Skip to content

Commit 52f80c5

Browse files
committed
Don't write out <tool_result> tags
1 parent aa57abc commit 52f80c5

File tree

1 file changed

+21
-20
lines changed

1 file changed

+21
-20
lines changed

.agents/base2/base2.ts

Lines changed: 21 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -143,6 +143,7 @@ For other questions, you can direct them to codebuff.com, or especially codebuff
143143
# Response guidelines
144144
145145
${buildArray(
146+
"- NEVER write out <tool_result> tags in your response. Tools will automatically append the results of their tool calls for you -- it's not something you need to do.",
146147
isSonnet &&
147148
`- **Don't create a summary markdown file:** The user doesn't want markdown files they didn't ask for. Don't create them.`,
148149
'- **Keep final summary extremely concise:** Write only a few words for each change you made in the final summary.',
@@ -235,6 +236,26 @@ ${buildArray(
235236
).join('\n')}`
236237
}
237238

239+
function buildImplementationStepPrompt({
240+
isMax,
241+
isGpt5,
242+
hasNoValidation,
243+
isSonnet,
244+
}: {
245+
isMax: boolean
246+
isGpt5: boolean
247+
hasNoValidation: boolean
248+
isSonnet: boolean
249+
}) {
250+
return buildArray(
251+
isMax &&
252+
`Keep working until the user's request is completely satisfied${!hasNoValidation ? ' and validated' : ''}, or until you require more information from the user.`,
253+
`After completing the user request, summarize your changes in a sentence 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.`,
254+
isGpt5 &&
255+
`IMPORTANT: if you are completely done with the user's request or require more information from the user, you must call the task_completed tool to end your turn.`,
256+
).join('\n')
257+
}
258+
238259
function buildPlanOnlyInstructionsPrompt({}: {}) {
239260
return `Orchestrate the completion of the user's request using your specialized sub-agents.
240261
@@ -288,26 +309,6 @@ On subsequent turns with the user, you should rewrite the spec to reflect the us
288309
).join('\n')}`
289310
}
290311

291-
function buildImplementationStepPrompt({
292-
isMax,
293-
isGpt5,
294-
hasNoValidation,
295-
isSonnet,
296-
}: {
297-
isMax: boolean
298-
isGpt5: boolean
299-
hasNoValidation: boolean
300-
isSonnet: boolean
301-
}) {
302-
return buildArray(
303-
isMax &&
304-
`Keep working until the user's request is completely satisfied${!hasNoValidation ? ' and validated' : ''}, or until you require more information from the user.`,
305-
`After completing the user request, summarize your changes in a sentence 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.`,
306-
isGpt5 &&
307-
`IMPORTANT: if you are completely done with the user's request or require more information from the user, you must call the task_completed tool to end your turn.`,
308-
).join('\n')
309-
}
310-
311312
function buildPlanOnlyStepPrompt({}: {}) {
312313
return buildArray(
313314
`Your are in plan mode. Do not make any file changes. Do not call write_file or str_replace. Do not spawn the best-of-n-editor agent. Do not use the write_todos tool.`,

0 commit comments

Comments
 (0)