Skip to content

docs: add authenticated star history chart#200

Merged
cyfyifanchen merged 4 commits into
mainfrom
fix/star_history_embed
Jul 23, 2026
Merged

docs: add authenticated star history chart#200
cyfyifanchen merged 4 commits into
mainfrom
fix/star_history_embed

Conversation

@cyfyifanchen

@cyfyifanchen cyfyifanchen commented Jul 23, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Replace the legacy anonymous Star History SVG with the official sealed-token embed in both English and Chinese READMEs.
  • Support light and dark color schemes for the Star History chart.
  • Keep the raw GitHub PAT out of the repository. The committed sealed token is the public embed value generated by Star History.

Type

  • Fix
  • Feature
  • Docs
  • CI / tooling
  • Refactor
  • Other

Verification

  • Relevant tests pass locally

  • Relevant lint / type checks pass locally

  • User-facing docs or screenshots are updated when needed

  • git diff --check origin/main...HEAD passed.

  • make check-large-files passed.

  • The interactive Star History chart rendered with the configured token. The embed endpoint also accepted and decrypted the sealed token, but a direct cold request intermittently returned HTTP 500 when its server-side GitHub fetch exceeded 10 seconds.

Risk

  • Security impact considered
  • Backward compatibility considered
  • Rollback path is clear for risky changes

The chart remains dependent on the remote Star History service rendering reliably on GitHub.

Related Issues

Tracks #202

@cyfyifanchen
cyfyifanchen marked this pull request as ready for review July 23, 2026 08:40
@cyfyifanchen
cyfyifanchen merged commit 7cad14a into main Jul 23, 2026
8 checks passed
@cyfyifanchen
cyfyifanchen deleted the fix/star_history_embed branch July 23, 2026 23:18
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