Skip to content

fix(kimi-web): clarify desktop notification title and icon - #1434

Merged
wbxl2000 merged 1 commit into
mainfrom
fix/web-notification-title-icon
Jul 6, 2026
Merged

fix(kimi-web): clarify desktop notification title and icon#1434
wbxl2000 merged 1 commit into
mainfrom
fix/web-notification-title-icon

Conversation

@wbxl2000

@wbxl2000 wbxl2000 commented Jul 6, 2026

Copy link
Copy Markdown
Collaborator

Related Issue

N/A — small UX polish to web desktop notifications.

Problem

Web desktop notifications previously reused the session title as the notification title and shipped with no explicit icon, so a finished turn and a question needing an answer looked nearly identical and showed a generic browser icon.

What changed

  • Use a fixed, event-aware title (Kimi Code · Turn finished / Kimi Code · Needs answer) and put the session title or question preview in the body, falling back to a neutral hint when neither is available.
  • Set the Kimi favicon as the notification icon.
  • Localize the new copy in English and Chinese, and add unit tests for the notification copy builder.

This keeps the existing opt-in model intact: completion notifications default on, question notifications default off to keep question text behind explicit consent.

Checklist

  • I have read the CONTRIBUTING document.
  • I have linked a related issue, or explained the problem above.
  • I have added tests that prove my feature works.
  • Ran gen-changesets skill, or this PR needs no changeset.
  • Ran gen-docs skill, or this PR needs no doc update.

@changeset-bot

changeset-bot Bot commented Jul 6, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: c239370

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@moonshot-ai/kimi-code Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@pkg-pr-new

pkg-pr-new Bot commented Jul 6, 2026

Copy link
Copy Markdown
pnpm dlx https://pkg.pr.new/@moonshot-ai/kimi-code@c239370
npx https://pkg.pr.new/@moonshot-ai/kimi-code@c239370

commit: c239370

@wbxl2000
wbxl2000 merged commit 4aacddc into main Jul 6, 2026
11 checks passed
@wbxl2000
wbxl2000 deleted the fix/web-notification-title-icon branch July 6, 2026 13:18
This was referenced Jul 6, 2026
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