@bettercms-ai/convert 0.12.0 → 0.12.1

package/CHANGELOG.md

@@ -1,5 +1,26 @@
 # @bettercms-ai/convert
 
+## 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;
@@ -9595,7 +9605,7 @@ function emitStubs2(dialects, slugs, contents, out) {
 // package.json
 var package_default = {
   name: "@bettercms-ai/convert",
-  version: "0.12.0",
+  version: "0.12.1",
   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 +9645,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"