chore: mark sponsor links as sponsored and shorten past-sponsors list - #58
Conversation
Sponsor links in the generated README sponsors section are paid placements, so they now carry the standard `sponsored` link type alongside `noopener` for the `target="_blank"` destination.
@standardserver/aws-lambda
@standardserver/bun
@standardserver/core
@standardserver/deno
@standardserver/fastify
@standardserver/fetch
@standardserver/node
@standardserver/peer
@standardserver/shared
commit: |
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
There was a problem hiding this comment.
✅ No new issues found.
Reviewed changes — Reviewed the single commit 50d0ba8 across 11 files (a script change plus 10 regenerated READMEs).
scripts/sync-sponsors.ts— both sponsor-link templates (tier tables and past-sponsors list) now emitrel="sponsored noopener"instead ofrel="noopener".- 10 checked-in
README.mdfiles — therelattribute on every sponsor link updated torel="sponsored noopener", no other sponsor data changed.
The change is consistent and correct: the script's two emission sites match the README output, and a repo-wide grep confirms no bare rel="noopener" remains in any sponsor link. This is a pure markup/SEO metadata change with no runtime behavioral surface.
DeepSeek Flash (default — pick a model for stronger reviews) | 𝕏
Merging this PR will not alter performance
Comparing Footnotes
|
The past-sponsors list rendered 38 avatar images, which dwarfed the active tiers above it. It is now one sentence crediting the count.
There was a problem hiding this comment.
✅ No new issues found.
Reviewed changes — Reviewed the incremental delta commit 8d458b2 (summarize past sponsors as a single line) on top of the previously-approved 50d0ba8.
- Replaced the past-sponsors avatar block in
buildSponsorsSectionwith a single thank-you sentence (With thanks to ${count} ${noun} who helped get us here., singular/plural aware) — the### Past Sponsorsheading,<p>, and 38-avatar loop are removed from the generator. - Regenerated all 10 checked-in
README.mdfiles; every sponsors section now ends at</table>followed by the summary sentence.
Repo-wide grep confirms the state is consistent: zero bare rel="noopener", zero residual ### Past Sponsors heading, no stray avatar <p> markup, and all 10 READMEs carrying the With thanks to … line — matching the generator output exactly. Pure README/generator change with no runtime behavioral surface.
DeepSeek Flash (default — pick a model for stronger reviews) | 𝕏
The blurb above the tier tables already links to GitHub Sponsors.
There was a problem hiding this comment.
✅ No new issues found.
Reviewed changes — Reviewed the incremental delta commit def0981 (link the past-sponsors line to GitHub Sponsors) on top of the previously-approved 8d458b2.
scripts/sync-sponsors.ts— the past-sponsors thank-you sentence is now emitted as a markdown link tohttps://github.com/sponsors/dinwwwh.- Regenerated all 10 checked-in
README.mdfiles, each carrying the identical linked line.
Repo-wide verification confirms the generator and all READMEs stay in sync. Pure README/generator markup change with no runtime behavioral surface.
DeepSeek Flash (default — pick a model for stronger reviews) | 𝕏

Two changes to the generated README sponsors section.
Sponsor links are paid placements, so they now declare that with the standard
sponsoredlink type:rel="noopener"becomesrel="sponsored noopener".The past-sponsors block rendered 38 avatar images, which visually dwarfed the active tiers above it. It is now a single sentence — "With thanks to 38 past sponsors who helped get us here." — with the
### Past Sponsorsheading dropped.Changes
scripts/sync-sponsors.tsemits both the newrelvalue and the summary line (with singular/plural handling for the count).README.mdfiles were regenerated, removing ~40 lines of avatar markup each.Testing
Regenerated via
node scripts/sync-sponsors.tsagainst live sponsor data; the diff touches only therelattribute and the past-sponsors block, with no unrelated sponsor churn. Lint-staged eslint passed on both commits.