@bettercms-ai/convert 0.16.1 → 0.17.1

package/dist/index.d.ts

@@ -114,6 +114,13 @@ interface BriefForm {
         key: string;
         type: string;
     }[];
+    /**
+     * Where the built form sat — footer, header, nav, main, aside, dialog or body. Breaks the tie
+     * between several source `<form>`s posting exactly these fields. @see forms.ts
+     */
+    landmark?: string;
+    /** The owner's answer to FORM_AMBIGUOUS: the file this form is in. Applied like `--pin`. */
+    pin?: string;
 }
 
 /**
@@ -402,7 +409,7 @@ interface ConversionReceipt {
      * reader must treat absent as unknown rather than as `"pending"`.
      */
     canvas?: {
-        lane: "bridge" | "pending";
+        lane: "bridge" | "draft-route" | "pending";
         files?: string[];
     };
     /**
@@ -431,6 +438,8 @@ interface PendingForm {
     file?: string;
     /** What exactly could not be done. Informative; the reason is the contract. */
     message?: string;
+    /** FORM_AMBIGUOUS only: the files that each post every field — what a person picks from. */
+    candidates?: string[];
 }
 interface FormsReceipt {
     /** A `<form>` this run pointed at the CMS: action, method, id, field markers and the script. */
@@ -1233,7 +1242,7 @@ interface CanvasSource {
     ast?: unknown;
 }
 interface CanvasResult {
-    lane: "bridge" | "pending";
+    lane: "bridge" | "draft-route" | "pending";
     /** Files the bridge was installed in. */
     files: string[];
     /** Rewritten contents, by path. Empty when the lane is pending. */
@@ -1248,8 +1257,8 @@ declare const ASTRO_LANE_PENDING: PendingPath;
  * Install the draft bridge, or say why not.
  *
  * `routes` maps a brief route (`/about`) to its slug, so a route written as a literal in the page
- * can name itself. An Astro project is refused wholesale: bridging it needs a server render, which
- * is a deploy-shaped decision and never a codemod's.
+ * can name itself. An Astro project is never bridged here: it either wired a lane itself (see
+ * {@link astroLane}) or it carries the SSR recipe.
  */
 declare function canvasBridge(sources: CanvasSource[], routes: Map<string, string>): CanvasResult;
 

package/dist/index.js

@@ -6090,9 +6090,28 @@ var pend = (reason, hint, file, line) => ({
   hint
 });
 var ASTRO_LANE_PENDING = pend("SSR_DRAFT_ROUTE", SSR_DRAFT_RECIPE);
+function mounts(content, component) {
+  const from = new RegExp(
+    `import\\s+([A-Za-z_$][\\w$]*)\\s+from\\s+["']@bettercms-ai/astro/components/${component}\\.astro["']`
+  );
+  const local = content.match(from)?.[1];
+  return local !== void 0 && new RegExp(`<${local}[\\s/>]`).test(content);
+}
+var ASTRO_CONFIG = /(^|\/)astro\.config\.(mjs|cjs|js|ts|mts)$/;
+function astroLane(sources) {
+  const astro = sources.filter((s) => s.path.toLowerCase().endsWith(".astro"));
+  const server = sources.some(
+    (s) => ASTRO_CONFIG.test(s.path) && /\boutput\s*:\s*["']server["']/.test(s.content)
+  );
+  if (server && astro.some((s) => mounts(s.content, "BcmsLive"))) return "draft-route";
+  if (astro.some((s) => mounts(s.content, "BcmsLiveBlocks"))) return "bridge";
+  return null;
+}
 function canvasBridge(sources, routes) {
-  if (sources.some((s) => s.path.toLowerCase().endsWith(".astro")))
-    return { lane: "pending", files: [], edits: /* @__PURE__ */ new Map(), pending: [ASTRO_LANE_PENDING] };
+  if (sources.some((s) => s.path.toLowerCase().endsWith(".astro"))) {
+    const lane = astroLane(sources);
+    return lane ? { lane, files: [], edits: /* @__PURE__ */ new Map(), pending: [] } : { lane: "pending", files: [], edits: /* @__PURE__ */ new Map(), pending: [ASTRO_LANE_PENDING] };
+  }
   const files = [];
   const edits = /* @__PURE__ */ new Map();
   const pending = [];
@@ -6387,6 +6406,7 @@ function scriptFor(dialect) {
 var messageFor = (dialect) => dialect === "jsx" ? `<p className="bcms-form-msg" hidden></p>` : `<p class="bcms-form-msg" hidden></p>`;
 var escapeAttr = (value) => value.replace(/&/g, "&amp;").replace(/</g, "&lt;").replace(/"/g, "&quot;");
 var attrOf = (el, name) => el.attrs.find((a) => a.name.toLowerCase() === name);
+var DECLARED_BINDING_ATTR = /^data-bcms-(field|kind|props)$/i;
 function* elementsIn(nodes) {
   for (const node of nodes) {
     if (node.type === "expr") {
@@ -6401,8 +6421,28 @@ function* elementsIn(nodes) {
 var childElements = (node) => node.children.flatMap(
   (child) => child.type === "element" ? [child] : child.type === "expr" ? childElements(child) : []
 );
+var LANDMARK_TAGS = /* @__PURE__ */ new Set(["footer", "header", "nav", "main", "aside", "dialog"]);
+function landmarkOf(chain) {
+  for (let i = chain.length - 1; i >= 0; i--) {
+    const el = chain[i];
+    if (attrOf(el, "role")?.value === "dialog") return "dialog";
+    if (LANDMARK_TAGS.has(el.tag.toLowerCase())) return el.tag.toLowerCase();
+  }
+  return "body";
+}
+function* formsIn(nodes, chain = []) {
+  for (const node of nodes) {
+    if (node.type === "expr") {
+      yield* formsIn(node.children, chain);
+      continue;
+    }
+    if (node.type !== "element") continue;
+    if (node.tag.toLowerCase() === "form") yield { el: node, landmark: landmarkOf(chain) };
+    yield* formsIn(node.children, [...chain, node]);
+  }
+}
 var isComponentTag = (dialect, tag) => dialect !== "html" && /^[A-Z]/.test(tag);
-function candidateOf(file, dialect, form, imported) {
+function candidateOf(file, dialect, form, imported, landmark) {
   const controls = /* @__PURE__ */ new Map();
   const wrapper = /* @__PURE__ */ new Map();
   const foreign = /* @__PURE__ */ new Set();
@@ -6439,7 +6479,7 @@ function candidateOf(file, dialect, form, imported) {
     const id = attrOf(control, "id")?.value;
     if (id && !controls.has(id)) controls.set(id, control);
   }
-  return { file, dialect, el: form, controls, wrapper, foreign };
+  return { file, dialect, el: form, controls, wrapper, foreign, landmark };
 }
 async function importedControls(source, parsed, dialect, project, cache) {
   const out = /* @__PURE__ */ new Map();
@@ -6474,14 +6514,21 @@ function wire(source, candidate, form) {
   };
   set("action", form.submitUrl);
   set("method", "post");
-  inserts.push(`data-bcms-form="${escapeAttr(form.id)}"`);
-  if (form.successMessage) inserts.push(`data-bcms-form-success="${escapeAttr(form.successMessage)}"`);
+  set("data-bcms-form", form.id);
+  if (form.successMessage) set("data-bcms-form-success", form.successMessage);
   source.appendLeft(el.attrInsertAt, ` ${inserts.join(" ")}`);
+  for (const inner of elementsIn(el.children)) {
+    for (const attr of inner.attrs) {
+      if (!DECLARED_BINDING_ATTR.test(attr.name)) continue;
+      const start = source.original[attr.range.start - 1] === " " ? attr.range.start - 1 : attr.range.start;
+      source.remove(start, attr.range.end);
+    }
+  }
   for (const field of form.fields) {
     const control = candidate.controls.get(field.key);
     if (candidate.foreign.has(control)) continue;
     const host = candidate.wrapper.get(control) ?? control;
-    source.appendLeft(host.attrInsertAt, ` data-bcms-form-field="${escapeAttr(field.key)}"`);
+    if (!attrOf(host, "data-bcms-form-field")) source.appendLeft(host.attrInsertAt, ` data-bcms-form-field="${escapeAttr(field.key)}"`);
     const name = attrOf(control, "name");
     if (!name) source.appendLeft(control.attrInsertAt, ` name="${escapeAttr(field.key)}"`);
     else if (!name.expression && name.value !== field.key && name.valueRange) {
@@ -6525,8 +6572,8 @@ async function wireForms(briefIn, sources, options_ = {}) {
       continue;
     }
     const imported = dialect === "html" ? /* @__PURE__ */ new Map() : await importedControls(source, parsed, dialect, project, componentFiles);
-    for (const el of elementsIn(parsed.roots)) {
-      if (el.tag.toLowerCase() === "form") candidates.push(candidateOf(source.path, dialect, el, imported));
+    for (const { el, landmark } of formsIn(parsed.roots)) {
+      candidates.push(candidateOf(source.path, dialect, el, imported, landmark));
     }
   }
   const holds = (c, f) => f.fields.filter((field) => c.controls.has(field.key)).length;
@@ -6540,6 +6587,9 @@ async function wireForms(briefIn, sources, options_ = {}) {
     if (file.trim() === "") throw new Error(`--pin ${ref}: no file named. Give "<form>=<file>".`);
     pinned.set(form.id, file);
   }
+  for (const form of forms) {
+    if (form.pin && form.pin.trim() !== "" && !pinned.has(form.id)) pinned.set(form.id, form.pin);
+  }
   for (const [id, file] of pinned) {
     options.set(id, options.get(id).filter((c) => samePath(c.file, file)));
     for (const other of forms) {
@@ -6559,6 +6609,15 @@ async function wireForms(briefIn, sources, options_ = {}) {
     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 placed = /* @__PURE__ */ new Map();
+  for (const form of forms) {
+    const list = options.get(form.id);
+    if (list.length < 2 || !form.landmark || form.landmark === "body") continue;
+    const kept = list.filter((c) => c.landmark === form.landmark);
+    if (kept.length === 0 || kept.length === list.length) continue;
+    options.set(form.id, kept);
+    placed.set(form.id, { kept, dropped: list.filter((c) => !kept.includes(c)) });
+  }
   const wanted = /* @__PURE__ */ new Map();
   for (const list of options.values()) for (const c of list) wanted.set(c, (wanted.get(c) ?? 0) + 1);
   const matched = [];
@@ -6569,12 +6628,13 @@ async function wireForms(briefIn, sources, options_ = {}) {
   for (const form of forms) {
     const list = options.get(form.id) ?? [];
     if (list.length > 1) {
-      pend2(
-        form,
-        "FORM_AMBIGUOUS",
-        void 0,
-        `${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>".`}`
-      );
+      pending.push({
+        id: form.id,
+        name: form.name,
+        reason: "FORM_AMBIGUOUS",
+        message: `${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>".`}`,
+        candidates: [...new Set(list.map((c) => c.file))]
+      });
       continue;
     }
     if (list.length === 1 && (wanted.get(list[0]) ?? 0) > 1) {
@@ -6612,7 +6672,7 @@ async function wireForms(briefIn, sources, options_ = {}) {
       pend2(form, "FIELD_UNMATCHED", candidate.file, `the control for ${misnamed.key} posts under an expression, so its name cannot be set to the CMS key.`);
       continue;
     }
-    if (attrOf(candidate.el, "data-bcms-form")) {
+    if (attrOf(candidate.el, "data-bcms-form")?.value === form.id) {
       alreadyWired++;
       onSite.push({ form, file: candidate.file });
       continue;
@@ -6644,6 +6704,12 @@ ${indent}${scriptFor(last.candidate.dialect)}`);
         `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.`
       );
     }
+    const where = placed.get(form.id);
+    if (where) {
+      notes.push(
+        `form ${form.name}: ${where.dropped.map((c) => c.file).join(", ")} post${where.dropped.length === 1 ? "s" : ""} every field of it too, but the built form sat in the ${form.landmark} and only ${candidate.file}'s does \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);