claude-smith 3.0.0 → 3.2.0

package/hooks/build-tracker.mjs

@@ -5,21 +5,19 @@
  * Records build pass/fail result for build-guard reference
  */
 
-import { readFileSync, writeFileSync, mkdirSync } from 'fs';
+import { writeFileSync, mkdirSync } from 'fs';
 import { join } from 'path';
 import { tmpdir } from 'os';
 import { getHookConfig } from '../lib/config.mjs';
 import { recordEvent } from '../lib/stats.mjs';
+import { appendEvent } from '../lib/event-log.mjs';
+import { sanitizeId, parseHookInput, notifyUser } from '../lib/hook-utils.mjs';
 
 const config = getHookConfig('build-tracker', process.cwd());
 if (!config.enabled) process.exit(0);
 
-let input;
-try {
-  input = JSON.parse(readFileSync(0, 'utf8'));
-} catch {
-  process.exit(0);
-}
+const input = parseHookInput();
+if (!input) process.exit(0);
 
 const command = input.tool_input?.command || '';
 
@@ -27,11 +25,6 @@ if (input.tool_name !== 'Bash') process.exit(0);
 
 // Detect build commands
 if (/pnpm build|npm run build|yarn build|tsc|next build/.test(command)) {
-  function sanitizeId(id) {
-    if (!id || typeof id !== 'string') return 'default';
-    return id.replace(/[^a-zA-Z0-9_-]/g, '').slice(0, 128) || 'default';
-  }
-
   const sessionId = sanitizeId(input.session_id);
   const stateDir = join(tmpdir(), '.claude-smith', sessionId);
   mkdirSync(stateDir, { recursive: true, mode: 0o700 });
@@ -39,6 +32,9 @@ if (/pnpm build|npm run build|yarn build|tsc|next build/.test(command)) {
   const response = JSON.stringify(input.tool_response || '').toLowerCase();
   const failed = /\b(?:build\s+failed|[1-9]\d*\s+errors?\b|error(?:s)?:\s*[1-9]|exit\s+code\s+[1-9]|failed\s+to\s+compile)/i.test(response)
     && !/\b0\s+errors?\b/.test(response);
-  writeFileSync(join(stateDir, 'last-build-result'), failed ? 'fail' : 'pass', { mode: 0o600 });
+  const result = failed ? 'fail' : 'pass';
+  writeFileSync(join(stateDir, 'last-build-result'), result, { mode: 0o600 });
   recordEvent(sessionId, 'build-tracker', 'fire');
+  appendEvent(process.cwd(), { hook: 'build-tracker', event: 'fire', message: result });
+  notifyUser(config.notify, 'Build Tracker', 'track', failed ? '❌ fail' : '✅ pass');
 }

package/hooks/commit-guard.mjs

@@ -12,16 +12,14 @@ import { join } from 'path';
 import { tmpdir } from 'os';
 import { getHookConfig } from '../lib/config.mjs';
 import { recordEvent } from '../lib/stats.mjs';
+import { appendEvent } from '../lib/event-log.mjs';
+import { sanitizeId, parseHookInput, notifyUser } from '../lib/hook-utils.mjs';
 
 const config = getHookConfig('commit-guard', process.cwd());
 if (!config.enabled) process.exit(0);
 
-let input;
-try {
-  input = JSON.parse(readFileSync(0, 'utf8'));
-} catch {
-  process.exit(0);
-}
+const input = parseHookInput();
+if (!input) process.exit(0);
 
 const toolName = input.tool_name;
 const command = input.tool_input?.command || '';
@@ -32,24 +30,19 @@ if (!command.includes('git commit')) process.exit(0);
 // Skip amend, merge commits
 if (command.includes('--allow-empty')) process.exit(0);
 
-function sanitizeId(id) {
-  if (!id || typeof id !== 'string') return 'default';
-  return id.replace(/[^a-zA-Z0-9_-]/g, '').slice(0, 128) || 'default';
-}
-
 const sessionId = sanitizeId(input.session_id);
 const stateDir = join(tmpdir(), '.claude-smith', sessionId);
 const lastTestFile = join(stateDir, 'last-test-run');
 const lastResultFile = join(stateDir, 'last-test-result');
 
-recordEvent(sessionId, 'commit-guard', 'fire');
-
 // Check test result first (hard block on failure)
 if (existsSync(lastResultFile)) {
   const result = readFileSync(lastResultFile, 'utf8').trim();
   if (result === 'fail') {
     recordEvent(sessionId, 'commit-guard', 'block');
-    process.stderr.write("🔨 Smith ✋ Commit blocked: last test run FAILED. Run your test suite and fix failures before committing.\n💡 Coaching: 1) Run your test suite. 2) Fix failing tests one at a time. 3) Verify all pass. 4) Then commit.");
+    appendEvent(process.cwd(), { hook: 'commit-guard', event: 'block', message: 'Tests failing' });
+    notifyUser(config.notify, 'Commit Guard', 'block', '테스트 실패 — 커밋 차단');
+    process.stderr.write("🕵️ Smith ✋ Commit blocked: last test run FAILED. Run your test suite and fix failures before committing.\n💡 Coaching: 1) Run your test suite. 2) Fix failing tests one at a time. 3) Verify all pass. 4) Then commit.");
     process.exit(2);
   }
 }
@@ -61,21 +54,32 @@ if (existsSync(lastTestFile)) {
   const diff = now - lastTest;
   if (diff > config.maxTestAge) {
     recordEvent(sessionId, 'commit-guard', 'warn');
+    appendEvent(process.cwd(), { hook: 'commit-guard', event: 'warn', message: 'Tests stale' });
+    notifyUser(config.notify, 'Commit Guard', 'warn', '테스트 경과 시간 초과');
     const output = JSON.stringify({
       hookSpecificOutput: {
         hookEventName: "PreToolUse",
-        additionalContext: `🔨 Smith > Commit Guard: Last test run was ${Math.floor(diff / 60)}min ago (>5min). Run tests before committing.`
+        additionalContext: `🕵️ Smith > Commit Guard: Last test run was ${Math.floor(diff / 60)}min ago (>5min). Run tests before committing.`
       }
     });
     process.stdout.write(output);
+    process.exit(0);
   }
 } else {
   recordEvent(sessionId, 'commit-guard', 'warn');
+  appendEvent(process.cwd(), { hook: 'commit-guard', event: 'warn', message: 'No test run detected' });
+  notifyUser(config.notify, 'Commit Guard', 'warn', '테스트 실행 기록 없음');
   const output = JSON.stringify({
     hookSpecificOutput: {
       hookEventName: "PreToolUse",
-      additionalContext: "🔨 Smith > Commit Guard: No test run detected this session. Consider running tests before committing.\n💡 Coaching: Run your test suite to verify nothing is broken. Even a quick smoke test prevents broken commits."
+      additionalContext: "🕵️ Smith > Commit Guard: No test run detected this session. Consider running tests before committing.\n💡 Coaching: Run your test suite to verify nothing is broken. Even a quick smoke test prevents broken commits."
     }
   });
   process.stdout.write(output);
+  process.exit(0);
 }
+
+// All checks passed - commit allowed
+recordEvent(sessionId, 'commit-guard', 'fire');
+appendEvent(process.cwd(), { hook: 'commit-guard', event: 'fire', message: 'Commit allowed' });
+notifyUser(config.notify, 'Commit Guard', 'fire', '커밋 허용');

package/hooks/commit-message.mjs

@@ -5,39 +5,32 @@
  * Enforces conventional commits format: type(scope): description
  */
 
-import { readFileSync } from 'fs';
 import { getHookConfig } from '../lib/config.mjs';
 import { recordEvent } from '../lib/stats.mjs';
+import { appendEvent } from '../lib/event-log.mjs';
+import { sanitizeId, parseHookInput, notifyUser } from '../lib/hook-utils.mjs';
 
 const config = getHookConfig('commit-message', process.cwd());
 if (!config.enabled) process.exit(0);
 
-let input;
-try {
-  input = JSON.parse(readFileSync(0, 'utf8'));
-} catch {
-  process.exit(0);
-}
+const input = parseHookInput();
+if (!input) process.exit(0);
 
 const command = input.tool_input?.command || '';
 
 if (input.tool_name !== 'Bash') process.exit(0);
 if (!command.includes('git commit')) process.exit(0);
 
-function sanitizeId(id) {
-  if (!id || typeof id !== 'string') return 'default';
-  return id.replace(/[^a-zA-Z0-9_-]/g, '').slice(0, 128) || 'default';
-}
-
 const sessionId = sanitizeId(input.session_id);
 
 // Extract commit message from -m flag
-const msgMatch = command.match(/-m\s+"([^"]+)"/s)
-  || command.match(/-m\s+'([^']+)'/s)
-  || command.match(/-m\s+"\$\(cat\s+<<'?EOF'?\s*\n([\s\S]*?)\n\s*EOF/);
+// Check HEREDOC format first (used by Claude Code)
+const msgMatch = command.match(/-m\s+"\$\(cat\s+<<'?EOF'?\s*\n([\s\S]*?)\n\s*EOF/)
+  || command.match(/-m\s+"([^"]+)"/s)
+  || command.match(/-m\s+'([^']+)'/s);
 if (!msgMatch) process.exit(0); // Can't parse message, let it through
 
-const msg = msgMatch[1] || msgMatch[2] || '';
+const msg = msgMatch[1] || '';
 const firstLine = msg.split('\n')[0].trim();
 
 // Conventional commit pattern: type(scope): description OR type: description
@@ -45,11 +38,18 @@ const pattern = /^(feat|fix|docs|style|refactor|perf|test|build|ci|chore|revert)
 
 if (!pattern.test(firstLine)) {
   recordEvent(sessionId, 'commit-message', 'warn');
+  appendEvent(process.cwd(), { hook: 'commit-message', event: 'warn', message: `Non-conventional: ${firstLine.slice(0, 50)}` });
+  notifyUser(config.notify, 'Commit Message', 'warn', `비표준 형식: ${firstLine.slice(0, 40)}`);
   const output = JSON.stringify({
     hookSpecificOutput: {
       hookEventName: "PreToolUse",
-      additionalContext: `🔨 Smith > Commit Message: "${firstLine}" doesn't follow conventional commits. Expected: type(scope): description (e.g., feat(auth): add login, fix: resolve crash)`
+      additionalContext: `🕵️ Smith > Commit Message: "${firstLine}" doesn't follow conventional commits. Expected: type(scope): description (e.g., feat(auth): add login, fix: resolve crash)`
     }
   });
   process.stdout.write(output);
+} else {
+  // Format is valid
+  recordEvent(sessionId, 'commit-message', 'fire');
+  appendEvent(process.cwd(), { hook: 'commit-message', event: 'fire', message: 'Format valid' });
+  notifyUser(config.notify, 'Commit Message', 'fire', '형식 유효');
 }

package/hooks/debug-loop.mjs

@@ -11,16 +11,14 @@ import { tmpdir } from 'os';
 import { createHash } from 'crypto';
 import { getHookConfig } from '../lib/config.mjs';
 import { recordEvent } from '../lib/stats.mjs';
+import { appendEvent } from '../lib/event-log.mjs';
+import { sanitizeId, parseHookInput, isOmcAutoMode, notifyUser } from '../lib/hook-utils.mjs';
 
 const config = getHookConfig('debug-loop', process.cwd());
 if (!config.enabled) process.exit(0);
 
-let input;
-try {
-  input = JSON.parse(readFileSync(0, 'utf8'));
-} catch {
-  process.exit(0);
-}
+const input = parseHookInput();
+if (!input) process.exit(0);
 
 const toolName = input.tool_name;
 const filePath = input.tool_input?.file_path || '';
@@ -31,11 +29,6 @@ if (!filePath) process.exit(0);
 // Skip non-source files
 if (!/\.(ts|tsx|js|jsx|py|go|rs)$/.test(filePath)) process.exit(0);
 
-function sanitizeId(id) {
-  if (!id || typeof id !== 'string') return 'default';
-  return id.replace(/[^a-zA-Z0-9_-]/g, '').slice(0, 128) || 'default';
-}
-
 // Isolate state per agent when sub-agents provide their own ID
 const sessionId = sanitizeId(input.session_id);
 const agentId = sanitizeId(input.agent_id || '');
@@ -44,15 +37,7 @@ const stateDir = join(tmpdir(), '.claude-smith', stateKey);
 mkdirSync(stateDir, { recursive: true, mode: 0o700 });
 
 // Raise thresholds during OMC autonomous modes to avoid false positives in parallel execution
-const omcStateDir = join(process.cwd(), '.omc', 'state');
-const omcAutoModes = ['autopilot-state.json', 'ultrawork-state.json', 'ralph-state.json'];
-const isOmcAutoMode = omcAutoModes.some(f => {
-  try {
-    const state = JSON.parse(readFileSync(join(omcStateDir, f), 'utf8'));
-    return state.active === true || state.status === 'running';
-  } catch { return false; }
-});
-if (isOmcAutoMode) {
+if (isOmcAutoMode(process.cwd())) {
   config.warnAt = Math.max(config.warnAt, 6);
   config.blockAt = Math.max(config.blockAt, 10);
 }
@@ -100,15 +85,19 @@ const patternHint = pattern === 'divergent'
 
 if (count === config.warnAt) {
   recordEvent(sessionId, 'debug-loop', 'warn');
+  appendEvent(process.cwd(), { hook: 'debug-loop', event: 'warn', file: filePath, message: `Edit #${count}${patternHint ? ' ' + patternHint : ''}` });
+  notifyUser(config.notify, 'Debug Loop', 'warn', `${name} ${count}회 편집${pattern === 'divergent' ? ' (반복 패턴)' : ''}`);
   const output = JSON.stringify({
     hookSpecificOutput: {
       hookEventName: "PostToolUse",
-      additionalContext: `🔨 Smith > Debug Loop: ${name} edited ${config.warnAt} times.${patternHint}\n💡 Coaching: 1) STOP editing. Read the error message carefully. 2) Add logging to trace data flow. 3) Check recent git changes: git diff HEAD~3. 4) Find a working similar pattern and compare. 5) Form a hypothesis before making the next edit.`
+      additionalContext: `🕵️ Smith > Debug Loop: ${name} edited ${config.warnAt} times.${patternHint}\n💡 Coaching: 1) STOP editing. Read the error message carefully. 2) Add logging to trace data flow. 3) Check recent git changes: git diff HEAD~3. 4) Find a working similar pattern and compare. 5) Form a hypothesis before making the next edit.`
     }
   });
   process.stdout.write(output);
 } else if (count >= config.blockAt) {
   recordEvent(sessionId, 'debug-loop', 'block');
-  process.stderr.write(`🔨 Smith ✋ Debug Loop blocked: ${name} edited ${count} times.${patternHint}\nPer rule 2-6: 3+ failed fixes → question architecture. Discuss with user before continuing.`);
+  appendEvent(process.cwd(), { hook: 'debug-loop', event: 'block', file: filePath, message: `Edit #${count} - blocked` });
+  notifyUser(config.notify, 'Debug Loop', 'block', `${name} ${count}회 편집 — 차단`);
+  process.stderr.write(`🕵️ Smith ✋ Debug Loop blocked: ${name} edited ${count} times.${patternHint}\nPer rule 2-6: 3+ failed fixes → question architecture. Discuss with user before continuing.`);
   process.exit(2);
 }

package/hooks/file-size-warn.mjs

@@ -9,16 +9,14 @@ import { readFileSync, existsSync } from 'fs';
 import { basename } from 'path';
 import { getHookConfig } from '../lib/config.mjs';
 import { recordEvent } from '../lib/stats.mjs';
+import { appendEvent } from '../lib/event-log.mjs';
+import { sanitizeId, parseHookInput, notifyUser } from '../lib/hook-utils.mjs';
 
 const config = getHookConfig('file-size-warn', process.cwd());
 if (!config.enabled) process.exit(0);
 
-let input;
-try {
-  input = JSON.parse(readFileSync(0, 'utf8'));
-} catch {
-  process.exit(0);
-}
+const input = parseHookInput();
+if (!input) process.exit(0);
 
 const filePath = input.tool_input?.file_path || '';
 
@@ -30,11 +28,6 @@ if (!/\.(ts|tsx|js|jsx|py|go|rs)$/.test(filePath)) process.exit(0);
 
 if (!existsSync(filePath)) process.exit(0);
 
-function sanitizeId(id) {
-  if (!id || typeof id !== 'string') return 'default';
-  return id.replace(/[^a-zA-Z0-9_-]/g, '').slice(0, 128) || 'default';
-}
-
 const sessionId = sanitizeId(input.session_id);
 
 const lines = readFileSync(filePath, 'utf8').split('\n').length;
@@ -42,10 +35,12 @@ const threshold = config.maxLines || 500;
 
 if (lines > threshold) {
   recordEvent(sessionId, 'file-size-warn', 'warn');
+  appendEvent(process.cwd(), { hook: 'file-size-warn', event: 'warn', file: filePath, message: `${lines} lines` });
+  notifyUser(config.notify, 'File Size', 'warn', `${basename(filePath)} ${lines}줄 (>${threshold})`);
   const output = JSON.stringify({
     hookSpecificOutput: {
       hookEventName: "PostToolUse",
-      additionalContext: `🔨 Smith > File Size: ${basename(filePath)} has ${lines} lines (>${threshold}). Consider splitting into smaller modules.`
+      additionalContext: `🕵️ Smith > File Size: ${basename(filePath)} has ${lines} lines (>${threshold}). Consider splitting into smaller modules.`
     }
   });
   process.stdout.write(output);

package/hooks/plan-guard.mjs

@@ -12,25 +12,18 @@ import { join, extname } from 'path';
 import { tmpdir } from 'os';
 import { getHookConfig } from '../lib/config.mjs';
 import { recordEvent } from '../lib/stats.mjs';
+import { appendEvent } from '../lib/event-log.mjs';
+import { sanitizeId, parseHookInput, isOmcAutoMode, notifyUser } from '../lib/hook-utils.mjs';
 
 const config = getHookConfig('plan-guard', process.cwd());
 if (!config.enabled) process.exit(0);
 
-let input;
-try {
-  input = JSON.parse(readFileSync(0, 'utf8'));
-} catch {
-  process.exit(0);
-}
+const input = parseHookInput();
+if (!input) process.exit(0);
 
 const filePath = input.tool_input?.file_path || '';
 if (!filePath) process.exit(0);
 
-function sanitizeId(id) {
-  if (!id || typeof id !== 'string') return 'default';
-  return id.replace(/[^a-zA-Z0-9_-]/g, '').slice(0, 128) || 'default';
-}
-
 const sessionId = sanitizeId(input.session_id);
 const agentId = sanitizeId(input.agent_id || '');
 const stateKey = agentId && agentId !== 'default' ? `${sessionId}-${agentId}` : sessionId;
@@ -45,6 +38,7 @@ if (isPostToolUse) {
   if (/plan/i.test(filePath) && /\.md$/.test(filePath)) {
     const planDetectedFile = join(stateDir, 'plan-detected');
     writeFileSync(planDetectedFile, String(Date.now()), { mode: 0o600 });
+    appendEvent(process.cwd(), { hook: 'plan-guard', event: 'fire', file: filePath, message: 'Plan file detected' });
   }
   process.exit(0);
 }
@@ -90,32 +84,28 @@ count++;
 writeFileSync(counterFile, String(count), { mode: 0o600 });
 
 // Raise threshold during OMC autonomous modes
-const omcStateDir = join(process.cwd(), '.omc', 'state');
-const omcAutoModes = ['autopilot-state.json', 'ultrawork-state.json', 'ralph-state.json'];
-const isOmcAutoMode = omcAutoModes.some(f => {
-  try {
-    const state = JSON.parse(readFileSync(join(omcStateDir, f), 'utf8'));
-    return state.active === true || state.status === 'running';
-  } catch { return false; }
-});
-
-const warnAt = isOmcAutoMode ? (config.warnAt || 5) * 2 : (config.warnAt || 5);
-const blockAt = isOmcAutoMode ? (config.blockAt || 10) * 2 : (config.blockAt || 10);
+const isOmc = isOmcAutoMode(process.cwd());
+const warnAt = isOmc ? (config.warnAt || 5) * 2 : (config.warnAt || 5);
+const blockAt = isOmc ? (config.blockAt || 10) * 2 : (config.blockAt || 10);
 
 // Block if threshold reached
 if (count >= blockAt) {
   recordEvent(sessionId, 'plan-guard', 'block');
-  process.stderr.write(`🔨 Smith ✋ Plan Guard: ${count} code files edited without a decomposition plan. Create a plan first.`);
+  appendEvent(process.cwd(), { hook: 'plan-guard', event: 'block', message: `${count} files without plan` });
+  notifyUser(config.notify, 'Plan Guard', 'block', `${count}개 파일 계획 없이 편집 — 차단`);
+  process.stderr.write(`🕵️ Smith ✋ Plan Guard: ${count} code files edited without a decomposition plan. Create a plan first.`);
   process.exit(2);
 }
 
 // Warn if approaching threshold
 if (count >= warnAt) {
   recordEvent(sessionId, 'plan-guard', 'warn');
+  appendEvent(process.cwd(), { hook: 'plan-guard', event: 'warn', message: `${count} files without plan` });
+  notifyUser(config.notify, 'Plan Guard', 'warn', `${count}개 파일 계획 없이 편집`);
   const output = JSON.stringify({
     hookSpecificOutput: {
       hookEventName: "PreToolUse",
-      additionalContext: `🔨 Smith > Plan Guard: ${count} code files edited without a plan. Complex work needs decomposition first.
+      additionalContext: `🕵️ Smith > Plan Guard: ${count} code files edited without a plan. Complex work needs decomposition first.
 💡 Create a plan file (docs/plans/*.md or plan.md) that includes:
 1. Each unit's responsibility (one sentence)
 2. Dependency direction between units

package/hooks/scope-guard.mjs

@@ -10,16 +10,14 @@ import { join, dirname } from 'path';
 import { tmpdir } from 'os';
 import { getHookConfig } from '../lib/config.mjs';
 import { recordEvent } from '../lib/stats.mjs';
+import { appendEvent } from '../lib/event-log.mjs';
+import { sanitizeId, parseHookInput, isOmcAutoMode, notifyUser } from '../lib/hook-utils.mjs';
 
 const config = getHookConfig('scope-guard', process.cwd());
 if (!config.enabled) process.exit(0);
 
-let input;
-try {
-  input = JSON.parse(readFileSync(0, 'utf8'));
-} catch {
-  process.exit(0);
-}
+const input = parseHookInput();
+if (!input) process.exit(0);
 
 const filePath = input.tool_input?.file_path || '';
 
@@ -27,11 +25,6 @@ if (!['Edit', 'Write'].includes(input.tool_name)) process.exit(0);
 if (!filePath) process.exit(0);
 if (!/\.(ts|tsx|js|jsx|py|go|rs|css|json)$/.test(filePath)) process.exit(0);
 
-function sanitizeId(id) {
-  if (!id || typeof id !== 'string') return 'default';
-  return id.replace(/[^a-zA-Z0-9_-]/g, '').slice(0, 128) || 'default';
-}
-
 // Isolate state per agent when sub-agents provide their own ID
 const sessionId = sanitizeId(input.session_id);
 const agentId = sanitizeId(input.agent_id || '');
@@ -39,16 +32,6 @@ const stateKey = agentId && agentId !== 'default' ? `${sessionId}-${agentId}` :
 const stateDir = join(tmpdir(), '.claude-smith', stateKey);
 mkdirSync(stateDir, { recursive: true, mode: 0o700 });
 
-// Raise threshold during OMC autonomous modes to avoid false positives in parallel execution
-const omcStateDir = join(process.cwd(), '.omc', 'state');
-const omcAutoModes = ['autopilot-state.json', 'ultrawork-state.json', 'ralph-state.json'];
-const isOmcAutoMode = omcAutoModes.some(f => {
-  try {
-    const state = JSON.parse(readFileSync(join(omcStateDir, f), 'utf8'));
-    return state.active === true || state.status === 'running';
-  } catch { return false; }
-});
-
 const stateFile = join(stateDir, 'scope-directories');
 const dir = dirname(filePath);
 
@@ -66,14 +49,17 @@ if (!dirs.includes(dir)) {
   writeFileSync(stateFile, JSON.stringify(dirs), { mode: 0o600 });
 }
 
-const threshold = isOmcAutoMode ? Math.max((config.maxDirectories || 5) * 2, 10) : (config.maxDirectories || 5);
+// Raise threshold during OMC autonomous modes to avoid false positives in parallel execution
+const threshold = isOmcAutoMode(process.cwd()) ? Math.max((config.maxDirectories || 5) * 2, 10) : (config.maxDirectories || 5);
 
 if (dirs.length >= threshold && (dirs.length === threshold || dirs.length % threshold === 0)) {
   recordEvent(sessionId, 'scope-guard', 'warn');
+  appendEvent(process.cwd(), { hook: 'scope-guard', event: 'warn', message: `${dirs.length} directories` });
+  notifyUser(config.notify, 'Scope Guard', 'warn', `${dirs.length}개 디렉토리 변경`);
   const output = JSON.stringify({
     hookSpecificOutput: {
       hookEventName: "PostToolUse",
-      additionalContext: `🔨 Smith > Scope Guard: Changes span ${dirs.length} directories (>=${threshold}). Is the scope too broad? Consider breaking into smaller tasks.`
+      additionalContext: `🕵️ Smith > Scope Guard: Changes span ${dirs.length} directories (>=${threshold}). Is the scope too broad? Consider breaking into smaller tasks.`
     }
   });
   process.stdout.write(output);

package/hooks/subagent-inject.mjs

@@ -5,29 +5,23 @@
  * Injects core TDD/verification rules into subagent context
  */
 
-import { readFileSync } from 'fs';
 import { getHookConfig } from '../lib/config.mjs';
 import { recordEvent } from '../lib/stats.mjs';
+import { appendEvent } from '../lib/event-log.mjs';
+import { sanitizeId, parseHookInput, notifyUser } from '../lib/hook-utils.mjs';
 
 const config = getHookConfig('subagent-inject', process.cwd());
 if (!config.enabled) process.exit(0);
 
 // Consume stdin (required by hook protocol)
-let input;
-try {
-  input = JSON.parse(readFileSync(0, 'utf8'));
-} catch {
-  process.exit(0);
-}
-
-function sanitizeId(id) {
-  if (!id || typeof id !== 'string') return 'default';
-  return id.replace(/[^a-zA-Z0-9_-]/g, '').slice(0, 128) || 'default';
-}
+const input = parseHookInput();
+if (!input) process.exit(0);
 
 const sessionId = sanitizeId(input.session_id);
 
 recordEvent(sessionId, 'subagent-inject', 'fire');
+appendEvent(process.cwd(), { hook: 'subagent-inject', event: 'fire', message: 'Rules injected into subagent' });
+notifyUser(config.notify, 'Subagent Inject', 'inject', '규칙 주입됨');
 const output = JSON.stringify({
   hookSpecificOutput: {
     hookEventName: "SubagentStart",

package/hooks/tdd-guard.mjs

@@ -5,20 +5,18 @@
  * Warns when editing a source file that has no corresponding test file
  */
 
-import { readFileSync, existsSync } from 'fs';
+import { existsSync } from 'fs';
 import { dirname, basename, join, extname } from 'path';
 import { getHookConfig } from '../lib/config.mjs';
 import { recordEvent } from '../lib/stats.mjs';
+import { appendEvent } from '../lib/event-log.mjs';
+import { sanitizeId, parseHookInput, notifyUser } from '../lib/hook-utils.mjs';
 
 const config = getHookConfig('tdd-guard', process.cwd());
 if (!config.enabled) process.exit(0);
 
-let input;
-try {
-  input = JSON.parse(readFileSync(0, 'utf8'));
-} catch {
-  process.exit(0);
-}
+const input = parseHookInput();
+if (!input) process.exit(0);
 
 const filePath = input.tool_input?.file_path || '';
 
@@ -41,11 +39,6 @@ if (/node_modules|\.next|dist|build|\.storybook/.test(filePath)) process.exit(0)
 // Skip layout, page files (Next.js App Router)
 if (/\/(layout|page|loading|error|not-found)\.(ts|tsx)$/.test(filePath)) process.exit(0);
 
-function sanitizeId(id) {
-  if (!id || typeof id !== 'string') return 'default';
-  return id.replace(/[^a-zA-Z0-9_-]/g, '').slice(0, 128) || 'default';
-}
-
 const sessionId = sanitizeId(input.session_id);
 
 const dir = dirname(filePath);
@@ -59,10 +52,12 @@ const testFileTs = join(dir, `${name}.test.ts`);
 
 if (!existsSync(testFile) && !existsSync(specFile) && !existsSync(testFileTsx) && !existsSync(testFileTs)) {
   recordEvent(sessionId, 'tdd-guard', 'warn');
+  appendEvent(process.cwd(), { hook: 'tdd-guard', event: 'warn', file: filePath, message: `No test file for ${basename(filePath)}` });
+  notifyUser(config.notify, 'TDD Guard', 'warn', `${basename(filePath)} 테스트 파일 없음`);
   const output = JSON.stringify({
     hookSpecificOutput: {
       hookEventName: "PreToolUse",
-      additionalContext: `🔨 Smith > TDD Guard: ${basename(filePath)} has no test file (${name}.test${ext}). TDD Iron Law: write failing test FIRST.\n💡 Coaching: 1) Create ${name}.test${ext} in the same directory. 2) Write a test that describes expected behavior. 3) Run the test to confirm it fails. 4) Then implement the code.`
+      additionalContext: `🕵️ Smith > TDD Guard: ${basename(filePath)} has no test file (${name}.test${ext}). TDD Iron Law: write failing test FIRST.\n💡 Coaching: 1) Create ${name}.test${ext} in the same directory. 2) Write a test that describes expected behavior. 3) Run the test to confirm it fails. 4) Then implement the code.`
     }
   });
   process.stdout.write(output);

package/hooks/test-tracker.mjs

@@ -10,16 +10,14 @@ import { join } from 'path';
 import { tmpdir } from 'os';
 import { getHookConfig } from '../lib/config.mjs';
 import { recordEvent } from '../lib/stats.mjs';
+import { appendEvent } from '../lib/event-log.mjs';
+import { sanitizeId, parseHookInput, notifyUser } from '../lib/hook-utils.mjs';
 
 const config = getHookConfig('test-tracker', process.cwd());
 if (!config.enabled) process.exit(0);
 
-let input;
-try {
-  input = JSON.parse(readFileSync(0, 'utf8'));
-} catch {
-  process.exit(0);
-}
+const input = parseHookInput();
+if (!input) process.exit(0);
 
 const toolName = input.tool_name;
 const command = input.tool_input?.command || '';
@@ -28,11 +26,6 @@ if (toolName !== 'Bash') process.exit(0);
 
 // Detect test commands
 if (/pnpm test|npm test|yarn test|bun test|vitest|jest|pytest|go test|cargo test|make test|mocha/.test(command)) {
-  function sanitizeId(id) {
-    if (!id || typeof id !== 'string') return 'default';
-    return id.replace(/[^a-zA-Z0-9_-]/g, '').slice(0, 128) || 'default';
-  }
-
   const sessionId = sanitizeId(input.session_id);
   const stateDir = join(tmpdir(), '.claude-smith', sessionId);
   mkdirSync(stateDir, { recursive: true, mode: 0o700 });
@@ -45,8 +38,11 @@ if (/pnpm test|npm test|yarn test|bun test|vitest|jest|pytest|go test|cargo test
   const response = JSON.stringify(input.tool_response || '').toLowerCase();
   const hasNonZeroFails = /[1-9]\d*\s+fail(?:ed|ing|ures?)?\b/i.test(response);
   const hasErrors = /[1-9]\d*\s+errors?\b/i.test(response) && !/0\s+errors?\b/.test(response);
-  writeFileSync(join(stateDir, 'last-test-result'), (hasNonZeroFails || hasErrors) ? 'fail' : 'pass', { mode: 0o600 });
+  const failed = hasNonZeroFails || hasErrors;
+  writeFileSync(join(stateDir, 'last-test-result'), failed ? 'fail' : 'pass', { mode: 0o600 });
   recordEvent(sessionId, 'test-tracker', 'fire');
+  appendEvent(process.cwd(), { hook: 'test-tracker', event: 'fire', message: `${failed ? 'fail' : 'pass'}` });
+  notifyUser(config.notify, 'Test Tracker', 'track', failed ? '❌ fail' : '✅ pass');
 
   // Extract coverage percentage from test output
   function parseCoverage(text) {
@@ -75,10 +71,11 @@ if (/pnpm test|npm test|yarn test|bun test|vitest|jest|pytest|go test|cargo test
 
     if (prevCoverage !== null && coverage < prevCoverage) {
       const delta = (coverage - prevCoverage).toFixed(1);
+      appendEvent(process.cwd(), { hook: 'test-tracker', event: 'warn', message: `Coverage: ${prevCoverage}% → ${coverage}%` });
       const output = JSON.stringify({
         hookSpecificOutput: {
           hookEventName: "PostToolUse",
-          additionalContext: `🔨 Smith 📊 Coverage Delta: ${prevCoverage}% → ${coverage}% (${delta}%). Test coverage decreased.\n💡 Coaching: 1) Check which lines lost coverage. 2) Add tests for uncovered paths. 3) Run coverage report: your-test-cmd --coverage`
+          additionalContext: `🕵️ Smith 📊 Coverage Delta: ${prevCoverage}% → ${coverage}% (${delta}%). Test coverage decreased.\n💡 Coaching: 1) Check which lines lost coverage. 2) Add tests for uncovered paths. 3) Run coverage report: your-test-cmd --coverage`
         }
       });
       process.stdout.write(output);

package/lib/config.mjs

@@ -9,6 +9,7 @@ import { join } from 'path';
 const DEFAULTS = {
   language: 'en',
   omcCompat: false,
+  notify: false,
   hooks: {
     'tdd-guard': { enabled: true },
     'commit-guard': { enabled: true, maxTestAge: 300 },
@@ -68,7 +69,9 @@ export function loadConfig(projectRoot) {
 
 export function getHookConfig(hookName, projectRoot) {
   const config = loadConfig(projectRoot);
-  return config.hooks?.[hookName] || DEFAULTS.hooks[hookName] || { enabled: true };
+  const hookConfig = config.hooks?.[hookName] || DEFAULTS.hooks[hookName] || { enabled: true };
+  hookConfig.notify = config.notify || false;
+  return hookConfig;
 }
 
 const DANGEROUS_KEYS = new Set(['__proto__', 'constructor', 'prototype']);