Fold small session, chrome, and compaction helpers into owning modules - #1036
Merged
TheGreatAxios merged 7 commits intoSep 14, 2026
Merged
TheGreatAxios merged 7 commits into
TheGreatAxios merged 7 commits into
Conversation
summary-excerpt.ts was a 100-line companion to summarizer.ts with a single caller. summarizer.ts now owns the budgeted archive excerpt directly.
deliver-agent-message.ts had one entry point and one caller family; queued-delivery.ts now owns the guard, the result type, and the operator copy.
session-chrome.ts duplicated the chrome zone model's vocabulary: TurnStatus, ACTIVITY_STATES, and the ticker's leak-prevention contract now live in chrome-state.ts alongside the zone content they describe. All importers follow; no behavior change.
archive-uri.ts owned the archive:// addressing scheme apart from the store that honors it. The ref helpers now live on compaction-archive.ts verbatim; plugins and session importers follow. No behavior change.
IMPLEMENTATION.md lists the excerpt and archive refs on their owning modules; TELEMETRY.md points at chrome-state.ts.
Revert 974e96d: sibling cl-7949-tui-micro-inlines renames queued-delivery.ts, so this branch keeps deliver-agent-message.ts separate to merge cleanly.
session-chrome, summary-excerpt, and archive-uri tests now ride with chrome-state, summarizer, and compaction-archive; the excerpt rationale header returns to summarizer.ts.
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.
Summary: Four verbatim micro-folds (summary-excerpt into summarizer, archive-uri into compaction-archive, session-chrome into chrome-state; delivery fold reverted to avoid colliding with the delivery-queue lane), orphan test files renamed to owning modules, dropped doc sentences restored. Verification: per-seam suites 218/0 plus runtime-bridge/telemetry 92/0; typecheck shows only known pre-existing errors. Related to CL-7953 (micro-folds; boxes remain open)