Emit the site.standard.publication link tag on every page - #62
Emit the site.standard.publication link tag on every page#62RobbieTheWagner wants to merge 1 commit into
Conversation
Bluesky's AppView needs a site-wide <link rel="site.standard.publication"> discovery hint on every page (home and episode pages alike) to render shared links as a verified publication; until now the package only emitted the per-episode site.standard.document tag and the /.well-known endpoint. Gated on STANDARD_SITE_DID and STANDARD_SITE_PUBLICATION_RKEY, so sites without standard.site publishing are unaffected. This restores a fix whiskey.fm carried in its fork before converting to the npm package. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 WalkthroughWalkthrough
ChangesPublication link
Estimated code review effort: 2 (Simple) | ~10 minutes Merge Risk: ⚪ Minimal · up to This localized change adds the publication discovery link when the required site configuration is present and leaves pages unchanged otherwise. No actionable merge-blocking risk remains beyond normal checks and review. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
Full details: Docstring CoverageExplanation No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.) ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
What
Adds the site-wide
<link rel="site.standard.publication">discovery hint to the package Layout, emitted whenSTANDARD_SITE_DIDandSTANDARD_SITE_PUBLICATION_RKEYare set (and omitted entirely otherwise).Why
Bluesky's AppView needs this tag on every page — home and episode pages alike — to render shared links as a verified publication. The package currently only emits the per-episode
site.standard.documenttag in[episode].astroand the/.well-known/site.standard.publicationendpoint.whiskey.fm carried this fix in its fork (www-starpod
846665e); converting that site to consumestarpodfrom npm dropped the tag, since the Layout has no override hook. This upstreams it.Verification
Built the reference site with test values set:
renders in the head of
/,/about, and episode pages; a build without the env vars emits nothing.🤖 Generated with Claude Code
Summary by CodeRabbit