create-claude-cabinet 0.46.0 → 0.48.0

package/lib/cli.js

@@ -3,10 +3,10 @@ const path = require('path');
 const fs = require('fs');
 const os = require('os');
 const crypto = require('crypto');
-const { copyTemplates, recordSkip } = require('./copy');
-const { mergeSettings, healUserSettings, mergeWatchtowerHooks, mergeMuxHooks, mergeBashCompressHooks } = require('./settings-merge');
-const { create: createMetadata, read: readMetadata } = require('./metadata');
-const { setupDb } = require('./db-setup');
+const { copyTemplates, recordSkip, isProjectOwnedSeed, classifyFileOwnership } = require('./copy');
+const { mergeSettings, healUserSettings, mergeWatchtowerHooks, mergeMuxHooks, mergeBashCompressHooks, mergeCcDevHooks, mergeSkillOverrides } = require('./settings-merge');
+const { create: createMetadata, read: readMetadata, readFrozenModules, setFrozenModules } = require('./metadata');
+const { setupDb, shouldSkipInit, readBundledSchemaVersion } = require('./db-setup');
 const { setupVerifyRuntime } = require('./verify-setup');
 const { setupSiteAuditRuntime } = require('./site-audit-setup');
 const { setupEngagement } = require('./engagement-setup');
@@ -442,255 +442,10 @@ function generateAgentWrappers(projectDir) {
   return wanted.size;
 }
 
-// MODULES is the manifest: every template path here is copied into a
-// consumer's project on install. A skill/hook/script that exists under
-// templates/ but is NOT listed in any module never ships — that is the
-// orphan bug that left the whole built-in-memory layer uninstalled
-// before v0.27.2 (test/manifest-integrity guards against recurrence).
-//
-// Intentional orphans (maintainer-only, must NOT ship to consumers) are
-// allowlisted in test/manifest-integrity. Currently: skills/cc-publish
-// (CC-source-repo release tooling, like scripts/migrate-all-consumers.js).
-const MODULES = {
-  'session-loop': {
-    name: 'Session Loop (orient + debrief)',
-    description: 'The briefing cycle. Claude starts each session informed, ends by preparing the next briefing.',
-    mandatory: true,
-    templates: [
-      'skills/orient',
-      'skills/orient-quick',
-      'skills/debrief',
-      'skills/debrief-quick',
-      'skills/session-handoff',
-      // Instruction phases — always ship, overriding the default skip-phases rule in copy.js
-      'skills/debrief/phases/audit-pattern-capture.md',
-      'skills/debrief/phases/methodology-capture.md',
-      'skills/debrief/phases/record-lessons.md',
-      'skills/debrief/phases/upstream-feedback.md',
-      'skills/debrief/phases/qa-handoff-sweep.md',
-      'skills/menu',
-    ],
-  },
-  'hooks': {
-    name: 'Git Guardrails + Telemetry',
-    description: 'Block destructive git ops (force push, hard reset). Track skill usage via JSONL telemetry.',
-    mandatory: false,
-    default: true,
-    lean: true,
-    templates: ['hooks/git-guardrails.sh', 'hooks/cc-upstream-guard.sh', 'hooks/skill-telemetry.sh', 'hooks/skill-tool-telemetry.sh', 'hooks/work-tracker-guard.sh', 'hooks/action-quality-gate.sh', 'hooks/action-completion-gate.sh', 'hooks/memory-index-guard.sh', 'scripts/cc-drift-check.cjs', 'scripts/skill-usage.mjs'],
-  },
-  'work-tracking': {
-    name: 'Work Tracking (pib-db or markdown)',
-    description: 'Track work items for orient/debrief. Default: SQLite (pib-db). Also supports markdown (tasks.md) or external systems. Choice made during /onboard.',
-    mandatory: false,
-    default: true,
-    lean: false,
-    templates: ['scripts/pib-db.mjs', 'scripts/pib-db-lib.mjs', 'scripts/pib-db-mcp-server.mjs', 'scripts/pib-db-schema.sql', 'scripts/work-tracker-server.mjs', 'scripts/work-tracker-ui.html', 'skills/work-tracker', 'skills/spring-clean'],
-    needsDb: true,
-  },
-  'planning': {
-    name: 'Planning + Execution (plan + execute)',
-    description: 'Structured planning with cabinet critique — members weigh in before you build.',
-    mandatory: false,
-    default: true,
-    lean: true,
-    templates: ['skills/plan', 'skills/execute', 'skills/execute/phases/post-impl-checklist.md', 'skills/debrief/phases/checklist-feedback.md', 'skills/checklist-discover', 'skills/generate-plan-groups', 'skills/execute-group', 'workflows/execute-group-implement.js', 'workflows/execute-group-complete.js', 'skills/investigate', 'cabinet/checkpoint-protocol.md', 'cabinet/elicitation-methods.md', 'cabinet/qa-dimensions-template.yaml', 'cabinet/checklist-stats-schema.md', 'scripts/qa-dimensions-validator.cjs', 'skills/orient/phases/checklist-status.md', 'skills/audit/phases/checklist-pruning.md'],
-  },
-  'compliance': {
-    name: 'Compliance Stack (rules + enforcement)',
-    description: 'Scoped instructions that load by path. Enforcement pipeline for promoting patterns to rules.',
-    mandatory: false,
-    default: true,
-    lean: false,
-    templates: ['rules/enforcement-pipeline.md', 'rules/maintainability.md', 'rules/markdown-prose.md', 'skills/unwrap', 'memory/patterns/_pattern-template.md', 'memory/patterns/pattern-intelligence-first.md'],
-  },
-  'memory': {
-    name: 'Built-In Memory (cc-remember + reader + validator)',
-    description: 'Curated write/validate layer over Claude Code\'s built-in file memory. /cc-remember writes indexed memories, /memory browses them, validate-memory.mjs guards MEMORY.md integrity. Replaced the retired omega engine in v0.27.',
-    mandatory: false,
-    default: true,
-    // lean:true is load-bearing — it keeps the skill triad, scripts, and
-    // rule installing together as an atomic unit. compliance is lean:false,
-    // so the memory-capture rule must NOT live there or lean installs get a
-    // partial, incoherent set.
-    lean: true,
-    templates: [
-      'skills/cc-remember',
-      'skills/memory',
-      'rules/memory-capture.md',
-      'scripts/write-memory-file.mjs',
-      'scripts/validate-memory.mjs',
-      // project-context.cjs ships as a co-located sibling: the two .mjs
-      // scripts require('./project-context.cjs') and consumers have no lib/.
-      'scripts/project-context.cjs',
-    ],
-  },
-  'audit': {
-    name: 'Audit Loop (audit + triage + cabinet)',
-    description: '27 expert cabinet members review your project. Convene the full cabinet or just one committee.',
-    mandatory: false,
-    default: true,
-    lean: true,
-    templates: [
-      'skills/audit', 'skills/pulse', 'skills/triage-audit', 'skills/cabinet',
-      'cabinet', 'briefing',
-      'skills/cabinet-accessibility', 'skills/cabinet-anti-confirmation',
-      'skills/cabinet-architecture', 'skills/cabinet-automation',
-      'skills/cabinet-boundary-man',
-      'skills/cabinet-anthropic-insider', 'skills/cabinet-cc-health',
-      'skills/cabinet-data-integrity',
-      'skills/cabinet-debugger', 'skills/cabinet-deployment',
-      'skills/cabinet-historian',
-      'skills/cabinet-organized-mind', 'skills/cabinet-process-therapist',
-      'skills/cabinet-qa', 'skills/cabinet-record-keeper',
-      'skills/cabinet-roster-check', 'skills/cabinet-security',
-      'skills/cabinet-seo',
-      'skills/cabinet-small-screen', 'skills/cabinet-speed-freak',
-      'skills/cabinet-system-advocate', 'skills/cabinet-technical-debt',
-      'skills/cabinet-usability', 'skills/cabinet-workflow-cop',
-      'skills/cabinet-framework-quality', 'skills/cabinet-goal-alignment',
-      'skills/cabinet-information-design', 'skills/cabinet-mantine-quality',
-      'skills/cabinet-ui-experimentalist', 'skills/cabinet-user-advocate',
-      'skills/cabinet-vision',
-      'skills/cabinet-narrative-architect', 'skills/cabinet-interactive-storyteller',
-      'skills/cabinet-elegance',
-      'scripts/merge-findings.js', 'scripts/load-triage-history.js',
-      'scripts/triage-server.mjs', 'scripts/triage-ui.html',
-      'scripts/finding-schema.json', 'scripts/resolve-committees.cjs',
-      'scripts/review-server.mjs', 'scripts/review-ui.html',
-      'workflows/deliberative-audit.js', 'cabinet/critique-contract.md',
-    ],
-  },
-  'lifecycle': {
-    name: 'Lifecycle (onboard + seed + cc-upgrade + link)',
-    description: 'Onboarding prepares the briefings. Seed proposes new cabinet members when you adopt new tech.',
-    mandatory: false,
-    default: true,
-    lean: true,
-    templates: ['skills/onboard', 'skills/seed', 'skills/cc-upgrade', 'skills/cc-link', 'skills/cc-unlink', 'skills/cc-extract', 'skills/cc-feedback', 'cabinet/elicitation-methods.md'],
-  },
-  'validate': {
-    name: 'Validate',
-    description: 'Structural validation checks with unified summary. Define your own validators.',
-    mandatory: false,
-    default: true,
-    lean: false,
-    templates: ['skills/validate', 'scripts/skill-validator.sh'],
-  },
-  'verify': {
-    name: 'Verification Harness (Cucumber + Playwright)',
-    description: 'Walkthrough verification with human-in-the-loop verdict pauses. Replaces flat AC lists with re-runnable user-journey scenarios. Includes /verify skeleton skill + cabinet-verify npm runtime + opt-in integration phases for /plan, /execute, /debrief.',
-    mandatory: false,
-    default: false,
-    lean: false,
-    templates: [
-      'skills/verify',
-      'skills/plan/phases/verify-plan.md',
-      'skills/execute/phases/verify-emit.md',
-      'skills/debrief/phases/verify-coverage.md',
-      'skills/orient/phases/verify-backfill.md',
-    ],
-    postInstall: 'verify-setup',
-  },
-  'site-audit': {
-    name: 'Site Audit (deployed-site quality)',
-    description: '14-check audit for deployed websites: performance, accessibility (3 engines), security, SEO, DNS, privacy, sustainability. Standalone HTML report + comparison mode.',
-    mandatory: false,
-    default: false,
-    lean: false,
-    templates: [
-      'skills/cc-site-audit',
-      'site-audit-runtime',
-    ],
-    postInstall: 'site-audit-setup',
-  },
-  engagement: {
-    name: 'Engagement management (with secure credential handoff)',
-    description: 'Ongoing client-engagement management built on pib-db: per-recipient packets (rendered projections of the work backlog), role-gated billing, client feedback flowing back as events, and secure credential handoff (encrypted capture via OS dialog, pluggable email/MCP/file transport). Three primary skills across consultant and client sides, plus redirect stubs. Requires work-tracking (the pib-db adapter is the engine\'s data source).',
-    mandatory: false,
-    default: false,
-    lean: false,
-    requires: ['work-tracking'],
-    postInstall: 'engagement-setup',
-    templates: [
-      'skills/collab-client',
-      'skills/collab-consultant',
-      'skills/engagement-help',
-      'skills/engagement-create',
-      'skills/setup-accounts',
-      'skills/guide',
-      'engagement',
-    ],
-  },
-  watchtower: {
-    name: 'Watchtower (continuous background state)',
-    description: 'Replaces orient/debrief with continuous background processing. Three rings (mechanical cron, Claude intelligence, session-aware), ambient state injection via SessionStart hook, and an inbox for extracted knowledge and signals. Sessions start informed with minimal context cost.',
-    mandatory: false,
-    default: false,
-    lean: false,
-    templates: [
-      'scripts/watchtower-validate.mjs',
-      'cabinet/watchtower-contracts.md',
-      'scripts/watchtower-lib.mjs',
-      'scripts/watchtower-queue.mjs',
-      'scripts/watchtower-routines.mjs',
-      'scripts/watchtower-advisories.mjs',
-      'cabinet/advisories-state-schema.md',
-      'skills/inbox',
-      'hooks/watchtower-session-start.sh',
-      'scripts/watchtower-build-context.mjs',
-      'scripts/watchtower-ring1.mjs',
-      'scripts/watchtower-ring1-runner.sh',
-      'skills/watchtower',
-      'watchtower/com.claude-cabinet.watchtower-ring1.plist',
-      'watchtower/watchtower-ring1.service',
-      'watchtower/watchtower-ring1.timer',
-      'scripts/watchtower-ring2.mjs',
-      'scripts/watchtower-ring2-runner.sh',
-      'watchtower/com.claude-cabinet.watchtower-ring2-fast.plist',
-      'watchtower/com.claude-cabinet.watchtower-ring2-slow.plist',
-      'watchtower/watchtower-ring2-fast.service',
-      'watchtower/watchtower-ring2-fast.timer',
-      'watchtower/watchtower-ring2-slow.service',
-      'watchtower/watchtower-ring2-slow.timer',
-      'hooks/watchtower-session-end.sh',
-      'scripts/watchtower-ring3-close.mjs',
-      'scripts/watchtower-status.sh',
-      'skills/briefing',
-      'skills/threads',
-      'skills/qa-handoff',
-      'skills/qa-drain',
-    ],
-  },
-  mux: {
-    name: 'Mux (tmux project manager)',
-    description: 'Multi-project terminal manager. Desks (tmux sessions), auto-worktrees, trail logging, sticky notes, portal color-switching, and an MCP server for Claude. Installs to user-level paths (~/.local/bin/, ~/.config/mux/).',
-    mandatory: false,
-    default: false,
-    lean: false,
-    postInstall: 'mux-setup',
-    templates: ['skills/orient/phases/dx-captures.md', 'skills/dx-feedback'],
-  },
-  'engagement-server': {
-    name: 'Engagement Server',
-    description: 'Central multi-engagement API server. Deploys once to Railway/Fly, serves all client engagements. User-level infrastructure.',
-    mandatory: false,
-    default: false,
-    lean: false,
-    requires: ['engagement', 'work-tracking'],
-    postInstall: 'engagement-server-setup',
-    templates: [],
-  },
-  'bash-compress': {
-    name: 'Bash Output Compression Hook',
-    description: 'PostToolUse hook that compresses noisy Bash stdout (git status walls, npm/yarn install output, find/ls dumps) to reclaim context in long sessions. Off by default. stderr and error/warning lines pass through verbatim; every rewrite carries a visible [compressed] marker; fail-open on any error. Requires the hooks module (it wires into .claude/settings.json).',
-    mandatory: false,
-    default: false,
-    lean: false,
-    requires: ['hooks'],
-    templates: ['hooks/bash-output-compress.sh'],
-  },
-};
+// The MODULES manifest (the module → template-path registry) lives in its
+// own file so this ~2000-line orchestrator stays navigable (architecture-0005).
+// Re-exported below for back-compat with importers of `require('../lib/cli').MODULES`.
+const { MODULES } = require('./modules');
 
 /** Recursively collect all relative file paths under a directory. */
 function walkDir(dir, base) {
@@ -708,6 +463,94 @@ function walkDir(dir, base) {
   return results;
 }
 
+// Map a module template path to its install-time manifest key (the path
+// relative to the project root). Top-level so both the install copy loop and
+// computeRemovals derive the key the same way — single source of truth.
+function manifestPath(tmpl) {
+  if (tmpl.startsWith('scripts/')) return tmpl;
+  return '.claude/' + tmpl;
+}
+
+/**
+ * Expand a declared frozen-module list by its `requires` closure (act:c1013967).
+ * Freezing a module must also pin the modules it depends on, or a frozen
+ * caller can be left bound to a refreshed callee contract (the cross-runtime
+ * version-skew failure: green in the dev tree, broken on a fresh install).
+ * E.g. freezing `engagement` (requires work-tracking) also pins work-tracking.
+ * Returns a Set of module keys. Unknown keys are ignored (validated elsewhere).
+ */
+function expandFrozenClosure(frozenKeys, modules = MODULES) {
+  const closure = new Set();
+  const queue = [...(frozenKeys || [])];
+  while (queue.length) {
+    const key = queue.shift();
+    if (closure.has(key)) continue;
+    closure.add(key);
+    const mod = modules[key];
+    for (const req of (mod && mod.requires) || []) {
+      if (!closure.has(req)) queue.push(req);
+    }
+  }
+  return closure;
+}
+
+/**
+ * Classify which manifest entries should be removed because they are no longer
+ * shipped upstream — the pure core of the cleanup/reconcile loop, extracted so
+ * it is unit-testable in isolation (the inline loop never was). Returns
+ * `{ toRemove, kept }`: `toRemove` is the list of project-relative paths whose
+ * files exist on disk and are safe to delete; `kept` carries `{path, reason}`
+ * for files deliberately spared, which the caller logs.
+ *
+ * The four safeguards from the original loop are preserved exactly: S1 only
+ * delete known-template paths; S2 only from selected modules; project-owned
+ * seeds and non-CC project skills are never deleted. A FROZEN file needs no
+ * special case here — it was RETAINED in `allManifest` (at its on-disk hash),
+ * so it never reaches the `!allManifest[oldPath]` test; a frozen file removed
+ * upstream is kept by S1 ("non-template").
+ */
+function computeRemovals({ existingManifest, allManifest, selectedModules, templateRoot, projectDir, modules = MODULES }) {
+  const collectPaths = (modKeys) => {
+    const set = new Set();
+    for (const modKey of modKeys) {
+      const mod = modules[modKey];
+      for (const tmpl of (mod && mod.templates) || []) {
+        const srcPath = path.join(templateRoot, tmpl);
+        if (fs.existsSync(srcPath) && fs.statSync(srcPath).isDirectory()) {
+          for (const rel of walkDir(srcPath)) set.add(manifestPath(tmpl) + '/' + rel);
+        } else if (fs.existsSync(srcPath)) {
+          set.add(manifestPath(tmpl));
+        }
+      }
+    }
+    return set;
+  };
+
+  const allTemplatePaths = collectPaths(Object.keys(modules));
+  const selectedTemplatePaths = collectPaths(selectedModules);
+
+  const ccSkillNames = new Set();
+  for (const mod of Object.values(modules)) {
+    for (const tmpl of mod.templates || []) {
+      if (tmpl.startsWith('skills/')) ccSkillNames.add(tmpl.split('/')[1]);
+    }
+  }
+
+  const toRemove = [];
+  const kept = [];
+  for (const oldPath of Object.keys(existingManifest)) {
+    if (allManifest[oldPath]) continue;
+    if (/\/phases\//.test(oldPath)) continue; // user-customizable
+    if (isProjectOwnedSeed(oldPath)) { kept.push({ path: oldPath, reason: 'project-owned seed' }); continue; }
+    if (!allTemplatePaths.has(oldPath)) { kept.push({ path: oldPath, reason: 'non-template file' }); continue; }
+    if (!selectedTemplatePaths.has(oldPath)) continue; // deselected module — leave it
+    const skillDirMatch = oldPath.match(/^\.claude\/skills\/([^/]+)\//);
+    if (skillDirMatch && !ccSkillNames.has(skillDirMatch[1])) { kept.push({ path: oldPath, reason: 'project skill' }); continue; }
+    if (fs.existsSync(path.join(projectDir, oldPath))) toRemove.push(oldPath);
+  }
+  return { toRemove, kept };
+}
+
 // Signals that a directory contains a real project (not just empty)
 const PROJECT_SIGNALS = [
   'package.json', 'Cargo.toml', 'requirements.txt', 'pyproject.toml',
@@ -730,6 +573,13 @@ function detectProjectState(dir) {
   return 'empty';
 }
 
+// A `--freeze`/`--unfreeze` value token vs the positional target dir. Module
+// keys are simple kebab-case identifiers; a flag or any path-like token
+// (separator, or . / ..) is the target directory, never a module list.
+function looksLikeModuleArg(s) {
+  return Boolean(s) && !s.startsWith('-') && !s.includes('/') && !s.includes('\\') && s !== '.' && s !== '..';
+}
+
 function parseArgs(argv) {
   const args = argv.slice(2);
   const flags = {
@@ -752,11 +602,32 @@ function parseArgs(argv) {
     else if (arg === '--help' || arg === '-h') flags.help = true;
     else if (arg === '--reset') flags.reset = true;
     else if (arg === '--force') flags.force = true;
+    else if (arg === '--reindex') flags.reindex = true;
     else if (arg === '--migrate-memory') flags.migrateMemory = true;
     else if (arg === '--unmigrate-memory') flags.unmigrateMemory = true;
+    else if (arg === '--list-frozen') flags.listFrozen = true;
     else if (arg === '--modules' && i + 1 < args.length) {
       flags.modules = args[++i].split(',').map(s => s.trim()).filter(Boolean);
     }
+    // --freeze <mods> / --unfreeze [mods]: edit .ccrc.json frozenModules and
+    // exit. Consume the next token as the module list ONLY when it looks like
+    // one — a flag, or a path-like token (contains a separator, or is . / ..),
+    // is the target directory, not a value. Module keys are simple kebab-case
+    // identifiers and never contain a path separator, so this can't swallow a
+    // real module while it does stop `--unfreeze /path/to/project` from being
+    // misread as a module (which would silently fail to clear-all). An empty
+    // --freeze is an error (dispatch warns, never a silent no-op); an empty
+    // --unfreeze clears all.
+    else if (arg === '--freeze') {
+      flags.freeze = looksLikeModuleArg(args[i + 1])
+        ? args[++i].split(',').map(s => s.trim()).filter(Boolean)
+        : [];
+    }
+    else if (arg === '--unfreeze') {
+      flags.unfreeze = looksLikeModuleArg(args[i + 1])
+        ? args[++i].split(',').map(s => s.trim()).filter(Boolean)
+        : []; // clear all
+    }
     else if (arg === '--frontier-model' && i + 1 < args.length) {
       // Empty/whitespace values are treated as absent: '' is a substring of
       // every model ID, which would match everything and leave the watchdog
@@ -782,6 +653,16 @@ function printHelp() {
     --modules <keys>  Comma-separated module keys to install (e.g., 'verify').
                   Mandatory modules are always included.
     --dry-run     Show what would be copied without writing
+    --reindex     Regenerate .claude/skills/_index.json in place (picks up
+                  directives-project.yaml mandate/routine edits without a full
+                  reinstall) and exit
+    --freeze <keys>    Pin module(s) (comma-separated) in .ccrc.json so the
+                  NEXT reinstall holds their CC-owned files at the current
+                  version (e.g. --freeze engagement while a client engagement
+                  is live). Also pins the requires-closure. Edits config, exits.
+    --unfreeze [keys]  Clear the freeze for the given module(s), or all if no
+                  keys. Re-run the installer to refresh the unfrozen module(s).
+    --list-frozen      Show which modules are frozen (+ their closure) and exit.
     --reset       Remove Claude Cabinet files (uses manifest for safety)
     --force       With --reset: remove even customized files;
                   with --migrate-memory: override already-migrated guard
@@ -823,6 +704,80 @@ async function run() {
     return;
   }
 
+  // Standalone skill-index regen (act:98d41fa2). Advisor standing-mandates and
+  // routine declarations are read from .claude/skills/_index.json, which is
+  // otherwise regenerated only by a full install — coupling every overlay
+  // change (e.g. adding a session-close mandate to directives-project.yaml) to
+  // an upgrade cycle. This regenerates the index in place so a change is
+  // verifiable immediately, no reinstall needed.
+  if (flags.reindex) {
+    const projectDir = path.resolve(flags.targetDir);
+    if (!fs.existsSync(path.join(projectDir, '.claude', 'skills'))) {
+      console.log(`  No .claude/skills/ in ${projectDir} — nothing to reindex.`);
+      return;
+    }
+    const n = generateSkillIndex(projectDir);
+    console.log(`  📇 Reindexed ${n} skill${n === 1 ? '' : 's'} in .claude/skills/_index.json`);
+    return;
+  }
+
+  // Standalone freeze management (act:c1013967) — edit .ccrc.json frozenModules
+  // and exit, no install. Freeze pins a module's CC-owned files across a
+  // reinstall (the "reinstall maginnis without disturbing Ed's engagement" fix);
+  // the install body reads frozenModules + expands the requires closure.
+  if (flags.listFrozen || flags.freeze || flags.unfreeze) {
+    const projectDir = path.resolve(flags.targetDir);
+    const current = readFrozenModules(projectDir);
+
+    const describe = (mods) => {
+      if (mods.length === 0) { console.log('  No modules are frozen.'); return; }
+      const closure = expandFrozenClosure(mods, MODULES);
+      const pulledIn = [...closure].filter(k => !mods.includes(k));
+      console.log(`  ❄ Frozen modules: ${mods.join(', ')}`);
+      if (pulledIn.length) console.log(`    + required closure (also pinned): ${pulledIn.join(', ')}`);
+      console.log('    A reinstall holds these modules’ CC-owned files at the current version.');
+    };
+
+    if (flags.listFrozen) {
+      describe(current);
+      return;
+    }
+
+    if (flags.freeze) {
+      if (flags.freeze.length === 0) {
+        console.log('  ✗ --freeze needs at least one module key, e.g. --freeze engagement');
+        return;
+      }
+      const known = Object.keys(MODULES);
+      const unknown = flags.freeze.filter(m => !known.includes(m));
+      if (unknown.length) console.log(`  ⚠ Unknown module key(s) ignored: ${unknown.join(', ')} (known: ${known.join(', ')})`);
+      const valid = flags.freeze.filter(m => known.includes(m));
+      const next = [...new Set([...current, ...valid])];
+      if (flags.dryRun) {
+        console.log(`  [dry run] Would set frozenModules to: ${next.join(', ') || '(none)'}`);
+      } else {
+        setFrozenModules(projectDir, next);
+        console.log('  ✓ Updated .ccrc.json frozenModules.');
+      }
+      describe(next);
+      return;
+    }
+
+    if (flags.unfreeze) {
+      const next = flags.unfreeze.length === 0
+        ? []                                              // --unfreeze → clear all
+        : current.filter(m => !flags.unfreeze.includes(m));
+      if (flags.dryRun) {
+        console.log(`  [dry run] Would set frozenModules to: ${next.join(', ') || '(none)'}`);
+      } else {
+        setFrozenModules(projectDir, next);
+        console.log('  ✓ Updated .ccrc.json frozenModules. Re-run the installer to refresh the unfrozen module(s).');
+      }
+      describe(next);
+      return;
+    }
+  }
+
   if (flags.migrateMemory) {
     const { migrateMemoryCmd } = require('./migrate-memory-cmd');
     const projectDir = path.resolve(flags.targetDir);
@@ -1151,18 +1106,24 @@ async function run() {
     return crypto.createHash('sha256').update(content).digest('hex').slice(0, 16);
   }
 
-  // Compute the relative path from projectDir for manifest entries
-  function manifestPath(tmpl) {
-    if (tmpl.startsWith('skills/') || tmpl.startsWith('hooks/') || tmpl.startsWith('rules/')) {
-      return '.claude/' + tmpl;
-    } else if (tmpl.startsWith('scripts/')) {
-      return tmpl;
-    }
-    return '.claude/' + tmpl;
-  }
+  // Frozen modules (act:c1013967): the operator pinned these in .ccrc.json so
+  // a reinstall holds their CC-owned files at the current version. Expand by
+  // the `requires` closure (freezing engagement also pins work-tracking, so a
+  // frozen caller is never left bound to a refreshed callee), then intersect
+  // with what is actually installed. `effectiveFrozen` gates ALL freeze
+  // behavior — never the raw declared list — so a typo'd / uninstalled / dead
+  // key can't silently pin global state or claim protection it isn't giving.
+  const declaredFrozen = readFrozenModules(projectDir);
+  const effectiveFrozen = new Set(
+    [...expandFrozenClosure(declaredFrozen, MODULES)].filter(
+      (k) => MODULES[k] && selectedModules.includes(k)
+    )
+  );
+  const frozenCounts = {}; // modKey -> # files held this install
 
   for (const modKey of selectedModules) {
     const mod = MODULES[modKey];
+    const frozen = effectiveFrozen.has(modKey);
     for (const tmpl of mod.templates) {
       const srcPath = path.join(templateRoot, tmpl);
       if (!fs.existsSync(srcPath)) {
@@ -1211,7 +1172,9 @@ async function run() {
           projectRoot: projectDir,
           existingManifest,
           instructionPhases: instrPhases,
+          frozen,
         });
+        if (results.frozen.length) frozenCounts[modKey] = (frozenCounts[modKey] || 0) + results.frozen.length;
         totalCopied += results.copied.length;
         totalSkipped += results.skipped.length;
         totalOverwritten += results.overwritten.length;
@@ -1229,53 +1192,88 @@ async function run() {
         const incoming = fs.readFileSync(srcPath, 'utf8');
         const incomingHash = hashContent(incoming);
         const mPath = manifestPath(tmpl);
+        const destExists = fs.existsSync(destPath);
+        const existingContent = destExists ? fs.readFileSync(destPath, 'utf8') : null;
+        // Instruction phases explicitly listed in MODULES are CC-owned and must
+        // always overwrite (they're not customizable); other phase files are
+        // preserved when the operator has edited them.
+        const isPhaseFile = tmpl.includes('/phases/') || tmpl.includes('\\phases\\');
+        const isInstructionPhase = isPhaseFile && mod.templates.includes(tmpl);
+
+        // Single decision spine — the ownership cascade lives in copy.js
+        // (classifyFileOwnership) so this branch can't drift from the directory
+        // walk. The leak this closes: the hand-inlined copy here was missing
+        // the project-owned-seed guard (elegance-0002, act:a6377c82).
+        const decision = classifyFileOwnership({
+          relPath: tmpl,
+          destExists,
+          existingContent,
+          incoming,
+          isPhaseFile,
+          isInstructionPhase,
+          frozen,
+          isCcOwned: Boolean(existingManifest[mPath]),
+          skipConflicts: flags.yes || dirState === 'existing-install',
+        });
 
-        if (fs.existsSync(destPath)) {
-          const existingContent = fs.readFileSync(destPath, 'utf8');
-          if (existingContent === incoming) {
+        switch (decision.action) {
+          case 'seed':
+            // Project-owned seed (act:2001bf54): seed if absent, NEVER
+            // overwrite, ALWAYS omit from the manifest.
+            if (!destExists) {
+              if (!flags.dryRun) fs.copyFileSync(srcPath, destPath);
+              totalCopied++;
+            } else {
+              totalSkipped++;
+            }
+            recordSkip(allManifest, mPath);
+            continue;
+          case 'preserve-phase':
+            console.log(`  Preserved customized phase: ${tmpl}`);
+            totalSkipped++;
+            recordSkip(allManifest, mPath);
+            continue;
+          case 'skip-identical':
             totalSkipped++;
             recordSkip(allManifest, mPath, { identical: true, incomingHash });
             continue;
-          }
-
-          // Phase file customization guard — mirrors copy.js:48-59 for the
-          // single-file install path. If the destination path is a phase
-          // file and the on-disk content differs from upstream (operator
-          // edited it), preserve it. This keeps the documented "customize
-          // by editing the phase file" affordance intact across upgrades.
-          // Exception: instruction phases explicitly listed in MODULES are
-          // CC-owned and must always overwrite (they're not customizable).
-          const isPhaseFile = tmpl.includes('/phases/') || tmpl.includes('\\phases\\');
-          const isInstructionPhase = isPhaseFile && mod.templates.includes(tmpl);
-          if (isPhaseFile && !isInstructionPhase && existingContent.trim() !== '' && existingContent.trim() !== incoming.trim()) {
-            console.log(`  Preserved customized phase: ${tmpl}`);
+          case 'freeze-hold':
             totalSkipped++;
-            // Customized phase = project-owned content → omit from manifest
-            // (recordSkip in copy.js — omission means "not ours").
+            frozenCounts[modKey] = (frozenCounts[modKey] || 0) + 1;
+            allManifest[mPath] = decision.manifestHash;
+            console.log(`  ❄ Frozen (held): ${path.relative(projectDir, destPath)}`);
+            continue;
+          case 'overwrite':
+            if (!flags.dryRun) fs.copyFileSync(srcPath, destPath);
+            totalOverwritten++;
+            // Log single-file overwrites too — the directory path (copy.js)
+            // already does. Without this, scripts/ updates are invisible in
+            // install output, masking whether a changed script propagated.
+            console.log(`  Updated: ${path.relative(projectDir, destPath)}`);
+            allManifest[mPath] = incomingHash;
+            continue;
+          case 'skip-omit':
+            totalSkipped++;
+            // Project-created file → omit from manifest entirely. Ownership
+            // classification is manifest-PRESENCE-based, so recording ANY hash
+            // here would mark the file upstream-owned and the NEXT install
+            // would silently overwrite it (act:bf21c95b).
             recordSkip(allManifest, mPath);
             continue;
-          }
-
-          if (flags.yes || dirState === 'existing-install') {
-            // If file is in the old manifest, it's upstream-managed — overwrite.
-            // If not, it's project-created — skip.
-            if (existingManifest[mPath]) {
-              if (!flags.dryRun) fs.copyFileSync(srcPath, destPath);
-              totalOverwritten++;
-              // Log single-file overwrites too — the directory path (copy.js)
-              // already does. Without this, scripts/ updates are invisible in
-              // install output, masking whether a changed script propagated.
-              console.log(`  Updated: ${path.relative(projectDir, destPath)}`);
-              allManifest[mPath] = incomingHash;
-            } else {
-              totalSkipped++;
-              // Project-created file → omit from manifest entirely. Ownership
-              // classification is manifest-PRESENCE-based, so recording ANY
-              // hash here would mark the file upstream-owned and the NEXT
-              // install would silently overwrite it (act:bf21c95b).
-              recordSkip(allManifest, mPath);
-            }
-          } else {
+          case 'freeze-hold-new':
+            // New file in a frozen module: hold it back (don't create) so the
+            // module stays internally consistent at its pinned version. Report
+            // only — never a manifest entry (no on-disk content to hash).
+            frozenCounts[modKey] = (frozenCounts[modKey] || 0) + 1;
+            console.log(`  ❄ Frozen (held new): ${path.relative(projectDir, destPath)}`);
+            continue;
+          case 'copy-new':
+            if (!flags.dryRun) fs.copyFileSync(srcPath, destPath);
+            totalCopied++;
+            allManifest[mPath] = incomingHash;
+            continue;
+          case 'prompt':
+          default: {
             const response = await prompts({
               type: 'select',
               name: 'action',
@@ -1296,11 +1294,8 @@ async function run() {
               // the manifest so it is never mistaken for upstream content.
               recordSkip(allManifest, mPath);
             }
+            continue;
           }
-        } else {
-          if (!flags.dryRun) fs.copyFileSync(srcPath, destPath);
-          totalCopied++;
-          allManifest[mPath] = incomingHash;
         }
       }
     }
@@ -1308,6 +1303,28 @@ async function run() {
 
   console.log(`  📁 Files: ${totalCopied} copied, ${totalOverwritten} overwritten, ${totalSkipped} unchanged`);
 
+  // --- Frozen-module report (act:c1013967) ---
+  // Plain-English, per-module: surface WHICH modules were held and how many
+  // files, so a gitignored freeze can never quietly change install behavior.
+  // Note in plain words when the requires-closure pinned a module the operator
+  // didn't name directly.
+  if (effectiveFrozen.size > 0) {
+    const declaredSet = new Set(declaredFrozen);
+    for (const modKey of effectiveFrozen) {
+      const n = frozenCounts[modKey] || 0;
+      let reason = '';
+      if (!declaredSet.has(modKey)) {
+        const namedBy = [...declaredSet].find(d => expandFrozenClosure([d], MODULES).has(modKey) && d !== modKey);
+        if (namedBy) reason = ` (pinned because ${namedBy} requires it)`;
+      }
+      if (n > 0) {
+        console.log(`  ❄ Frozen: ${n} file${n === 1 ? '' : 's'} in module '${modKey}'${reason} held at current version — unfreeze to update.`);
+      } else {
+        console.log(`  ❄ Frozen: module '${modKey}'${reason} — no CC-owned files differed (nothing to hold this run).`);
+      }
+    }
+  }
+
   // --- Make hook scripts executable ---
   const hooksDir = path.join(claudeDir, 'hooks');
   if (!flags.dryRun && fs.existsSync(hooksDir)) {
@@ -1339,6 +1356,27 @@ async function run() {
       mergeBashCompressHooks(settingsPath);
       console.log('  ⚙️  Registered bash-output compression PostToolUse hook');
     }
+
+    // Repo-local maintainer hooks — no-op unless this IS the CC source repo
+    // (dogfood). Registers the node --check PostToolUse gate durably so a
+    // reinstall can't silently regenerate it away (architecture-0004).
+    if (mergeCcDevHooks(projectDir)) {
+      console.log('  ⚙️  Registered repo-local node --check PostToolUse hook (CC source repo)');
+    }
+
+    // cabinet-* skills are ~half the skill-listing budget and are spawned by
+    // agentType (name), never auto-invoked by description — so set them
+    // "name-only" to free budget for the routing skills that need full trigger
+    // keywords (act:bee0fa16). Only-if-absent, so a user override is preserved.
+    if (selectedModules.includes('audit')) {
+      const cabinetSkills = (MODULES.audit.templates || [])
+        .map((t) => (t.match(/^skills\/(cabinet-[^/]+)$/) || [])[1])
+        .filter(Boolean);
+      const added = mergeSkillOverrides(projectDir, cabinetSkills);
+      if (added > 0) {
+        console.log(`  ⚙️  Set ${added} cabinet-* skill${added === 1 ? '' : 's'} to name-only (frees skill-listing budget)`);
+      }
+    }
   }
 
   // --- Heal user-level ~/.claude/settings.json ---
@@ -1378,10 +1416,34 @@ async function run() {
   }
 
   // --- Set up database ---
+  // dbSchemaVersion is recorded into .ccrc.json so a future reinstall can
+  // tell, WITHOUT loading better-sqlite3, that no migration is pending and
+  // skip the init step entirely (lib/db-setup.js shouldSkipInit).
+  let dbSchemaVersion;
   if (includeDb && selectedModules.includes('work-tracking') && !flags.dryRun) {
     try {
-      const dbResults = setupDb(projectDir);
-      for (const r of dbResults) console.log(`  🗄️  ${r}`);
+      // Freeze insurance (act:c1013967): when a module is frozen and a schema
+      // migration is actually pending, back up pib.db before db-setup runs so
+      // the operator has true one-file rollback for the store holding live
+      // engagement data. Reuses db-setup's own skip decision (no logic
+      // duplication): if work-tracking is itself frozen, its pib-db-lib.mjs
+      // stayed old → bundled == recorded → no migration → no backup. The
+      // existsSync guard means a fresh install (no pib.db yet) still creates
+      // the db normally; there's just nothing to back up.
+      if (effectiveFrozen.size > 0) {
+        const dbPath = path.join(projectDir, 'pib.db');
+        const recordedVersion = (readMetadata(projectDir) || {}).dbSchemaVersion;
+        const bundledVersion = readBundledSchemaVersion(path.join(projectDir, 'scripts'));
+        const willMigrate = !shouldSkipInit({ dbExists: fs.existsSync(dbPath), recordedVersion, bundledVersion });
+        if (willMigrate && fs.existsSync(dbPath)) {
+          const backupPath = `${dbPath}.cc-bak-${new Date().toISOString().replace(/[:.]/g, '-')}`;
+          fs.copyFileSync(dbPath, backupPath);
+          console.log(`  ❄ Backed up pib.db → ${path.basename(backupPath)} before migration (freeze active).`);
+        }
+      }
+      const dbResult = setupDb(projectDir);
+      for (const r of dbResult.results) console.log(`  🗄️  ${r}`);
+      if (dbResult.schemaVersion != null) dbSchemaVersion = dbResult.schemaVersion;
     } catch (err) {
       console.log(`  ⚠ Database setup failed: ${err.message}`);
       console.log('    You can set it up later: node scripts/pib-db.mjs init');
@@ -1487,78 +1549,26 @@ async function run() {
   //   S3: Itemize — list all deletions and confirm before proceeding
   //   S4: Backup — copy files to .cc-backup/<timestamp>/ before deletion
   if (Object.keys(existingManifest).length > 0) {
-    // S1: Build complete template path set (ALL modules) for classification
-    const allTemplatePaths = new Set();
-    for (const mod of Object.values(MODULES)) {
-      for (const tmpl of mod.templates || []) {
-        const srcPath = path.join(templateRoot, tmpl);
-        if (fs.existsSync(srcPath) && fs.statSync(srcPath).isDirectory()) {
-          for (const rel of walkDir(srcPath)) {
-            allTemplatePaths.add(manifestPath(tmpl) + '/' + rel);
-          }
-        } else if (fs.existsSync(srcPath)) {
-          allTemplatePaths.add(manifestPath(tmpl));
-        }
-      }
-    }
-
-    // S2: Build selected-module template paths for scoping
-    const selectedTemplatePaths = new Set();
-    for (const modKey of selectedModules) {
-      const mod = MODULES[modKey];
-      for (const tmpl of mod.templates || []) {
-        const srcPath = path.join(templateRoot, tmpl);
-        if (fs.existsSync(srcPath) && fs.statSync(srcPath).isDirectory()) {
-          for (const rel of walkDir(srcPath)) {
-            selectedTemplatePaths.add(manifestPath(tmpl) + '/' + rel);
-          }
-        } else if (fs.existsSync(srcPath)) {
-          selectedTemplatePaths.add(manifestPath(tmpl));
-        }
-      }
-    }
-
-    // Build CC template skill name set for project-skill guard (act:4ac281ba)
-    const ccSkillNames = new Set();
-    for (const mod of Object.values(MODULES)) {
-      for (const tmpl of mod.templates || []) {
-        if (tmpl.startsWith('skills/')) {
-          const skillName = tmpl.split('/')[1];
-          ccSkillNames.add(skillName);
-        }
-      }
-    }
-
-    // Collect files that would be removed
-    const toRemove = [];
-    for (const oldPath of Object.keys(existingManifest)) {
-      if (!allManifest[oldPath]) {
-        // Skip phase files — they may be user-customized
-        if (/\/phases\//.test(oldPath)) continue;
-
-        // S1: Only delete if the path maps to a known CC template
-        if (!allTemplatePaths.has(oldPath)) {
-          console.log(`  Keeping non-template file: ${oldPath}`);
-          continue;
-        }
-
-        // S2: Only delete from selected modules (don't purge deselected modules)
-        if (!selectedTemplatePaths.has(oldPath)) {
-          continue;
-        }
-
-        // Project-skill guard: never delete skills CC didn't create
-        const skillDirMatch = oldPath.match(/^\.claude\/skills\/([^/]+)\//);
-        if (skillDirMatch && !ccSkillNames.has(skillDirMatch[1])) {
-          console.log(`  Keeping project skill: ${oldPath}`);
-          continue;
-        }
-
-        const fullPath = path.join(projectDir, oldPath);
-        if (fs.existsSync(fullPath)) {
-          toRemove.push(oldPath);
-        }
-      }
+    // Classification is the pure `computeRemovals` (top-level, unit-tested): S1
+    // only known-template paths, S2 only selected modules, plus the
+    // project-owned-seed and project-skill guards. A FROZEN file needs no case
+    // here — retain-in-manifest (the freeze copy path) kept it in allManifest,
+    // so it never reaches the removal test; a frozen file removed upstream is
+    // kept by S1 ("non-template"). The S3 itemize/confirm + S4 backup/delete
+    // stay inline below.
+    const { toRemove, kept } = computeRemovals({
+      existingManifest,
+      allManifest,
+      selectedModules,
+      templateRoot,
+      projectDir,
+    });
+    for (const k of kept) {
+      // 'deselected module' files are silently left (matching the prior loop's
+      // bare `continue` for that S2 case); the rest log their keep reason.
+      if (k.reason === 'project-owned seed') console.log(`  Keeping project-owned seed: ${k.path}`);
+      else if (k.reason === 'non-template file') console.log(`  Keeping non-template file: ${k.path}`);
+      else if (k.reason === 'project skill') console.log(`  Keeping project skill: ${k.path}`);
     }
 
     // S3: Itemize and confirm before deleting
@@ -1629,6 +1639,7 @@ async function run() {
       skipped: skippedModules,
       version: VERSION,
       manifest: allManifest,
+      dbSchemaVersion,
     });
     console.log('  📝 Created .ccrc.json');
   }
@@ -1684,9 +1695,13 @@ async function run() {
       } else {
         console.log('  📋 Registered in project registry');
       }
-      // Self-announcing: print the effective designation on every run.
+      // Self-announcing: print the effective designation on every run, and
+      // the inert-state notice when none is set — a never-configured
+      // watchdog must not read the same as a passing one (act:77f7b7b2).
       if (registry.frontierModel) {
         console.log(`  🛰  Frontier model: ${registry.frontierModel} (visibility watchdog — /orient + SessionStart warn on mismatch; nothing is pinned)`);
+      } else {
+        console.log('  🛰  Frontier model: none designated — watchdog inert. Set with: npx create-claude-cabinet --frontier-model <value>');
       }
     } catch (err) {
       // Non-fatal — registry is nice-to-have
@@ -1710,4 +1725,4 @@ async function run() {
   console.log('');
 }
 
-module.exports = { run, MODULES, generateAgentWrappers };
+module.exports = { run, MODULES, generateAgentWrappers, generateSkillIndex, expandFrozenClosure, computeRemovals, manifestPath };