Skip to content

chore: sync sponsors with new upstream schema - #71

Merged
dinwwwh merged 1 commit into
mainfrom
claude/update-generated-readme-b2fe25
Aug 20, 2026
Merged

chore: sync sponsors with new upstream schema#71
dinwwwh merged 1 commit into
mainfrom
claude/update-generated-readme-b2fe25

Conversation

@dinwwwh

@dinwwwh dinwwwh commented Aug 20, 2026

Copy link
Copy Markdown
Member

Ports the sponsor-sync changes from middleapi/orpc#1897 to this repo's scripts/sync-sponsors.ts and regenerates the sponsors section in every README. Slot sponsors now appear as featured full-width cards above the tier tables, and every link carries the tracking and rel tokens the upstream data provides.

Changes

  • Sponsor links are no longer rewritten with ref=orpc — they arrive pre-tracked from upstream, and each link's rel is noopener plus the sponsor's own tokens instead of a hardcoded sponsored noopener.
  • Sponsors that bought an ad slot render as full-width cards (floated logo, name over tagline) and are excluded from the tier tables, so nobody appears twice.
  • Tier headings use the new plural tier titles; image sizes rank against all active tiers including the featured ones, so the tables keep their previous 167/139/119 widths.
  • The past-sponsors line links to the full sponsor wall SVG.
  • Section replacement trims to a single trailing newline, so re-syncs no longer fight eslint's markdown fixer over the last line of a README.

The PR's apps/content parts (slots.ts generation, ad-card component) have no counterpart here and were skipped.

Testing

  • pnpm run sponsors:sync passes with eslint at --max-warnings=0, and a second run leaves every README byte-identical.
  • scripts/sync-sponsors.ts passes eslint and tsc --noEmit.

Port the sync-sponsors changes from middleapi/orpc#1897: links arrive
pre-tracked with per-sponsor rel tokens, slot sponsors render as featured
full-width cards above the tier tables, tier headings use plural titles
with slot-aware image sizing, the past-sponsors line links to the sponsor
wall SVG, and section replacement trims to a single trailing newline so
re-syncs stay idempotent against eslint's markdown fixer.
@pkg-pr-new

pkg-pr-new Bot commented Aug 20, 2026

Copy link
Copy Markdown
@standardserver/aws-lambda

npm i https://pkg.pr.new/@standardserver/aws-lambda@71

@standardserver/core

npm i https://pkg.pr.new/@standardserver/core@71

@standardserver/fastify

npm i https://pkg.pr.new/@standardserver/fastify@71

@standardserver/fetch

npm i https://pkg.pr.new/@standardserver/fetch@71

@standardserver/node

npm i https://pkg.pr.new/@standardserver/node@71

@standardserver/peer

npm i https://pkg.pr.new/@standardserver/peer@71

@standardserver/shared

npm i https://pkg.pr.new/@standardserver/shared@71

commit: df75d29

@codecov

codecov Bot commented Aug 20, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@codspeed-hq

codspeed-hq Bot commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

Merging this PR will not alter performance

✅ 26 untouched benchmarks
⏩ 108 skipped benchmarks1


Comparing claude/update-generated-readme-b2fe25 (df75d29) with main (d6d2794)2

Open in CodSpeed

Footnotes

  1. 108 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports.

  2. No successful run was found on main (2c489f1) during the generation of this report, so d6d2794 was used instead as the comparison base. There might be some changes unrelated to this pull request in this report.

@pullfrog pullfrog Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

ℹ️ Minor suggestion only.

Reviewed changes

The PR ports the sponsor-sync behavior from middleapi/orpc#1897 into scripts/sync-sponsors.ts and regenerates the ## Sponsors section across all 8 READMEs (root symlinks to packages/core/README.md; the generator script is the other changed file). I read the full generator diff and verified the regenerated README output end-to-end across core, fastify, and the tests/bun files — the blocks are structurally identical (same generator writes every file), so the remaining generated READMEs carry the same changes. Everything lines up with the PR description.

  • Ad slot cards — sponsors with a slot value now render as full-width featured cards (buildSlotCards) above the tier tables and are excluded from the tables, so no one appears twice.
  • Tracking/relwithTracking (?ref=orpc) is removed since links arrive pre-tracked upstream; rel is now noopener plus the sponsor's own tokens via relAttribute, on both card and tier-link emission sites.
  • Tier sizing — image widths rank against all active tiers (sizeTierLevels), keeping the tables at their prior 167/139/119 widths even though the top tiers now render as cards.
  • Headings / past sponsors — tier headings use the new plural tierTitlePlural; the past-sponsors line links to the sponsor-wall SVG.
  • IdempotencyreplaceSponsorsSection trims to a single trailing newline, ending the README-EOF/elint-fixer tug-of-war the author describes.

One minor suggestion, inline. This is otherwise a careful, well-scoped chore PR.

Pullfrog  | Fix all ➔Fix 👍s ➔View workflow run | Using DeepSeek Flash (default — pick a model for stronger reviews) | 𝕏

Comment thread scripts/sync-sponsors.ts
const escapedName = escapeHtml(displayName)

lines.push(` <td align="center"><a href="${escapeHtml(href)}" target="_blank" rel="sponsored noopener" title="${escapedName}"><img src="${escapeHtml(sponsor.avatar)}" width="${imageSize}" alt="${escapedName}"/><br />${escapedName}</a></td>`)
lines.push(` <td align="center"><a href="${escapeHtml(href)}" target="_blank" rel="${relAttribute(sponsor)}" title="${escapedName}"><img src="${escapeHtml(sponsor.avatar)}" width="${imageSize}" alt="${escapedName}"/><br />${escapedName}</a></td>`)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Nit: sponsor.rel is interpolated into the rel="..." attribute unescaped, unlike every other trusted-but-escaped field on this line (and the card site on L97). If a sponsor's rel ever carried a ", it would break out of the attribute. Wrap the value with the existing escapeHtml helper for consistency with the rest of the templated line.

@dinwwwh
dinwwwh merged commit 4ef2051 into main Aug 20, 2026
10 of 11 checks passed
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