fix(ui): replace repeated live hints#1624
Conversation
|
Thanks — fix itself is tight and the reducer behavior test is exactly the right shape. One blocker though: the branch is behind main and would silently revert four merges that landed today. Drift
GH shows What to do
Main's been moving fast this week — if the rebase takes a while, you may need a second one. Push when ready and I'll merge. |
a97d3f7 to
f763454
Compare
|
let more message apply to this mode? like edit mode change or else? |
|
@dacec354 Thanks for the suggestion. Since this PR is already merged and was intentionally scoped to repeated live hints, could you open a separate issue for this request? It would be helpful to include the exact mode(s), the messages you want covered, and the expected behavior, so we can discuss the scope clearly there. |
What
Make
live.showreplace an existing live card when the same id is reused, and route Alt+S stash/recall notices through a stable hint id so repeated presses update the same card instead of adding stale scrollback entries.Why
Fixes #1573. Repeated transient hints such as stash/recall confirmations were accumulating because every
pushInfocall generated a fresh card id.How to verify
npm test -- tests/ui-reducer.test.tsnpm run typechecknpm run verifyChecklist
npm run verifypasses locally (lint + typecheck + tests + comment-policy gate)Co-Authored-By: Claudetrailer in commitsCHANGELOG.md— release notes are maintainer-written at release time