diff options
| author | Ogulcan Celik <ogulcancelik@gmail.com> | 2026-05-16 17:29:37 +0300 |
|---|---|---|
| committer | Ogulcan Celik <ogulcancelik@gmail.com> | 2026-05-16 18:31:35 +0300 |
| commit | 170d5674872fe030777ceb97e18b38a0e9ddbc2f (patch) | |
| tree | fd7d19494902c5b3e93fee7e34b2e30b0569b23a /.pi | |
| parent | 72770305d388fd5ec247493e05a89b686c6d0026 (diff) | |
fix: respect integration config dirs
refs #121
Diffstat (limited to '.pi')
| -rw-r--r-- | .pi/docs/CHANGELOG.md | 1 | ||||
| -rw-r--r-- | .pi/docs/INTEGRATIONS.md | 29 |
2 files changed, 18 insertions, 12 deletions
diff --git a/.pi/docs/CHANGELOG.md b/.pi/docs/CHANGELOG.md index 6655806e..2af756a5 100644 --- a/.pi/docs/CHANGELOG.md +++ b/.pi/docs/CHANGELOG.md @@ -4,6 +4,7 @@ ### Fixed - GitHub Copilot is now correctly detected when its process name is `copilot`. +- Integration installs now respect `PI_CODING_AGENT_DIR`, `CLAUDE_CONFIG_DIR`, and `CODEX_HOME` when choosing Pi, Claude Code, and Codex config paths. - Split pane resize hit areas no longer overlap the first content column or row, making text selection work from the start of right and bottom panes. ## [0.5.9] - 2026-05-15 diff --git a/.pi/docs/INTEGRATIONS.md b/.pi/docs/INTEGRATIONS.md index ec4e4884..3905e834 100644 --- a/.pi/docs/INTEGRATIONS.md +++ b/.pi/docs/INTEGRATIONS.md @@ -55,6 +55,14 @@ this writes the bundled pi extension to: ~/.pi/agent/extensions/herdr-agent-state.ts ``` +if `PI_CODING_AGENT_DIR` is set, herdr uses that agent directory instead and writes to: + +```text +$PI_CODING_AGENT_DIR/extensions/herdr-agent-state.ts +``` + +`~` is expanded in `PI_CODING_AGENT_DIR`. + pi is the cleanest integration. it already has an authoritative hook model, so herdr can get direct state reports over the socket api without guessing as much from the terminal. bundled source: [`src/integration/assets/pi/herdr-agent-state.ts`](./src/integration/assets/pi/herdr-agent-state.ts) @@ -65,11 +73,7 @@ uninstall: herdr integration uninstall pi ``` -this removes: - -```text -~/.pi/agent/extensions/herdr-agent-state.ts -``` +this removes the same extension path herdr would install, using `PI_CODING_AGENT_DIR` when it is set. ## claude code @@ -84,6 +88,8 @@ this: - writes the hook script to `~/.claude/hooks/herdr-agent-state.sh` - updates `~/.claude/settings.json` +if `CLAUDE_CONFIG_DIR` is set, herdr uses that directory instead, for example `$CLAUDE_CONFIG_DIR/hooks/herdr-agent-state.sh` and `$CLAUDE_CONFIG_DIR/settings.json`. `~` is expanded in `CLAUDE_CONFIG_DIR`. + bundled source: [`src/integration/assets/claude/herdr-agent-state.sh`](./src/integration/assets/claude/herdr-agent-state.sh) current hook mapping: @@ -109,10 +115,7 @@ uninstall: herdr integration uninstall claude ``` -this: - -- removes `~/.claude/hooks/herdr-agent-state.sh` -- removes herdr-owned hook entries from `~/.claude/settings.json` +this removes the same hook path and settings entries herdr would install, using `CLAUDE_CONFIG_DIR` when it is set. ## codex @@ -129,6 +132,8 @@ this: - ensures `hooks = true` under `[features]` in `~/.codex/config.toml` - migrates the deprecated top-level `[features] codex_hooks = true` setting to `hooks = true` +if `CODEX_HOME` is set, herdr uses that directory instead, for example `$CODEX_HOME/herdr-agent-state.sh`, `$CODEX_HOME/hooks.json`, and `$CODEX_HOME/config.toml`. `~` is expanded in `CODEX_HOME`. + bundled source: [`src/integration/assets/codex/herdr-agent-state.sh`](./src/integration/assets/codex/herdr-agent-state.sh) current hook mapping: @@ -153,9 +158,9 @@ herdr integration uninstall codex this: -- removes `~/.codex/herdr-agent-state.sh` -- removes herdr-owned hook entries from `~/.codex/hooks.json` -- intentionally leaves `~/.codex/config.toml` alone +- removes the same hook path herdr would install, using `CODEX_HOME` when it is set +- removes herdr-owned hook entries from the matching `hooks.json` +- intentionally leaves the matching `config.toml` alone that last point is deliberate: herdr does **not** try to guess whether `hooks = true` is still needed for some other codex hook setup. |
