@su-record/vibe 2.12.4 → 2.13.0

package/hooks/scripts/step-counter.js

@@ -2,9 +2,19 @@
 /**
  * PostToolUse Hook — 툴콜 스텝 카운터 + 패턴 로거 + 3-fail 감지기
  *
- * 책임 1) 모든 성공 툴콜을 1 스텝으로 집계 → `current-run.json`
+ * hooks.json matcher: "Edit|Write|Bash|Task|SlashCommand|NotebookEdit"
+ * → Read/Grep/Glob/WebFetch/WebSearch/TodoWrite/ToolSearch/ListMcpResourcesTool
+ *   는 matcher 단에서 제외되어 이 프로세스 자체가 spawn 되지 않는다.
+ *   아래 READ_ONLY_TOOLS 는 defense-in-depth 용 추가 가드이다.
+ *
+ * 책임 1) 액션 툴콜을 1 스텝으로 집계 → `current-run.json`
  *        ↳ /vibe.verify 가 history.jsonl에 append 후 출력
- * 책임 2) 각 툴콜 한 줄을 `current-run.jsonl` 에 append (post-task-curation SPEC)
+ *        ↳ recipe-extractor 가 meta(startedAt, steps) 소비
+ *        [쓰기 스로틀] current-run.json 전체 재작성은 (a) 마지막 재작성 후
+ *        ≥2초 경과, 또는 (b) 10이벤트마다 1회로 제한.
+ *        액션 툴(Edit/Write/Bash) 호출 + ≥2초 조건을 동시 충족할 때도 재작성.
+ *        Stop 경로 등 최종 상태가 필요한 소비자는 staleness ≤10 이벤트를 허용.
+ * 책임 2) 각 툴콜 한 줄을 `current-run.jsonl` 에 append (항상 즉시 — 스로틀 없음)
  *        ↳ Phase 3 의 recipe extractor 가 소비
  * 책임 3) (Phase 2) 실패 메시지 분류 + 같은 (file, category) 3회 반복 감지 →
  *        `.vibe/anti-patterns/<slug>.md` 로 자동 저장.
@@ -24,6 +34,16 @@ const MAX_JSONL_LINES = 5000;
 const FAIL_WINDOW = 10;
 const FAIL_THRESHOLD = 3;
 
+// 스로틀 파라미터
+const REWRITE_INTERVAL_MS = 2000; // 최소 재작성 간격
+const REWRITE_EVERY_N = 10;       // N이벤트마다 강제 재작성
+
+// 읽기 전용 툴 — defense-in-depth (matcher 단에서 이미 제거됨)
+const READ_ONLY_TOOLS = new Set([
+  'Read', 'Grep', 'Glob', 'WebFetch', 'WebSearch',
+  'TodoWrite', 'ToolSearch', 'ListMcpResourcesTool',
+]);
+
 // ─────────────────────────────────────────────────────
 // stdin / env 에서 PostToolUse payload 추출
 // ─────────────────────────────────────────────────────
@@ -97,9 +117,43 @@ function classifyError(toolResponse) {
 }
 
 // ─────────────────────────────────────────────────────
-// 책임 1: current-run.json 카운터
+// 스로틀: current-run.json 재작성 여부 결정
+// ─────────────────────────────────────────────────────
+/**
+ * 재작성해야 하는지 판단.
+ * - jsonlLineCount: jsonl 에 추가한 후의 총 라인 수 (이벤트 카운터 대리)
+ * - toolName: Edit/Write/Bash 인 경우 mtime 조건 충족 시 강제 재작성
+ * @param {string} toolName
+ * @param {number} jsonlLineCount
+ * @returns {boolean}
+ */
+function shouldRewriteJson(toolName, jsonlLineCount) {
+  // N이벤트마다 강제 재작성 (race-tolerant — jsonl 줄 수가 카운터 역할)
+  if (jsonlLineCount % REWRITE_EVERY_N === 0) return true;
+
+  // 마지막 재작성 후 ≥2초 경과 여부 (mtime 기반)
+  try {
+    const stat = fs.statSync(CURRENT_RUN_JSON);
+    const elapsed = Date.now() - stat.mtimeMs;
+    if (elapsed >= REWRITE_INTERVAL_MS) return true;
+  } catch {
+    // 파일 없음 → 첫 쓰기, 허용
+    return true;
+  }
+
+  return false;
+}
+
+// ─────────────────────────────────────────────────────
+// 책임 1: current-run.json 카운터 (스로틀 적용)
+//
+// steps 는 current-run.jsonl 의 라인 수에서 파생한다.
+// 이렇게 하면 스로틀로 재작성을 건너뛰어도 재작성 시점에
+// 정확한 steps 가 보장되며, 프로세스간 누적도 자연스럽게 처리된다.
 // ─────────────────────────────────────────────────────
-function bumpCounter() {
+function bumpCounter(toolName, currentLineCount) {
+  if (!shouldRewriteJson(toolName, currentLineCount)) return; // 스로틀 — 재작성 생략
+
   let data = { feature: null, startedAt: null, steps: 0 };
   if (fs.existsSync(CURRENT_RUN_JSON)) {
     try {
@@ -107,12 +161,24 @@ function bumpCounter() {
     } catch { /* 손상 → 새로 시작 */ }
   }
   if (!data.startedAt) data.startedAt = new Date().toISOString();
-  data.steps = (data.steps || 0) + 1;
-  fs.writeFileSync(CURRENT_RUN_JSON, JSON.stringify(data, null, 2));
+  // steps = jsonl 라인 수에서 파생 (매 재작성 시 최신값 보장)
+  data.steps = currentLineCount;
+
+  // 원자적 쓰기 (tmp → rename) — 동시 에이전트 파일 손상 방지
+  const tmp = CURRENT_RUN_JSON + '.tmp';
+  try {
+    fs.writeFileSync(tmp, JSON.stringify(data, null, 2));
+    fs.renameSync(tmp, CURRENT_RUN_JSON);
+  } catch {
+    // rename 실패 시 직접 쓰기 (fail-open)
+    try { fs.writeFileSync(CURRENT_RUN_JSON, JSON.stringify(data, null, 2)); } catch { /* ignore */ }
+    try { fs.unlinkSync(tmp); } catch { /* ignore */ }
+  }
 }
 
 // ─────────────────────────────────────────────────────
 // 책임 2: current-run.jsonl append + error_category 채움
+// 항상 즉시 append — 스로틀 없음 (3-fail detector, recipe-extractor 가 소비)
 // ─────────────────────────────────────────────────────
 function appendJsonl(stdinPayload) {
   const toolName = stdinPayload?.tool_name || process.argv[2] || '';
@@ -147,6 +213,21 @@ function appendJsonl(stdinPayload) {
   return record;
 }
 
+/**
+ * jsonl 의 현재 라인 수를 반환 (stat 으로만 — 전체 파일 읽기 없음).
+ * 경합 안전 — 정확한 값이 아니어도 스로틀 판단에 충분하다.
+ */
+function jsonlLineCount() {
+  try {
+    // 라인 수 근사: 파일 크기 / 평균 라인 길이(100바이트) — 정확도 불필요
+    // 실제 라인 수는 appendJsonl 직후 여기서 읽으면 +1 반영됨
+    const content = fs.readFileSync(CURRENT_RUN_JSONL, 'utf-8');
+    return content.split('\n').filter(Boolean).length;
+  } catch {
+    return 1; // 파일 없음 → 첫 이벤트로 간주
+  }
+}
+
 // ─────────────────────────────────────────────────────
 // 책임 3b: 3-fail detector → anti-pattern md
 // ─────────────────────────────────────────────────────
@@ -238,10 +319,22 @@ try {
   }
 
   const stdinPayload = readStdinSync();
+  const toolName = stdinPayload?.tool_name || process.argv[2] || '';
 
-  try { bumpCounter(); } catch { /* 카운터 실패 무시 */ }
+  // defense-in-depth: 읽기 전용 툴은 조기 종료 (matcher 단에서 이미 제거됨)
+  if (toolName && READ_ONLY_TOOLS.has(toolName)) {
+    process.exit(0);
+  }
+
+  // 책임 2를 먼저 실행 — jsonl 라인 수가 bumpCounter 스로틀 판단에 쓰임
   let lastRecord = null;
   try { lastRecord = appendJsonl(stdinPayload); } catch { /* 로깅 실패 무시 */ }
+
+  // 책임 1: jsonl append 후 라인 수로 스로틀 판단
+  const lineCount = jsonlLineCount();
+  try { bumpCounter(toolName, lineCount); } catch { /* 카운터 실패 무시 */ }
+
+  // 책임 3
   try {
     if (lastRecord && !lastRecord.ok && lastRecord.error_category) {
       const trip = detectThreeFail();

package/hooks/scripts/stop-dispatcher.js

@@ -7,6 +7,7 @@
  * 현재: 단일 디스패처에서 순차 실행.
  *
  * 실행 순서:
+ *   0. verify-skip 경고/차단 (in-process)
  *   1. codex-review-gate — 리뷰 필요 여부 판단 (stdout → Claude 지시 주입)
  *   2. stop-notify       — 완료 알림
  *   3. auto-commit       — 변경 자동 커밋 (git hook cascade 주의)
@@ -16,9 +17,34 @@
  * 이 Stop dispatcher도 건너뛴다 (자식 세션에서 auto-commit 등이 돌 이유 없음).
  */
 import { dispatch } from './lib/dispatcher.js';
+import { readLedger, markStopWarned } from './lib/run-ledger.js';
+import { PROJECT_DIR, readProjectConfig } from './utils.js';
 
 if (process.env.VIBE_HOOK_DEPTH) process.exit(0);
 
+// verify-skip 게이트 — runStarted가 있고 verifyPassed가 false이면 경고 또는 차단.
+// stopWarned가 이미 true이면 루프 방지를 위해 스킵.
+try {
+  const ledger = readLedger(PROJECT_DIR);
+  if (ledger && ledger.runStarted && !ledger.verifyPassed && !ledger.stopWarned) {
+    const config = readProjectConfig();
+    const gateMode = config?.verifyGate?.mode;
+
+    if (gateMode === 'block') {
+      // 1회에 한해 Stop 차단 — 루프 방지 플래그 먼저 세팅
+      markStopWarned(PROJECT_DIR);
+      const reason = '/vibe.verify not run after /vibe.run — stop blocked (verifyGate.mode=block)';
+      process.stdout.write(JSON.stringify({ decision: 'block', reason }) + '\n');
+      process.exit(0);
+    } else {
+      // 기본: warn 모드 — stderr에 경고, 루프 방지 플래그 세팅
+      markStopWarned(PROJECT_DIR);
+      const feature = ledger.runFeature ? ` (feature: ${ledger.runFeature})` : '';
+      process.stderr.write(`[vibe] WARNING: /vibe.run${feature} completed but /vibe.verify was not run.\n`);
+    }
+  }
+} catch { /* verify-skip 게이트 실패는 이후 실행을 막지 않음 */ }
+
 try {
   await dispatch([
     { name: 'codex-review-gate', script: 'codex-review-gate.js' },

package/hooks/scripts/utils.js

@@ -75,32 +75,44 @@ export function projectMemoryDir(projectDir = PROJECT_DIR) {
   return path.join(projectDir, '.vibe', 'memories');
 }
 
+// config 캐시 — 훅 스크립트는 단명 프로세스이므로 프로세스 생명주기 내에서
+// config 파일이 바뀌지 않는다고 가정한다 (llm-orchestrate처럼 한 프로세스에서
+// 3회 이상 읽는 경로의 중복 read+parse 제거).
+let _vibeConfigCache = null;
+let _projectConfigCache = null;
+
 /**
- * ~/.vibe/config.json 읽기
+ * ~/.vibe/config.json 읽기 (프로세스 내 캐시)
  * @returns {object} 파싱된 config 또는 빈 객체
  */
 export function readVibeConfig() {
+  if (_vibeConfigCache !== null) return _vibeConfigCache;
   const configPath = path.join(VIBE_HOME_DIR, 'config.json');
   try {
     if (fs.existsSync(configPath)) {
-      return JSON.parse(fs.readFileSync(configPath, 'utf-8'));
+      _vibeConfigCache = JSON.parse(fs.readFileSync(configPath, 'utf-8'));
+      return _vibeConfigCache;
     }
   } catch { /* ignore */ }
-  return {};
+  _vibeConfigCache = {};
+  return _vibeConfigCache;
 }
 
 /**
- * 프로젝트 설정(.vibe/config.json) 읽기 — legacy `.claude/vibe/` fallback 포함
+ * 프로젝트 설정(.vibe/config.json) 읽기 — legacy `.claude/vibe/` fallback 포함 (프로세스 내 캐시)
  * @returns {object} 파싱된 config 또는 빈 객체
  */
 export function readProjectConfig() {
+  if (_projectConfigCache !== null) return _projectConfigCache;
   const configPath = projectVibePath(PROJECT_DIR, 'config.json');
   try {
     if (fs.existsSync(configPath)) {
-      return JSON.parse(fs.readFileSync(configPath, 'utf-8'));
+      _projectConfigCache = JSON.parse(fs.readFileSync(configPath, 'utf-8'));
+      return _projectConfigCache;
     }
   } catch { /* ignore */ }
-  return {};
+  _projectConfigCache = {};
+  return _projectConfigCache;
 }
 
 /**
@@ -151,29 +163,71 @@ function toFileUrl(fsPath) {
   return url.endsWith('/') ? url : url + '/';
 }
 
-// 전역 npm 경로 캐시
-let _globalNpmPath = null;
-function getGlobalNpmPath() {
-  if (_globalNpmPath === null) {
-    try {
-      _globalNpmPath = execSync('npm root -g', { encoding: 'utf8' }).trim();
-    } catch {
-      // 폴백: 플랫폼별 기본 경로
-      if (IS_WINDOWS) {
-        // Windows: %APPDATA%\npm\node_modules 또는 %LOCALAPPDATA%\npm\node_modules
-        const appData = process.env.APPDATA || process.env.LOCALAPPDATA || '';
-        _globalNpmPath = path.join(appData, 'npm', 'node_modules');
-      } else {
-        // macOS/Linux: /usr/local/lib/node_modules 또는 ~/.npm-global/lib/node_modules
-        const homeDir = os.homedir();
-        const defaultPaths = [
-          '/usr/local/lib/node_modules',
-          path.join(homeDir, '.npm-global', 'lib', 'node_modules'),
-          path.join(homeDir, '.nvm', 'versions', 'node', process.version, 'lib', 'node_modules'),
-        ];
-        _globalNpmPath = defaultPaths.find(p => fs.existsSync(p)) || defaultPaths[0];
-      }
+// 전역 npm 경로 캐시 — L1: 인-프로세스, L2: 파일 캐시 (TTL 24h)
+// VIBE_NPM_ROOT_CACHE_FILE 환경 변수로 테스트 시 격리된 경로 주입 가능
+const NPM_ROOT_CACHE_FILE = process.env.VIBE_NPM_ROOT_CACHE_FILE ||
+  path.join(os.homedir(), '.vibe', 'cache', 'npm-root.json');
+const NPM_ROOT_CACHE_TTL_MS = 24 * 60 * 60 * 1000; // 24시간
+
+/** npm root -g 결과를 파일로 캐시 (원자적 쓰기). 실패는 조용히 무시. */
+function saveNpmRootCache(npmRoot) {
+  try {
+    const dir = path.dirname(NPM_ROOT_CACHE_FILE);
+    if (!fs.existsSync(dir)) fs.mkdirSync(dir, { recursive: true });
+    const payload = JSON.stringify({ npmRoot, savedAt: Date.now() });
+    const tmp = NPM_ROOT_CACHE_FILE + '.tmp';
+    fs.writeFileSync(tmp, payload, { mode: 0o600 });
+    fs.renameSync(tmp, NPM_ROOT_CACHE_FILE);
+  } catch { /* 캐시 저장 실패는 무시 */ }
+}
+
+/** 파일 캐시에서 npm root 읽기. TTL 초과 또는 손상 시 null 반환. */
+function loadNpmRootCache() {
+  try {
+    const raw = fs.readFileSync(NPM_ROOT_CACHE_FILE, 'utf-8');
+    const { npmRoot, savedAt } = JSON.parse(raw);
+    if (typeof npmRoot === 'string' && typeof savedAt === 'number') {
+      if (Date.now() - savedAt < NPM_ROOT_CACHE_TTL_MS) return npmRoot;
     }
+  } catch { /* 파일 없음 / 손상 / 권한 → null */ }
+  return null;
+}
+
+/** npm root -g 플랫폼 폴백 (execSync 실패 시 사용). */
+function npmRootFallback() {
+  if (IS_WINDOWS) {
+    const appData = process.env.APPDATA || process.env.LOCALAPPDATA || '';
+    return path.join(appData, 'npm', 'node_modules');
+  }
+  const homeDir = os.homedir();
+  const candidates = [
+    '/usr/local/lib/node_modules',
+    path.join(homeDir, '.npm-global', 'lib', 'node_modules'),
+    path.join(homeDir, '.nvm', 'versions', 'node', process.version, 'lib', 'node_modules'),
+  ];
+  return candidates.find(p => fs.existsSync(p)) || candidates[0];
+}
+
+// L1 인-프로세스 캐시
+let _globalNpmPath = null;
+export function getGlobalNpmPath() {
+  // L1: 인-프로세스 캐시 히트
+  if (_globalNpmPath !== null) return _globalNpmPath;
+
+  // L2: 파일 캐시 히트
+  const cached = loadNpmRootCache();
+  if (cached !== null) {
+    _globalNpmPath = cached;
+    return _globalNpmPath;
+  }
+
+  // L3: execSync 실행 후 파일 캐시에 저장
+  try {
+    _globalNpmPath = execSync('npm root -g', { encoding: 'utf8' }).trim();
+    saveNpmRootCache(_globalNpmPath);
+  } catch {
+    _globalNpmPath = npmRootFallback();
+    // 폴백은 캐시에 저장하지 않음 — 다음 프로세스에서 재시도
   }
   return _globalNpmPath;
 }
@@ -190,20 +244,32 @@ function hasRuntimeDeps(packageRoot) {
  * 패키지 하위 경로의 file:// URL 반환 (크로스 플랫폼)
  * 우선순위: 로컬 빌드 → ~/.vibe/ → 전역 npm
  * 각 후보는 probeFile 존재 + 런타임 deps 해석 가능해야 채택 (dist-only 복사본 회피)
+ *
+ * 프로세스 내 메모이제이션: getToolsBaseUrl/getLibBaseUrl/getCliBaseUrl이 한 훅
+ * 프로세스에서 각각 호출되면 최악 6회 동기 stat이 반복되므로 결과를 캐싱한다.
  */
+const _packageUrlCache = new Map();
 function getPackageUrl(subpath, probeFile) {
+  const cacheKey = `${subpath}|${probeFile}`;
+  const cached = _packageUrlCache.get(cacheKey);
+  if (cached) return cached;
+
   const localRoot = VIBE_PATH;
   const vibeRoot = path.join(VIBE_HOME_DIR, 'node_modules', '@su-record', 'vibe');
   const globalRoot = path.join(getGlobalNpmPath(), '@su-record', 'vibe');
 
   const candidates = [localRoot, vibeRoot, globalRoot];
+  let result = null;
   for (const root of candidates) {
     const target = path.join(root, subpath);
     if (fs.existsSync(path.join(target, probeFile)) && hasRuntimeDeps(root)) {
-      return toFileUrl(target);
+      result = toFileUrl(target);
+      break;
     }
   }
-  return toFileUrl(path.join(globalRoot, subpath));
+  if (result === null) result = toFileUrl(path.join(globalRoot, subpath));
+  _packageUrlCache.set(cacheKey, result);
+  return result;
 }
 
 export function getToolsBaseUrl() {

package/hooks/scripts/verify-ledger.js

@@ -0,0 +1,22 @@
+#!/usr/bin/env node
+/**
+ * verify-ledger CLI — vibe.verify 결과를 run-ledger에 기록.
+ *
+ * 사용법: node hooks/scripts/verify-ledger.js pass|fail
+ *
+ * vibe.verify/SKILL.md 에서 검증 완료 시 호출.
+ * stdout 출력은 에이전트가 Bash로 실행하는 컨텍스트이므로 허용.
+ * 항상 exit 0.
+ */
+
+import { recordVerify } from './lib/run-ledger.js';
+
+const arg = process.argv[2];
+const passed = arg === 'pass';
+const projectDir = process.env.CLAUDE_PROJECT_DIR || process.cwd();
+
+recordVerify(projectDir, passed);
+
+const status = passed ? 'pass' : 'fail';
+process.stdout.write(`[verify-ledger] recorded: verifyPassed=${passed} (${status})\n`);
+process.exit(0);

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@su-record/vibe",
-  "version": "2.12.4",
-  "description": "AI Coding Framework for Claude Code — 56 agents, 45 skills, multi-LLM orchestration",
+  "version": "2.13.0",
+  "description": "AI Coding Framework for Claude Code — 60+ agents, 69 skills, multi-LLM orchestration",
   "type": "module",
   "main": "dist/cli/index.js",
   "exports": {

package/skills/arch-guard/SKILL.md

@@ -3,8 +3,8 @@ name: arch-guard
 user-invocable: false
 invocation: [auto]
 tier: core
-description: "Generate architecture boundary tests that mechanically enforce layer constraints. Use when adding new modules, refactoring layers, or after detecting circular dependencies. Creates import-rule tests (e.g., 'UI must not import DB') that fail CI on violation. Must use this skill when user mentions layer enforcement, dependency rules, or architectural boundaries — even casually like 'make sure services don't import controllers'."
-triggers: [arch guard, architecture test, layer test, boundary test, structural test, arch validation]
+description: "Generate import-rule tests that mechanically enforce architecture layer constraints (e.g., 'UI must not import DB') — for new modules, layer refactors, or circular dependencies."
+triggers: [arch guard, architecture test, layer test, boundary test, structural test, arch validation, layer enforcement, dependency rules, architectural boundaries, circular dependency]
 priority: 60
 ---
 

package/skills/characterization-test/SKILL.md

@@ -3,8 +3,8 @@ name: characterization-test
 user-invocable: false
 invocation: [auto]
 tier: core
-description: "Lock existing behavior with characterization tests before modifying code. Use BEFORE any refactor, rewrite, or large-scale modification of existing code — especially legacy code without tests. Captures current input/output behavior as test cases so regressions are caught immediately. Must use this skill when touching files >200 lines with no existing tests, when user says 'refactor', 'rewrite', 'modernize', or 'clean up' existing code."
-triggers: [legacy, characterization test, lock behavior, regression prevention, before refactor, large file]
+description: "Lock existing behavior with characterization tests BEFORE refactoring or modifying legacy/untested code (files >200 lines without tests)."
+triggers: [legacy, characterization test, lock behavior, regression prevention, before refactor, large file, refactor, rewrite, modernize, clean up]
 priority: 65
 ---
 

package/skills/exec-plan/SKILL.md

@@ -3,8 +3,8 @@ name: exec-plan
 user-invocable: false
 invocation: [auto, chain]
 tier: core
-description: "Convert SPEC into a self-contained execution plan that agents can run autonomously for hours. Use when a SPEC has 3+ phases, multiple files to change, or complex dependencies between tasks. Produces a plan with explicit file paths, interface extracts, and acceptance criteria so agents never need to 'figure out' context. Must use this skill when launching long-running autonomous work, parallelizing implementation across agents, or when user says 'execute this spec' or 'run this plan'."
-triggers: [exec plan, execution plan, autonomous plan, self-contained plan, long-running]
+description: "Convert a SPEC (3+ phases or multi-file) into a self-contained execution plan — explicit file paths, interfaces, acceptance criteria — that agents can run autonomously."
+triggers: [exec plan, execution plan, autonomous plan, self-contained plan, long-running, execute this spec, run this plan, parallel implementation]
 priority: 70
 chain-next: [commit-push-pr]
 ---