Skip to content

feat(chat): show compaction summary on token hover - #985

Merged
vastsa merged 2 commits into
mainfrom
feat/context-compaction-summary-hover
Sep 24, 2026
Merged

vastsa merged 2 commits into
mainfrom
feat/context-compaction-summary-hover

Conversation

@vastsa

@vastsa vastsa commented Sep 24, 2026

Copy link
Copy Markdown
Owner

Hover the ≈N tokens estimate in the transcript compaction row or the context inspector to see the full persisted summary text.

Changes

  • app-state.ts — extend sessionCompactions type with summary?: string
  • session-coordination.ts — carry record.summary when mapping marks on session load
  • app-store.ts — preserve existing summary when a live event replaces a mark
  • AssistantTurn.tsx / ContextUsageInspector.tsx — add title on the token count (only when summarized, no fallback)
  • context-compaction.test.mjs — sync two drifted regex assertions

Not changed

  • Event protocol stays lightweight (no summary in compaction_end)
  • Summary appears after session load; a just-completed compaction shows it on next open

Pass record.summary through to the renderer when loading session
compactions, and display it as a title tooltip on the token count
in both the transcript divider row and the context usage inspector.
Fallback and no-summary checkpoints never expose their marker text.
Copilot AI lite review requested due to automatic review settings September 24, 2026 02:44

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@vastsa
vastsa merged commit d4b9402 into main Sep 24, 2026
3 of 4 checks passed
@vastsa
vastsa deleted the feat/context-compaction-summary-hover branch October 6, 2026 16:56

This branch was previously deployed

1 inactive deployment
Preview — 34fcbbb2 Deployed Sep 24, 2026 by vercel[bot]
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