elias/oh-my-opencode
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

update: always wait for Oracle results instead of blanket background_cancel(all=true)

Browse Source
This commit is contained in:
YeonGyu-Kim 2026-02-17 03:53:02 +09:00
parent 6208c07809
commit 219c1f8225
2 changed files with 8 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/agents/dynamic-agent-prompt-builder.ts
View File

@ -336,6 +336,10 @@ ${avoidWhen.map((w) => `- ${w}`).join("\n")}
Briefly announce "Consulting Oracle for [reason]" before invocation. Briefly announce "Consulting Oracle for [reason]" before invocation.
**Exception**: This is the ONLY case where you announce before acting. For all other work, start immediately without status updates. **Exception**: This is the ONLY case where you announce before acting. For all other work, start immediately without status updates.
### Oracle Background Task Policy:
- Oracle takes 20+ min by design. Always wait for Oracle results via \`background_output\` before final answer.
- Oracle provides independent analysis from a different angle that catches blind spots — even when you believe you already have sufficient context, Oracle's perspective is worth the wait.
</Oracle_Usage>` </Oracle_Usage>`
} }

7
src/agents/sisyphus.ts
View File

@ -310,7 +310,7 @@ result = task(..., run_in_background=false) // Never wait synchronously for exp
1. Launch parallel agents → receive task_ids 1. Launch parallel agents → receive task_ids
2. Continue immediate work 2. Continue immediate work
3. When results needed: \`background_output(task_id="...")\` 3. When results needed: \`background_output(task_id="...")\`
4. BEFORE final answer: \`background_cancel(all=true)\` 4. Before final answer: cancel disposable tasks (explore, librarian) individually via \`background_cancel(taskId="...")\`. Always wait for Oracle — collect its result via \`background_output\` before answering.
### Search Stop Conditions ### Search Stop Conditions
@ -449,8 +449,9 @@ If verification fails:
3. Report: "Done. Note: found N pre-existing lint errors unrelated to my changes." 3. Report: "Done. Note: found N pre-existing lint errors unrelated to my changes."
### Before Delivering Final Answer: ### Before Delivering Final Answer:
- Cancel ALL running background tasks: \`background_cancel(all=true)\` - Cancel disposable background tasks (explore, librarian) individually via \`background_cancel(taskId="...")\`
- This conserves resources and ensures clean workflow completion - **Always wait for Oracle**: Oracle takes 20+ min by design and always provides valuable independent analysis from a different angle — even when you already have enough context. Collect Oracle results via \`background_output\` before answering.
- When Oracle is running, cancel disposable tasks individually instead of using \`background_cancel(all=true)\`.
</Behavior_Instructions> </Behavior_Instructions>
${oracleSection} ${oracleSection}