Skip to content

feat(session): record the compaction trigger on fold records - #1525

Merged
TheGreatAxios merged 2 commits into
mainfrom
cl-10869-persist-compaction-trigger-reason-on-the-manifest-record
Oct 19, 2026
Merged

TheGreatAxios merged 2 commits into
mainfrom
cl-10869-persist-compaction-trigger-reason-on-the-manifest-record

Conversation

@TheGreatAxios

Copy link
Copy Markdown
Collaborator

Summary

  • Compaction manifest records now store the invocation trigger on decisions.trigger (director:operator-request, director:context-threshold, or director:context-overflow).
  • The existing human reason strings are unchanged, so a fold log distinguishes a typed /compact from the governor.

Verification

  • Pre-commit tsc --noEmit and bun build passed
  • Compaction unit tests passed (compactor-trigger, context-compactor, compaction-verify, compactor-excerpt-once, compactor-pairing: 93 pass). Omitting trigger fails the new tests.
  • Full bun run test was not re-run

Fixes CL-10869

@linear-code

linear-code Bot commented Oct 19, 2026

Copy link
Copy Markdown

CL-10869

@corbits-triage

corbits-triage Bot commented Oct 19, 2026 •

Copy link
Copy Markdown
Contributor

Triage: Pass

No changes needed from you. A maintainer will review this pull request.

@TheGreatAxios
TheGreatAxios merged commit b6d3f74 into main Oct 19, 2026
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant