hypomnema 1.3.4 → 1.4.0

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/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.0"
 created: YYYY-MM-DD
 ---