Skip to content

fix: namespace reviewer agent as deepwork:reviewer#375

Merged
nhorton merged 3 commits intomainfrom
fix/reviewer-agent-namespace
Apr 14, 2026
Merged

fix: namespace reviewer agent as deepwork:reviewer#375
nhorton merged 3 commits intomainfrom
fix/reviewer-agent-namespace

Conversation

@nhorton
Copy link
Copy Markdown
Contributor

@nhorton nhorton commented Apr 14, 2026

Summary

  • Changed the default reviewer agent name from reviewer to deepwork:reviewer in the MCP get_review_instructions output so it resolves correctly as a plugin-namespaced agent.
  • Updated the /review skill to check for deepwork:reviewer availability before proceeding, directing users to run /reload-plugins if the agent is missing.
  • Renamed the agent definition in plugins/claude/agents/reviewer.md to match.

Test plan

  • Unit tests updated and passing (88 tests, all green)
  • Verify /review skill correctly dispatches agents with subagent_type: deepwork:reviewer
  • Verify /review skill stops with reload message when agent is unavailable

🤖 Generated with Claude Code

nhorton and others added 3 commits April 14, 2026 10:22
Use the fully-qualified plugin agent name "deepwork:reviewer" instead of
bare "reviewer" so the agent is correctly resolved from the plugin. The
/review skill now checks for agent availability and tells users to run
/reload-plugins if the agent is missing.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- Update REVIEW-REQ-006.3.3c to reference "deepwork:reviewer" default
- Add REVIEW-REQ-007.1.10 for agent availability pre-flight check
- Update README_REVIEWS.md example output and contributor setup section
- Add plugins/**/agents/**/*.md to prompt_best_practices review rule
- Add changelog entries for the reviewer rename

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…ode_review output format

- Condensed requirements_traceability inline instructions (~94 → ~45 lines)
  while preserving all review criteria
- Added explicit Output Format section to shell_code_review rule

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@nhorton nhorton added this pull request to the merge queue Apr 14, 2026
Merged via the queue into main with commit 8033f20 Apr 14, 2026
5 checks passed
@nhorton nhorton deleted the fix/reviewer-agent-namespace branch April 14, 2026 16:45
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.

1 participant