Skip to content

fix(agent): run the pre-push gate without pnpm exec #305

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
patroza merged 1 commit into from
Aug 4, 2026
Merged

fix(agent): run the pre-push gate without pnpm exec #305

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 6 additions & 2 deletions .githooks/pre-push
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
# Agent-only ship gate (humans: no-op). See scripts/agent-pre-push.mjs.
# Draft / no PR: free agent push. Ready PR: vp check + typecheck + tests.
# Publish: pnpm pr:ready (not raw gh pr ready). Agent gh shim: .tools/bin/gh.
# Publish: pnpm pr:ready, or raw `gh pr ready` — the shim runs the same gate first.
#
# Humans: SKIP_AGENT_PREPUSH=1 git push
# Agents: never SKIP_AGENT_PREPUSH / never --no-verify
Expand All @@ -9,4 +9,8 @@
for name in $(git rev-parse --local-env-vars); do
unset "$name"
done
pnpm exec node scripts/agent-pre-push.mjs
# Plain `node`, not `pnpm exec node`: pnpm runs a dependency-status check first
# and will try to purge node_modules when the lockfile moved (a rebase does it),
# which aborts without a TTY — the hook then fails with a pnpm error instead of
# anything about the gate. The gate puts node_modules/.bin on PATH itself.
node scripts/agent-pre-push.mjs
Loading