Skip to content

fix(commands): use fully-qualified agent namespace in Task invocations#248

Merged
tmchow merged 2 commits intoEveryInc:mainfrom
mvanhorn:osc/199-fix-agent-namespace
Mar 14, 2026
Merged

fix(commands): use fully-qualified agent namespace in Task invocations#248
tmchow merged 2 commits intoEveryInc:mainfrom
mvanhorn:osc/199-fix-agent-namespace

Conversation

@mvanhorn
Copy link
Contributor

Summary

  • Update all bare agent name references in ce:plan, ce:brainstorm, and ce:review to use fully-qualified compound-engineering:category:name namespace
  • Bare names like Task repo-research-analyst(...) fail with "Agent type not found" because the plugin registers agents under the compound-engineering:research: or compound-engineering:review: namespace

Fixes #199

Changes

commands/ce/plan.md (4 refs):

  • repo-research-analyst -> compound-engineering:research:repo-research-analyst
  • learnings-researcher -> compound-engineering:research:learnings-researcher
  • best-practices-researcher -> compound-engineering:research:best-practices-researcher
  • framework-docs-researcher -> compound-engineering:research:framework-docs-researcher

commands/ce/brainstorm.md (1 ref):

  • repo-research-analyst -> compound-engineering:research:repo-research-analyst

commands/ce/review.md (6 refs):

  • agent-native-reviewer -> compound-engineering:review:agent-native-reviewer
  • learnings-researcher -> compound-engineering:research:learnings-researcher
  • schema-drift-detector -> compound-engineering:review:schema-drift-detector
  • data-migration-expert -> compound-engineering:review:data-migration-expert
  • deployment-verification-agent -> compound-engineering:review:deployment-verification-agent
  • code-simplicity-reviewer -> compound-engineering:review:code-simplicity-reviewer

The general-purpose agent type references were left unchanged as that is a built-in Claude Code agent type.

Test plan

  • Run /ce:plan and verify agents are dispatched without "Agent type not found" errors
  • Run /ce:review and verify review agents launch correctly

This contribution was developed with AI assistance (Claude Code).

Copy link
Collaborator

@tmchow tmchow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mvanhorn can you rebase and retarget the fixes on the new skill files given the changes we merged in from your other PR?

Rebased onto main after EveryInc#251 restructured commands into skills/.
Applied the same namespace fix to the new skill file locations
(ce-brainstorm, ce-plan, ce-review SKILL.md files).

Short agent names like `repo-research-analyst` are replaced with
fully-qualified `compound-engineering:research:repo-research-analyst`
to ensure correct agent resolution across all plugin targets.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@mvanhorn mvanhorn force-pushed the osc/199-fix-agent-namespace branch from 87e2626 to 026602e Compare March 13, 2026 16:29
@mvanhorn
Copy link
Contributor Author

Rebased onto main and retargeted the fixes to the new skill file locations (skills/ce-brainstorm/SKILL.md, skills/ce-plan/SKILL.md, skills/ce-review/SKILL.md). All 11 Task references now use fully-qualified agent namespaces.

Force-pushed in 026602e.

@tmchow
Copy link
Collaborator

tmchow commented Mar 14, 2026

@mvanhorn one more thing, sorry. Let's add an AGENTS.md directive to follow this pattern from now on so we dont make the mistake down the road.

Prevents future agent resolution failures by requiring skills to use
compound-engineering:<category>:<agent-name> instead of short names.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@mvanhorn
Copy link
Contributor Author

Added an AGENTS.md directive requiring fully-qualified agent namespaces in skill files. Pushed in 52e3e14.

@tmchow tmchow self-requested a review March 14, 2026 05:20
@tmchow tmchow merged commit 0c0c69b into EveryInc:main Mar 14, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bug: Agent namespace mismatch - workflows:plan fails to invoke compound-engineering agents

2 participants