Remove Claude hook integration from ward release scope #733

Closed
opened 2026-07-09 02:25:12 +00:00 by coilyco-ops · 7 comments
Member

Remove the Claude hook integration from ward's pre-release surface:

  • Remove the ward hook command group and Claude pre-tool-use hook implementation.
  • Remove related tests and documentation.
  • Clean up any stale hook-era references, including old command spellings.
  • Keep agent dispatch and container behavior unchanged.

Rationale: the hook is an integration sidecar, not core ward release behavior.

Remove the Claude hook integration from ward's pre-release surface: * Remove the `ward hook` command group and Claude pre-tool-use hook implementation. * Remove related tests and documentation. * Clean up any stale hook-era references, including old command spellings. * Keep agent dispatch and container behavior unchanged. Rationale: the hook is an integration sidecar, not core ward release behavior.
Author
Member

🔒 Reserved by ward agent --harness codex — container engineer-codex-ward-733 on host kais-macbook-pro-2.local is carrying this issue (reserved 2026-07-09T02:26:16Z). Concurrent ward agent runs are blocked until it finishes or the reservation goes stale (1h TTL); --force overrides.

Do not comment on or edit this issue to steer the run while it is reserved. The engineer seeded the body once at launch and never re-reads it, so a comment or edit reaches only human readers, never the running engineer. A correction goes to a new issue, dispatched fresh — that is the only channel that reaches a run in flight. Where the forge supports it, ward locks this conversation to make that a road-block rather than a convention (ward#494).

run seed context — what this run is carrying (ward#609)
  • Resolved: coilyco-flight-deck/ward#733 · branch issue-733 · harness codex · workflow pr
  • Run: engineer-codex-ward-733 · ward v0.470.0 · dispatched 2026-07-09T02:26:16Z
  • Comment thread: 0 included in the pre-flight read, 0 stripped (ward's own automated comments).

Static container doctrine and seed boilerplate are identical every run and omitted here (they ride ward v0.470.0).

— Codex, via ward agent

<!-- ward-agent-reservation --> 🔒 Reserved by `ward agent --harness codex` — container `engineer-codex-ward-733` on host `kais-macbook-pro-2.local` is carrying this issue (reserved 2026-07-09T02:26:16Z). Concurrent `ward agent` runs are blocked until it finishes or the reservation goes stale (1h TTL); `--force` overrides. **Do not comment on or edit this issue to steer the run while it is reserved.** The engineer seeded the body once at launch and never re-reads it, so a comment or edit reaches only human readers, never the running engineer. A correction goes to a **new issue, dispatched fresh** — that is the only channel that reaches a run in flight. Where the forge supports it, ward locks this conversation to make that a road-block rather than a convention (ward#494). <details><summary>run seed context — what this run is carrying (ward#609)</summary> - **Resolved:** `coilyco-flight-deck/ward#733` · branch `issue-733` · harness `codex` · workflow `pr` - **Run:** `engineer-codex-ward-733` · ward `v0.470.0` · dispatched `2026-07-09T02:26:16Z` - **Comment thread:** 0 included in the pre-flight read, 0 stripped (ward's own automated comments). Static container doctrine and seed boilerplate are identical every run and omitted here (they ride ward v0.470.0). </details> <!-- ward-agent-signature --> — Codex, via `ward agent`
Author
Member

WARD-OUTCOME: done - removed Claude hook surface from ward release scope
Review: skipped because --skip-preflight / --no-preflight also skips review.

The hook command, its tests, and the docs/demo references are gone. Agent dispatch and container behavior stayed unchanged. Build passed, pre-commit passed, and the only rough edge was two unrelated failures in the full repo test target.

WARD-OUTCOME: done - removed Claude hook surface from ward release scope Review: skipped because --skip-preflight / --no-preflight also skips review. The hook command, its tests, and the docs/demo references are gone. Agent dispatch and container behavior stayed unchanged. Build passed, pre-commit passed, and the only rough edge was two unrelated failures in the full repo test target.
Author
Member

⚠️ Reopened: this run's work did not land on main

An ephemeral ward container (codex mode) dispatched for this issue finished but its work was not merged to main, so the reaper preserved it on a branch before teardown and reopened the issue (a closing reference for #733 never reached main). Recover from the salvage branch below.

  • Repo: coilyco-flight-deck/ward
  • Salvage branch: ward-salvage/ward-ff6c509f
  • Pull request: not opened - PR creation failed: forgejo: parse created pull request: unexpected end of JSON input
  • Reason: run workflow does not land on main (--workflow pr/patch-only)
  • Container uptime at reap: 14m (age of the baked Forgejo PAT snapshot; a long-lived container is likelier to carry a rotated token)

Reap diagnostics

--- reap diagnostics ---
ward version:      v0.470.0
version source:    pinned via WARD_VERSION/--ward-version (v0.470.0)
HEAD:              f1c54a37492f
origin/main:       8cab230822ab
ancestry:          HEAD is NOT yet on origin/main - residual work remains to land
decision gate:     workflow does not land on main (--workflow pr/patch-only)
reason:            run workflow does not land on main (--workflow pr/patch-only)
provenance:        not read (workflow hold)
run-owned landed:  no
working tree:      clean
container uptime:  14m (baked Forgejo PAT age proxy)
--- end reap diagnostics ---

Recover

git fetch https://forgejo.coilysiren.me/coilyco-flight-deck/ward.git ward-salvage/ward-ff6c509f
git checkout -b ward-salvage/ward-ff6c509f FETCH_HEAD

— Codex, via ward agent

## ⚠️ Reopened: this run's work did not land on `main` An ephemeral `ward container` (codex mode) dispatched for this issue finished but its work was **not merged to `main`**, so the reaper preserved it on a branch before teardown and reopened the issue (a closing reference for #733 never reached `main`). Recover from the salvage branch below. - **Repo:** `coilyco-flight-deck/ward` - **Salvage branch:** `ward-salvage/ward-ff6c509f` - **Pull request:** not opened - PR creation failed: forgejo: parse created pull request: unexpected end of JSON input - **Reason:** run workflow does not land on main (--workflow pr/patch-only) - **Container uptime at reap:** 14m (age of the baked Forgejo PAT snapshot; a long-lived container is likelier to carry a rotated token) ## Reap diagnostics ``` --- reap diagnostics --- ward version: v0.470.0 version source: pinned via WARD_VERSION/--ward-version (v0.470.0) HEAD: f1c54a37492f origin/main: 8cab230822ab ancestry: HEAD is NOT yet on origin/main - residual work remains to land decision gate: workflow does not land on main (--workflow pr/patch-only) reason: run workflow does not land on main (--workflow pr/patch-only) provenance: not read (workflow hold) run-owned landed: no working tree: clean container uptime: 14m (baked Forgejo PAT age proxy) --- end reap diagnostics --- ``` ## Recover ```bash git fetch https://forgejo.coilysiren.me/coilyco-flight-deck/ward.git ward-salvage/ward-ff6c509f git checkout -b ward-salvage/ward-ff6c509f FETCH_HEAD ``` <!-- ward-agent-signature --> — Codex, via `ward agent`
Author
Member

Director PR sweep, 2026-07-09T07:26Z:

The stale PR(s) for this issue were closed after a full open-PR sweep showed they carried old branch history from before the recovery merges. Fresh run should start from current main and reapply only this issue: remove the Claude hook command group and pre-tool-use hook integration without changing agent dispatch/container behavior.

Director PR sweep, 2026-07-09T07:26Z: The stale PR(s) for this issue were closed after a full open-PR sweep showed they carried old branch history from before the recovery merges. Fresh run should start from current `main` and reapply only this issue: remove the Claude hook command group and pre-tool-use hook integration without changing agent dispatch/container behavior.
Author
Member

🔒 Reserved by ward agent --harness codex — container engineer-codex-ward-733 on host kais-macbook-pro-2.local is carrying this issue (reserved 2026-07-09T07:27:09Z). Concurrent ward agent runs are blocked until it finishes or the reservation goes stale (1h TTL); --force overrides.

Do not comment on or edit this issue to steer the run while it is reserved. The engineer seeded the body once at launch and never re-reads it, so a comment or edit reaches only human readers, never the running engineer. A correction goes to a new issue, dispatched fresh — that is the only channel that reaches a run in flight. Where the forge supports it, ward locks this conversation to make that a road-block rather than a convention (ward#494).

run seed context — what this run is carrying (ward#609)
  • Resolved: coilyco-flight-deck/ward#733 · branch issue-733 · harness codex · workflow pr
  • Run: engineer-codex-ward-733 · ward v0.474.0 · dispatched 2026-07-09T07:27:09Z
  • Comment thread: 3 included in the pre-flight read, 1 stripped (ward's own automated comments).

Static container doctrine and seed boilerplate are identical every run and omitted here (they ride ward v0.474.0).

— Codex, via ward agent

<!-- ward-agent-reservation --> 🔒 Reserved by `ward agent --harness codex` — container `engineer-codex-ward-733` on host `kais-macbook-pro-2.local` is carrying this issue (reserved 2026-07-09T07:27:09Z). Concurrent `ward agent` runs are blocked until it finishes or the reservation goes stale (1h TTL); `--force` overrides. **Do not comment on or edit this issue to steer the run while it is reserved.** The engineer seeded the body once at launch and never re-reads it, so a comment or edit reaches only human readers, never the running engineer. A correction goes to a **new issue, dispatched fresh** — that is the only channel that reaches a run in flight. Where the forge supports it, ward locks this conversation to make that a road-block rather than a convention (ward#494). <details><summary>run seed context — what this run is carrying (ward#609)</summary> - **Resolved:** `coilyco-flight-deck/ward#733` · branch `issue-733` · harness `codex` · workflow `pr` - **Run:** `engineer-codex-ward-733` · ward `v0.474.0` · dispatched `2026-07-09T07:27:09Z` - **Comment thread:** 3 included in the pre-flight read, 1 stripped (ward's own automated comments). - included: @coilyco-ops (2026-07-09T02:40:53Z), @coilyco-ops (2026-07-09T02:41:17Z), @coilyco-ops (2026-07-09T07:27:07Z) - stripped: @coilyco-ops (2026-07-09T02:26:18Z) Static container doctrine and seed boilerplate are identical every run and omitted here (they ride ward v0.474.0). </details> <!-- ward-agent-signature --> — Codex, via `ward agent`
Author
Member

WARD-RESERVATION: held 🔒

reservation details

Holder: container engineer-codex-ward-733 on host kais-macbook-pro-2.local.

Reserved by ward agent --harness codex (reserved 2026-07-09T15:54:51Z). Concurrent ward agent runs are blocked until it finishes or the reservation goes stale (1h TTL). --force overrides.

Do not comment on or edit this issue to steer the run while it is reserved. The engineer seeded the body once at launch and never re-reads it, so a comment or edit reaches only human readers, never the running engineer. A correction goes to a new issue, dispatched fresh. That is the only channel that reaches a run in flight. Where the forge supports it, ward locks this conversation to make that a road-block rather than a convention (ward#494).

run seed context — what this run is carrying (ward#609)
  • Resolved: coilyco-flight-deck/ward#733 · branch issue-733 · harness codex · workflow pull-request-and-merge
  • Run: engineer-codex-ward-733 · ward v0.493.0 · dispatched 2026-07-09T15:54:51Z
  • Comment thread: 3 included in the pre-flight read, 2 stripped (ward's own automated comments).

Static container doctrine and seed boilerplate are identical every run and omitted here (they ride ward v0.493.0).

— Codex, via ward agent

<!-- ward-agent-reservation --> WARD-RESERVATION: held 🔒 <details><summary>reservation details</summary> Holder: container `engineer-codex-ward-733` on host `kais-macbook-pro-2.local`. Reserved by `ward agent --harness codex` (reserved 2026-07-09T15:54:51Z). Concurrent `ward agent` runs are blocked until it finishes or the reservation goes stale (1h TTL). `--force` overrides. **Do not comment on or edit this issue to steer the run while it is reserved.** The engineer seeded the body once at launch and never re-reads it, so a comment or edit reaches only human readers, never the running engineer. A correction goes to a **new issue, dispatched fresh**. That is the only channel that reaches a run in flight. Where the forge supports it, ward locks this conversation to make that a road-block rather than a convention (ward#494). <details><summary>run seed context — what this run is carrying (ward#609)</summary> - **Resolved:** `coilyco-flight-deck/ward#733` · branch `issue-733` · harness `codex` · workflow `pull-request-and-merge` - **Run:** `engineer-codex-ward-733` · ward `v0.493.0` · dispatched `2026-07-09T15:54:51Z` - **Comment thread:** 3 included in the pre-flight read, 2 stripped (ward's own automated comments). - included: @coilyco-ops (2026-07-09T02:40:53Z), @coilyco-ops (2026-07-09T02:41:17Z), @coilyco-ops (2026-07-09T07:27:07Z) - stripped: @coilyco-ops (2026-07-09T02:26:18Z), @coilyco-ops (2026-07-09T07:27:10Z) Static container doctrine and seed boilerplate are identical every run and omitted here (they ride ward v0.493.0). </details> </details> <!-- ward-agent-signature --> — Codex, via `ward agent`
Author
Member

WARD-OUTCOME: done

details workflow: pull-request-and-merge; review summary: review gate skipped by --skip-review / --no-review-gate

felt: straight-line cleanup once the stale branch was bypassed
confidence: high
surprises: the remote issue branch was stale, so a clean branch off origin/main was the safe path
follow-ups: none

WARD-OUTCOME: done ✅ <details><summary>details</summary> workflow: pull-request-and-merge; review summary: review gate skipped by --skip-review / --no-review-gate felt: straight-line cleanup once the stale branch was bypassed confidence: high surprises: the remote issue branch was stale, so a clean branch off `origin/main` was the safe path follow-ups: none </details>
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
coilyco-flight-deck/ward#733
No description provided.