Add Pi support for MCP, plugins, and subagents - #179
Conversation
There was a problem hiding this comment.
Sorry @yourconscience, you've used your own review budget of 250,000 diff characters for the last 7 days.
You can request another review in 17 hours and 49 minutes by commenting @sourcery-ai review. Upgrade to get a review now.
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Advanced Run ID: 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 |
Reviewer's GuidePi is upgraded from skills-only support to a managed integration: canonical roles are rendered for pi-subagents, MCP and Agent Plugin servers are synchronized through pi-mcp-adapter, and root instructions are linked into Pi’s agent directory, with tests and documentation updated while OMP remains separate. Sequence diagram for syncing Pi roles and MCP serverssequenceDiagram
participant User
participant Dotagents
participant PiAgents as Pi user agents
participant PiConfig as Pi mcp.json
participant Subagents as pi-subagents
participant Adapter as pi-mcp-adapter
User->>Dotagents: sync
Dotagents->>PiAgents: renderPiAgentRole
Dotagents->>PiConfig: patchJSONMCPServer
PiAgents-->>Subagents: load rendered roles
PiConfig-->>Adapter: load managed MCP entries
Subagents-->>User: subagent commands available
Adapter-->>User: MCP and Agent Plugin servers available
File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: bffa130f78
ℹ️ 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".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| {Name: agentOMP, Enabled: true, SkillRoot: "~/.omp/agent/skills", AgentRoot: "~/.omp/agent/agents", Detect: "omp"}, | ||
| {Name: agentOpenCode, Enabled: true, SkillRoot: "~/.config/opencode/skills", AgentRoot: "~/.config/opencode/agents", Detect: "opencode"}, | ||
| {Name: agentPi, Enabled: true, SkillRoot: "~/.pi/agent/skills", Detect: "pi"}, | ||
| {Name: agentPi, Enabled: true, SkillRoot: "~/.pi/agent/skills", AgentRoot: "~/.pi/agent/agents", Detect: "pi"}, |
There was a problem hiding this comment.
Reconcile imported Pi roles before first sync
When setup detects Pi with an existing third-party pi-subagents role, this newly configured agent root makes scanNativeRoles offer the role for import. After the user accepts, the canonical copy is created but the source remains unchanged; the first runSync renders a version containing the dotagents marker, classifies the original source as a non-managed conflict, and aborts setup. Accepted Pi role imports therefore cannot complete their required first sync and need to be recognized or otherwise reconciled without modifying the source.
AGENTS.md reference: AGENTS.md:L17-L18
Useful? React with 👍 / 👎.
What changed
pi-subagentspi-mcp-adapterVerification
go test ./...Summary by Sourcery
Enable managed Pi roles, MCP, Agent Plugins, and root instructions while preserving OMP as a separate harness target.
New Features:
Enhancements:
Documentation:
Tests: