@ngockhoale/ukit 3.4.47 → 3.4.48

package/scripts/perf/diff-perf-findings.mjs

@@ -1,136 +0,0 @@
-#!/usr/bin/env node
-/**
- * diff-perf-findings.mjs — findings-level diff between two audit snapshots
- * (TASK-006, SPEC §5 FR-008).
- *
- * Why a separate script and not a mode of `audit-perf.mjs`: the audit takes live
- * measurements, so "compare two snapshots" is a read-only operation that must
- * work against archived JSON long after the telemetry that produced it rotated
- * away. This script never touches telemetry at all — it reads two reports and
- * prints their deltas.
- *
- * Every snapshot is a point in time over a live dataset (see the O2 finding:
- * 30k → 34k rows in two hours), so both snapshots' `generatedAt` /
- * `telemetryRows` / `telemetryFiles` provenance is printed before any delta —
- * two reports are only comparable once their provenance is known.
- *
- * Deltas reported, per finding id:
- *   + <id> added    — present in b, absent from a
- *   - <id> removed  — present in a, absent from b (never a crash)
- *   ~ <id> changed  — same id, different status and/or different measured numbers
- *
- * Usage:
- *   node scripts/perf/diff-perf-findings.mjs <a.json> <b.json>
- *
- * Exit: 0 = compared (with or without deltas, including a self-diff); 2 = usage,
- * unreadable file, or a report that is not a findings snapshot.
- */
-import fs from 'node:fs';
-
-const METRIC_KEYS = ['p50ms', 'p95ms', 'totalMs', 'invocations'];
-
-function usage() {
-  process.stderr.write('usage: node scripts/perf/diff-perf-findings.mjs <a.json> <b.json>\n');
-}
-
-function loadSnapshot(file) {
-  let report;
-  try {
-    report = JSON.parse(fs.readFileSync(file, 'utf8'));
-  } catch (error) {
-    throw new Error(`cannot read findings snapshot ${file}: ${error.message}`);
-  }
-  if (!report || !Array.isArray(report.findings)) {
-    throw new Error(`${file} is not a findings snapshot (no findings[] array)`);
-  }
-  return report;
-}
-
-function formatNumber(value) {
-  if (typeof value !== 'number' || !Number.isFinite(value)) return 'n/a';
-  return Number.isInteger(value) ? String(value) : value.toFixed(1);
-}
-
-function provenanceLine(label, file, report) {
-  const rows = formatNumber(report.telemetryRows);
-  const files = formatNumber(report.telemetryFiles);
-  const generatedAt = typeof report.generatedAt === 'string' ? report.generatedAt : 'n/a';
-  return `${label}: ${file} — generatedAt ${generatedAt}, telemetryRows ${rows} / telemetryFiles ${files}`;
-}
-
-/** Metric drift for one finding pair, as `key a → b` fragments. */
-function metricDrift(before, after) {
-  return METRIC_KEYS
-    .filter((key) => before[key] !== after[key])
-    .map((key) => `${key} ${formatNumber(before[key])} → ${formatNumber(after[key])}`);
-}
-
-function diffFindings(a, b) {
-  const before = new Map(a.findings.map((item) => [item.id, item]));
-  const after = new Map(b.findings.map((item) => [item.id, item]));
-  const deltas = [];
-
-  for (const [id, item] of after) {
-    if (!before.has(id)) {
-      deltas.push({ kind: 'added', id, line: `+ ${id} added (status ${item.status})` });
-      continue;
-    }
-    const previous = before.get(id);
-    const drift = metricDrift(previous, item);
-    if (previous.status !== item.status) {
-      const suffix = drift.length ? ` (${drift.join(', ')})` : '';
-      deltas.push({
-        kind: 'changed',
-        id,
-        line: `~ ${id} status ${previous.status} → ${item.status}${suffix}`,
-      });
-    } else if (drift.length) {
-      deltas.push({ kind: 'changed', id, line: `~ ${id} ${drift.join(', ')}` });
-    }
-  }
-
-  for (const [id, item] of before) {
-    if (after.has(id)) continue;
-    deltas.push({ kind: 'removed', id, line: `- ${id} removed (was ${item.status})` });
-  }
-
-  return deltas;
-}
-
-function main(argv) {
-  const [fileA, fileB] = argv;
-  if (!fileA || !fileB || argv.length > 2) {
-    usage();
-    return 2;
-  }
-
-  let snapshotA;
-  let snapshotB;
-  try {
-    snapshotA = loadSnapshot(fileA);
-    snapshotB = loadSnapshot(fileB);
-  } catch (error) {
-    process.stderr.write(`${error.message}\n`);
-    return 2;
-  }
-
-  const deltas = diffFindings(snapshotA, snapshotB);
-  const count = (kind) => deltas.filter((delta) => delta.kind === kind).length;
-  const lines = [
-    '# perf-findings diff',
-    provenanceLine('a', fileA, snapshotA),
-    provenanceLine('b', fileB, snapshotB),
-    '',
-  ];
-  if (deltas.length) {
-    lines.push(...deltas.map((delta) => delta.line), '');
-  }
-  lines.push(
-    `${deltas.length} delta${deltas.length === 1 ? '' : 's'}: `
-    + `${count('added')} added, ${count('removed')} removed, ${count('changed')} changed`,
-  );
-  process.stdout.write(`${lines.join('\n')}\n`);
-  return 0;
-}
-
-process.exitCode = main(process.argv.slice(2));

package/scripts/perf/perf-findings.json

@@ -1,474 +0,0 @@
-{
-  "generatedAt": "2026-09-20T23:07:33.744Z",
-  "schema": "ukit-perf-findings/1",
-  "telemetryRows": 62854,
-  "telemetryFiles": 120,
-  "sources": {
-    "telemetry": {
-      "dir": "/Volumes/KHOA_EXTENAL/WORKING_PROJECT/WORKING/UKit/.ukit/storage/cache/hook-latency",
-      "rows": 62854,
-      "files": 120
-    },
-    "settings": "/Volumes/KHOA_EXTENAL/WORKING_PROJECT/WORKING/UKit/template_project/.claude/settings.json",
-    "hooksDir": "/Volumes/KHOA_EXTENAL/WORKING_PROJECT/WORKING/UKit/template_project/.claude/hooks",
-    "benchmarkIterations": 2,
-    "gitLogWindowDays": 14
-  },
-  "findings": [
-    {
-      "id": "chain-pretooluse-read-grep-glob",
-      "area": "hook chain — PreToolUse Read|Grep|Glob — 1 hook",
-      "evidence": "telemetry",
-      "evidenceDetail": ".ukit/storage/cache/hook-latency — 62854 rows across 120 session files, span 2026-09-17T10:50:18.670Z → 2026-09-20T23:07:33.053Z; measured per-call hook cost for the group's matching tools (n=2340, p50 37ms, 2 hook rows/call; paths: 404 direct / 1936 chain-runner). Hooks: sensitive-data-guard.mjs:8 (18s)",
-      "p50ms": 37,
-      "p95ms": 72,
-      "totalMs": 78128,
-      "invocations": 2340,
-      "rootCause": "3 process spawns per call (1 settings rows × 3: wrapper bash + node runtime + advisory telemetry node child), against a measured floor of bash 3.1ms + node 16.1ms + telemetry finish 19.5ms. Measured p50 is under the 50ms hotspot bar; consolidation is still the mechanism that removes the spawns.",
-      "fixTask": "TASK-234",
-      "status": "confirmed"
-    },
-    {
-      "id": "chain-pretooluse-edit-write",
-      "area": "hook chain — PreToolUse Edit|Write — 1 hook",
-      "evidence": "telemetry",
-      "evidenceDetail": ".ukit/storage/cache/hook-latency — 62854 rows across 120 session files, span 2026-09-17T10:50:18.670Z → 2026-09-20T23:07:33.053Z; measured per-call hook cost for the group's matching tools (n=728, p50 331ms, 6 hook rows/call; paths: 402 direct / 326 chain-runner). Hooks: context-hardcap-gate.sh:8 (73s)",
-      "p50ms": 331,
-      "p95ms": 429,
-      "totalMs": 249733,
-      "invocations": 728,
-      "rootCause": "3 process spawns per call (1 settings rows × 3: wrapper bash + node runtime + advisory telemetry node child), against a measured floor of bash 3.1ms + node 16.1ms + telemetry finish 19.5ms. Measured p50 exceeds the 50ms hotspot bar, so the spawn multiplier is the dominant term.",
-      "fixTask": "TASK-234",
-      "status": "confirmed"
-    },
-    {
-      "id": "chain-pretooluse-bash",
-      "area": "hook chain — PreToolUse Bash — 1 hook",
-      "evidence": "telemetry",
-      "evidenceDetail": ".ukit/storage/cache/hook-latency — 62854 rows across 120 session files, span 2026-09-17T10:50:18.670Z → 2026-09-20T23:07:33.053Z; measured per-call hook cost for the group's matching tools (n=4853, p50 307ms, 6 hook rows/call; paths: 2519 direct / 2334 chain-runner). Hooks: verification-guard.sh:8 (65s)",
-      "p50ms": 307,
-      "p95ms": 474,
-      "totalMs": 1559455,
-      "invocations": 4853,
-      "rootCause": "3 process spawns per call (1 settings rows × 3: wrapper bash + node runtime + advisory telemetry node child), against a measured floor of bash 3.1ms + node 16.1ms + telemetry finish 19.5ms. Measured p50 exceeds the 50ms hotspot bar, so the spawn multiplier is the dominant term.",
-      "fixTask": "TASK-234",
-      "status": "confirmed"
-    },
-    {
-      "id": "chain-posttooluse-read-grep-glob",
-      "area": "hook chain — PostToolUse Read|Grep|Glob — 1 hook",
-      "evidence": "telemetry",
-      "evidenceDetail": ".ukit/storage/cache/hook-latency — 62854 rows across 120 session files, span 2026-09-17T10:50:18.670Z → 2026-09-20T23:07:33.053Z; measured per-call hook cost for the group's matching tools (n=2329, p50 37ms, 2 hook rows/call; paths: 393 direct / 1936 chain-runner). Hooks: record-execution.mjs:8 (18s)",
-      "p50ms": 37,
-      "p95ms": 118,
-      "totalMs": 100380,
-      "invocations": 2329,
-      "rootCause": "3 process spawns per call (1 settings rows × 3: wrapper bash + node runtime + advisory telemetry node child), against a measured floor of bash 3.1ms + node 16.1ms + telemetry finish 19.5ms. Measured p50 is under the 50ms hotspot bar; consolidation is still the mechanism that removes the spawns.",
-      "fixTask": "TASK-234",
-      "status": "confirmed"
-    },
-    {
-      "id": "chain-posttooluse-edit-write",
-      "area": "hook chain — PostToolUse Edit|Write — 1 hook",
-      "evidence": "telemetry",
-      "evidenceDetail": ".ukit/storage/cache/hook-latency — 62854 rows across 120 session files, span 2026-09-17T10:50:18.670Z → 2026-09-20T23:07:33.053Z; measured per-call hook cost for the group's matching tools (n=713, p50 130ms, 3 hook rows/call; paths: 397 direct / 316 chain-runner). Hooks: task-watchdog.sh:8 (34s)",
-      "p50ms": 130,
-      "p95ms": 222,
-      "totalMs": 102401,
-      "invocations": 713,
-      "rootCause": "3 process spawns per call (1 settings rows × 3: wrapper bash + node runtime + advisory telemetry node child), against a measured floor of bash 3.1ms + node 16.1ms + telemetry finish 19.5ms. Measured p50 exceeds the 50ms hotspot bar, so the spawn multiplier is the dominant term.",
-      "fixTask": "TASK-234",
-      "status": "confirmed"
-    },
-    {
-      "id": "chain-posttooluse-bash",
-      "area": "hook chain — PostToolUse Bash — 1 hook",
-      "evidence": "telemetry",
-      "evidenceDetail": ".ukit/storage/cache/hook-latency — 62854 rows across 120 session files, span 2026-09-17T10:50:18.670Z → 2026-09-20T23:07:33.053Z; measured per-call hook cost for the group's matching tools (n=4701, p50 87ms, 2 hook rows/call; paths: 2406 direct / 2295 chain-runner). Hooks: record-execution.mjs:8 (30s)",
-      "p50ms": 87,
-      "p95ms": 150,
-      "totalMs": 449914,
-      "invocations": 4701,
-      "rootCause": "3 process spawns per call (1 settings rows × 3: wrapper bash + node runtime + advisory telemetry node child), against a measured floor of bash 3.1ms + node 16.1ms + telemetry finish 19.5ms. Measured p50 exceeds the 50ms hotspot bar, so the spawn multiplier is the dominant term.",
-      "fixTask": "TASK-234",
-      "status": "confirmed"
-    },
-    {
-      "id": "chain-userpromptsubmit-all",
-      "area": "hook chain — UserPromptSubmit (all tools) — 1 hook",
-      "evidence": "telemetry",
-      "evidenceDetail": ".ukit/storage/cache/hook-latency — 62854 rows across 120 session files, span 2026-09-17T10:50:18.670Z → 2026-09-20T23:07:33.053Z; measured per-call hook cost for the group's matching tools (n=349, p50 291ms, 4 hook rows/call; paths: 200 direct / 149 chain-runner). Hooks: context-window-guard.sh:10 (60s)",
-      "p50ms": 291,
-      "p95ms": 582,
-      "totalMs": 101608,
-      "invocations": 349,
-      "rootCause": "3 process spawns per call (1 settings rows × 3: wrapper bash + node runtime + advisory telemetry node child), against a measured floor of bash 3.1ms + node 16.1ms + telemetry finish 19.5ms. Measured p50 exceeds the 50ms hotspot bar, so the spawn multiplier is the dominant term.",
-      "fixTask": "TASK-234",
-      "status": "confirmed"
-    },
-    {
-      "id": "chain-stop-all",
-      "area": "hook chain — Stop (all tools) — 1 hook",
-      "evidence": "telemetry",
-      "evidenceDetail": ".ukit/storage/cache/hook-latency — 62854 rows across 120 session files, span 2026-09-17T10:50:18.670Z → 2026-09-20T23:07:33.053Z; measured per-call hook cost for the group's matching tools (n=116, p50 81ms, 1 hook rows/call; paths: 113 direct / 3 chain-runner). Hooks: completion-gate.sh:8 (18s)",
-      "p50ms": 81,
-      "p95ms": 111,
-      "totalMs": 9890,
-      "invocations": 116,
-      "rootCause": "3 process spawns per call (1 settings rows × 3: wrapper bash + node runtime + advisory telemetry node child), against a measured floor of bash 3.1ms + node 16.1ms + telemetry finish 19.5ms. Measured p50 exceeds the 50ms hotspot bar, so the spawn multiplier is the dominant term.",
-      "fixTask": "TASK-234",
-      "status": "confirmed"
-    },
-    {
-      "id": "chain-precompact-all",
-      "area": "hook chain — PreCompact (all tools) — 1 hook",
-      "evidence": "telemetry",
-      "evidenceDetail": ".ukit/storage/cache/hook-latency — 62854 rows across 120 session files, span 2026-09-17T10:50:18.670Z → 2026-09-20T23:07:33.053Z; measured per-call hook cost for the group's matching tools (n=6, p50 49ms, 1 hook rows/call; paths: 0 direct / 6 chain-runner). Hooks: reinject-context.sh:8 (18s)",
-      "p50ms": 49,
-      "p95ms": 53,
-      "totalMs": 260,
-      "invocations": 6,
-      "rootCause": "3 process spawns per call (1 settings rows × 3: wrapper bash + node runtime + advisory telemetry node child), against a measured floor of bash 3.1ms + node 16.1ms + telemetry finish 19.5ms. Measured p50 is under the 50ms hotspot bar; consolidation is still the mechanism that removes the spawns.",
-      "fixTask": "TASK-234",
-      "status": "confirmed"
-    },
-    {
-      "id": "chain-sessionstart-all",
-      "area": "hook chain — SessionStart (all tools) — 1 hook",
-      "evidence": "telemetry",
-      "evidenceDetail": ".ukit/storage/cache/hook-latency — 62854 rows across 120 session files, span 2026-09-17T10:50:18.670Z → 2026-09-20T23:07:33.053Z; measured per-call hook cost for the group's matching tools (n=113, p50 119ms, 2 hook rows/call; paths: 38 direct / 75 chain-runner). Hooks: handoff-resume.sh:8 (50s)",
-      "p50ms": 119,
-      "p95ms": 289,
-      "totalMs": 16463,
-      "invocations": 113,
-      "rootCause": "3 process spawns per call (1 settings rows × 3: wrapper bash + node runtime + advisory telemetry node child), against a measured floor of bash 3.1ms + node 16.1ms + telemetry finish 19.5ms. Measured p50 exceeds the 50ms hotspot bar, so the spawn multiplier is the dominant term.",
-      "fixTask": "TASK-234",
-      "status": "confirmed"
-    },
-    {
-      "id": "chain-sessionend-all",
-      "area": "hook chain — SessionEnd (all tools) — 1 hook",
-      "evidence": "telemetry",
-      "evidenceDetail": ".ukit/storage/cache/hook-latency — 62854 rows across 120 session files, span 2026-09-17T10:50:18.670Z → 2026-09-20T23:07:33.053Z; measured per-call hook cost for the group's matching tools (n=1, p50 2020ms, 1 hook rows/call; paths: 0 direct / 1 chain-runner). Hooks: session-episode.sh:8 (18s)",
-      "p50ms": 2020,
-      "p95ms": 2020,
-      "totalMs": 2020,
-      "invocations": 1,
-      "rootCause": "3 process spawns per call (1 settings rows × 3: wrapper bash + node runtime + advisory telemetry node child), against a measured floor of bash 3.1ms + node 16.1ms + telemetry finish 19.5ms. Measured p50 exceeds the 50ms hotspot bar, so the spawn multiplier is the dominant term.",
-      "fixTask": "TASK-234",
-      "status": "confirmed"
-    },
-    {
-      "id": "hook-hook-chain-runner",
-      "area": "single hook — hook-chain-runner",
-      "evidence": "telemetry",
-      "evidenceDetail": ".ukit/storage/cache/hook-latency — 62854 rows across 120 session files, span 2026-09-17T10:50:18.670Z → 2026-09-20T23:07:33.053Z; standalone re-run p50 not benchmarked (script not resolvable from this root)",
-      "p50ms": 80,
-      "p95ms": 375,
-      "totalMs": 1269815,
-      "invocations": 9453,
-      "rootCause": "per-invocation cost is dominated by the 3-spawn hook row (wrapper bash 3.1ms + node 16.1ms + telemetry finish 19.5ms), plus this script's own work",
-      "fixTask": "TASK-234",
-      "status": "confirmed"
-    },
-    {
-      "id": "hook-skill-router-sh",
-      "area": "single hook — skill-router.sh",
-      "evidence": "telemetry",
-      "evidenceDetail": ".ukit/storage/cache/hook-latency — 62854 rows across 120 session files, span 2026-09-17T10:50:18.670Z → 2026-09-20T23:07:33.053Z; standalone re-run p50 not benchmarked (script not resolvable from this root)",
-      "p50ms": 52,
-      "p95ms": 134,
-      "totalMs": 68680,
-      "invocations": 1089,
-      "rootCause": "per-invocation cost is dominated by the 3-spawn hook row (wrapper bash 3.1ms + node 16.1ms + telemetry finish 19.5ms), plus this script's own work",
-      "fixTask": "TASK-234",
-      "status": "confirmed"
-    },
-    {
-      "id": "hook-vision-router-sh",
-      "area": "single hook — vision-router.sh",
-      "evidence": "telemetry",
-      "evidenceDetail": ".ukit/storage/cache/hook-latency — 62854 rows across 120 session files, span 2026-09-17T10:50:18.670Z → 2026-09-20T23:07:33.053Z; standalone re-run p50 not benchmarked (script not resolvable from this root)",
-      "p50ms": 73,
-      "p95ms": 220,
-      "totalMs": 29834,
-      "invocations": 325,
-      "rootCause": "per-invocation cost is dominated by the 3-spawn hook row (wrapper bash 3.1ms + node 16.1ms + telemetry finish 19.5ms), plus this script's own work",
-      "fixTask": "TASK-234",
-      "status": "confirmed"
-    },
-    {
-      "id": "hook-block-dangerous-sh",
-      "area": "single hook — block-dangerous.sh",
-      "evidence": "telemetry",
-      "evidenceDetail": ".ukit/storage/cache/hook-latency — 62854 rows across 120 session files, span 2026-09-17T10:50:18.670Z → 2026-09-20T23:07:33.053Z; standalone re-run p50 not benchmarked (script not resolvable from this root)",
-      "p50ms": 83,
-      "p95ms": 237,
-      "totalMs": 461434,
-      "invocations": 5107,
-      "rootCause": "per-invocation cost is dominated by the 3-spawn hook row (wrapper bash 3.1ms + node 16.1ms + telemetry finish 19.5ms), plus this script's own work",
-      "fixTask": "TASK-234",
-      "status": "confirmed"
-    },
-    {
-      "id": "hook-context-hardcap-gate-sh",
-      "area": "single hook — context-hardcap-gate.sh",
-      "evidence": "telemetry",
-      "evidenceDetail": ".ukit/storage/cache/hook-latency — 62854 rows across 120 session files, span 2026-09-17T10:50:18.670Z → 2026-09-20T23:07:33.053Z; standalone re-run p50 not benchmarked (script not resolvable from this root)",
-      "p50ms": 55,
-      "p95ms": 88,
-      "totalMs": 334184,
-      "invocations": 5650,
-      "rootCause": "per-invocation cost is dominated by the 3-spawn hook row (wrapper bash 3.1ms + node 16.1ms + telemetry finish 19.5ms), plus this script's own work",
-      "fixTask": "TASK-234",
-      "status": "confirmed"
-    },
-    {
-      "id": "hook-completion-gate-sh",
-      "area": "single hook — completion-gate.sh",
-      "evidence": "telemetry",
-      "evidenceDetail": ".ukit/storage/cache/hook-latency — 62854 rows across 120 session files, span 2026-09-17T10:50:18.670Z → 2026-09-20T23:07:33.053Z; standalone re-run p50 not benchmarked (script not resolvable from this root)",
-      "p50ms": 80,
-      "p95ms": 108,
-      "totalMs": 9881,
-      "invocations": 119,
-      "rootCause": "per-invocation cost is dominated by the 3-spawn hook row (wrapper bash 3.1ms + node 16.1ms + telemetry finish 19.5ms), plus this script's own work",
-      "fixTask": "TASK-234",
-      "status": "confirmed"
-    },
-    {
-      "id": "hook-project-important-tail",
-      "area": "project-important.sh — PROJECT_IMPORTANT.md injection tail",
-      "evidence": "telemetry",
-      "evidenceDetail": ".ukit/storage/cache/hook-latency — 62854 rows across 120 session files, span 2026-09-17T10:50:18.670Z → 2026-09-20T23:07:33.053Z; bimodal across 47 invocations: p50 38ms vs p95 61ms vs max 62ms (paths seen: PreToolUse, SessionStart) — a cheap path and an expensive path with no cache between them",
-      "p50ms": null,
-      "p95ms": 61,
-      "totalMs": 1962,
-      "invocations": 47,
-      "rootCause": "the renderer re-reads PROJECT_IMPORTANT.md and re-runs its full work (strict UTF-8 decode, control-character validation, sensitive-value scan, envelope render) on every invocation; there is no mtime/content-keyed cache, so repeat runs pay the whole cost instead of replaying the previous result",
-      "fixTask": "TASK-235",
-      "status": "confirmed"
-    },
-    {
-      "id": "record-execution-multiplicity",
-      "area": "record-execution.sh — PostToolUse receipt writer registered in 0 matcher groups",
-      "evidence": "telemetry",
-      "evidenceDetail": ".ukit/storage/cache/hook-latency — 62854 rows across 120 session files, span 2026-09-17T10:50:18.670Z → 2026-09-20T23:07:33.053Z; measured fire count per tool call = 4 (distribution {\"1\":4873,\"3\":1,\"4\":1} over 4875 calls). The \"once per matching matcher group\" repeat hypothesis is REFUTED: settings.json names the hook in 0 groups, but those matchers are mutually exclusive per tool, so it runs exactly once per call. Its real cost is breadth — highest-invocation hook measured (n=4880, p50 39ms, total 207226ms).",
-      "p50ms": 39,
-      "p95ms": 63,
-      "totalMs": 207226,
-      "invocations": 4880,
-      "rootCause": "not a duplication bug: the matchers (Read|Grep|Glob, Edit|Write, Bash) are mutually exclusive, so at most one group fires per tool. The cost is that one full 3-spawn hook row is paid on every tool call of every kind — breadth, not repeats.",
-      "fixTask": null,
-      "status": "not-a-bug"
-    },
-    {
-      "id": "index-router-cold-start",
-      "area": "index/router helpers invoked per prompt (skill-router → resolve-context / route-task / query-index)",
-      "evidence": "telemetry",
-      "evidenceDetail": ".ukit/storage/cache/hook-latency — 62854 rows across 120 session files, span 2026-09-17T10:50:18.670Z → 2026-09-20T23:07:33.053Z; UserPromptSubmit groups: n=349, p50 291ms, p95 582ms, 4 hook rows/group. Standalone cold starts: skill-router.sh 105.2ms, query-index.mjs 23.6ms, route-task.mjs 26.4ms, resolve-context.mjs 21.9ms",
-      "p50ms": 291,
-      "p95ms": 582,
-      "totalMs": 101608,
-      "invocations": 349,
-      "rootCause": "every router helper pays a fresh node cold start (no warm process is reused), and skill-router.sh additionally imports the index core in-process per invocation; the cost is process boot plus module load, not the routing decision itself",
-      "fixTask": "TASK-234",
-      "status": "confirmed"
-    },
-    {
-      "id": "c29-c32-hot-path-additions",
-      "area": "recent hot-path additions — 24 files across 18 commits",
-      "evidence": "git-log",
-      "evidenceDetail": "git log --since=14 days ago --diff-filter=A -- template_project/.claude/hooks template_project/.claude/ukit/runtime template_project/.claude/ukit/index: template_project/.claude/hooks/block-dangerous.mjs, template_project/.claude/hooks/record-execution.mjs, template_project/.claude/hooks/sensitive-data-guard.mjs, template_project/.claude/ukit/runtime/hook-field-salvage.mjs, template_project/.claude/hooks/session-episode.sh, template_project/.claude/hooks/project-important.sh, template_project/.claude/hooks/project-important.sh, template_project/.claude/ukit/runtime/project-important.mjs, template_project/.claude/ukit/runtime/sensitive-value-scanner.mjs, template_project/.claude/ukit/runtime/hook-chain-budget.mjs, template_project/.claude/ukit/runtime/stop-coordinator.mjs, template_project/.claude/ukit/runtime/hook-payload-store.mjs, +12 more",
-      "p50ms": null,
-      "p95ms": null,
-      "totalMs": null,
-      "invocations": 24,
-      "rootCause": "each added hook file becomes another settings.json row on the event it joins, i.e. another 3-spawn hook row per call (SPEC §1.3 consolidation is what removes them); each added runtime module is an extra cold-start import paid per invocation. The regression is additive spawn count, not one slow file.",
-      "fixTask": "TASK-234",
-      "status": "confirmed"
-    },
-    {
-      "id": "existing-chain-runner-baseline",
-      "area": "hook-chain-runner.mjs — already-consolidated omp path (TASK-234 reuse target)",
-      "evidence": "telemetry",
-      "evidenceDetail": ".ukit/storage/cache/hook-latency — 62854 rows across 120 session files, span 2026-09-17T10:50:18.670Z → 2026-09-20T23:07:33.053Z; a whole chain of 1 scripts runs in ONE node process at p50 80ms / p95 375ms (n=9453), while a Claude settings group of comparable size pays 3 spawns per script. Nested per-script rows: record-execution.mjs p50 5ms, sensitive-data-guard.mjs p50 4ms, handoff-model-guard.sh p50 39ms, context-hardcap-gate.sh p50 68ms, auto-allow-bash.sh p50 72ms, compress-output.sh p50 73ms",
-      "p50ms": 80,
-      "p95ms": 375,
-      "totalMs": 1269815,
-      "invocations": 9453,
-      "rootCause": "no defect — this is the measured proof that one-process-per-group execution is achievable with per-script telemetry intact, so TASK-234 should re-point settings.json at this runner rather than build a new one",
-      "fixTask": "TASK-234",
-      "status": "confirmed"
-    },
-    {
-      "id": "process-floor",
-      "area": "process spawn floor (bash + node + advisory telemetry child)",
-      "evidence": "micro-benchmark",
-      "evidenceDetail": "live spawnSync over 2 iterations: bash -c true p50 3.13ms, node -e 0 p50 16.11ms, node hook-telemetry.mjs --finish p50 19.50ms",
-      "p50ms": 38.73741700000001,
-      "p95ms": 38.73741700000001,
-      "totalMs": null,
-      "invocations": 0,
-      "rootCause": "each settings.json hook row is a bash wrapper that boots node, then arms a second node child for advisory telemetry; this floor is paid per row, so it multiplies by the row count of the event",
-      "fixTask": "TASK-234",
-      "status": "not-a-bug"
-    },
-    {
-      "id": "telemetry-append-overhead",
-      "area": "telemetry append overhead — appendTelemetryRow, in-process against a throwaway project root",
-      "evidence": "micro-benchmark",
-      "evidenceDetail": "appendTelemetryRow over 2 iterations against a temp project root (the audited telemetry dir is never written): p50 0.407ms, p95 0.407ms, max 0.407ms, total 0.464ms; writer template_project/.claude/ukit/runtime/hook-telemetry.mjs",
-      "p50ms": 0.407125,
-      "p95ms": 0.407125,
-      "totalMs": 0.464083,
-      "invocations": 2,
-      "rootCause": "one synchronous append costs p50 0.407ms (under the 50ms hotspot bar): the observability layer's own I/O is bounded by a single append plus an amortized 1-in-16 sampled sweep, so it is not the hook chain's dominant term",
-      "fixTask": null,
-      "status": "not-a-bug"
-    },
-    {
-      "id": "stdin-stage-vs-exec-split",
-      "area": "stdin stage vs execution split (O3 — the wall-time tail attributed)",
-      "evidence": "unavailable",
-      "evidenceDetail": ".ukit/storage/cache/hook-latency — 62854 rows across 120 session files, span 2026-09-17T10:50:18.670Z → 2026-09-20T23:07:33.053Z; no row carries `stdinStageMs` or `stageMs`, so the exec-vs-stage split is unmeasured on this tree. Old rows keep aggregating exactly as before — the fields are additive and optional, so their absence is reported rather than guessed at as 0ms.",
-      "p50ms": null,
-      "p95ms": null,
-      "totalMs": null,
-      "invocations": 0,
-      "rootCause": "the split needs rows written after the fields shipped; a dataset predating them yields no numbers at all (null, not zero), which is the null-safe posture the audit reports here",
-      "fixTask": null,
-      "status": "deferred"
-    },
-    {
-      "id": "hook-sensitive-data-guard-mjs-8-unmeasured",
-      "area": "single hook — sensitive-data-guard.mjs:8 (registered in settings.json, no telemetry rows)",
-      "evidence": "unavailable",
-      "evidenceDetail": ".ukit/storage/cache/hook-latency — 62854 rows across 120 session files, span 2026-09-17T10:50:18.670Z → 2026-09-20T23:07:33.053Z; no row exists for this script, so no per-invocation number is reported. It still costs 3 spawns on its event and is therefore already counted inside that chain's total.",
-      "p50ms": null,
-      "p95ms": null,
-      "totalMs": null,
-      "invocations": 0,
-      "rootCause": "no measurement exists for sensitive-data-guard.mjs:8 on this tree — its cost stays unset (evidenced by absence) rather than estimated; the chain it belongs to carries the spawn floor",
-      "fixTask": null,
-      "status": "deferred"
-    },
-    {
-      "id": "hook-context-hardcap-gate-sh-8-unmeasured",
-      "area": "single hook — context-hardcap-gate.sh:8 (registered in settings.json, no telemetry rows)",
-      "evidence": "unavailable",
-      "evidenceDetail": ".ukit/storage/cache/hook-latency — 62854 rows across 120 session files, span 2026-09-17T10:50:18.670Z → 2026-09-20T23:07:33.053Z; no row exists for this script, so no per-invocation number is reported. It still costs 3 spawns on its event and is therefore already counted inside that chain's total.",
-      "p50ms": null,
-      "p95ms": null,
-      "totalMs": null,
-      "invocations": 0,
-      "rootCause": "no measurement exists for context-hardcap-gate.sh:8 on this tree — its cost stays unset (evidenced by absence) rather than estimated; the chain it belongs to carries the spawn floor",
-      "fixTask": null,
-      "status": "deferred"
-    },
-    {
-      "id": "hook-verification-guard-sh-8-unmeasured",
-      "area": "single hook — verification-guard.sh:8 (registered in settings.json, no telemetry rows)",
-      "evidence": "unavailable",
-      "evidenceDetail": ".ukit/storage/cache/hook-latency — 62854 rows across 120 session files, span 2026-09-17T10:50:18.670Z → 2026-09-20T23:07:33.053Z; no row exists for this script, so no per-invocation number is reported. It still costs 3 spawns on its event and is therefore already counted inside that chain's total.",
-      "p50ms": null,
-      "p95ms": null,
-      "totalMs": null,
-      "invocations": 0,
-      "rootCause": "no measurement exists for verification-guard.sh:8 on this tree — its cost stays unset (evidenced by absence) rather than estimated; the chain it belongs to carries the spawn floor",
-      "fixTask": null,
-      "status": "deferred"
-    },
-    {
-      "id": "hook-record-execution-mjs-8-unmeasured",
-      "area": "single hook — record-execution.mjs:8 (registered in settings.json, no telemetry rows)",
-      "evidence": "unavailable",
-      "evidenceDetail": ".ukit/storage/cache/hook-latency — 62854 rows across 120 session files, span 2026-09-17T10:50:18.670Z → 2026-09-20T23:07:33.053Z; no row exists for this script, so no per-invocation number is reported. It still costs 3 spawns on its event and is therefore already counted inside that chain's total.",
-      "p50ms": null,
-      "p95ms": null,
-      "totalMs": null,
-      "invocations": 0,
-      "rootCause": "no measurement exists for record-execution.mjs:8 on this tree — its cost stays unset (evidenced by absence) rather than estimated; the chain it belongs to carries the spawn floor",
-      "fixTask": null,
-      "status": "deferred"
-    },
-    {
-      "id": "hook-task-watchdog-sh-8-unmeasured",
-      "area": "single hook — task-watchdog.sh:8 (registered in settings.json, no telemetry rows)",
-      "evidence": "unavailable",
-      "evidenceDetail": ".ukit/storage/cache/hook-latency — 62854 rows across 120 session files, span 2026-09-17T10:50:18.670Z → 2026-09-20T23:07:33.053Z; no row exists for this script, so no per-invocation number is reported. It still costs 3 spawns on its event and is therefore already counted inside that chain's total.",
-      "p50ms": null,
-      "p95ms": null,
-      "totalMs": null,
-      "invocations": 0,
-      "rootCause": "no measurement exists for task-watchdog.sh:8 on this tree — its cost stays unset (evidenced by absence) rather than estimated; the chain it belongs to carries the spawn floor",
-      "fixTask": null,
-      "status": "deferred"
-    },
-    {
-      "id": "hook-context-window-guard-sh-10-unmeasured",
-      "area": "single hook — context-window-guard.sh:10 (registered in settings.json, no telemetry rows)",
-      "evidence": "unavailable",
-      "evidenceDetail": ".ukit/storage/cache/hook-latency — 62854 rows across 120 session files, span 2026-09-17T10:50:18.670Z → 2026-09-20T23:07:33.053Z; no row exists for this script, so no per-invocation number is reported. It still costs 3 spawns on its event and is therefore already counted inside that chain's total.",
-      "p50ms": null,
-      "p95ms": null,
-      "totalMs": null,
-      "invocations": 0,
-      "rootCause": "no measurement exists for context-window-guard.sh:10 on this tree — its cost stays unset (evidenced by absence) rather than estimated; the chain it belongs to carries the spawn floor",
-      "fixTask": null,
-      "status": "deferred"
-    },
-    {
-      "id": "hook-completion-gate-sh-8-unmeasured",
-      "area": "single hook — completion-gate.sh:8 (registered in settings.json, no telemetry rows)",
-      "evidence": "unavailable",
-      "evidenceDetail": ".ukit/storage/cache/hook-latency — 62854 rows across 120 session files, span 2026-09-17T10:50:18.670Z → 2026-09-20T23:07:33.053Z; no row exists for this script, so no per-invocation number is reported. It still costs 3 spawns on its event and is therefore already counted inside that chain's total.",
-      "p50ms": null,
-      "p95ms": null,
-      "totalMs": null,
-      "invocations": 0,
-      "rootCause": "no measurement exists for completion-gate.sh:8 on this tree — its cost stays unset (evidenced by absence) rather than estimated; the chain it belongs to carries the spawn floor",
-      "fixTask": null,
-      "status": "deferred"
-    },
-    {
-      "id": "hook-reinject-context-sh-8-unmeasured",
-      "area": "single hook — reinject-context.sh:8 (registered in settings.json, no telemetry rows)",
-      "evidence": "unavailable",
-      "evidenceDetail": ".ukit/storage/cache/hook-latency — 62854 rows across 120 session files, span 2026-09-17T10:50:18.670Z → 2026-09-20T23:07:33.053Z; no row exists for this script, so no per-invocation number is reported. It still costs 3 spawns on its event and is therefore already counted inside that chain's total.",
-      "p50ms": null,
-      "p95ms": null,
-      "totalMs": null,
-      "invocations": 0,
-      "rootCause": "no measurement exists for reinject-context.sh:8 on this tree — its cost stays unset (evidenced by absence) rather than estimated; the chain it belongs to carries the spawn floor",
-      "fixTask": null,
-      "status": "deferred"
-    },
-    {
-      "id": "hook-handoff-resume-sh-8-unmeasured",
-      "area": "single hook — handoff-resume.sh:8 (registered in settings.json, no telemetry rows)",
-      "evidence": "unavailable",
-      "evidenceDetail": ".ukit/storage/cache/hook-latency — 62854 rows across 120 session files, span 2026-09-17T10:50:18.670Z → 2026-09-20T23:07:33.053Z; no row exists for this script, so no per-invocation number is reported. It still costs 3 spawns on its event and is therefore already counted inside that chain's total.",
-      "p50ms": null,
-      "p95ms": null,
-      "totalMs": null,
-      "invocations": 0,
-      "rootCause": "no measurement exists for handoff-resume.sh:8 on this tree — its cost stays unset (evidenced by absence) rather than estimated; the chain it belongs to carries the spawn floor",
-      "fixTask": null,
-      "status": "deferred"
-    },
-    {
-      "id": "hook-session-episode-sh-8-unmeasured",
-      "area": "single hook — session-episode.sh:8 (registered in settings.json, no telemetry rows)",
-      "evidence": "unavailable",
-      "evidenceDetail": ".ukit/storage/cache/hook-latency — 62854 rows across 120 session files, span 2026-09-17T10:50:18.670Z → 2026-09-20T23:07:33.053Z; no row exists for this script, so no per-invocation number is reported. It still costs 3 spawns on its event and is therefore already counted inside that chain's total.",
-      "p50ms": null,
-      "p95ms": null,
-      "totalMs": null,
-      "invocations": 0,
-      "rootCause": "no measurement exists for session-episode.sh:8 on this tree — its cost stays unset (evidenced by absence) rather than estimated; the chain it belongs to carries the spawn floor",
-      "fixTask": null,
-      "status": "deferred"
-    }
-  ]
-}