hypomnema 1.3.2 → 1.3.4

package/scripts/graph.mjs

@@ -17,7 +17,7 @@
 import { existsSync, readFileSync, readdirSync, statSync } from 'fs';
 import { join, relative, extname, basename } from 'path';
 import { resolveHypoRoot, expandHome } from './lib/hypo-root.mjs';
-import { loadHypoIgnore, isIgnored } from './lib/hypo-ignore.mjs';
+import { loadHypoIgnore, isScanIgnored } from './lib/hypo-ignore.mjs';
 
 // ── arg parsing ───────────────────────────────────────────────────────────────
 
@@ -38,7 +38,7 @@ function collectPages(dir, root, pages = [], ignorePatterns = []) {
   if (!existsSync(dir)) return pages;
   for (const entry of readdirSync(dir)) {
     const full = join(dir, entry);
-    if (isIgnored(full, root, ignorePatterns)) continue;
+    if (isScanIgnored(full, root, ignorePatterns)) continue;
     const st = statSync(full);
     if (st.isDirectory()) {
       collectPages(full, root, pages, ignorePatterns);

package/scripts/init.mjs

@@ -32,7 +32,7 @@ import {
   renameSync,
   unlinkSync,
 } from 'fs';
-import { join, basename } from 'path';
+import { join, basename, dirname } from 'path';
 import { homedir } from 'os';
 import { execSync, spawnSync } from 'child_process';
 import { fileURLToPath } from 'url';
@@ -171,7 +171,19 @@ docstring at the top of scripts/<command>.mjs.`);
 
 // ── result tracking ──────────────────────────────────────────────────────────
 
-const results = { created: [], skipped: [], merged: [], errors: [] };
+const results = { created: [], skipped: [], merged: [], deduped: [], errors: [] };
+
+// Stock template basename → legacy hand-authored equivalents to honor instead of
+// injecting a duplicate. The hypo-/wiki- namespace split (the rename that
+// produced hypo-guide from wiki-guide, etc.) means an exact-filename check misses
+// a user's existing page and drops a 0-reference duplicate orphan beside it.
+// Mirrors the explicit HOOK_RENAMES idiom in upgrade.mjs. hypo-guide.md is
+// intentionally absent: the runtime reads it by name, so a mid-migration vault
+// must still receive it even when wiki-guide.md is present.
+const PAGE_EQUIVALENTS = {
+  'hypo-automation.md': ['wiki-automation.md'],
+  'hypo-help.md': ['wiki-help.md'],
+};
 
 function log(action, path) {
   results[action].push(path);
@@ -213,6 +225,19 @@ function copyTemplate(srcName, destPath, dryRun, transform) {
     log('skipped', destPath);
     return;
   }
+  // If a legacy hand-authored equivalent already exists, do NOT inject the stock
+  // page — that is exactly what produced a duplicate orphan. Skip LOUDLY (own
+  // bucket, ⚠ in the summary) so the user can merge manually. Fires in --dry-run
+  // too, so a dry run previews the suppression.
+  const equivalents = PAGE_EQUIVALENTS[basename(srcName)] || [];
+  const existingEquivalent = equivalents.find((name) => existsSync(join(dirname(destPath), name)));
+  if (existingEquivalent) {
+    log(
+      'deduped',
+      `${destPath} — kept existing ${existingEquivalent}; stock ${basename(srcName)} not installed (merge manually if you want the template content)`,
+    );
+    return;
+  }
   if (!dryRun) {
     let content = readFileSync(src, 'utf-8');
     content = content.replace(/YYYY-MM-DD/g, new Date().toISOString().slice(0, 10));
@@ -1017,6 +1042,10 @@ if (results.skipped.length)
   );
 if (results.merged.length)
   lines.push(`↪ Merged into settings.json:\n${results.merged.map((p) => `  ${p}`).join('\n')}`);
+if (results.deduped.length)
+  lines.push(
+    `⚠ Skipped to avoid a duplicate — an equivalent page already exists (${results.deduped.length}):\n${results.deduped.map((p) => `  ${p}`).join('\n')}`,
+  );
 if (results.errors.length)
   lines.push(`✗ Errors:\n${results.errors.map((p) => `  ${p}`).join('\n')}`);
 

package/scripts/lib/hypo-ignore.mjs

@@ -23,6 +23,30 @@ function globToRegex(glob) {
   );
 }
 
+// Generated, regenerable tool artifacts that live at the WIKI ROOT and must not
+// pollute the knowledge catalog. These are intentionally NOT added to
+// .hypoignore: that list also drives the pre-commit secret gate
+// (hooks/hypo-pre-commit.mjs), so listing them there would block the report's
+// commit and freeze every auto-commit while it sits at root. Instead they are
+// excluded from catalog/scan only, via isScanIgnored(). The patterns are
+// anchored to the root-relative path (no '/'), so an equivalent name nested
+// under pages/, projects/, or sources/ is left untouched.
+const GENERATED_ARTIFACT_RES = [/^MIGRATION-v[^/]*\.md$/, /^GRAPH_REPORT\.md$/];
+
+export function isGeneratedArtifact(filePath, hypoDir) {
+  const rel = relative(hypoDir, filePath).replace(/\\/g, '/');
+  return GENERATED_ARTIFACT_RES.some((re) => re.test(rel));
+}
+
+// Catalog/scan exclusion = privacy/.hypoignore patterns PLUS generated root
+// artifacts. Use this in tools that build the knowledge catalog (lint, graph,
+// stats, query, verify, crystallize, rename, doctor broken-links). Do NOT use it
+// in the pre-commit gate or ingest --check, which are privacy boundaries that
+// must stay on isIgnored()/.hypoignore alone.
+export function isScanIgnored(filePath, hypoDir, patterns) {
+  return isIgnored(filePath, hypoDir, patterns) || isGeneratedArtifact(filePath, hypoDir);
+}
+
 export function isIgnored(filePath, hypoDir, patterns) {
   const rel = relative(hypoDir, filePath).replace(/\\/g, '/');
   const base = basename(filePath);

package/scripts/lint.mjs

@@ -20,7 +20,7 @@ import { existsSync, readFileSync, writeFileSync, readdirSync, statSync } from '
 import { join, relative, extname, basename } from 'path';
 import { resolveHypoRoot, expandHome } from './lib/hypo-root.mjs';
 import { SESSION_STATE_NEXT_HEADINGS } from '../hooks/hypo-shared.mjs';
-import { loadHypoIgnore, isIgnored } from './lib/hypo-ignore.mjs';
+import { loadHypoIgnore, isScanIgnored } from './lib/hypo-ignore.mjs';
 import { parseSchemaVocab, checkForbidden, parseSchemaPageDirs } from './lib/schema-vocab.mjs';
 import { findDesignHistoryStale } from './lib/design-history-stale.mjs';
 import { FEEDBACK_SCOPE_RE } from './lib/feedback-scope.mjs';
@@ -117,7 +117,7 @@ function collectPages(dir, root, pages = [], ignorePatterns = []) {
   if (!existsSync(dir)) return pages;
   for (const entry of readdirSync(dir)) {
     const full = join(dir, entry);
-    if (isIgnored(full, root, ignorePatterns)) continue;
+    if (isScanIgnored(full, root, ignorePatterns)) continue;
     const st = statSync(full);
     if (st.isDirectory()) {
       // `_`-prefixed dirs (e.g. pages/feedback/_drafts) hold scaffolds / scratch
@@ -164,13 +164,14 @@ function collectLinkTargets(hypoDir, ignorePatterns = []) {
   if (existsSync(hypoDir)) {
     for (const entry of readdirSync(hypoDir)) {
       const full = join(hypoDir, entry);
-      // root-level *.md FILES only (no recursion), honoring .hypoignore exactly
-      // like collectPages — otherwise an ignored root file (e.g. a secret) would
-      // resolve [[its-slug]] as valid, a false negative.
+      // root-level *.md FILES only (no recursion), honoring .hypoignore plus the
+      // scan-only generated-artifact exclusions (isScanIgnored) like collectPages
+      // — otherwise an ignored root file (e.g. a secret) or a regenerable report
+      // would resolve [[its-slug]] as valid, a false negative.
       if (
         extname(entry) === '.md' &&
         !entry.startsWith('.') &&
-        !isIgnored(full, hypoDir, ignorePatterns) &&
+        !isScanIgnored(full, hypoDir, ignorePatterns) &&
         statSync(full).isFile()
       ) {
         targets.push(entry.replace(/\.md$/, ''));

package/scripts/query.mjs

@@ -19,7 +19,7 @@
 import { existsSync, readFileSync, readdirSync, statSync } from 'fs';
 import { join, relative, extname } from 'path';
 import { resolveHypoRoot, expandHome } from './lib/hypo-root.mjs';
-import { loadHypoIgnore, isIgnored } from './lib/hypo-ignore.mjs';
+import { loadHypoIgnore, isScanIgnored } from './lib/hypo-ignore.mjs';
 
 // ── arg parsing ──────────────────────────────────────────────────────────────
 
@@ -42,7 +42,7 @@ function collectMdFiles(dir, root, acc = [], ignorePatterns = []) {
   for (const entry of readdirSync(dir)) {
     if (entry.startsWith('.')) continue;
     const full = join(dir, entry);
-    if (isIgnored(full, root, ignorePatterns)) continue;
+    if (isScanIgnored(full, root, ignorePatterns)) continue;
     const st = statSync(full);
     if (st.isDirectory()) collectMdFiles(full, root, acc, ignorePatterns);
     else if (extname(entry) === '.md') acc.push({ path: full, rel: relative(root, full) });