Skip to content

Commit fe7cf86

Browse files
committed
Fix: tell code reviewer its a subagent. Drop "opus" from code reviewer id
1 parent 42f2252 commit fe7cf86

File tree

3 files changed

+6
-17
lines changed

3 files changed

+6
-17
lines changed

.agents/base2/base2.ts

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -70,7 +70,7 @@ export function createBase2(
7070
isDefault && 'editor',
7171
isMax && 'editor-multi-prompt',
7272
isMax && 'thinker-best-of-n-opus',
73-
!isLite && 'code-reviewer-opus',
73+
!isLite && 'code-reviewer',
7474
'context-pruner',
7575
),
7676

@@ -131,7 +131,7 @@ Use the spawn_agents tool to spawn specialized agents to help you complete the u
131131
'- Spawn commanders sequentially if the second command depends on the the first.',
132132
!isFast &&
133133
!isLite &&
134-
'- Spawn a code-reviewer-opus to review the changes after you have implemented the changes.',
134+
'- Spawn a code-reviewer to review the changes after you have implemented the changes.',
135135
).join('\n ')}
136136
- **No need to include context:** When prompting an agent, realize that many agents can already see the entire conversation history, so you can be brief in prompting them without needing to include context.
137137
@@ -305,7 +305,7 @@ ${buildArray(
305305
isFast &&
306306
'- Do a single typecheck targeted for your changes at most (if applicable for the project). Or skip this step if the change was small.',
307307
(isDefault || isMax) &&
308-
'- Spawn a code-reviewer-opus to review the changes after you have implemented the changes. (Skip this step only if the change is extremely straightforward and obvious.)',
308+
'- Spawn a code-reviewer to review the changes after you have implemented the changes. (Skip this step only if the change is extremely straightforward and obvious.)',
309309
!hasNoValidation &&
310310
`- Test your changes by running appropriate validation commands for the project (e.g. typechecks, tests, lints, etc.). Try to run all appropriate commands in parallel. ${isMax ? ' Typecheck and test the specific area of the project that you are editing *AND* then typecheck and test the entire project if necessary.' : ' If you can, only test the area of the project that you are editing, rather than the entire project.'} You may have to explore the project to find the appropriate commands. Don't skip this step!`,
311311
`- Inform the user that you have completed the task in one sentence or a few short bullet points.${isSonnet ? " Don't create any markdown summary files or example documentation files, unless asked by the user." : ''}`,
@@ -333,7 +333,7 @@ function buildImplementationStepPrompt({
333333
isMax &&
334334
`You must spawn the 'editor-multi-prompt' agent to implement code changes, since it will generate the best code changes.`,
335335
(isDefault || isMax) &&
336-
'Spawn code-reviewer-opus to review the changes after you have implemented the changes and in parallel with typechecking or testing.',
336+
'Spawn code-reviewer to review the changes after you have implemented the changes and in parallel with typechecking or testing.',
337337
`After completing the user request, summarize your changes in a sentence${isFast ? '' : ' or a few short bullet points'}.${isSonnet ? " Don't create any summary markdown files or example documentation files, unless asked by the user." : ''} Don't repeat yourself, especially if you have already concluded and summarized the changes in a previous step -- just end your turn.`,
338338
!isFast &&
339339
`After a successful implementation, use the suggest_followups tool to suggest around 3 next steps the user might want to take.`,

.agents/reviewer/code-reviewer-opus.ts

Lines changed: 0 additions & 11 deletions
This file was deleted.

.agents/reviewer/code-reviewer.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ export const createReviewer = (
2525
inheritParentSystemPrompt: true,
2626
includeMessageHistory: true,
2727

28-
instructionsPrompt: `For reference, here is the original user request:
28+
instructionsPrompt: `You are a subagent that reviews code changes. Do not use any tools. For reference, here is the original user request:
2929
<user_message>
3030
${PLACEHOLDER.USER_INPUT_PROMPT}
3131
</user_message>
@@ -57,7 +57,7 @@ Be extremely concise.`,
5757
const definition: SecretAgentDefinition = {
5858
id: 'code-reviewer',
5959
publisher,
60-
...createReviewer('anthropic/claude-sonnet-4.5'),
60+
...createReviewer('anthropic/claude-opus-4.5'),
6161
}
6262

6363
export default definition

0 commit comments

Comments
 (0)