Skip to content

Conversation

@ammar-agent
Copy link
Collaborator

What changed

  • Use a short resume sentinel ("Continue") for forced compaction.
  • Don't inject that sentinel into the compaction prompt (summary request), while still keeping it in compaction metadata so the backend can auto-send it after compaction.
  • Add unit coverage for the default vs. explicit follow-up behavior.

Why

Appending a verbose auto-injected instruction can compete with the user's actual message; treating "Continue" as a control sentinel and keeping it out of the summarization prompt reduces instruction confusion.


Generated with mux • Model: unknown • Thinking: unknown

@chatgpt-codex-connector
Copy link

Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits.
Repo admins can enable using credits for code reviews in their settings.

@ammario ammario merged commit 7053c2b into main Dec 14, 2025
20 checks passed
@ammario ammario deleted the messaging-59cn branch December 14, 2025 03:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants