claude-code-session-manager 0.121.0 → 0.122.0

package/src/main/lib/telemetry/auditLog.cjs

@@ -7,8 +7,8 @@
  * state a project's single-writer law needs to arbitrate, and it must
  * survive even if the project's ops root doesn't exist yet.
  *
- * Rotation: on the first append in a new calendar month (UTC) the live file is
- * gzipped to audit-log-<YYYY-MM>.jsonl.gz beside it (YYYY-MM = the month of the
+ * Rotation: on the first append in a new calendar month (UTC), or once the live file passes
+ * 32 MB, the live file is renamed synchronously and then stream-gzipped (async) to audit-log-<YYYY-MM>.jsonl.gz beside it (YYYY-MM = the month of the
  * file's last write) and a fresh live file starts; the newest `auditLogMonths` rolled
  * files are kept. The live file stays plain JSONL, so readers are unaffected.
  *
@@ -23,6 +23,7 @@
 const fs = require('node:fs');
 const path = require('node:path');
 const zlib = require('node:zlib');
+const { pipeline } = require('node:stream/promises');
 
 const { auditLogPath } = require('../scheduler/schedulerPaths.cjs');
 
@@ -55,23 +56,73 @@ const ROLLED_RE = /^audit-log-\d{4}-\d{2}\.jsonl\.gz$/;
 
 function monthKey(d) { return d.toISOString().slice(0, 7); }
 
-/** Roll the live file if its last write was in an earlier month than `now`; prune old rolls. */
-function rotateIfNewMonth(target, now) {
+const MAX_LIVE_BYTES = 32 * 1024 * 1024;
+let rollSeq = 0;
+const inflight = new Set(); // renamed files currently being compressed
+let chain = Promise.resolve();
+const settling = new Set(); // promises of in-progress compressions
+
+function escapeRe(s) { return s.replace(/[.*+?^${}()|[\]\\]/g, '\\$&'); }
+
+function pruneRolled(dir) {
+  const rolls = fs.readdirSync(dir).filter((f) => ROLLED_RE.test(f)).sort();
+  for (const f of rolls.slice(0, Math.max(0, rolls.length - maxRolled()))) {
+    try { fs.unlinkSync(path.join(dir, f)); } catch { /* best-effort */ }
+  }
+}
+
+/** Stream-gzip `pending` onto the month archive (append: members concatenate), then unlink it. */
+async function compressPending(pending, rolled, dir) {
+  let before = 0;
+  try { before = fs.statSync(rolled).size; } catch { /* new archive */ }
+  try {
+    await pipeline(fs.createReadStream(pending), zlib.createGzip(), fs.createWriteStream(rolled, { flags: 'a' }));
+  } catch (e) {
+    // Drop any partial gzip member so the archive stays valid; the pending file is kept for retry.
+    try { if (fs.existsSync(rolled)) fs.truncateSync(rolled, before); } catch { /* best-effort */ }
+    throw e;
+  }
+  fs.unlinkSync(pending);
+  try { pruneRolled(dir); } catch { /* best-effort */ }
+}
+
+function startCompress(pending, rolled, dir) {
+  if (inflight.has(pending)) return;
+  inflight.add(pending);
+  // Serialized: two members appended to one archive concurrently would interleave.
+  const p = chain.then(() => compressPending(pending, rolled, dir))
+    .catch((e) => { console.error('[auditLog] roll compress failed; will retry', e?.message ?? String(e)); })
+    .finally(() => { inflight.delete(pending); settling.delete(p); });
+  chain = p;
+  settling.add(p);
+}
+
+/** Resolves once every in-flight roll compression has finished (success or failure). Test seam. */
+async function settleRolls() {
+  while (settling.size) await Promise.all([...settling]);
+}
+
+/**
+ * Roll the live file if its last write was in an earlier month than `now`, or it
+ * exceeds `maxBytes`. The rename is synchronous; compression is async. Leftover
+ * renamed files from a failed compress are retried here.
+ */
+function rotateIfNeeded(target, now, maxBytes) {
   let st;
   try { st = fs.statSync(target); } catch { return; }
   if (!st.isFile() || st.size === 0) return;
   const oldMonth = monthKey(st.mtime);
-  if (oldMonth === monthKey(now)) return;
+  if (oldMonth === monthKey(now) && st.size <= maxBytes) return;
   const dir = path.dirname(target);
   const base = path.basename(target, path.extname(target));
-  const rolled = path.join(dir, `${base}-${oldMonth}.jsonl.gz`);
-  // appendFileSync: gzip members concatenate, so a pre-existing roll for that month is extended.
-  fs.appendFileSync(rolled, zlib.gzipSync(fs.readFileSync(target)));
-  fs.unlinkSync(target);
-  const rolls = fs.readdirSync(dir).filter((f) => ROLLED_RE.test(f)).sort();
-  for (const f of rolls.slice(0, Math.max(0, rolls.length - maxRolled()))) {
-    try { fs.unlinkSync(path.join(dir, f)); } catch { /* best-effort */ }
+  const pending = path.join(dir, `${base}-${oldMonth}.jsonl.rolling-${Date.now().toString(36)}${(rollSeq++).toString(36)}`);
+  fs.renameSync(target, pending);
+  const pendingRe = new RegExp(`^${escapeRe(base)}-(\\d{4}-\\d{2})\\.jsonl\\.rolling-[a-z0-9]+$`);
+  for (const f of fs.readdirSync(dir)) {
+    const m = pendingRe.exec(f);
+    if (m) startCompress(path.join(dir, f), path.join(dir, `${base}-${m[1]}.jsonl.gz`), dir);
   }
+  try { pruneRolled(dir); } catch { /* best-effort */ }
 }
 
 /**
@@ -92,7 +143,7 @@ function appendAuditEvent(kind, fields = {}, opts = {}) {
     const target = opts.target ?? auditLogPath();
     fs.mkdirSync(path.dirname(target), { recursive: true });
     try {
-      rotateIfNewMonth(target, now);
+      rotateIfNeeded(target, now, opts.maxBytes ?? MAX_LIVE_BYTES);
     } catch (e) {
       console.error('[auditLog] rotation failed', e?.message ?? String(e));
     }
@@ -127,4 +178,4 @@ function readTail(maxBytes, target = auditLogPath()) {
   }
 }
 
-module.exports = { appendAuditEvent, auditLogPath, readTail };
+module.exports = { appendAuditEvent, settleRolls, MAX_LIVE_BYTES, auditLogPath, readTail };

package/src/main/loopEfficiencyHandlers.cjs

@@ -0,0 +1,160 @@
+/**
+ * loopEfficiencyHandlers.cjs — `history:loop-efficiency` IPC. Gathers this
+ * usage week's run records + validator verdicts from the runs dir and feeds
+ * the pure calculator in lib/scheduler/loopEfficiency.cjs.
+ *
+ * Complexity O(runs in window); run logs are immutable after finalize, so
+ * validator verdicts are memoized per log path.
+ */
+
+'use strict';
+
+const fsp = require('fs/promises');
+const path = require('path');
+const { ipcMain } = require('electron');
+const { computeLoopEfficiency } = require('./lib/scheduler/loopEfficiency.cjs');
+const { parseValidationSentinels } = require('./lib/scheduler/validationSentinels.cjs');
+
+const WEEK_MS = 7 * 24 * 3600 * 1000;
+const LOG_TAIL_BYTES = 1024 * 1024;
+const USAGE_TIMEOUT_MS = 5000;
+const VALIDATOR_SLUG_RE = /(^|-)validate-/;
+
+/** @type {Map<string, Array<{ slug: string, verdict: string, reason: string }>>} */
+const validationMemo = new Map();
+
+/** `2026-10-23T01-26-37-026Z` -> epoch ms (NaN if not a run dir name). */
+function runIdToMs(runId) {
+  const m = /^(\d{4}-\d{2}-\d{2})T(\d{2})-(\d{2})-(\d{2})-(\d{3})Z$/.exec(runId);
+  return m ? Date.parse(`${m[1]}T${m[2]}:${m[3]}:${m[4]}.${m[5]}Z`) : NaN;
+}
+
+async function readJson(p) {
+  try { return JSON.parse(await fsp.readFile(p, 'utf8')); } catch { return null; }
+}
+
+/** Last 1 MB of a file as text ('' on failure). */
+async function readTail(p) {
+  let fh;
+  try {
+    fh = await fsp.open(p, 'r');
+    const { size } = await fh.stat();
+    const len = Math.min(size, LOG_TAIL_BYTES);
+    const buf = Buffer.alloc(len);
+    await fh.read(buf, 0, len, size - len);
+    return buf.toString('utf8');
+  } catch {
+    return '';
+  } finally {
+    if (fh) await fh.close().catch(() => {});
+  }
+}
+
+async function validationsForLog(logPath) {
+  const hit = validationMemo.get(logPath);
+  if (hit) return hit;
+  let out = [];
+  const tail = await readTail(logPath);
+  const lines = tail.split('\n');
+  for (let i = lines.length - 1; i >= 0; i--) {
+    if (!lines[i].includes('"type":"result"')) continue;
+    try {
+      const obj = JSON.parse(lines[i]);
+      if (typeof obj.result === 'string') out = parseValidationSentinels(obj.result);
+    } catch { /* torn line — treat as no verdicts */ }
+    break;
+  }
+  validationMemo.set(logPath, out);
+  return out;
+}
+
+/** Never throws; resolves null on any failure or after 5 s. */
+async function defaultUsage(fetchUsage) {
+  let timer;
+  try {
+    const fetchFn = fetchUsage || require('./usage.cjs').fetchUsage;
+    const r = await Promise.race([
+      fetchFn(),
+      new Promise((resolve) => { timer = setTimeout(() => resolve(null), USAGE_TIMEOUT_MS); }),
+    ]);
+    if (!r) return null;
+    // 'ok-stale' = last cached payload served while the API fails / rate-limits.
+    if (r.kind === 'ok' || r.kind === 'ok-stale') return r.data && r.data.usage ? r.data.usage : null;
+    return null;
+  } catch {
+    return null;
+  } finally {
+    if (timer) clearTimeout(timer);
+  }
+}
+
+/**
+ * @param {{ runsRoot?: string, nowMs?: number, usage?: any }} [opts]
+ *   `usage` is the usage payload (`{ seven_day: { utilization, resets_at } }`) or null.
+ */
+async function gatherLoopEfficiency({ runsRoot, nowMs, usage } = {}) {
+  const root = runsRoot || require('./lib/scheduler/schedulerPaths.cjs').runsDir();
+  const now = typeof nowMs === 'number' ? nowMs : Date.now();
+
+  let fromMs = now - WEEK_MS;
+  let weeklyUtilizationPct = null;
+  const sd = usage && usage.seven_day;
+  if (sd && sd.resets_at && typeof sd.utilization === 'number') {
+    const reset = typeof sd.resets_at === 'number' ? sd.resets_at : Date.parse(sd.resets_at);
+    if (Number.isFinite(reset)) {
+      fromMs = reset - WEEK_MS;
+      weeklyUtilizationPct = sd.utilization;
+    }
+  }
+
+  const runs = [];
+  const validations = [];
+  let dirs = [];
+  try { dirs = await fsp.readdir(root); } catch { /* no runs dir */ }
+
+  for (const runId of dirs) {
+    const t = runIdToMs(runId);
+    if (!Number.isFinite(t) || t < fromMs) continue;
+    const runDir = path.join(root, runId);
+    let files;
+    try { files = await fsp.readdir(runDir); } catch { continue; }
+    for (const f of files) {
+      if (!f.endsWith('.meta.json')) continue;
+      const meta = await readJson(path.join(runDir, f));
+      if (!meta || typeof meta !== 'object') continue;
+      // Slug comes from the filename, never meta content (no path traversal via meta.slug).
+      const slug = f.slice(0, -'.meta.json'.length);
+      const vd = await readJson(path.join(runDir, `${slug}.verdicts.json`));
+      const startedAtMs = typeof meta.startedAt === 'number' ? meta.startedAt : t;
+      const cwd = String(meta.cwd == null ? '' : meta.cwd);
+      runs.push({
+        cwd,
+        slug,
+        startedAtMs,
+        costUsd: meta.totalCostUsd,
+        verdict: vd && typeof vd.verdict === 'string' ? vd.verdict : null,
+      });
+      if (VALIDATOR_SLUG_RE.test(slug)) {
+        for (const v of await validationsForLog(path.join(runDir, `${slug}.log`))) {
+          validations.push({
+            cwd,
+            slug: v.slug,
+            verdict: v.verdict === 'verified' ? 'VERIFIED' : 'REFUTED',
+            atMs: startedAtMs,
+          });
+        }
+      }
+    }
+  }
+
+  return computeLoopEfficiency({ runs, validations, fromMs, toMs: now, weeklyUtilizationPct });
+}
+
+function registerLoopEfficiencyHandlers() {
+  ipcMain.handle('history:loop-efficiency', async () => {
+    const usage = await defaultUsage();
+    return gatherLoopEfficiency({ nowMs: Date.now(), usage });
+  });
+}
+
+module.exports = { gatherLoopEfficiency, registerLoopEfficiencyHandlers, defaultUsage };

package/src/main/queueOps.cjs

@@ -18,7 +18,8 @@
  *   - `^until ` at the start of a logical line  → fizzpop-style poll hang
  *   - `^while true` (case insensitive)          → unbounded while loop
  *   - `for .* in $(seq 1 [5-9][0-9][0-9]+)`     → unbounded large seq
- *   - missing frontmatter title / cwd / estimateMinutes
+ *   - missing frontmatter title / cwd (estimateMinutes is optional; if present
+ *     it must be a positive integer)
  *   - cwd doesn't exist on disk (fs.accessSync)
  *   - `--no-verify` or `--no-gpg-sign`          → hook-skip flags
  *
@@ -200,14 +201,8 @@ function lintParsed(slug, raw) {
       }
     }
   }
-  if (fm.estimateMinutes === undefined || !String(fm.estimateMinutes).trim()) {
-    findings.push({
-      rule: 'missing-estimate',
-      line: 1,
-      snippet: 'frontmatter "estimateMinutes" is required',
-      severity: 'warn',
-    });
-  } else {
+  // estimateMinutes is optional (budget.cjs floors a missing one); validate only when present.
+  if (fm.estimateMinutes !== undefined && String(fm.estimateMinutes).trim()) {
     const v = Number(fm.estimateMinutes);
     if (!Number.isInteger(v) || v <= 0) {
       findings.push({