mosaicstack/stack
16
0
Fork 1
Code Issues 330 Pull Requests 12 Actions Packages Projects Releases 15 Wiki Activity

wrapper-guard.sh PreToolUse hook crashed twice with no stderr on a fleet-comms push, then passed identical shapes #1384

New Issue
Open
opened 2026-08-24 20:09:57 +00:00 by orch-01 · 0 comments
orch-01 commented 2026-08-24 20:09:57 +00:00
Contributor
Copy Link
Copy Source

Observed 2026-08-24 by orch-01 (Claude Code harness, auto mode), filing per operator instruction (Q44).

  1. Symptom: PreToolUse hook wrapper-guard.sh failed twice in a row on a git push to the fleet-comms repo (hook error, no stderr reaching the session), then passed on later pushes of identical shape (same repo, same command form, same identity env).
  2. First failing attempt contained a stray escaped line (git add "$F") in a heredoc-built script; the clean retry also failed, so the escape alone does not explain it.
  3. Impact: transient false-block of a legitimate push; cost a retry cycle. Not bypassed (no --no-verify), consistent with the hooks-are-the-gate rule.
  4. Ask: make wrapper-guard.sh fail loud (emit its refusal reason to stderr unconditionally) and add a trace mode so a crash is distinguishable from a policy block.
  5. Related harness observation recorded fleet-side, not a stack defect: the Claude Code harness strips a leading cd-to-workdir prefix from some Bash commands, so hook and tool code must never depend on cwd or a leading cd (use git -C / absolute paths). Noted here for wrapper-guard test coverage only.
Observed 2026-08-24 by orch-01 (Claude Code harness, auto mode), filing per operator instruction (Q44). 1. Symptom: PreToolUse hook wrapper-guard.sh failed twice in a row on a git push to the fleet-comms repo (hook error, no stderr reaching the session), then passed on later pushes of identical shape (same repo, same command form, same identity env). 2. First failing attempt contained a stray escaped line (git add \"\$F\") in a heredoc-built script; the clean retry also failed, so the escape alone does not explain it. 3. Impact: transient false-block of a legitimate push; cost a retry cycle. Not bypassed (no --no-verify), consistent with the hooks-are-the-gate rule. 4. Ask: make wrapper-guard.sh fail loud (emit its refusal reason to stderr unconditionally) and add a trace mode so a crash is distinguishable from a policy block. 5. Related harness observation recorded fleet-side, not a stack defect: the Claude Code harness strips a leading cd-to-workdir prefix from some Bash commands, so hook and tool code must never depend on cwd or a leading cd (use git -C / absolute paths). Noted here for wrapper-guard test coverage only.
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
bug
Defect in framework tooling or delivered code
 fleet-enhancement
Fleet-wide reliability/speed/cost improvements surfaced by the nightly enhance pass
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
code-be-01 (Mosaic fleet seat code-be-01) code-be-02 (Mosaic fleet seat code-be-02) code-infra-01 (Mosaic fleet seat code-infra-01) fargo fred gate-merge-01 (Mosaic fleet seat gate-merge-01) happy jason.woltje (Jason Woltje) merge-gate ops-01 (Mosaic fleet seat ops-01) ops-02 (Mosaic fleet seat ops-02) ops-03 (Mosaic fleet seat ops-03) ops-ci-01 (Mosaic fleet seat ops-ci-01) ops-deploy-01 (Mosaic fleet seat ops-deploy-01) pepper rev-code-01 rev-code-02 rev-security-01 rev-security-02 rev-security-03 (Mosaic fleet seat rev-security-03) sanity scooby (Scooby) scrappy shaggy tiny velma veronica (Mosaic fleet seat veronica) vision woodpecker
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: mosaicstack/stack#1384