oh-my-harness 0.10.0 → 0.10.2

package/dist/catalog/blocks/tdd-guard.js

@@ -44,17 +44,14 @@ SRC_RE='{{{srcPattern}}}'
 TEST_RE="\${TEST_RE//\\\\\\\\/\\\\}"
 SRC_RE="\${SRC_RE//\\\\\\\\/\\\\}"
 if [[ "\$FILE_PATH" =~ \$TEST_RE ]]; then
-  # 테스트 파일 수정 → 기록 + 통과 (flock으로 원자적 읽기/쓰기)
-  (
-    flock -x 200
-    if [[ ! -f "\$HISTORY_FILE" ]]; then
-      echo '{"edits":[]}' > "\$HISTORY_FILE"
-    fi
-    UPDATED=$(jq --arg f "\$FILE_PATH" '.edits += [$f] | .edits |= unique' "\$HISTORY_FILE" 2>/dev/null) || true
-    if [[ -n "\$UPDATED" ]]; then
-      echo "\$UPDATED" > "\$HISTORY_FILE"
-    fi
-  ) 200>"\$HISTORY_FILE.lock"
+  # 테스트 파일 수정 → 기록 + 통과 (tmp+mv로 원자적 쓰기, 크로스 플랫폼)
+  if [[ ! -f "\$HISTORY_FILE" ]]; then
+    echo '{"edits":[]}' > "\$HISTORY_FILE"
+  fi
+  UPDATED=$(jq --arg f "\$FILE_PATH" '.edits += [$f] | .edits |= unique' "\$HISTORY_FILE" 2>/dev/null) || true
+  if [[ -n "\$UPDATED" ]]; then
+    echo "\$UPDATED" > "\$HISTORY_FILE.tmp" && mv "\$HISTORY_FILE.tmp" "\$HISTORY_FILE"
+  fi
   exit 0
 fi
 
@@ -73,25 +70,20 @@ if [[ ! -f "\$HISTORY_FILE" ]]; then
   exit 0
 fi
 
-# edit-history에서 테스트 파일 검색 (flock으로 원자적 읽기/쓰기)
-DECISION=$(
-  (
-    flock -x 200
-    if jq -e --arg b "\$BASENAME" '.edits[] | select(contains($b) and (contains(".test.") or contains(".spec.") or contains("test_")))' "\$HISTORY_FILE" >/dev/null 2>&1; then
-      # 테스트 먼저 수정됨 → 매칭 테스트 기록 소비(제거) + 소스 기록 + 통과
-      UPDATED=$(jq --arg b "\$BASENAME" --arg f "\$FILE_PATH" '
-        .edits |= [.[] | select((contains($b) and (contains(".test.") or contains(".spec.") or contains("test_"))) | not)]
-        | .edits += [$f] | .edits |= unique
-      ' "\$HISTORY_FILE" 2>/dev/null) || true
-      if [[ -n "\$UPDATED" ]]; then
-        echo "\$UPDATED" > "\$HISTORY_FILE"
-      fi
-      echo "allow"
-    else
-      echo "block"
-    fi
-  ) 200>"\$HISTORY_FILE.lock"
-)
+# edit-history에서 테스트 파일 검색 (tmp+mv로 원자적 쓰기, 크로스 플랫폼)
+if jq -e --arg b "\$BASENAME" '.edits[] | select(contains($b) and (contains(".test.") or contains(".spec.") or contains("test_")))' "\$HISTORY_FILE" >/dev/null 2>&1; then
+  # 테스트 먼저 수정됨 → 매칭 테스트 기록 소비(제거) + 소스 기록 + 통과
+  UPDATED=$(jq --arg b "\$BASENAME" --arg f "\$FILE_PATH" '
+    .edits |= [.[] | select((contains($b) and (contains(".test.") or contains(".spec.") or contains("test_"))) | not)]
+    | .edits += [$f] | .edits |= unique
+  ' "\$HISTORY_FILE" 2>/dev/null) || true
+  if [[ -n "\$UPDATED" ]]; then
+    echo "\$UPDATED" > "\$HISTORY_FILE.tmp" && mv "\$HISTORY_FILE.tmp" "\$HISTORY_FILE"
+  fi
+  DECISION="allow"
+else
+  DECISION="block"
+fi
 
 if [[ "\$DECISION" == "allow" ]]; then
   exit 0

package/dist/cli/command-checker.js

@@ -61,7 +61,7 @@ export async function checkHarnessCommands(hooks, projectDir) {
     const fs = await import("node:fs/promises");
     const path = await import("node:path");
     for (const hook of hooks) {
-        const scriptPath = hook.command.replace(/^bash\s+/, "");
+        const scriptPath = hook.command.replace(/^bash\s+/, "").replace(/^"|"$/g, "");
         const fullPath = path.join(projectDir, scriptPath);
         let content;
         try {

package/dist/cli/event-logger.d.ts

@@ -2,7 +2,7 @@ export interface HookEvent {
     ts: string;
     event: string;
     hook: string;
-    decision: "block" | "allow";
+    decision: "block" | "allow" | "error";
     reason?: string;
     tool?: string;
 }
@@ -13,9 +13,11 @@ export interface EventStats {
     totalEvents: number;
     blockCount: number;
     allowCount: number;
+    errorCount: number;
     byHook: Record<string, {
         block: number;
         allow: number;
+        error: number;
     }>;
 }
 export declare function aggregateStats(events: HookEvent[]): EventStats;

package/dist/cli/event-logger.js

@@ -27,7 +27,7 @@ export async function readEvents(projectDir) {
             if (typeof parsed.ts === "string" &&
                 typeof parsed.event === "string" &&
                 typeof parsed.hook === "string" &&
-                (parsed.decision === "block" || parsed.decision === "allow")) {
+                (parsed.decision === "block" || parsed.decision === "allow" || parsed.decision === "error")) {
                 events.push(parsed);
             }
         }
@@ -48,15 +48,18 @@ export function aggregateStats(events) {
         totalEvents: events.length,
         blockCount: 0,
         allowCount: 0,
+        errorCount: 0,
         byHook: {},
     };
     for (const e of events) {
         if (e.decision === "block")
             stats.blockCount++;
+        else if (e.decision === "error")
+            stats.errorCount++;
         else
             stats.allowCount++;
         if (!stats.byHook[e.hook]) {
-            stats.byHook[e.hook] = { block: 0, allow: 0 };
+            stats.byHook[e.hook] = { block: 0, allow: 0, error: 0 };
         }
         stats.byHook[e.hook][e.decision]++;
     }

package/dist/cli/harness-tester.js

@@ -125,11 +125,11 @@ export function generateBlockTestCases(hookEntries, blocks, currentBranch, regis
         };
         const aliases = blockIdAliases[block.id] ?? [block.id];
         const matchedHook = registeredHooks?.find((h) => {
-            const scriptName = h.command.replace(/^bash\s+/, "");
+            const scriptName = h.command.replace(/^bash\s+/, "").replace(/^"|"$/g, "");
             return aliases.some((alias) => scriptName.includes(alias));
         });
         const hookScript = matchedHook
-            ? matchedHook.command.replace(/^bash\s+/, "")
+            ? matchedHook.command.replace(/^bash\s+/, "").replace(/^"|"$/g, "")
             : `.claude/hooks/catalog-${block.id}.sh`;
         switch (block.id) {
             case "path-guard": {

package/dist/generators/hooks.d.ts

@@ -1,5 +1,5 @@
 import type { MergedConfig } from "../core/preset-types.js";
-export declare function wrapWithLogger(script: string, event?: string): string;
+export declare function wrapWithLogger(script: string, event?: string, projectDir?: string): string;
 export interface GenerateHooksOptions {
     projectDir: string;
     config: MergedConfig;

package/dist/generators/hooks.js

@@ -1,8 +1,11 @@
 import { mkdir, writeFile, chmod, readFile, unlink } from "node:fs/promises";
 import { join } from "node:path";
-function buildLoggerSnippet(event) {
+function buildLoggerSnippet(event, projectDir) {
+    const stateDir = projectDir
+        ? `${projectDir}/.claude/hooks/.state`
+        : ".claude/hooks/.state";
     return `# --- oh-my-harness event logger ---
-_OMH_STATE_DIR=".claude/hooks/.state"
+_OMH_STATE_DIR="${stateDir}"
 mkdir -p "$_OMH_STATE_DIR" 2>/dev/null || true
 _OMH_HOOK_NAME="$(basename "$0")"
 _OMH_EVENT="${event}"
@@ -14,11 +17,11 @@ _log_event() {
     "$(date -u +%Y-%m-%dT%H:%M:%SZ)" "$_OMH_EVENT" "$_OMH_HOOK_NAME" "$decision" "$reason" \\
     >> "$_OMH_STATE_DIR/events.jsonl"
 }
-trap '[ "$_OMH_LOGGED" -eq 0 ] && _log_event "allow"' EXIT
+trap '_OMH_EXIT_CODE=$?; if [ "$_OMH_LOGGED" -eq 0 ]; then if [ "$_OMH_EXIT_CODE" -ne 0 ]; then _log_event "error" "hook exited with code $_OMH_EXIT_CODE"; else _log_event "allow"; fi; fi' EXIT
 # --- end logger ---`;
 }
-export function wrapWithLogger(script, event = "unknown") {
-    const snippet = buildLoggerSnippet(event);
+export function wrapWithLogger(script, event = "unknown", projectDir) {
+    const snippet = buildLoggerSnippet(event, projectDir);
     if (script.includes("INPUT=$(cat)")) {
         return script.replace("INPUT=$(cat)", `INPUT=$(cat)\n\n${snippet}`);
     }
@@ -108,13 +111,13 @@ export async function generateHooks(options) {
         }
         usedScriptNames.add(scriptName);
         const scriptPath = join(hooksDir, scriptName);
-        const wrappedScript = wrapWithLogger(hook.inline, hook.event);
+        const wrappedScript = wrapWithLogger(hook.inline, hook.event, projectDir);
         await writeFile(scriptPath, wrappedScript, "utf8");
         await chmod(scriptPath, 0o755);
         generatedFiles.push(scriptPath);
         const entry = {
             matcher: hook.matcher,
-            hooks: [{ type: "command", command: `bash .claude/hooks/${scriptName}` }],
+            hooks: [{ type: "command", command: `bash "${scriptPath}"` }],
         };
         if (!hooksConfig[hook.event]) {
             hooksConfig[hook.event] = [];

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "oh-my-harness",
-  "version": "0.10.0",
+  "version": "0.10.2",
   "description": "Tame your AI coding agents with natural language",
   "type": "module",
   "bin": {