fix: update deprecated Plan+ agent references to Build+#229
fix: update deprecated Plan+ agent references to Build+#229marcusquinn merged 4 commits intomainfrom
Conversation
Plan+ was consolidated into Build+ (PR #226) but several files still referenced Plan+ as a primary agent. This caused the list-todo command to fail by routing to a non-existent primary agent. Changes: - list-todo, save-todo, show-plan: agent frontmatter Plan+ → Build+ - conversation-starter: Plan+ section → Build+ deliberation mode - add-new-mcp-to-aidevops: remove Plan+ from primary agent lists - troubleshooting, opencode: update test examples to Build+ - context-guardrails: reference @plan-plus subagent instead - README: update agent structure description Historical references in TODO.md, PLANS.md, CHANGELOG.md preserved. plan-plus.md (the subagent itself) left unchanged - already correct.
Summary of ChangesHello @marcusquinn, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request refines the agent architecture by updating references to the Highlights
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
|
Note Other AI code review bot(s) detectedCodeRabbit has detected other AI code review bot(s) in this pull request and will avoid duplicating their findings in the review comments. This may lead to a less comprehensive review. WalkthroughPlan+ is demoted from a main agent to a planning-only subagent named Changes
Sequence Diagram(s)(omitted — changes are documentation/configuration and do not introduce multi-component control-flow.) Estimated code review effort🎯 2 (Simple) | ⏱️ ~9 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request effectively addresses the deprecation of the Plan+ agent by replacing its references with Build+ or @plan-plus as appropriate. The changes are consistent across all modified files, fixing broken commands and updating documentation to reflect the new agent architecture. I have one minor suggestion to improve the clarity of the agent structure description in the README.md file.
README.md
Outdated
| ### Agent Structure | ||
|
|
||
| - 15 primary agents (Plan+, Build+, SEO, Marketing, etc.) | ||
| - 15 primary agents (Build+, SEO, Marketing, etc.) with @plan-plus subagent for planning-only mode |
There was a problem hiding this comment.
This line states there are '15 primary agents', but with the removal of Plan+ (a primary agent) and the clarification that @plan-plus is a subagent, this count is no longer accurate for primary agents. The table on line 1239 lists 15 agents in total, including @plan-plus. To improve clarity and avoid confusion for users, I suggest removing the specific count from this summary line.
| - 15 primary agents (Build+, SEO, Marketing, etc.) with @plan-plus subagent for planning-only mode | |
| - Primary agents (Build+, SEO, Marketing, etc.) with @plan-plus subagent for planning-only mode |
🔍 Code Quality Report�[0;35m[MONITOR]�[0m Code Review Monitoring Report �[0;34m[INFO]�[0m Latest Quality Status: �[0;34m[INFO]�[0m Recent monitoring activity: 📈 Current Quality Metrics
Generated on: Sun Jan 25 22:22:36 UTC 2026 Generated by AI DevOps Framework Code Review Monitoring |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
🤖 Augment PR SummarySummary: Replaces deprecated Changes:
🤖 Was this summary useful? React with 👍 or 👎 |
![augmentcode[bot]](https://avatars.githubusercontent.com/in/1027498?s=60&v=4){kind=small}
|
|
||
| # Test agent permissions | ||
| ~/.aidevops/agents/scripts/opencode-test-helper.sh test-agent Plan+ | ||
| ~/.aidevops/agents/scripts/opencode-test-helper.sh test-agent Build+ |
There was a problem hiding this comment.
This doc now shows opencode-test-helper.sh test-agent Build+, but the helper script’s own show_help() examples still reference test-agent Plan+, which can lead to conflicting guidance. Consider aligning the examples so users aren’t unsure which agent name is expected.
Other Locations
.agent/aidevops/troubleshooting.md:251
🤖 Was this useful? React with 👍 or 👎
| | Name | File | Purpose | MCPs Enabled | | ||
| |------|------|---------|--------------| | ||
| | Plan+ | `plan-plus.md` | Planning with semantic search, writes to TODO.md/todo/ | context7, augment, repomix | | ||
| | @plan-plus | `plan-plus.md` | Planning-only subagent (Build+ handles planning by default) | context7, augment, repomix | |
There was a problem hiding this comment.
In the “Main Agents” table, the @plan-plus row is described as a “subagent”, which conflicts with the section label (and the earlier “primary agents … with @plan-plus subagent” line). Consider clarifying whether @plan-plus is meant to be tab-selectable as a main agent or only @mentionable.
🤖 Was this useful? React with 👍 or 👎
| > | ||
| > **Main Agents Available**: | ||
| > Plan+, Build+, Accounts, AI-DevOps, Content, Health, Legal, Marketing, | ||
| > Build+, Accounts, AI-DevOps, Content, Health, Legal, Marketing, |
There was a problem hiding this comment.
This prompt asks about “main agents and subagents”, but the “Main Agents Available” list no longer includes the planning-only option that’s referenced later as @plan-plus, which makes the example a bit inconsistent for enablement decisions. Consider either listing @plan-plus under subagents here or adjusting the wording to match the list.
🤖 Was this useful? React with 👍 or 👎
🔍 Code Quality Report�[0;35m[MONITOR]�[0m Code Review Monitoring Report �[0;34m[INFO]�[0m Latest Quality Status: �[0;34m[INFO]�[0m Recent monitoring activity: 📈 Current Quality Metrics
Generated on: Sun Jan 25 22:25:51 UTC 2026 Generated by AI DevOps Framework Code Review Monitoring |
…rID TypeError
OpenCode iterates all agent entries and tries to resolve model.providerID
even for disabled agents. Since Plan+ and AI-DevOps have no model config,
this causes: TypeError: undefined is not an object (evaluating 'input.model.providerID')
Fix: Stop emitting {"disable": true} entries for Plan+ and AI-DevOps in
the generator script. They exist only as subagents (@plan-plus, @AIDevops)
and don't need primary agent entries at all. OpenCode's built-in build/plan
agents still need disable entries since they'd otherwise appear by default.
🔍 Code Quality Report�[0;35m[MONITOR]�[0m Code Review Monitoring Report �[0;34m[INFO]�[0m Latest Quality Status: �[0;34m[INFO]�[0m Recent monitoring activity: 📈 Current Quality Metrics
Generated on: Sun Jan 25 22:35:05 UTC 2026 Generated by AI DevOps Framework Code Review Monitoring |
The providerID TypeError is not caused by disabled agent entries. Reverting to original behavior while investigating the actual cause.
|
🔍 Code Quality Report�[0;35m[MONITOR]�[0m Code Review Monitoring Report �[0;34m[INFO]�[0m Latest Quality Status: �[0;34m[INFO]�[0m Recent monitoring activity: 📈 Current Quality Metrics
Generated on: Sun Jan 25 22:40:28 UTC 2026 Generated by AI DevOps Framework Code Review Monitoring |
1 participant
Summary
Plan+primary agent in their frontmatterPlan+references across 9 files to useBuild+(the unified agent) or@plan-plus(the subagent) as appropriateContext
PR #226 consolidated Plan+ and AI-DevOps into Build+ as the unified coding agent. However, several files still referenced
Plan+as a primary agent, causing the/list-todocommand (and other planning commands) to error out when trying to route to a non-existent primary agent.Changes
list-todo.mdagent: Plan+→agent: Build+save-todo.mdagent: Plan+→agent: Build+show-plan.mdagent: Plan+→agent: Build+conversation-starter.mdadd-new-mcp-to-aidevops.mdtroubleshooting.mdcontext-guardrails.mdopencode.mdREADME.mdNot Changed (Intentionally)
plan-plus.md- Already correctly describes itself as a subagentarchitecture.md- Historical context ("consolidates the former Plan+") is accurateonboarding.md- Already uses correct@plan-plussubagent referenceTODO.md,PLANS.md,CHANGELOG.md- Historical records preservedSummary by CodeRabbit
Documentation
Configuration
Troubleshooting & Commands
✏️ Tip: You can customize this high-level summary in your review settings.