claude-smith 3.1.0 → 3.3.0

package/README.ko.md

@@ -1,8 +1,8 @@
 [English](README.md) | **한국어**
 
-# 🔨 claude-smith
+# 🕵️ claude-smith
 
-> **Claude Code의 ESLint** — AI 코딩 세션에 물리적 강제를 더합니다.
+> **코드의 조용한 수호자** — AI 코딩 세션의 물리적 규율 강제
 
 ## 문제
 
@@ -39,6 +39,24 @@ npx claude-smith init
 
 언어를 선택하면(en/ko/zh/ja) 끝. 12개 Hook 설치, 규칙 주입, 강제 활성화.
 
+## 실시간 알림
+
+Claude Code 세션 중 Hook 활동을 실시간으로 확인하세요:
+
+```json
+{ "notify": true }
+```
+
+활성화하면 터미널에서 간략한 상태 메시지를 볼 수 있습니다:
+```
+🕵️ Smith — ⚠️ TDD Guard: LoginForm.tsx 테스트 파일 없음
+🕵️ Smith — ✅ Commit Guard: 커밋 허용
+🕵️ Smith — 🚫 Debug Loop: app.tsx 5회 편집 차단
+🕵️ Smith — 🤖 Subagent Inject: 규칙 주입됨
+```
+
+`"notify": false` (기본값)로 설정하면 비활성화됩니다.
+
 ## 12개 Hook
 
 ### 차단하는 Hook (exit code 2 — 진행 자체를 막음)
@@ -124,6 +142,7 @@ Claude Code 세션
 | `/smith-report` | 상세 규정 준수 리포트 |
 | `/smith-check` | 설치 검증 |
 | `/smith-plan` | 분해 계획 템플릿 생성 |
+| `/smith-decompose` | 복잡한 작업을 위한 5단계 가이드 분해 |
 | `/smith-update` | 버전 체크 + 업그레이드 |
 
 ## 명령어
@@ -148,6 +167,7 @@ claude-smith uninstall             # 모든 구성 요소 제거
 ```json
 {
   "language": "ko",
+  "notify": false,
   "hooks": {
     "commit-guard": { "maxTestAge": 300 },
     "debug-loop": { "warnAt": 3, "blockAt": 5 },
@@ -187,13 +207,36 @@ claude-smith uninstall             # 모든 구성 요소 제거
 - **안전한 권한**: 디렉터리 `0o700`, 파일 `0o600`
 - **경쟁 조건 방지**: Hook별 원자적 카운터 (공유 JSON 미사용)
 - **제로 의존성**: Node.js 기본 모듈만 사용
-- **63개 테스트**: 보안 엣지 케이스, 에러 처리, 크로스 플랫폼 시나리오
+- **포괄적 테스트 스위트**: 보안 엣지 케이스, 에러 처리, 크로스 플랫폼 시나리오
 
 ## 요구 사항
 
 - Node.js >= 18
 - Hook을 지원하는 Claude Code (`.claude/settings.json`)
 
+## 문제 해결
+
+### Hook이 작동하지 않음
+1. 설치 검증: `claude-smith check`
+2. `.claude/settings.json`에 Hook 정의가 있는지 확인
+3. Node.js >= 18 확인: `node --version`
+
+### 병렬 에이전트에서 오탐 발생
+`.claude-smith.json`에서 OMC 호환 모드 활성화:
+```json
+{ "omcCompat": true }
+```
+debug-loop, batch-checkpoint, scope-guard, plan-guard 임계값이 상향됩니다.
+
+### 테스트가 "실행 안 됨"으로 표시됨
+test-tracker Hook은 Bash 도구 호출의 테스트 명령만 감지합니다. 인라인이 아닌 터미널을 통해 테스트를 실행하세요.
+
+### Hook 에러
+개별 Hook을 수동으로 실행하여 디버깅:
+```bash
+echo '{"tool_name":"Edit","tool_input":{"file_path":"test.ts"},"session_id":"debug"}' | node .claude/hooks/tdd-guard.mjs
+```
+
 ## 라이선스
 
 MIT

package/README.md

@@ -1,8 +1,8 @@
 **English** | [한국어](README.ko.md)
 
-# 🔨 claude-smith
+# 🕵️ claude-smith
 
-> **Claude Code's ESLint** — Physical enforcement for AI coding sessions.
+> **Your code's silent guardian** — Physical enforcement for AI coding sessions.
 
 ## The Problem
 
@@ -39,6 +39,24 @@ npx claude-smith init
 
 Select your language (en/ko/zh/ja), and you're done. 12 hooks installed, rules injected, enforcement active.
 
+## Live Notifications
+
+See hook activity in real-time during Claude Code sessions:
+
+```json
+{ "notify": true }
+```
+
+When enabled, hooks write brief status messages visible in your terminal:
+```
+🕵️ Smith — ⚠️ TDD Guard: LoginForm.tsx test file missing
+🕵️ Smith — ✅ Commit Guard: Commit allowed
+🕵️ Smith — 🚫 Debug Loop: app.tsx 5 edits blocked
+🕵️ Smith — 🤖 Subagent Inject: Rules injected
+```
+
+Set `"notify": false` (default) to disable.
+
 ## The 12 Hooks
 
 ### Hooks That Block (exit code 2 — action physically prevented)
@@ -124,6 +142,7 @@ After installation, these commands are available in Claude Code:
 | `/smith-report` | Detailed compliance report |
 | `/smith-check` | Validate installation |
 | `/smith-plan` | Generate decomposition plan template |
+| `/smith-decompose` | Guided 5-step problem decomposition for complex tasks |
 | `/smith-update` | Check for updates and upgrade |
 
 ## Commands
@@ -148,6 +167,7 @@ Project-level settings in `.claude-smith.json` (all optional):
 ```json
 {
   "language": "en",
+  "notify": false,
   "hooks": {
     "commit-guard": { "maxTestAge": 300 },
     "debug-loop": { "warnAt": 3, "blockAt": 5 },
@@ -187,13 +207,36 @@ User settings always take priority over `omcCompat` defaults.
 - **Secure permissions**: Directories `0o700`, files `0o600`
 - **Race-condition safe**: Per-hook atomic counters (no shared JSON)
 - **Zero dependencies**: Node.js built-in modules only
-- **63 tests**: Security edge cases, error handling, cross-platform scenarios
+- **Comprehensive test suite**: Security edge cases, error handling, cross-platform scenarios
 
 ## Requirements
 
 - Node.js >= 18
 - Claude Code with hook support (`.claude/settings.json`)
 
+## Troubleshooting
+
+### Hooks not firing
+1. Verify installation: `claude-smith check`
+2. Check `.claude/settings.json` contains hook definitions
+3. Ensure Node.js >= 18: `node --version`
+
+### False positives in parallel agents
+Enable OMC compatibility mode in `.claude-smith.json`:
+```json
+{ "omcCompat": true }
+```
+This raises thresholds for debug-loop, batch-checkpoint, scope-guard, and plan-guard.
+
+### Tests show as "not run"
+The test-tracker hook only detects test commands in Bash tool calls. Ensure you run tests via the terminal, not inline.
+
+### Hook errors
+Run individual hooks manually to debug:
+```bash
+echo '{"tool_name":"Edit","tool_input":{"file_path":"test.ts"},"session_id":"debug"}' | node .claude/hooks/tdd-guard.mjs
+```
+
 ## License
 
 MIT

package/bin/cli.mjs

@@ -79,7 +79,7 @@ switch (command) {
 
 async function init() {
   const cwd = process.cwd();
-  console.log(`\n🔨 Smith v${VERSION} - init\n`);
+  console.log(`\n🕵️ Smith v${VERSION} - init\n`);
 
   // 0. Determine language
   const lang = await resolveLanguage();
@@ -265,7 +265,7 @@ async function resolveLanguage() {
 
 function update() {
   const cwd = process.cwd();
-  console.log(`\n🔨 Smith v${VERSION} - update\n`);
+  console.log(`\n🕵️ Smith v${VERSION} - update\n`);
 
   // 1. Overwrite hooks (always latest)
   const hooksDir = join(cwd, '.claude', 'hooks');
@@ -319,7 +319,8 @@ function injectRules(claudeMdPath, lang = 'en') {
   const fallback = join(TEMPLATES_DIR, 'rules.en.md');
   const rulesPath = existsSync(rulesFile) ? rulesFile : fallback;
   const rules = readFileSync(rulesPath, 'utf8')
-    .replace(/claude-smith v\d+\.\d+\.\d+/g, `claude-smith v${VERSION}`);
+    .replace(/claude-smith v\d+\.\d+\.\d+/g, `claude-smith v${VERSION}`)
+    .replace(/\{\{SMITH_VERSION\}\}/g, `v${VERSION}`);
 
   if (existsSync(claudeMdPath)) {
     let content = readFileSync(claudeMdPath, 'utf8');
@@ -385,7 +386,7 @@ function check() {
   const ciMode = process.argv.includes('--ci');
   const results = { version: VERSION, ok: true, hooks: {}, settings: false, rules: false };
 
-  if (!ciMode) console.log(`\n🔨 Smith v${VERSION} - check\n`);
+  if (!ciMode) console.log(`\n🕵️ Smith v${VERSION} - check\n`);
 
   // Check hooks
   const hooksDir = join(cwd, '.claude', 'hooks');
@@ -402,14 +403,14 @@ function check() {
 
   // Check commands
   const commandsDir = join(cwd, '.claude', 'commands');
-  const expectedCommands = ['smith.md', 'smith-report.md', 'smith-check.md', 'smith-plan.md'];
+  const expectedCommands = ['smith.md', 'smith-report.md', 'smith-check.md', 'smith-plan.md', 'smith-decompose.md'];
   const missingCommands = expectedCommands.filter(c => !existsSync(join(commandsDir, c)));
-  results.commands = { installed: 4 - missingCommands.length, total: 4, missing: missingCommands };
+  results.commands = { installed: 5 - missingCommands.length, total: 5, missing: missingCommands };
 
   if (!ciMode) {
     console.log(missingCommands.length === 0
-      ? `✅ Commands: 4/4 installed`
-      : `❌ Commands: ${4 - missingCommands.length}/4 (missing: ${missingCommands.join(', ')})`);
+      ? `✅ Commands: 5/5 installed`
+      : `❌ Commands: ${5 - missingCommands.length}/5 (missing: ${missingCommands.join(', ')})`);
   }
   if (missingCommands.length > 0) results.ok = false;
 
@@ -448,7 +449,7 @@ function check() {
 }
 
 function stats() {
-  console.log(`\n🔨 Smith v${VERSION} - stats\n`);
+  console.log(`\n🕵️ Smith v${VERSION} - stats\n`);
 
   // Try to find stats from any active session
   const baseDir = join(tmpdir(), '.claude-smith');
@@ -500,7 +501,7 @@ function stats() {
 
 async function uninstall() {
   const cwd = process.cwd();
-  console.log(`\n🔨 Smith v${VERSION} - uninstall\n`);
+  console.log(`\n🕵️ Smith v${VERSION} - uninstall\n`);
 
   const SMITH_HOOKS = ['tdd-guard.mjs', 'commit-guard.mjs', 'test-tracker.mjs', 'debug-loop.mjs',
     'batch-checkpoint.mjs', 'subagent-inject.mjs', 'commit-message.mjs', 'build-guard.mjs',
@@ -595,7 +596,7 @@ async function uninstall() {
 }
 
 function report() {
-  console.log(`\n🔨 Smith v${VERSION} - session report\n`);
+  console.log(`\n🕵️ Smith v${VERSION} - session report\n`);
 
   const baseDir = join(tmpdir(), '.claude-smith');
   if (!existsSync(baseDir)) {
@@ -784,7 +785,7 @@ ${!isPreTool ? "const toolResponse = input.tool_response || {};\n" : ""}
     writeFileSync(settingsPath, JSON.stringify(settings, null, 2) + '\n');
   }
 
-  console.log(`\n🔨 Smith v${VERSION} - create-hook\n`);
+  console.log(`\n🕵️ Smith v${VERSION} - create-hook\n`);
   console.log(`✅ Hook created → .claude/hooks/${fileName}`);
   console.log(`✅ Registered in .claude/settings.json (${event})`);
   console.log(`\nNext steps:`);
@@ -803,7 +804,7 @@ function escapeHtml(str) {
 }
 
 function dashboard() {
-  console.log(`\n🔨 Smith v${VERSION} - dashboard\n`);
+  console.log(`\n🕵️ Smith v${VERSION} - dashboard\n`);
 
   const baseDir = join(tmpdir(), '.claude-smith');
   if (!existsSync(baseDir)) {
@@ -1056,7 +1057,7 @@ function dashboard() {
 <head>
 <meta charset="UTF-8">
 <meta name="viewport" content="width=device-width, initial-scale=1.0">
-<title>🔨 Smith Dashboard</title>
+<title>🕵️ Smith Dashboard</title>
 <style>
   * { margin: 0; padding: 0; box-sizing: border-box; }
   body { font-family: -apple-system, BlinkMacSystemFont, 'Segoe UI', sans-serif; background: #0d1117; color: #c9d1d9; padding: 2rem; }
@@ -1113,7 +1114,7 @@ function dashboard() {
 </style>
 </head>
 <body>
-  <h1>🔨 Smith</h1>
+  <h1>🕵️ Smith</h1>
   <p class="subtitle">Session Protection Report — ${new Date().toISOString().split('T')[0]}</p>
 
   <div class="hero-grid">
@@ -1188,7 +1189,7 @@ ${hookNames.map((name, i) => {
 ${insightsHtml}
   </div>
 
-  <footer>Generated by 🔨 Smith v${escapeHtml(VERSION)} | ${escapeHtml(new Date().toISOString())}</footer>
+  <footer>Generated by 🕵️ Smith v${escapeHtml(VERSION)} | ${escapeHtml(new Date().toISOString())}</footer>
 </body>
 </html>`;
 
@@ -1200,7 +1201,7 @@ ${insightsHtml}
 
 function help() {
   console.log(`
-🔨 Smith v${VERSION}
+🕵️ Smith v${VERSION}
 Claude Code workflow enforcement CLI - forging coding discipline into every session
 
 Usage:

package/hooks/batch-checkpoint.mjs

@@ -5,34 +5,23 @@
  * Counts distinct files edited, reminds to report progress at threshold (default: 3)
  */
 
-import { readFileSync, writeFileSync, existsSync, mkdirSync } from 'fs';
+import { writeFileSync, existsSync, mkdirSync, readFileSync } 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, isOmcAutoMode, notifyUser } from '../lib/hook-utils.mjs';
 
 const config = getHookConfig('batch-checkpoint', process.cwd());
 if (!config.enabled) process.exit(0);
 
 // Skip in OMC autonomous execution modes (autopilot, ultrawork, ralph)
 // These modes manage their own progress reporting and checkpoint would interfere
-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) process.exit(0);
+if (isOmcAutoMode(process.cwd())) 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 || '';
@@ -43,11 +32,6 @@ if (!filePath) process.exit(0);
 // Skip non-source files
 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';
-}
-
 const sessionId = sanitizeId(input.session_id);
 const stateDir = join(tmpdir(), '.claude-smith', sessionId);
 mkdirSync(stateDir, { recursive: true, mode: 0o700 });
@@ -73,10 +57,11 @@ if (!editedFiles.includes(filePath)) {
 if (editedFiles.length > 0 && editedFiles.length % config.fileThreshold === 0) {
   recordEvent(sessionId, 'batch-checkpoint', 'warn');
   appendEvent(process.cwd(), { hook: 'batch-checkpoint', event: 'warn', message: `${editedFiles.length} files edited` });
+  notifyUser(config.notify, 'Batch Checkpoint', 'warn', `${editedFiles.length}개 파일 편집`);
   const output = JSON.stringify({
     hookSpecificOutput: {
       hookEventName: "PostToolUse",
-      additionalContext: `🔨 Smith 📋 Batch Checkpoint (rule 2-9): ${editedFiles.length} files edited. Report progress to user before continuing.`
+      additionalContext: `🕵️ Smith 📋 Batch Checkpoint (rule 2-9): ${editedFiles.length} files edited. Report progress to user before continuing.`
     }
   });
   process.stdout.write(output);

package/hooks/build-guard.mjs

@@ -11,16 +11,13 @@ 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-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 command = input.tool_input?.command || '';
 
@@ -28,11 +25,6 @@ if (input.tool_name !== 'Bash') process.exit(0);
 if (!command.includes('git commit')) process.exit(0);
 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 lastBuildResult = join(stateDir, 'last-build-result');
@@ -44,7 +36,8 @@ if (existsSync(lastBuildResult)) {
   if (result === 'fail') {
     recordEvent(sessionId, 'build-guard', 'block');
     appendEvent(process.cwd(), { hook: 'build-guard', event: 'block', message: 'Build failing' });
-    process.stderr.write("🔨 Smith ✋ Commit blocked: last build FAILED. Run build and fix errors before committing.");
+    notifyUser(config.notify, 'Build Guard', 'block', '빌드 실패 — 커밋 차단');
+    process.stderr.write("🕵️ Smith ✋ Commit blocked: last build FAILED. Run build and fix errors before committing.");
     process.exit(2);
   }
 }

package/hooks/build-tracker.mjs

@@ -5,22 +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 || '';
 
@@ -28,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 });
@@ -44,4 +36,5 @@ if (/pnpm build|npm run build|yarn build|tsc|next build/.test(command)) {
   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

@@ -13,16 +13,13 @@ 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 || '';
@@ -33,26 +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');
-appendEvent(process.cwd(), { hook: 'commit-guard', event: 'fire', message: 'Tests passed, commit allowed' });
-
 // 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');
     appendEvent(process.cwd(), { hook: 'commit-guard', event: 'block', message: 'Tests failing' });
-    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.");
+    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);
   }
 }
@@ -65,22 +55,31 @@ if (existsSync(lastTestFile)) {
   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,40 +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
@@ -47,11 +39,17 @@ 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', '형식 유효');
 }