@cmssy/codemod 7.0.0 → 8.0.0

package/dist/index.js

@@ -162,8 +162,62 @@ function transform2(source) {
   return { code, changed };
 }
 
+// src/v8.ts
+var HAND_TYPED_CONTENT = /content\s*:\s*(Record<string,\s*unknown>|\{[^}]*\}|[A-Z]\w*(Content|Props))/;
+function stripTypeArguments(source) {
+  const marker = "defineBlock<";
+  let code = source;
+  let changed = false;
+  for (let start = code.indexOf(marker); start !== -1; ) {
+    const open = start + marker.length - 1;
+    let depth = 0;
+    let end = -1;
+    for (let i = open; i < code.length; i++) {
+      const char = code[i];
+      if (char === "<") depth++;
+      else if (char === ">") {
+        depth--;
+        if (depth === 0) {
+          end = i;
+          break;
+        }
+      } else if (char === "(" || char === ";" || char === "\n") {
+        break;
+      }
+    }
+    if (end === -1) {
+      start = code.indexOf(marker, start + marker.length);
+      continue;
+    }
+    code = code.slice(0, open) + code.slice(end + 1);
+    changed = true;
+    start = code.indexOf(marker);
+  }
+  return { code, changed };
+}
+function transform3(source) {
+  const { code, changed } = stripTypeArguments(source);
+  const notes = [];
+  const usesBlockProps = /\bBlockProps\s*</.test(code);
+  if (/\bdefineBlock\s*[<(]/.test(code) && !usesBlockProps) {
+    notes.push(
+      "declares a block - type its component as BlockProps<typeof props>"
+    );
+  } else if (HAND_TYPED_CONTENT.test(code) && !usesBlockProps) {
+    notes.push(
+      "types a block's content by hand - derive it from the schema instead"
+    );
+  }
+  return notes.length > 0 ? { code, changed, notes } : { code, changed };
+}
+
 // src/index.ts
-var TRANSFORMS = { v5: transform, v7: transform2 };
+var TRANSFORMS = { v5: transform, v7: transform2, v8: transform3 };
+var PREVIOUS_MAJOR = {
+  v5: "4.x",
+  v7: "6.x",
+  v8: "7.x"
+};
 var SKIP = /* @__PURE__ */ new Set(["node_modules", "dist", "build", "out", "coverage"]);
 var EXTENSIONS = [".ts", ".tsx", ".js", ".jsx", ".mjs"];
 function skipDirectory(name) {
@@ -185,9 +239,9 @@ async function sourceFiles(dir) {
 async function main() {
   const args = process.argv.slice(2);
   const version = args[0];
-  const transform3 = TRANSFORMS[version];
-  if (!transform3) {
-    console.error("usage: cmssy-codemod v5|v7 [path] [--dry]");
+  const transform4 = TRANSFORMS[version];
+  if (!transform4) {
+    console.error("usage: cmssy-codemod v5|v7|v8 [path] [--dry]");
     process.exitCode = 1;
     return;
   }
@@ -197,24 +251,44 @@ async function main() {
   );
   const files = await sourceFiles(target);
   const touched = [];
+  const manual = [];
   let needsCore = false;
   for (const file of files) {
     const source = await readFile(file, "utf8");
-    const { code, changed } = transform3(source);
+    const { code, changed, notes } = transform4(source);
+    if (notes && notes.length > 0) manual.push({ file, notes });
     if (!changed) continue;
     touched.push(file);
     if (code.includes('from "@cmssy/core"')) needsCore = true;
     if (!dry) await writeFile(file, code);
   }
-  if (touched.length === 0) {
-    console.log("cmssy: nothing to migrate - no 4.x imports found.");
+  const report = (file) => file.slice(target.length + 1);
+  if (touched.length === 0 && manual.length === 0) {
+    console.log(
+      `cmssy: nothing to migrate - no ${PREVIOUS_MAJOR[version]} imports found.`
+    );
     return;
   }
-  console.log(
-    `cmssy: ${dry ? "would rewrite" : "rewrote"} ${touched.length} file(s):`
-  );
-  for (const file of touched) {
-    console.log(`  ${file.slice(target.length + 1)}`);
+  if (touched.length > 0) {
+    console.log(
+      `cmssy: ${dry ? "would rewrite" : "rewrote"} ${touched.length} file(s):`
+    );
+    for (const file of touched) console.log(`  ${report(file)}`);
+  }
+  if (manual.length > 0) {
+    console.log(
+      `
+cmssy: ${manual.length} file(s) need a human - a block's content must be
+derived from its schema, and only you know which fields it means to read:
+`
+    );
+    for (const { file, notes } of manual) {
+      console.log(`  ${report(file)}
+    ${notes.join("\n    ")}`);
+    }
+    console.log(
+      "\n  https://github.com/cmssy-io/cmssy-sdk/blob/main/docs/migrations/v7-to-v8.md"
+    );
   }
   if (needsCore && !await dependsOnCore(target)) {
     console.log(

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cmssy/codemod",
-  "version": "7.0.0",
+  "version": "8.0.0",
   "description": "Rewrites cmssy imports across major versions, so a breaking release costs minutes instead of an afternoon.",
   "keywords": [
     "cmssy",