Repository navigation
chore(deps): update the plugin SDK for dropped shows and rating gates - #14
Conversation
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Advanced Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (1)
Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 WalkthroughWalkthroughThe required version of ChangesSDK dependency
Priority: ➖ Normal Estimated code review effort: 1 (Trivial) | ~3 minutes Change: Other Merge Risk: ⚪ Minimal · up to The SDK upgrade preserves the watch-provider fields through catalog ingestion, with no concrete merge-blocking issue identified. Build and test results remain to be confirmed by CI; the PR reports they pass. Architecture SummaryArchitecture risk: 🔵 Low · up to The change affects 1 system. Changed systems: Architecture concerns Review detailsSystems and components
Before / after behavior
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ 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 |
Problem
Related issue: N/A
Validation tasks: none
Used by: the first-party Trakt, Simkl, and MDBList watch-provider plugins (
Silo-Server/silo-plugin-watchprovider-trakt,-simkl,-mdblist), and Silo-Server/silo-plugin-sdk#33.The catalog decodes plugin manifests with the SDK it pins and discards fields it doesn't know. On v0.17.0 it would drop
sync_droppedandrating_export_requires_watchedfrom the new Trakt and Simkl releases, so their catalog entries would leave out dropped-show sync and Simkl's movie-rating gate.Approach
Update the SDK to v0.21.0, which adds those fields. This has to land before the catalog ingests the first watch-provider plugin releases.
Validation
go build ./...andgo test ./...pass.BuildPackageFromReleasekeepssync_dropped(Trakt, Simkl) and the movie rating gate (Simkl), and the results passmanifest.Validate.ValidateCatalogPresentationresult on v0.17.0 and v0.21.0. All pass exceptsilo.requests.arrandsilo.requests.seerr, which lack a presentation block on both versions.Risks
None expected: v0.18–v0.21 add contract fields only, and ingestion validates just the plugin being released.
Checklist
AI Disclosure
🤖 Generated with Claude Code
Note
Update
silo-plugin-sdkfrom v0.17.0 to v0.21.0Bumps
github.com/Silo-Server/silo-plugin-sdkto v0.21.0 in go.mod, with matching checksums in go.sum. The SDK update covers dropped shows and rating gates.Macroscope summarized 6bd77c6.