feat: add blocked for human input feature#184
Merged
leonvanzyl merged 1 commit intoAutoForgeAI:masterfrom Feb 12, 2026
Merged
Conversation
Agents can now request structured human input when they encounter genuine blockers (API keys, design choices, external configs). The request is displayed in the UI with a dynamic form, and the human's response is stored and made available when the agent resumes. Changes span 21 files + 1 new component: - Database: 3 new columns (needs_human_input, human_input_request, human_input_response) with migration - MCP: new feature_request_human_input tool + guards on existing tools - API: new resolve-human-input endpoint, 4th feature bucket - Orchestrator: skip needs_human_input features in scheduling - Progress: 4-tuple return from count_passing_tests - WebSocket: needs_human_input count in progress messages - UI: conditional 4th Kanban column, HumanInputForm component, amber status indicators, dependency graph support Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Collaborator
|
Thanks for the thoughtful full-stack implementation here. This is a strong direction overall and the feature is very useful. I found a few issues that should be addressed before merge:
If you can patch these, this should be in good shape to merge. Appreciate the work on this. |
leonvanzyl
added a commit
that referenced
this pull request
Feb 12, 2026
A) Graph view: add needs_human_input bucket to handleGraphNodeClick so clicking blocked nodes opens the feature modal B) MCP validation: validate field type enum, require options for select, enforce unique non-empty field IDs and labels C) Progress fallback: include needs_human_input in non-WebSocket total D) WebSocket: track needs_human_input count in progress state E) Cleanup guard: remove unnecessary needs_human_input check in _cleanup_stale_features (resolved via merge conflict) F) Defensive SQL: require in_progress=1 in feature_request_human_input Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
leonvanzyl
added a commit
that referenced
this pull request
Feb 12, 2026
…nput feat: add blocked for human input feature
CaitlynByrne
added a commit
to CaitlynByrne/autoforge
that referenced
this pull request
Feb 14, 2026
4 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Key changes
needs_human_input,human_input_request,human_input_response) with migrationfeature_request_human_inputtool + guards on existing toolsresolve-human-inputendpoint, 4th feature bucketneeds_human_inputfeatures in schedulingcount_passing_testsneeds_human_inputcount in progress messagesHumanInputFormcomponent, amber status indicators, dependency graph supportTest plan
ruff check .andcd ui && npm run buildto verify lint/type checks pass🤖 Generated with Claude Code