Skip to content

Remove GitHub star giveaway notification#4592

Merged
bturcotte520 merged 2 commits into
mainfrom
session/agent_9d3172f7-67a1-492a-8997-e83c2879f0d7
Jul 16, 2026
Merged

Remove GitHub star giveaway notification#4592
bturcotte520 merged 2 commits into
mainfrom
session/agent_9d3172f7-67a1-492a-8997-e83c2879f0d7

Conversation

@bturcotte520

Copy link
Copy Markdown
Contributor

Summary

Removes the extension notification promoting a "star our GitHub repo for a chance to win $500 of AI Credits" giveaway.

The entry (id: 'star-giveaway-june-2026') lived in the static normalUnconditionalNotifications array in apps/web/src/lib/notifications.ts, which feeds the /api/users/notifications route consumed by the VS Code extension. It has been deleted entirely, along with its title, message copy, and action link.

Verification

  • Confirmed no other file references the notification id, title, or message text (repo-wide grep).
  • No tests specifically covered this notification entry; existing generic tests in notifications.test.ts are unaffected by the removal.
  • Confirmed pnpm format:changed produces no formatting diffs for the change.
  • Could not run the full Jest suite for this file in this sandbox because it requires a running Postgres test database (Docker is unavailable here); the change itself is a self-contained array-entry deletion with no other code depending on it.

Built for Brian Turcotte by Kilo for Slack

Co-authored-by: kiloconnect[bot] <240665456+kiloconnect[bot]@users.noreply.github.com>
@kilo-code-bot

kilo-code-bot Bot commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Executive Summary

Clean deletion of a single static notification entry (star-giveaway-june-2026) from the normalUnconditionalNotifications array, with no remaining references or side effects.

Files Reviewed (1 files)
  • apps/web/src/lib/notifications.ts

Reviewed by claude-sonnet-5 · Input: 12 · Output: 1.7K · Cached: 205.3K

Review guidance: REVIEW.md from base branch main

@bturcotte520 bturcotte520 enabled auto-merge (squash) July 16, 2026 17:30
@bturcotte520 bturcotte520 merged commit 4b2a9f0 into main Jul 16, 2026
14 checks passed
@bturcotte520 bturcotte520 deleted the session/agent_9d3172f7-67a1-492a-8997-e83c2879f0d7 branch July 16, 2026 17:40
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