infernoflow 0.32.6 → 0.32.8

package/dist/lib/commands/check.mjs

@@ -102,7 +102,9 @@ export async function checkCommand(args) {
         console.log(JSON.stringify({ ok: false, errors, warnings }, null, 2));
         process.exit(1);
       }
-      const registryIds = new Set((registry.capabilities || []).map(c => c?.id).filter(Boolean));
+      // Support both bare array format (written by scan) and { capabilities: [...] } object format
+      const regArray = Array.isArray(registry) ? registry : (registry.capabilities || []);
+      const registryIds = new Set(regArray.map(c => c?.id).filter(Boolean));
 
       const missingInRegistry = caps.filter(c => !registryIds.has(c));
       if (missingInRegistry.length > 0) {
@@ -127,9 +129,12 @@ export async function checkCommand(args) {
         if (!jsonOut) ok(`${scenarioFiles.length} scenario file(s) found`);
 
         if (uncovered.length > 0 && requireCoverage) {
+          // Scenario coverage is advisory — missing coverage is a warning, not a hard error.
+          // This allows `infernoflow_apply` to add new capabilities without immediately
+          // requiring scenarios (which can't exist for brand-new capabilities yet).
           uncovered.forEach(c => {
-            if (!jsonOut) fail(`"${c}" has no scenario coverage`, `Add to capabilitiesCovered in a scenario file`);
-            errors.push(`"${c}" uncovered`);
+            if (!jsonOut) warn(`"${c}" has no scenario coverage`, `Add to capabilitiesCovered in a scenario file`);
+            warnings.push(`"${c}" uncovered`);
           });
         } else if (!jsonOut) {
           ok(`All capabilities covered by scenarios`);

package/dist/lib/commands/explain.mjs

@@ -343,7 +343,7 @@ export async function explainCommand(rawArgs) {
 
     if (!jsonMode) {
       console.log(gray(`\n  infernoflow explain  →  ${bold(arg)}`));
-      console.log(gray(`  Found ${capIdsToExplain.length} mapped capability${capIdsToExplain.length > 1 ? "ies" : "y"}: ${capIdsToExplain.join(", ")}`));
+      console.log(gray(`  Found ${capIdsToExplain.length} mapped ${capIdsToExplain.length > 1 ? "capabilities" : "capability"}: ${capIdsToExplain.join(", ")}`));
       console.log();
     }
   } else {

package/dist/lib/commands/graph.mjs

@@ -48,7 +48,8 @@ function buildGraph(scanCaps, allCaps) {
   const reverse = {};
 
   // Build function → capId index
-  const funcIndex = {}; // functionName → capId
+  // Use Map (not plain object) to avoid collisions with inherited properties like toString, constructor, etc.
+  const funcIndex = new Map(); // functionName → capId
   for (const entry of scanCaps) {
     const capFull = allCaps.find(c => c.id === entry.id) || {};
     nodes[entry.id] = {
@@ -66,8 +67,8 @@ function buildGraph(scanCaps, allCaps) {
 
     for (const fn of (entry.codeAnalysis?.functions || [])) {
       const bare = fn.replace(/\(\)$/, "");
-      funcIndex[bare] = entry.id;
-      funcIndex[bare.toLowerCase()] = entry.id;
+      funcIndex.set(bare, entry.id);
+      funcIndex.set(bare.toLowerCase(), entry.id);
     }
   }
 
@@ -75,7 +76,7 @@ function buildGraph(scanCaps, allCaps) {
   for (const [capId, node] of Object.entries(nodes)) {
     for (const call of node.calls) {
       const bare   = call.replace(/\(\)$/, "");
-      const target = funcIndex[bare] || funcIndex[bare.toLowerCase()];
+      const target = funcIndex.get(bare) || funcIndex.get(bare.toLowerCase());
       if (target && target !== capId) {
         edges[capId].add(target);
         reverse[target].add(capId);

package/dist/lib/commands/scan.mjs

@@ -462,12 +462,18 @@ export async function scanCommand(rawArgs) {
     process.exit(1);
   }
   let capabilities;
+  let capsFileIsObject = false; // track original format so we can write it back correctly
+  let capsFileWrapper = null;
   try { capabilities = JSON.parse(fs.readFileSync(capsPath, "utf8")); }
   catch (e) { console.error(red("✗ Failed to parse capabilities.json: " + e.message)); process.exit(1); }
 
   if (!Array.isArray(capabilities)) {
     // handle object format { capabilities: [...] }
-    if (capabilities.capabilities) capabilities = capabilities.capabilities;
+    if (capabilities.capabilities) {
+      capsFileIsObject = true;
+      capsFileWrapper = capabilities; // preserve other top-level keys
+      capabilities = capabilities.capabilities;
+    }
     else { console.error(red("✗ Unexpected capabilities.json format.")); process.exit(1); }
   }
 
@@ -572,7 +578,11 @@ export async function scanCommand(rawArgs) {
   });
 
   if (changed > 0) {
-    fs.writeFileSync(capsPath, JSON.stringify(updatedCaps, null, 2));
+    // Preserve the original file format: object wrapper or bare array
+    const toWrite = capsFileIsObject
+      ? { ...capsFileWrapper, capabilities: updatedCaps }
+      : updatedCaps;
+    fs.writeFileSync(capsPath, JSON.stringify(toWrite, null, 2));
     console.log(gray(`  Updated ${changed} capability entries in capabilities.json`));
   }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "infernoflow",
-  "version": "0.32.6",
+  "version": "0.32.8",
   "description": "The forge for liquid code - keep capabilities, contracts, and docs in sync.",
   "type": "module",
   "bin": {