Skip to content

Emit the site.standard.publication link tag on every page - #62

Open
RobbieTheWagner wants to merge 1 commit into
mainfrom
fix-standard-site-publication-tag
Open

Emit the site.standard.publication link tag on every page#62
RobbieTheWagner wants to merge 1 commit into
mainfrom
fix-standard-site-publication-tag

Conversation

@RobbieTheWagner

@RobbieTheWagner RobbieTheWagner commented Aug 30, 2026

Copy link
Copy Markdown
Member

What

Adds the site-wide <link rel="site.standard.publication"> discovery hint to the package Layout, emitted when STANDARD_SITE_DID and STANDARD_SITE_PUBLICATION_RKEY are 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.document tag in [episode].astro and the /.well-known/site.standard.publication endpoint.

whiskey.fm carried this fix in its fork (www-starpod 846665e); converting that site to consume starpod from npm dropped the tag, since the Layout has no override hook. This upstreams it.

Verification

Built the reference site with test values set:

<link rel="site.standard.publication" href="at://did:plc:test123/site.standard.publication/testrkey">

renders in the head of /, /about, and episode pages; a build without the env vars emits nothing.

🤖 Generated with Claude Code

Summary by CodeRabbit

  • New Features
    • Added support for linking configured standard-site publications in the page metadata.
    • The link is included automatically when the required publication settings are available.

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>
@vercel

vercel Bot commented Aug 30, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
starpod Ready Ready Preview Aug 30, 2026 12:28am

Request Review

@coderabbitai

coderabbitai Bot commented Aug 30, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 1fa9bdba-9c67-4a72-9807-e4061154da62

📥 Commits

Reviewing files that changed from the base of the PR and between 3d9a168 and d9285a9.

📒 Files selected for processing (1)
  • packages/starpod/src/layouts/Layout.astro

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.


📝 Walkthrough

Walkthrough

Layout.astro now generates a standard publication URI from environment variables and emits it as a document-head link when both values are present.

Changes

Publication link

Layer / File(s) Summary
Build and render publication link
packages/starpod/src/layouts/Layout.astro
The layout imports getPublicationAtUri, reads STANDARD_SITE_DID and STANDARD_SITE_PUBLICATION_RKEY, builds the publication link when both values exist, and renders it in the document head.

Estimated code review effort: 2 (Simple) | ~10 minutes

Merge Risk: ⚪ Minimal · up to d9285

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)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the main change: emitting the site.standard.publication link tag on every page.
Docstring Coverage ✅ Passed 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…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Full details: Docstring Coverage

Explanation

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)
  • Create PR with unit tests
  • Commit unit tests in branch fix-standard-site-publication-tag

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

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