Skip to content

Add top-level announcement notification - #11379

Merged
bturcotte520 merged 4 commits into
mainfrom
add-top-level-announcement-notification
Jun 17, 2026
Merged

Add top-level announcement notification#11379
bturcotte520 merged 4 commits into
mainfrom
add-top-level-announcement-notification

Conversation

@kilo-code-bot

@kilo-code-bot kilo-code-bot Bot commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Updates the static notification message to: "We're giving away $500 of AI Credits when we reach 25,000 stars on GitHub. Support us:" with a link to https://github.com/Kilo-Org/kilocode/
  • Makes the notification dismissible — removed the STATIC_IDS guard that blocked dismissal, so it now behaves like a normal notification
  • Notification is still prepended before API-fetched notifications until dismissed

Built for Brian Turcotte by Kilo for Slack

// Static notifications always shown unconditionally (not fetched from API, not dismissable)
const STATIC_NOTIFICATIONS: KilocodeNotification[] = [
{
id: "star-giveaway-june-2026",

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

SUGGESTION: This notification promotes a time-limited event ("June 24th") but has no expiry mechanism. After June 24, 2026 passes, this notification will continue showing to all users indefinitely until the code is manually removed in a follow-up PR. Consider adding a date guard so it automatically stops displaying after the event ends, or ensure a follow-up PR is planned to remove it.


Reply with @kilocode-bot fix it to have Kilo Code address this issue.

@kilo-code-bot

kilo-code-bot Bot commented Jun 17, 2026

Copy link
Copy Markdown
Contributor Author

Code Review Summary

Status: No Issues Found | Recommendation: Merge

The incremental changes since the last review (commit 7cd018af) are limited to a prettier formatting fix in notifications.tsx (message collapsed to a single line) and an auto-generated source-links.md update. No new issues introduced.

The previous SUGGESTION about the time-limited promotion lacking an expiry mechanism is resolved — the notification is dismissible via dismiss(id).

Other Observations (not in diff)
File Line Issue
N/A N/A No changeset present for this user-facing change (notification message update + dismissibility). A .changeset/*.md describing the user-visible change would be appropriate.
Files Reviewed (2 files)
  • packages/kilo-vscode/webview-ui/src/context/notifications.tsx - 0 issues
  • packages/kilo-docs/source-links.md - 0 issues (auto-generated)

Fix these issues in Kilo Cloud

Previous Review Summaries (2 snapshots, latest commit 7cd018a)

Current summary above is authoritative. Previous snapshots are kept for context only.

Previous review (commit 7cd018a)

Status: 1 Issue Found | Recommendation: Address before merge

Overview

Severity Count
CRITICAL 0
WARNING 0
SUGGESTION 0

Note: The previous SUGGESTION (line 17, "Time-limited promotion has no expiry mechanism") is now resolved — the notification is dismissible.

Other Observations (not in diff)
File Line Issue
N/A N/A Missing changeset — this is a user-facing change (updating notification message and making it dismissible in the extension UI). A .changeset/*.md file should be included with a user-facing description.
Files Reviewed (1 file)
  • packages/kilo-vscode/webview-ui/src/context/notifications.tsx - 0 new issues

Fix these issues in Kilo Cloud

Previous review (commit f02deb5)

Status: 1 Issue Found | Recommendation: Address before merge

Overview

Severity Count
CRITICAL 0
WARNING 0
SUGGESTION 1
Issue Details (click to expand)

SUGGESTION

File Line Issue
packages/kilo-vscode/webview-ui/src/context/notifications.tsx 17 Time-limited promotion has no expiry mechanism
Other Observations (not in diff)
File Line Issue
N/A N/A Missing changeset — this is a user-facing change (adding a promotional notification banner to the extension UI). A .changeset/*.md file should be included with a user-facing description.
Files Reviewed (1 file)
  • packages/kilo-vscode/webview-ui/src/context/notifications.tsx - 1 issue

Fix these issues in Kilo Cloud


Reviewed by glm-5.2-20260616 · 153,817 tokens

Review guidance: REVIEW.md from base branch main

@bturcotte520
bturcotte520 requested a review from olearycrew June 17, 2026 21:55
@bturcotte520
bturcotte520 merged commit ebac27a into main Jun 17, 2026
24 checks passed
@bturcotte520
bturcotte520 deleted the add-top-level-announcement-notification branch June 17, 2026 22:24
NJ-2020 pushed a commit to NJ-2020/kilocode that referenced this pull request Jul 4, 2026
t7tran pushed a commit to t7tran/kilocode that referenced this pull request Aug 14, 2026
t7tran pushed a commit to t7tran/kilocode that referenced this pull request Aug 14, 2026
…ement-notification

Add top-level announcement notification
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