Repository navigation
Assistant: help / new feature / bug report modes, with bug reports on existing features - #278
Open
zwanski-start wants to merge 1 commit into
Open
zwanski-start wants to merge 1 commit into
zwanski-start wants to merge 1 commit into
Conversation
Add a feature_report_bug MCP tool that appends a bug report (and optional verification steps) to an existing feature, marks it not passing and moves it to the front of the queue. Expose it to the project assistant, teach the system prompt the help / new feature / bug report modes, and add quick-start mode chips to the assistant chat. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Author
🔍 QA report: ✅ PassedTester: Dennis (cubefarm agent, in a fresh session) · Round: 1 · Author: Dennis The new feature_report_bug MCP tool works as described: an existing feature goes back to pending at the front of the queue with the bug report and steps appended, in-progress and unknown features are refused, and bad input is rejected. The assistant can use the tool and the coding-agent tools stay blocked. The Help / New feature / Report a bug buttons and the tool-call label work in a real browser, including at phone width. Lint, build and the new tests pass; the pytest failures and mypy errors are the same as on master.
🧪 Commands run
No browser screenshots were taken in this round. Posted by cubefarm · merges automatically once GitHub's checks pass |
cabana8471-arch
pushed a commit
to cabana8471-arch/autoforge
that referenced
this pull request
Nov 8, 2026
Adaptations: AssistantChat merged with my-features' messagesContainerRef scrolling; feature_report_bug also clears needs_manual_testing (AutoForgeAI#245) so a feature in the Manual Testing lane returns to the agents (test RED->GREEN), and writes with an in_progress = 0 guard so it cannot reset a feature an agent claimed after the check. AutoForgeAI#278's move-to-front not taken (optional in the plan). Integrates AutoForgeAI#331 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01AeH26pcsqoxPb3CDs8g1VN
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.
Closes #135
What changed
feature_report_bug(feature_id, bug_description, verification_steps?)(mcp_server/feature_mcp.py): instead of creating a near-duplicate feature, it appendsBUG REPORT: …to the existing feature's description, appends any verification steps, setspasses=false/in_progress=false, and moves the feature to the front of the queue (MIN(priority) - 1). Runs in anatomic_transaction. It refuses features that an agent is currently working on (in_progress), so the report can't be lost when that agent marks the feature passing.server/services/assistant_chat_session.py): the tool is now in the assistant's allowed tools. The system prompt describes three modes: Help (answer only), New feature (feature_create/_bulk), and Bug report (find the existing feature and callfeature_report_bug; create a new feature only if no existing feature covers the behavior). The greeting mentions the three options.AssistantChat.tsx,useAssistantChat.ts): three quick-start chips (Get help / New feature / Report a bug) above the input. They show until the user sends a first message, and clicking one pre-fills the input with a mode prefix. The input placeholder is updated, and the tool call is shown as "Reporting bug on feature #N (back to pending)".Verification
test_feature_report_bug.py(5 tests, passing): passing feature → pending at front of queue with report and steps appended; steps unchanged when none are given; in-progress feature refused and left unchanged; unknown ID; tool is allowed for the assistant and not in its disallowed list.ruff check .clean,python test_security.pypasses, and mypy shows no new errors. Its only output is pre-existing unused-type: ignorenotes on lines this PR doesn't change.npm run lint(0 errors) andnpm run buildpass.CLAUDE.mdinto a registered project. The UI change is small and covered by the type check and build.Assumptions
passes=false, in_progress=false. I also move the feature to the front of the queue so the bug gets fixed next, which seemed the most useful default.feature_get_by_id.🤖 Generated with Claude Code