feat(hooks): port the PreToolUse guard hooks to pi's tool_call contract #561

New issue

Open

opened 2026-09-21 20:46:48 +00:00 by hexajon · 0 comments

hexajon commented

2026-09-21 20:46:48 +00:00

Owner

Phase 2 of ADR 0039 (docs/decisions/0039-pi-first-class-harness.md). Refs #546.

ADR 0039 records the gap: the guard hooks that ship for Claude Code do not fire under pi, so a pi user runs on the server-side behavioural guards alone (ADR 0007, ADR 0009). Those guards stay the safety floor either way.

pi takes TypeScript hook factories at .omp/hooks/pre|post/*.ts, or the same under a plugin's hooks/ directory. Whether the existing bash scripts can serve that contract is unproven.

Blocked on #559: if omp imports the claude-plugins hooks as-is, this shrinks to documentation.

Acceptance criteria:

Each guard hook either fires under pi or is listed as not ported, with the reason.
The ported set has tests that prove a real blocked call is denied with a reason.
ADR 0039 is updated with the outcome.

Phase 2 of ADR 0039 (docs/decisions/0039-pi-first-class-harness.md). Refs #546. ADR 0039 records the gap: the guard hooks that ship for Claude Code do not fire under pi, so a pi user runs on the server-side behavioural guards alone (ADR 0007, ADR 0009). Those guards stay the safety floor either way. pi takes TypeScript hook factories at .omp/hooks/pre|post/*.ts, or the same under a plugin's hooks/ directory. Whether the existing bash scripts can serve that contract is unproven. Blocked on #559: if omp imports the claude-plugins hooks as-is, this shrinks to documentation. Acceptance criteria: - Each guard hook either fires under pi or is listed as not ported, with the reason. - The ported set has tests that prove a real blocked call is denied with a reason. - ADR 0039 is updated with the outcome.

hexajon added a new dependency

2026-09-21 20:46:53 +00:00

#559 spike(pi): check whether omp imports the existing claude-plugins hooks as-is

hexajon referenced this issue

2026-09-21 20:47:26 +00:00

docs(adr): track the ADR 0039 phase-2 follow-ups as issues #562

hexajon referenced this issue

2026-09-28 18:16:43 +00:00

decision: move the ADR tooling into its own Go module, or keep it in madtea and improve it? #643

hexajon referenced this issue

2026-09-29 15:53:47 +00:00

feat(serve): add pi (Oh My Pi) as a first-class MCP harness #546