oh-my-customcode 1.1.91 → 1.1.93

package/README.md

@@ -249,7 +249,7 @@ oh-my-customcode ships 42 lifecycle hook scripts covering security, drift detect
 
 Most hooks are advisory (exit 0) — they warn but never block. A small number of hard-block hooks (e.g. `stage-blocker`, `rule-deletion-guard`) reject the tool call outright (exit 2). oh-my-customcode's own governance rules (`.claude/rules/`) follow an **advisory-first enforcement model**: prompt-based guidance is the default, and a rule is only promoted to a blocking hook after repeated, observed violations. Rules also retire — clauses tied to since-fixed platform bugs or that go unused for two minor releases are wrapped in HTML comments (still readable via the source file, invisible to the agent's context) rather than left to accumulate indefinitely.
 
-The hook source of truth is `.claude/hooks/hooks.json`; `omcustom init` compiles it into the `hooks` block of `.claude/settings.json` (via `src/core/hooks-settings.ts`), which is the file Claude Code actually loads.
+The hook source of truth is `.claude/hooks/hooks.json`; `omcustom init` compiles it into the `hooks` block of `.claude/settings.json` (via `src/core/hooks-settings.ts`), which is the file Claude Code actually loads. Hook commands are anchored as `bash "${CLAUDE_PROJECT_DIR:-.}/.claude/hooks/<path>"`, so hooks keep working when the session's working directory moves to a subdirectory. Contributors: do not hand-edit the `hooks` blocks in settings; edit `hooks.json` and run `bun run sync:hooks` (also run at the end of `bun run build`) to regenerate them. Existing installs need no manual step: `omcustom update` (full or hooks component) and `omcustom init --backup` automatically migrate old relative hook commands in `.claude/settings.local.json`, and custom hooks are left untouched.
 
 ---
 

package/dist/cli/index.js

@@ -217,7 +217,7 @@ var init_package = __esm(() => {
     workspaces: [
       "packages/*"
     ],
-    version: "1.1.91",
+    version: "1.1.93",
     description: "Batteries-included agent harness for Claude Code",
     type: "module",
     bin: {
@@ -241,7 +241,8 @@ var init_package = __esm(() => {
     },
     scripts: {
       dev: "bun run src/cli/index.ts",
-      build: "bun build src/cli/index.ts --outdir dist/cli --target node && bun build src/index.ts --outdir dist --target node && bun run scripts/sync-source-lockfile.ts",
+      build: "bun build src/cli/index.ts --outdir dist/cli --target node && bun build src/index.ts --outdir dist --target node && bun run scripts/sync-source-lockfile.ts && bun run scripts/sync-hooks-settings.ts",
+      "sync:hooks": "bun run scripts/sync-hooks-settings.ts",
       test: "bun test tests/ packages/eval-core/",
       "test:unit": "bun test tests/unit",
       "test:integration": "bun test tests/integration",
@@ -15615,6 +15616,8 @@ var MESSAGES = {
     "install.entry_md_skipped": "{{entry}} skipped ({{reason}})",
     "install.lockfile_generated": "Lockfile generated ({{files}} files tracked)",
     "install.lockfile_failed": "Failed to generate lockfile: {{error}}",
+    "install.hook_commands_migrated": "Migrated {{rewritten}} hook command(s) in settings.local.json",
+    "install.hook_commands_migration_failed": "Failed to migrate hook commands in settings.local.json: {{error}}",
     "lockfile.not_found": "Lockfile not found: {{path}}",
     "lockfile.invalid_version": "Invalid lockfile version: {{path}}",
     "lockfile.invalid_structure": "Invalid lockfile structure: {{path}}",
@@ -15636,6 +15639,8 @@ var MESSAGES = {
     "update.lockfile_failed": "Failed to regenerate lockfile: {{error}}",
     "update.protected_file_updated": "⟳ Protected file {{file}} in {{component}} updated: {{hint}}",
     "update.namespace_synced": "Namespace synced: {{file}} ({{component}})",
+    "update.hook_commands_migrated": "Migrated {{count}} hook command(s) to CLAUDE_PROJECT_DIR-anchored paths",
+    "update.hook_commands_migration_failed": "Failed to migrate hook commands in {{path}}",
     "config.load_failed": "Failed to load config: {{error}}",
     "config.not_found": "Config not found at {{path}}, using defaults",
     "config.saved": "Config saved to {{path}}",
@@ -15661,6 +15666,8 @@ var MESSAGES = {
     "install.entry_md_skipped": "{{entry}} 건너뜀 ({{reason}})",
     "install.lockfile_generated": "잠금 파일 생성 완료 ({{files}}개 파일 추적)",
     "install.lockfile_failed": "잠금 파일 생성 실패: {{error}}",
+    "install.hook_commands_migrated": "settings.local.json의 훅 명령 {{rewritten}}개를 마이그레이션했습니다",
+    "install.hook_commands_migration_failed": "settings.local.json의 훅 명령 마이그레이션에 실패했습니다: {{error}}",
     "lockfile.not_found": "잠금 파일 없음: {{path}}",
     "lockfile.invalid_version": "잠금 파일 버전 유효하지 않음: {{path}}",
     "lockfile.invalid_structure": "잠금 파일 구조 유효하지 않음: {{path}}",
@@ -15682,6 +15689,8 @@ var MESSAGES = {
     "update.lockfile_failed": "잠금 파일 재생성 실패: {{error}}",
     "update.protected_file_updated": "⟳ 보호 파일 {{file}} ({{component}}) 업데이트됨: {{hint}}",
     "update.namespace_synced": "네임스페이스 동기화: {{file}} ({{component}})",
+    "update.hook_commands_migrated": "CLAUDE_PROJECT_DIR 기준 경로로 훅 명령 {{count}}개를 마이그레이션했습니다",
+    "update.hook_commands_migration_failed": "{{path}}의 훅 명령 마이그레이션에 실패했습니다",
     "config.load_failed": "설정 로드 실패: {{error}}",
     "config.not_found": "{{path}}에 설정 없음, 기본값 사용",
     "config.saved": "설정 저장: {{path}}",
@@ -17349,6 +17358,52 @@ function getDefaultWorkflow() {
   };
 }
 
+// src/core/hook-command-migration.ts
+var OLD_HOOK_COMMAND = /^(bash )?(?:\.\/)?\.claude\/hooks\/((?:scripts\/)?[A-Za-z0-9._-]+\.sh)$/;
+function rewriteRelativeHookCommand(command) {
+  const match = OLD_HOOK_COMMAND.exec(command);
+  if (match === null) {
+    return null;
+  }
+  const interpreter = match[1] ?? "";
+  return `${interpreter}"\${CLAUDE_PROJECT_DIR:-.}/.claude/hooks/${match[2]}"`;
+}
+function isRecord(value) {
+  return typeof value === "object" && value !== null && !Array.isArray(value);
+}
+function rewriteNode(node, counter) {
+  if (Array.isArray(node)) {
+    return node.map((item) => rewriteNode(item, counter));
+  }
+  if (!isRecord(node)) {
+    return node;
+  }
+  const out = {};
+  for (const [key, value] of Object.entries(node)) {
+    if (key === "command" && typeof value === "string") {
+      const rewritten = rewriteRelativeHookCommand(value);
+      if (rewritten !== null) {
+        counter.count++;
+        out[key] = rewritten;
+        continue;
+      }
+    }
+    out[key] = rewriteNode(value, counter);
+  }
+  return out;
+}
+function migrateHookCommands(settings) {
+  if (!isRecord(settings.hooks)) {
+    return { settings, rewritten: 0 };
+  }
+  const counter = { count: 0 };
+  const hooks = rewriteNode(settings.hooks, counter);
+  if (counter.count === 0) {
+    return { settings, rewritten: 0 };
+  }
+  return { settings: { ...settings, hooks }, rewritten: counter.count };
+}
+
 // src/core/hooks-settings.ts
 init_fs();
 import { dirname as dirname3 } from "node:path";
@@ -18036,6 +18091,32 @@ async function installHooksSettings(targetDir, _options, result) {
     warn("install.hooks_settings_failed", { error: message });
   }
 }
+var BOM = "\uFEFF";
+async function migrateRestoredHookCommands(targetDir, result) {
+  const layout = getProviderLayout();
+  const settingsPath = join8(targetDir, layout.rootDir, "settings.local.json");
+  if (!await fileExists(settingsPath)) {
+    return;
+  }
+  try {
+    const raw = await readTextFile(settingsPath);
+    const hasBom = raw.startsWith(BOM);
+    const trailingNewline = raw.endsWith(`
+`) ? `
+` : "";
+    const current = JSON.parse(hasBom ? raw.slice(BOM.length) : raw);
+    const { settings, rewritten } = migrateHookCommands(current);
+    if (rewritten > 0) {
+      const body = `${JSON.stringify(settings, null, 2)}${trailingNewline}`;
+      await writeTextFile(settingsPath, hasBom ? `${BOM}${body}` : body);
+      debug("install.hook_commands_migrated", { rewritten: String(rewritten) });
+    }
+  } catch (err) {
+    const message = err instanceof Error ? err.message : String(err);
+    result.warnings.push(`Failed to migrate hook commands in settings.local.json: ${message}`);
+    warn("install.hook_commands_migration_failed", { error: message });
+  }
+}
 async function installEntryDocWithTracking(targetDir, options, result) {
   const language = options.language ?? DEFAULT_LANGUAGE2;
   const overwrite = !!(options.force || options.backup);
@@ -18111,6 +18192,7 @@ async function install(options) {
           result.warnings.push(`Failed to restore ${failure.path}: ${failure.reason}`);
         }
       }
+      await migrateRestoredHookCommands(options.targetDir, result);
       await cleanupPreservation(preservation.tempDir);
     }
     await updateInstallConfig(options.targetDir, options, result.installedComponents);
@@ -20948,6 +21030,37 @@ async function backfillStatusLineRefreshInterval(targetDir, options) {
     });
   }
 }
+async function migrateHookCommandsInSettingsLocal(targetDir, options) {
+  if (options.dryRun) {
+    return;
+  }
+  const layout = getProviderLayout();
+  const settingsPath = join18(targetDir, layout.rootDir, "settings.local.json");
+  if (!await fileExists(settingsPath)) {
+    return;
+  }
+  try {
+    const raw = await readTextFile(settingsPath);
+    const bom = raw.startsWith("\uFEFF") ? "\uFEFF" : "";
+    const parsed = JSON.parse(bom ? raw.slice(bom.length) : raw);
+    if (typeof parsed !== "object" || parsed === null || Array.isArray(parsed)) {
+      return;
+    }
+    const { settings, rewritten } = migrateHookCommands(parsed);
+    if (rewritten === 0) {
+      return;
+    }
+    const trailingNewline = raw.endsWith(`
+`) ? `
+` : "";
+    await writeTextFile(settingsPath, `${bom}${JSON.stringify(settings, null, 2)}${trailingNewline}`);
+    info("update.hook_commands_migrated", { count: String(rewritten) });
+  } catch {
+    warn("update.hook_commands_migration_failed", {
+      path: settingsPath
+    });
+  }
+}
 async function runFullUpdatePostProcessing(options, result, config) {
   const isFullUpdate = !options.components || options.components.length === 0;
   if (isFullUpdate) {
@@ -20960,6 +21073,9 @@ async function runFullUpdatePostProcessing(options, result, config) {
       await backfillStatusLineRefreshInterval(options.targetDir, options);
     }
   }
+  if (isFullUpdate || options.components?.includes("hooks")) {
+    await migrateHookCommandsInSettingsLocal(options.targetDir, options);
+  }
   if (!options.dryRun) {
     config.version = result.newVersion;
     config.lastUpdated = new Date().toISOString();

package/dist/index.js

@@ -330,6 +330,8 @@ var MESSAGES = {
     "install.entry_md_skipped": "{{entry}} skipped ({{reason}})",
     "install.lockfile_generated": "Lockfile generated ({{files}} files tracked)",
     "install.lockfile_failed": "Failed to generate lockfile: {{error}}",
+    "install.hook_commands_migrated": "Migrated {{rewritten}} hook command(s) in settings.local.json",
+    "install.hook_commands_migration_failed": "Failed to migrate hook commands in settings.local.json: {{error}}",
     "lockfile.not_found": "Lockfile not found: {{path}}",
     "lockfile.invalid_version": "Invalid lockfile version: {{path}}",
     "lockfile.invalid_structure": "Invalid lockfile structure: {{path}}",
@@ -351,6 +353,8 @@ var MESSAGES = {
     "update.lockfile_failed": "Failed to regenerate lockfile: {{error}}",
     "update.protected_file_updated": "⟳ Protected file {{file}} in {{component}} updated: {{hint}}",
     "update.namespace_synced": "Namespace synced: {{file}} ({{component}})",
+    "update.hook_commands_migrated": "Migrated {{count}} hook command(s) to CLAUDE_PROJECT_DIR-anchored paths",
+    "update.hook_commands_migration_failed": "Failed to migrate hook commands in {{path}}",
     "config.load_failed": "Failed to load config: {{error}}",
     "config.not_found": "Config not found at {{path}}, using defaults",
     "config.saved": "Config saved to {{path}}",
@@ -376,6 +380,8 @@ var MESSAGES = {
     "install.entry_md_skipped": "{{entry}} 건너뜀 ({{reason}})",
     "install.lockfile_generated": "잠금 파일 생성 완료 ({{files}}개 파일 추적)",
     "install.lockfile_failed": "잠금 파일 생성 실패: {{error}}",
+    "install.hook_commands_migrated": "settings.local.json의 훅 명령 {{rewritten}}개를 마이그레이션했습니다",
+    "install.hook_commands_migration_failed": "settings.local.json의 훅 명령 마이그레이션에 실패했습니다: {{error}}",
     "lockfile.not_found": "잠금 파일 없음: {{path}}",
     "lockfile.invalid_version": "잠금 파일 버전 유효하지 않음: {{path}}",
     "lockfile.invalid_structure": "잠금 파일 구조 유효하지 않음: {{path}}",
@@ -397,6 +403,8 @@ var MESSAGES = {
     "update.lockfile_failed": "잠금 파일 재생성 실패: {{error}}",
     "update.protected_file_updated": "⟳ 보호 파일 {{file}} ({{component}}) 업데이트됨: {{hint}}",
     "update.namespace_synced": "네임스페이스 동기화: {{file}} ({{component}})",
+    "update.hook_commands_migrated": "CLAUDE_PROJECT_DIR 기준 경로로 훅 명령 {{count}}개를 마이그레이션했습니다",
+    "update.hook_commands_migration_failed": "{{path}}의 훅 명령 마이그레이션에 실패했습니다",
     "config.load_failed": "설정 로드 실패: {{error}}",
     "config.not_found": "{{path}}에 설정 없음, 기본값 사용",
     "config.saved": "설정 저장: {{path}}",
@@ -1094,6 +1102,52 @@ async function cleanupPreservation(tempDir) {
   }
 }
 
+// src/core/hook-command-migration.ts
+var OLD_HOOK_COMMAND = /^(bash )?(?:\.\/)?\.claude\/hooks\/((?:scripts\/)?[A-Za-z0-9._-]+\.sh)$/;
+function rewriteRelativeHookCommand(command) {
+  const match = OLD_HOOK_COMMAND.exec(command);
+  if (match === null) {
+    return null;
+  }
+  const interpreter = match[1] ?? "";
+  return `${interpreter}"\${CLAUDE_PROJECT_DIR:-.}/.claude/hooks/${match[2]}"`;
+}
+function isRecord(value) {
+  return typeof value === "object" && value !== null && !Array.isArray(value);
+}
+function rewriteNode(node, counter) {
+  if (Array.isArray(node)) {
+    return node.map((item) => rewriteNode(item, counter));
+  }
+  if (!isRecord(node)) {
+    return node;
+  }
+  const out = {};
+  for (const [key, value] of Object.entries(node)) {
+    if (key === "command" && typeof value === "string") {
+      const rewritten = rewriteRelativeHookCommand(value);
+      if (rewritten !== null) {
+        counter.count++;
+        out[key] = rewritten;
+        continue;
+      }
+    }
+    out[key] = rewriteNode(value, counter);
+  }
+  return out;
+}
+function migrateHookCommands(settings) {
+  if (!isRecord(settings.hooks)) {
+    return { settings, rewritten: 0 };
+  }
+  const counter = { count: 0 };
+  const hooks = rewriteNode(settings.hooks, counter);
+  if (counter.count === 0) {
+    return { settings, rewritten: 0 };
+  }
+  return { settings: { ...settings, hooks }, rewritten: counter.count };
+}
+
 // src/core/hooks-settings.ts
 init_fs();
 import { dirname as dirname2 } from "node:path";
@@ -2026,6 +2080,32 @@ async function installHooksSettings(targetDir, _options, result) {
     warn("install.hooks_settings_failed", { error: message });
   }
 }
+var BOM = "\uFEFF";
+async function migrateRestoredHookCommands(targetDir, result) {
+  const layout = getProviderLayout();
+  const settingsPath = join5(targetDir, layout.rootDir, "settings.local.json");
+  if (!await fileExists(settingsPath)) {
+    return;
+  }
+  try {
+    const raw = await readTextFile(settingsPath);
+    const hasBom = raw.startsWith(BOM);
+    const trailingNewline = raw.endsWith(`
+`) ? `
+` : "";
+    const current = JSON.parse(hasBom ? raw.slice(BOM.length) : raw);
+    const { settings, rewritten } = migrateHookCommands(current);
+    if (rewritten > 0) {
+      const body = `${JSON.stringify(settings, null, 2)}${trailingNewline}`;
+      await writeTextFile(settingsPath, hasBom ? `${BOM}${body}` : body);
+      debug("install.hook_commands_migrated", { rewritten: String(rewritten) });
+    }
+  } catch (err) {
+    const message = err instanceof Error ? err.message : String(err);
+    result.warnings.push(`Failed to migrate hook commands in settings.local.json: ${message}`);
+    warn("install.hook_commands_migration_failed", { error: message });
+  }
+}
 async function installEntryDocWithTracking(targetDir, options, result) {
   const language = options.language ?? DEFAULT_LANGUAGE;
   const overwrite = !!(options.force || options.backup);
@@ -2101,6 +2181,7 @@ async function install(options) {
           result.warnings.push(`Failed to restore ${failure.path}: ${failure.reason}`);
         }
       }
+      await migrateRestoredHookCommands(options.targetDir, result);
       await cleanupPreservation(preservation.tempDir);
     }
     await updateInstallConfig(options.targetDir, options, result.installedComponents);
@@ -2326,7 +2407,7 @@ var package_default = {
   workspaces: [
     "packages/*"
   ],
-  version: "1.1.91",
+  version: "1.1.93",
   description: "Batteries-included agent harness for Claude Code",
   type: "module",
   bin: {
@@ -2350,7 +2431,8 @@ var package_default = {
   },
   scripts: {
     dev: "bun run src/cli/index.ts",
-    build: "bun build src/cli/index.ts --outdir dist/cli --target node && bun build src/index.ts --outdir dist --target node && bun run scripts/sync-source-lockfile.ts",
+    build: "bun build src/cli/index.ts --outdir dist/cli --target node && bun build src/index.ts --outdir dist --target node && bun run scripts/sync-source-lockfile.ts && bun run scripts/sync-hooks-settings.ts",
+    "sync:hooks": "bun run scripts/sync-hooks-settings.ts",
     test: "bun test tests/ packages/eval-core/",
     "test:unit": "bun test tests/unit",
     "test:integration": "bun test tests/integration",
@@ -5157,6 +5239,37 @@ async function backfillStatusLineRefreshInterval(targetDir, options) {
     });
   }
 }
+async function migrateHookCommandsInSettingsLocal(targetDir, options) {
+  if (options.dryRun) {
+    return;
+  }
+  const layout = getProviderLayout();
+  const settingsPath = join7(targetDir, layout.rootDir, "settings.local.json");
+  if (!await fileExists(settingsPath)) {
+    return;
+  }
+  try {
+    const raw = await readTextFile(settingsPath);
+    const bom = raw.startsWith("\uFEFF") ? "\uFEFF" : "";
+    const parsed = JSON.parse(bom ? raw.slice(bom.length) : raw);
+    if (typeof parsed !== "object" || parsed === null || Array.isArray(parsed)) {
+      return;
+    }
+    const { settings, rewritten } = migrateHookCommands(parsed);
+    if (rewritten === 0) {
+      return;
+    }
+    const trailingNewline = raw.endsWith(`
+`) ? `
+` : "";
+    await writeTextFile(settingsPath, `${bom}${JSON.stringify(settings, null, 2)}${trailingNewline}`);
+    info("update.hook_commands_migrated", { count: String(rewritten) });
+  } catch {
+    warn("update.hook_commands_migration_failed", {
+      path: settingsPath
+    });
+  }
+}
 async function runFullUpdatePostProcessing(options, result, config) {
   const isFullUpdate = !options.components || options.components.length === 0;
   if (isFullUpdate) {
@@ -5169,6 +5282,9 @@ async function runFullUpdatePostProcessing(options, result, config) {
       await backfillStatusLineRefreshInterval(options.targetDir, options);
     }
   }
+  if (isFullUpdate || options.components?.includes("hooks")) {
+    await migrateHookCommandsInSettingsLocal(options.targetDir, options);
+  }
   if (!options.dryRun) {
     config.version = result.newVersion;
     config.lastUpdated = new Date().toISOString();

package/package.json

@@ -3,7 +3,7 @@
   "workspaces": [
     "packages/*"
   ],
-  "version": "1.1.91",
+  "version": "1.1.93",
   "description": "Batteries-included agent harness for Claude Code",
   "type": "module",
   "bin": {
@@ -27,7 +27,8 @@
   },
   "scripts": {
     "dev": "bun run src/cli/index.ts",
-    "build": "bun build src/cli/index.ts --outdir dist/cli --target node && bun build src/index.ts --outdir dist --target node && bun run scripts/sync-source-lockfile.ts",
+    "build": "bun build src/cli/index.ts --outdir dist/cli --target node && bun build src/index.ts --outdir dist --target node && bun run scripts/sync-source-lockfile.ts && bun run scripts/sync-hooks-settings.ts",
+    "sync:hooks": "bun run scripts/sync-hooks-settings.ts",
     "test": "bun test tests/ packages/eval-core/",
     "test:unit": "bun test tests/unit",
     "test:integration": "bun test tests/integration",