hypomnema 1.3.4 → 1.4.1

package/scripts/lint.mjs

@@ -17,13 +17,21 @@
  */
 
 import { existsSync, readFileSync, writeFileSync, readdirSync, statSync } from 'fs';
-import { join, relative, extname, basename } from 'path';
+import { join, extname, basename } from 'path';
 import { resolveHypoRoot, expandHome } from './lib/hypo-root.mjs';
 import { SESSION_STATE_NEXT_HEADINGS } from '../hooks/hypo-shared.mjs';
 import { loadHypoIgnore, isScanIgnored } from './lib/hypo-ignore.mjs';
-import { parseSchemaVocab, checkForbidden, parseSchemaPageDirs } from './lib/schema-vocab.mjs';
+import {
+  parseSchemaVocab,
+  checkForbidden,
+  parseSchemaPageDirs,
+  parseSchemaTypes,
+} from './lib/schema-vocab.mjs';
 import { findDesignHistoryStale } from './lib/design-history-stale.mjs';
 import { FEEDBACK_SCOPE_RE } from './lib/feedback-scope.mjs';
+import { FAILURE_TYPE_ENUM } from './lib/failure-type.mjs';
+import { collectPagesLint, slugForms } from './lib/wikilink.mjs';
+import { parseFrontmatter, SEQUENCE_ENTRY_RE } from './lib/frontmatter.mjs';
 
 // ── arg parsing ──────────────────────────────────────────────────────────────
 
@@ -41,19 +49,45 @@ function parseArgs(argv) {
 
 // ── frontmatter parser ────────────────────────────────────────────────────────
 
-function parseFrontmatter(content) {
-  const m = content.match(/^---\r?\n([\s\S]*?)\r?\n---/);
-  if (!m) return null;
-  const fm = {};
-  for (const line of m[1].split('\n')) {
+// ── W9: narrow invalid-YAML detector ───────────────────────────────────────────
+// NOT a full YAML parser (zero-dep policy). Catches only specific classes the
+// lenient line-scanner (parseFrontmatter, imported from lib) silently passes but
+// a real YAML parser (js-yaml, what Obsidian uses) rejects. Each class is
+// conservative — it may miss, but must NOT false-positive on valid YAML, so it
+// inspects only top-level (unindented) lines: indented lines may be block-scalar
+// content where ": "/tabs are legal, and reliably telling content from structure
+// needs a real parser. (A leading-tab class was considered and dropped for this
+// reason — it cannot distinguish `relations:\n\t- x` from a tabbed block-scalar
+// body without tracking block context.) Returns an array of reasons.
+function checkYamlInvalid(block) {
+  const reasons = [];
+  const seen = new Set();
+  for (const raw of block.split('\n')) {
+    const line = raw.replace(/\r$/, '');
+    if (!line.trim()) continue;
+    if (/^\s/.test(line) || SEQUENCE_ENTRY_RE.test(line)) continue; // top-level keys only
     const idx = line.indexOf(':');
     if (idx < 0) continue;
-    fm[line.slice(0, idx).trim()] = line
-      .slice(idx + 1)
-      .trim()
-      .replace(/^["']|["']$/g, '');
+    const key = line.slice(0, idx).trim();
+    if (!key) continue;
+    // duplicate top-level key (YAML rejects; line-scanner silently keeps one)
+    if (seen.has(key)) reasons.push(`duplicate key: "${key}"`);
+    else seen.add(key);
+    // colon-space inside an unquoted, non-flow plain scalar. A plain scalar may
+    // not contain ": " — js-yaml hard-errors or silently truncates. Skip quoted
+    // ("/') and flow ([/{) values where ": " is legal, and strip an inline
+    // " #" comment before scanning.
+    let val = line.slice(idx + 1).trim();
+    const c = val[0];
+    if (val && c !== '"' && c !== "'" && c !== '[' && c !== '{') {
+      const hash = val.indexOf(' #');
+      if (hash >= 0) val = val.slice(0, hash);
+      if (/:\s/.test(val)) {
+        reasons.push(`unquoted "${key}" value contains ": " (quote it): "${val.slice(0, 40)}"`);
+      }
+    }
   }
-  return fm;
+  return reasons;
 }
 
 function parseTagsField(rawValue) {
@@ -108,31 +142,14 @@ const TYPE_ENUM_FIELDS = {
     status: ['active', 'superseded', 'archived'],
     tier: ['L1', 'L2'],
     sensitivity: ['public', 'sanitized'],
+    // FEAT-1: optional failure taxonomy. The enum loop guards on `if (fm[field])`,
+    // so an omitted failure_type is never validated (optional, migration-safe).
+    failure_type: FAILURE_TYPE_ENUM,
   },
 };
 
 // ── page collector ────────────────────────────────────────────────────────────
 
-function collectPages(dir, root, pages = [], ignorePatterns = []) {
-  if (!existsSync(dir)) return pages;
-  for (const entry of readdirSync(dir)) {
-    const full = join(dir, entry);
-    if (isScanIgnored(full, root, ignorePatterns)) continue;
-    const st = statSync(full);
-    if (st.isDirectory()) {
-      // `_`-prefixed dirs (e.g. pages/feedback/_drafts) hold scaffolds / scratch
-      // not yet promoted to content — skip so incomplete frontmatter never errors
-      // (mirrors loadFeedbackPages in feedback-sync.mjs). `_`-prefixed *files*
-      // (e.g. _index.md) are still linted.
-      if (entry.startsWith('_')) continue;
-      collectPages(full, root, pages, ignorePatterns);
-    } else if (extname(entry) === '.md' && !entry.startsWith('.')) {
-      pages.push({ path: full, rel: relative(root, full) });
-    }
-  }
-  return pages;
-}
-
 // ── slug map ─────────────────────────────────────────────────────────────────
 
 // `extraTargets` are link-target-only slugs (root *.md, sources/*) that resolve
@@ -142,13 +159,14 @@ function buildSlugMap(pages, extraTargets = []) {
   const map = new Set();
   for (const { rel } of pages) {
     const noExt = rel.replace(/\.md$/, '').replace(/\\/g, '/');
-    map.add(noExt);
-    map.add(basename(rel, '.md'));
-    // Dir-relative alias: drop the leading scan-dir segment so the vault's
-    // convention link [[learnings/foo]] resolves to pages/learnings/foo.md.
-    // (A page directly under a scan dir has no extra segment to drop.)
-    const slash = noExt.indexOf('/');
-    if (slash !== -1) map.add(noExt.slice(slash + 1));
+    // full slug + bare basename + dir-relative alias (drop the leading scan-dir
+    // segment so the convention link [[learnings/foo]] resolves to
+    // pages/learnings/foo.md). slugForms returns dirRel=null when the slug has no
+    // `/` (a page directly under a scan dir has no extra segment to drop).
+    const { full, bare, dirRel } = slugForms(noExt);
+    map.add(full);
+    map.add(bare);
+    if (dirRel) map.add(dirRel);
   }
   for (const t of extraTargets) map.add(t);
   return map;
@@ -165,7 +183,7 @@ function collectLinkTargets(hypoDir, ignorePatterns = []) {
     for (const entry of readdirSync(hypoDir)) {
       const full = join(hypoDir, entry);
       // root-level *.md FILES only (no recursion), honoring .hypoignore plus the
-      // scan-only generated-artifact exclusions (isScanIgnored) like collectPages
+      // scan-only generated-artifact exclusions (isScanIgnored) like collectPagesLint
       // — otherwise an ignored root file (e.g. a secret) or a regenerable report
       // would resolve [[its-slug]] as valid, a false negative.
       if (
@@ -180,7 +198,7 @@ function collectLinkTargets(hypoDir, ignorePatterns = []) {
   }
   // sources/*: linkable as the full 'sources/<name>' slug only — deliberately no
   // bare basename, so a stale [[name]] can't silently resolve to a source file.
-  for (const { rel } of collectPages(join(hypoDir, 'sources'), hypoDir, [], ignorePatterns)) {
+  for (const { rel } of collectPagesLint(join(hypoDir, 'sources'), hypoDir, ignorePatterns)) {
     targets.push(rel.replace(/\.md$/, '').replace(/\\/g, '/'));
   }
   return targets;
@@ -259,10 +277,15 @@ const issues = [];
 // STRICT_PROMOTE_IDS (OQ-E1, frozen as a code constant): confirmed content
 // defects only.
 //   W1 no-frontmatter / W2 unknown-type / W4 broken-wikilink → promote.
+//   W9 invalid-YAML → promote (frontmatter a real YAML parser rejects).
 //   W3 missing-updated  → excluded (auto-repaired by --fix).
 //   W8 design-history-stale → excluded (hypo-personal-check handles it; would
 //                             double-gate).
-const STRICT_PROMOTE_IDS = new Set(['W1', 'W2', 'W4']);
+// NOTE: no gate currently passes --strict (npm run lint / CI / release.yml /
+// crystallize / the close-gate all run plain lint), so promotion is
+// forward-looking — these surface as warnings today. IMPR-3's deliverable is
+// "stop silently green-passing invalid YAML"; the W9 warning satisfies that.
+const STRICT_PROMOTE_IDS = new Set(['W1', 'W2', 'W4', 'W9']);
 
 function issue(severity, rel, msg, fullPath = null, id = null) {
   issues.push({ severity, file: rel, message: msg, path: fullPath, id });
@@ -285,7 +308,7 @@ function lintSessionStateHeadings(content, rel) {
   }
 }
 
-function lintPage({ path, rel }, slugMap, tagVocab, pageDirs) {
+function lintPage({ path, rel }, slugMap, tagVocab, pageDirs, validTypes) {
   // Directory whitelist: a content page under pages/<subdir>/ must live in a
   // SCHEMA-defined directory. Catches typo'd dirs (e.g. pages/learning/ vs the
   // canonical pages/learnings/) regardless of frontmatter, since a directory
@@ -319,6 +342,16 @@ function lintPage({ path, rel }, slugMap, tagVocab, pageDirs) {
     return;
   }
 
+  // W9: invalid-YAML frontmatter the lenient line-scanner would otherwise pass
+  // green. Runs on the raw `---` block (the `content.match` above guarantees an
+  // opening fence; an unclosed fence falls through to the W3/parse path below).
+  const fmBlock = content.match(/^---\r?\n([\s\S]*?)\r?\n---/);
+  if (fmBlock) {
+    for (const reason of checkYamlInvalid(fmBlock[1])) {
+      issue('warn', rel, `Invalid YAML frontmatter: ${reason}`, null, 'W9');
+    }
+  }
+
   const fm = parseFrontmatter(content);
   if (!fm) {
     issue('error', rel, 'Malformed frontmatter (unclosed ---)');
@@ -329,7 +362,7 @@ function lintPage({ path, rel }, slugMap, tagVocab, pageDirs) {
     if (!fm[field]) issue('error', rel, `Missing required frontmatter field: ${field}`);
   }
 
-  if (fm.type && !VALID_TYPES.includes(fm.type)) {
+  if (fm.type && !validTypes.has(fm.type)) {
     issue('warn', rel, `Unknown type: "${fm.type}"`, null, 'W2');
   }
 
@@ -413,13 +446,19 @@ const args = parseArgs(process.argv);
 
 const ignorePatterns = loadHypoIgnore(args.hypoDir);
 const scanDirs = ['pages', 'projects', 'journal'].map((d) => join(args.hypoDir, d));
-const pages = scanDirs.flatMap((d) => collectPages(d, args.hypoDir, [], ignorePatterns));
+const pages = scanDirs.flatMap((d) => collectPagesLint(d, args.hypoDir, ignorePatterns));
 const linkTargets = collectLinkTargets(args.hypoDir, ignorePatterns);
 const slugMap = buildSlugMap(pages, linkTargets);
 const tagVocab = parseSchemaVocab(args.hypoDir);
 const pageDirs = parseSchemaPageDirs(args.hypoDir);
-
-for (const page of pages) lintPage(page, slugMap, tagVocab, pageDirs);
+// Accepted page types = hardcoded core ∪ the vault SCHEMA's taxonomy. Union (not
+// replace) so core types lint specially handles (session-state, source, …) are
+// never lost when a vault's SCHEMA omits or predates them, while a vault-local
+// extension (working-doc / draft / qa-run) is honored without editing lint —
+// mirroring how tags and page dirs are SCHEMA-derived.
+const validTypes = new Set([...VALID_TYPES, ...parseSchemaTypes(args.hypoDir)]);
+
+for (const page of pages) lintPage(page, slugMap, tagVocab, pageDirs, validTypes);
 
 // W8: design-history.md stale relative to session-log.md. Emitted once per
 // project (not per page) — runs outside the page loop. POSIX-separated path

package/scripts/rename.mjs

@@ -47,9 +47,10 @@ import {
   lstatSync,
   realpathSync,
 } from 'fs';
-import { join, relative, extname, basename, dirname, normalize, isAbsolute, sep } from 'path';
+import { join, basename, dirname, normalize, isAbsolute, sep } from 'path';
 import { resolveHypoRoot, expandHome } from './lib/hypo-root.mjs';
-import { loadHypoIgnore, isScanIgnored } from './lib/hypo-ignore.mjs';
+import { loadHypoIgnore } from './lib/hypo-ignore.mjs';
+import { collectPagesRename, slugForms } from './lib/wikilink.mjs';
 
 // ── arg parsing ───────────────────────────────────────────────────────────────
 
@@ -66,28 +67,6 @@ function parseArgs(argv) {
   return args;
 }
 
-// ── page collection (mirrors graph.mjs / crystallize.mjs collectPages) ──────────
-
-function collectPages(dir, root, pages = [], ignorePatterns = []) {
-  if (!existsSync(dir)) return pages;
-  for (const entry of readdirSync(dir)) {
-    const full = join(dir, entry);
-    if (isScanIgnored(full, root, ignorePatterns)) continue;
-    // lstat (never follow): a symlinked dir or file is skipped entirely so the
-    // whole-vault scan can never traverse out of the vault via a symlink.
-    const st = lstatSync(full);
-    if (st.isSymbolicLink()) {
-      continue;
-    } else if (st.isDirectory()) {
-      collectPages(full, root, pages, ignorePatterns);
-    } else if (extname(entry) === '.md' && !entry.startsWith('.')) {
-      const rel = relative(root, full).replace(/\\/g, '/');
-      pages.push({ path: full, rel, slug: rel.replace(/\.md$/, ''), bare: basename(full, '.md') });
-    }
-  }
-  return pages;
-}
-
 // ── preservation class of a link-source path ───────────────────────────────────
 // Two distinct reasons a file is normally skipped as a link SOURCE, kept separate
 // because directory mode treats them differently (codex design BLOCKER):
@@ -123,10 +102,7 @@ function isPreservedSource(rel) {
 // needs to KNOW when a form is shared, so it maps each form → the set of page
 // rels that expose it. precedence forms per page: full noExt slug, bare
 // basename, dir-relative (drop the leading scan-dir segment).
-function dirRelForm(slug) {
-  const slash = slug.indexOf('/');
-  return slash === -1 ? null : slug.slice(slash + 1);
-}
+const dirRelForm = (slug) => slugForms(slug).dirRel;
 
 function buildFormIndex(pages) {
   const index = new Map(); // form → Set<rel>
@@ -473,7 +449,7 @@ function runDirectory(args, fromDirRel, ignorePatterns) {
     fail(args, `--from subtree contains a symlink — refusing (move could escape the vault)`);
   }
 
-  const pages = collectPages(args.hypoDir, args.hypoDir, [], ignorePatterns);
+  const pages = collectPagesRename(args.hypoDir, args.hypoDir, ignorePatterns);
   const formIndex = buildFormIndex(pages);
 
   // The moving pages: every collected page whose rel is under the from-directory.
@@ -676,7 +652,7 @@ function run(args) {
     return runDirectory(args, fromNorm, ignorePatterns);
   }
 
-  const pages = collectPages(args.hypoDir, args.hypoDir, [], ignorePatterns);
+  const pages = collectPagesRename(args.hypoDir, args.hypoDir, ignorePatterns);
   const formIndex = buildFormIndex(pages);
 
   const fromPage = resolveArgToPage(args.from, pages, formIndex);

package/scripts/stats.mjs

@@ -45,16 +45,25 @@ function collectMdFiles(dir, acc = [], hypoDir = '', ignorePatterns = []) {
   return acc;
 }
 
+// Local top-level-only extractor, behavior-aligned with lib/frontmatter.mjs:
+// skip indented / list lines, first-wins, strip a trailing `# comment`. Without
+// the comment strip a `failure_type: overreach # note` would aggregate under the
+// literal "overreach # note" key (FEAT-1). Kept local (not the shared import) to
+// hold stats' dependency surface flat per the FEAT-1 scope split.
 function parseFrontmatter(content) {
   const m = content.match(/^---\r?\n([\s\S]*?)\r?\n---/);
   if (!m) return null;
   const fm = {};
-  for (const line of m[1].split('\n')) {
+  for (const line of m[1].split(/\r?\n/)) {
+    if (/^\s/.test(line) || /^-(\s|$)/.test(line)) continue; // nested / list item
     const idx = line.indexOf(':');
     if (idx < 0) continue;
-    fm[line.slice(0, idx).trim()] = line
+    const key = line.slice(0, idx).trim();
+    if (!key || Object.hasOwn(fm, key)) continue; // first-wins
+    fm[key] = line
       .slice(idx + 1)
       .trim()
+      .replace(/\s+#.*$/, '')
       .replace(/^["']|["']$/g, '');
   }
   return fm;
@@ -92,6 +101,7 @@ const sources = existsSync(join(args.hypoDir, 'sources'))
   : [];
 
 const typeCounts = {};
+const failureTypeCounts = {}; // FEAT-1: feedback pages carrying a failure_type
 let missingFrontmatter = 0;
 
 for (const f of pageFiles) {
@@ -108,6 +118,9 @@ for (const f of pageFiles) {
   }
   const t = fm.type || 'unknown';
   typeCounts[t] = (typeCounts[t] || 0) + 1;
+  if (t === 'feedback' && fm.failure_type) {
+    failureTypeCounts[fm.failure_type] = (failureTypeCounts[fm.failure_type] || 0) + 1;
+  }
 }
 
 let adrCount = 0;
@@ -122,6 +135,9 @@ const lastActivity = getLastActivity(args.hypoDir);
 
 const stats = {
   pages: { total: pageFiles.length, byType: typeCounts, missingFrontmatter },
+  // Omit the key entirely when no feedback page is classified (OQ-4: no empty
+  // section noise) so consumers can branch on presence.
+  ...(Object.keys(failureTypeCounts).length ? { failureTypes: failureTypeCounts } : {}),
   projects: projects.length,
   sources: sources.length,
   adrs: adrCount,
@@ -139,6 +155,10 @@ if (args.json) {
   if (missingFrontmatter) {
     console.log(`  missing frontmatter: ${missingFrontmatter}`);
   }
+  const ftEntries = Object.entries(failureTypeCounts).sort((a, b) => b[1] - a[1]);
+  if (ftEntries.length) {
+    console.log(`  failure types: ${ftEntries.map(([t, n]) => `${t} (${n})`).join(', ')}`);
+  }
   console.log(`Projects: ${projects.length}`);
   console.log(`Sources:  ${sources.length}`);
   console.log(`ADRs:     ${adrCount}`);

package/scripts/upgrade.mjs

@@ -42,6 +42,7 @@ import { fileURLToPath } from 'url';
 import { createHash } from 'crypto';
 import { resolveHypoRoot, expandHome } from './lib/hypo-root.mjs';
 import { parseFrontmatter } from './lib/frontmatter.mjs';
+import { templateSchemaVersion } from './lib/template-schema-version.mjs';
 import {
   readPkgJson as readPkgJsonSafe,
   writePkgJsonAtomic,
@@ -477,7 +478,14 @@ function writeMigrationReport(hypoDir, fromVersion, toVersion, { pluginMode = fa
   // right to keep SCHEMA.md as user-owned (Option C); auto-stub of the 9 new
   // fields is rejected because scope/tier/targets/sensitivity/reason/source
   // are semantic decisions whose wrong defaults would project wrong behavior.
-  const isV1ToV2 = fromVersion === '1.0' && toVersion === '2.0';
+  // Fire for any 1.x → 2.x major crossing, not just 1.0 → 2.0 exactly: the ADR
+  // 0031 feedback-field backfill that this body explains was introduced at 2.0
+  // and still applies to a user landing on any later 2.x (e.g. 2.1's additive
+  // failure_type — FEAT-1). Keying on the exact target string would silently
+  // drop this guidance the moment the template minor moves.
+  const fromV = parseVersion(fromVersion);
+  const toV = parseVersion(toVersion);
+  const isV1ToV2 = fromV?.major === 1 && toV?.major === 2;
   const specificBody = isV1ToV2
     ? `## What changed in SCHEMA 2.0
 
@@ -755,7 +763,7 @@ function applyCommands(commandResults, force) {
     ...existing,
     pkgRoot: PKG_ROOT,
     pkgVersion,
-    schemaVersion: '2.0',
+    schemaVersion: templateSchemaVersion(PKG_ROOT) ?? '2.1',
     commands: newSHAs,
   });
   return applied;
@@ -781,7 +789,7 @@ function writePluginModeMetadata() {
     ...existing,
     pkgRoot: PKG_ROOT,
     pkgVersion,
-    schemaVersion: '2.0',
+    schemaVersion: templateSchemaVersion(PKG_ROOT) ?? '2.1',
   });
   return true;
 }

package/scripts/weekly-report.mjs

@@ -123,7 +123,7 @@ export function buildReport(
   return { week: weekLabel, count: weekResults.length, counts, score, results: weekResults };
 }
 
-function renderMarkdown(report) {
+function renderMarkdown(report, hypoDir) {
   const { week, count, counts, score, results } = report;
   const today = new Date().toISOString().slice(0, 10);
   const lines = [];
@@ -138,8 +138,14 @@ function renderMarkdown(report) {
   lines.push('');
   lines.push(`# Observability — Week ${week}`);
   lines.push('');
+  // Only emit the wikilink when its target page exists, so a fresh vault
+  // (or a deleted _index) does not stamp a broken link into every weekly.
+  const indexExists = !!hypoDir && existsSync(join(hypoDir, 'pages', 'observability', '_index.md'));
+  const heuristicRef = indexExists
+    ? '[[pages/observability/_index]]'
+    : '`pages/observability/_index.md` (not present in this vault)';
   lines.push(
-    `> Generated by \`scripts/weekly-report.mjs\`. Heuristic definition: [[pages/observability/_index]].`,
+    `> Generated by \`scripts/weekly-report.mjs\`. Heuristic definition: ${heuristicRef}.`,
   );
   lines.push('');
   lines.push(`## Autonomy score (heuristic v0): **${score}%**`);
@@ -196,7 +202,7 @@ if (isMain()) {
     process.exit(0);
   }
 
-  const md = renderMarkdown(report);
+  const md = renderMarkdown(report, args.hypoDir);
 
   if (args.write) {
     const dir = join(args.hypoDir, 'journal', 'weekly');

package/skills/crystallize/SKILL.md

@@ -76,7 +76,27 @@ reports and re-run until it prints **"Compact-ready"** — that is the signal th
 session is closed. A close-files-only pass is not enough; the real `/compact`
 also blocks on those other checks.
 
-Once it passes, report each item with ✓ and ask: "Session closed. Would you like to also run knowledge synthesis now, or stop here?"
+**When using `--apply-session-close --session-id=<id>`** (the payload-driven
+path), the `--session-id` must be the main conversation's session id. Do NOT
+extract it from a background task or Agent output path (e.g., a UUID from
+`/tmp/.../<uuid>/tasks/...`). Such a UUID is a background task id, not the
+main conversation id. Passing the wrong id causes `markerWritten: false` with
+`markerSkipReason: "transcript-unresolved"` or `"no-user-close-signal"`: the
+5 mandatory files are written (ok: true) but the Stop-chain marker is withheld,
+so the session is not actually closed and the Stop hook re-prompts. The correct
+id comes from the `[WIKI_AUTOCLOSE]` block reason or `$CLAUDE_SESSION_ID`.
+
+Once it passes, report each item with ✓:
+- ✓ session-state.md
+- ✓ hot.md (project + root)
+- ✓ session-log entry
+- ✓ open-questions (or skipped if unchanged)
+- ✓ log.md entry
+- **marker written?** (required, if `--apply-session-close --session-id` was used): check `markerWritten` in the JSON output. If `true`, report "session-close marker written." If `false`, do NOT declare the session closed or complete. Instead report: "Files applied and verified (ok: true), but the session-close marker was not written (reason: `<markerSkipReason>`). The Stop-chain is still active. Re-run with the correct main-conversation `--session-id`."
+
+If `markerWritten: true` (or `--session-id` was not passed), ask: "Session closed. Would you like to also run knowledge synthesis now, or stop here?"
+
+If `markerWritten: false`, do NOT say "session closed." Surface the skip reason and instruct a re-run with the correct main-conversation `--session-id` (see the bullet above) before treating the session as closed.
 
 ---
 

package/templates/SCHEMA.md

@@ -2,7 +2,7 @@
 title: Wiki Schema
 type: schema
 updated: YYYY-MM-DD
-version: 2.0
+version: 2.1
 ---
 
 # Wiki Schema
@@ -118,11 +118,35 @@ memory_summary: <one line for the MEMORY.md index>
 reason: <why this rule is needed>
 source: session:YYYY-MM-DD | commit:<hash> | pr:<n> | https://...
 
+# optional (2.1) — failure taxonomy for incident-driven corrections:
+failure_type: <enum>                     # one of the 8 values below; OMIT for pure
+                                         # preferences / new conventions ("always do X")
+
 # conditional — required when `targets` includes `claude-learned`:
 global_summary: <one line for the <learned_behaviors> entry>
 promote_to_global: true                  # explicit opt-in to global projection
 ```
 
+`failure_type` (optional, added 2.1) classifies feedback that came from a real
+failure incident so recurring mistake types become machine-aggregatable
+(surfaced by `hypomnema stats`). Leave it off when the page records a preference
+rather than a failure. The eight values, in classification-precedence order
+(most specific first — a failure matching several takes the earliest):
+
+| value | when |
+|-------|------|
+| `hallucination` | fabricated a fact / API / path |
+| `false-completion` | declared "done" without running the required gate or test |
+| `process-stall` | stopped instead of asking / continuing when it should have |
+| `over-caution` | re-asked or re-gated despite standing authority |
+| `overreach` | acted beyond the requested scope |
+| `incompleteness` | started correctly but omitted a required step or scope |
+| `instruction-miss` | ignored an explicit this-session instruction |
+| `convention-violation` | broke a standing documented convention (not restated) |
+
+`lint` rejects any value outside this set; an omitted `failure_type` is always
+allowed.
+
 Edit the feedback page only — never hand-edit the generated
 `<!-- HYPO:FEEDBACK-SYNC:START … -->` managed blocks (sync detects tampering as a conflict).
 

package/templates/hypo-config.md

@@ -1,7 +1,7 @@
 ---
 title: Hypomnema Config
 type: config
-version: "1.3.4"
+version: "1.4.1"
 created: YYYY-MM-DD
 ---
 

package/scripts/bump-version.mjs

@@ -1,63 +0,0 @@
-#!/usr/bin/env node
-// Sync version across package.json, .claude-plugin/plugin.json, and templates/hypo-config.md.
-// Usage: node scripts/bump-version.mjs <new-version>
-
-import { readFileSync, writeFileSync } from 'node:fs';
-import { resolve, dirname } from 'node:path';
-import { fileURLToPath } from 'node:url';
-
-const root = resolve(dirname(fileURLToPath(import.meta.url)), '..');
-const next = process.argv[2];
-
-if (!next || !/^\d+\.\d+\.\d+(-[\w.]+)?$/.test(next)) {
-  console.error('Usage: node scripts/bump-version.mjs <semver>');
-  console.error('Example: node scripts/bump-version.mjs 1.0.0');
-  process.exit(1);
-}
-
-const targets = [
-  {
-    path: 'package.json',
-    pattern: /("version"\s*:\s*")([^"]+)(")/,
-  },
-  {
-    path: '.claude-plugin/plugin.json',
-    pattern: /("version"\s*:\s*")([^"]+)(")/,
-  },
-  {
-    path: '.claude-plugin/marketplace.json',
-    pattern: /("version"\s*:\s*")([^"]+)(")/,
-  },
-  {
-    path: 'templates/hypo-config.md',
-    pattern: /(^version:\s*")([^"]+)(")/m,
-  },
-];
-
-for (const { path, pattern } of targets) {
-  const abs = resolve(root, path);
-  const before = readFileSync(abs, 'utf-8');
-  const match = before.match(pattern);
-  if (!match) {
-    console.error(`✗ ${path}: version pattern not found`);
-    process.exit(1);
-  }
-  const current = match[2];
-  if (current === next) {
-    console.log(`= ${path} already ${next}`);
-    continue;
-  }
-  writeFileSync(abs, before.replace(pattern, `$1${next}$3`));
-  console.log(`✓ ${path}: ${current} → ${next}`);
-}
-
-console.log(`\nNext steps:`);
-console.log(`  1. Edit CHANGELOG.md — ensure the "## [${next}]" section has a`);
-console.log(`     "### 한글 요약" sub-section (release.yml check-bilingual gate).`);
-console.log(`  2. node scripts/check-bilingual.mjs --changelog   # local pre-check`);
-console.log(`  3. git add -A && git commit -m "chore(release): v${next}"`);
-console.log(`  4. Create an ANNOTATED tag (lightweight tags are rejected by CI):`);
-console.log(`       git tag -a v${next} -m "<English body>\\n\\n---\\n\\n<한글 요약>"`);
-console.log(`     See docs/CONTRIBUTING.md "Cutting a release" for the full template.`);
-console.log(`  5. node scripts/check-bilingual.mjs --tag v${next}   # local pre-check`);
-console.log(`  6. git push origin <branch> --tags`);