@bettercms-ai/convert 0.24.1 → 0.25.1

package/CHANGELOG.md

@@ -1,5 +1,23 @@
 # @bettercms-ai/convert
 
+## 0.25.1
+
+### Patch Changes
+
+- b1e5eb0: Prices and other numbers are now editable: a value written as `49` or `price={49}` in a data object, a `const`, or a component prop is bound like any other copy, and still renders the same text. `--brief` no longer writes a second `data-bcms-field` on an element, and two paths that want the same element, or a rewrite inside text another rewrite replaces, are reported as ambiguous instead of corrupting the file; a second run over converted source changes nothing.
+- Updated dependencies [56335ff]
+  - @bettercms-ai/types@1.32.0
+
+## 0.25.0
+
+### Minor Changes
+
+- 832e08a: A converted repository whose `build` runs checks before the build (`astro check && astro build`, `tsc --noEmit && vite build`, `npm run lint && next build`) gets a `build:bcms` script beside it with the same build minus those checks, `prebuild`/`postbuild` folded in. BetterCMS builds with `build:bcms` when a project declares one, so a deploy no longer pays for the project's CI checks; `build` is left unchanged. Every writing run adds it once (a repository that already has one is never touched), and only for a plain `&&` chain. Exported as `bcmsBuildScript`.
+
+### Patch Changes
+
+- c6fd790: The codemod never walks into a directory holding its own `.git` (a git worktree, submodule or vendored clone), so a tool's worktree copy of the repo is no longer converted alongside the real source.
+
 ## 0.24.1
 
 ### Patch Changes

package/dist/cli.js

@@ -2,6 +2,7 @@
 
 // src/cli.ts
 import { execFileSync } from "child_process";
+import { existsSync } from "fs";
 import { readdir, readFile, writeFile, mkdir, stat } from "fs/promises";
 import { dirname, join as join2, relative, resolve, sep } from "path";
 
@@ -2714,8 +2715,9 @@ function collectSites(roots, literals) {
     }
     const bindingsAttr = node.attrs.find((a) => a.name === "bcmsBindings")?.range;
     for (const attr of node.attrs) {
-      if (attr.expression || !attr.valueRange || BCMS_ATTRS.includes(attr.name)) continue;
-      const value = norm(attr.value);
+      const digits = attr.expression && attr.expressionRange ? attr.number : void 0;
+      if (attr.expression && digits === void 0 || !digits && !attr.valueRange || BCMS_ATTRS.includes(attr.name)) continue;
+      const value = norm(digits ?? attr.value);
       for (const literal of wanted) {
         if (value !== literal) continue;
         matched.add(literal);
@@ -2723,7 +2725,7 @@ function collectSites(roots, literals) {
           file: "",
           tag: node.tag,
           attrInsertAt: node.attrInsertAt,
-          range: attr.valueRange,
+          range: digits === void 0 ? attr.valueRange : attr.expressionRange,
           inner: node.inner,
           // 🔴 A COMPONENT'S ATTRIBUTE IS A PROP, and a prop is a different conversion: the value
           // is rendered in another file, by an element only that file knows. @see props.ts
@@ -2831,6 +2833,8 @@ function attrsOf2(open, content) {
     const value = attr.value;
     const range = { start: attr.start ?? 0, end: attr.end ?? 0 };
     const literal = value?.type === "StringLiteral";
+    const inside = value?.type === "JSXExpressionContainer" ? value.expression : null;
+    const numeric = inside?.type === "NumericLiteral" ? String(inside.extra.raw) : "";
     return [
       {
         name: nameOf(attr.name),
@@ -2839,7 +2843,8 @@ function attrsOf2(open, content) {
         valueRange: literal ? { start: (value.start ?? 0) + 1, end: (value.end ?? 0) - 1 } : null,
         expression: !literal,
         range,
-        expressionRange: value?.type === "JSXExpressionContainer" ? { start: value.start ?? 0, end: value.end ?? 0 } : null
+        expressionRange: value?.type === "JSXExpressionContainer" ? { start: value.start ?? 0, end: value.end ?? 0 } : null,
+        ...numeric ? { number: numeric } : {}
       }
     ];
   });
@@ -3315,14 +3320,21 @@ function exportedNames(ast) {
   });
   return out;
 }
+function printed(node) {
+  if (node?.type === "StringLiteral") return [String(node.value)];
+  if (node?.type === "NumericLiteral") return [String(node.value), String(node.extra?.raw ?? "")];
+  const quasis = node?.type === "TemplateLiteral" ? node.quasis ?? [] : [];
+  const expressions = node?.expressions ?? [];
+  return quasis.length === 1 && expressions.length === 0 ? [String(quasis[0].value.cooked ?? "")] : [];
+}
 function propertyFor(row, literal) {
   if (row?.type !== "ObjectExpression") return null;
   for (const property of row.properties ?? []) {
     if (property.type !== "ObjectProperty" && property.type !== "Property") continue;
     const key = property.key;
     const value = property.value;
-    if (value?.type !== "StringLiteral" || key?.type !== "Identifier") continue;
-    if (norm(String(value.value)) === literal) return String(key.name);
+    if (key?.type !== "Identifier") continue;
+    if (printed(value).some((text) => norm(text) === literal)) return String(key.name);
   }
   return null;
 }
@@ -3396,7 +3408,7 @@ function stringLiterals(ast) {
     if (node.type !== "VariableDeclarator") return;
     const id = node.id;
     const init = node.init;
-    if (id?.type !== "Identifier" || init?.type !== "StringLiteral") return;
+    if (id?.type !== "Identifier" || !init || printed(init).length === 0) return;
     out.set(String(id.name), init);
   });
   return out;
@@ -3427,7 +3439,7 @@ function scalarUses(content, roots, id) {
 }
 function bindScalar(ctx, member, literals, offset) {
   const rule = DIALECT_RULES[ctx.dialect];
-  const named = [...literals].filter(([, init2]) => norm(String(init2.value)) === member.literal);
+  const named = [...literals].filter(([, init2]) => printed(init2).some((text) => norm(text) === member.literal));
   if (named.length !== 1) return { splices: [], refusal: null };
   const [id, init] = named[0];
   const uses = scalarUses(ctx.content, ctx.parsed.roots, id);
@@ -3642,16 +3654,17 @@ function literalTable(init) {
   const first = elements[0];
   if (!first || elements.length !== (init.elements ?? []).length) return null;
   if (elements.some((element) => element.type !== first.type)) return null;
-  if (first.type === "StringLiteral") {
-    return { tuple: false, rows: elements.map((element) => [norm(String(element.value))]) };
+  if (printed(first).length > 0) {
+    if (elements.some((element) => printed(element).length === 0)) return null;
+    return { tuple: false, rows: elements.map((element) => [norm(printed(element)[0])]) };
   }
   if (first.type !== "ArrayExpression") return null;
   const rows = [];
   for (const element of elements) {
     const inner = (element.elements ?? []).filter((x) => x !== null);
-    if (inner.length === 0 || inner.some((x) => x.type !== "StringLiteral")) return null;
+    if (inner.length === 0 || inner.some((x) => printed(x).length === 0)) return null;
     if (inner.length !== (element.elements ?? []).length) return null;
-    rows.push(inner.map((x) => norm(String(x.value))));
+    rows.push(inner.map((x) => norm(printed(x)[0])));
   }
   if (new Set(rows.map((row) => row.length)).size !== 1) return null;
   return { tuple: true, rows };
@@ -5459,7 +5472,8 @@ function overlapping(rewrites) {
   for (const a of rewrites) {
     for (const b of rewrites) {
       if (a.raw !== void 0 || b.raw !== void 0) continue;
-      if (a !== b && a.key !== b.key && a.site.where === "text" && b.site.where === "text" && overlaps(a.site.range, b.site.range)) {
+      if (a === b || a.key === b.key) continue;
+      if (collide(a, b)) {
         out.add(a);
         out.add(b);
       }
@@ -5467,6 +5481,16 @@ function overlapping(rewrites) {
   }
   return [...out];
 }
+var writes = ({ site }) => [site.range, site.attrRange, site.where === "text" ? site.inner : null].filter((r) => r != null);
+function collide(a, b) {
+  const inside = (at, spans) => spans.some((r) => r.start < at && at < r.end);
+  const [wa, wb] = [writes(a), writes(b)];
+  if (wa.some((x) => wb.some((y) => overlaps(x, y)))) return true;
+  if (inside(a.site.attrInsertAt, wb) || inside(b.site.attrInsertAt, wa)) return true;
+  const [fa, fb] = [field(a), field(b)];
+  return a.site.attrInsertAt === b.site.attrInsertAt && !!fa && !!fb && fa !== fb;
+}
+var field = (r) => r.site.where === "prop" || r.site.where === "text" && !ownsSubtree(r.kind, r.site, r.inline) && r.site.mixed && r.site.textRange ? void 0 : declaration(r).plain[0];
 function declaration(rewrite) {
   const { site, path, kind, scope } = rewrite;
   if (site.where === "attr" && kind !== "image") {
@@ -5481,8 +5505,8 @@ function rewriteFile(content, dialect, rewrites, imports, extra = []) {
   const bindings = /* @__PURE__ */ new Map();
   const addDeclaration = (at, add) => {
     const held = attributes.get(at) ?? { plain: [], props: [] };
-    held.plain.push(...add.plain);
-    held.props.push(...add.props);
+    held.plain.push(...add.plain.filter((text) => !held.plain.includes(text)));
+    held.props.push(...add.props.filter((p) => !held.props.some((q) => JSON.stringify(q) === JSON.stringify(p))));
     attributes.set(at, held);
   };
   for (const rewrite of rewrites) {
@@ -5905,7 +5929,8 @@ function attrsOf4(node, content) {
         valueRange: quoted && rawStart >= 0 && attr.value !== "" ? { start: rawStart + (unquoted ? 0 : 1), end: rawStart + raw.length - (unquoted ? 0 : 1) } : null,
         expression: !quoted,
         range,
-        expressionRange: !quoted && open >= 0 && close > open ? { start: open, end: close + 1 } : null
+        expressionRange: !quoted && open >= 0 && close > open ? { start: open, end: close + 1 } : null,
+        ...attr.kind === "expression" && /^\d+(\.\d+)?$/.test(attr.value.trim()) ? { number: attr.value.trim() } : {}
       }
     ];
   });
@@ -6024,6 +6049,9 @@ function attrsOf5(node) {
       return [{ name: `\0${name}`, value: "", valueRange: null, expression: true, expressionRange: null, range }];
     }
     const literal = literalValue(attr);
+    const tag = attr.value;
+    const raw = tag?.type === "ExpressionTag" ? tag.expression : void 0;
+    const digits = typeof raw?.value === "number" ? String(raw.raw ?? "") : "";
     return [
       {
         name,
@@ -6031,7 +6059,8 @@ function attrsOf5(node) {
         valueRange: literal && literal.value !== "" ? literal.range : null,
         expression: literal === null,
         // Not read: svelte sections are DIALECT_UNSUPPORTED. @see componentize.ts
-        expressionRange: null,
+        expressionRange: digits ? { start: tag.start ?? 0, end: tag.end ?? 0 } : null,
+        ...digits ? { number: digits } : {},
         range
       }
     ];
@@ -6136,13 +6165,15 @@ var spanOf = (loc) => ({
 function attrsOf6(node) {
   return (node.props ?? []).flatMap((prop) => {
     if (prop.type === DIRECTIVE && prop.name === "bind" && prop.arg?.content) {
+      const digits = /^\d+(\.\d+)?$/.test(String(prop.exp?.content ?? "").trim()) ? String(prop.exp.content).trim() : "";
       return [
         {
           name: prop.arg.content,
           value: "",
           valueRange: null,
           expression: true,
-          expressionRange: null,
+          expressionRange: digits ? spanOf(prop.exp?.loc) : null,
+          ...digits ? { number: digits } : {},
           range: spanOf(prop.loc)
         }
       ];
@@ -6626,8 +6657,8 @@ function readPlan(value) {
       check.optionalString(section.sectionType, `${at2}.sectionType`);
       check.optionalString(section.copy, `${at2}.copy`);
       if (section.fields !== void 0) {
-        for (const [f, field] of check.array(section.fields, `${at2}.fields`).entries()) {
-          const shape = check.record(field, `${at2}.fields[${f}]`);
+        for (const [f, field2] of check.array(section.fields, `${at2}.fields`).entries()) {
+          const shape = check.record(field2, `${at2}.fields[${f}]`);
           check.optionalString(shape.key, `${at2}.fields[${f}].key`);
           check.optionalString(shape.path, `${at2}.fields[${f}].path`);
         }
@@ -6946,15 +6977,15 @@ function wire(source, candidate, form) {
       source.remove(start, attr.range.end);
     }
   }
-  for (const field of form.fields) {
-    const control = candidate.controls.get(field.key);
+  for (const field2 of form.fields) {
+    const control = candidate.controls.get(field2.key);
     if (candidate.foreign.has(control)) continue;
     const host = candidate.wrapper.get(control) ?? control;
-    if (!attrOf(host, "data-bcms-form-field")) 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(field2.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) {
-      source.overwrite(name.valueRange.start, name.valueRange.end, escapeAttr(field.key));
+    if (!name) source.appendLeft(control.attrInsertAt, ` name="${escapeAttr(field2.key)}"`);
+    else if (!name.expression && name.value !== field2.key && name.valueRange) {
+      source.overwrite(name.valueRange.start, name.valueRange.end, escapeAttr(field2.key));
     }
   }
   if (el.inner && !hasMessage(el)) {
@@ -6998,7 +7029,7 @@ async function wireForms(briefIn, sources, options_ = {}) {
       candidates.push(candidateOf(source.path, dialect, el, imported, landmark, source.content, parsed.importAt));
     }
   }
-  const holds = (c, f) => f.fields.filter((field) => c.controls.has(field.key)).length;
+  const holds = (c, f) => f.fields.filter((field2) => c.controls.has(field2.key)).length;
   const byId = (c, f) => !attrOf(c.el, "data-bcms-form")?.expression && attrOf(c.el, "data-bcms-form")?.value === f.id;
   const programAction = (c) => !!attrOf(c.el, "action")?.expression && calledName(exprOf(c, "action"), "bcmsFormAction") === null;
   const wiredTo = (c, f) => BY_NAME.includes(c.dialect) ? calledName(exprOf(c, "data-bcms-form"), "bcmsFormId") === f.name || byId(c, f) && programAction(c) : byId(c, f);
@@ -7120,9 +7151,9 @@ async function wireForms(briefIn, sources, options_ = {}) {
       pend2(form, "ACTION_IS_EXPRESSION", candidate.file, "its action is an expression, so this run will not replace it.");
       continue;
     }
-    const misnamed = form.fields.find((field) => {
-      const attr = attrOf(candidate.controls.get(field.key), "name");
-      return !!attr && attr.value !== field.key && (attr.expression || !attr.valueRange);
+    const misnamed = form.fields.find((field2) => {
+      const attr = attrOf(candidate.controls.get(field2.key), "name");
+      return !!attr && attr.value !== field2.key && (attr.expression || !attr.valueRange);
     });
     if (misnamed) {
       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.`);
@@ -7202,7 +7233,7 @@ ${indent}${scriptFor(last.candidate.dialect)}`);
     );
   }
   for (const { form, candidate } of matched) {
-    const keys = form.fields.filter((field) => candidate.foreign.has(candidate.controls.get(field.key))).map((field) => field.key);
+    const keys = form.fields.filter((field2) => candidate.foreign.has(candidate.controls.get(field2.key))).map((field2) => field2.key);
     if (keys.length > 0) {
       notes.push(
         `form ${form.name}: ${keys.join(", ")} matched a control inside an imported component, so no data-bcms-form-field marker was written for ${keys.length === 1 ? "it" : "them"}.`
@@ -7664,6 +7695,54 @@ function moduleScope(parsed, content) {
   return out;
 }
 
+// src/build-script.ts
+var CHECK = /^(?:npx\s+|pnpm\s+(?:exec\s+)?|yarn\s+|bunx\s+)?(?:astro\s+check|svelte-check|vue-tsc|tsc(?:\s|$)|eslint|next\s+lint|biome\s+(?:check|lint|ci)|prettier\s+(?:--check|-c)\b|stylelint|oxlint)/;
+var RUN_CHECK = /^(?:npm\s+run|pnpm(?:\s+run)?|yarn(?:\s+run)?|bun\s+run)\s+(?:lint|typecheck|type-check|check|check:types|types|format:check|test)(?::[\w-]+)?\s*$/;
+var UNSAFE = /;|\|\||\||(?:^|[^&])&(?:[^&]|$)|\$\(|`|\n/;
+var isCheck = (step) => CHECK.test(step) || RUN_CHECK.test(step);
+function steps(script) {
+  if (UNSAFE.test(script)) return null;
+  const out = script.split("&&").map((s) => s.trim());
+  return out.some((s) => s === "") ? null : out;
+}
+function bcmsBuildScript(packageJson) {
+  let pkg;
+  try {
+    pkg = JSON.parse(packageJson);
+  } catch {
+    return null;
+  }
+  const scripts = pkg?.scripts;
+  if (!scripts || typeof scripts !== "object" || "build:bcms" in scripts) return null;
+  const build = scripts.build;
+  if (typeof build !== "string") return null;
+  const parts = [];
+  for (const name of ["prebuild", "build", "postbuild"]) {
+    const value = scripts[name];
+    if (value === void 0) continue;
+    if (typeof value !== "string") return null;
+    const s = steps(value);
+    if (!s) return null;
+    parts.push(s);
+  }
+  const all = parts.flat();
+  const dropped = all.filter(isCheck);
+  const kept = all.filter((s) => !isCheck(s));
+  if (dropped.length === 0 || kept.length === 0) return null;
+  const script = kept.join(" && ");
+  const next = {};
+  for (const [key, value] of Object.entries(scripts)) {
+    next[key] = value;
+    if (key === "build") next["build:bcms"] = script;
+  }
+  const indent = /^[ \t]+(?=")/m.exec(packageJson)?.[0] ?? "  ";
+  const eol = packageJson.includes("\r\n") ? "\r\n" : "\n";
+  let content = JSON.stringify({ ...pkg, scripts: next }, null, indent);
+  if (eol !== "\n") content = content.replace(/\n/g, eol);
+  if (/\r?\n$/.test(packageJson)) content += eol;
+  return { content, script, dropped };
+}
+
 // src/componentize.ts
 import MagicString5 from "magic-string";
 var SectionInvariantError = class extends Error {
@@ -8413,7 +8492,7 @@ var inSection = (section, path) => {
   const target = rowless(path);
   return fieldPaths(section).some((fieldPath) => rowless(fieldPath) === target);
 };
-var fieldPaths = (section) => (section.fields ?? []).map((field) => field.path ?? `${section.groupKey}.${field.key ?? ""}`);
+var fieldPaths = (section) => (section.fields ?? []).map((field2) => field2.path ?? `${section.groupKey}.${field2.key ?? ""}`);
 function elementIndex(roots) {
   const out = /* @__PURE__ */ new Map();
   const visit = (node, chain) => {
@@ -8571,8 +8650,8 @@ function componentFile(dialect, path, fromFile, content, parsed, resolved, pathP
   const carriedDeclarations = carry.declarations.length > 0 ? `${carry.declarations.join("\n")}
 ` : "";
   const copy = !snapshot ? "" : flat3 ? `const ${snapshot} = bcmsSectionPaths(page, bind, overrides, ${JSON.stringify(pathPrefix)}, ${JSON.stringify(pathMap)}, bindPaths);` : `const ${snapshot} = { ${key}: bcmsSection(page, bind, overrides, bindPaths) };`;
-  const field = bound > 0 ? `const ${fieldName} = ${fieldsName}({ blockId, bind, bindPaths, overrides }, ${JSON.stringify(pathPrefix)}${flat3 ? `, ${JSON.stringify(pathMap)}` : ""});` : "";
-  const declared = [copy, field].filter(Boolean).join("\n");
+  const field2 = bound > 0 ? `const ${fieldName} = ${fieldsName}({ blockId, bind, bindPaths, overrides }, ${JSON.stringify(pathPrefix)}${flat3 ? `, ${JSON.stringify(pathMap)}` : ""});` : "";
+  const declared = [copy, field2].filter(Boolean).join("\n");
   const styles = dialect === "astro" ? carriedStyles(content) : "";
   if (dialect === "astro") {
     return { free: [], duplicates: [], unconvertible: [], content: `---
@@ -8885,8 +8964,8 @@ function propPrefix(section) {
 }
 function exactPrefix(section) {
   const prefixes = /* @__PURE__ */ new Set();
-  for (const field of section.fields ?? []) {
-    const { path, key } = field;
+  for (const field2 of section.fields ?? []) {
+    const { path, key } = field2;
     if (typeof path !== "string" || typeof key !== "string") continue;
     if (!path.endsWith(key)) return null;
     prefixes.add(path.slice(0, path.length - key.length));
@@ -8898,11 +8977,11 @@ var foldKey = (remainder) => remainder.replace(/\[(\d+)\]/g, "_$1").replace(/[^a
 function flatPrefix(section) {
   const prefixes = /* @__PURE__ */ new Set();
   let informative = false;
-  for (const field of section.fields ?? []) {
-    const { path, key } = field;
+  for (const field2 of section.fields ?? []) {
+    const { path, key } = field2;
     if (typeof path !== "string" || typeof key !== "string") continue;
     informative = true;
-    const prefix = ["-", "_"].map((sep2) => `${section.groupKey}${sep2}`).find((candidate) => path.startsWith(candidate) && (typeof field.overrideKey === "string" || foldKey(path.slice(candidate.length)) === key));
+    const prefix = ["-", "_"].map((sep2) => `${section.groupKey}${sep2}`).find((candidate) => path.startsWith(candidate) && (typeof field2.overrideKey === "string" || foldKey(path.slice(candidate.length)) === key));
     if (!prefix) return null;
     prefixes.add(prefix);
   }
@@ -8911,15 +8990,15 @@ function flatPrefix(section) {
 function flatPathMap(section, prefix) {
   const scalars = {};
   const tables = {};
-  for (const field of section.fields ?? []) {
-    const { path, key } = field;
+  for (const field2 of section.fields ?? []) {
+    const { path, key } = field2;
     if (typeof path !== "string" || typeof key !== "string" || !path.startsWith(prefix)) continue;
     const row = /^([^[]+)\[\d+\]\.(.+)$/.exec(path);
     if (!row) {
-      scalars[path] = field.overrideKey ?? key;
+      scalars[path] = field2.overrideKey ?? key;
       continue;
     }
-    const address = typeof field.overrideKey === "string" ? /^([^[]+)\[\d+\]\.(.+)$/.exec(field.overrideKey) : null;
+    const address = typeof field2.overrideKey === "string" ? /^([^[]+)\[\d+\]\.(.+)$/.exec(field2.overrideKey) : null;
     if (!address) return null;
     const table = tables[row[1]] ??= { key: address[1], columns: {} };
     if (table.key !== address[1]) return null;
@@ -10784,7 +10863,7 @@ function styleEdits(before, after) {
 // package.json
 var package_default = {
   name: "@bettercms-ai/convert",
-  version: "0.24.1",
+  version: "0.25.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: {
@@ -10817,7 +10896,7 @@ var package_default = {
   dependencies: {
     "@astrojs/compiler": "^2.13.1",
     "@babel/parser": "^7.29.2",
-    "@bettercms-ai/types": "^1.30.0",
+    "@bettercms-ai/types": "^1.32.0",
     "@vue/compiler-sfc": "^3.5.0",
     "magic-string": "^0.30.21",
     parse5: "^7.3.0",
@@ -10961,7 +11040,7 @@ async function readTree(root, verbose) {
       const full = join2(dir, entry.name);
       const rel = relative(root, full).split(sep).join("/");
       if (entry.isDirectory()) {
-        if (isSourceCandidate(`${rel}/x.html`)) await walk(full);
+        if (isSourceCandidate(`${rel}/x.html`) && !existsSync(join2(full, ".git"))) await walk(full);
         continue;
       }
       if (!entry.isFile() || !isSourceCandidate(rel)) continue;
@@ -11142,6 +11221,21 @@ async function verifyStyles(root, base, verbose) {
   );
   return 1;
 }
+async function addBcmsBuild(root, dryRun) {
+  let packageJson;
+  try {
+    packageJson = await readFile(join2(root, "package.json"), "utf-8");
+  } catch {
+    return;
+  }
+  const added = bcmsBuildScript(packageJson);
+  if (!added) return;
+  if (!dryRun) await writeFile(join2(root, "package.json"), added.content, "utf-8");
+  process.stderr.write(
+    `package.json: ${dryRun ? "would add" : "added"} "build:bcms": "${added.script}" \u2014 BetterCMS builds with it, without ${added.dropped.join(", ")}; "build" is unchanged.
+`
+  );
+}
 async function main() {
   const args = parseArgs(process.argv.slice(2));
   if (args.help) {
@@ -11173,6 +11267,7 @@ async function main() {
         await writeFile(target, file.content, "utf-8");
       }
     }
+    await addBcmsBuild(root2, args.dryRun);
     if (args.receipt) await writeFile(resolve(args.receipt), `${JSON.stringify(receipt2, null, 2)}
 `, "utf-8");
     reportUpgrade(files2, receipt2, args.dryRun);
@@ -11200,12 +11295,14 @@ async function main() {
   if (args.verbose) process.stderr.write(`read ${sources.length} candidate files under ${root}
 `);
   const written = async (files2) => {
-    if (args.dryRun) return;
-    for (const file of files2) {
-      const target = under(root, file.path);
-      await mkdir(dirname(target), { recursive: true });
-      await writeFile(target, file.content, "utf-8");
+    if (!args.dryRun) {
+      for (const file of files2) {
+        const target = under(root, file.path);
+        await mkdir(dirname(target), { recursive: true });
+        await writeFile(target, file.content, "utf-8");
+      }
     }
+    await addBcmsBuild(root, args.dryRun);
   };
   const saveReceipt = async (receipt2) => {
     if (!args.receipt) return;