Skip to content

feat(web): add GitHub star giveaway notification - #4095

Merged
markijbema merged 1 commit into
mainfrom
mark/star-giveaway-notification
Jun 18, 2026
Merged

feat(web): add GitHub star giveaway notification#4095
markijbema merged 1 commit into
mainfrom
mark/star-giveaway-notification

Conversation

@markijbema

Copy link
Copy Markdown
Contributor

Summary

Adds the GitHub star giveaway announcement as a server-side notification in the cloud platform, where client-facing notifications belong — rather than hardcoding it into the VS Code extension webview (as done in kilocode#11379).

  • Adds a star-giveaway-june-2026 entry to the static notification list in apps/web/src/lib/notifications.ts, served via GET /api/users/notifications to the extension and CLI.
  • Message: "We're giving away $500 of AI Credits when we reach 25,000 stars on GitHub. Support us:" with an action linking to https://github.com/Kilo-Org/kilocode/
  • Because it flows through the centralized notification system, it ships to existing clients without an extension release, is dismissible by id like any other notification, and can be edited/removed server-side.

Verification

Visual Changes

N/A

Reviewer Notes

  • No expiresAt is set, mirroring the source PR (shows until dismissed). The existing entries generally favor an expiry to auto-hide server-side — happy to add one (e.g. end of July) if preferred.
  • No showIn is set, so it targets all clients with notification support.

Co-authored-by: kiloconnect[bot] <240665456+kiloconnect[bot]@users.noreply.github.com>
@markijbema markijbema self-assigned this Jun 18, 2026
@kilo-code-bot

kilo-code-bot Bot commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Executive Summary

Reviewed the single-file addition of a star-giveaway-june-2026 static notification; the entry matches the KiloNotification shape, uses a unique id, and introduces no memory leaks or runtime risks.

Files Reviewed (1 file)
  • apps/web/src/lib/notifications.ts — added one entry to normalUnconditionalNotifications
Notes (not blockers)
  • No expiresAt is set, so the notification persists until dismissed server-side. This is consistent with several existing entries (kilo-cli-jan-5, kilo-cloud-agents-jan-15, kilo-console-beta, kilo-pass-announcement-jan-12) and is already acknowledged in the PR's Reviewer Notes.
  • No showIn is set, so it surfaces in all clients with notification support. Also called out in Reviewer Notes.
  • No new closures, event listeners, or module-scope I/O clients are introduced — no memory-leak concerns.

Reviewed by glm-5.2-20260616 · 211,173 tokens

Review guidance: REVIEW.md from base branch main

@markijbema
markijbema enabled auto-merge (squash) June 18, 2026 12:19
@markijbema
markijbema merged commit db7008a into main Jun 18, 2026
16 checks passed
@markijbema
markijbema deleted the mark/star-giveaway-notification branch June 18, 2026 12:23
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