fix: use the running binary in agent hooks

[Pimpmuckl]

What

Agent hooks that call the Herdr CLI now use HERDR_BIN_PATH, with herdr on PATH as a fallback.

This fixes the CLI callback for:

• Claude Code
• Codex
• Kimi
• GitHub Copilot CLI
• Droid
• Qoder CLI on Windows and Unix
• Antigravity CLI

The other supported integrations already use HERDR_BIN_PATH or report through the socket directly.

Why

I hit this while dogfooding a current master build on Windows. The running server used the current protocol, but the installed herdr on PATH was older. Codex started normally, but its session hook called the older client and the report failed silently.

That meant Herdr could detect the agent, but never saved its thread ID. After a reboot, panes with older saved IDs resumed correctly while newer panes came back as shells.

Herdr already gives every pane the exact running binary path. The hooks now use it consistently, so their client always matches the server.

Testing

• just check
• PowerShell and shell syntax checks
• direct Codex hook callback with an injected HERDR_BIN_PATH

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 29d066ad-9fd0-422f-9146-a41c9b7ad4f4

📥 Commits

Reviewing files that changed from the base of the PR and between ceb2376 and d9695e4.

📒 Files selected for processing (1)

• src/integration/tests.rs

---

📝 Walkthrough

Walkthrough

The integration version markers were incremented for seven integrations. PowerShell agent-state scripts now resolve Herdr from HERDR_BIN_PATH, with herdr as the fallback. The Qoder shell wrapper uses the same resolution path.

Changes

Herdr integration updates

Layer / File(s)	Summary
Integration version contract src/integration/mod.rs, src/integration/assets/*/herdr-agent-state.*, src/integration/tests.rs	Integration version constants and asset markers were incremented. Claude and Codex status tests now expect the updated versions. The asset consistency test now includes Antigravity CLI.
Configurable executable resolution src/integration/assets/*/herdr-agent-state.ps1, src/integration/assets/qodercli/herdr-agent-state.sh	Reporting commands resolve Herdr from HERDR_BIN_PATH and fall back to herdr. The Qoder shell wrapper no longer performs its separate Herdr availability check.

Estimated code review effort: 2 (Simple) | ~10 minutes

Possibly related PRs

• herdrdev/herdr#2087: Updates Antigravity CLI integration scripts and version constants.
• herdrdev/herdr#2496: Updates PowerShell integration hooks with HERDR_BIN_PATH resolution.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly summarizes the main change: agent hooks now use the running Herdr binary.
Description check	✅ Passed	The description accurately explains the HERDR_BIN_PATH change, affected integrations, rationale, and testing.
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

_{Comment @coderabbitai help to get the list of available commands.}

[coderabbitai]

🧹 Nitpick comments (1)

src/integration/mod.rs (1)

230-230: 🗄️ Data Integrity & Integration | 🔵 Trivial | ⚡ Quick win

Add Antigravity CLI to the asset-version invariant.

ANTIGRAVITY_CLI_INTEGRATION_VERSION changes here, but bundled_integration_asset_versions_match_expected_versions in src/integration/tests.rs omits the Antigravity asset. Add the asset and expected-version pair so a marker/constant mismatch fails the contract test instead of going unnoticed.

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 860cf89c-b2c7-4847-9677-47f2bb3f6fa5

📥 Commits

Reviewing files that changed from the base of the PR and between 5600197 and ceb2376.

📒 Files selected for processing (16)

• src/integration/assets/antigravity_cli/herdr-agent-state.ps1
• src/integration/assets/antigravity_cli/herdr-agent-state.sh
• src/integration/assets/claude/herdr-agent-state.ps1
• src/integration/assets/claude/herdr-agent-state.sh
• src/integration/assets/codex/herdr-agent-state.ps1
• src/integration/assets/codex/herdr-agent-state.sh
• src/integration/assets/copilot/herdr-agent-state.ps1
• src/integration/assets/copilot/herdr-agent-state.sh
• src/integration/assets/droid/herdr-agent-state.ps1
• src/integration/assets/droid/herdr-agent-state.sh
• src/integration/assets/kimi/herdr-agent-state.ps1
• src/integration/assets/kimi/herdr-agent-state.sh
• src/integration/assets/qodercli/herdr-agent-state.ps1
• src/integration/assets/qodercli/herdr-agent-state.sh
• src/integration/mod.rs
• src/integration/tests.rs

[greptile-apps]

Greptile Summary

The PR makes agent hooks invoke the exact Herdr executable that created the pane, while retaining herdr on PATH as a fallback.

• Updates the affected PowerShell hooks and Qoder CLI’s Unix hook to consume HERDR_BIN_PATH.
• Bumps integration asset versions so existing installations are detected as outdated and can be refreshed.
• Extends bundled asset-version consistency coverage to Antigravity CLI.

Confidence Score: 5/5

The PR appears safe to merge; the executable-selection changes preserve fallback and error behavior while keeping asset versions synchronized.

The pane environment supplies the running executable through HERDR_BIN_PATH, restored panes receive a freshly derived value, the new invocation forms safely preserve complete paths, and the affected version markers match their expected constants.

Important Files Changed

Filename	Overview
src/integration/mod.rs	Bumps expected integration versions consistently with the changed bundled hook assets.
src/integration/assets/qodercli/herdr-agent-state.sh	Selects HERDR_BIN_PATH inside a safely constructed subprocess argument list and preserves silent failure handling.
src/integration/assets/codex/herdr-agent-state.ps1	Invokes the pane-provided executable path through PowerShell’s call operator, with PATH fallback and existing error suppression.
src/integration/tests.rs	Updates expected integration versions and adds Antigravity CLI to bundled asset-marker consistency checks.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart LR
  Server[Running Herdr binary] -->|sets HERDR_BIN_PATH| Pane[Managed pane environment]
  Pane --> Agent[Agent CLI hook]
  Agent -->|invoke HERDR_BIN_PATH| Callback[Herdr pane report command]
  Callback --> Session[Persist agent session state]
  Agent -. variable absent .-> Fallback[herdr on PATH]
  Fallback --> Callback

Loading

_{Reviews (1): Last reviewed commit: "fix: use the running binary in agent hoo..." | Re-trigger Greptile}

[ogulcancelik]

hmm, i really need to work on agent registry v1. we have so much problems on what an agent means inside herdr, how to update or add new ones and now integrations share same versions across operating systems which is bad :/