fix: address PR #184 review feedback

[CaitlynByrne]

Summary

• Adds stricter validation for feature_request_human_input field schema in mcp_server/feature_mcp.py, addressing review feedback from feat: add blocked for human input feature #184

Changes

• Select options validation: Each option in a select field must be a dict with non-empty value and label strings
• Spurious options rejection: Fields with non-select types that include options are now rejected with a clear error

Already addressed in master

The other two items from the review were already fixed before merge:

• Graph view handleGraphNodeClick already includes needs_human_input (App.tsx:130-135)
• Progress fallback total already includes needs_human_input (App.tsx:249)

Test plan

• Call feature_request_human_input with a select field missing value/label on options — should return error
• Call with well-formed select options — should succeed
• Call with options on a non-select field type — should return error
• Run ruff check . — passes

🤖 Generated with Claude Code

[leonvanzyl]

Thanks for the clean follow-up, @CaitlynByrne! The stricter select option validation and spurious options rejection look great. Merged.