@su-record/vibe 2.12.5 → 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

@@ -163,29 +163,71 @@ function toFileUrl(fsPath) {
   return url.endsWith('/') ? url : url + '/';
 }
 
-// 전역 npm 경로 캐시
+// 전역 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() {
-  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];
-      }
-    }
+  // 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;
 }

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.5",
-  "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/spec/references/templates.md

@@ -86,19 +86,24 @@ Define AI role and expertise for implementation
 
 ## Task
 <task>
+<!-- ⚠️ MANDATORY: Every functional requirement item MUST carry a REQ-ID so the RTM can extract it.
+     Format: REQ-{feature-name}-NNN  (feature = SPEC file basename, NNN = 3-digit zero-padded number)
+     Example: REQ-auth-login-001
+     Regex the RTM uses: /\b(REQ-[a-z0-9-]+-\d{3})[:\s]/gi  — use uppercase REQ-, lowercase feature segment, exactly 3 digits.
+     status === 'empty' means the RTM gate MUST be treated as failed/not-applicable, never as 100% pass. -->
 ### Phase 1: {phase-name}
-1. [ ] {specific task}
+1. [ ] REQ-{feature-name}-001: {specific task}
    - File: `path/to/file`
    - Verify: `command`
-2. [ ] {specific task}
+2. [ ] REQ-{feature-name}-002: {specific task}
 
 ### Phase 2: {phase-name}
-1. [ ] {specific task}
-2. [ ] {specific task}
+1. [ ] REQ-{feature-name}-003: {specific task}
+2. [ ] REQ-{feature-name}-004: {specific task}
 
 ### Phase 3: Testing and Verification
-1. [ ] Unit Tests
-2. [ ] Integration Tests
+1. [ ] REQ-{feature-name}-005: Unit Tests
+2. [ ] REQ-{feature-name}-006: Integration Tests
 </task>
 
 ## Constraints