Skip to content

feat(pi): add agent integration#812

Open
YHM404 wants to merge 1 commit into
rtk-ai:developfrom
YHM404:feat/pi-agent-integration
Open

feat(pi): add agent integration#812
YHM404 wants to merge 1 commit into
rtk-ai:developfrom
YHM404:feat/pi-agent-integration

Conversation

@YHM404

@YHM404 YHM404 commented Mar 25, 2026

Copy link
Copy Markdown

Summary

  • add a thin pi extension that overrides pi's bash tool and delegates to rtk rewrite
  • add rtk init -g --agent pi install, uninstall, and config-status support
  • document pi in the supported tools list and installation examples

Test plan

  • cargo fmt --all --check && cargo clippy --all-targets && cargo test
  • Manual testing: installed the extension in pi, ran /reload, and verified git status was rewritten to RTK output

Signed-off-by: yhm404 <yaohuiming789@gmail.com>
@YHM404 YHM404 mentioned this pull request Mar 25, 2026
@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@aeppling

Copy link
Copy Markdown
Contributor

Hey

We are cleaning up the codebase and improving the project structure for better onboarding. As part of this effort, PR #826 reorganizes src/ from a flat layout into subfolders.

No logic changes — only file moves and import path updates.

What you need to do

Rebase your branch on develop when receiving this comment:

git fetch origin && git rebase origin/develop

Git 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

@snitramodranoel

Copy link
Copy Markdown

Should be closed in favor of #1741

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.

5 participants