@lh8ppl/claude-memory-kit 0.1.2 → 0.2.1

package/src/lessons-promote.mjs

@@ -0,0 +1,137 @@
+// lessons-promote.mjs — `cmk lessons promote <id>`: move a project-tier fact
+// into the user tier (LESSONS.md by default) through the SAFE promote path.
+//
+// This is the EXPLICIT half of the wedge (D-27/D-30): a project observation the
+// user wants to carry across ALL their projects. Before this, the subcommand
+// was a stub and the memory-write skill hand-edited LESSONS.md — bypassing
+// home-path sanitization, Poison_Guard, dedup, and the audit trail.
+//
+// It routes through promoteCandidatesToUserTier (D-13) at confidence:'high'
+// (an explicit user action is the highest-trust signal there is, so it promotes
+// rather than queuing). NEVER hand-edit ~/.claude-memory-kit/*.md.
+//
+// Composes on: forget.resolveFact (read a project fact by id) +
+// auto-persona.promoteCandidatesToUserTier (safe user-tier write).
+
+import { resolveFact } from './forget.mjs';
+import { promoteCandidatesToUserTier } from './auto-persona.mjs';
+import { findBulletScratchpad } from './bullet-lookup.mjs';
+import { errorResult, notFoundResult } from './result-shapes.mjs';
+
+const VALID_TARGETS = new Set(['USER.md', 'HABITS.md', 'LESSONS.md']);
+
+// Sensible default landing section per target. Each name passes
+// auto-persona's SAFE_SECTION_NAME guard; ensureSectionExists creates it if the
+// user's scaffold doesn't already have it.
+const DEFAULT_SECTION = Object.freeze({
+  'LESSONS.md': 'Cross-Project Lessons',
+  'HABITS.md': 'Working Style',
+  'USER.md': 'Profile',
+});
+
+/**
+ * Promote a project-tier fact to the user tier through the safe path.
+ *
+ * @param {object} opts
+ * @param {string} opts.id          citation id of the project fact (e.g. P-XXXXXXXX)
+ * @param {string} opts.projectRoot project root (for resolving the source fact)
+ * @param {string} opts.userDir     user-tier dir (~/.claude-memory-kit)
+ * @param {string} [opts.to]        target user-tier file (default LESSONS.md)
+ * @param {string} [opts.section]   landing section (default per-target)
+ * @param {string} [opts.now]       ISO timestamp override (tests)
+ * @returns {{action:string, id?:string, target?:string, section?:string, ...}}
+ */
+export function lessonsPromote({ id, projectRoot, userDir, to = 'LESSONS.md', section, now } = {}) {
+  if (!userDir) {
+    return errorResult({ category: 'schema', errors: ['userDir is required (lessons promote writes to the user tier)'] });
+  }
+  if (!VALID_TARGETS.has(to)) {
+    return errorResult({ category: 'schema', errors: [`invalid target '${to}' (expected USER.md | HABITS.md | LESSONS.md)`] });
+  }
+  // `lessons promote` carries a PROJECT observation to the user tier. Reject a
+  // U-tier id (already user-tier — nothing to promote) and an L-tier id (local
+  // is gitignored/machine-specific on purpose — promoting it to the
+  // machine-global user tier would surface deliberately-unshared content in
+  // every project's persona). Source must be the committed project tier.
+  if (typeof id === 'string' && (id[0] === 'U' || id[0] === 'L')) {
+    return errorResult({
+      category: 'schema',
+      errors: [`lessons promote moves a PROJECT-tier (P-) fact; got a ${id[0]}-tier id '${id}'`],
+      id,
+    });
+  }
+
+  const found = resolveFact({ id, projectRoot, userDir });
+  if (found.state === 'not-found') {
+    // The id might be a scratchpad BULLET (the common `cmk search` mix-up):
+    // search surfaces bullet ids too, but promote carries FACTS. Say so.
+    const bulletIn = findBulletScratchpad(id, { projectRoot, userDir });
+    if (bulletIn) {
+      return notFoundResult({
+        errors: [
+          `'${id}' is a scratchpad bullet in ${bulletIn}, not a graduated fact — \`cmk lessons promote\` carries facts (in context/memory/) to the user tier. In \`cmk search\` output, pick an id whose location is a context/memory/*.md file, not a ${bulletIn}:NN bullet.`,
+        ],
+        id,
+      });
+    }
+    return notFoundResult({ errors: [`no fact with id '${id}'`], id });
+  }
+  if (found.state === 'tombstoned') {
+    return notFoundResult({ errors: [`fact '${id}' is tombstoned (forgotten); cannot promote`], id });
+  }
+
+  // A scratchpad bullet is single-line (the provenance HTML-comment must sit on
+  // the very next line). A RICH fact body is multi-line — `headline\n\n**Why:**
+  // …\n\n**How to apply:** …` — which writeBullet rejects outright (newlines
+  // break the 2-line bullet+comment shape). Flatten all whitespace to single
+  // spaces so the rule + its rationale promote as one well-formed bullet (the
+  // primary wedge case: an explicitly-captured rich architecture rule). The
+  // scratchpad byte cap still applies downstream via memoryWrite.
+  const text = (found.body ?? '').replace(/\s+/g, ' ').trim();
+  if (!text) {
+    return errorResult({ category: 'schema', errors: [`fact '${id}' has no body to promote`], id });
+  }
+
+  const candidate = {
+    target: to,
+    section: section || DEFAULT_SECTION[to],
+    text,
+    confidence: 'high', // explicit user action → clears the confidence gate (promotes, not queued)
+  };
+
+  // trust:'high' + source:'user-explicit' — a user-attested promotion is durable
+  // (never aged out / auto-superseded by the maintenance passes — the 45.4
+  // invariant). The auto path leaves these at the default medium.
+  const res = promoteCandidatesToUserTier({
+    candidates: [candidate],
+    userDir,
+    now,
+    trust: 'high',
+    source: 'user-explicit',
+  });
+
+  const promotedHit = res.promoted.find((p) => p.target === to);
+  if (promotedHit) {
+    return { action: 'promoted', id, target: to, section: candidate.section, newId: promotedHit.id ?? null };
+  }
+  // A supersede is ALSO success: the promotion replaced an existing same-topic
+  // lesson with this updated one (common when the user re-promotes a refined rule).
+  const supersededHit = res.superseded.find((s) => s.target === to);
+  if (supersededHit) {
+    return { action: 'promoted', id, target: to, section: candidate.section, newId: supersededHit.newId, superseded: supersededHit.oldId };
+  }
+  // Routed to the conflict queue (e.g. it clashes with a hand-curated entry the
+  // kit won't silently overwrite) or otherwise didn't land — surface honestly.
+  const conflictHit = res.conflicts.find((q) => q.target === to);
+  if (conflictHit) {
+    return { action: 'queued', id, target: to, section: candidate.section, reason: 'conflict' };
+  }
+  const queuedHit = res.queued.find((q) => q.target === to);
+  return {
+    action: 'queued',
+    id,
+    target: to,
+    section: candidate.section,
+    reason: queuedHit?.reason ?? 'not-promoted',
+  };
+}

package/src/memory-write.mjs

@@ -54,7 +54,7 @@ import {
 import { nowIso, appendAuditEntry, REASON_CODES } from './audit-log.mjs';
 import { ERROR_CATEGORIES, errorResult } from './result-shapes.mjs';
 import { appendScratchpadBullet } from './scratchpad.mjs';
-import { parseBulletProvenance } from './provenance.mjs';
+import { parseBulletProvenance, isProvenanceCommentLine } from './provenance.mjs';
 import { checkPoisonGuard, logPoisonGuardRejection } from './poison-guard.mjs';
 import { detectConflicts, writeConflictEntry } from './conflict-queue.mjs';
 import { sanitizeHomePaths } from './sanitize.mjs';
@@ -190,7 +190,7 @@ function findMatchingBullet({ lines, substring, sectionTitle }) {
     const [, tier, idShort, bulletText] = m;
     if (!bulletText.includes(substring)) continue;
     const commentLine = lines[i + 1];
-    if (!commentLine || !/^\s*<!--.*-->\s*$/.test(commentLine)) continue;
+    if (!isProvenanceCommentLine(commentLine)) continue;
     return {
       bulletIdx: i,
       commentIdx: i + 1,

package/src/native-memory.mjs

@@ -0,0 +1,98 @@
+// Native Anthropic Auto Memory coexistence (Task 60, ADR-0011).
+//
+// Claude Code ships its own Auto Memory (v2.1.59+, ON by default), writing
+// machine-local `~/.claude/projects/<slug>/memory/` in the same shape the kit
+// uses in-repo. With the kit installed BOTH inject at session start → context
+// bloat. Per ADR-0011 the kit is ADDITIVE, not enforcing: the default is
+// coexist (we never touch the user's setting); `cmk disable-native-memory`
+// is a one-command, committable opt-in that writes `autoMemoryEnabled: false`
+// into the project's `.claude/settings.json` (which travels with `git clone`,
+// unlike the user-only `autoMemoryDirectory`). `cmk enable-native-memory`
+// reverses it (explicit `true`).
+//
+// Public boundary:
+//   setNativeAutoMemory({ projectRoot, enabled })
+//     → { action: 'written' | 'unchanged', settingsPath, enabled }
+//     → errorResult({ category: SCHEMA }) when the existing file is unparseable
+//       (NEVER clobber a hand-broken file — surface it).
+//   getNativeAutoMemoryState({ projectRoot })
+//     → { state: 'enabled' | 'disabled' | 'default' | 'unknown', settingsPath }
+//   (`default` = key absent ⇒ Anthropic's default, which is ON.)
+
+import { existsSync, readFileSync, writeFileSync, mkdirSync } from 'node:fs';
+import { join, dirname } from 'node:path';
+import { errorResult, ERROR_CATEGORIES } from './result-shapes.mjs';
+
+const SETTINGS_REL = ['.claude', 'settings.json'];
+
+export function nativeMemorySettingsPath(projectRoot) {
+  return join(projectRoot, ...SETTINGS_REL);
+}
+
+function readSettings(settingsPath) {
+  if (!existsSync(settingsPath)) return { settings: {}, existed: false };
+  const raw = readFileSync(settingsPath, 'utf8');
+  return { settings: JSON.parse(raw), existed: true };
+}
+
+/**
+ * Read the project's `.claude/settings.json` and report the native-memory
+ * state. `default` means the user has not set `autoMemoryEnabled` at all, so
+ * Anthropic's default (enabled) applies.
+ */
+export function getNativeAutoMemoryState({ projectRoot }) {
+  const settingsPath = nativeMemorySettingsPath(projectRoot);
+  if (!existsSync(settingsPath)) return { state: 'default', settingsPath };
+  let settings;
+  try {
+    ({ settings } = readSettings(settingsPath));
+  } catch (err) {
+    return { state: 'unknown', settingsPath, error: err?.message ?? String(err) };
+  }
+  const v = settings?.autoMemoryEnabled;
+  if (v === false) return { state: 'disabled', settingsPath };
+  if (v === true) return { state: 'enabled', settingsPath };
+  return { state: 'default', settingsPath };
+}
+
+/**
+ * The one-line `cmk install` heads-up about native-vs-kit coexistence
+ * (ADR-0011). Returns the note string when the heads-up is relevant (the user
+ * has NOT already opted out), or `null` when they've disabled native memory
+ * (no point nagging). Pure + trivially testable; runInstall just prints it.
+ */
+export function nativeMemoryInstallNote(projectRoot) {
+  if (getNativeAutoMemoryState({ projectRoot }).state === 'disabled') return null;
+  return "  Note: Claude Code's native Auto Memory keeps running alongside the kit (both fill over time). For one lean memory layer, run `cmk disable-native-memory`.";
+}
+
+/**
+ * Write `autoMemoryEnabled: <enabled>` into the project's committable
+ * `.claude/settings.json`. Idempotent (a no-op write reports `unchanged` and
+ * leaves the file byte-identical). Preserves every sibling key. On a parse
+ * error of an existing file, returns an error WITHOUT overwriting.
+ */
+export function setNativeAutoMemory({ projectRoot, enabled }) {
+  const settingsPath = nativeMemorySettingsPath(projectRoot);
+
+  let settings = {};
+  if (existsSync(settingsPath)) {
+    try {
+      ({ settings } = readSettings(settingsPath));
+    } catch (err) {
+      return errorResult({
+        category: ERROR_CATEGORIES.SCHEMA,
+        errors: [`${settingsPath} parse error: ${err?.message ?? err}`],
+      });
+    }
+  }
+
+  if (settings.autoMemoryEnabled === enabled) {
+    return { action: 'unchanged', settingsPath, enabled };
+  }
+
+  settings.autoMemoryEnabled = enabled;
+  mkdirSync(dirname(settingsPath), { recursive: true });
+  writeFileSync(settingsPath, JSON.stringify(settings, null, 2) + '\n', 'utf8');
+  return { action: 'written', settingsPath, enabled };
+}

package/src/persona-portability.mjs

@@ -0,0 +1,253 @@
+// persona-portability.mjs — Task 72. `cmk persona export` / `cmk persona import`.
+//
+// The persona (the user tier — USER/HABITS/LESSONS + fragments/) follows the
+// HUMAN, not the repo (design §1.1, D-27): it lives machine-local at
+// ~/.claude-memory-kit and is deliberately OUT of any project repo, because
+// committing it would leak your working-style to teammates who clone. So
+// portability across YOUR machines is per-human, not per-repo: export the user
+// tier to one OS-agnostic bundle file, carry it (USB / private repo / Dropbox),
+// import it on the other machine.
+//
+// This is the EXPLICIT primitive (decided in Task 72): no merge, no collision
+// control. Import OVERWRITES, backing up anything it would replace so nothing is
+// lost. The seamless auto-merge path (`cmk persona sync <git-url>`, Task 72.2)
+// is deferred — git handles transport + conflicts there.
+//
+// Bundle format: a single self-describing JSON file (no tar/zip dependency, and
+// human-inspectable). `{ kind, version, exportedAt, fileCount, files: { relpath:
+// content } }`.
+
+import {
+  existsSync,
+  readFileSync,
+  writeFileSync,
+  mkdirSync,
+  readdirSync,
+  statSync,
+  renameSync,
+  unlinkSync,
+} from 'node:fs';
+import { join, dirname } from 'node:path';
+import { reindex } from './reindex.mjs';
+import { appendAuditEntry, nowIso, REASON_CODES } from './audit-log.mjs';
+
+const BUNDLE_KIND = 'cmk-persona-bundle';
+const BUNDLE_VERSION = 1;
+
+// The persona surface to bundle: the 3 user-tier scratchpads + a settings
+// override, plus the fact-store / queue subdirs (walked recursively). Everything
+// else under the user tier is machine-local + regenerable and is NEVER bundled —
+// runtime locks/audit (.locks/), the FTS cache (.index/), and prior import
+// backups (.import-backups/). Using an explicit allow-list (rather than
+// "everything minus excludes") guarantees a new runtime dir can't leak in later.
+const TOP_LEVEL_FILES = ['USER.md', 'HABITS.md', 'LESSONS.md', 'settings.json'];
+const SUBDIRS = ['fragments', 'queues'];
+
+function walkFiles(absDir, relPrefix, out) {
+  for (const name of readdirSync(absDir)) {
+    const abs = join(absDir, name);
+    const rel = relPrefix ? `${relPrefix}/${name}` : name;
+    if (statSync(abs).isDirectory()) walkFiles(abs, rel, out);
+    else out.push({ rel, abs });
+  }
+}
+
+/**
+ * Export the user tier to a portable bundle file.
+ *
+ * @param {object} opts
+ * @param {string} opts.userDir - the user-tier root to export.
+ * @param {string} opts.outFile - where to write the bundle.
+ * @param {string} [opts.now] - ISO timestamp override (tests).
+ * @returns {{action:'exported'|'error', path?, fileCount?, bytes?, errorCategory?, errors?}}
+ */
+export function exportPersona({ userDir, outFile, now } = {}) {
+  if (!userDir || !existsSync(userDir)) {
+    return {
+      action: 'error',
+      errorCategory: 'not-found',
+      errors: [`user tier not found at ${userDir} — run \`cmk init-user-tier\` first`],
+    };
+  }
+  if (!outFile) {
+    return { action: 'error', errorCategory: 'schema', errors: ['no output file given'] };
+  }
+
+  const files = {};
+  for (const f of TOP_LEVEL_FILES) {
+    const abs = join(userDir, f);
+    if (existsSync(abs) && statSync(abs).isFile()) {
+      files[f] = readFileSync(abs, 'utf8');
+    }
+  }
+  for (const sub of SUBDIRS) {
+    const absSub = join(userDir, sub);
+    if (existsSync(absSub) && statSync(absSub).isDirectory()) {
+      const collected = [];
+      walkFiles(absSub, sub, collected);
+      for (const { rel, abs } of collected) files[rel] = readFileSync(abs, 'utf8');
+    }
+  }
+
+  const bundle = {
+    kind: BUNDLE_KIND,
+    version: BUNDLE_VERSION,
+    exportedAt: now ?? nowIso(),
+    fileCount: Object.keys(files).length,
+    files,
+  };
+  const json = JSON.stringify(bundle, null, 2);
+  mkdirSync(dirname(outFile), { recursive: true });
+  writeFileSync(outFile, json, 'utf8');
+
+  return {
+    action: 'exported',
+    path: outFile,
+    fileCount: bundle.fileCount,
+    bytes: Buffer.byteLength(json, 'utf8'),
+  };
+}
+
+// Read + validate a bundle file. Returns { bundle } on success, or { error: <the
+// error result> } on any problem. Kept separate so importPersona stays simple.
+function readAndValidateBundle(inFile) {
+  const err = (msg, cat = 'schema') => ({ error: { action: 'error', errorCategory: cat, errors: [msg] } });
+  if (!inFile || !existsSync(inFile)) return err(`bundle not found at ${inFile}`, 'not-found');
+  let bundle;
+  try {
+    bundle = JSON.parse(readFileSync(inFile, 'utf8'));
+  } catch (e) {
+    return err(`bundle is not valid JSON: ${e.message}`);
+  }
+  if (bundle?.kind !== BUNDLE_KIND) return err(`not a cmk persona bundle (kind: ${bundle?.kind ?? 'missing'})`);
+  if (bundle.version !== BUNDLE_VERSION) {
+    return err(`unsupported bundle version ${bundle.version} (this cmk supports v${BUNDLE_VERSION})`);
+  }
+  if (!bundle.files || typeof bundle.files !== 'object') return err('bundle carries no files');
+  return { bundle };
+}
+
+// Undo a partial import: remove the files we created, restore the ones we moved
+// aside. Best-effort per item — a leaked backup is recoverable; a clobbered live
+// file is not, so we always try to put the originals back.
+function rollbackImport(created, renamed) {
+  for (const dest of created) {
+    try {
+      if (existsSync(dest)) unlinkSync(dest);
+    } catch {
+      /* best-effort */
+    }
+  }
+  for (const { dest, bkp } of renamed) {
+    try {
+      if (existsSync(bkp)) {
+        mkdirSync(dirname(dest), { recursive: true });
+        renameSync(bkp, dest);
+      }
+    } catch {
+      /* best-effort — the backup copy still exists for manual recovery */
+    }
+  }
+}
+
+// Apply the bundle's files TRANSACTIONALLY (the Task-91 rollback discipline):
+// back up every existing target first, then write all files, and if ANY write
+// fails partway, roll the whole thing back so a mid-import disk/permission error
+// never leaves the persona half-applied. Returns the count of backed-up files;
+// throws on unrecoverable failure (after rolling back).
+function applyBundleAtomic(userDir, files, backupRoot) {
+  const renamed = []; // {dest, bkp} — existing files moved aside
+  const created = []; // dest — files that did NOT exist before (new this import)
+  try {
+    for (const rel of Object.keys(files)) {
+      const dest = join(userDir, ...rel.split('/'));
+      if (existsSync(dest)) {
+        const bkp = join(backupRoot, ...rel.split('/'));
+        mkdirSync(dirname(bkp), { recursive: true });
+        renameSync(dest, bkp);
+        renamed.push({ dest, bkp });
+      } else {
+        created.push(dest);
+      }
+    }
+    for (const [rel, content] of Object.entries(files)) {
+      const dest = join(userDir, ...rel.split('/'));
+      mkdirSync(dirname(dest), { recursive: true });
+      writeFileSync(dest, content, 'utf8');
+    }
+  } catch (err) {
+    rollbackImport(created, renamed);
+    throw err;
+  }
+  return renamed.length;
+}
+
+// Best-effort user-tier reindex — `cmk search` works immediately after import;
+// `cmk reindex` can rebuild later if this throws.
+function tryReindexUserTier(userDir) {
+  try {
+    reindex({ tier: 'U', userDir, warn: () => {} });
+    return true;
+  } catch {
+    return false;
+  }
+}
+
+// Door 4: one operational audit entry (the user tier was bulk-rewritten). The
+// individual facts keep their own provenance inside the bundled fact files; this
+// records the import event + where overwritten files were backed up. Best-effort.
+function writeImportAudit(userDir, { ts, fileCount, backedUp, backupRoot, inFile }) {
+  try {
+    appendAuditEntry(userDir, {
+      ts,
+      action: 'persona-imported',
+      tier: 'U',
+      id: 'persona-bundle',
+      reasonCode: REASON_CODES.PERSONA_IMPORTED,
+      paths: backedUp > 0 ? { archive: backupRoot } : undefined,
+      extra: { fileCount, backedUp, source: inFile },
+    });
+  } catch {
+    /* never fail the import because the audit write failed */
+  }
+}
+
+/**
+ * Import a persona bundle onto this machine's user tier. OVERWRITES, backing up
+ * any file it would replace to <userDir>/.import-backups/<ts>/ first (no data
+ * loss; transactional — rolls back on a mid-import failure). Rebuilds the
+ * user-tier search index from the imported fragments.
+ *
+ * @param {object} opts
+ * @param {string} opts.userDir - the target user-tier root.
+ * @param {string} opts.inFile - the bundle to import.
+ * @param {string} [opts.now] - ISO timestamp override (tests).
+ * @returns {{action:'imported'|'error', fileCount?, backedUp?, backupPath?, reindexed?, errorCategory?, errors?}}
+ */
+export function importPersona({ userDir, inFile, now } = {}) {
+  const { bundle, error } = readAndValidateBundle(inFile);
+  if (error) return error;
+
+  const ts = now ?? nowIso();
+  mkdirSync(userDir, { recursive: true });
+  const backupRoot = join(userDir, '.import-backups', ts.replace(/[:.]/g, '-'));
+
+  let backedUp;
+  try {
+    backedUp = applyBundleAtomic(userDir, bundle.files, backupRoot);
+  } catch (err) {
+    return { action: 'error', errorCategory: 'io', errors: [`import failed and was rolled back: ${err?.message ?? err}`] };
+  }
+
+  const fileCount = Object.keys(bundle.files).length;
+  const reindexed = tryReindexUserTier(userDir);
+  writeImportAudit(userDir, { ts, fileCount, backedUp, backupRoot, inFile });
+
+  return {
+    action: 'imported',
+    fileCount,
+    backedUp,
+    backupPath: backedUp > 0 ? backupRoot : null,
+    reindexed,
+  };
+}

package/src/provenance.mjs

@@ -75,8 +75,27 @@ const BULLET_RE = new RegExp(
   `^- \\((${ID_PATTERN.source.replace(/^\^/, '').replace(/\$$/, '')})\\)\\s+(.+)$`,
 );
 
-// Match a provenance comment, tolerant of leading indentation.
-const COMMENT_RE = /^\s*<!--.*-->\s*$/;
+// Is `line` a single-line HTML comment (the shape the kit writes provenance
+// in: `  <!-- source: …, trust: … -->`), tolerant of leading indentation?
+// String-scanning, NOT a regex, on purpose: a `/<!--.*-->/` regex trips
+// CodeQL js/bad-tag-filter (`.` skips newlines; ignores the `--!>` end-tag
+// variant). Our provenance comments are always single-line, so a literal
+// prefix/suffix check is equivalent AND clears the alert (the PR #72
+// pattern). Shared so scratchpad / memory-write / inject-context don't each
+// re-roll the flagged regex.
+export function isProvenanceCommentLine(line) {
+  if (typeof line !== 'string') return false;
+  const t = line.trim();
+  return t.length >= 7 && t.startsWith('<!--') && t.endsWith('-->');
+}
+
+// Strip the `<!--` (4 chars) / `-->` (3 chars) delimiters from a line already
+// confirmed by isProvenanceCommentLine. Slicing, not a regex, for the same
+// js/bad-tag-filter reason.
+function stripCommentDelimiters(line) {
+  const t = line.trim();
+  return t.slice(4, t.length - 3);
+}
 
 function validateBulletInput({ id, text, provenance }) {
   const errors = [];
@@ -183,10 +202,9 @@ export function writeBullet(opts = {}) {
 }
 
 export function parseBulletProvenance(line) {
-  if (typeof line !== 'string') return null;
-  if (!COMMENT_RE.test(line)) return null;
+  if (!isProvenanceCommentLine(line)) return null;
 
-  const inner = line.replace(/^\s*<!--/, '').replace(/-->\s*$/, '');
+  const inner = stripCommentDelimiters(line);
   const fields = {};
   for (const part of inner.split(',')) {
     const idx = part.indexOf(':');

package/src/read-hook-stdin.mjs

@@ -0,0 +1,47 @@
+// Hook-stdin drain (Task: cmk-compress-session manual-invocation hang fix).
+//
+// The SessionEnd hook bins (npm: packages/cli/bin/; plugin: plugin/bin/) drain
+// stdin so Claude Code's hook pipe closes cleanly. The PAYLOAD is discarded —
+// the bins read their state from disk (sessions/now.md, the fact corpus), not
+// from the hook JSON — so the read exists ONLY to drain the pipe.
+//
+// The hazard this module fixes: `readFileSync(0, 'utf8')` BLOCKS until stdin
+// reaches EOF. When the bin is run as a real hook, Claude Code pipes the JSON
+// payload and closes the pipe → EOF arrives → the read returns instantly. But
+// when the bin is run MANUALLY without redirecting stdin (e.g. the v0.2.0
+// cut-gate B7 probe: `cmk-compress-session | Out-Null` pipes stdout but leaves
+// stdin on the interactive console), the console never sends EOF, so the read
+// blocks forever — before ANY of the bin's body runs. The 60s SessionEnd hook
+// ceiling then kills it (exit 124, zero stderr), which looked for days like a
+// graduation/Haiku/lock hang but was the wrapper never executing. See
+// DECISION-LOG 2026-06-06 FIX/RESOLVED.
+//
+// Boundary: readHookStdin({ isTTY }) → string. When stdin is an interactive TTY
+// (no piped payload to drain, and reading would block), return '' without
+// touching the fd. Otherwise drain the fd as before. isTTY is INJECTED by the
+// caller (`process.stdin.isTTY`) so the function is pure and unit-testable
+// without a real terminal.
+
+import { readFileSync } from 'node:fs';
+
+/**
+ * Drain the hook payload from stdin without blocking an interactive console.
+ *
+ * @param {object} [opts]
+ * @param {boolean} [opts.isTTY] - the caller's `process.stdin.isTTY`. When
+ *   truthy, stdin is an interactive terminal: there is no piped payload and a
+ *   blocking read would hang, so we return '' and never touch the fd.
+ * @param {number} [opts.fd=0] - the stdin file descriptor (override for tests).
+ * @returns {string} the drained payload, or '' for a TTY / unconnected stdin.
+ */
+export function readHookStdin({ isTTY, fd = 0 } = {}) {
+  // Interactive console → no payload to drain and readFileSync(fd) would block
+  // waiting for an EOF the terminal never sends. Treat as empty.
+  if (isTTY) return '';
+  try {
+    return readFileSync(fd, 'utf8');
+  } catch {
+    // stdin not connected (e.g. fd closed) — fine; the hook still proceeds.
+    return '';
+  }
+}

package/src/register-crons.mjs

@@ -121,13 +121,17 @@ function buildWindowsSchtasks({ command, entryName, hour, minute, dayOfWeek }) {
   // already exists (idempotency primitive). /RL LIMITED (not HIGHEST)
   // because daily distill doesn't need admin.
   const time = `${String(hour).padStart(2, '0')}:${String(minute).padStart(2, '0')}`;
-  // Escape inner double-quotes per schtasks /TR convention: `\"`.
-  // Task 33 B2 fix — pre-fix `/TR "${command}"` produced malformed
-  // nested quotes for any command containing `"` (typical when the
-  // command points at a Windows path with spaces). Skill-review caught
-  // this; the test fixture (cli-register-crons.test.js) didn't surface
-  // it because earlier tests only passed `'node bin.mjs'` (no quotes).
-  const escapedCommand = command.replace(/"/g, '\\"');
+  // The command is wrapped in `/TR "${command}"`. cmd.exe/schtasks treats
+  // backslash as a PATH separator, not an escape character, so the only
+  // char that can break out of the quotes is a literal `"` — and that is
+  // rejected at the registerCron boundary (see the validation below), the
+  // same way an embedded `'` is rejected for the Linux cron line. So no
+  // escaping is applied here. (The earlier `command.replace(/"/g, '\\"')`
+  // was both unnecessary — controlled bin-name+path commands never contain
+  // `"` — and CodeQL-flagged js/incomplete-sanitization, since it didn't
+  // escape backslashes; but `\"`/`\\` is not how cmd.exe quotes anyway, and
+  // doubling backslashes would corrupt Windows paths. Reject-at-boundary is
+  // the correct, safe contract.)
   // Task 34: /SC WEEKLY /D <SUN|MON|...> for weekly cadence; /SC DAILY otherwise.
   let scheduleFlags;
   if (dayOfWeek !== undefined && dayOfWeek !== null) {
@@ -139,7 +143,7 @@ function buildWindowsSchtasks({ command, entryName, hour, minute, dayOfWeek }) {
   } else {
     scheduleFlags = '/SC DAILY';
   }
-  return `schtasks /Create /TN "${entryName}" ${scheduleFlags} /ST ${time} /TR "${escapedCommand}" /RL LIMITED /F`;
+  return `schtasks /Create /TN "${entryName}" ${scheduleFlags} /ST ${time} /TR "${command}" /RL LIMITED /F`;
 }
 
 /**
@@ -165,6 +169,11 @@ export function registerCron(opts = {}) {
     // cron command needs to either escape POSIX-style ('\'') or
     // we extend this helper with a sanitizer (v0.1.x candidate).
     errors.push("command: must not contain single quotes (Linux cron-line shell-quoting contract)");
+  } else if (opts.command.includes('"')) {
+    // Windows schtasks /TR wraps the command in double-quotes; an embedded
+    // `"` would break out of the quoting. Reject at the boundary (cmd.exe
+    // has no usable in-quote escape for `"`), mirroring the `'` contract.
+    errors.push('command: must not contain double quotes (Windows schtasks /TR quoting contract)');
   }
   const entryName = opts.entryName ?? CRON_ENTRY_NAME;
   if (!entryName || typeof entryName !== 'string' || !/^[a-zA-Z0-9_.-]+$/.test(entryName)) {