Skip to content

fix: restore Prairie module path and pin the Prairie SDK - #12

Merged
JonahMMay merged 1 commit into
mainfrom
fix/sdk-module-path
Oct 12, 2026
Merged

JonahMMay merged 1 commit into
mainfrom
fix/sdk-module-path

Conversation

@JonahMMay

@JonahMMay JonahMMay commented Oct 12, 2026 •

Copy link
Copy Markdown

Why main is red

The Silo sync (8ad664d) rebranded upstream's go.mod instead of keeping Prairie's:

  • It declared module github.com/Prairie-Server/prairie-plugins, but cmd/update-catalog imports github.com/prairie-server/prairie-plugins/catalog.
  • It required github.com/Prairie-Server/prairie-plugin-sdk v0.21.0. That tag doesn't exist on the Prairie SDK fork, and the casing is wrong. CI failed with missing go.sum entry.

Fix

  • The module path goes back to the pre-sync github.com/prairie-server/prairie-plugins. This matches the code's imports and .golangci.yml local-prefixes.
  • The SDK is pinned to Prairie SDK main: github.com/prairie-server/prairie-plugin-sdk v0.12.1-0.20261012211536-3ea8c11a9099 (3ea8c11).
  • go mod tidy picks up the SDK's newer grpc, x/net and x/sys requirements.

No Go source changes were needed.

Verification (golang 1.26.9, GOWORK=off)

  • go test ./..., go vet ./... and go build ./... pass. This is the full current CI.
  • golangci-lint run ./... (v2.14.0) reports 0 issues.
  • Coverage is 98.2% (floor 95%).
  • python3 -m json.tool manifest.json and python3 scripts/validate-manifest.py manifest.json report "17 plugin(s) OK".

Follow-ups (not in this PR)

  • The sync replaced Prairie's ci.yml (manifest validation, lint, coverage) with Silo's minimal workflow.
  • manifest.json again lists the prairie-plugins-requests-arr and -seerr entries that fbc270c dropped as unpublished. Those repos don't exist.

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Chores
    • Updated internal project configuration and supporting components. These changes do not add, remove, or alter any user-facing features or behavior. There are no new workflows, interface changes, or other end-user actions associated with this update.

The Silo sync (8ad664d) rebranded upstream's go.mod instead of keeping
Prairie's: it declared module github.com/Prairie-Server/prairie-plugins
(the code imports github.com/prairie-server/prairie-plugins/catalog)
and required github.com/Prairie-Server/prairie-plugin-sdk v0.21.0, a
tag that does not exist on the Prairie SDK fork, so CI failed with a
missing go.sum entry.

Restore the pre-sync module path github.com/prairie-server/
prairie-plugins and pin the SDK to Prairie SDK main (3ea8c11); go mod
tidy picks up the SDK's newer grpc/x/net requirements.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Oct 12, 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: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 04e09ae2-43b2-4743-aafb-eb8ce42991e0

📥 Commits

Reviewing files that changed from the base of the PR and between 15bec0f and 52df064.


⛔ 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

Walkthrough

The module path in go.mod changed to lowercase. The plugin SDK and several indirect dependency versions changed. The Go version and protobuf requirement remain unchanged.

Changes

Module and dependency updates

Layer / File(s) Summary
Module path and dependency versions
go.mod
The module path changed to prairie-server. The plugin SDK and five indirect dependencies now use different versions. The Go version and protobuf requirement are unchanged.

Priority: ➖ Normal

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

Change: Bug fix

Suggested reviewers: quick104


Merge Risk: ⚪ Minimal · up to 52df0

The module path aligns with the repository’s imports, and the pinned SDK retains the catalog interfaces in use. No concrete build or catalog compatibility failure is established, so no merge-blocking risk is identified.

Pre-merge checks · Passed 5
Passed 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 identifies the two main changes: restoring the Prairie module path and pinning the Prairie SDK.
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.

✨ 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.

@JonahMMay
JonahMMay merged commit 470e461 into main Oct 12, 2026
2 checks passed
@JonahMMay
JonahMMay deleted the fix/sdk-module-path branch October 12, 2026 22:21
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