Skip to content

Commit f1721cf

Browse files
wkoutreclaude
andcommitted
fix(development-planning): remove round floor and repo-only agent check
Two residues in plan-implementation that undercut the Opus 5 migration: - Quick Process step 3 mandated "2-3 rounds" while the ceiling table below caps Simple at 1 and Medium at 2. The numbered step reads as an instruction, so the floor survived the removal. It now defers to the table. - The agent-existence check grepped `packages/plugins/*/agents/`, a path that exists only in this repo. In a consumer repo it matches nothing, and the adjacent "fails at dispatch" warning then reads as "do not dispatch" - a self-disabling guard. Replaced with the session's own agent inventory, plus the naming rule stated as fact (frontmatter `name:`, `-agent` suffix, cross-plugin `plugin-name:agent-name`). Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
1 parent 50e1250 commit f1721cf

2 files changed

Lines changed: 10 additions & 7 deletions

File tree

packages/plugins/development-planning/skills/plan-implementation/SKILL.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ Create clear, actionable implementation plans through collaborative multi-agent
2222

2323
1. **Analyze Context**: Understand the task, leverage any prior exploration
2424
2. **Select Agents**: Choose specialists for the dimensions the task actually raises, up to 10
25-
3. **Collaborative Discussion**: 2-3 rounds of multi-agent refinement
25+
3. **Collaborative Discussion**: Multi-agent refinement, up to the round ceiling in the table below
2626
4. **Synthesize Plan**: Generate comprehensive implementation plan
2727
5. **Output File**: Write plan to `.claude-output/plan-[timestamp].md`
2828

packages/plugins/development-planning/skills/plan-implementation/planning-guide.md

Lines changed: 9 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -20,12 +20,15 @@ Select agents based on technical domains in the task. The names below are the di
2020
| External research | `researcher-agent` |
2121
| Debugging | `debug-assistant-agent` |
2222

23-
Before dispatching an agent that is not in this table, confirm it exists - a name that does not
24-
match an agent's frontmatter `name:` fails at dispatch:
25-
26-
```bash
27-
grep -rhE '^name: ' packages/plugins/*/agents/ | sort -u
28-
```
23+
Before dispatching an agent that is not in this table, confirm it appears in the agent inventory
24+
available in the current session - the agent types listed in your own context are the authoritative
25+
set, and a name that is not among them fails at dispatch. Do not try to verify a name by searching
26+
the filesystem: these agents ship from installed plugins rather than from the user's project, so a
27+
repo-relative search returns nothing even when the agent is available.
28+
29+
Dispatch names come from an agent's frontmatter `name:` field; marketplace agents carry an `-agent`
30+
suffix. Agents from a different plugin are addressed as `plugin-name:agent-name`, for example
31+
`development-codebase-tools:security-analyzer-agent`.
2932

3033
### By Complexity
3134

0 commit comments

Comments
 (0)