Repository navigation
fix: reject a template-echo summary; close an open turn on an idle notification - #40
Merged
Merged
Conversation
added 2 commits
October 5, 2026 16:45
The condense prompt shows the frame with two placeholders in it, "<free-form condensed summary>" and "<new durable fact, one per line>". A model that answered with that template alone produced a valid frame. The placeholder then replaced the condensed history and was pinned as a durable fact. The file got smaller, so nothing downstream objected, and the backup was the only way back. A frame whose summary is the summary placeholder, or whose hoist list includes the fact placeholder, is now rejected like any other unusable answer. A summary that merely mentions the placeholder is still accepted. The prompt reads the placeholders from the same constants, so its text is unchanged.
…nput
An open turn keeps a worker off the stall re-nudge until Stop arrives,
bounded by a twenty-minute ceiling for a Stop that never comes. An
interrupted turn sends no Stop, and some engines never send one. For
those, the turn stayed open for the whole ceiling, and a worker that was
in fact idle and stalled on its mail waited out the full twenty minutes.
The CLI's idle notification ("waiting for your input") says the same
thing Stop does, so it now closes the turn too. A permission prompt is a
turn waiting on the human, not a finished one, so it leaves the turn
open, and the HITL hold covers the prompt itself.
🚫 This PR is missing its before/after evidenceEvery pull request here has to show its work. Screenshots or a short screen recording, before the change and after it.
How to fix it: edit the description, keep the A bug fix with no visible surface still needs it: show the failing behaviour, then the same steps passing. A terminal recording is fine. Genuinely nothing to show — a CI tweak, a typo, a dependency bump? A maintainer can apply the |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Two small hardening fixes.
Both test-first with mutation checks; touched suites pass.