@bettercms-ai/convert 0.12.0 → 0.13.0

package/CHANGELOG.md

@@ -1,5 +1,38 @@
 # @bettercms-ai/convert
 
+## 0.13.0
+
+### Minor Changes
+
+- 0b05aa6: The forms receipt names the file each wired form is in
+
+  `receipt.forms.wiredForms` lists `{ id, file }` for every form on the site after a
+  `--forms` run — wired now or by an earlier one. The server matches the file to the
+  component recorded as its owner, so the Forms tab and a component's form prop can say
+  which component renders each form. Additive: `wired` is still a number, and a server
+  that has never heard of the list accepts the receipt unchanged.
+
+## 0.12.1
+
+### Patch Changes
+
+- 6d826cf: The trunk-zero rule is per-country data, not a regex
+
+  `bcmsPhone` stripped a leading zero from every national number with a bare
+  `.replace(/^0/, "")`. That is only correct for countries that actually use a trunk
+  prefix. Where the leading digit is part of the number, the submitted value was
+  silently corrupted — one digit short, and wrong in a way nobody notices until a
+  customer cannot be called back.
+
+  The country list now carries a `trunk` flag derived from libphonenumber's own example
+  numbers, rendered as `data-trunk` on each option, and the zero is dropped only when
+  that country says it is a trunk prefix.
+
+  The inline script exists once per vanilla lane — the Astro component, the converter and
+  the server renderer — because each ships as inline script text on someone else's page
+  and cannot import at runtime. All three carry the identical change, which
+  `src/__tests__/content/form-script-pin.test.ts` pins byte for byte.
+
 ## 0.12.0
 
 ### Minor Changes

package/dist/cli.js

@@ -6321,6 +6321,7 @@ import MagicString4 from "magic-string";
 var WIRABLE = ["html", "astro", "jsx"];
 var CONTROLS = /* @__PURE__ */ new Set(["input", "select", "textarea"]);
 var FORM_SCRIPT = `<script>
+function bcmsPhone(cc,raw,trunk){var d=String(raw||'').trim();if(!d)return '';if(d.charAt(0)==='+'){var i=d.replace(/D/g,'');return i?'+'+i:'';}if(!cc)return d;var n=d.replace(/D/g,'');if(trunk&&n.charAt(0)==='0')n=n.slice(1);return n?'+'+cc+n:'';}
 document.querySelectorAll('form[data-bcms-form]').forEach(function(f){
   f.addEventListener('submit',function(e){
     e.preventDefault();
@@ -6336,6 +6337,15 @@ document.querySelectorAll('form[data-bcms-form]').forEach(function(f){
       if(Object.prototype.hasOwnProperty.call(data,k))data[k]=(Array.isArray(data[k])?data[k]:[data[k]]).concat(v);
       else data[k]=v;
     });
+    // Country-picker phone fields post ONE key. The __country select is scaffolding: it is
+    // composed into the field's own value here, AFTER collection, so a showIf that points at a
+    // phone key still compares the raw digits. Then it is removed from the body.
+    f.querySelectorAll('select[name$="__country"]').forEach(function(sel){
+      var k=sel.name.slice(0,-9);
+      var o=sel.options[sel.selectedIndex];
+      data[k]=bcmsPhone(o?(o.getAttribute('data-cc')||''):'',data[k],o?o.getAttribute('data-trunk')==='1':false);
+      delete data[k+'__country'];
+    });
     var body={data:data};
     var tok=f.querySelector('[name="cf-turnstile-response"]');
     if(tok&&tok.value)body['cf-turnstile-response']=tok.value;
@@ -6501,11 +6511,11 @@ async function wireForms(briefIn, sources) {
     }
     if (attrOf(candidate.el, "data-bcms-form")) {
       alreadyWired++;
-      onSite.push(form);
+      onSite.push({ form, file: candidate.file });
       continue;
     }
     matched.push({ form, candidate });
-    onSite.push(form);
+    onSite.push({ form, file: candidate.file });
   }
   const contents = new Map(sources.map((s) => [s.path, s.content]));
   const byFile = /* @__PURE__ */ new Map();
@@ -6523,7 +6533,7 @@ ${indent}${scriptFor(last.candidate.dialect)}`);
     }
     files.push({ path, operation: "modify", content: source.toString() });
   }
-  const notes = onSite.filter((f) => f.status === "draft").map((f) => `publish form ${f.name} in the Forms tab \u2014 a draft form rejects every submission with 403.`);
+  const notes = onSite.filter(({ form }) => form.status === "draft").map(({ form }) => `publish form ${form.name} in the Forms tab \u2014 a draft form rejects every submission with 403.`);
   return {
     files,
     receipt: {
@@ -6540,7 +6550,9 @@ ${indent}${scriptFor(last.candidate.dialect)}`);
         wired: matched.length,
         alreadyWired,
         pending,
-        ...notes.length > 0 ? { notes } : {}
+        ...notes.length > 0 ? { notes } : {},
+        // Always, even empty: a run where every form went pending must retire the files an earlier one named.
+        wiredForms: onSite.map(({ form, file }) => ({ id: form.id, file }))
       })
     }
   };
@@ -9595,7 +9607,7 @@ function emitStubs2(dialects, slugs, contents, out) {
 // package.json
 var package_default = {
   name: "@bettercms-ai/convert",
-  version: "0.12.0",
+  version: "0.13.0",
   type: "module",
   description: "Deterministic codemod that turns an imported site's text-matched bindings into declared ones \u2014 parser offsets in, one magic-string pass out, plus a receipt of everything it could not do.",
   bin: {
@@ -9635,7 +9647,7 @@ var package_default = {
   },
   devDependencies: {
     "@types/node": "^20",
-    esbuild: "^0.27.7",
+    esbuild: "^0.28.1",
     tsup: "^8.5.1",
     typescript: "^5",
     vitest: "^4.1.4"