Skip to content

docs: add Star History chart image with sealed token #66

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
afonsoft merged 1 commit into from
Jul 11, 2026
Merged

docs: add Star History chart image with sealed token #66

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -337,7 +337,7 @@ reportgenerator \

## Star History

See the [star history chart](https://www.star-history.com/?repos=afonsoft%2FQRCoder.Core&type=date&legend=top-left) for this repository.
[![Star History Chart](https://api.star-history.com/chart?repos=afonsoft/QRCoder.Core&type=date&legend=top-left&sealed_token=a1KQPztifboHTq3RlLRkwvvPGw3dtFbndqZQpJxDeVkKCiIiJQyi1kuTEocyVlxTQ_wR7U0f6lQ1gBk00x7MNA3wOhureCKWKGRw0nT27SQSXMsSECsH8w)](https://www.star-history.com/?repos=afonsoft%2FQRCoder.Core&type=date&legend=top-left)

@devin-ai-integration devin-ai-integration Bot Jul 11, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

📝 Info: Star-history sealed_token in URL is a public embedding token, not a secret

The sealed_token parameter in the star-history API URL (e.g., https://api.star-history.com/chart?...&sealed_token=a1KQPz...) might appear to be a committed secret. However, star-history.com provides these tokens publicly for anyone embedding their chart badges — they are not private API keys or credentials. This does not violate the AGENTS.md Hard Rule H4 ("Secrets: nunca commitar .env, tokens ou API keys") since it is not a secret credential.

Open in Devin Review

Was this helpful? React with 👍 or 👎 to provide feedback.


## StarMapper

Expand Down
2 changes: 1 addition & 1 deletion README.pt-br.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -361,7 +361,7 @@ reportgenerator \

## Historico de Estrelas

Veja o [grafico de historico de estrelas](https://www.star-history.com/?repos=afonsoft%2FQRCoder.Core&type=date&legend=top-left) deste repositorio.
[![Star History Chart](https://api.star-history.com/chart?repos=afonsoft/QRCoder.Core&type=date&legend=top-left&sealed_token=a1KQPztifboHTq3RlLRkwvvPGw3dtFbndqZQpJxDeVkKCiIiJQyi1kuTEocyVlxTQ_wR7U0f6lQ1gBk00x7MNA3wOhureCKWKGRw0nT27SQSXMsSECsH8w)](https://www.star-history.com/?repos=afonsoft%2FQRCoder.Core&type=date&legend=top-left)

## StarMapper

Expand Down
3 changes: 1 addition & 2 deletions readme.md
View file
Open in desktop

@devin-ai-integration devin-ai-integration Bot Jul 11, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

🔍 Duplicate README files may conflict on case-insensitive file systems

The repository has both README.md and readme.md at the root with different content. On case-insensitive file systems (Windows, macOS default), these two files would collide — only one would be visible, and git operations could behave unpredictably. This is a pre-existing issue, but since this PR modifies both files and the project explicitly claims cross-platform support (Windows, Linux, macOS per AGENTS.md:103), it's worth noting. GitHub renders README.md by default, so readme.md may not be serving its intended purpose.

Open in Devin Review

Was this helpful? React with 👍 or 👎 to provide feedback.

Original file line number Diff line number Diff line change
Expand Up @@ -284,8 +284,7 @@ The project uses GitHub Actions for automated build, test, security scanning, an

## Star History / Histórico de Estrelas

See the [star history chart](https://www.star-history.com/?repos=afonsoft%2FQRCoder.Core&type=date&legend=top-left) for this repository.<br/>
Veja o [gráfico de histórico de estrelas](https://www.star-history.com/?repos=afonsoft%2FQRCoder.Core&type=date&legend=top-left) deste repositório.
[![Star History Chart](https://api.star-history.com/chart?repos=afonsoft/QRCoder.Core&type=date&legend=top-left&sealed_token=a1KQPztifboHTq3RlLRkwvvPGw3dtFbndqZQpJxDeVkKCiIiJQyi1kuTEocyVlxTQ_wR7U0f6lQ1gBk00x7MNA3wOhureCKWKGRw0nT27SQSXMsSECsH8w)](https://www.star-history.com/?repos=afonsoft%2FQRCoder.Core&type=date&legend=top-left)

@devin-ai-integration devin-ai-integration Bot Jul 11, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

🟡 Portuguese description removed from the bilingual README's star history section

The Portuguese translation of the star history description is dropped (readme.md:287) when replacing the two bilingual lines with a single English-only image badge, so the bilingual README loses its Portuguese content in that section.

Impact: The bilingual README no longer provides Portuguese text for the star history section, breaking the project's bilingual documentation standard.

Bilingual documentation rule violation details

The readme.md file is the project's bilingual (EN/PT-BR) README — all section headers use the dual-language pattern (e.g., "Star History / Histórico de Estrelas", "License / Licença").

Before this change, the Star History section had two lines:

  • English: See the [star history chart](...) for this repository.
  • Portuguese: Veja o [gráfico de histórico de estrelas](...) deste repositório.

After this change, both lines are replaced with a single image badge with English-only alt text (Star History Chart). The Portuguese description is entirely removed.

AGENTS.md explicitly requires under "DO (Faça)": "Documentação bilíngue (pt-BR + en-US)"

Suggested change
[![Star History Chart](https://api.star-history.com/chart?repos=afonsoft/QRCoder.Core&type=date&legend=top-left&sealed_token=a1KQPztifboHTq3RlLRkwvvPGw3dtFbndqZQpJxDeVkKCiIiJQyi1kuTEocyVlxTQ_wR7U0f6lQ1gBk00x7MNA3wOhureCKWKGRw0nT27SQSXMsSECsH8w)](https://www.star-history.com/?repos=afonsoft%2FQRCoder.Core&type=date&legend=top-left)
[![Star History Chart](https://api.star-history.com/chart?repos=afonsoft/QRCoder.Core&type=date&legend=top-left&sealed_token=a1KQPztifboHTq3RlLRkwvvPGw3dtFbndqZQpJxDeVkKCiIiJQyi1kuTEocyVlxTQ_wR7U0f6lQ1gBk00x7MNA3wOhureCKWKGRw0nT27SQSXMsSECsH8w)](https://www.star-history.com/?repos=afonsoft%2FQRCoder.Core&type=date&legend=top-left)
See the star history chart for this repository. / Veja o gráfico de histórico de estrelas deste repositório.
Open in Devin Review

Was this helpful? React with 👍 or 👎 to provide feedback.


## StarMapper

Expand Down
Loading