Skip to content

fix: keep star-tracker count fresh on webhook#56

Merged
wavekat-eason merged 1 commit into
mainfrom
fix/star-tracker-stale-count
May 15, 2026
Merged

fix: keep star-tracker count fresh on webhook#56
wavekat-eason merged 1 commit into
mainfrom
fix/star-tracker-stale-count

Conversation

@wavekat-eason
Copy link
Copy Markdown
Contributor

Summary

  • repos.stargazers_count was only written by backfill (register + nightly cron), so live star webhooks updated the chart series but left the "Tracked repos — N stars" line stale.
  • Use the post-action repository.stargazers_count GitHub includes on the star payload to refresh the cached count on every event.

Repro

On /wavekat, wavekat-sip showed +1 in Recent activity but 0 stars in the Tracked repos list. The repo was backfilled at 0 stars; the subsequent webhook never bumped the count.

Test plan

  • Star a tracked repo and confirm the count on /:slug updates without waiting for nightly reconcile.
  • Unstar and confirm the count decrements (GitHub sends the post-action count for deleted too).

🤖 Generated with Claude Code

stargazers_count was only written by backfill, so live star webhooks
updated the chart but left the "Tracked repos — N stars" line frozen
at the last backfill value. Use the post-action count GitHub includes
on the star payload to refresh it on every event.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@github-actions
Copy link
Copy Markdown
Contributor

@wavekat-eason wavekat-eason merged commit 1fdf264 into main May 15, 2026
4 checks passed
@wavekat-eason wavekat-eason deleted the fix/star-tracker-stale-count branch May 15, 2026 23:47
wavekat-eason pushed a commit that referenced this pull request May 15, 2026
🤖 I have created a release *beep* *boop*
---


##
[0.0.21](wavekat-com-v0.0.20...wavekat-com-v0.0.21)
(2026-05-15)


### Features

* add star-tracker worker
([#52](#52))
([6f13ee1](6f13ee1))
* add wavekat-asr to ecosystem
([#50](#50))
([f26f29b](f26f29b))
* auto-theme star-tracker embed snippet
([#55](#55))
([4989603](4989603))
* star-tracker recent trend, range, per-repo charts
([#54](#54))
([7189f91](7189f91))


### Bug Fixes

* keep star-tracker count fresh on webhook
([#56](#56))
([1fdf264](1fdf264))
* stop long embed snippets blowing out page width
([#57](#57))
([7b6b096](7b6b096))
* wrap star-tracker header on mobile
([#53](#53))
([82e92b9](82e92b9))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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