@bettercms-ai/convert 0.16.0 → 0.16.1

package/CHANGELOG.md

@@ -1,5 +1,18 @@
 # @bettercms-ai/convert
 
+## 0.16.1
+
+### Patch Changes
+
+- Fix `--forms` tie-breaking. 0.16.0's second tie-break rule ("the form posting nothing beyond its
+  fields wins") could bind a look-alike form — for example a "forgot password" dialog posting only
+  `email` — to the newsletter without reporting it. 0.16.1 removes that rule: such cases stay
+  `FORM_AMBIGUOUS` until you name the file with `--pin "<form>=<file>"`. When the remaining rule
+  (the form posting more of a `<form>`'s fields wins) decides a tie, the receipt now carries a `notes`
+  line naming the file and the rival that lost it. `--pin` is also stricter: an empty value or a
+  repeated pin for the same form is refused, `--pin` cannot be combined with `--componentize`, and
+  two forms pinned to one file both wire.
+
 ## 0.16.0
 
 ### Minor Changes

package/README.md

@@ -630,6 +630,13 @@ a reason — `FORM_NOT_IN_SOURCE`, `FORM_AMBIGUOUS`, `FIELD_UNMATCHED`, `ACTION_
 leads into another form's inbox, silently, forever. A `<form>` that already carries
 `data-bcms-form` is `alreadyWired` and is left alone, so a second run is a no-op.
 
+Two `<form>`s that both post exactly a form's fields are `FORM_AMBIGUOUS`, and no rule can break
+that tie: `--pin "<form>=<file>"` (repeatable, form by id or name) names the file, and a pin is
+exclusive only against forms that carry no pin of their own, so two forms living in one file can
+both be pinned to it. **Keep passing a pin on every later run** — a pin is not written into the
+tree, so an unpinned run over the same repository reports that form ambiguous again and its
+receipt stops listing the file it is wired to, even though the markup still is.
+
 `--receipt <file>` writes this run's own receipt (`forms-receipt.json` — a separate file from the
 P2 run's, since the two runs describe different work), and `--strict` exits 2 on any pending form.
 `--forms` and `--componentize` are separate runs; doing both in one command is refused.

package/dist/cli.js

@@ -6521,32 +6521,33 @@ async function wireForms(briefIn, sources, options_ = {}) {
   }
   const holds = (c, f) => f.fields.filter((field) => c.controls.has(field.key)).length;
   const covers2 = (c, f) => f.fields.length > 0 && holds(c, f) === f.fields.length;
-  const posts = (c) => new Set(c.controls.values()).size;
   const samePath = (a, b) => a.replace(/\\/g, "/").replace(/^\.\//, "") === b.replace(/\\/g, "/").replace(/^\.\//, "");
   const options = new Map(forms.map((f) => [f.id, candidates.filter((c) => covers2(c, f))]));
   const pinned = /* @__PURE__ */ new Map();
   for (const [ref, file] of Object.entries(options_.pins ?? {})) {
     const form = forms.find((f) => f.id === ref || f.name === ref);
     if (!form) throw new Error(`--pin ${ref}: the brief lists no form with that id or name.`);
+    if (file.trim() === "") throw new Error(`--pin ${ref}: no file named. Give "<form>=<file>".`);
     pinned.set(form.id, file);
-    options.set(form.id, options.get(form.id).filter((c) => samePath(c.file, file)));
+  }
+  for (const [id, file] of pinned) {
+    options.set(id, options.get(id).filter((c) => samePath(c.file, file)));
     for (const other of forms) {
-      if (other.id !== form.id) options.set(other.id, options.get(other.id).filter((c) => !samePath(c.file, file)));
+      const its = pinned.get(other.id);
+      if (other.id === id || its !== void 0 && samePath(its, file)) continue;
+      options.set(other.id, options.get(other.id).filter((c) => !samePath(c.file, file)));
     }
   }
+  const beat = /* @__PURE__ */ new Map();
   for (const c of candidates) {
     const claimants = forms.filter((f) => options.get(f.id).includes(c));
     if (claimants.length < 2) continue;
     const top = Math.max(...claimants.map((f) => holds(c, f)));
     const winners = claimants.filter((f) => holds(c, f) === top);
     if (winners.length !== 1) continue;
-    for (const f of claimants) if (f !== winners[0]) options.set(f.id, options.get(f.id).filter((x) => x !== c));
-  }
-  for (const f of forms) {
-    const list = options.get(f.id);
-    if (list.length < 2) continue;
-    const exact = list.filter((c) => posts(c) === f.fields.length);
-    if (exact.length === 1) options.set(f.id, exact);
+    const rivals = claimants.filter((f) => f !== winners[0]);
+    for (const f of rivals) options.set(f.id, options.get(f.id).filter((x) => x !== c));
+    beat.set(c, { winner: winners[0], rivals });
   }
   const wanted = /* @__PURE__ */ new Map();
   for (const list of options.values()) for (const c of list) wanted.set(c, (wanted.get(c) ?? 0) + 1);
@@ -6562,7 +6563,7 @@ async function wireForms(briefIn, sources, options_ = {}) {
         form,
         "FORM_AMBIGUOUS",
         void 0,
-        `${list.length} forms post every field of "${form.name}": ${list.map((c) => c.file).join(", ")}. Name the right one with --pin "${form.name}=<file>".`
+        `${list.length} forms post every field of "${form.name}": ${list.map((c) => c.file).join(", ")}.${pinned.has(form.id) ? " The pin names that file, and it holds more than one." : ` Name the right one with --pin "${form.name}=<file>".`}`
       );
       continue;
     }
@@ -6626,6 +6627,14 @@ ${indent}${scriptFor(last.candidate.dialect)}`);
     files.push({ path, operation: "modify", content: source.toString() });
   }
   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.`);
+  for (const { form, candidate } of matched) {
+    const tie = beat.get(candidate);
+    if (tie?.winner === form) {
+      notes.push(
+        `form ${form.name}: ${candidate.file} posts every field of ${tie.rivals.map((f) => f.name).join(", ")} too, and more of ${form.name}'s, so it went to ${form.name} \u2014 name the file yourself with --pin "<form>=<file>" if that is wrong.`
+      );
+    }
+  }
   for (const { form, candidate } of matched) {
     const keys = form.fields.filter((field) => candidate.foreign.has(candidate.controls.get(field.key))).map((field) => field.key);
     if (keys.length > 0) {
@@ -9726,7 +9735,7 @@ function emitStubs2(dialects, slugs, contents, out) {
 // package.json
 var package_default = {
   name: "@bettercms-ai/convert",
-  version: "0.16.0",
+  version: "0.16.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: {
@@ -9805,9 +9814,14 @@ function parseArgs(argv) {
     else if (arg === "--componentize") args.componentize = true;
     else if (arg === "--forms") args.forms = true;
     else if (arg === "--pin") {
-      const at = (argv[++i] ?? "").indexOf("=");
-      if (at < 1) throw new CliError(`--pin takes <form>=<file>, e.g. --pin "Newsletter=src/components/FooterSection.astro". See --help.`);
-      args.pins[argv[i].slice(0, at)] = argv[i].slice(at + 1);
+      const raw = argv[++i] ?? "";
+      const at = raw.indexOf("=");
+      if (at < 1 || at === raw.length - 1) {
+        throw new CliError(`--pin takes <form>=<file>, e.g. --pin "Newsletter=src/components/FooterSection.astro". See --help.`);
+      }
+      const form = raw.slice(0, at);
+      if (Object.hasOwn(args.pins, form)) throw new CliError(`--pin ${form} twice: give one file. See --help.`);
+      args.pins[form] = raw.slice(at + 1);
     } else if (arg === "--xray") args.xray = argv[++i];
     else if (arg === "--plan") args.plan = argv[++i];
     else if (arg === "--copy") {
@@ -10029,6 +10043,9 @@ async function main() {
   if (args.forms && args.componentize) {
     throw new CliError("--forms and --componentize are separate runs. Do one, then the other. See --help.");
   }
+  if (Object.keys(args.pins).length > 0 && !args.forms) {
+    throw new CliError("--pin only means something with --forms. See --help.");
+  }
   const root = resolve(args.root);
   const brief = JSON.parse(await readFile(resolve(args.brief), "utf-8"));
   if (!Array.isArray(brief.pages)) {
@@ -10069,9 +10086,6 @@ async function main() {
     reportSections(files2, receipt2, args.dryRun);
     return args.strict && receipt2.sections.pending.length > 0 ? 2 : 0;
   }
-  if (Object.keys(args.pins).length > 0 && !args.forms) {
-    throw new CliError("--pin only means something with --forms. See --help.");
-  }
   if (args.forms) {
     const { files: files2, receipt: receipt2 } = await wireForms(brief, sources, { pins: args.pins });
     await written(files2);