feat(review): default to minimal 'reviewer' subagent on Sonnet - #364
Merged
Conversation
Ship a new 'reviewer' subagent in the DeepWork Claude plugin pinned to Sonnet with minimal instructions, and default review dispatch to it. Opus→Sonnet across parallel review fan-out is the dominant cost lever. Per-rule agent overrides still take precedence. The reviewer's tools list includes dual-wildcard MCP prefixes (mcp__plugin_deepwork_deepwork__* and mcp__deepwork-dev__*) so the same agent file resolves mark_review_as_passed in both production (plugin install only) and development (dev MCP server, with or without plugin) contexts. Spec REVIEW-REQ-006.3.c and test_formatter.py are updated in lockstep; stale REVIEW-REQ-006.3.3b/c cross-references corrected. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- Fix test_formatter.py traceability comment IDs: restore the .3.3 hierarchy level (REVIEW-REQ-006.3.3c, .3.3b) instead of the incorrectly shortened .3.c, .3.b format. - Update README_REVIEWS.md example output to match current formatter format (name/description/subagent_type/prompt fields, reviewer default). - Add agents/ directory to CLAUDE.md and doc/architecture.md project structure trees. - Add PLUG-REQ-001.14 (Default Reviewer Subagent) formalizing the reviewer agent's existence, model, tools, and behavioral constraints. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
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
1 participant
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.
Summary
reviewersubagent in the DeepWork Claude plugin (plugins/claude/agents/reviewer.md) pinned tomodel: sonnetwith minimal instructions, reducing per-review cost by switching from Opus to Sonnet across parallel review fan-out.subagent_typein review dispatch from"general-purpose"to"reviewer"(single-line change atformatter.py:109), with per-rule.deepreviewagent overrides still taking precedence.mcp__plugin_deepwork_deepwork__*andmcp__deepwork-dev__*) in the agent'stools:frontmatter.Test plan
uv run pytest tests/unit/review/test_formatter.py— 23 passed (default assertion updated)uv run pytest tests/unit/review/— 267 passed, no regressionsuv run deepwork review --instructions-for claude— confirmed unscoped rules emitsubagent_type: reviewer;requirements_traceabilitystill routes torequirements-revieweroverride/reviewdispatch: verify thereviewersubagent resolves, completes a review, and callsmark_review_as_passedsuccessfullytools:syntax works in Claude Code agent frontmatter at dispatch time; if not, swap to explicit six-tool enumeration🤖 Generated with Claude Code