Skip to content

fix(desktop): disable "Summarize after this" on the last message#6001

Merged
SivanCola merged 2 commits into
esengine:main-v2from
HUQIANTAO:fix/disable-summ-from-on-last-turn
Jul 5, 2026
Merged

fix(desktop): disable "Summarize after this" on the last message#6001
SivanCola merged 2 commits into
esengine:main-v2from
HUQIANTAO:fix/disable-summ-from-on-last-turn

Conversation

@HUQIANTAO

Copy link
Copy Markdown
Contributor

Problem

The "Summarize after this" button (summ-from) is enabled on the last user message, but there is nothing after it to summarize. This is inconsistent with the existing behavior that correctly disables "Summarize before this" on the first message (turn <= 0).

Issue: #5744

Root Cause

Message.tsx:574-589actionDisabledReason("summ-from") checks canConversation but does not check whether the current turn is the last turn.

Changes

File Change
Message.tsx Add isLastTurn prop to TurnActions; check it in actionDisabledReason for summ-from
Transcript.tsx Pass isLastTurn={turn === maxTurn} from hot zone and warm zone rendering
en.ts Add rewind.disabledNoLater key
zh.ts Add rewind.disabledNoLater key
zh-TW.ts Add rewind.disabledNoLater key

Testing

  • TypeScript compiles with no new errors
  • The disable logic mirrors the existing summ-upto check on turn 0
  • Manual verification: "Summarize after this" should be greyed out on the last message with tooltip "There is nothing after this message to summarize"

The "Summarize after this" (summ-from) button was enabled on the last
user message even though there is nothing after it to summarize. This
mirrors the existing check that disables "Summarize before this" on
the first message (turn <= 0).

- Add isLastTurn prop to TurnActions component
- Check isLastTurn in actionDisabledReason for summ-from scope
- Pass isLastTurn from Transcript hot zone and WarmTurnItems
- Add i18n key rewind.disabledNoLater (en/zh/zh-TW)

Closes esengine#5744
@github-actions github-actions Bot added v2 Go rewrite (1.x) — main-v2 branch, active development desktop Wails desktop app (desktop/**) labels Jul 5, 2026
Use the same authoritative question turn mapping for the last-turn check that TurnActions uses for rewind actions. This keeps "Summarize after this" disabled on the last visible turn even when restored or paged history uses non-contiguous checkpoint turns.

Co-authored-by: SivanCola <[email protected]>
@SivanCola SivanCola merged commit 98166cb into esengine:main-v2 Jul 5, 2026
21 of 22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

desktop Wails desktop app (desktop/**) v2 Go rewrite (1.x) — main-v2 branch, active development

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants