Skip to content

fix(website): refresh repository stars at runtime - #371

Merged
veryCrunchy merged 4 commits into
mainfrom
fix/website-star-count
Aug 13, 2026
Merged

fix(website): refresh repository stars at runtime#371
veryCrunchy merged 4 commits into
mainfrom
fix/website-star-count

Conversation

@veryCrunchy

Copy link
Copy Markdown
Member

Summary

  • refresh the repository star count at runtime through a same-origin endpoint
  • cache GitHub responses for ten minutes and serve stale data during temporary upstream failures
  • retain a validated pre-rendered fallback so the website remains usable before or without a successful refresh

Validation

  • npm run --prefix website test (49 tests)
  • npm run --prefix website build (client, SSR, 31 prerendered routes, 222 capture assets)
  • bash tools/check-repository.sh
  • docker build -f website/Dockerfile -t nc-native-website-star-count:final .
  • production image smoke test: homepage and repository endpoint returned HTTP 200; repeated requests used one Nginx cache entry

Deployment note

The website container needs outbound HTTPS access to api.github.com. The displayed count can intentionally lag GitHub by up to ten minutes.

Closes #369

@obiente-cloud
obiente-cloud Bot temporarily deployed to Obiente Preview / PR #371 / NC Native August 13, 2026 19:36 Destroyed
@obiente-cloud

obiente-cloud Bot commented Aug 13, 2026

Copy link
Copy Markdown

Obiente preview

NC Native · 001bee34e51d · Removed

View preview status

View in Obiente

Obiente updates this comment as the preview changes.

@veryCrunchy veryCrunchy added priority:P1 Required for the pre-release product experience area:website Public website, documentation presentation, and roadmap embedding area:ux Navigation, accessibility, responsive design, and interaction type:reliability Correctness, crash prevention, recovery, and data safety type:bug labels Aug 13, 2026
@veryCrunchy veryCrunchy added this to the 0.6 Platform polish milestone Aug 13, 2026
@obiente-cloud
obiente-cloud Bot temporarily deployed to Obiente Preview / PR #371 / NC Native August 13, 2026 19:37 Destroyed

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 43fac33b90

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread website/nginx.conf
Comment thread website/nginx.conf Outdated
Comment thread website/nginx.conf Outdated
@obiente-cloud
obiente-cloud Bot temporarily deployed to Obiente Preview / PR #371 / NC Native August 13, 2026 19:45 Destroyed

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 8daab628ab

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread website/nginx.conf Outdated
Comment thread website/nginx.conf
@obiente-cloud
obiente-cloud Bot temporarily deployed to Obiente Preview / PR #371 / NC Native August 13, 2026 20:00 Destroyed
@veryCrunchy
veryCrunchy merged commit 3bac270 into main Aug 13, 2026
3 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:ux Navigation, accessibility, responsive design, and interaction area:website Public website, documentation presentation, and roadmap embedding priority:P1 Required for the pre-release product experience type:bug type:reliability Correctness, crash prevention, recovery, and data safety

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix(website): keep the repository star count current

1 participant