chati-dev 4.2.1 → 4.3.0

package/src/memory/dream.js

@@ -19,6 +19,35 @@ const MAX_ENTRIES = 200;
 const MEMORY_BASE = '.chati/memories';
 const ARCHIVE_DIR = '.chati/memories/shared/archive';
 
+/**
+ * Normalize confidence to a number in [0, 1]. agent-memory.js emits either
+ * numeric (0.0-1.0) OR string tokens ('high'|'medium'|'low'). Article XIII
+ * promotion rule requires confidence > 0.9 — without coercion here the
+ * Math.max(string, string) path produces NaN and the rule never fires.
+ *
+ * F-A5-010 fix.
+ *
+ * @param {number|string|null|undefined} raw
+ * @returns {number}
+ */
+export function normalizeConfidence(raw) {
+  if (typeof raw === 'number' && Number.isFinite(raw)) {
+    return Math.min(1, Math.max(0, raw));
+  }
+  if (typeof raw === 'string') {
+    switch (raw.toLowerCase().trim()) {
+      case 'high':   return 0.9;
+      case 'medium': return 0.7;
+      case 'low':    return 0.5;
+      default: {
+        const parsed = parseFloat(raw);
+        return Number.isFinite(parsed) ? Math.min(1, Math.max(0, parsed)) : 0;
+      }
+    }
+  }
+  return 0;
+}
+
 /**
  * Run the complete 4-phase dream consolidation.
  * @param {string} projectDir - Project root directory
@@ -149,11 +178,13 @@ function consolidate(allEntries) {
   let contradictions = 0;
 
   for (const entry of allEntries) {
+    // Coerce confidence to numeric BEFORE merge (F-A5-010).
+    entry.confidence = normalizeConfidence(entry.confidence);
     const existing = hashMap.get(entry._hash);
     if (existing) {
       // Merge: keep highest confidence, combine tags, increment evidence
       mergedCount++;
-      existing.confidence = Math.max(existing.confidence || 0, entry.confidence || 0);
+      existing.confidence = Math.max(normalizeConfidence(existing.confidence), entry.confidence);
       existing.evidence_count = (existing.evidence_count || 1) + (entry.evidence_count || 1);
 
       // Merge tags

package/src/merger/replace-merger.js

@@ -1,22 +1,35 @@
+/**
+ * Framework-owned subdirectories. These are matched against relative paths
+ * from the project root. Both 'chati.dev/' and '.chati.dev/' prefixes are
+ * matched so the merger works with both legacy and new project structures.
+ */
+const FRAMEWORK_OWNED_SUBDIRS = [
+  'agents/',
+  'schemas/',
+  'orchestrator/',
+  'workflows/',
+  'quality-gates/',
+  'templates/',
+  'frameworks/',
+  'patterns/',
+  'i18n/',
+  'migrations/',
+  'hooks/',
+  'domains/',
+  'intelligence/',
+];
+
+const FRAMEWORK_OWNED_FILES = [
+  'constitution.md',
+];
+
 /**
  * Framework-owned file paths that should always be fully replaced.
- * These are paths relative to the project root.
+ * These are paths relative to the project root. Matches both chati.dev/ and .chati.dev/ prefixes.
  */
 const FRAMEWORK_OWNED_PATTERNS = [
-  /^chati\.dev\/agents\//,
-  /^chati\.dev\/schemas\//,
-  /^chati\.dev\/constitution\.md$/,
-  /^chati\.dev\/orchestrator\//,
-  /^chati\.dev\/workflows\//,
-  /^chati\.dev\/quality-gates\//,
-  /^chati\.dev\/templates\//,
-  /^chati\.dev\/frameworks\//,
-  /^chati\.dev\/patterns\//,
-  /^chati\.dev\/i18n\//,
-  /^chati\.dev\/migrations\//,
-  /^chati\.dev\/hooks\//,
-  /^chati\.dev\/domains\//,
-  /^chati\.dev\/intelligence\//,
+  ...FRAMEWORK_OWNED_SUBDIRS.map(d => new RegExp(`^(?:\\.chati\\.dev|chati\\.dev)\\/${d.replace('/', '\\/')}`)),
+  ...FRAMEWORK_OWNED_FILES.map(f => new RegExp(`^(?:\\.chati\\.dev|chati\\.dev)\\/${f}$`)),
 ];
 
 /**

package/src/orchestrator/agent-selector.js

@@ -15,11 +15,12 @@ export const AGENT_PIPELINE = [
   { name: 'detail', phase: 'plan', group: 'plan-parallel', parallel: true },
   { name: 'architect', phase: 'plan', group: 'plan-parallel', parallel: true },
   { name: 'ux', phase: 'plan', group: 'plan-parallel', parallel: true },
+  { name: 'qa-planning', phase: 'plan', group: 'quality-plan', parallel: false },
   { name: 'phases', phase: 'plan', group: 'plan', parallel: false },
   { name: 'tasks', phase: 'plan', group: 'plan', parallel: false },
-  { name: 'qa-planning', phase: 'plan', group: 'quality', parallel: false },
   { name: 'dev', phase: 'build', group: 'build', parallel: false },
   { name: 'qa-implementation', phase: 'build', group: 'quality', parallel: false },
+  { name: 'qa-visual', phase: 'build', group: 'quality', parallel: false },
   { name: 'devops', phase: 'deploy', group: 'deploy', parallel: false },
 ];