feat: add Pi agent support for hook installation#750
Conversation
- Add 'pi' to AgentTarget enum - Create pi-rtk-rewrite.sh hook script similar to Cursor - Add installation logic in init.rs for Pi hooks (~/.omp/hooks/) - Add --agent pi flag support to CLI - Update uninstall to support Pi hooks
|
Philippe seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account. You have signed the CLA already but the status is still pending? Let us recheck it. |
|
Hi! Two things needed before we can review:
Thanks! |
|
Hey We are cleaning up the codebase and improving the project structure for better onboarding. As part of this effort, PR #826 reorganizes No logic changes — only file moves and import path updates. What you need to doRebase your branch on git fetch origin && git rebase origin/developGit detects renames automatically. If you get import conflicts, update the paths: use crate::git; // now: use crate::cmds::git::git;
use crate::tracking; // now: use crate::core::tracking;
use crate::config; // now: use crate::core::config;
use crate::init; // now: use crate::hooks::init;
use crate::gain; // now: use crate::analytics::gain;Need help rebasing? Tag @aeppling |
|
Thanks @zogzog26 for the early Pi support! Pi integration has since been picked up in #1741, which takes the modern approach (a TypeScript extension delegating to Closing this as superseded by #1741 — there's been no activity here for ~8 weeks. Thanks again, and feel free to chime in on #1741. |
Summary
This PR adds support for the Pi agent (oh-my-pi / omp) in RTK, following the same pattern as the existing Cursor agent support.
Changes
Usage
The hook will be installed to where Pi should automatically detect it.
Testing