mirror of
https://github.com/affaan-m/everything-claude-code.git
synced 2026-02-02 04:33:08 +08:00
a0b84f7b86
Fixes #78 ## Problem The Stop hook used inline JavaScript code with `node -e`, which caused shell syntax errors on macOS/zsh due to special characters (parentheses, braces, arrow functions) being misinterpreted by the shell. Error message: /bin/sh: -c: line 0: syntax error near unexpected token \`(' ## Solution - Created scripts/hooks/check-console-log.js with the hook logic - Updated hooks/hooks.json to reference the external script - This follows the same pattern as other hooks in the plugin ## Benefits - Fixes shell compatibility issues across different environments - Improves code maintainability (separate, well-documented script) - Follows plugin's own best practices - Makes the code easier to test and debug ## Testing Tested on macOS with zsh - no more syntax errors. The hook still functions correctly to detect console.log statements.
62 lines
1.6 KiB
JavaScript
Executable File
62 lines
1.6 KiB
JavaScript
Executable File
#!/usr/bin/env node
|
|
|
|
/**
|
|
* Stop Hook: Check for console.log statements in modified files
|
|
*
|
|
* This hook runs after each response and checks if any modified
|
|
* JavaScript/TypeScript files contain console.log statements.
|
|
* It provides warnings to help developers remember to remove
|
|
* debug statements before committing.
|
|
*/
|
|
|
|
const { execSync } = require('child_process');
|
|
const fs = require('fs');
|
|
|
|
let data = '';
|
|
|
|
// Read stdin
|
|
process.stdin.on('data', chunk => {
|
|
data += chunk;
|
|
});
|
|
|
|
process.stdin.on('end', () => {
|
|
try {
|
|
// Check if we're in a git repository
|
|
try {
|
|
execSync('git rev-parse --git-dir', { stdio: 'pipe' });
|
|
} catch {
|
|
// Not in a git repo, just pass through the data
|
|
console.log(data);
|
|
process.exit(0);
|
|
}
|
|
|
|
// Get list of modified files
|
|
const files = execSync('git diff --name-only HEAD', {
|
|
encoding: 'utf8',
|
|
stdio: ['pipe', 'pipe', 'pipe']
|
|
})
|
|
.split('\n')
|
|
.filter(f => /\.(ts|tsx|js|jsx)$/.test(f) && fs.existsSync(f));
|
|
|
|
let hasConsole = false;
|
|
|
|
// Check each file for console.log
|
|
for (const file of files) {
|
|
const content = fs.readFileSync(file, 'utf8');
|
|
if (content.includes('console.log')) {
|
|
console.error(`[Hook] WARNING: console.log found in ${file}`);
|
|
hasConsole = true;
|
|
}
|
|
}
|
|
|
|
if (hasConsole) {
|
|
console.error('[Hook] Remove console.log statements before committing');
|
|
}
|
|
} catch (error) {
|
|
// Silently ignore errors (git might not be available, etc.)
|
|
}
|
|
|
|
// Always output the original data
|
|
console.log(data);
|
|
});
|