zsp/everything-claude-code
1
0
Fork 0
mirror of https://github.com/affaan-m/everything-claude-code.git synced 2026-04-15 14:33:33 +08:00
Code Issues Packages Projects Releases Wiki Activity

fix: preserve commit-quality blocking in bash dispatcher

Browse Source
This commit is contained in:
Affaan Mustafa
2026-04-14 21:17:17 -07:00
parent 1c45152c6d
commit a8bb5979a5
2 changed files with 8 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
scripts/hooks/pre-bash-commit-quality.js
View File

@@ -380,7 +380,11 @@ function evaluate(rawInput) {
} }
function run(rawInput) { function run(rawInput) {
return evaluate(rawInput).output; const result = evaluate(rawInput);
return {
stdout: result.output,
exitCode: result.exitCode,
};
} }
// ── stdin entry point ──────────────────────────────────────────── // ── stdin entry point ────────────────────────────────────────────

6
tests/hooks/bash-hook-dispatcher.test.js
View File

@@ -74,7 +74,7 @@ function runTests() {
if (test('post dispatcher writes both bash audit and cost logs in one pass', () => { if (test('post dispatcher writes both bash audit and cost logs in one pass', () => {
const homeDir = fs.mkdtempSync(path.join(os.tmpdir(), 'ecc-bash-dispatcher-')); const homeDir = fs.mkdtempSync(path.join(os.tmpdir(), 'ecc-bash-dispatcher-'));
const payload = { tool_input: { command: 'npm publish --token abc123' } }; const payload = { tool_input: { command: 'npm publish --token fixture-token' } };
try { try {
const result = runScript(postDispatcher, payload, { const result = runScript(postDispatcher, payload, {
@@ -89,8 +89,8 @@ function runTests() {
assert.ok(auditLog.includes('--token=<REDACTED>')); assert.ok(auditLog.includes('--token=<REDACTED>'));
assert.ok(costLog.includes('tool=Bash command=npm publish --token=<REDACTED>')); assert.ok(costLog.includes('tool=Bash command=npm publish --token=<REDACTED>'));
assert.ok(!auditLog.includes('abc123')); assert.ok(!auditLog.includes('fixture-token'));
assert.ok(!costLog.includes('abc123')); assert.ok(!costLog.includes('fixture-token'));
} finally { } finally {
fs.rmSync(homeDir, { recursive: true, force: true }); fs.rmSync(homeDir, { recursive: true, force: true });
} }