@bettercms-ai/convert 0.24.1 → 0.25.0

package/CHANGELOG.md

@@ -1,5 +1,15 @@
 # @bettercms-ai/convert
 
+## 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";
 
@@ -7664,6 +7665,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 {
@@ -10784,7 +10833,7 @@ function styleEdits(before, after) {
 // package.json
 var package_default = {
   name: "@bettercms-ai/convert",
-  version: "0.24.1",
+  version: "0.25.0",
   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: {
@@ -10961,7 +11010,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 +11191,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 +11237,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 +11265,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;