@deneb-ui/cli 2.0.83 → 2.0.84

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@deneb-ui/cli",
-  "version": "2.0.83",
+  "version": "2.0.84",
   "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.83",
+    "@deneb-ui/core": "^2.0.84",
     "@octokit/rest": "^22.0.1",
     "adm-zip": "^0.6.0",
     "dotenv": "^17.4.2",

package/src/arc/index.cjs

@@ -613,6 +613,23 @@ function runArcTransformations(projectDir, projectName, opts, profile, graph, an
     }
   }
 
+  // Mandatory Post-Pass: Guarantee all files referencing siteData have useSiteData hook injected
+  const { healMissingSiteDataHooks } = require('./transformer.cjs');
+  for (const relativeFile of profile.jsxFiles || []) {
+    const abs = path.join(projectDir, relativeFile);
+    if (!fs.existsSync(abs)) continue;
+    const currentCode = fs.readFileSync(abs, 'utf8');
+    if (currentCode.includes('siteData')) {
+      const healed = healMissingSiteDataHooks(currentCode, relativeFile);
+      if (healed && healed !== currentCode) {
+        backupFile(projectDir, backupDir, abs);
+        fs.writeFileSync(abs, healed, 'utf8');
+        afterFiles[relativeFile] = healed;
+        if (!changedFiles.includes(relativeFile)) changedFiles.push(relativeFile);
+      }
+    }
+  }
+
   if (profile.framework === 'nextjs') {
     const before = findNextConfig(projectDir);
     if (before) backupFile(projectDir, backupDir, before.abs);
@@ -682,13 +699,39 @@ function runArcTransformations(projectDir, projectName, opts, profile, graph, an
   const alreadyEditable = analyses.filter((a) => a.alreadyEditable).length;
   const skippedDynamic = plan.skipped.filter((s) => /dynamic|api/.test(s.reason || '')).length;
 
-  const fivoraAudit = auditFivoraContract({
+  let fivoraAudit = auditFivoraContract({
     profile,
     siteData: dataBundle.siteData,
     manifest: dataBundle.manifest,
     inventory: collectMarkerInventory(projectDir, profile, graph),
   });
 
+  // Auto-reconciliation: If fivoraAudit flagged unknown paths, auto-register them to controlOnlyPaths
+  if (!fivoraAudit.passed && fivoraAudit.errors?.length > 0) {
+    let manifestModified = false;
+    for (const err of fivoraAudit.errors) {
+      const unknownMatch = err.match(/references unknown path "([^"]+)"/);
+      if (unknownMatch) {
+        const unknownPath = unknownMatch[1];
+        dataBundle.manifest.visualEditing = dataBundle.manifest.visualEditing || {};
+        dataBundle.manifest.visualEditing.controlOnlyPaths = dataBundle.manifest.visualEditing.controlOnlyPaths || [];
+        if (!dataBundle.manifest.visualEditing.controlOnlyPaths.includes(unknownPath)) {
+          dataBundle.manifest.visualEditing.controlOnlyPaths.push(unknownPath);
+          manifestModified = true;
+        }
+      }
+    }
+    if (manifestModified) {
+      writeDataBank(projectDir, dataBundle.siteData, dataBundle.manifest);
+      fivoraAudit = auditFivoraContract({
+        profile,
+        siteData: dataBundle.siteData,
+        manifest: dataBundle.manifest,
+        inventory: collectMarkerInventory(projectDir, profile, graph),
+      });
+    }
+  }
+
   const coverage = coverageMetrics({
     analyses,
     plan,

package/src/arc/learning.cjs

@@ -108,6 +108,10 @@ const BASELINE_VERIFIED_FINGERPRINTS = {
   'empty-state-array-guard': { id: 'empty-state-array-guard', state: 'verified', successfulApplications: 50, failedApplications: 0 },
   'action-label-split': { id: 'action-label-split', state: 'verified', successfulApplications: 50, failedApplications: 0 },
   'section-overflow-clip': { id: 'section-overflow-clip', state: 'verified', successfulApplications: 50, failedApplications: 0 },
+  'auto-injected-site-data-hook': { id: 'auto-injected-site-data-hook', state: 'verified', successfulApplications: 50, failedApplications: 0 },
+  'hidden-preview-marker-strip': { id: 'hidden-preview-marker-strip', state: 'verified', successfulApplications: 50, failedApplications: 0 },
+  'unknown-marker-auto-prune': { id: 'unknown-marker-auto-prune', state: 'verified', successfulApplications: 50, failedApplications: 0 },
+  'hidden-element-strict-contract': { id: 'hidden-element-strict-contract', state: 'deprecated', successfulApplications: 0, failedApplications: 50 },
 };
 
 function loadFingerprintBoost(fingerprint) {

package/src/arc/residual.cjs

@@ -427,8 +427,16 @@ function applyResidualPass({ code, file, ownerScope, usedPaths, componentName, r
     return { code, changed: false, fields: [], applied: 0 };
   }
 
+  let finalCode = printSource(ast, code);
+  if (finalCode.includes('siteData')) {
+    try {
+      const { healMissingSiteDataHooks } = require('./transformer.cjs');
+      finalCode = healMissingSiteDataHooks(finalCode, file);
+    } catch {}
+  }
+
   return {
-    code: printSource(ast, code),
+    code: finalCode,
     changed: true,
     fields,
     applied,

package/src/arc/semantic.cjs

@@ -286,6 +286,34 @@ function hasStaticMarker(node) {
   return hasJsxAttribute(node, 'data-preview-static');
 }
 
+function isElementOrParentHidden(pathNode) {
+  let current = pathNode;
+  while (current) {
+    const node = current.node || current.value || current;
+    if (node && (node.type === 'JSXElement' || node.type === 'JSXOpeningElement')) {
+      const attrs = (node.type === 'JSXElement' ? node.openingElement?.attributes : node.attributes) || [];
+      if (attrs.some((a) => a.type === 'JSXAttribute' && a.name && (
+        a.name.name === 'hidden' ||
+        (a.name.name === 'aria-hidden' && /true/.test(recast.print(a).code))
+      ))) {
+        return true;
+      }
+      const classAttr = attrs.find((a) => a.type === 'JSXAttribute' && a.name && (a.name.name === 'className' || a.name.name === 'class'));
+      if (classAttr) {
+        const rawClass = recast.print(classAttr).code;
+        if (/(?<![\w-])hidden(?![a-zA-Z0-9_-])/.test(rawClass)) return true;
+      }
+      const styleAttr = attrs.find((a) => a.type === 'JSXAttribute' && a.name && a.name.name === 'style');
+      if (styleAttr) {
+        const rawStyle = recast.print(styleAttr).code;
+        if (/(?:display\s*:\s*['"]none['"]|visibility\s*:\s*['"]hidden['"])/.test(rawStyle)) return true;
+      }
+    }
+    current = current.parentPath || current.parent;
+  }
+  return false;
+}
+
 function parentNames(pathNode) {
   const names = [];
   let current = pathNode.parent;
@@ -523,6 +551,21 @@ function analyzeFile({ code, relativeFile, profile, graph, ownerScope, component
         return;
       }
 
+      if (isElementOrParentHidden(pathNode)) {
+        candidates.push({
+          loc,
+          tag: name,
+          kind: 'decoration',
+          confidence: 0.1,
+          skip: true,
+          reason: 'hidden-element-strict-contract',
+          file: relativeFile,
+          ownerScope,
+        });
+        this.traverse(pathNode);
+        return;
+      }
+
       const importSource = imports[name.split('.')[0]];
       const recognition = recognizeWithAdapters(node, { profile, imports }, adapters);
       const mapInfo = inMapCallback(pathNode);

package/src/arc/transformer.cjs

@@ -1537,7 +1537,7 @@ function sanitizeContradictoryMarkers(ast) {
 }
 
 function sanitizeContradictoryMarkersInSource(code, relativeFile) {
-  if (!code.includes('data-preview-static') && !code.includes('data-preview-field-path') && !code.includes('overflow-hidden')) {
+  if (!code.includes('data-preview-static') && !code.includes('data-preview-field-path') && !code.includes('data-preview-list-path') && !code.includes('data-preview-item-path') && !code.includes('overflow-hidden') && !code.includes('hidden')) {
     return { code, updated: false };
   }
   let ast;
@@ -1720,10 +1720,15 @@ function healHiddenPreviewMarkers(ast) {
           if (expr.type === 'TemplateLiteral') classText = (expr.quasis || []).map((q) => q.value.cooked || q.value.raw || '').join(' ');
         }
       }
-      const hiddenAttr = attrs.some(
-        (attr) => attr.type === 'JSXAttribute' && attr.name && (attr.name.name === 'hidden' || (attr.name.name === 'aria-hidden' && recast.print(attr).code.includes('true')))
-      );
-      if (hiddenAttr || /(^|\s)hidden(\s|$)/.test(classText)) {
+      const isHidden =
+        attrs.some((attr) => attr.type === 'JSXAttribute' && attr.name && (
+          attr.name.name === 'hidden' ||
+          (attr.name.name === 'aria-hidden' && /true/.test(recast.print(attr).code))
+        )) ||
+        /(?<![\w-])hidden(?![a-zA-Z0-9_-])/.test(classText) ||
+        attrs.some((attr) => attr.type === 'JSXAttribute' && attr.name && attr.name.name === 'style' && /(?:display\s*:\s*['"]none['"]|visibility\s*:\s*['"]hidden['"])/.test(recast.print(attr).code));
+
+      if (isHidden) {
         pathNode.node.attributes = attrs.filter(
           (attr) =>
             !(

package/src/tools/deneb-doctor.cjs

@@ -201,6 +201,7 @@ function runDoctor(targetDirInput = '.', options = {}) {
   // =========================================================================
   if (!isJson) console.log('\n\x1b[1m[2/7] Project Package Configuration:\x1b[0m');
   const suite2 = 'Package Configuration';
+  const sourceFiles = findSourceFiles(path.join(targetDir, 'src'));
 
   const pkgPath = path.join(targetDir, 'package.json');
   let pkg = null;
@@ -288,6 +289,20 @@ function runDoctor(targetDirInput = '.', options = {}) {
         addCheck(suite2, 'err', 'Platform Engine Compatibility', 'Detected packages requiring Node >=22. Run "deneb doctor --fix" to inject Node 20 overrides.', { code: 'DNB-ENG-001' }) //, 'Detected packages requiring Node >=22. Run "deneb doctor --fix" to inject Node 20 overrides.');
       }
 
+      // Pre-heal missing useSiteData hooks before tsc runs when --fix is enabled
+      if (shouldFix) {
+        for (const file of sourceFiles) {
+          if (!/\.(tsx|jsx|ts|js)$/.test(file)) continue;
+          let c = fs.readFileSync(file, 'utf-8');
+          if (c.includes('siteData')) {
+            const healed = healMissingSiteDataHooks(c, file);
+            if (healed && healed !== c) {
+              fs.writeFileSync(file, healed, 'utf8');
+            }
+          }
+        }
+      }
+
       // Check TypeScript compilation preflight
       const tsconfigPath = path.join(targetDir, 'tsconfig.json');
       if (fs.existsSync(tsconfigPath)) {
@@ -480,7 +495,7 @@ function runDoctor(targetDirInput = '.', options = {}) {
   }
 
   // Scan all source files for visual editing contract compliance
-  const sourceFiles = findSourceFiles(path.join(targetDir, 'src'));
+  // (sourceFiles already loaded)
   const foundFieldPaths = new Set();
   const orphanPaths = [];
   const missingInSchema = [];
@@ -869,7 +884,7 @@ function runDoctor(targetDirInput = '.', options = {}) {
   for (const file of sourceFiles) {
     if (!/\.(tsx|jsx|ts|js)$/.test(file)) continue;
     let c = fs.readFileSync(file, 'utf-8');
-    if (c.includes('siteData') && (c.includes('useSiteData') || c.includes('@deneb-ui/ui'))) {
+    if (c.includes('siteData')) {
       const healed = healMissingSiteDataHooks(c, file);
       const norm = (s) => s.replace(/\r\n/g, '\n');
       if (healed && norm(healed) !== norm(c)) {