From 71af170a08e1487c83d4f0e69fe20e8f64e79862 Mon Sep 17 00:00:00 2001 From: Tom Boucher Date: Fri, 17 Apr 2026 12:22:09 -0400 Subject: [PATCH] fix(tests): clear CLAUDECODE env var in read-guard test runner (#2375) The hook skips its advisory on two env vars: CLAUDE_SESSION_ID and CLAUDECODE. runHook() cleared CLAUDE_SESSION_ID but inherited CLAUDECODE from process.env, so tests run inside a Claude Code session silently no-oped and produced no stdout, causing JSON.parse to throw. Co-authored-by: Claude Sonnet 4.6 --- tests/read-guard.test.cjs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/read-guard.test.cjs b/tests/read-guard.test.cjs index 55f0df08d..5fa545787 100644 --- a/tests/read-guard.test.cjs +++ b/tests/read-guard.test.cjs @@ -29,7 +29,7 @@ const HOOK_PATH = path.join(__dirname, '..', 'hooks', 'gsd-read-guard.js'); function runHook(payload, envOverrides = {}) { const input = JSON.stringify(payload); // Sanitize CLAUDE_SESSION_ID so positive-path tests work inside Claude Code sessions - const env = { ...process.env, CLAUDE_SESSION_ID: '', ...envOverrides }; + const env = { ...process.env, CLAUDE_SESSION_ID: '', CLAUDECODE: '', ...envOverrides }; try { const stdout = execFileSync(process.execPath, [HOOK_PATH], { input,