P1-T8: Update /config examples for broker setup first#132
Merged
Conversation
There was a problem hiding this comment.
Pull request overview
Updates the repository’s MCP client configuration templates to present broker-mode setup first (P1-T8), and archives the task’s validation/review artifacts under SPECS/ARCHIVE/.
Changes:
- Reordered/expanded
config/templates for Cursor and Zed to show broker-mode options before non-broker alternatives. - Updated Claude Code and Codex CLI setup snippets to make broker-mode the recommended “Option 1”.
- Added P1-T8 planning/archive entries and archived validation + review reports.
Reviewed changes
Copilot reviewed 10 out of 10 changed files in this pull request and generated 7 comments.
Show a summary per file
| File | Description |
|---|---|
| config/zed-agent.json | Adds broker-first option blocks and renumbers other options accordingly. |
| config/cursor-mcp.json | Adds broker-first option blocks and renumbers other options accordingly. |
| config/codex-cli.txt | Reorders CLI setup instructions to put broker-mode first. |
| config/claude-code.txt | Reorders CLI setup instructions to put broker-mode first. |
| SPECS/Workplan.md | Marks P1-T8 as completed and records outputs/acceptance criteria. |
| SPECS/INPROGRESS/next.md | Resets status to idle and records P1-T8 as recently archived. |
| SPECS/ARCHIVE/_Historical/REVIEW_p1_t8_config_broker_setup_first.md | Adds the historical review report for P1-T8. |
| SPECS/ARCHIVE/P1-T8_Update_config_examples_for_borker_setup_first/P1-T8_Validation_Report.md | Adds the validation report artifact for P1-T8. |
| SPECS/ARCHIVE/P1-T8_Update_config_examples_for_borker_setup_first/P1-T8_Update_config_examples_for_borker_setup_first.md | Adds the archived PRD/spec artifact for P1-T8. |
| SPECS/ARCHIVE/INDEX.md | Updates archive index/log to include P1-T8 and the review report reference. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
SoundBlaster
changed the title
Owner
Author
|
Addressed all Copilot review comments in commit f34502f:\n- corrected |
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.
Description
Implement
P1-T8end-to-end via FLOW and updateconfig/templates to present broker setup first.Key changes:
P1-T8inSPECS/Workplan.md/SPECS/INPROGRESS/next.md.config/cursor-mcp.jsonconfig/zed-agent.jsonconfig/claude-code.txtconfig/codex-cli.txtSPECS/ARCHIVE/P1-T8_Update_config_examples_for_borker_setup_first/SPECS/INPROGRESS/next.mdto idle.Validation results:
pytest→735 passed, 5 skippedruff check src/→ passmypy src/→ passpytest --cov→91.26%Type of Change
Quality Gates
Before submitting, ensure all quality gates pass:
Or run individually:
make test- All tests pass with ≥90% coveragemake lint- No linting errorsmake format- Code is properly formattedmake typecheck- Type checking passesmake doccheck- Documentation is synced with DocC (if docs changed)Documentation Sync
If you modified files in
docs/, ensure corresponding DocC files are also updated:docs/installation.mdmcpbridge-wrapper.docc/Installation.mddocs/cursor-setup.mdmcpbridge-wrapper.docc/CursorSetup.mddocs/claude-setup.mdmcpbridge-wrapper.docc/ClaudeCodeSetup.mddocs/codex-setup.mdmcpbridge-wrapper.docc/CodexCLISetup.mddocs/troubleshooting.mdmcpbridge-wrapper.docc/Troubleshooting.mddocs/architecture.mdmcpbridge-wrapper.docc/Architecture.mddocs/environment-variables.mdmcpbridge-wrapper.docc/EnvironmentVariables.mdREADME.mdmcpbridge-wrapper.docc/mcpbridge-wrapper.mdTesting
Checklist