Skip to content

feat: add Pi origin support to code review UI#263

Merged
backnotprop merged 2 commits intomainfrom
feat/pi-origin-support
Mar 10, 2026
Merged

feat: add Pi origin support to code review UI#263
backnotprop merged 2 commits intomainfrom
feat/pi-origin-support

Conversation

@backnotprop
Copy link
Owner

Summary

  • Add 'pi' to origin type definitions in review and annotate servers
  • Show violet "Pi" badge in code review header (matching plan editor's existing Pi support)
  • Update banner now copies pi install npm:@plannotator/pi-extension for Pi users instead of the curl install script
  • Completion overlay shows "Pi" instead of falling through to "OpenCode"

Test plan

  • Build: bun run build:review && bun run build:hook
  • Run plannotator review from Pi — verify violet "Pi" badge in header
  • Trigger update banner with ?preview-update=99.0.0&preview-origin=pi — verify correct install command is copied
  • Submit feedback from Pi code review — verify completion overlay says "Pi"

🤖 Generated with Claude Code

backnotprop and others added 2 commits March 9, 2026 22:56
Pi users were seeing "OpenCode" badge and getting the wrong update
command. Adds 'pi' to origin types in review/annotate servers, shows
violet "Pi" badge in code review header, and copies the correct
`pi install` command in the update banner.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
The install scripts now check if `pi` is on PATH and run
`pi install npm:@plannotator/pi-extension` to update the extension.
Silent for users without Pi installed.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@backnotprop
Copy link
Owner Author

Code review

No issues found. Checked for bugs and CLAUDE.md compliance.

🤖 Generated with Claude Code

- If this code review was useful, please react with 👍. Otherwise, react with 👎.

@backnotprop backnotprop merged commit b6bec69 into main Mar 10, 2026
3 checks passed
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