Skip to content

New versions go into the notice list - #290

Merged
fylorn merged 1 commit into
devfrom
notice-update-in-list
Oct 7, 2026
Merged

fylorn merged 1 commit into
devfrom
notice-update-in-list

Conversation

@fylorn

@fylorn fylorn commented Oct 7, 2026

Copy link
Copy Markdown
Contributor

An update found by the daily check used to be a one-off system notification; once dismissed from Notification Center, only the menu item was left. It is now a regular notice:

  • Listed in the bell at info level (grey dot, the badge does not turn red); shown as a system notification when notices are set to that, list-only when set to in-app
  • One notice per version (update:<version>): the same version found again is left alone; a newer one withdraws the older
  • Withdrawn when a check finds nothing newer, and at startup (the found version only lives in memory, so a leftover notice would open an empty update window)
  • Clicking it in the list, the menu bar or Notification Center opens the update window and marks it read

New Signal::tell() lets an info-level signal still interrupt; everything else (dedup, token bucket) is unchanged.

Local: clippy --all-targets -D warnings, cargo test (436 + control-plane 9 against twcore v0.64.0), pnpm typecheck, pnpm test (872).

🤖 Generated with Claude Code

An update found by the daily check used to be a one-off system notification:
once it was dismissed from Notification Center, the only trace left was the
menu item. It is now a notice like the others: listed in the bell (grey, not
red), shown as a system notification when notices are set to that, and listed
only when set to in-app.

- One notice per version (`update:<version>`): finding the same version again
  the next day leaves it alone; finding a newer one withdraws the older one
- Withdrawn when a check finds nothing newer, and at startup: the found version
  only lives in memory, so a notice left from the last run would open an empty
  update window
- Clicking it in the list, the menu bar or Notification Center opens the
  update window and marks it read

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@fylorn
fylorn merged commit e0e3a09 into dev Oct 7, 2026
4 checks passed
@fylorn
fylorn deleted the notice-update-in-list branch October 7, 2026 05:47
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.

1 participant