diff --git a/hooks/hooks.json b/hooks/hooks.json index 344b1ab..697d83c 100644 --- a/hooks/hooks.json +++ b/hooks/hooks.json @@ -105,7 +105,7 @@ "hooks": [ { "type": "command", - "command": "node -e \"const{execFileSync}=require('child_process');const fs=require('fs');let d='';process.stdin.on('data',c=>d+=c);process.stdin.on('end',()=>{const i=JSON.parse(d);const p=i.tool_input?.file_path;if(p&&/\\.(ts|tsx|js|jsx)$/.test(p)&&fs.existsSync(p)){try{execFileSync('npx',['prettier','--write',p],{stdio:['pipe','pipe','pipe']})}catch(e){}}console.log(d)})\"" + "command": "node \"${CLAUDE_PLUGIN_ROOT}/scripts/hooks/post-edit-format.js\"" } ], "description": "Auto-format JS/TS files with Prettier after edits" @@ -115,7 +115,7 @@ "hooks": [ { "type": "command", - "command": "node -e \"const{execFileSync}=require('child_process');const fs=require('fs');const path=require('path');let d='';process.stdin.on('data',c=>d+=c);process.stdin.on('end',()=>{const i=JSON.parse(d);const p=i.tool_input?.file_path;if(p&&/\\.(ts|tsx)$/.test(p)&&fs.existsSync(p)){let dir=path.dirname(p);while(dir!==path.dirname(dir)&&!fs.existsSync(path.join(dir,'tsconfig.json'))){dir=path.dirname(dir)}if(fs.existsSync(path.join(dir,'tsconfig.json'))){try{const r=execFileSync('npx',['tsc','--noEmit','--pretty','false'],{cwd:dir,encoding:'utf8',stdio:['pipe','pipe','pipe']});const lines=r.split('\\n').filter(l=>l.includes(p)).slice(0,10);if(lines.length)console.error(lines.join('\\n'))}catch(e){const lines=(e.stdout||'').split('\\n').filter(l=>l.includes(p)).slice(0,10);if(lines.length)console.error(lines.join('\\n'))}}}console.log(d)})\"" + "command": "node \"${CLAUDE_PLUGIN_ROOT}/scripts/hooks/post-edit-typecheck.js\"" } ], "description": "TypeScript check after editing .ts/.tsx files" @@ -125,7 +125,7 @@ "hooks": [ { "type": "command", - "command": "node -e \"const fs=require('fs');let d='';process.stdin.on('data',c=>d+=c);process.stdin.on('end',()=>{const i=JSON.parse(d);const p=i.tool_input?.file_path;if(p&&/\\.(ts|tsx|js|jsx)$/.test(p)&&fs.existsSync(p)){const c=fs.readFileSync(p,'utf8');const lines=c.split('\\n');const matches=[];lines.forEach((l,idx)=>{if(/console\\.log/.test(l))matches.push((idx+1)+': '+l.trim())});if(matches.length){console.error('[Hook] WARNING: console.log found in '+p);matches.slice(0,5).forEach(m=>console.error(m));console.error('[Hook] Remove console.log before committing')}}console.log(d)})\"" + "command": "node \"${CLAUDE_PLUGIN_ROOT}/scripts/hooks/post-edit-console-warn.js\"" } ], "description": "Warn about console.log statements after edits" diff --git a/scripts/hooks/post-edit-console-warn.js b/scripts/hooks/post-edit-console-warn.js new file mode 100644 index 0000000..e4f5c3c --- /dev/null +++ b/scripts/hooks/post-edit-console-warn.js @@ -0,0 +1,47 @@ +#!/usr/bin/env node +/** + * PostToolUse Hook: Warn about console.log statements after edits + * + * Cross-platform (Windows, macOS, Linux) + * + * Runs after Edit tool use. If the edited JS/TS file contains console.log + * statements, warns with line numbers to help remove debug statements + * before committing. + */ + +const fs = require('fs'); + +let data = ''; + +process.stdin.on('data', chunk => { + data += chunk; +}); + +process.stdin.on('end', () => { + try { + const input = JSON.parse(data); + const filePath = input.tool_input?.file_path; + + if (filePath && /\.(ts|tsx|js|jsx)$/.test(filePath) && fs.existsSync(filePath)) { + const content = fs.readFileSync(filePath, 'utf8'); + const lines = content.split('\n'); + const matches = []; + + lines.forEach((line, idx) => { + if (/console\.log/.test(line)) { + matches.push((idx + 1) + ': ' + line.trim()); + } + }); + + if (matches.length > 0) { + console.error('[Hook] WARNING: console.log found in ' + filePath); + matches.slice(0, 5).forEach(m => console.error(m)); + console.error('[Hook] Remove console.log before committing'); + } + } + } catch { + // Invalid input — pass through + } + + console.log(data); +}); diff --git a/scripts/hooks/post-edit-format.js b/scripts/hooks/post-edit-format.js new file mode 100644 index 0000000..8769cd6 --- /dev/null +++ b/scripts/hooks/post-edit-format.js @@ -0,0 +1,40 @@ +#!/usr/bin/env node +/** + * PostToolUse Hook: Auto-format JS/TS files with Prettier after edits + * + * Cross-platform (Windows, macOS, Linux) + * + * Runs after Edit tool use. If the edited file is a JS/TS file, + * formats it with Prettier. Fails silently if Prettier isn't installed. + */ + +const { execFileSync } = require('child_process'); +const fs = require('fs'); + +let data = ''; + +process.stdin.on('data', chunk => { + data += chunk; +}); + +process.stdin.on('end', () => { + try { + const input = JSON.parse(data); + const filePath = input.tool_input?.file_path; + + if (filePath && /\.(ts|tsx|js|jsx)$/.test(filePath) && fs.existsSync(filePath)) { + try { + execFileSync('npx', ['prettier', '--write', filePath], { + stdio: ['pipe', 'pipe', 'pipe'], + timeout: 15000 + }); + } catch { + // Prettier not installed or failed — non-blocking + } + } + } catch { + // Invalid input — pass through + } + + console.log(data); +}); diff --git a/scripts/hooks/post-edit-typecheck.js b/scripts/hooks/post-edit-typecheck.js new file mode 100644 index 0000000..51c013e --- /dev/null +++ b/scripts/hooks/post-edit-typecheck.js @@ -0,0 +1,69 @@ +#!/usr/bin/env node +/** + * PostToolUse Hook: TypeScript check after editing .ts/.tsx files + * + * Cross-platform (Windows, macOS, Linux) + * + * Runs after Edit tool use on TypeScript files. Walks up from the file's + * directory to find the nearest tsconfig.json, then runs tsc --noEmit + * and reports only errors related to the edited file. + */ + +const { execFileSync } = require('child_process'); +const fs = require('fs'); +const path = require('path'); + +let data = ''; + +process.stdin.on('data', chunk => { + data += chunk; +}); + +process.stdin.on('end', () => { + try { + const input = JSON.parse(data); + const filePath = input.tool_input?.file_path; + + if (filePath && /\.(ts|tsx)$/.test(filePath) && fs.existsSync(filePath)) { + // Find nearest tsconfig.json by walking up (max 20 levels to prevent infinite loop) + let dir = path.dirname(path.resolve(filePath)); + const root = path.parse(dir).root; + let depth = 0; + + while (dir !== root && depth < 20) { + if (fs.existsSync(path.join(dir, 'tsconfig.json'))) { + break; + } + dir = path.dirname(dir); + depth++; + } + + if (fs.existsSync(path.join(dir, 'tsconfig.json'))) { + try { + execFileSync('npx', ['tsc', '--noEmit', '--pretty', 'false'], { + cwd: dir, + encoding: 'utf8', + stdio: ['pipe', 'pipe', 'pipe'], + timeout: 30000 + }); + } catch (err) { + // tsc exits non-zero when there are errors — filter to edited file + const output = err.stdout || ''; + const relevantLines = output + .split('\n') + .filter(line => line.includes(filePath) || line.includes(path.basename(filePath))) + .slice(0, 10); + + if (relevantLines.length > 0) { + console.error('[Hook] TypeScript errors in ' + path.basename(filePath) + ':'); + relevantLines.forEach(line => console.error(line)); + } + } + } + } + } catch { + // Invalid input — pass through + } + + console.log(data); +});