Repository navigation
Merge latest upstream starpod changes - #29
Merged
Merged
Conversation
…e (#58)
* Only validate feed fields the publish script uses
Flightcast now emits a second enclosure per episode (the cover image)
with no type attribute, which the copied-from-rss.ts schema required —
so every publish run died in FeedSchema parse. Trim the schema to the
fields the script reads (title, published, description, content_encoded,
itunes_episodeType) so irrelevant feed changes can't break publishing.
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
* Rename publish script to publish-atproto-episodes; rebuild once per feed change
The script only publishes standard.site documents to ATProto — episode
pages come from the Astro build — so name it accordingly (pnpm scripts
are now publish:atproto / publish:atproto:backfill).
The gate also now records which feed content it already triggered a
Vercel rebuild for, separately from the publish-success hash. A
persistently failing publish (like the recent FeedSchema crash) keeps
retrying every poll but no longer fires a fresh build each time; the
publish script still waits on the pages from the one build it did
trigger.
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
* Restore relative dasherize import
An in-progress workspace import path ('starpod/src/utils/dasherize')
was accidentally captured during the rename; main still has src/ at the
repo root, so astro check failed with ts(2307).
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
---------
Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Contributor
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (5)
📝 WalkthroughWalkthroughThe PR switches episode publishing commands and workflows to the AtProto publisher, narrows feed validation to used fields, and adds rebuild-hash tracking around site rebuild webhooks. ChangesAtProto publishing
Estimated code review effort: 3 (Moderate) | ~20 minutes Sequence Diagram(s)sequenceDiagram
participant PublishWorkflow as publish-episodes workflow
participant RebuildHash as rebuild hash state
participant SiteWebhook as site rebuild webhook
PublishWorkflow->>RebuildHash: restore recorded rebuild hash
PublishWorkflow->>SiteWebhook: trigger rebuild with current feed hash
SiteWebhook-->>PublishWorkflow: return webhook result
PublishWorkflow->>RebuildHash: save hash after successful trigger
Suggested reviewers: ✨ Finishing Touches📝 Generate docstrings
🧪 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 |
This branch was successfully deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Merges the latest commit from upstream starpod:
The merge was clean — no conflicts. Upstream renamed
scripts/publish-episodes.tstoscripts/publish-atproto-episodes.tsand git carried this fork'sSTANDARD_SITE_URLhandling through the rename; verified the merged script, package.json scripts, and workflows are consistent.Verification
pnpm build(astro check + astro build + md-negotiation) passespnpm lintpassespnpm test:unit— 156/156 tests pass🤖 Generated with Claude Code
Summary by CodeRabbit
New Features
Bug Fixes
Documentation