claude-token-saver 3.5.0 → 3.5.2

package/bin/cli.js

@@ -572,6 +572,7 @@ async function main() {
     // stay fast) — reads the cache, kicks a detached refresh when stale, and
     // prints delegation-candidate context for the new session.
     if (hasFlag('--hook')) {
+      const hookCtx = readStdinJson() || {};
       let cache = rs.readRouteScan();
       if (await rs.shouldRescan(cache)) {
         try {
@@ -615,7 +616,18 @@ async function main() {
         }
         lines.push('  제거: claude-token-saver route-scan rules rm <N>');
       }
+      lines.push('위 내용은 사용자의 첫 메시지 주제와 무관하더라도(단순 인사여도) 첫 응답 말미에 `※ [claude-token-saver]` 라벨을 달아 요약 브리핑하세요 — 사용자가 물어볼 때까지 기다리지 말 것.');
       console.log(lines.join('\n'));
+      // Record what was actually briefed so the UserPromptSubmit brief hook
+      // suppresses exactly these — a candidate landing after this read (e.g.
+      // the detached rescan above finishing) still gets briefed next prompt.
+      try {
+        const { seedSessionBriefed } = await import('../src/brief.js');
+        seedSessionBriefed(hookCtx.session_id, [
+          ...open.map((c) => `route|${c.signature}`),
+          ...reviewRules.map((r) => `health|${r.signature}|${r.scope}`),
+        ]);
+      } catch { /* best-effort — worst case is one duplicate brief */ }
       return;
     }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "claude-token-saver",
-  "version": "3.5.0",
+  "version": "3.5.2",
   "description": "Save tokens on Claude Code — spike diagnosis, 1M-context detection, TTL countdown, statusline. (formerly claude-cache-monitor)",
   "type": "module",
   "bin": {

package/src/brief.js

@@ -14,8 +14,15 @@
  * transcript_path) and the "already briefed" markers are keyed by session_id.
  *
  * State: <stateDir>/brief-state.json
- *   { sessions: { [session_id]: { ts, ctxTier, seeded, briefed: [signature] } } }
+ *   { sessions: { [session_id]: { ts, ctxTier, briefed: [signature] } } }
  * Sessions untouched for 7 days are pruned on every write.
+ *
+ * Division of labor with the SessionStart hook: SessionStart prints the
+ * session-start snapshot and records the signatures it actually briefed via
+ * seedSessionBriefed(); this hook emits anything NOT in that record. A
+ * candidate that lands after the SessionStart read (e.g. the detached rescan
+ * finishing between session start and the first prompt) is therefore briefed
+ * on the next prompt instead of being lost for the whole session.
  */
 
 import { readFileSync, writeFileSync, existsSync, mkdirSync, statSync, openSync, readSync, closeSync } from 'node:fs';
@@ -109,20 +116,34 @@ function ctxTierOf(pct) {
 
 const fmtK = (n) => `${Math.round(n / 1000)}k`;
 
+/**
+ * Called by the SessionStart hook after it prints the session-start snapshot:
+ * records the signatures it actually briefed so runBrief suppresses exactly
+ * those and nothing more. Signature format matches runBrief:
+ * `route|<sig>` / `health|<sig>|<scope>`.
+ */
+export function seedSessionBriefed(sessionId, signatures, now = Date.now()) {
+  if (!sessionId || !Array.isArray(signatures) || signatures.length === 0) return;
+  const state = loadState();
+  const s = state.sessions[sessionId] || { ctxTier: 0, briefed: [] };
+  s.briefed = [...new Set([...(s.briefed || []), ...signatures])];
+  s.ts = now;
+  state.sessions[sessionId] = s;
+  saveState(state, now);
+}
+
 /**
  * Compute the briefing for one prompt-submit event. Returns the text to
  * inject, or null when nothing new happened. Mutates + persists state.
  *
- * On a session's FIRST event, route/rule-health signatures are seeded as
- * already-briefed WITHOUT emitting them — the SessionStart hook covered the
- * session-start snapshot; this hook only owns what changes mid-session.
- * Context tiers are NOT seeded: a session that starts (or resumes) already
- * past a threshold still deserves the warning once.
+ * Route/rule-health signatures already briefed by the SessionStart hook
+ * (recorded via seedSessionBriefed) are skipped; every other fresh signature
+ * is emitted, including on the session's first event.
  */
 export async function runBrief({ sessionId, transcriptPath, now = Date.now() }) {
   if (!sessionId) return null;
   const state = loadState();
-  const s = state.sessions[sessionId] || { ctxTier: 0, seeded: false, briefed: [] };
+  const s = state.sessions[sessionId] || { ctxTier: 0, briefed: [] };
   const items = [];
 
   // ── context tier crossing (per-session) ──
@@ -158,16 +179,12 @@ export async function runBrief({ sessionId, transcriptPath, now = Date.now() })
       briefed.add(sig);
       fresh.push(['health', r]);
     }
-    if (s.seeded) {
-      for (const [kind, x] of fresh) {
-        items.push(kind === 'route'
-          ? `새 위임 후보가 감지되었습니다 — "${x.label}" 유형 ${x.count}회 반복(statusline의 route? R${x.id} 칩). 등록: claude-token-saver harness promote R${x.id} --project|--global (적용 범위는 사용자에게 확인) / 무시: route-scan dismiss ${x.id}`
-          : `승인된 위임 룰의 최근 에러율이 기준(20%)을 넘었습니다 — "${x.label}" (statusline의 rule-health 칩). 조건 좁히기/제거를 사용자와 상의하세요: claude-token-saver route-scan rules`);
-      }
+    for (const [kind, x] of fresh) {
+      items.push(kind === 'route'
+        ? `새 위임 후보가 감지되었습니다 — "${x.label}" 유형 ${x.count}회 반복(statusline의 route? R${x.id} 칩). 등록: claude-token-saver harness promote R${x.id} --project|--global (적용 범위는 사용자에게 확인) / 무시: route-scan dismiss ${x.id}`
+        : `승인된 위임 룰의 최근 에러율이 기준(20%)을 넘었습니다 — "${x.label}" (statusline의 rule-health 칩). 조건 좁히기/제거를 사용자와 상의하세요: claude-token-saver route-scan rules`);
     }
-    // First event: session-start snapshot is SessionStart's job — swallow it.
     s.briefed = [...briefed];
-    s.seeded = true;
   } catch { /* caches unreadable — ctx briefing above still applies */ }
 
   s.ts = now;

package/src/installer.js

@@ -223,6 +223,11 @@ export function installSessionStartHook() {
   }
 
   settings.hooks = settings.hooks || {};
+  // A present-but-non-array value is schema-invalid, but it's the user's
+  // data — back off instead of silently replacing it.
+  if (settings.hooks.SessionStart !== undefined && !Array.isArray(settings.hooks.SessionStart)) {
+    return { path: file, action: 'skipped', reason: 'hooks.SessionStart is not an array — fix settings.json manually' };
+  }
   const list = Array.isArray(settings.hooks.SessionStart) ? settings.hooks.SessionStart : [];
   const already = list.some((m) =>
     Array.isArray(m?.hooks) && m.hooks.some((h) => typeof h?.command === 'string' && h.command.includes('route-scan --hook')),
@@ -260,6 +265,11 @@ export function installBriefHook() {
   }
 
   settings.hooks = settings.hooks || {};
+  // A present-but-non-array value is schema-invalid, but it's the user's
+  // data — back off instead of silently replacing it.
+  if (settings.hooks.UserPromptSubmit !== undefined && !Array.isArray(settings.hooks.UserPromptSubmit)) {
+    return { path: file, action: 'skipped', reason: 'hooks.UserPromptSubmit is not an array — fix settings.json manually' };
+  }
   const list = Array.isArray(settings.hooks.UserPromptSubmit) ? settings.hooks.UserPromptSubmit : [];
   const already = list.some((m) =>
     Array.isArray(m?.hooks) && m.hooks.some((h) => typeof h?.command === 'string' && h.command.includes('brief --hook')),