-
Notifications
You must be signed in to change notification settings - Fork 7.1k
Refocus compaction summaries on the active task #8605
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
|
All contributors have signed the CLA ✍️ ✅ |
|
I have read the CLA Document and I hereby sign the CLA |
|
Is there a bug report filed for this issue? In our contributor's guidance we ask that all bug fix PRs start with a bug report. This helps us prioritize and track issues, especially in cases where we decide to reject a PR. |
|
We do significant testing with our system prompts. Changes to these prompts can easily introduce regressions, so we have a high bar for accepting PRs that modify prompts. |
Understood, would that mean delete this PR and start a new one quoting the big report, or I can just simply add it here? |
|
@ReyNeill, no need to create a new PR. Just add a link to a bug report. |
|
I happen to run with the issue right now and submitted the bug: https://github.com/openai/codex/issues/new?template=2-bug-report.yml&steps=Uploaded%20thread:%20019b704d-04f3-7031-b1bb-fb5af42be238 thread ID 019b704d-04f3-7031-b1bb-fb5af42be238 The model wastes a lot of tokens and time verifying all of the session's tasks instead of continuing the task it was doing when the session history auto-compacted |
|
How much testing have you done with your modified prompt? I'm trying to get a sense for whether the solution is speculative or whether you have strong evidence that it improves the behavior, at least for your usage. |
|
Not enough testing, but I'll do it and comeback. Thanks for the fast feedback speed. |
Problem
When auto-compaction triggers mid-implementation, the summary plus preserved user messages can cause the model to re-check the entire session instead of continuing the current task.
Solution
Refocus the compaction prompt and summary prefix so the summary targets only the active task and explicitly avoids re-opening completed work unless the user asks.
Notes / Future options
Changes
codex-rs/core/templates/compact/prompt.mdcodex-rs/core/templates/compact/summary_prefix.mdTests
Not run (content-only changes).