feat(pi): add Sentry package distribution - #304
Conversation
![sentry-warden[bot]](https://avatars.githubusercontent.com/in/2755838?s=60&v=4){kind=small}
![sentry[bot]](https://avatars.githubusercontent.com/in/12637?s=60&v=4){kind=small}
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
Co-Authored-By: GPT-5.6 <noreply@openai.com>
Co-Authored-By: GPT-5.6 <noreply@openai.com>
Co-Authored-By: GPT-5.6 <noreply@openai.com>
319b761 to
2995592
Compare
Capture CLI output once before grepping to avoid pipe-related Codex panics on macOS. Co-Authored-By: GPT-5.6 <noreply@openai.com>
| detect: async () => detectOnPath(system, "pi"), | ||
|
|
||
| isInstalled: async () => { | ||
| const result = await system.run("pi list --no-approve"); |
There was a problem hiding this comment.
There's no json output for this is there?
There was a problem hiding this comment.
Correct — verified against the pinned pi 0.83.0: pi list only accepts --approve/--no-approve, and pi list --json errors with "Unknown option". String matching on the output is the only option for now. The matcher keys on the package source line pi list prints (git:github.com/getsentry/plugin-pi), which is stable across user/project sections.
— Claude Code
Resolves conflicts against the harness build-on-demand refactor (#321), the develop/main deploy split (#317), and the flowmark reformat (#315/#316): - harnesses/index.ts: register the Pi harness inside buildHarnesses() and drop its module-level constant, matching the single-construction-path refactor - deploy-plugins.yml: keep this branch's build/validate -> artifact -> deploy job split (no third-party code runs while the deploy token is in scope) while adopting main's develop/main target branches, dev-version stamping, and release tagging - AGENTS.md / READMEs: fold the Pi mentions into main's reflowed prose and drop the root .mcp.json compat note main removed Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
|
🚧 Skipped: PR exceeds review size limit. Please split into smaller PRs and re-run. |
|
🚧 Skipped: PR exceeds review size limit. Please split into smaller PRs and re-run. |
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 68ebacf. Configure here.
The pnpm workspace hoists packages into a root node_modules/, which the root .gitignore never covered (only packages/installer/ ignores its own), so the merge commit accidentally swept it into the tree. Untrack it and add the ignore so it cannot recur. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
2 participants
Adds Pi as a first-class Sentry distribution alongside Claude Code, Cursor, Codex, and Grok. The new native Pi package ships the hydrated skill tree unchanged, adapts the hosted Sentry MCP server behind a namespaced
sentry_mcpgateway, and coexists with users who already install the genericpi-mcp-adapterpackage.The
@sentry/aiinstaller now detects Pi and manages the package through Pi’s native install, update, and remove commands. Deployment and cross-platform installer smoke workflows include Pi, with package validation covering skill hydration, package contents, extension loading, and MCP-adapter coexistence.The target distribution repository has been created at https://github.com/getsentry/plugin-pi. Before merging, the
PLUGIN_DEPLOY_APP_IDGitHub App must be granted access to that repository so the deploy matrix can push generated content.