therookie 0.4.30 → 0.4.32

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "therookie",
-  "version": "0.4.30",
+  "version": "0.4.32",
   "description": "the Rookie — the AI new hire that remembers what you teach it. A CLI that connects a personal AI memory to AI tools like Claude and Codex.",
   "license": "MIT",
   "keywords": [

package/src/cli.js

@@ -111,6 +111,9 @@ const VALUE_FLAGS = new Set([
   'projectId',
   'repo-path',
   'repo-url',
+  // spec 2026-09-08 §2.4 — `rookie save --session <id>`. 화이트리스트에 없으면 값이 안 붙어
+  //  flags.session === true 가 되고 저장 마커가 조용히 안 남는다(실측으로 잡힌 버그).
+  'session',
 ])
 
 export function parseFlags(args) {

package/src/commands/doctor.js

@@ -460,28 +460,44 @@ function printOrphanPending() {
 // 서버 전송 없음. 파일이 없으면(아직 안 돌았거나 구버전) 침묵한다.
 export function summarizeRecallPush(lines, max = 1000) {
   const rows = []
+  // spec 2026-09-08 훅 타임아웃 유실 — 시작 줄(phase:'start')과 완료 줄을 turn 으로 짝지어,
+  //  짝 없는 시작이 곧 "훅이 제한 시간을 넘겨 통째로 죽은 턴"이다. 유실은 무주입과 다르다 —
+  //  무주입은 넣을 게 없던 것이고 유실은 넣으려다 못 넣은 것이라, 무주입률에 섞지 않는다.
+  const startTurns = new Set()
+  const endTurns = new Set()
   // 파일은 append 전용이라 최신이 뒤에 온다 — 뒤에서부터 max 줄만 집계한다.
   for (let i = lines.length - 1; i >= 0 && rows.length < max; i--) {
     try {
       const o = JSON.parse(lines[i])
-      if (o && typeof o.injected === 'number') rows.push(o)
+      if (!o) continue
+      if (o.phase === 'start') {
+        if (o.turn) startTurns.add(o.turn)
+        continue
+      }
+      if (o.turn) endTurns.add(o.turn)
+      // 구버전 줄에는 phase 가 없다 — 완료 줄로 읽는다(하위호환).
+      if (typeof o.injected === 'number') rows.push(o)
     } catch {
       /* 깨진 줄은 건너뛴다 */
     }
   }
-  if (rows.length === 0) return null
+  let lost = 0
+  for (const t of startTurns) if (!endTurns.has(t)) lost++
+  if (rows.length === 0 && lost === 0) return null
   const silent = rows.filter((r) => r.injected === 0).length
   // 중앙값은 chars 기준이다 — 컨텍스트 예산은 문자·토큰 감각이고, 한국어는 바이트가 3배라
   // 바이트로 재면 합격 기준(§4.1)이 왜곡된다. bytes 는 파일에만 남긴다.
   const chars = rows.map((r) => (typeof r.chars === 'number' ? r.chars : 0)).sort((a, b) => a - b)
-  const median = chars[Math.floor(chars.length / 2)]
+  // 완료 줄이 하나도 없는 창(시작 줄만 남은 경우)에서 undefined 가 새어 나가지 않게 0 으로 닫는다.
+  const median = chars.length > 0 ? chars[Math.floor(chars.length / 2)] : 0
   const epochs = new Set(rows.map((r) => String(r.epoch ?? '')))
   return {
     turns: rows.length,
     silent,
-    silentRatio: silent / rows.length,
+    silentRatio: rows.length > 0 ? silent / rows.length : 0,
     medianChars: median,
     epochs: epochs.size,
+    lost,
   }
 }
 
@@ -502,9 +518,17 @@ function printRecallPush() {
   }
   if (!s) return
   const pct = Math.round(s.silentRatio * 100)
-  console.log(
-    `— 회상 푸시: 최근 ${s.turns}턴 중 무주입 ${s.silent}건(${pct}%), 주입 길이 중앙값 ${s.medianChars}자, epoch ${s.epochs}개`
-  )
+  if (s.turns > 0) {
+    console.log(
+      `— 회상 푸시: 최근 ${s.turns}턴 중 무주입 ${s.silent}건(${pct}%), 주입 길이 중앙값 ${s.medianChars}자, epoch ${s.epochs}개`
+    )
+  }
+  // 유실 0 이면 침묵 — 정상 상태를 매번 알리지 않는다.
+  if (s.lost > 0) {
+    console.log(
+      `⚠️ 훅 유실: 회상 ${s.lost}건 — 훅이 제한 시간을 넘겨 주입이 버려졌습니다. 부하가 높았는지 확인하세요.`
+    )
+  }
 }
 
 // B6-3 (spec 2026-07-11 §B6): 세션 identity pinning 위반 가시화 — 세션 도중 계정 전환으로

package/templates/hook/bin/recall-push.sh

@@ -55,15 +55,14 @@ process.stdin.on("data",c=>d+=c).on("end",async()=>{
   // 병렬 세션 안전: 쓰기는 append 전용(O_APPEND, 한 줄은 원자적)이고, 크기 초과 시에만
   // rename 으로 로테이트한다. 파일을 읽어 다시 쓰는 방식은 두 세션이 같은 스냅샷을 보고
   // 서로의 줄을 덮어쓸 수 있어 쓰지 않는다(codex 코드검토 R1 P1).
-  const metric=(layer,candidates,injected,text)=>{
+  // spec 2026-09-08 훅 타임아웃 유실 — 시작 줄과 완료 줄을 turn 으로 짝지어, 짝 없는 시작이
+  //  곧 "프로세스가 통째로 죽은 턴"이 된다. 시작 줄은 node 진입 직후에만 쓴다(집계 범위 = node 단계).
+  const turn=crypto.randomBytes(4).toString("hex");
+  const writeLine=(obj)=>{
     try{
       const dir=process.env.ROOKIE_PUSH_DIR;
       const f=path.join(dir,"metrics.jsonl");
-      const s=String(text||"");
-      const line=JSON.stringify({
-        ts:new Date().toISOString(),epoch,layer,candidates,injected,
-        chars:s.length,bytes:Buffer.byteLength(s)
-      })+"\n";
+      const line=JSON.stringify({ts:new Date().toISOString(),epoch,layer:"B",turn,...obj})+"\n";
       fs.mkdirSync(dir,{recursive:true,mode:0o700});
       try{
         if(fs.statSync(f).size>METRIC_MAX_BYTES) fs.renameSync(f,f+".1");
@@ -71,11 +70,18 @@ process.stdin.on("data",c=>d+=c).on("end",async()=>{
       fs.appendFileSync(f,line,{mode:0o600});
     }catch{}
   };
+  const metricStart=()=>writeLine({phase:"start"});
+  // 종료 경로마다 한 번. reason 은 코드의 return·catch 와 1:1 (설계 §3.4).
+  const metricEnd=(reason,candidates,injected,text)=>{
+    const t=String(text||"");
+    writeLine({phase:"end",reason,candidates,injected,chars:t.length,bytes:Buffer.byteLength(t)});
+  };
   try{
+    metricStart();
     const p=JSON.parse(d);
     const sid=String(p.session_id||p.sessionId||"");
     let prompt=String(p.prompt||"").trim();
-    if(!prompt) return;                       // 빈 발화 = 질의 불성립(유일한 기계 조건)
+    if(!prompt){ metricEnd("empty_prompt",0,0,""); return; }  // 빈 발화 = 질의 불성립
     prompt=prompt.slice(0,500);               // RecallQuerySchema q max(500) — 초과는 400 무주입
     const K=parseInt(process.env.ROOKIE_RECALL_PUSH_TOPK||"3",10);
     const CAP=parseInt(process.env.ROOKIE_RECALL_PUSH_CAP||"600",10);
@@ -101,7 +107,7 @@ process.stdin.on("data",c=>d+=c).on("end",async()=>{
       headers:{Authorization:"Bearer "+process.env.ROOKIE_TK,"x-rookie-hostname":process.env.ROOKIE_HOST||""},
       signal:AbortSignal.timeout(TIMEOUT)
     });
-    if(!res.ok) return;
+    if(!res.ok){ metricEnd("http_error",0,0,""); return; }
     const body=await res.json();
     const results=Array.isArray(body&&body.results)?body.results:[];
     // 임계 없음 — 걸러내는 것은 형식 불량(id·요약 없음, 점수 비수치)과 기주입뿐이다.
@@ -109,7 +115,7 @@ process.stdin.on("data",c=>d+=c).on("end",async()=>{
       .filter(r=>r&&typeof r.relevance==="number"&&r.id&&typeof r.summary==="string"&&r.summary.trim()&&!seen.has(r.id))
       .sort((a,b)=>b.relevance-a.relevance)
       .slice(0,K);
-    if(!cands.length){ metric("B",results.length,0,""); return; }
+    if(!cands.length){ metricEnd("no_hit",results.length,0,""); return; }
 
     // 주입 문구는 지시가 아니라 자료다. 관련도를 보증하지 않으므로 라벨에 명시하고,
     // 날짜·출처를 앞에 놓아 ITEM 절삭에 잘려 나가지 않게 한다.
@@ -123,16 +129,20 @@ process.stdin.on("data",c=>d+=c).on("end",async()=>{
       if(msg.length+item.length>CAP) break;   // 총량 상한 — relevance 순 절삭
       msg+=item; used.push(r.id);
     }
-    if(!used.length){ metric("B",results.length,0,""); return; }
+    if(!used.length){ metricEnd("no_hit",results.length,0,""); return; }
     if(stFile){
       try{
         fs.mkdirSync(process.env.ROOKIE_PUSH_DIR,{recursive:true,mode:0o700});
         fs.writeFileSync(stFile,JSON.stringify({epoch,ids:st.ids.concat(used)}),{mode:0o600});
       }catch{}                                // dedup 기록 실패 → 다음 턴 중복 주입 위험만, 주입은 진행
     }
-    metric("B",results.length,used.length,msg);
+    metricEnd("ok",results.length,used.length,msg);
     process.stdout.write(JSON.stringify({hookSpecificOutput:{hookEventName:"UserPromptSubmit",additionalContext:msg}}));
-  }catch{ /* fail-soft — 조용히 무주입 */ }
+  }catch{
+    // fetch 타임아웃·네트워크·JSON 파싱 실패 전부. 세분하지 않는다(설계 §3.4) — 이 계측의 목적은
+    //  타임아웃 유실을 세는 것이지 실패 원인을 분류하는 것이 아니다.
+    try{ metricEnd("error",0,0,""); }catch{}
+  }
 });
 ' 2>/dev/null
 

package/version.json

@@ -1,4 +1,4 @@
 {
-  "cli_version": "0.4.30",
+  "cli_version": "0.4.32",
   "skill_version": "1.40.0"
 }