Skip to content

chore: remove the Trakt watch-provider plugin from the catalog - #18

Merged
Quick104 merged 1 commit into
mainfrom
chore/retire-trakt
Oct 16, 2026
Merged

Quick104 merged 1 commit into
mainfrom
chore/retire-trakt

Conversation

@Quick104

@Quick104 Quick104 commented Oct 16, 2026 •

Copy link
Copy Markdown
Contributor

Problem

Silo is dropping Trakt support (Silo-Server/silo-server#2586), and Silo-Server/silo-plugin-watchprovider-trakt is archived. The catalog still offers silo.watchprovider.trakt v0.2.0, so admins could keep installing a plugin the server will no longer register.

Approach

Remove the silo.watchprovider.trakt entry from manifest.json; no other entry changes. Servers skip installed plugins that the catalog no longer lists, so existing installs see no update errors. The server change disables any installed copy on update. The archived repository's release assets stay downloadable.

Validation

  • go test ./..., go vet ./..., go build ./...: pass; gofmt -l . prints nothing.
  • manifest.json parses and lists 16 plugins; the diff removes only the Trakt entry.

AI Disclosure

  • Harness: Claude Code in T3 Code
  • Tool(s): Claude Code, GitHub CLI
  • Model(s): claude-opus-5-5
  • Involvement: Fully AI-generated, human verified
  • Adversarial review: n/a

🤖 Generated with Claude Code

Note

Remove the Trakt watch-provider plugin silo.watchprovider.trakt from the catalog

Deletes the manifest entry for plugin silo.watchprovider.trakt version 0.2.0 in manifest.json. This includes its platforms, watch-sync capability, config schema, metadata, repository URLs, and binary download mappings. Risk: manifest consumers no longer get an entry to locate or install the Trakt plugin binaries.

Macroscope summarized 4cc41d5.

Silo no longer supports Trakt (Silo-Server/silo-server#2586), and the plugin
repository is archived.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 16, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-16T14:11:15.233566Z 4cc41d5 PR opened
🔒 Security Review ✅ Completed 2026-10-16T14:12:02.053080Z 4cc41d5 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@silo-triage silo-triage Bot added the size/M 50–199 added + deleted lines (all files); change volume, not priority. label Oct 16, 2026
@coderabbitai

coderabbitai Bot commented Oct 16, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: f2b504ba-69d7-4abe-be72-b11b17ffa9c9
📥 Commits

Reviewing files that changed from the base of the PR and between 3c63376 and 4cc41d5.

📒 Files selected for processing: 1
  • manifest.json
💤 Files with no reviewable changes: 1
  • manifest.json

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

📝 Walkthrough

The manifest no longer includes the silo.watchprovider.trakt entry, its watch-sync capabilities, Trakt app configuration, presentation metadata, or platform release URLs.

Changes: Trakt provider removal

Layer / File(s) Summary
Remove Trakt manifest entry
manifest.json
Removed the silo.watchprovider.trakt declaration, including its capabilities, configuration, metadata, and release URLs.

Priority: ⬇️ Low

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

Change: Other

Merge Risk: ⚪ Minimal · up to 4cc41

No merge-blocking issue is identified with the intended Trakt catalog removal.

Pre-merge checks: Passed 5
Check name Status Explanation
Description check Passed The description clearly explains why the Trakt plugin catalog entry is removed, the implementation, and the validation performed.
Title check Passed The title clearly and concisely identifies the removal of the Trakt watch-provider plugin from the catalog.
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.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

@silo-triage silo-triage Bot added the triaged High-confidence review completed; does not mean fixed, validated, or accepted for release. label Oct 16, 2026
@Quick104
Quick104 merged commit bda4ee0 into main Oct 16, 2026
4 checks passed
@Quick104
Quick104 deleted the chore/retire-trakt branch October 16, 2026 14:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/M 50–199 added + deleted lines (all files); change volume, not priority. triaged High-confidence review completed; does not mean fixed, validated, or accepted for release.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant