selftune 0.2.13 → 0.2.15

package/cli/selftune/evolution/evolve.ts

@@ -36,10 +36,12 @@ import type {
   SessionTelemetryRecord,
   SkillUsageRecord,
 } from "../types.js";
+import { CLIError, handleCLIError } from "../utils/cli-error.js";
 import { parseFrontmatter, replaceDescription } from "../utils/frontmatter.js";
 import { createEvolveTUI } from "../utils/tui.js";
 import { appendAuditEntry } from "./audit.js";
 import { checkConstitution } from "./constitutional.js";
+import { scoreDescription } from "./description-quality.js";
 import { appendEvidenceEntry } from "./evidence.js";
 import { extractFailurePatterns } from "./extract-patterns.js";
 import {
@@ -94,6 +96,8 @@ export interface EvolveResult {
   baselineResult?: BaselineMeasurement;
   gateValidation?: ValidationResult;
   sync_result?: SyncResult;
+  descriptionQualityBefore?: number;
+  descriptionQualityAfter?: number;
 }
 
 /**
@@ -247,16 +251,26 @@ export async function evolve(
     );
 
   /** Stamp every return with pipeline stats so callers always get them. */
-  const withStats = (r: Omit<EvolveResult, "llmCallCount" | "elapsedMs">): EvolveResult => ({
-    ...r,
-    llmCallCount,
-    elapsedMs: Date.now() - pipelineStart,
-    ...(syncResult ? { sync_result: syncResult } : {}),
-  });
+  const withStats = (r: Omit<EvolveResult, "llmCallCount" | "elapsedMs">): EvolveResult => {
+    const descQualityAfterScore = r.proposal
+      ? scoreDescription(r.proposal.proposed_description, options.skillName).composite
+      : undefined;
+    return {
+      ...r,
+      llmCallCount,
+      elapsedMs: Date.now() - pipelineStart,
+      ...(syncResult ? { sync_result: syncResult } : {}),
+      ...(descQualityBeforeScore != null
+        ? { descriptionQualityBefore: descQualityBeforeScore }
+        : {}),
+      ...(descQualityAfterScore != null ? { descriptionQualityAfter: descQualityAfterScore } : {}),
+    };
+  };
 
-  // Hoisted so catch block can preserve partial results on error
+  // Hoisted so catch block and withStats can preserve partial results on error
   let lastProposal: EvolutionProposal | null = null;
   let lastValidation: ValidationResult | null = null;
+  let descQualityBeforeScore: number | undefined;
 
   try {
     // -----------------------------------------------------------------------
@@ -281,7 +295,11 @@ export async function evolve(
     const versionTag = skillVersion ? `, v${skillVersion}` : "";
     const createdAuditDetails = (message: string) =>
       `original_description:${rawContent}\n${message}`;
-    tui.done(`Loaded SKILL.md (desc: ${currentDescription.length} chars${versionTag})`);
+    const descQualityBefore = scoreDescription(currentDescription, skillName);
+    descQualityBeforeScore = descQualityBefore.composite;
+    tui.done(
+      `Loaded SKILL.md (desc: ${currentDescription.length} chars${versionTag}, quality: ${descQualityBefore.composite})`,
+    );
 
     if (options.syncFirst) {
       tui.step(`Syncing source-truth telemetry${options.syncForce ? " (force)" : ""}...`);
@@ -1111,38 +1129,36 @@ Options:
   }
 
   if (!values.skill || !values["skill-path"]) {
-    console.error("[ERROR] --skill and --skill-path are required");
-    process.exit(1);
+    throw new CLIError(
+      "--skill and --skill-path are required",
+      "MISSING_FLAG",
+      "selftune evolve --skill <name> --skill-path <path>",
+    );
   }
   if ((values["sync-force"] ?? false) && !(values["sync-first"] ?? false)) {
-    console.error("[ERROR] --sync-force requires --sync-first");
-    process.exit(1);
+    throw new CLIError(
+      "--sync-force requires --sync-first",
+      "INVALID_FLAG",
+      "Add --sync-first when using --sync-force",
+    );
   }
 
   const { detectAgent } = await import("../utils/llm-call.js");
   const requestedAgent = values.agent;
   if (requestedAgent && !Bun.which(requestedAgent)) {
-    console.error(
-      JSON.stringify({
-        level: "error",
-        code: "agent_not_in_path",
-        message: `Agent CLI '${requestedAgent}' not found in PATH.`,
-        action: "Install it or omit --agent to use auto-detection.",
-      }),
+    throw new CLIError(
+      `Agent CLI '${requestedAgent}' not found in PATH.`,
+      "AGENT_NOT_FOUND",
+      "Install it or omit --agent to use auto-detection.",
     );
-    process.exit(1);
   }
   const agent = requestedAgent ?? detectAgent();
   if (!agent) {
-    console.error(
-      JSON.stringify({
-        level: "error",
-        code: "agent_not_found",
-        message: "No agent CLI (claude/codex/opencode) found in PATH.",
-        action: "Install Claude Code, Codex, or OpenCode.",
-      }),
+    throw new CLIError(
+      "No agent CLI (claude/codex/opencode) found in PATH.",
+      "AGENT_NOT_FOUND",
+      "Install Claude Code, Codex, or OpenCode.",
     );
-    process.exit(1);
   }
 
   // -------------------------------------------------------------------------
@@ -1150,20 +1166,27 @@ Options:
   // -------------------------------------------------------------------------
   const skillPath = values["skill-path"];
   if (!skillPath) {
-    console.error("[ERROR] --skill-path is required.");
-    process.exit(1);
+    throw new CLIError(
+      "--skill-path is required.",
+      "MISSING_FLAG",
+      "selftune evolve --skill <name> --skill-path <path>",
+    );
   }
   if (!existsSync(skillPath)) {
-    console.error(`[ERROR] SKILL.md not found at: ${skillPath}`);
-    console.error("  Verify the --skill-path argument points to an existing SKILL.md file.");
-    process.exit(1);
+    throw new CLIError(
+      `SKILL.md not found at: ${skillPath}`,
+      "FILE_NOT_FOUND",
+      "Verify the --skill-path argument points to an existing SKILL.md file.",
+    );
   }
 
   const evalSetPath = values["eval-set"];
   if (evalSetPath && !existsSync(evalSetPath)) {
-    console.error(`[ERROR] Eval set file not found at: ${evalSetPath}`);
-    console.error("  Verify the --eval-set argument points to an existing JSON file.");
-    process.exit(1);
+    throw new CLIError(
+      `Eval set file not found at: ${evalSetPath}`,
+      "FILE_NOT_FOUND",
+      "Verify the --eval-set argument points to an existing JSON file.",
+    );
   }
 
   // If no eval-set provided, check that log files exist for auto-generation
@@ -1172,12 +1195,11 @@ Options:
     const hasSkillLog = querySkillUsageRecords(dbCheck).length > 0;
     const hasQueryLog = existsSync(QUERY_LOG);
     if (!hasSkillLog && !hasQueryLog) {
-      console.error("[ERROR] No eval set provided and no telemetry logs found.");
-      console.error(
-        "  Either pass --eval-set <path> or generate logs first by using selftune-enabled skills.",
+      throw new CLIError(
+        `No eval set provided and no telemetry logs found. Expected logs at: ${SKILL_LOG} and ${QUERY_LOG}`,
+        "MISSING_DATA",
+        "Either pass --eval-set <path> or generate logs first by using selftune-enabled skills.",
       );
-      console.error(`  Expected logs at: ${SKILL_LOG} and ${QUERY_LOG}`);
-      process.exit(1);
     }
   }
 
@@ -1244,6 +1266,12 @@ Options:
       rationale: result.proposal?.rationale ?? "",
       ...(result.skillVersion ? { version: result.skillVersion } : {}),
       dashboard_url: `http://localhost:3141/report/${encodeURIComponent(values.skill)}`,
+      ...(result.descriptionQualityBefore != null
+        ? { description_quality_before: result.descriptionQualityBefore }
+        : {}),
+      ...(result.descriptionQualityAfter != null
+        ? { description_quality_after: result.descriptionQualityAfter }
+        : {}),
     };
     console.log(JSON.stringify(summary, null, 2));
   }
@@ -1276,20 +1304,5 @@ Options:
 }
 
 if (import.meta.main) {
-  cliMain().catch((err) => {
-    const message = err instanceof Error ? err.message : String(err);
-    const stack = err instanceof Error ? err.stack : undefined;
-    console.error(`[FATAL] ${message}`);
-    if (stack && process.env.SELFTUNE_VERBOSE === "1") {
-      console.error(stack);
-    }
-    console.error(
-      "\nTroubleshooting:\n" +
-        "  - Verify --skill-path points to a valid SKILL.md file\n" +
-        "  - Ensure eval data exists (run `selftune eval generate` first) or pass --eval-set\n" +
-        "  - Check that ANTHROPIC_API_KEY is set if using Claude\n" +
-        "  - Re-run with --verbose for full diagnostic output",
-    );
-    process.exit(1);
-  });
+  cliMain().catch(handleCLIError);
 }

package/cli/selftune/evolution/rollback.ts

@@ -13,6 +13,7 @@ import { parseArgs } from "node:util";
 
 import { updateContextAfterRollback } from "../memory/writer.js";
 import type { EvolutionAuditEntry } from "../types.js";
+import { CLIError, handleCLIError } from "../utils/cli-error.js";
 import { replaceDescription } from "../utils/frontmatter.js";
 import { appendAuditEntry, getLastDeployedProposal, readAuditTrail } from "./audit.js";
 
@@ -233,8 +234,11 @@ Options:
   }
 
   if (!values.skill || !values["skill-path"]) {
-    console.error("[ERROR] --skill and --skill-path are required");
-    process.exit(1);
+    throw new CLIError(
+      "--skill and --skill-path are required",
+      "MISSING_FLAG",
+      "selftune evolve rollback --skill <name> --skill-path <path>",
+    );
   }
 
   const result = await rollback({
@@ -248,8 +252,5 @@ Options:
 }
 
 if (import.meta.main) {
-  cliMain().catch((err) => {
-    console.error(`[FATAL] ${err}`);
-    process.exit(1);
-  });
+  cliMain().catch(handleCLIError);
 }

package/cli/selftune/grading/auto-grade.ts

@@ -17,7 +17,7 @@ import { AGENT_CANDIDATES, TELEMETRY_LOG } from "../constants.js";
 import { getDb } from "../localdb/db.js";
 import { querySessionTelemetry, querySkillUsageRecords } from "../localdb/queries.js";
 import type { GradingResult, SessionTelemetryRecord, SkillUsageRecord } from "../types.js";
-import { readJsonl } from "../utils/jsonl.js";
+import { CLIError, handleCLIError } from "../utils/cli-error.js";
 import { detectAgent as _detectAgent } from "../utils/llm-call.js";
 import { readExcerpt } from "../utils/transcript.js";
 import {
@@ -63,8 +63,7 @@ Options:
 
   const skill = values.skill;
   if (!skill) {
-    console.error("[ERROR] --skill is required");
-    process.exit(1);
+    throw new CLIError("--skill is required", "MISSING_FLAG", "selftune auto-grade --skill <name>");
   }
 
   // --- Determine agent ---
@@ -72,10 +71,11 @@ Options:
   const validAgents = [...AGENT_CANDIDATES];
   if (values.agent) {
     if (!validAgents.includes(values.agent)) {
-      console.error(
-        `[ERROR] Invalid --agent '${values.agent}'. Expected one of: ${validAgents.join(", ")}`,
+      throw new CLIError(
+        `Invalid --agent '${values.agent}'. Expected one of: ${validAgents.join(", ")}`,
+        "INVALID_FLAG",
+        `selftune auto-grade --skill <name> --agent ${validAgents[0]}`,
       );
-      process.exit(1);
     }
     agent = values.agent;
   } else {
@@ -83,28 +83,19 @@ Options:
   }
 
   if (!agent) {
-    console.error(
-      `[ERROR] No supported agent CLI (${AGENT_CANDIDATES.join("/")}) found in PATH.\n` +
-        "Install one of the supported agent CLIs.",
+    throw new CLIError(
+      `No supported agent CLI (${AGENT_CANDIDATES.join("/")}) found in PATH`,
+      "AGENT_NOT_FOUND",
+      "Install one of the supported agent CLIs",
     );
-    process.exit(1);
   }
 
   console.error(`[INFO] Auto-grade via agent: ${agent}`);
 
   // --- Auto-find session ---
-  const telemetryLog = values["telemetry-log"] ?? TELEMETRY_LOG;
-  let telRecords: SessionTelemetryRecord[];
-  let skillUsageRecords: SkillUsageRecord[];
-  if (telemetryLog === TELEMETRY_LOG) {
-    const db = getDb();
-    telRecords = querySessionTelemetry(db) as SessionTelemetryRecord[];
-    skillUsageRecords = querySkillUsageRecords(db) as SkillUsageRecord[];
-  } else {
-    // Intentional JSONL fallback: custom --telemetry-log path overrides SQLite reads
-    telRecords = readJsonl<SessionTelemetryRecord>(telemetryLog);
-    skillUsageRecords = [];
-  }
+  const db = getDb();
+  const telRecords = querySessionTelemetry(db) as SessionTelemetryRecord[];
+  const skillUsageRecords = querySkillUsageRecords(db) as SkillUsageRecord[];
 
   let telemetry: SessionTelemetryRecord;
   let sessionId: string;
@@ -114,21 +105,22 @@ Options:
     sessionId = values["session-id"];
     const resolved = resolveSessionById(telRecords, sessionId);
     if (!resolved) {
-      console.error(
-        `[ERROR] Session '${sessionId}' not found in telemetry or recoverable transcript data. ` +
-          "Check the session ID or omit --session-id to auto-select the latest matching session.",
+      throw new CLIError(
+        `Session '${sessionId}' not found in telemetry or recoverable transcript data`,
+        "MISSING_DATA",
+        "Check the session ID or omit --session-id to auto-select the latest matching session",
       );
-      process.exit(1);
     }
     telemetry = resolved.telemetry;
     transcriptPath = resolved.transcriptPath;
   } else {
     const resolved = resolveLatestSessionForSkill(telRecords, skillUsageRecords, skill);
     if (!resolved) {
-      console.error(
-        `[ERROR] No session found for skill '${skill}'. Run the skill first, or pass --session-id.`,
+      throw new CLIError(
+        `No session found for skill '${skill}'`,
+        "MISSING_DATA",
+        "Run the skill first, or pass --session-id",
       );
-      process.exit(1);
     }
     telemetry = resolved.telemetry;
     sessionId = resolved.sessionId ?? "unknown";
@@ -169,8 +161,11 @@ Options:
       agent,
     });
   } catch (err) {
-    console.error(`[ERROR] ${err instanceof Error ? err.message : String(err)}`);
-    process.exit(1);
+    throw new CLIError(
+      `Grading failed: ${err instanceof Error ? err.message : String(err)}`,
+      "OPERATION_FAILED",
+      "Check agent availability and try again",
+    );
   }
 
   const outputPath = values.output ?? buildDefaultGradingOutputPath(sessionId);
@@ -203,8 +198,5 @@ Options:
 
 // Guard: only run when invoked directly
 if (import.meta.main) {
-  cliMain().catch((err) => {
-    console.error(`[FATAL] ${err}`);
-    process.exit(1);
-  });
+  cliMain().catch(handleCLIError);
 }

package/cli/selftune/grading/grade-session.ts

@@ -28,7 +28,7 @@ import type {
   SessionTelemetryRecord,
   SkillUsageRecord,
 } from "../types.js";
-import { readJsonl } from "../utils/jsonl.js";
+import { CLIError, handleCLIError } from "../utils/cli-error.js";
 import {
   detectAgent as _detectAgent,
   stripMarkdownFences as _stripMarkdownFences,
@@ -744,8 +744,7 @@ Options:
 
   const skill = values.skill;
   if (!skill) {
-    console.error("[ERROR] --skill is required");
-    process.exit(1);
+    throw new CLIError("--skill is required", "MISSING_FLAG", "selftune grade --skill <name>");
   }
 
   // --- Determine agent ---
@@ -753,10 +752,11 @@ Options:
   const validAgents = [...AGENT_CANDIDATES];
   if (values.agent) {
     if (!validAgents.includes(values.agent)) {
-      console.error(
-        `[ERROR] Invalid --agent '${values.agent}'. Expected one of: ${validAgents.join(", ")}`,
+      throw new CLIError(
+        `Invalid --agent '${values.agent}'. Expected one of: ${validAgents.join(", ")}`,
+        "INVALID_FLAG",
+        `selftune grade --skill <name> --agent ${validAgents[0]}`,
       );
-      process.exit(1);
     }
     agent = values.agent;
   } else {
@@ -764,11 +764,11 @@ Options:
   }
 
   if (!agent) {
-    console.error(
-      `[ERROR] No supported agent CLI (${AGENT_CANDIDATES.join("/")}) found in PATH.\n` +
-        "Install one of the supported agent CLIs.",
+    throw new CLIError(
+      `No supported agent CLI (${AGENT_CANDIDATES.join("/")}) found in PATH`,
+      "AGENT_NOT_FOUND",
+      "Install claude, codex, or opencode CLI, then retry",
     );
-    process.exit(1);
   }
 
   console.error(`[INFO] Grading via agent: ${agent}`);
@@ -778,8 +778,11 @@ Options:
   if (values["evals-json"] && values["eval-id"] != null) {
     const evalIdNum = Number(values["eval-id"]);
     if (!Number.isFinite(evalIdNum) || !Number.isInteger(evalIdNum)) {
-      console.error(`[ERROR] --eval-id must be a finite integer, got: ${values["eval-id"]}`);
-      process.exit(1);
+      throw new CLIError(
+        `--eval-id must be a finite integer, got: ${values["eval-id"]}`,
+        "INVALID_FLAG",
+        "selftune grade --eval-id <integer>",
+      );
     }
     expectations = loadExpectationsFromEvalsJson(values["evals-json"], evalIdNum);
   } else if (values.expectations?.length) {
@@ -804,18 +807,9 @@ Options:
   let transcriptPath = "";
   let sessionId = "unknown";
 
-  const telemetryLog = values["telemetry-log"] ?? TELEMETRY_LOG;
-  let telRecords: SessionTelemetryRecord[];
-  let skillUsageRecords: SkillUsageRecord[];
-  if (telemetryLog === TELEMETRY_LOG) {
-    const db = getDb();
-    telRecords = querySessionTelemetry(db) as SessionTelemetryRecord[];
-    skillUsageRecords = querySkillUsageRecords(db) as SkillUsageRecord[];
-  } else {
-    // Intentional JSONL fallback: custom --telemetry-log path overrides SQLite reads
-    telRecords = readJsonl<SessionTelemetryRecord>(telemetryLog);
-    skillUsageRecords = [];
-  }
+  const db = getDb();
+  const telRecords = querySessionTelemetry(db) as SessionTelemetryRecord[];
+  const skillUsageRecords = querySkillUsageRecords(db) as SkillUsageRecord[];
 
   if (values.transcript) {
     transcriptPath = values.transcript;
@@ -873,8 +867,11 @@ Options:
       agent,
     });
   } catch (err) {
-    console.error(`[ERROR] ${err instanceof Error ? err.message : String(err)}`);
-    process.exit(1);
+    throw new CLIError(
+      `Grading failed: ${err instanceof Error ? err.message : String(err)}`,
+      "OPERATION_FAILED",
+      "Check agent availability and try again",
+    );
   }
 
   const outputPath = values.output ?? buildDefaultGradingOutputPath(sessionId);
@@ -898,8 +895,5 @@ Options:
 
 // Guard: only run when invoked directly
 if (import.meta.main) {
-  cliMain().catch((err) => {
-    console.error(`[FATAL] ${err}`);
-    process.exit(1);
-  });
+  cliMain().catch(handleCLIError);
 }

package/cli/selftune/hooks/auto-activate.ts

@@ -179,9 +179,18 @@ if (import.meta.main) {
       const statePath = sessionStatePath(sessionId);
       const suggestions = evaluateRules(DEFAULT_RULES, ctx, statePath);
 
-      for (const s of suggestions) {
-        // Output to stderr — Claude Code shows stderr as system messages
-        process.stderr.write(`[selftune] 💡 Suggestion: ${s}\n`);
+      if (suggestions.length > 0) {
+        // Output as JSON with additionalContext — Claude Code adds this to
+        // Claude's context on UserPromptSubmit (more reliable than stderr)
+        const context = suggestions.map((s) => `[selftune] Suggestion: ${s}`).join("\n");
+        process.stdout.write(
+          JSON.stringify({
+            hookSpecificOutput: {
+              hookEventName: "UserPromptSubmit",
+              additionalContext: context,
+            },
+          }),
+        );
       }
     }
   } catch {

package/cli/selftune/hooks/evolution-guard.ts

@@ -19,7 +19,6 @@ import { basename, dirname, join } from "node:path";
 
 import { EVOLUTION_AUDIT_LOG, SELFTUNE_CONFIG_DIR } from "../constants.js";
 import type { PreToolUsePayload } from "../types.js";
-import { readJsonl } from "../utils/jsonl.js";
 
 // ---------------------------------------------------------------------------
 // Detection helpers (same pattern as skill-change-guard)
@@ -35,40 +34,31 @@ function extractSkillName(filePath: string): string {
 }
 
 // ---------------------------------------------------------------------------
-// Active monitoring check (SQLite-first — JSONL only for test/custom paths)
+// Active monitoring check (always SQLite)
 // ---------------------------------------------------------------------------
 
 /**
  * Check if a skill has an active deployed evolution (meaning it's under monitoring).
- * SQLite is the default read path; JSONL is used only for test/custom-path overrides.
  *
  * A skill is "actively monitored" if its last audit action is "deployed".
  * If the last action is "rolled_back", it's no longer monitored.
  */
 export async function checkActiveMonitoring(
   skillName: string,
-  auditLogPath: string,
+  _auditLogPath: string,
 ): Promise<boolean> {
-  // SQLite is the default path; JSONL fallback only for non-default paths (tests)
-  let entries: Array<{ skill_name?: string; action: string }>;
-  if (auditLogPath === EVOLUTION_AUDIT_LOG) {
-    const { getDb } = await import("../localdb/db.js");
-    const { queryEvolutionAudit } = await import("../localdb/queries.js");
-    const db = getDb();
-    entries = queryEvolutionAudit(db, skillName) as Array<{
-      skill_name?: string;
-      action: string;
-    }>;
-  } else {
-    // test/custom-path fallback
-    entries = readJsonl<{ skill_name?: string; action: string }>(auditLogPath);
-  }
-
-  // Filter entries for this skill by skill_name field
-  const skillEntries = entries.filter((e) => e.skill_name === skillName);
-  if (skillEntries.length === 0) return false;
-
-  const lastEntry = skillEntries[skillEntries.length - 1];
+  const { getDb } = await import("../localdb/db.js");
+  const { queryEvolutionAudit } = await import("../localdb/queries.js");
+  const db = getDb();
+  const entries = queryEvolutionAudit(db, skillName) as Array<{
+    skill_name?: string;
+    action: string;
+  }>;
+
+  if (entries.length === 0) return false;
+
+  // queryEvolutionAudit returns DESC order, so [0] is the most recent entry
+  const lastEntry = entries[0];
   return lastEntry.action === "deployed";
 }
 

package/cli/selftune/hooks/prompt-log.ts

@@ -21,7 +21,6 @@ import {
   reservePromptIdentity,
 } from "../normalization.js";
 import type { ImprovementSignalRecord, PromptSubmitPayload, QueryLogRecord } from "../types.js";
-import { appendJsonl } from "../utils/jsonl.js";
 
 // ---------------------------------------------------------------------------
 // Installed skill name cache
@@ -155,7 +154,13 @@ export async function processPrompt(
   promptStatePath?: string,
   _signalLogPath?: string,
 ): Promise<QueryLogRecord | null> {
-  const query = (payload.user_prompt ?? "").trim();
+  const rawPrompt =
+    typeof payload.prompt === "string"
+      ? payload.prompt
+      : typeof payload.user_prompt === "string"
+        ? payload.user_prompt
+        : "";
+  const query = rawPrompt.trim();
 
   if (!query) return null;
 
@@ -179,13 +184,6 @@ export async function processPrompt(
     /* hooks must never block */
   }
 
-  // JSONL backup (best-effort, hooks must never block)
-  try {
-    appendJsonl(logPath, record);
-  } catch {
-    /* hooks must never block */
-  }
-
   // Emit canonical prompt record (additive)
   const baseInput: CanonicalBaseInput = {
     platform: "claude_code",

package/cli/selftune/hooks/session-stop.ts

@@ -20,7 +20,6 @@ import {
   getLatestPromptIdentity,
 } from "../normalization.js";
 import type { SessionTelemetryRecord, StopPayload } from "../types.js";
-import { appendJsonl } from "../utils/jsonl.js";
 import { parseTranscript } from "../utils/transcript.js";
 
 const LOCK_STALE_MS = 30 * 60 * 1000;
@@ -120,13 +119,6 @@ export async function processSessionStop(
     /* hooks must never block */
   }
 
-  // JSONL backup (append-only, fail-open)
-  try {
-    appendJsonl(logPath, record);
-  } catch {
-    /* JSONL is a backup — never block on failure */
-  }
-
   // Emit canonical session + execution fact records (additive)
   const baseInput: CanonicalBaseInput = {
     platform: "claude_code",