Skip to content

chore(deps): bump silo-plugin-sdk to v0.27.0 - #19

Merged
Quick104 merged 1 commit into
mainfrom
chore/sdk-v0.27-watch-sync-batch
Oct 17, 2026
Merged

Quick104 merged 1 commit into
mainfrom
chore/sdk-v0.27-watch-sync-batch

Conversation

@Quick104

@Quick104 Quick104 commented Oct 17, 2026 •

Copy link
Copy Markdown
Contributor

Problem

Related issue: N/A

The catalog tool cannot add a watch-sync plugin release that declares max_batch_size above 100. update-catalog validates each release's manifest with the plugin SDK, and the pinned v0.21.0 rejects it with watch sync max_batch_size must be between 1 and 100. The MDBList watch-provider plugin is about to declare 1000.

Approach

Bump silo-plugin-sdk to v0.27.0 (Silo-Server/silo-plugin-sdk#44), which accepts 1 to 1000. The SDK's own requirements bring along newer grpc, genproto and x/text. No catalog entries change.

Automation impact: update-catalog and update-manifest run from main, so this needs to merge before the MDBList plugin release that declares 1000.

Validation

  • GOWORK=off go test ./..., go vet ./..., go build ./...: pass. gofmt -l .: clean.
  • With v0.27.0, BuildPackageFromRelease accepts the MDBList plugin's manifest with max_batch_size: 1000; with v0.21.0 it rejects it with the error above.

Evidence

Evidence: https://evidence.siloserver.org/r/silo-plugins/pr-19/

Risks

  • None for existing entries: every manifest that validated under v0.21.0 still validates.

Checklist

  • I read and can explain the complete diff.
  • This pull request addresses one concern.

AI Disclosure

  • Harness: Claude Code (T3 Code)
  • Tool(s): Claude Code, GitHub CLI
  • Model(s): claude-opus-5-5
  • Involvement: AI-assisted. The maintainer approved raising the batch size across the SDK, server, catalog and plugin.
  • Adversarial review: n/a. This is a dependency bump; the SDK change it picks up was reviewed in feat(watchsync): allow watch-sync batch sizes up to 1000 silo-plugin-sdk#44.

🤖 Generated with Claude Code

The catalog tool validates each plugin release with the SDK's manifest
validator. v0.27.0 accepts watch-sync max_batch_size up to 1000, so a
release that declares more than 100 can be added to the catalog.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@silo-triage silo-triage Bot added the size/M 50–199 added + deleted lines (all files); change volume, not priority. label Oct 17, 2026
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 17, 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-17T03:17:20.965704Z ce1de7c PR opened
🔒 Security Review ✅ Completed 2026-10-17T03:19:07.217118Z ce1de7c 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.

@coderabbitai

coderabbitai Bot commented Oct 17, 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: 20438de9-4cdd-41e6-8f02-a4e778b9ed44
📥 Commits

Reviewing files that changed from the base of the PR and between bda4ee0 and ce1de7c.

⛔ Files ignored due to path filters: 1
  • go.sum is excluded by !**/*.sum
📒 Files selected for processing: 1
  • go.mod

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 Go module upgrades the Silo plugin SDK from v0.21.0 to v0.27.0 and updates five indirect dependency requirements. The protobuf requirement remains unchanged.

Changes: Go dependency upgrades

Layer / File(s) Summary
Update module requirements
go.mod
The SDK requirement and indirect requirements for x/net, x/sys, x/text, genproto/googleapis/rpc, and grpc are upgraded. The protobuf requirement is unchanged.

Priority: ➖ Normal

Estimated code review effort: 1 (Trivial) | ~5 minutes

Change: Other

Merge Risk: ⚪ Minimal · up to ce1de

No actionable merge-blocking issue is identified; the dependency update is mergeable after normal checks.

Pre-merge checks: Passed 5
Check name Status Explanation
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.
Title check Passed The title clearly and concisely identifies the dependency update to silo-plugin-sdk v0.27.0, which is the main change.
Description check Passed The description directly explains the max_batch_size validation issue, the SDK upgrade, validation results, and expected impact.
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)
  • 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 dependencies Pull requests that update a dependency file go Pull requests that update go code triaged High-confidence review completed; does not mean fixed, validated, or accepted for release. labels Oct 17, 2026
@Quick104
Quick104 merged commit b3ed7e5 into main Oct 17, 2026
4 checks passed
@Quick104
Quick104 deleted the chore/sdk-v0.27-watch-sync-batch branch October 17, 2026 03:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file go Pull requests that update go code 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