@deneb-ui/cli 2.0.50 → 2.0.52

package/bin/index.js

@@ -187,6 +187,18 @@ function detectPages(projectDir) {
               continue;
             }
             if (!foundRoutes.has('/' + routeName)) {
+              const pageFile = ['page.tsx', 'page.jsx', 'page.js']
+                .map((name) => path.join(cDir, routeName, name))
+                .find((file) => fs.existsSync(file));
+              const pagesFile = ['tsx', 'jsx', 'js']
+                .map((ext) => path.join(cDir, `${routeName}.${ext}`))
+                .find((file) => fs.existsSync(file));
+              if (!pageFile && !pagesFile && path.basename(cDir) !== 'pages' && path.basename(cDir) !== 'src') {
+                continue;
+              }
+              if (!pageFile && path.basename(cDir) !== 'pages' && !cDir.endsWith(`${path.sep}pages`)) {
+                continue;
+              }
               foundRoutes.add('/' + routeName);
               const label = routeName
                 .replace(/[-_]/g, ' ')

package/package.json

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

package/src/arc/__tests__/arc.test.cjs

@@ -158,8 +158,8 @@ test('AST transformer preserves className and uses nullish fallbacks', () => {
   assert.match(result.code, /data-preview-field-path=/);
   assert.match(result.code, /\?\?/);
   assert.match(result.code, /<span[\s\S]*data-preview-field-path="/);
-  assert.match(result.code, /<span[^>]*hidden[^>]*data-preview-field-path="/);
-  assert.match(result.code, /data-preview-static="action-link"/);
+  assert.match(result.code, /<a\b[^>]*data-preview-field-path=/);
+  assert.doesNotMatch(result.code, /<span[^>]*hidden[^>]*data-preview-field-path/);
   assert.doesNotMatch(result.code, /'use client'/);
   assert.match(result.code, /site-data\.json|@\/data\/site-data\.json/);
   assert.match(result.code, /data-preview-style-target=/);
@@ -409,12 +409,14 @@ test('static-array collections become list contracts without changing render log
   assert.equal(siteData.content.home.products[0].title, 'Minimalist Smart Watch');
 });
 
-test('collections holding component references are left alone', () => {
+test('collections holding component references still bind primitive item fields', () => {
   const dir = copyOf(STOREFRONT_FIXTURE);
   silence(() => runDenebArc(dir, 'acme-store', { telemetry: 'off' }));
   const features = fs.readFileSync(path.join(dir, 'src', 'components', 'Features.tsx'), 'utf8');
-  assert.ok(!features.includes('data-preview-list-path'), 'icon component refs are not merchant content');
+  assert.match(features, /data-preview-list-path=/);
   assert.match(features, /icon: Truck/);
+  assert.match(features, /data-preview-static="component-ref"/);
+  assert.match(features, /data-preview-field-path=\{`[^`]*\.title`\}/);
 });
 
 test('literal text inside a broad container is wrapped instead of marked illegally', () => {
@@ -435,8 +437,8 @@ test('shadcn Button asChild keeps the action on the link and the label in a span
   assert.match(hero, /<Button asChild>/);
   assert.match(hero, /href=\{siteData\?\.content\?\.home\?\.hero\?\.shopCollectionUrl \?\? "\/products"\}/);
   assert.match(hero, /<span[\s\S]*data-preview-field-path="home\.hero\.shopCollectionLabel"/);
-  // The decorative icon stays static.
-  assert.match(hero, /<ArrowRight className="ml-2 size-4" aria-hidden="true" \/>/);
+  // The decorative icon stays static (may also receive data-preview-static).
+  assert.match(hero, /<ArrowRight[^>]*className="ml-2 size-4"[^>]*aria-hidden="true"/);
 });
 
 test('a second init run converges on identical sources and manifest', () => {
@@ -729,7 +731,12 @@ test('classifyActionIntent identifies action keywords and assigns correct fallba
   assert.equal(shop?.defaultUrl, '/shop');
   assert.equal(shop?.external, false);
 
-  const nonAction = classifyActionIntent('Submit Form', null);
+  const formSubmit = classifyActionIntent('Submit Form', null);
+  assert.equal(formSubmit?.action, 'form-submit');
+  assert.equal(formSubmit?.defaultUrl, 'https://wa.me/1234567890');
+  assert.equal(formSubmit?.external, true);
+
+  const nonAction = classifyActionIntent('Random Content Text', null);
   assert.equal(nonAction, null);
 });
 
@@ -816,11 +823,12 @@ export function ActionPanel() {
   assert.match(result.code, /<a\s+[^>]*href=\{siteData\?\.content\?\.home\?\.hero\?\.whatsappUrl \?\? "https:\/\/wa\.me\/1234567890"\}/);
   assert.match(result.code, /target="_blank"/);
   assert.match(result.code, /rel="noopener noreferrer"/);
-  assert.match(result.code, /data-preview-static="action-link"/);
+  // URL field path is bound directly to outer <a> element (never hidden, never static conflict)
+  assert.match(result.code, /<a\s+[^>]*data-preview-field-path=\{?"home\.hero\.whatsappUrl"?\}/);
   // Text label wrapped in editable span
   assert.match(result.code, /<span\s+data-preview-field-path="home\.hero\.whatsappLabel"[^>]*>\{siteData\?\.content\?\.home\?\.hero\?\.whatsappLabel \?\? "Order on WhatsApp"\}<\/span>/);
-  // Hidden URL span present for Fivora contract
-  assert.match(result.code, /<span hidden aria-hidden="true" data-preview-field-path="home\.hero\.whatsappUrl">/);
+  // Never emit hidden preview markers
+  assert.doesNotMatch(result.code, /<span[^>]*hidden[^>]*data-preview-field-path/);
   // No parse errors
   assert.doesNotThrow(() => parseSource(result.code, 'ActionPanel.tsx'));
 });
@@ -1215,3 +1223,204 @@ export function CartDrawer() {
   assert.match(splitAction.urlField, /whatsapp/i);
 });
 
+test('classifyActionIntent recognizes form submit action keywords', () => {
+  const submit = classifyActionIntent('Submit', null);
+  assert.equal(submit?.action, 'form-submit');
+  assert.equal(submit?.defaultUrl, 'https://wa.me/1234567890');
+
+  const sendMessage = classifyActionIntent('Send Message', null);
+  assert.equal(sendMessage?.action, 'form-submit');
+
+  const confirmBooking = classifyActionIntent('Confirm Booking', null);
+  assert.equal(confirmBooking?.action, 'form-submit');
+
+  const getQuote = classifyActionIntent('Get Quote', null);
+  assert.equal(getQuote?.action, 'form-submit');
+});
+
+test('semantic engine recognizes form inputs, labels, placeholders, and form-submit-action buttons', () => {
+  const code = `
+export function BookingForm() {
+  return (
+    <form className="booking-form">
+      <h2>Book Appointment</h2>
+      <label>Full Name</label>
+      <input type="text" placeholder="Enter your full name" />
+      <label>Email Address</label>
+      <input type="email" placeholder="you@example.com" />
+      <label>Special Instructions</label>
+      <textarea placeholder="Describe your request..." />
+      <button type="submit">Confirm Booking</button>
+    </form>
+  );
+}
+`;
+  const profile = {
+    root: os.tmpdir(),
+    framework: 'nextjs',
+    router: 'next-app',
+    language: 'typescript',
+    cssSystems: ['tailwind'],
+    hasSrc: true,
+  };
+
+  const analysis = analyzeFile({
+    code,
+    relativeFile: 'src/components/BookingForm.tsx',
+    profile,
+    graph: { sharedFiles: [] },
+    ownerScope: 'home',
+    componentMeta: { name: 'BookingForm', role: 'form' },
+  });
+
+  const placeholders = analysis.candidates.filter((c) => c.kind === 'placeholder');
+  assert.equal(placeholders.length, 3, 'expected 3 placeholder candidates');
+
+  const labels = analysis.candidates.filter((c) => c.kind === 'text' && c.tag === 'label');
+  assert.equal(labels.length, 3, 'expected 3 label candidates');
+
+  const submitAction = analysis.candidates.find((c) => c.kind === 'form-submit-action');
+  assert.ok(submitAction, 'expected form-submit-action candidate');
+  assert.equal(submitAction.label, 'Confirm Booking');
+  assert.equal(submitAction.extra.action, 'form-submit');
+  assert.equal(submitAction.extra.insideForm, true);
+});
+
+test('planner and transformer make entire form editable with submit button and WhatsApp URL binding', () => {
+  const code = `
+import React from 'react';
+
+export function ContactSection() {
+  return (
+    <form className="contact-form">
+      <label>Your Name</label>
+      <input type="text" placeholder="John Doe" />
+      <button type="submit">Send Message</button>
+    </form>
+  );
+}
+`;
+  const profile = {
+    root: os.tmpdir(),
+    framework: 'nextjs',
+    router: 'next-app',
+    language: 'typescript',
+    cssSystems: ['tailwind'],
+    hasSrc: true,
+    aliasMap: { '@/*': ['src/*'] },
+  };
+
+  const analysis = analyzeFile({
+    code,
+    relativeFile: 'src/components/ContactSection.tsx',
+    profile,
+    graph: { sharedFiles: [] },
+    ownerScope: 'home',
+    componentMeta: { name: 'ContactSection', role: 'contact' },
+  });
+  analysis.code = code;
+  analysis.relativeFile = 'src/components/ContactSection.tsx';
+
+  const plan = planTransformations({
+    profile,
+    analyses: [analysis],
+  });
+
+  const filePlan = plan.files[0];
+  const formSubmit = filePlan.transformations.find((t) => t.operation === 'form-submit-action');
+  assert.ok(formSubmit, 'form-submit-action must be planned');
+  assert.match(formSubmit.urlField, /formWhatsappUrl/i);
+  assert.match(formSubmit.labelField, /formSubmitLabel/i);
+
+  const transformed = applyFilePlan(filePlan, profile);
+  assert.ok(transformed.changed, 'file must be changed');
+  assert.match(transformed.code, /data-preview-field-path/);
+  assert.match(transformed.code, /formSubmitLabel/);
+  assert.match(transformed.code, /type="submit"/);
+  assert.doesNotMatch(transformed.code, /<span[^>]*hidden/);
+  assert.doesNotMatch(transformed.code, /data-preview-static[\s\S]*data-preview-field-path/);
+});
+
+test('auditMarkerPlacement detects and rejects hidden preview markers', () => {
+  const hiddenCode = `
+    <div>
+      <span className="hidden" aria-hidden="true" data-preview-field-path="home.hero.buttonUrl">https://wa.me/123</span>
+      <span>Click me</span>
+    </div>
+  `;
+  const errors = contract.auditMarkerPlacement(hiddenCode, 'TestHidden.tsx');
+  assert.ok(errors.some((e) => e.includes('data-preview field/list/item marker is hidden')));
+});
+
+test('auditCoupledListMarkers detects and rejects coupling parallel list arrays by index', () => {
+  const coupledCode = `
+    <div data-preview-item-path="seasonal.items[0]">
+      <span data-preview-field-path="seasonal.preOrderCta[0].buttonLabel">Pre-Order</span>
+    </div>
+  `;
+  const errors = contract.auditCoupledListMarkers(coupledCode, 'TestCoupled.tsx');
+  assert.ok(errors.some((e) => e.includes('belongs to a different list')));
+});
+
+test('detectedPages without a page file are not invented in the manifest', () => {
+  const dir = copyOf(FIXTURE);
+  silence(() =>
+    runDenebArc(dir, 'basic-store', {
+      telemetry: 'off',
+      detectedPages: [{ id: 'contact', label: 'Contact', route: '/contact', required: true }],
+    })
+  );
+  const { manifest } = auditFivora(dir);
+  assert.ok(!manifest.pages.some((page) => page.id === 'contact' || page.route === '/contact'));
+});
+
+test('learning records contract failure instead of success', () => {
+  const { honestOutcome } = require('../learning.cjs');
+  assert.equal(
+    honestOutcome({ syntaxPassed: true, contractPassed: true, fivoraContractPassed: false }, 'success'),
+    'failure'
+  );
+  assert.equal(
+    honestOutcome({ syntaxPassed: true, contractPassed: true, fivoraContractPassed: true, uncoveredVisibleText: 0 }, 'success'),
+    'success'
+  );
+});
+
+test('residual pass marks leftover decorative copy static with a reason', () => {
+  const { applyResidualPass } = require('../residual.cjs');
+  const code = `export function Chrome() { return <button aria-hidden="true">Close</button>; }`;
+  const result = applyResidualPass({
+    code,
+    file: 'src/components/Chrome.tsx',
+    ownerScope: 'home',
+    usedPaths: new Set(),
+  });
+  assert.equal(result.changed, true);
+  assert.match(result.code, /data-preview-static="[^"]+"/);
+});
+
+test('style-bind grid and card attributes are written onto collection nodes', () => {
+  const dir = copyOf(STOREFRONT_FIXTURE);
+  silence(() => runDenebArc(dir, 'acme-store', { telemetry: 'off' }));
+  const grid = fs.readFileSync(path.join(dir, 'src', 'components', 'ProductGrid.tsx'), 'utf8');
+  assert.match(grid, /data-preview-style-type="grid"/);
+  assert.match(grid, /data-preview-style-type="card"/);
+  assert.match(grid, /data-preview-style-target="home\.products\.grid"/);
+});
+
+test('ContactActions stays mounted when all channels are empty', () => {
+  const sourcePath = path.join(__dirname, '..', '..', '..', '..', '..', 'packages', 'deneb-ui', 'src', 'contact', 'ContactActions.tsx');
+  const source = fs.readFileSync(sourcePath, 'utf8');
+  assert.doesNotMatch(source, /if \(!hasPhone && !hasWhatsApp && !hasEmail\)/);
+  assert.doesNotMatch(source, /return null/);
+});
+
+test('empty-state source audit flags gated preview markers', () => {
+  const errors = contract.auditEmptyStateSource(
+    '{title && <span data-preview-field-path="home.hero.title">{title}</span>}',
+    'Hero.tsx'
+  );
+  assert.ok(errors.some((error) => error.includes('gated behind')));
+});
+
+

package/src/arc/adapters.cjs

@@ -23,6 +23,8 @@ const HEADING_TAGS = new Set(['h1', 'h2', 'h3', 'h4', 'h5', 'h6', 'Heading', 'Ti
 const TEXT_TAGS = new Set(['p', 'span', 'li', 'blockquote', 'figcaption', 'label', 'CardDescription', 'Description', 'Subtitle', 'Typography', 'Text', 'Badge', 'badge']);
 const ACTION_TAGS = new Set(['a', 'Link', 'NavLink', 'Button', 'button', 'IconButton', 'NavbarBrand']);
 const IMAGE_TAGS = new Set(['img', 'Image', 'Img', 'BackgroundImage']);
+const FORM_INPUT_TAGS = new Set(['input', 'Input', 'textarea', 'Textarea', 'select', 'Select']);
+const FORM_CONTAINER_TAGS = new Set(['form', 'Form']);
 
 const shadcn = createAdapter('shadcn', {
   detect: (project) => detectFromProfile(project, 'shadcn') || project.shadcn,
@@ -106,6 +108,9 @@ const deneb = createAdapter('deneb', {
     Boolean(project.dependencies && project.dependencies['@deneb-ui/ui']),
   recognizeNode(node) {
     const name = getJsxName(node);
+    if (['EditableContactForm', 'ContactForm'].includes(name)) {
+      return { library: 'deneb', kind: 'form', tag: name };
+    }
     if (['EditableGoogleFeedback', 'EditableCustomerReviews', 'CustomerReviews', 'GoogleFeedback'].includes(name)) {
       return { library: 'deneb', kind: 'feedback', tag: name };
     }
@@ -192,6 +197,18 @@ function classifyActionIntent(text, href) {
   const t = String(text || '').trim().toLowerCase();
   const h = String(href || '').trim().toLowerCase();
 
+  // 0. Form Submit / WhatsApp Form Action
+  if (
+    /^(submit|send\s*message|send\s*inquiry|confirm\s*booking|place\s*order|book\s*now|confirm|get\s*quote|request\s*quote|send\s*request|schedule|register|sign\s*up|subscribe|apply|enroll)$/i.test(t) ||
+    /\b(?:send\s*message|confirm\s*booking|submit\s*form|submit\s*inquiry|send\s*inquiry|request\s*quote|get\s*quote|send\s*request)\b/i.test(t)
+  ) {
+    return {
+      action: 'form-submit',
+      defaultUrl: /^https?:\/\//i.test(h) && !h.includes('#') ? href : 'https://wa.me/1234567890',
+      external: true,
+    };
+  }
+
   // 1. WhatsApp
   if (
     /wa\.me|whatsapp/i.test(h) ||
@@ -280,6 +297,8 @@ module.exports = {
   TEXT_TAGS,
   ACTION_TAGS,
   IMAGE_TAGS,
+  FORM_INPUT_TAGS,
+  FORM_CONTAINER_TAGS,
   collectJsxText,
   getJsxAttributeLiteral,
 };

package/src/arc/field-paths.cjs

@@ -47,6 +47,7 @@ function inferSection(context) {
     ['testimonials', /testimonial/],
     ['map', /map\b|location-map|google-map/],
     ['faq', /faq|accordion/],
+    ['form', /form|booking|inquiry|registration/],
     ['contact', /contact|whatsapp|mailto/],
     ['featuredProducts', /featured|product-grid|collection/],
     ['newsletter', /newsletter|subscribe/],
@@ -65,6 +66,7 @@ function inferSection(context) {
 
 function inferFieldName(kind, tag, text, extra = {}) {
   if (kind === 'url') {
+    if (extra.action === 'form-submit') return 'formWhatsappUrl';
     if (extra.platform) return `${extra.platform}Url`;
     if (extra.action === 'whatsapp') return 'whatsappUrl';
     if (extra.action === 'phone') return 'phoneUrl';
@@ -74,6 +76,7 @@ function inferFieldName(kind, tag, text, extra = {}) {
     return fromText || (extra.action ? `${extra.action}Url` : 'ctaUrl');
   }
   if (kind === 'label' && extra.paired) {
+    if (extra.action === 'form-submit') return 'formSubmitLabel';
     if (extra.action === 'whatsapp') return 'whatsappLabel';
     if (extra.action === 'phone') return 'phoneLabel';
     if (extra.action === 'email') return 'emailLabel';
@@ -100,6 +103,7 @@ function inferFieldName(kind, tag, text, extra = {}) {
     Description: 'description',
     Typography: 'text',
     Badge: 'badge',
+    label: 'label',
     button: 'label',
     Button: 'label',
     span: 'label',