@mindrian_os/install 1.13.0-beta.11

package/lib/import/report.test.cjs

@@ -0,0 +1,186 @@
+'use strict';
+
+/**
+ * Phase 80 Plan 80-05 -- lib/import/report.cjs tests.
+ *
+ * Covers:
+ *   1. generateImportReport returns a markdown string with all required sections
+ *   2. Summary counts match manifest decision breakdown
+ *   3. Main-topic extraction: top-2 sections join
+ *   4. Main-topic extraction: single-section dominance (> 60%)
+ *   5. Main-topic extraction: --topic override
+ *   6. UNDONE: true frontmatter when manifest.undo.is_undone
+ *   7. /mos: Usability Check placeholder section is always present
+ */
+
+const assert = require('node:assert/strict');
+const {
+  generateImportReport,
+  extractMainTopicSlug,
+  countDecisions
+} = require('./report.cjs');
+
+function makeManifest(overrides) {
+  return Object.assign({
+    schema_version: '1.0',
+    import_id: '2026-04-13-problem-solution',
+    source_vault: '/tmp/src',
+    status: 'complete',
+    main_topic: null,
+    files: [],
+    people: [],
+    meetings: [],
+    warnings: [],
+    collisions: [],
+    undo: { is_undone: false }
+  }, overrides || {});
+}
+
+function file(source_path, section, confidence, decision, destination) {
+  return {
+    source_path: source_path,
+    destination_path: destination || null,
+    classification: {
+      section: section,
+      confidence: confidence,
+      decision: decision,
+      evidence: 'test'
+    }
+  };
+}
+
+// ---------------------------------------------------------------------------
+// Test 1: required sections present
+// ---------------------------------------------------------------------------
+(function testRequiredSections() {
+  const m = makeManifest({
+    files: [
+      file('a.md', 'problem-definition', 0.9, 'AUTO', 'problem-definition/a/a.md'),
+      file('b.md', 'business-model', 0.5, 'SUGGEST', 'inbox/suggested/b/b.md')
+    ],
+    people: [{ name: 'Jane Doe', role_guess: 'core-team', mention_files: ['f1'], profile_path: 'team/core-team/jane-doe/jane-doe.md' }],
+    meetings: [{ detected_date: '2026-04-10', detection_signals: ['filename-date'], routed_to: 'meetings/2026-04-10-kickoff/2026-04-10-kickoff.md' }]
+  });
+  const out = generateImportReport(m);
+  assert.equal(typeof out, 'string');
+  assert.ok(out.startsWith('---\n'), 'starts with frontmatter');
+  assert.ok(/\ndate: 2026-04-13\n/.test(out), 'date frontmatter');
+  assert.ok(/\n# Import Report -- /.test(out), 'H1 present');
+  assert.ok(/\n## Summary\n/.test(out), 'Summary section');
+  assert.ok(/\n## Routing Table\n/.test(out), 'Routing Table section');
+  assert.ok(/\n## People Detected\n/.test(out), 'People Detected section');
+  assert.ok(/\n## Meetings Detected\n/.test(out), 'Meetings Detected section');
+  assert.ok(/\n## Warnings\n/.test(out), 'Warnings section');
+  assert.ok(/\n## Manifest Reference\n/.test(out), 'Manifest Reference section');
+  assert.ok(out.indexOf('Jane Doe') !== -1, 'person name rendered');
+  assert.ok(out.indexOf('2026-04-10') !== -1, 'meeting date rendered');
+  process.stdout.write('ok required sections\n');
+})();
+
+// ---------------------------------------------------------------------------
+// Test 2: Summary counts
+// ---------------------------------------------------------------------------
+(function testSummaryCounts() {
+  const m = makeManifest({
+    files: [
+      file('a.md', 'problem-definition', 0.9, 'AUTO'),
+      file('b.md', 'problem-definition', 0.85, 'AUTO'),
+      file('c.md', 'business-model', 0.6, 'SUGGEST'),
+      file('d.md', 'inbox', 0.3, 'INBOX')
+    ],
+    people: [{ name: 'P1' }, { name: 'P2' }, { name: 'P3' }],
+    meetings: [{ detected_date: '2026-04-01' }]
+  });
+  const out = generateImportReport(m);
+  assert.ok(out.indexOf('- 4 files ingested') !== -1, 'total files');
+  assert.ok(out.indexOf('- 2 auto-routed') !== -1, 'auto count');
+  assert.ok(out.indexOf('- 1 in inbox/suggested') !== -1, 'suggest count');
+  assert.ok(out.indexOf('- 1 in inbox/unclassified') !== -1, 'inbox count');
+  assert.ok(out.indexOf('- 3 people detected') !== -1, 'people count');
+  assert.ok(out.indexOf('- 1 meetings detected') !== -1, 'meetings count');
+  const c = countDecisions(m.files);
+  assert.deepEqual(c, { auto: 2, sugg: 1, inbox: 1 });
+  process.stdout.write('ok summary counts\n');
+})();
+
+// ---------------------------------------------------------------------------
+// Test 3: Main-topic extraction -- top-2 sections
+// ---------------------------------------------------------------------------
+(function testMainTopicTopTwo() {
+  const m = makeManifest({
+    files: [
+      file('a.md', 'problem-definition', 0.9, 'AUTO'),
+      file('b.md', 'problem-definition', 0.9, 'AUTO'),
+      file('c.md', 'solution-design', 0.9, 'AUTO'),
+      file('d.md', 'solution-design', 0.9, 'AUTO')
+    ]
+  });
+  const slug = extractMainTopicSlug(m);
+  assert.equal(slug, 'problem-definition-solution-design', 'joins top two');
+  process.stdout.write('ok main topic top-two\n');
+})();
+
+// ---------------------------------------------------------------------------
+// Test 4: Single-section dominance (> 60%)
+// ---------------------------------------------------------------------------
+(function testMainTopicDominance() {
+  const m = makeManifest({
+    files: [
+      file('a.md', 'problem-definition', 0.9, 'AUTO'),
+      file('b.md', 'problem-definition', 0.9, 'AUTO'),
+      file('c.md', 'problem-definition', 0.9, 'AUTO'),
+      file('d.md', 'solution-design', 0.9, 'AUTO')
+    ]
+  });
+  const slug = extractMainTopicSlug(m);
+  assert.equal(slug, 'problem-definition', 'dominant section only');
+  process.stdout.write('ok main topic dominance\n');
+})();
+
+// ---------------------------------------------------------------------------
+// Test 5: --topic override
+// ---------------------------------------------------------------------------
+(function testTopicOverride() {
+  const m = makeManifest({
+    files: [file('a.md', 'problem-definition', 0.9, 'AUTO')]
+  });
+  const slug = extractMainTopicSlug(m, 'My Custom Label');
+  assert.equal(slug, 'my-custom-label', 'override slugified');
+  const out = generateImportReport(m, { topic: 'My Custom Label' });
+  assert.ok(/main_topic: my-custom-label/.test(out), 'override in frontmatter');
+  process.stdout.write('ok topic override\n');
+})();
+
+// ---------------------------------------------------------------------------
+// Test 6: UNDONE frontmatter
+// ---------------------------------------------------------------------------
+(function testUndoneFrontmatter() {
+  const m = makeManifest({
+    undo: { is_undone: true, undone_at: '2026-04-13T12:00:00Z' },
+    files: [file('a.md', 'problem-definition', 0.9, 'AUTO')]
+  });
+  const out = generateImportReport(m);
+  assert.ok(/^---\nUNDONE: true\n/.test(out), 'UNDONE: true prepended');
+  assert.ok(out.indexOf('This import has been UNDONE') !== -1, 'undone summary note');
+  process.stdout.write('ok undone frontmatter\n');
+})();
+
+// ---------------------------------------------------------------------------
+// Test 7: /mos: Usability Check placeholder always present
+// ---------------------------------------------------------------------------
+(function testUsabilityPlaceholder() {
+  const m = makeManifest({
+    files: [file('a.md', 'problem-definition', 0.9, 'AUTO')]
+  });
+  const out = generateImportReport(m);
+  assert.ok(out.indexOf('## /mos: Usability Check') !== -1, 'Usability Check section');
+  assert.ok(out.indexOf('Phase 80 Plan 06') !== -1, 'placeholder mentions 80-06');
+
+  // With smoke result passed in, the placeholder is replaced by details
+  const out2 = generateImportReport(m, { smokeTestResult: { status: 'pass', details: 'all green' } });
+  assert.ok(out2.indexOf('status: pass') !== -1, 'smoke status rendered');
+  assert.ok(out2.indexOf('all green') !== -1, 'smoke details rendered');
+  process.stdout.write('ok usability placeholder\n');
+})();
+
+process.stdout.write('\nall report.test.cjs passing\n');

package/lib/import/room-md-scaffolder.cjs

@@ -0,0 +1,49 @@
+'use strict';
+
+/**
+ * Phase 80 Decision 15 helper -- ROOM.md scaffolder for the imports/ tree.
+ *
+ * The vault-import orchestrator (scripts/vault-import.cjs) MUST call
+ * writeImportsRoomMd() after every fs.mkdirSync it performs inside
+ * `imports/{import_id}/` so that EVERY directory in the staging tree carries
+ * an ICM Layer 0 identity file. This is Locked Fix 1 / Blocker 1 from the
+ * 80-CONTEXT.md plan-check, and Decision 15 in CLAUDE.md.
+ *
+ * The function is idempotent: if ROOM.md already exists in dirPath, it does
+ * nothing and returns false. This lets the orchestrator call it freely on
+ * folders that may have been written by an earlier stage.
+ *
+ * The output is intentionally minimal and deterministic (no LLM, no fields
+ * that change between runs) so tests can pin it.
+ */
+
+const fs = require('node:fs');
+const path = require('node:path');
+
+function writeImportsRoomMd(dirPath, layerLabel, importId, stageName, purpose) {
+  if (!fs.existsSync(dirPath)) {
+    fs.mkdirSync(dirPath, { recursive: true });
+  }
+  const abs = path.join(dirPath, 'ROOM.md');
+  if (fs.existsSync(abs)) return false;
+  const stage = stageName || '(staging)';
+  const lines = [
+    '---',
+    'icm_layer: 0',
+    'import_id: ' + importId,
+    'stage: ' + stage,
+    'generated_by: phase-80-vault-import',
+    '---',
+    '',
+    '# ' + (layerLabel || 'Vault Import') + ' -- ICM Layer 0',
+    '',
+    'This folder is part of the Vault Import pipeline (import_id: ' + importId + ').',
+    'Stage: ' + stage,
+    'Purpose: ' + (purpose || 'Layer 4 working artifact for the ' + stage + ' stage.'),
+    ''
+  ];
+  fs.writeFileSync(abs, lines.join('\n'));
+  return true;
+}
+
+module.exports = { writeImportsRoomMd: writeImportsRoomMd };

package/lib/import/router.cjs

@@ -0,0 +1,224 @@
+'use strict';
+
+/**
+ * Phase 80 Stage 03 -- Deterministic router.
+ *
+ * Moves classified files from the source vault into the destination room
+ * following the Decision 16 nested layout: room/{section}/{slug}/{slug}.md
+ *
+ * IMPORT-09 (Blocker 3 Locked Fix): when classification.section === 'inbox'
+ * the router branches on confidence:
+ *   confidence >= 0.45 -> inbox/suggested/{slug}/{slug}.md
+ *   confidence  < 0.45 -> inbox/unclassified/{slug}/{slug}.md
+ *
+ * Filename collisions never overwrite: a second artifact that would land in
+ * an already-occupied destination folder gets a "-imported-{YYYY-MM-DD}"
+ * suffix (and -2, -3, ... if that collides too). Every collision is recorded
+ * via recordCollision on the manifest.
+ *
+ * The router refuses to run if destination_room/STATE.md does not exist. This
+ * prevents accidental routing into non-room directories.
+ *
+ * Meetings detour: files whose id appears in manifest.meetings[] land in
+ * imports/{import_id}/meetings-pending/ instead of a room section. The
+ * 80-04 orchestrator then pipes them through /mos:file-meeting.
+ *
+ * MWP edge names used by downstream consumers of routed artifacts:
+ *   INFORMS, CONVERGES (canonical per docs/MWP-SPECIFICATION.md)
+ */
+
+const fs = require('node:fs');
+const path = require('node:path');
+const matter = require('gray-matter');
+const { updateFile, recordCollision } = require('./manifest.cjs');
+
+const MAX_SLUG_LEN = 60;
+const INBOX_SUGGESTED_THRESHOLD = 0.45;
+
+function slugifyTitle(stem) {
+  if (!stem) return '';
+  return String(stem)
+    .toLowerCase()
+    .normalize('NFD')
+    .replace(/[\u0300-\u036f]/g, '')
+    .replace(/[^a-z0-9\s-]/g, '')
+    .trim()
+    .replace(/\s+/g, '-')
+    .replace(/-+/g, '-')
+    .slice(0, MAX_SLUG_LEN)
+    .replace(/^-+|-+$/g, '');
+}
+
+function pickSlug(file) {
+  const fmTitle = file.source_frontmatter && file.source_frontmatter.title;
+  if (fmTitle) {
+    const s = slugifyTitle(String(fmTitle));
+    if (s) return s;
+  }
+  const base = path.basename(file.source_path || file.source_abs_path || 'untitled');
+  const stem = base.replace(/\.[^.]+$/, '');
+  return slugifyTitle(stem) || 'untitled';
+}
+
+// IMPORT-09 Locked Fix: inbox substructure branching.
+function resolveSectionSubpath(classification, slug) {
+  const section = classification.section;
+  const confidence = Number(classification.confidence) || 0;
+  if (section === 'inbox') {
+    if (confidence >= INBOX_SUGGESTED_THRESHOLD) {
+      return path.join('inbox', 'suggested', slug);
+    }
+    return path.join('inbox', 'unclassified', slug);
+  }
+  return path.join(section, slug);
+}
+
+function nextNonCollidingFolder(roomDir, baseFolder, dateSlug) {
+  const abs = path.join(roomDir, baseFolder);
+  if (!fs.existsSync(abs)) return { folder: baseFolder, collided: false };
+  const parent = path.dirname(baseFolder);
+  const leaf = path.basename(baseFolder);
+  const dated = path.join(parent, leaf + '-imported-' + dateSlug);
+  if (!fs.existsSync(path.join(roomDir, dated))) {
+    return { folder: dated, collided: true };
+  }
+  let n = 2;
+  while (
+    fs.existsSync(
+      path.join(roomDir, parent, leaf + '-imported-' + dateSlug + '-' + n)
+    )
+  ) {
+    n++;
+  }
+  return {
+    folder: path.join(parent, leaf + '-imported-' + dateSlug + '-' + n),
+    collided: true
+  };
+}
+
+function injectProvenance(content, file, manifest) {
+  const parsed = matter(content || '');
+  parsed.data._imported_from = {
+    source_path: file.source_path,
+    source_vault: manifest.source_vault,
+    import_date: (manifest.import_id || '').slice(0, 10)
+  };
+  return matter.stringify(parsed.content, parsed.data);
+}
+
+function routeFiles(manifest, opts) {
+  opts = opts || {};
+  const roomDir = opts.roomDir || manifest.destination_room;
+  if (!roomDir) {
+    throw new Error('routeFiles: destination_room not set on manifest');
+  }
+  if (!fs.existsSync(path.join(roomDir, 'STATE.md'))) {
+    throw new Error(
+      'routeFiles: refusing to route, destination room missing STATE.md at ' +
+        roomDir
+    );
+  }
+
+  const meetingFileIds = new Set(
+    (manifest.meetings || []).map(function (m) {
+      return m.source_file_id;
+    })
+  );
+  const importStaging = path.join(roomDir, 'imports', manifest.import_id);
+  const dateSlug = (manifest.import_id || '').slice(0, 10);
+  const mode = opts.mode || manifest.mode || 'copy';
+
+  for (const file of manifest.files) {
+    if (!file.classification || !file.classification.section) continue;
+
+    // Meeting detour (Blocker 5 fix).
+    if (meetingFileIds.has(file.id)) {
+      const meetingsPending = path.join(importStaging, 'meetings-pending');
+      fs.mkdirSync(meetingsPending, { recursive: true });
+      const target = path.join(
+        meetingsPending,
+        path.basename(file.source_path)
+      );
+      const content = fs.readFileSync(file.source_abs_path, 'utf8');
+      fs.writeFileSync(target, content);
+      updateFile(manifest, file.id, {
+        destination_section: 'meetings-pending',
+        destination_folder: path.relative(roomDir, path.dirname(target)),
+        destination_path: path.relative(roomDir, target),
+        destination_abs_path: target
+      });
+      if (mode === 'move') {
+        try {
+          fs.unlinkSync(file.source_abs_path);
+        } catch (e) {
+          /* soft-fail */
+        }
+      }
+      continue;
+    }
+
+    const slug = pickSlug(file);
+    const baseFolder = resolveSectionSubpath(file.classification, slug);
+    const { folder, collided } = nextNonCollidingFolder(
+      roomDir,
+      baseFolder,
+      dateSlug
+    );
+    const folderAbs = path.join(roomDir, folder);
+    const targetSlug = path.basename(folder);
+    const targetAbs = path.join(folderAbs, targetSlug + '.md');
+
+    fs.mkdirSync(folderAbs, { recursive: true });
+    const sourceContent = fs.readFileSync(file.source_abs_path, 'utf8');
+    const enriched = injectProvenance(sourceContent, file, manifest);
+    fs.writeFileSync(targetAbs, enriched);
+
+    if (collided) {
+      recordCollision(manifest, {
+        source_file_id: file.id,
+        intended_target: path.join(baseFolder, slug + '.md'),
+        actual_target: path.relative(roomDir, targetAbs),
+        reason: 'destination_folder_exists'
+      });
+    }
+
+    // destination_section records the TOP-LEVEL section (inbox, problem-definition, ...)
+    // not the sub-branch (inbox/suggested). This preserves backward compatibility
+    // with manifest.files[].destination_section consumers while the full path lives
+    // in destination_folder.
+    const topSection = folder.split(path.sep)[0];
+    updateFile(manifest, file.id, {
+      destination_section: topSection,
+      destination_folder: folder,
+      destination_path: path.relative(roomDir, targetAbs),
+      destination_abs_path: targetAbs,
+      collision: !!collided
+    });
+
+    if (mode === 'move') {
+      try {
+        fs.unlinkSync(file.source_abs_path);
+      } catch (e) {
+        /* soft-fail */
+      }
+    }
+  }
+
+  manifest.stage_states.route.status = 'complete';
+  manifest.stage_states.route.completed_at = new Date().toISOString();
+  manifest.stage_states.route.files_moved = manifest.files.filter(function (f) {
+    return f.destination_path;
+  }).length;
+  manifest.stage_states.route.collisions = (manifest.collisions || []).length;
+  return manifest;
+}
+
+module.exports = {
+  routeFiles: routeFiles,
+  pickSlug: pickSlug,
+  slugifyTitle: slugifyTitle,
+  nextNonCollidingFolder: nextNonCollidingFolder,
+  resolveSectionSubpath: resolveSectionSubpath,
+  INBOX_SUGGESTED_THRESHOLD: INBOX_SUGGESTED_THRESHOLD,
+  MAX_SLUG_LEN: MAX_SLUG_LEN
+};