feat: add Pi coding agent integration guide#2286
Conversation
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
|
Note Reviews pausedIt looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the Use the following commands to manage reviews:
Use the checkboxes below for quick actions:
WalkthroughAdds Pi terminal agent integration: exports a new ChangesPi Integration Setup
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~25 minutes Possibly related PRs
Suggested reviewers
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Tip 💬 Introducing Slack Agent: The best way for teams to turn conversations into code.Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.
Built for teams:
One agent for your entire SDLC. Right inside Slack. 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 |
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@apps/docs/content/guides/pi.mdx`:
- Around line 124-139: The example in the "Using Environment Variables for the
API Key" block uses "apiKey": "LLM_GATEWAY_API_KEY" which may be interpreted as
a literal string by Pi; update the example in apps/docs/content/guides/pi.mdx so
the "apiKey" field uses the exact environment-variable reference syntax Pi
expects (replace the bare token with the correct form such as $VAR or ${VAR}
only if Pi requires that), and add a short parenthetical note citing Pi's
official docs (link or exact quoted syntax) that confirms the required format
for ~/.pi/agent/models.json; ensure you modify the JSON snippet around the
"apiKey" property and mention the Pi docs reference for verification.
In `@pnpm-workspace.yaml`:
- Around line 6-15: Remove the deprecated pnpm build-config keys by deleting the
ignoredBuiltDependencies and onlyBuiltDependencies entries and rely solely on
the allowBuilds mapping; ensure the allowBuilds object (keys like core-js,
esbuild, protobufjs, sharp, unrs-resolver) contains every package you intend to
permit building and remove any arrays or keys named ignoredBuiltDependencies or
onlyBuiltDependencies from the file so build policy is controlled only via
allowBuilds.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Repository UI
Review profile: CHILL
Plan: Pro
Run ID: b81760e4-8654-40ae-8ba0-665afa768d4b
📒 Files selected for processing (3)
apps/docs/content/guides/pi.mdxapps/ui/src/content/guides/pi.mdpnpm-workspace.yaml
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Use only allowBuilds (pnpm v11+), remove ignoredBuiltDependencies and onlyBuiltDependencies which are no longer supported. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Summary
Guide covers:
~/.pi/agent/models.jsonwith LLM Gateway as providerTest plan
api.llmgateway.io— GPT-5 and Claude Sonnet 4.5 both respond correctly/guides/pion both docs and UI🤖 Generated with Claude Code
Summary by CodeRabbit
New Features
UI
Documentation
Chores