@deneb-ui/cli 2.0.103 → 2.0.105

package/bin/index.js

@@ -929,7 +929,13 @@ async function initProject(targetInput, options = {}) {
       conversionRes = pipelineRes.result || pipelineRes;
       if (pipelineRes.rolledBack) {
         console.log('\n\x1b[31m✖ DENEB CONVERSION BLOCKED — Original project files preserved intact.\x1b[0m');
-        console.log(`\x1b[33m  Reason:\x1b[0m ${pipelineRes.reasons?.[0] || '100% Fivora Contract / Acceptance Gate validation failed'}`);
+        const primaryReason = pipelineRes.reasons?.[0] || pipelineRes.error || '100% Fivora Contract / Acceptance Gate validation failed';
+        console.log(`\x1b[33m  Reason:\x1b[0m ${primaryReason}`);
+        if (pipelineRes.reasons && pipelineRes.reasons.length > 1) {
+          for (let i = 1; i < pipelineRes.reasons.length; i++) {
+            console.log(`\x1b[90m    - ${pipelineRes.reasons[i]}\x1b[0m`);
+          }
+        }
         console.log(`\n  Run \x1b[36m\x1b[1mnpx @deneb-ui/cli explain --blocked\x1b[0m for exact file locations, line numbers, and fixes.\n`);
         process.exit(1);
       }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@deneb-ui/cli",
-  "version": "2.0.103",
+  "version": "2.0.105",
   "description": "Official DENEB CLI — scaffold, convert, validate, and package Fivora-ready storefront templates.",
   "bin": {
     "deneb": "bin/index.js",
@@ -49,7 +49,7 @@
   "license": "MIT",
   "dependencies": {
     "@babel/parser": "^7.28.0",
-    "@deneb-ui/core": "^2.0.103",
+    "@deneb-ui/core": "^2.0.105",
     "@octokit/rest": "^22.0.1",
     "adm-zip": "^0.6.0",
     "dotenv": "^17.4.2",

package/src/arc/acceptance-gates.cjs

@@ -83,7 +83,7 @@ function evaluateAcceptanceGates(metrics = {}) {
     },
     designPreservation: {
       name: ACCEPTANCE_GATES.DESIGN_PRESERVATION,
-      passed: Boolean((metrics.designPreservationScore ?? 100) >= 95),
+      passed: Boolean((metrics.designPreservationScore ?? 100) >= 90),
       critical: false,
     },
     controlOnlyProtection: {

package/src/arc/fivora-contract.cjs

@@ -615,16 +615,26 @@ function findUncoveredVisibleTextRegex(code, filePath) {
 function auditEmptyStateSource(code, filePath) {
   const errors = [];
   const source = String(code);
+  const isInteractiveOrPageToggle = (condition) => {
+    return /(?:open|active|selected|modal|dialog|lightbox|drawer|popup|sheet|portal|loading|pending|tab|filter|step|enabled|ispageenabled|haspage|pagestatus|pageenabled)/i.test(
+      String(condition || '')
+    );
+  };
+
   const andPattern =
     /\{([^{}]{0,120}?)\s*&&\s*\(?\s*<\s*([a-zA-Z][\w.:-]*)([^>]*data-preview-(?:field-path|list-path|item-path)[^>]*)>/g;
   for (const match of source.matchAll(andPattern)) {
+    const condition = String(match[1]).trim();
+    if (isInteractiveOrPageToggle(condition)) continue;
     errors.push(
-      `${filePath}:${lineNumberAt(source, match.index ?? 0)} editable marker is gated behind "${String(match[1]).trim()} &&". Keep the target mounted when its value is empty, false, or zero.`
+      `${filePath}:${lineNumberAt(source, match.index ?? 0)} editable marker is gated behind "${condition} &&". Keep the target mounted when its value is empty, false, or zero.`
     );
   }
   const ternaryPattern =
     /\{([^{}]{0,80}?)\s*\?\s*<\s*([a-zA-Z][\w.:-]*)([^>]*data-preview-(?:field-path|list-path|item-path)[^>]*)>[\s\S]{0,200}?\?\s*(?:null|false|undefined)/g;
   for (const match of source.matchAll(ternaryPattern)) {
+    const condition = String(match[1]).trim();
+    if (isInteractiveOrPageToggle(condition)) continue;
     errors.push(
       `${filePath}:${lineNumberAt(source, match.index ?? 0)} editable marker unmounts on a falsy ternary. Keep the target mounted when its value is empty.`
     );

package/src/arc/index.cjs

@@ -21,7 +21,8 @@ const { planTransformations } = require('./planner.cjs');
 const { applyFilePlan, instrumentLayoutSource, instrumentPageKey, resolveSiteDataSpecifier, resolveSiteDataRuntimeSpecifier, rewriteRecursiveSiteDataContext, ensureJsonModule, sanitizeContradictoryMarkersInSource } = require('./transformer.cjs');
 const { applyResidualPass } = require('./residual.cjs');
 const { parseSource } = require('./ast.cjs');
-const { buildSiteDataAndManifest, writeDataBank, loadExistingData, countSchemaFields } = require('./manifest.cjs');
+const { buildSiteDataAndManifest, writeDataBank, loadExistingData, countSchemaFields, upsertSchemaField, getDeep } = require('./manifest.cjs');
+const { classifyFieldType, humanLabel } = require('./field-paths.cjs');
 const { collectFontIdsFromSiteData } = require('./font-plan.cjs');
 const { validateAstFiles, validateContracts, designPreservationScore, coverageMetrics } = require('./validator.cjs');
 const { recordExperience, registryArchitecture } = require('./learning.cjs');
@@ -751,7 +752,7 @@ function runArcTransformations(projectDir, projectName, opts, profile, graph, an
       .map((f) => ({ abs: path.join(projectDir, f), rel: f }))
   );
   const syntaxPassed = astResults.every((r) => r.passed);
-  const contracts = validateContracts(projectDir, dataBundle.siteData, dataBundle.manifest);
+  let contracts = validateContracts(projectDir, dataBundle.siteData, dataBundle.manifest);
   const design = designPreservationScore(plan.files, afterFiles);
   const alreadyEditable = analyses.filter((a) => a.alreadyEditable).length;
   const skippedDynamic = plan.skipped.filter((s) => /dynamic|api/.test(s.reason || '')).length;
@@ -786,6 +787,26 @@ function runArcTransformations(projectDir, projectName, opts, profile, graph, an
         manifest: dataBundle.manifest,
         inventory: collectMarkerInventory(projectDir, profile, graph),
       });
+      contracts = validateContracts(projectDir, dataBundle.siteData, dataBundle.manifest);
+    }
+  }
+
+  // Auto-reconciliation: If contracts flagged missingSchema, register them in manifest.editorSchema
+  if (contracts.missingSchema.length > 0) {
+    let schemaModified = false;
+    dataBundle.manifest.editorSchema = dataBundle.manifest.editorSchema || { version: 1, sections: [] };
+    dataBundle.manifest.editorSchema.sections = dataBundle.manifest.editorSchema.sections || [];
+    for (const item of contracts.missingSchema) {
+      const fp = item.fieldPath;
+      const leaf = fp.split('.').pop();
+      const val = getDeep(dataBundle.siteData.content, fp);
+      const type = classifyFieldType('text', val, leaf);
+      upsertSchemaField(dataBundle.manifest.editorSchema.sections, fp, type, humanLabel(leaf));
+      schemaModified = true;
+    }
+    if (schemaModified) {
+      writeDataBank(projectDir, dataBundle.siteData, dataBundle.manifest);
+      contracts = validateContracts(projectDir, dataBundle.siteData, dataBundle.manifest);
     }
   }
 
@@ -882,7 +903,7 @@ function runArcTransformations(projectDir, projectName, opts, profile, graph, an
 
   const criticalFailure = !syntaxPassed || (contracts.actionCollisions > 0 && appliedCount === 0) || (buildResult.passed === false);
   const contractFailure = !fivoraAudit.passed || fivoraAudit.uncoveredVisibleText.length > 0;
-  const designFailure = design.score < 95 && design.total > 0;
+  const designFailure = design.score < 90 && design.total > 0;
   const assetFailure = !assetAudit.passed && assetAudit.errors.length > 0;
   let outcome = 'success';
   if (criticalFailure) {
@@ -909,7 +930,7 @@ function runArcTransformations(projectDir, projectName, opts, profile, graph, an
     const reasons = [];
     if (!fivoraAudit.passed) reasons.push(`${fivoraAudit.errors.length} Fivora contract error(s)`);
     if (fivoraAudit.uncoveredVisibleText.length > 0) reasons.push(`${fivoraAudit.uncoveredVisibleText.length} uncovered visible text node(s)`);
-    if (designFailure) reasons.push(`Design preservation score below threshold (${design.score} < 98)`);
+    if (designFailure) reasons.push(`Design preservation score below threshold (${design.score} < 90)`);
     if (assetFailure) reasons.push(`Asset integrity failed (${assetAudit.errors.length} missing asset(s))`);
     if (!acceptance.allCriticalPassed) reasons.push(`Critical acceptance gates failed: ${acceptance.status}`);
     printer.printRollback(`Strict Fivora contract failed: ${reasons.join(', ')}`);
@@ -1180,7 +1201,9 @@ function runTransactionalPipeline(targetDirInput = '.', options = {}) {
       const syntaxOk = result.validation?.syntaxPassed !== false;
       const assetOk = result.validation?.assetIntegrity?.passed !== false;
       const gatesOk = result.validation?.acceptance ? result.validation.acceptance.passed : true;
-      const isSuccess = result.outcome === 'success' && (!opts.strict || (contractOk && syntaxOk && assetOk && gatesOk));
+      const isSuccess =
+        result.outcome !== 'rolled-back' &&
+        (!opts.strict || (result.outcome === 'success' && contractOk && syntaxOk && assetOk && gatesOk));
 
       if (isSuccess) {
         // Collect all files to commit from workspace to project
@@ -1219,7 +1242,7 @@ function runTransactionalPipeline(targetDirInput = '.', options = {}) {
         if (result.validation?.fivoraContractPassed === false) reasons.push('Fivora strict contract validation failed');
         if (result.validation?.uncoveredVisibleText > 0) reasons.push(`${result.validation.uncoveredVisibleText} uncovered visible text node(s)`);
         if (result.validation?.syntaxPassed === false) reasons.push('AST syntax validation failed');
-        if (result.validation?.designPreservation < 95 && result.validation?.designPreservation > 0) reasons.push(`Design preservation (${result.validation.designPreservation}%) below threshold`);
+        if (result.validation?.designPreservation < 90 && result.validation?.designPreservation > 0) reasons.push(`Design preservation (${result.validation.designPreservation}%) below threshold`);
         if (result.validation?.assetIntegrity?.passed === false) reasons.push(`Asset integrity failed: ${result.validation.assetIntegrity.errors?.[0] || 'missing asset'}`);
         if (result.validation?.acceptance && !result.validation.acceptance.passed) reasons.push(`15-Gate Acceptance Matrix failed: ${result.validation.acceptance.status}`);
 
@@ -1251,6 +1274,7 @@ function runTransactionalPipeline(targetDirInput = '.', options = {}) {
         rolledBack: true,
         outcome: 'rolled-back',
         error: err.message,
+        reasons: [err.message],
       };
     }
   }

package/src/arc/manifest.cjs

@@ -180,7 +180,31 @@ function upsertSchemaList(sections, listPath, itemFields, items) {
   }
 
   const key = rest[rest.length - 1];
-  if (fields.some((f) => f.key === key)) return;
+  const existingList = fields.find((f) => f.key === key);
+  if (existingList) {
+    existingList.fields = existingList.fields || [];
+    for (const field of itemFields || []) {
+      if (!existingList.fields.some((ef) => ef.key === field.key)) {
+        existingList.fields.push({
+          key: field.key,
+          type:
+            field.type === 'url'
+              ? 'url'
+              : field.type === 'image'
+                ? 'image'
+                : field.type === 'textarea'
+                  ? 'textarea'
+                  : field.type === 'tel' || field.type === 'phone'
+                    ? 'tel'
+                    : field.type === 'email'
+                      ? 'email'
+                      : field.type || 'text',
+          label: humanLabel(field.key),
+        });
+      }
+    }
+    return;
+  }
 
   const isPrimitiveList = (itemFields || []).length === 0;
   fields.push({
@@ -255,6 +279,9 @@ function enrichSchemasFromContent(content, sections) {
         if (Object.prototype.hasOwnProperty.call(node, labelKey)) {
           upsertSchemaField(sections, `${prefix}.${labelKey}`, 'text', humanLabel(labelKey));
         }
+      } else if (typeof value === 'string' || typeof value === 'number' || typeof value === 'boolean') {
+        const fieldType = classifyFieldType('text', value, key);
+        upsertSchemaField(sections, nextPath, fieldType, humanLabel(key));
       } else if (typeof value === 'object' && value !== null && !Array.isArray(value)) {
         walk(value, nextPath);
       }
@@ -627,6 +654,35 @@ function buildSiteDataAndManifest({
     }
   }
 
+  // Ensure all bound field paths (e.g., from existing template code) are registered in editorSchema and content
+  for (const bPath of boundFieldPaths || []) {
+    if (!bPath || typeof bPath !== 'string') continue;
+    const listMatch = bPath.match(/^(.*?)\[(?:\d+|\*|[a-zA-Z0-9_]+)\]\.(.*)$/);
+    if (listMatch) {
+      const listPath = listMatch[1];
+      const itemKey = listMatch[2];
+      const listVal = getDeep(content, listPath) || [];
+      const sample = Array.isArray(listVal) && listVal[0] ? listVal[0] : {};
+      const itemVal = getDeep(sample, itemKey);
+      const itemType = classifyFieldType('text', itemVal, itemKey);
+      upsertSchemaList(editorSections, listPath, [{ key: itemKey, type: itemType }], listVal);
+      continue;
+    }
+    if (!isBound(bPath) && !isAllowedControlOnly(bPath)) continue;
+    let val = getDeep(content, bPath);
+    if (val === undefined && existingSiteData?.content) {
+      val = getDeep(existingSiteData.content, bPath);
+      if (val !== undefined) setDeep(content, bPath, val);
+    }
+    if (val === undefined) {
+      setDeep(content, bPath, '');
+      val = '';
+    }
+    const leafKey = bPath.split('.').pop();
+    const type = classifyFieldType('text', val, leafKey);
+    upsertSchemaField(editorSections, bPath, type, humanLabel(leafKey));
+  }
+
   pruneUnboundLeaves(content, isBound);
 
   const siteData = {
@@ -747,6 +803,7 @@ module.exports = {
   collectFieldsFromPlan,
   computeControlOnlyPaths,
   assignSectionPageKeys,
+  upsertSchemaField,
   upsertSchemaList,
   enrichSchemasFromContent,
   isListActionCtaKey,

package/src/arc/pr-agent.cjs

@@ -32,6 +32,11 @@ function getGithubConfig() {
  * @returns {{ ready: boolean, reason?: string }}
  */
 function checkGithubReady() {
+  // Automated branch/PR creation is permanently disabled by default.
+  // Set DENEB_ENABLE_PR=true only if you explicitly want to create GitHub branches.
+  if (process.env.DENEB_ENABLE_PR !== 'true') {
+    return { ready: false, reason: 'Automated GitHub branch creation is disabled.' };
+  }
   loadEnv();
   const config = getGithubConfig();
   if (!config.pat) {
@@ -204,10 +209,10 @@ async function createComponentPR(params) {
   let corePr = null;
   let uiPr = null;
 
-  if (dryRun) {
+  if (dryRun || process.env.DENEB_ENABLE_PR !== 'true') {
     return {
-      corePr: { number: 0, url: '(dry-run — PR not created)' },
-      uiPr: docsCode ? { number: 0, url: '(dry-run — PR not created)' } : null,
+      corePr: { number: 0, url: '(branch/PR creation disabled)' },
+      uiPr: docsCode ? { number: 0, url: '(branch/PR creation disabled)' } : null,
       errors: [],
     };
   }

package/src/arc/semantic.cjs

@@ -595,6 +595,21 @@ function analyzeFile(optionsOrFile, codeArg, profileArg) {
         return;
       }
 
+      if (hasStaticMarker(node)) {
+        candidates.push({
+          loc,
+          tag: name,
+          kind: 'static',
+          confidence: 1,
+          skip: true,
+          reason: 'already-static-marker',
+          file: relativeFile,
+          ownerScope,
+        });
+        this.traverse(pathNode);
+        return;
+      }
+
       if (isElementOrParentHidden(pathNode)) {
         candidates.push({
           loc,

package/src/arc/transforms/healing/conditionals.cjs

@@ -31,47 +31,23 @@ function isUnsafeToStripCondition(leftExpr, rightExpr, pathNode) {
 
   if (leftIdentifiers.size === 0) return true;
 
-  // 2. State & modal variable keywords (e.g., selectedDish, lightboxIndex, isOpen, activeTab)
-  const stateKeywordRegex = /(?:open|active|selected|current|expanded|visible|show|modal|dialog|lightbox|drawer|loading|tab|filter|step|index)/i;
+  // 2. Interactive state & modal variable keywords (e.g. lightboxIndex, selectedDish, isOpen, activeTab, isModalOpen)
+  const interactiveStateKeywordRegex = /(?:modal|dialog|lightbox|drawer|popup|sheet|portal|isopen|activetab|selecteditem|selecteddish|lightboxindex|expanded|loading|pending)/i;
   for (const id of leftIdentifiers) {
-    if (stateKeywordRegex.test(id)) {
-      return true; // NEVER strip modal/state/selection guards
+    if (interactiveStateKeywordRegex.test(id)) {
+      return true; // NEVER strip modal/interactive state guards
     }
   }
 
-  // 3. Null / undefined / equality / comparison checks
-  let hasComparisonOrNullCheck = false;
-  recast.types.visit(leftExpr, {
-    visitBinaryExpression(bPath) {
-      const op = bPath.node.operator;
-      if (['!==', '!=', '===', '==', '>', '<', '>=', '<='].includes(op)) {
-        hasComparisonOrNullCheck = true;
-      }
-      this.traverse(bPath);
-    },
-    visitUnaryExpression(uPath) {
-      if (uPath.node.operator === '!') {
-        hasComparisonOrNullCheck = true;
-      }
-      this.traverse(uPath);
-    },
-  });
-  if (hasComparisonOrNullCheck) return true;
-
-  // 4. Check if ANY identifier in leftExpr is referenced inside rightExpr
-  let rightReferencesLeftId = false;
-  recast.types.visit(rightExpr, {
-    visitIdentifier(idPath) {
-      if (leftIdentifiers.has(idPath.node.name)) {
-        rightReferencesLeftId = true;
-        return false;
-      }
-      this.traverse(idPath);
-    },
-  });
-  if (rightReferencesLeftId) return true;
+  // 3. Page & feature toggles (e.g. inventoryEnabled, contactEnabled, isPageEnabled)
+  const pageToggleRegex = /(?:enabled|ispageenabled|haspage|pagestatus|pageenabled)/i;
+  for (const id of leftIdentifiers) {
+    if (pageToggleRegex.test(id)) {
+      return true; // NEVER strip page/route enablement guards
+    }
+  }
 
-  // 5. Check if rightExpr or any of its ancestors is an interactive modal / dialog / overlay / AnimatePresence
+  // 4. Check if rightExpr or any of its ancestors is an interactive modal / dialog / overlay / AnimatePresence
   let curr = pathNode;
   while (curr && curr.parent) {
     const parentNode = curr.parent.node;
@@ -111,11 +87,6 @@ function isUnsafeToStripCondition(leftExpr, rightExpr, pathNode) {
   });
   if (isModalOrOverlay) return true;
 
-  // 6. Only allow stripping if leftExpr is purely a direct siteData content member chain
-  const leftCode = recast.print(leftExpr).code;
-  const isDirectContentCheck = /siteData(?:\?\.|\.)content(?:\?\.|\.)/i.test(leftCode);
-  if (!isDirectContentCheck) return true;
-
   return false;
 }
 
@@ -141,6 +112,15 @@ function healEmptyStateConditionals(ast) {
       ) {
         return true;
       }
+      const attrs = node.openingElement?.attributes || [];
+      for (const a of attrs) {
+        if (a.type === 'JSXSpreadAttribute') {
+          const code = recast.print(a).code;
+          if (/data-preview-(?:field-path|list-path|item-path)/.test(code)) {
+            return true;
+          }
+        }
+      }
       return (node.children || []).some((child) => jsxHasPreviewMarker(child));
     }
     if (node.type === 'JSXFragment') {

package/src/arc/validator.cjs

@@ -48,7 +48,7 @@ function validateContracts(projectDir, siteData, manifest) {
       if (siteData?.content && getDeep(siteData.content, fieldPath) === undefined) {
         orphans.push({ fieldPath, file: relative });
       }
-      if (manifest && !isFieldInEditorSchema(manifest, fieldPath)) {
+      if (manifest && !isFieldInEditorSchema(manifest, fieldPath) && !isControlOnly(fieldPath, manifest.visualEditing?.controlOnlyPaths)) {
         missingSchema.push({ fieldPath, file: relative });
       }
     }
@@ -82,24 +82,49 @@ function validateContracts(projectDir, siteData, manifest) {
 }
 
 function isFieldInEditorSchema(manifest, fieldPath) {
-  const parts = String(fieldPath).split('.');
-  const sectionId = parts[0];
-  const fieldKey = parts.slice(1).join('.');
-  const section = (manifest.editorSchema?.sections || []).find((s) => s.id === sectionId || s.path === sectionId);
+  if (!manifest?.editorSchema?.sections) return false;
+  const cleanPath = String(fieldPath)
+    .replace(/\[(?:\d+|\*|[a-zA-Z0-9_]+)\]/g, '.[*]')
+    .replace(/\.\.+/g, '.');
+  const parts = cleanPath.split('.').filter(Boolean);
+  const rawSectionId = parts[0];
+  const sectionId = rawSectionId ? rawSectionId.replace(/\[\*\]/g, '') : '';
+
+  const section = (manifest.editorSchema?.sections || []).find(
+    (s) => s.id === sectionId || s.path === sectionId || s.id === rawSectionId
+  );
   if (!section) return false;
+  const fieldParts = parts.slice(1);
+  if (fieldParts.length === 0) return true;
+
+  function check(fields, remainingParts) {
+    if (!Array.isArray(fields) || remainingParts.length === 0) return false;
+    const current = remainingParts[0];
+    const rest = remainingParts.slice(1);
+
+    if (current === '[*]') {
+      if (rest.length === 0) return true;
+      return check(fields, rest);
+    }
 
-  function checkFields(fields, targetKey) {
-    if (!Array.isArray(fields)) return false;
     for (const f of fields) {
-      if (f.key === targetKey) return true;
-      if (f.fields && targetKey.startsWith(f.key + '.')) {
-        const subKey = targetKey.substring(f.key.length + 1);
-        if (checkFields(f.fields, subKey)) return true;
+      if (f.key === current) {
+        if (rest.length === 0) return true;
+        const subFields = f.fields || f.itemFields;
+        if (subFields) {
+          if (rest[0] === '[*]') {
+            if (rest.length === 1) return true;
+            if (check(subFields, rest.slice(1))) return true;
+          }
+          if (check(subFields, rest)) return true;
+        }
       }
+      if (f.key === remainingParts.join('.')) return true;
     }
     return false;
   }
-  return checkFields(section.fields, fieldKey);
+
+  return check(section.fields || section.itemFields, fieldParts);
 }
 
 function designPreservationScore(filePlans, afterFiles) {