@bettercms-ai/convert 0.11.0 → 0.12.1

package/dist/index.d.ts

@@ -197,6 +197,83 @@ declare function locate(brief: BriefLike, sources: SourceFile[], options?: Locat
     unlocated: UnlocatedPath[];
 };
 
+/**
+ * What to DO about a path this run could not bind — an instruction, never a category name.
+ *
+ * 🔴 A REASON IS A DIAGNOSIS AND AN AGENT CANNOT ACT ON ONE. `IN_EXPRESSION` tells a coding agent
+ * that something, somewhere, is an expression; it does not say which file, which line, or what to
+ * type. So every pending row leaves with a `fix`: one file, one line, the bytes that are there
+ * now, and the one edit that makes the path bindable on the next run. The closed `action` set is
+ * the contract the server validates (`src/lib/validation/schemas/conversion-receipt.ts`) and the
+ * loop `get_next_steps` reads — an unknown action is a 400, so it is minted in exactly one place.
+ */
+
+/**
+ * The edits an agent can be asked for, and nothing else.
+ *
+ * `wrap-span`         the value has no element of its own — wrap it in `<span data-bcms-field>`.
+ * `declare-attr`      the element is there — add `data-bcms-field` to it at file:line.
+ * `declare-richtext`  the container renders markup — bind it with the richtext helper.
+ * `bind-expression`   the value is computed — replace the expression with the helper's read.
+ * `bind-data`         the copy lives in a data file or a content entry — bind THAT field.
+ * `manual`            none of the above is true, and `why` says what a human has to decide.
+ */
+type FixAction = "wrap-span" | "declare-attr" | "bind-expression" | "declare-richtext" | "bind-data" | "manual";
+/** One mechanical instruction, at one place in one file. @see FixAction */
+interface PendingFix {
+    file: string;
+    /** 1-based. */
+    line: number;
+    /** 1-based, when the literal was found on the line. */
+    col?: number;
+    /** The bytes at `line`, trimmed — so the agent can see it is looking at the right place. */
+    snippet: string;
+    action: FixAction;
+    /** One sentence, present whenever the action alone does not say enough. */
+    why?: string;
+}
+/** Everything about one pending path a fix is derived from. Pure input; no filesystem. */
+interface FixInput {
+    /** The copy the repository renders, flattened. Empty when the brief captured none. */
+    literal: string;
+    kind: string;
+    /** The richtext value IS the element's whole inline subtree. @see brief.ts */
+    inline?: true;
+    /** Files this path's copy was found in, best first. May be empty. */
+    candidates: string[];
+    /** Source, by path — only the files named in `candidates` need be present. */
+    contents: ReadonlyMap<string, string>;
+    /** How many ROUTES share this `(scope, path)` in the pending list. @see sharedChrome */
+    routes: number;
+}
+/**
+ * Where `literal` sits in `content` — tolerant of the two things every template does to copy.
+ *
+ * A template wraps its copy across lines and indents it, and it puts inline markup in the middle
+ * of it (`Read <strong>this</strong> now`). Both are invisible to a reader and fatal to
+ * `indexOf`, so the words are matched with whitespace OR a tag between them. Bounded at
+ * `MAX_WORDS`, because a paragraph's first dozen words identify it as well as all ninety do.
+ */
+declare function findLiteral(content: string, literal: string): {
+    index: number;
+    length: number;
+} | null;
+/**
+ * The fix for one pending row. Always answers: a row without one is a row an agent cannot act on.
+ *
+ * The FILE is the row's own when it has one (the lane that refused named it), and otherwise the
+ * first candidate `locate` found — and when there is neither, the fix still names the route's page
+ * file with line 1, because "go and look here" beats "go and look".
+ */
+declare function fixFor(row: PendingPath, input: FixInput): PendingFix;
+/**
+ * How many ROUTES each `(scope, path)` is pending on — the shared-chrome tell. @see actionFor
+ *
+ * Counted over the whole pending list rather than per row: one nav label pending on twelve routes
+ * is one element, and the fix for every one of those rows is the same sentence about promotion.
+ */
+declare function routeCounts(pending: readonly PendingPath[]): Map<string, number>;
+
 /**
  * What the conversion did, and — the half that matters — what it did not.
  *
@@ -209,6 +286,7 @@ declare function locate(brief: BriefLike, sources: SourceFile[], options?: Locat
  * sentence appears in three files" and "this field is bound" are different questions — and a path
  * is only `rewritten` when EVERY located occurrence of it was.
  */
+
 type PendingReason = "NO_ORIGINAL" | "NOT_IN_SOURCE" | "DIALECT_UNSUPPORTED" | "IN_EXPRESSION" | "IN_SCRIPT_OR_COMMENT" | "IN_DATA_FILE" | "AMBIGUOUS_LITERAL" | "KIND_MISMATCH" | "SUBSTRING_ONLY"
 /** An INLINE richtext value whose element holds a component, a `<slot>` or an expression. */
  | "SUBTREE_NOT_HTML" | "PROP_TARGET_NOT_FOUND" | "PROP_DRILLED_DEEP" | "REPEATER_FIXED_LENGTH"
@@ -231,6 +309,15 @@ interface PendingPath {
     line?: number;
     /** What to do about it, in prose — a recipe, not a diagnosis. @see message */
     hint?: string;
+    /**
+     * The EDIT that makes this path bindable on the next run — one file, one line, one action.
+     *
+     * 🔴 THE REASON IS A DIAGNOSIS AND NOBODY CAN ACT ON ONE. Every row carries this, because a
+     * conversion ends when nothing is pending and the only way an agent gets there is by being told
+     * what to type and where. OPTIONAL in the type so a receipt written by an older version of this
+     * package still parses; this package emits one for every row. @see fix.ts
+     */
+    fix?: PendingFix;
 }
 /**
  * One path that was already DECLARED before this run — and how honestly.
@@ -869,7 +956,7 @@ declare function isKnownHelper(content: string, dialect: Dialect): number | null
  * renderer's constant occurs in it byte for byte, which is the only thing that keeps the copy a
  * copy. Change it there first, then paste it here.
  */
-declare const FORM_SCRIPT = "<script>\ndocument.querySelectorAll('form[data-bcms-form]').forEach(function(f){\n  f.addEventListener('submit',function(e){\n    e.preventDefault();\n    var msg=f.querySelector('.bcms-form-msg');\n    var data={};\n    // A checkboxes group posts one entry per checked box under the SAME name; the old\n    // one-at-a-time assignment kept only the last, so \"pick many\" delivered one answer.\n    // One checked box still arrives as a scalar \u2014 FormData cannot tell a group from a\n    // lone field \u2014 which is exactly why lib/forms/validate.ts's chosenValues() accepts\n    // both an array and a comma string.\n    new FormData(f).forEach(function(v,k){\n      if(k==='cf-turnstile-response')return;\n      if(Object.prototype.hasOwnProperty.call(data,k))data[k]=(Array.isArray(data[k])?data[k]:[data[k]]).concat(v);\n      else data[k]=v;\n    });\n    var body={data:data};\n    var tok=f.querySelector('[name=\"cf-turnstile-response\"]');\n    if(tok&&tok.value)body['cf-turnstile-response']=tok.value;\n    fetch(f.action,{method:'POST',headers:{'Content-Type':'application/json'},body:JSON.stringify(body)})\n      .then(function(r){return r.ok;})\n      .then(function(ok){if(msg){msg.hidden=false;msg.textContent=ok?(f.getAttribute('data-bcms-form-success')||'Thanks! Your submission was received.'):'Sorry, something went wrong.';}if(ok)f.reset();})\n      .catch(function(){if(msg){msg.hidden=false;msg.textContent='Sorry, something went wrong.';}});\n  });\n});\n</script>";
+declare const FORM_SCRIPT = "<script>\nfunction 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:'';}\ndocument.querySelectorAll('form[data-bcms-form]').forEach(function(f){\n  f.addEventListener('submit',function(e){\n    e.preventDefault();\n    var msg=f.querySelector('.bcms-form-msg');\n    var data={};\n    // A checkboxes group posts one entry per checked box under the SAME name; the old\n    // one-at-a-time assignment kept only the last, so \"pick many\" delivered one answer.\n    // One checked box still arrives as a scalar \u2014 FormData cannot tell a group from a\n    // lone field \u2014 which is exactly why lib/forms/validate.ts's chosenValues() accepts\n    // both an array and a comma string.\n    new FormData(f).forEach(function(v,k){\n      if(k==='cf-turnstile-response')return;\n      if(Object.prototype.hasOwnProperty.call(data,k))data[k]=(Array.isArray(data[k])?data[k]:[data[k]]).concat(v);\n      else data[k]=v;\n    });\n    // Country-picker phone fields post ONE key. The __country select is scaffolding: it is\n    // composed into the field's own value here, AFTER collection, so a showIf that points at a\n    // phone key still compares the raw digits. Then it is removed from the body.\n    f.querySelectorAll('select[name$=\"__country\"]').forEach(function(sel){\n      var k=sel.name.slice(0,-9);\n      var o=sel.options[sel.selectedIndex];\n      data[k]=bcmsPhone(o?(o.getAttribute('data-cc')||''):'',data[k],o?o.getAttribute('data-trunk')==='1':false);\n      delete data[k+'__country'];\n    });\n    var body={data:data};\n    var tok=f.querySelector('[name=\"cf-turnstile-response\"]');\n    if(tok&&tok.value)body['cf-turnstile-response']=tok.value;\n    fetch(f.action,{method:'POST',headers:{'Content-Type':'application/json'},body:JSON.stringify(body)})\n      .then(function(r){return r.ok;})\n      .then(function(ok){if(msg){msg.hidden=false;msg.textContent=ok?(f.getAttribute('data-bcms-form-success')||'Thanks! Your submission was received.'):'Sorry, something went wrong.';}if(ok)f.reset();})\n      .catch(function(){if(msg){msg.hidden=false;msg.textContent='Sorry, something went wrong.';}});\n  });\n});\n</script>";
 /**
  * Wire every form the brief lists, and say what could not be.
  *
@@ -1720,4 +1807,4 @@ declare function convertSources(briefIn: Brief, sources: SourceFile[], options?:
     receipt: ConversionReceipt;
 }>;
 
-export { ASTRO_LANE_PENDING, type AstNode, type AttrBinding, type Brief, type BriefForm, type BriefPage, type BriefPath, COMPONENT_DIR, CONTENT_DIR, type CanvasResult, type CanvasSource, type Carried, type ComponentizeOptions, type ComponentizePlan, type ComponentizePlanComponent, type ComponentizePlanField, type ComponentizePlanPage, type ComponentizePlanSection, type ComponentizeReceipt, type ConversionReceipt, ConvertError, type ConvertErrorCode, type ConvertOptions, DIALECT_RULES, type DataMember, type DataResult, type DeclaredPath, type Dialect, type DynamicBinding, FORM_SCRIPT, type FileDeclaration, type FindSitesResult, FormInvariantError, type FormPendingReason, type FormsReceipt, HELPER_PATH, HELPER_VERSION, type ImageResult, type ImageTarget, type ImportedFrom, type LlmFallback, type LocatedPath, type Node, PARSE_FILE, type ParsedFile, type ParserError, type PathLocator, type PendingForm, type PendingPath, type PendingReason, type PendingSection, type PlanFile, REGISTRY_MARKER, type ReceiptFile, ReceiptInvariantError, type ReceiptNote, type Rewrite, SECTIONS_LIB, SECTION_MARKER, SKIP_DIRS, SKIP_FILES, SOURCE_EXTENSIONS, SSR_DRAFT_RECIPE, STUB_CONTENT, SectionInvariantError, type SectionPendingReason, type SectionsReceipt, type Site, type SiteWhere, type SourceFile, type Splice, type TargetIdentity, type UnlocatedPath, type ValidationCode, ValidationError, type Xray, type XrayBucket, aliasesFrom, assertForms, assertReceipt, assertSections, assetCandidates, assetImport, attrsFor, bindDataLiterals, bindImages, briefDigest, canvasBridge, carryFor, componentizeSources, convertSources, convertedHere, coverageOf, declaringElements, dialectOf, exportedFunction, extendedConfigs, findSites, findSitesTolerant, flat, helperSource, importsIn, isDecorativeGlyph, isDynamicRoute, isKnownHelper, isSourceCandidate, locate, measureUnaddressable, overlapping, pageFilesFor, parseFile, propsAttribute, readBrief, readComponentizePlan, readDeclarations, readExpr, readPlan, relativeImport, resolveSpecifier, rewriteFile, routeOfFile, stripTags, unreadableConfigs, walkAst, wireForms };
+export { ASTRO_LANE_PENDING, type AstNode, type AttrBinding, type Brief, type BriefForm, type BriefPage, type BriefPath, COMPONENT_DIR, CONTENT_DIR, type CanvasResult, type CanvasSource, type Carried, type ComponentizeOptions, type ComponentizePlan, type ComponentizePlanComponent, type ComponentizePlanField, type ComponentizePlanPage, type ComponentizePlanSection, type ComponentizeReceipt, type ConversionReceipt, ConvertError, type ConvertErrorCode, type ConvertOptions, DIALECT_RULES, type DataMember, type DataResult, type DeclaredPath, type Dialect, type DynamicBinding, FORM_SCRIPT, type FileDeclaration, type FindSitesResult, type FixAction, FormInvariantError, type FormPendingReason, type FormsReceipt, HELPER_PATH, HELPER_VERSION, type ImageResult, type ImageTarget, type ImportedFrom, type LlmFallback, type LocatedPath, type Node, PARSE_FILE, type ParsedFile, type ParserError, type PathLocator, type PendingFix, type PendingForm, type PendingPath, type PendingReason, type PendingSection, type PlanFile, REGISTRY_MARKER, type ReceiptFile, ReceiptInvariantError, type ReceiptNote, type Rewrite, SECTIONS_LIB, SECTION_MARKER, SKIP_DIRS, SKIP_FILES, SOURCE_EXTENSIONS, SSR_DRAFT_RECIPE, STUB_CONTENT, SectionInvariantError, type SectionPendingReason, type SectionsReceipt, type Site, type SiteWhere, type SourceFile, type Splice, type TargetIdentity, type UnlocatedPath, type ValidationCode, ValidationError, type Xray, type XrayBucket, aliasesFrom, assertForms, assertReceipt, assertSections, assetCandidates, assetImport, attrsFor, bindDataLiterals, bindImages, briefDigest, canvasBridge, carryFor, componentizeSources, convertSources, convertedHere, coverageOf, declaringElements, dialectOf, exportedFunction, extendedConfigs, findLiteral, findSites, findSitesTolerant, fixFor, flat, helperSource, importsIn, isDecorativeGlyph, isDynamicRoute, isKnownHelper, isSourceCandidate, locate, measureUnaddressable, overlapping, pageFilesFor, parseFile, propsAttribute, readBrief, readComponentizePlan, readDeclarations, readExpr, readPlan, relativeImport, resolveSpecifier, rewriteFile, routeCounts, routeOfFile, stripTags, unreadableConfigs, walkAst, wireForms };

package/dist/index.js

@@ -5318,6 +5318,138 @@ function coverageOf(receipt) {
   };
 }
 
+// src/fix.ts
+var MAX_SNIPPET = 160;
+var MAX_WORDS = 12;
+var escapeRegExp2 = (value) => value.replace(/[.*+?^${}()|[\]\\]/g, "\\$&");
+function findLiteral(content, literal) {
+  const words = literal.split(/\s+/).filter(Boolean).slice(0, MAX_WORDS);
+  if (words.length === 0) return null;
+  const pattern = new RegExp(words.map(escapeRegExp2).join("(?:\\s|<[^>]*>)+"), "u");
+  const match = pattern.exec(content);
+  return match ? { index: match.index, length: match[0].length } : null;
+}
+function placeOf(content, index) {
+  const before = content.slice(0, index);
+  const line = before.split("\n").length;
+  const start = before.lastIndexOf("\n") + 1;
+  const end = content.indexOf("\n", index);
+  return {
+    line,
+    col: index - start + 1,
+    snippet: content.slice(start, end < 0 ? content.length : end).trim().slice(0, MAX_SNIPPET)
+  };
+}
+function fileHead(content) {
+  return { line: 1, snippet: (content ?? "").split("\n", 1)[0].trim().slice(0, MAX_SNIPPET) };
+}
+var isMarkdown = (file) => /\.mdx?$/i.test(file);
+var isDataModule = (file) => /\.(json|ya?ml|[mc]?[jt]s)$/i.test(file) && !/\.d\.ts$/i.test(file);
+function inDataRegion(content, index) {
+  if (content.startsWith("---")) {
+    const close = content.indexOf("\n---", 3);
+    if (close >= 0 && index < close) return true;
+  }
+  for (const match of content.matchAll(/<script\b[^>]*>([\s\S]*?)<\/script>/gi)) {
+    const start = match.index + match[0].indexOf(">") + 1;
+    if (index >= start && index < start + match[1].length) return true;
+  }
+  return false;
+}
+var ELEMENT_ACTIONS = /* @__PURE__ */ new Set([
+  "wrap-span",
+  "declare-attr",
+  "declare-richtext"
+]);
+function actionFor(reason, input, file, inData) {
+  const chosen = reasonAction(reason, input, file, inData);
+  if (inData && ELEMENT_ACTIONS.has(chosen.action)) {
+    return { action: "bind-data", why: "The copy is a row in this file's own data \u2014 bind that field." };
+  }
+  return chosen;
+}
+function reasonAction(reason, input, file, inData) {
+  const richtext = input.kind === "richtext" || input.inline === true;
+  switch (reason) {
+    case "AMBIGUOUS_LITERAL":
+      if (input.routes > 1) {
+        return {
+          action: "manual",
+          why: `${file} renders this value on ${input.routes} routes, so it is shared chrome rather than page copy: promote it to the project Layout (a Layout field is one value for the whole site) and rerun, or give each route its own element.`
+        };
+      }
+      return {
+        action: "declare-attr",
+        why: input.candidates.length > 1 ? `This copy appears in ${input.candidates.join(", ")}; declare the one this route renders.` : "Two paths want this element and nothing says which \u2014 declare the right one by hand."
+      };
+    case "REPEATER_AMBIGUOUS":
+    case "REPEATER_FIXED_LENGTH":
+      return {
+        action: "declare-attr",
+        why: "Declare this row's leaf on the element itself, or rewrite the loop to read the CMS array."
+      };
+    case "IN_EXPRESSION":
+      if (inData) {
+        return { action: "bind-data", why: "The copy is a row in this file's own data \u2014 bind that field." };
+      }
+      return richtext ? { action: "declare-richtext", why: "The container is fed markup \u2014 bind it with the richtext helper." } : { action: "bind-expression", why: "Replace the computed value with the helper's read for this path." };
+    case "DIALECT_UNSUPPORTED":
+      if (isMarkdown(file)) {
+        return {
+          action: "bind-data",
+          why: "This is a content-collection body: bind the element that renders it and make the entry the source."
+        };
+      }
+      return isDataModule(file) ? { action: "bind-data", why: `${file} holds this copy as data \u2014 bind the element that renders it.` } : { action: "manual", why: `No parser here reads ${file}, so the declaration has to be written by hand.` };
+    case "NOT_IN_SOURCE":
+    case "NO_ORIGINAL":
+      return isDataModule(file) || inData ? { action: "bind-data", why: "The template computes this from data \u2014 bind the data field instead of the text." } : {
+        action: "manual",
+        why: "The built page renders this and no source file holds it verbatim; find what formats it."
+      };
+    case "IN_DATA_FILE":
+    case "IMAGE_ASSET_UNRESOLVED":
+      return { action: "bind-data", why: "Bind the data field this value comes from." };
+    case "PROP_TARGET_NOT_FOUND":
+    case "PROP_DRILLED_DEEP":
+      return {
+        action: "wrap-span",
+        why: "The component renders this in a position no attribute reaches \u2014 wrap it in a span of its own."
+      };
+    case "SUBSTRING_ONLY":
+      return { action: "wrap-span", why: "Only part of the element's text is this value \u2014 give it its own span." };
+    case "SUBTREE_NOT_HTML":
+      return { action: "declare-richtext", why: "The value is the element's whole subtree \u2014 bind it as richtext." };
+    default:
+      return { action: "manual", why: `${reason} \u2014 see the receipt's message for what blocked this path.` };
+  }
+}
+function fixFor(row, input) {
+  const file = row.file ?? input.candidates[0] ?? "";
+  const content = input.contents.get(file);
+  const at = content ? findLiteral(content, input.literal) : null;
+  const place = at ? placeOf(content, at.index) : { ...fileHead(content), col: void 0 };
+  const { action, why } = actionFor(row.reason, input, file, at !== null && inDataRegion(content, at.index));
+  return {
+    file,
+    line: row.line ?? place.line,
+    ...place.col !== void 0 ? { col: place.col } : {},
+    snippet: place.snippet,
+    action,
+    ...why ? { why } : {}
+  };
+}
+function routeCounts(pending) {
+  const routes = /* @__PURE__ */ new Map();
+  for (const row of pending) {
+    const id = `${row.scope}|${row.path}`;
+    const held = routes.get(id) ?? /* @__PURE__ */ new Set();
+    held.add(row.route);
+    routes.set(id, held);
+  }
+  return new Map([...routes].map(([id, set]) => [id, set.size]));
+}
+
 // src/sites/html.ts
 import { parse as parseHtml } from "parse5";
 var RAW_TAGS = /* @__PURE__ */ new Set(["script", "style", "noscript", "template"]);
@@ -6199,6 +6331,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();
@@ -6214,6 +6347,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;
@@ -6438,7 +6580,9 @@ var SKIP_DIRS = [
   ".vercel/",
   ".cache/",
   "coverage/",
-  "vendor/"
+  "vendor/",
+  "__tests__/",
+  "__mocks__/"
 ];
 var SKIP_FILES = ["package-lock.json", "bun.lock", "bun.lockb", "yarn.lock", "pnpm-lock.yaml"];
 var SOURCE_EXTENSIONS = [
@@ -6476,11 +6620,13 @@ var TEMPLATE_DIRS = /* @__PURE__ */ new Set([
   "_includes",
   "_layouts"
 ]);
+var isAboutTheSite = (lower) => /\.(test|spec)\.[^./]+$/.test(lower) || /^[^/]*\.config\.[^./]+$/.test(lower);
 function isSourceCandidate(path) {
   const lower = path.toLowerCase();
   if (SKIP_DIRS.some((dir) => lower === dir.slice(0, -1) || lower.startsWith(dir) || lower.includes(`/${dir}`)))
     return false;
   if (SKIP_FILES.includes(lower.split("/").pop() ?? "")) return false;
+  if (isAboutTheSite(lower)) return false;
   if (/(^|\/)[jt]sconfig\.json$/.test(lower)) return true;
   if (!SOURCE_EXTENSIONS.some((ext) => lower.endsWith(ext))) return false;
   if (!PROSE_EXTENSIONS.some((ext) => lower.endsWith(ext))) return true;
@@ -8429,7 +8575,7 @@ var ConvertError = class extends Error {
   code;
 };
 var keyOf = (route, scope, path) => scope === "layout" ? `layout::${path}` : `${route}::page::${path}`;
-var escapeRegExp2 = (value) => value.replace(/[.*+?^${}()|[\]\\]/g, "\\$&");
+var escapeRegExp3 = (value) => value.replace(/[.*+?^${}()|[\]\\]/g, "\\$&");
 var identifierFor2 = (slug) => `bcms${slug.replace(/[^A-Za-z0-9]+(.)?/g, (_, c) => c ? c.toUpperCase() : "").replace(/^./, (c) => c.toUpperCase())}`;
 var UNPLACED = /* @__PURE__ */ new Set(["NOT_IN_SOURCE", "IN_EXPRESSION"]);
 async function drillProp(ctx, fromFile, component, prop, kind, hop) {
@@ -8776,6 +8922,33 @@ async function convertSources(briefIn, sources, options) {
     else outFiles.push({ path, operation: "modify", content });
   }
   pending.push(...canvas.pending);
+  for (const file of outFiles) if (file.operation === "modify") latest.set(file.path, file.content);
+  const pageFilesOrTemplate = (route, files) => {
+    const all = [...files];
+    const exact = pageFilesFor(route, all);
+    if (exact.length > 0) return exact;
+    const wanted = route.split("/").filter(Boolean);
+    return all.filter((file) => {
+      if (!isDynamicRoute(file)) return false;
+      const pattern = routeOfFile(file)?.split("/").filter(Boolean);
+      if (!pattern) return false;
+      const rest = pattern.some((segment) => segment.startsWith("[..."));
+      if (rest ? wanted.length < pattern.length : wanted.length !== pattern.length) return false;
+      return pattern.every((segment, i) => segment.startsWith("[") || segment === wanted[i]);
+    }).sort();
+  };
+  const routes = routeCounts(pending);
+  for (const row of pending) {
+    const target = targets.get(keyOf(row.route, row.scope, row.path));
+    row.fix = fixFor(row, {
+      literal: target?.literal ?? "",
+      kind: row.kind,
+      ...target?.inline ? { inline: true } : {},
+      candidates: locatedByKey.get(keyOf(row.route, row.scope, row.path))?.files ?? pageFilesOrTemplate(row.route, latest.keys()),
+      contents: latest,
+      routes: routes.get(`${row.scope}|${row.path}`) ?? 1
+    });
+  }
   const receipt = assertReceipt({
     briefDigest: digest,
     paths: {
@@ -8988,13 +9161,13 @@ async function plan(file, content, dialect, targets, sites, importAt, outcomes,
       for (const target of sharing) outcomes.set(target.key, missing(literal));
       continue;
     }
-    const assigned = sharing.length === 1 ? /* @__PURE__ */ new Map([[sharing[0].key, forLiteral]]) : resolvePositionally(sharing, forLiteral);
-    if (!assigned) {
-      for (const target of sharing) outcomes.set(target.key, "AMBIGUOUS_LITERAL");
-      continue;
-    }
+    const resolved = sharing.length === 1 ? { assigned: /* @__PURE__ */ new Map([[sharing[0].key, forLiteral]]), ambiguous: /* @__PURE__ */ new Set() } : resolvePositionally(sharing, forLiteral);
     for (const target of sharing) {
-      const mine = assigned.get(target.key) ?? [];
+      if (resolved.ambiguous.has(target.key)) {
+        outcomes.set(target.key, "AMBIGUOUS_LITERAL");
+        continue;
+      }
+      const mine = resolved.assigned.get(target.key) ?? [];
       const rewritable = mine.filter((s) => s.where !== "script" && s.where !== "comment" && !s.partial);
       if (rewritable.length === 0) {
         outcomes.set(
@@ -9339,19 +9512,25 @@ function repeatGroups(targets) {
 }
 function resolvePositionally(targets, sites) {
   const assigned = /* @__PURE__ */ new Map();
-  const claimed = /* @__PURE__ */ new Set();
+  const ambiguous = /* @__PURE__ */ new Set();
+  const picked = /* @__PURE__ */ new Map();
+  const wanters = /* @__PURE__ */ new Map();
   for (const target of targets) {
     const locator = target.locator;
-    if (!locator) return null;
-    const candidates = sites.filter(
-      (s) => s.siblingIndex === locator.siblingIndex && isSubsequence(locator.ancestors, s.ancestors)
-    );
-    const site = locator.repeat ? candidates[locator.repeat.index] : candidates.length === 1 ? candidates[0] : void 0;
-    if (!site || claimed.has(site)) return null;
-    claimed.add(site);
-    assigned.set(target.key, [site]);
+    const candidates = locator ? sites.filter((s) => s.siblingIndex === locator.siblingIndex && isSubsequence(locator.ancestors, s.ancestors)) : [];
+    const site = locator?.repeat ? candidates[locator.repeat.index] : candidates.length === 1 ? candidates[0] : void 0;
+    if (!site) {
+      ambiguous.add(target.key);
+      continue;
+    }
+    picked.set(target.key, site);
+    wanters.set(site, (wanters.get(site) ?? 0) + 1);
+  }
+  for (const [key, site] of picked) {
+    if ((wanters.get(site) ?? 0) > 1) ambiguous.add(key);
+    else assigned.set(key, [site]);
   }
-  return assigned;
+  return { assigned, ambiguous };
 }
 function isSubsequence(wanted, chain) {
   let at = 0;
@@ -9374,7 +9553,7 @@ async function tierThree(file, content, dialect, targets, llmFallback) {
   if (elements.length !== targets.length) return { ok: false, reason: "unverifiable" };
   const locals = helperLocals(reparsed, dialect, file);
   const reads = DIALECT_RULES[dialect].text !== null;
-  const boundary = (name) => `(?<![\\w$])${escapeRegExp2(name)}(?![\\w$])`;
+  const boundary = (name) => `(?<![\\w$])${escapeRegExp3(name)}(?![\\w$])`;
   const claimed = /* @__PURE__ */ new Set();
   for (const element of elements) {
     const candidates = targets.filter(
@@ -9382,7 +9561,7 @@ async function tierThree(file, content, dialect, targets, llmFallback) {
     );
     const match = candidates.find((target) => {
       if (!reads) return true;
-      const quoted = escapeRegExp2(JSON.stringify(target.path));
+      const quoted = escapeRegExp3(JSON.stringify(target.path));
       if (target.scope === "layout") {
         return locals.bcmsLayout.some(
           (name) => new RegExp(`${boundary(name)}\\s*\\(\\s*${quoted}`).test(element.own)
@@ -9482,8 +9661,10 @@ export {
   dialectOf,
   exportedFunction,
   extendedConfigs,
+  findLiteral,
   findSites,
   findSitesTolerant,
+  fixFor,
   flat,
   helperSource,
   importsIn,
@@ -9505,6 +9686,7 @@ export {
   relativeImport,
   resolveSpecifier,
   rewriteFile,
+  routeCounts,
   routeOfFile,
   stripTags,
   unreadableConfigs,