Skip to content

Fix broken star history chart in README - #1229

Merged
KeitaW merged 2 commits into
awslabs:mainfrom
OctoBored:fix/star-history-chart
Aug 25, 2026
Merged

Fix broken star history chart in README#1229
KeitaW merged 2 commits into
awslabs:mainfrom
OctoBored:fix/star-history-chart

Conversation

@OctoBored

Copy link
Copy Markdown
Contributor

The star history chart embedded in the README is broken, as it relies on a data source that is now restricted by GitHub's stargazer API limits. This change updates the chart to use a working alternative that requires no API token, so the chart renders correctly again. No other functionality is affected.

The star history chart in the README is currently broken because the GitHub stargazer API restricts anonymous requests. Point the chart to a working alternative that serves the same data without requiring an API token.

@KeitaW KeitaW left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Review Batch 1/1 — README Asset URLs & Host Provenance

Thanks for chasing this down — I want to start by confirming your diagnosis, because it is correct and it is not obvious from an HTTP status alone. The current chart really is broken: https://api.star-history.com/svg?repos=…&type=Date returns HTTP 200 image/svg+xml, so every link checker passes it, but the SVG body is a placeholder whose text nodes read "GitHub restricted access to star data" / "GitHub team is looking into it" (verified live, 2026-08-25). So the README does render a dead chart today and this is worth fixing.

The problem is that the replacement URL as written doesn't fix it — it swaps a placeholder image for a broken-image icon. One-line fix in the inline comment on README.md:109, with a one-click suggestion block.

Which host should an AWS-owned README point at? (a maintainer call, not a defect)

star-history.dera.page is a different domain from the official star-history.com, and this line makes it both a remote image source and a reader-facing link from an awslabs README. That's a maintainer decision rather than a code defect, so I'm flagging it rather than asking for a change — but it is worth a deliberate answer before merge, since the repo would be handing a third party control over a rendered asset on its front page.

For what it's worth, the placeholder SVG currently being served names an official path back: https://star-history.com/blog/github-stargazer-api-restriction describes a workaround repo owners can apply to restore their own chart on the original host. If that works for this repo, it keeps the README on the domain it already trusted. If maintainers prefer this fork, the inline suggestion is what makes it actually render.

Pre-existing, non-blocking: the contributors line carries the same stale slug

README.md:105 uses awslabs/awsome-distributed too. I checked it rather than assuming, and it works — https://contrib.rocks/image?repo=awslabs/awsome-distributed returns 200 image/svg+xml, 365,443 bytes (verified live, 2026-08-25), because that service follows the GitHub rename redirect. So there is nothing to fix today. Mentioning it only because it is the same latent staleness, and canonicalizing both slugs in one pass would remove the class — your call whether that belongs in this focused PR.

Things That Look Great

  • The diagnosis is right, and it was not cheap to reach. A status-code check says the old URL is fine; only reading the SVG body reveals the "GitHub restricted access to star data" placeholder. The PR description names the cause (GitHub's stargazer API restriction) accurately.
  • Well-scoped. One file, +1/-1, no drive-by edits — exactly the shape a README fix should have, and it made this straightforward to verify end-to-end.
  • The markdown itself is correct, including the unescaped & in the query string, which renders fine in GitHub-flavored Markdown.
  • The replacement host does work. With the corrected slug it serves a real chart with the full history — so the approach is sound and only the slug stands between this and a working chart.

Sources

Live verification (all 2026-08-25, curl -sS -L):

  • https://api.star-history.com/svg?repos=awslabs/awsome-distributed&type=Date200 image/svg+xml, 60,125 B; body text: "GitHub restricted access to star data". Identical placeholder for the -ai slug.
  • https://star-history.dera.page/svg?repos=awslabs/awsome-distributed&type=Date404 text/plain, "Repo not found" (3/3 attempts).
  • https://star-history.dera.page/svg?repos=awslabs/awsome-distributed-ai&type=Date200 image/svg+xml, 67,387 B, chart with 2024–2026 axis.
  • https://contrib.rocks/image?repo=awslabs/awsome-distributed200 image/svg+xml, 365,443 B.
  • https://api.github.com/repos/awslabs/awsome-distributed301 → repository 698678789 = awslabs/awsome-distributed-ai, 471 stars.

Docs:

Comment thread README.md Outdated
@KeitaW
KeitaW merged commit 6e866bf into awslabs:main Aug 25, 2026
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