@decocms/start 0.33.0 → 0.34.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@decocms/start",
-  "version": "0.33.0",
+  "version": "0.34.0",
   "type": "module",
   "description": "Deco framework for TanStack Start - CMS bridge, admin protocol, hooks, schema generation",
   "main": "./src/index.ts",

package/scripts/migrate/phase-transform.ts

@@ -7,6 +7,7 @@ import { transformJsx } from "./transforms/jsx.ts";
 import { transformFreshApis } from "./transforms/fresh-apis.ts";
 import { transformDenoIsms } from "./transforms/deno-isms.ts";
 import { transformTailwind } from "./transforms/tailwind.ts";
+import { transformDeadCode } from "./transforms/dead-code.ts";
 
 /**
  * Apply all transforms to a file's content in the correct order.
@@ -22,11 +23,12 @@ function applyTransforms(content: string, filePath: string): TransformResult {
     return { content, changed: false, notes: [] };
   }
 
-  // Pipeline: imports → jsx → fresh-apis → deno-isms → tailwind
+  // Pipeline: imports → jsx → fresh-apis → dead-code → deno-isms → tailwind
   const pipeline = [
     { name: "imports", fn: transformImports },
     { name: "jsx", fn: transformJsx },
     { name: "fresh-apis", fn: transformFreshApis },
+    { name: "dead-code", fn: transformDeadCode },
     { name: "deno-isms", fn: transformDenoIsms },
     { name: "tailwind", fn: transformTailwind },
   ];

package/scripts/migrate/phase-verify.ts

@@ -260,6 +260,67 @@ const checks: Check[] = [
       return true;
     },
   },
+  {
+    name: "No dead cache/cacheKey/loader exports",
+    severity: "warning",
+    fn: (ctx) => {
+      const srcDir = path.join(ctx.sourceDir, "src");
+      if (!fs.existsSync(srcDir)) return true;
+      const bad = findFilesWithPattern(srcDir, /^export\s+const\s+(?:cache|cacheKey|loader)\s*=/m);
+      if (bad.length > 0) {
+        console.log(`    Dead exports found (old cache/loader system): ${bad.join(", ")}`);
+        return false;
+      }
+      return true;
+    },
+  },
+  {
+    name: "No invoke.resend calls (should use sendEmail)",
+    severity: "warning",
+    fn: (ctx) => {
+      const srcDir = path.join(ctx.sourceDir, "src");
+      if (!fs.existsSync(srcDir)) return true;
+      const bad = findFilesWithPattern(srcDir, /invoke\.resend\./);
+      if (bad.length > 0) {
+        console.log(`    Still uses invoke.resend: ${bad.join(", ")}`);
+        return false;
+      }
+      return true;
+    },
+  },
+  {
+    name: "No Deno-only crypto.subtle.digestSync",
+    severity: "warning",
+    fn: (ctx) => {
+      const srcDir = path.join(ctx.sourceDir, "src");
+      if (!fs.existsSync(srcDir)) return true;
+      const bad = findFilesWithPattern(srcDir, /digestSync/);
+      if (bad.length > 0) {
+        console.log(`    Deno-only digestSync found: ${bad.join(", ")}`);
+        return false;
+      }
+      return true;
+    },
+  },
+  {
+    name: ".gitignore exists with new stack entries",
+    severity: "warning",
+    fn: (ctx) => {
+      const gitignorePath = path.join(ctx.sourceDir, ".gitignore");
+      if (!fs.existsSync(gitignorePath)) {
+        console.log("    .gitignore missing");
+        return false;
+      }
+      const content = fs.readFileSync(gitignorePath, "utf-8");
+      const required = [".wrangler/", "node_modules/", ".tanstack/", "src/routeTree.gen.ts"];
+      const missing = required.filter((r) => !content.includes(r));
+      if (missing.length > 0) {
+        console.log(`    .gitignore missing entries: ${missing.join(", ")}`);
+        return false;
+      }
+      return true;
+    },
+  },
 ];
 
 function findFilesWithPattern(

package/scripts/migrate/transforms/dead-code.ts

@@ -0,0 +1,103 @@
+import type { TransformResult } from "../types.ts";
+
+/**
+ * Removes dead code patterns from the old Deco stack that don't work
+ * in TanStack Start:
+ *
+ * - `export const cache = "stale-while-revalidate"` (old cache system)
+ * - `export const cacheKey = ...` (old cache key generation)
+ * - `export const loader = (props, req, ctx) => ...` (old section loader pattern)
+ * - `crypto.subtle.digestSync(...)` (Deno-only sync API)
+ * - `invoke.resend.actions.emails.send(...)` → `sendEmail(...)` from @decocms/apps
+ */
+export function transformDeadCode(content: string): TransformResult {
+  const notes: string[] = [];
+  let changed = false;
+  let result = content;
+
+  // Remove old cache export: export const cache = "stale-while-revalidate";
+  if (/^export\s+const\s+cache\s*=\s*["'][^"']*["']/m.test(result)) {
+    result = result.replace(
+      /^export\s+const\s+cache\s*=\s*["'][^"']*["'];?\s*\n?/gm,
+      "",
+    );
+    changed = true;
+    notes.push("Removed dead `export const cache` (old caching system)");
+  }
+
+  // Remove old cacheKey export (can be multiline)
+  if (/^export\s+const\s+cacheKey\s*=/m.test(result)) {
+    // Try to remove the entire cacheKey function — find matching closing brace/semicolon
+    result = result.replace(
+      /^export\s+const\s+cacheKey\s*=\s*\([^)]*\)\s*(?::\s*\w+\s*)?=>\s*\{[\s\S]*?\n\};\s*\n?/gm,
+      "",
+    );
+    // Also handle simpler inline forms
+    result = result.replace(
+      /^export\s+const\s+cacheKey\s*=[^;]*;\s*\n?/gm,
+      "",
+    );
+    changed = true;
+    notes.push("Removed dead `export const cacheKey` (old caching system)");
+  }
+
+  // Remove old section loader export: export const loader = (props, req, ctx) => { ... };
+  // This is the old pattern where sections had co-located loaders.
+  // In TanStack Start, section loaders are handled differently.
+  if (/^export\s+const\s+loader\s*=\s*\(/m.test(result)) {
+    result = result.replace(
+      /^export\s+const\s+loader\s*=\s*\([^)]*\)\s*(?::\s*[\w<>[\]|&\s]+)?\s*=>\s*\{[\s\S]*?\n\};\s*\n?/gm,
+      "",
+    );
+    // Also handle simpler inline forms
+    result = result.replace(
+      /^export\s+const\s+loader\s*=\s*\([^)]*\)\s*=>[^;]*;\s*\n?/gm,
+      "",
+    );
+    changed = true;
+    notes.push("Removed dead `export const loader` (old section loader — use section loaders in @decocms/start)");
+  }
+
+  // Replace crypto.subtle.digestSync (Deno-only) with a note
+  if (result.includes("digestSync")) {
+    result = result.replace(
+      /crypto\.subtle\.digestSync\(/g,
+      "/* MIGRATION: digestSync is Deno-only, use await crypto.subtle.digest( */ crypto.subtle.digest(",
+    );
+    changed = true;
+    notes.push("MANUAL: crypto.subtle.digestSync is Deno-only — replaced with crypto.subtle.digest (needs await)");
+  }
+
+  // Migrate invoke.resend.actions.emails.send → sendEmail from @decocms/apps
+  if (result.includes("invoke.resend.actions.emails.send")) {
+    // Replace the invoke call with sendEmail
+    result = result.replace(
+      /(?:await\s+)?invoke\.resend\.actions\.emails\.send\(/g,
+      "await sendEmail(",
+    );
+
+    // Remove the old runtime import if only used for resend
+    result = result.replace(
+      /^import\s+\{\s*invoke\s*\}\s+from\s+["'][^"']*runtime["'];?\s*\n?/gm,
+      "",
+    );
+
+    // Add the sendEmail import if not present
+    if (!result.includes('"@decocms/apps/resend/actions/send"')) {
+      result = `import { sendEmail } from "@decocms/apps/resend/actions/send";\n${result}`;
+    }
+
+    changed = true;
+    notes.push("Migrated invoke.resend.actions.emails.send → sendEmail from @decocms/apps/resend");
+  }
+
+  // Generic invoke.X pattern — flag for manual review
+  if (/\binvoke\.\w+\.\w+/.test(result) && !result.includes("sendEmail")) {
+    notes.push("MANUAL: invoke.* calls found — need manual migration to server functions or @decocms/apps actions");
+  }
+
+  // Clean up blank lines
+  result = result.replace(/\n{3,}/g, "\n\n");
+
+  return { content: result, changed, notes };
+}