@decocms/start 0.36.3 → 0.36.5

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@decocms/start",
-  "version": "0.36.3",
+  "version": "0.36.5",
   "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

@@ -72,6 +72,24 @@ export function transform(ctx: MigrationContext): void {
       }
     }
 
+    // Flag files with HTMX patterns for manual React migration
+    if (/\bhx-(?:get|post|put|delete|trigger|target|swap|on|indicator|sync|select)\b/.test(result.content)) {
+      ctx.manualReviewItems.push({
+        file: targetPath,
+        reason: "HTMX attributes (hx-*) found — needs manual migration to React state/effects. HTMX server-side rendering (hx-get/hx-post with useSection) must be converted to React components with useState/useEffect or server functions.",
+        severity: "warning",
+      });
+    }
+
+    // Flag files with hx-on:click that use useScript (simpler pattern)
+    if (/hx-on:click=\{useScript/.test(result.content)) {
+      ctx.manualReviewItems.push({
+        file: targetPath,
+        reason: "hx-on:click with useScript found — convert to onClick with React event handler. The useScript serialization won't work as onClick value.",
+        severity: "warning",
+      });
+    }
+
     if (ctx.dryRun) {
       if (result.changed) {
         log(ctx, `[DRY] Would transform: ${record.path} → ${targetPath}`);

package/scripts/migrate/phase-verify.ts

@@ -325,6 +325,20 @@ const checks: Check[] = [
       return true;
     },
   },
+  {
+    name: "No HTMX attributes (hx-*) in components",
+    severity: "warning",
+    fn: (ctx) => {
+      const srcDir = path.join(ctx.sourceDir, "src");
+      if (!fs.existsSync(srcDir)) return true;
+      const bad = findFilesWithPattern(srcDir, /\bhx-(?:get|post|put|delete|patch|trigger|target|swap|on|indicator|sync|select)\b/);
+      if (bad.length > 0) {
+        console.log(`    HTMX attributes found (needs manual React migration): ${bad.join(", ")}`);
+        return false;
+      }
+      return true;
+    },
+  },
 ];
 
 function findFilesWithPattern(

package/scripts/migrate/transforms/deno-isms.ts

@@ -38,6 +38,13 @@ export function transformDenoIsms(content: string): TransformResult {
     notes.push("Removed npm: prefix from imports");
   }
 
+  // @ts-ignore → @ts-expect-error (TypeScript 5+ prefers @ts-expect-error)
+  if (/@ts-ignore/.test(result)) {
+    result = result.replace(/@ts-ignore/g, "@ts-expect-error");
+    changed = true;
+    notes.push("Replaced @ts-ignore with @ts-expect-error");
+  }
+
   // Remove Deno.* API usages — flag for manual review
   if (result.includes("Deno.")) {
     notes.push("MANUAL: Deno.* API usage found — needs Node.js equivalent");

package/scripts/migrate/transforms/jsx.ts

@@ -171,6 +171,55 @@ export function transformJsx(content: string): TransformResult {
     notes.push("Replaced 'class' in interface definitions with 'className'");
   }
 
+  // Remove `alt` prop from non-img elements (<a>, <iframe>, <div>, etc.)
+  // In React, `alt` is only valid on <img>, <input type="image">, <area>
+  const altOnNonImgRegex = /(<(?:a|iframe|div|span|button|section)\s[^>]*?)\s+alt=(?:\{[^}]*\}|"[^"]*"|'[^']*')/g;
+  if (altOnNonImgRegex.test(result)) {
+    result = result.replace(
+      /(<(?:a|iframe|div|span|button|section)\s[^>]*?)\s+alt=(?:\{[^}]*\}|"[^"]*"|'[^']*')/g,
+      "$1",
+    );
+    changed = true;
+    notes.push("Removed invalid alt prop from non-img elements");
+  }
+
+  // Remove `type` prop from non-form elements (<span>, <div>, etc.)
+  // In React, `type` is only valid on <input>, <button>, <select>, <textarea>, <script>, <style>, <link>
+  const typeOnInvalidRegex = /(<(?:span|div|p|section|header|footer|nav|main|article|aside)\s[^>]*?)\s+type=(?:\{[^}]*\}|"[^"]*"|'[^']*')/g;
+  if (typeOnInvalidRegex.test(result)) {
+    result = result.replace(
+      /(<(?:span|div|p|section|header|footer|nav|main|article|aside)\s[^>]*?)\s+type=(?:\{[^}]*\}|"[^"]*"|'[^']*')/g,
+      "$1",
+    );
+    changed = true;
+    notes.push("Removed invalid type prop from non-form elements");
+  }
+
+  // setTimeout/setInterval return type: use window.setTimeout for correct typing
+  // In Node/CF Workers, setTimeout returns Timeout object, not number
+  // window.setTimeout always returns number
+  if (/\bsetTimeout\b/.test(result) && /:\s*number/.test(result)) {
+    // Only replace bare setTimeout when it's assigned to a typed variable
+    result = result.replace(
+      /\b(?<!window\.)setTimeout\(/g,
+      "window.setTimeout(",
+    );
+    result = result.replace(
+      /\b(?<!window\.)setInterval\(/g,
+      "window.setInterval(",
+    );
+    result = result.replace(
+      /\b(?<!window\.)clearTimeout\(/g,
+      "window.clearTimeout(",
+    );
+    result = result.replace(
+      /\b(?<!window\.)clearInterval\(/g,
+      "window.clearInterval(",
+    );
+    changed = true;
+    notes.push("Prefixed setTimeout/setInterval with window. for correct typing");
+  }
+
   // Ensure React import exists if we introduced React.* references
   if (
     (result.includes("React.") || result.includes("React,")) &&

package/src/apps/autoconfig.ts

@@ -25,7 +25,7 @@ interface AppAutoconfigurator {
 }
 
 const KNOWN_APPS: Record<string, AppAutoconfigurator> = {
-	"deco-resend": async (block: any) => {
+	"deco-resend": async (block: any): Promise<Record<string, InvokeAction>> => {
 		try {
 			const [resendClient, resendActions] = await Promise.all([
 				import("@decocms/apps/resend/client" as string),
@@ -40,7 +40,7 @@ const KNOWN_APPS: Record<string, AppAutoconfigurator> = {
 					"[autoconfig] deco-resend: no API key found." +
 					" Set DECO_CRYPTO_KEY to decrypt CMS secrets, or set RESEND_API_KEY as fallback.",
 				);
-				return {};
+				return {} as Record<string, InvokeAction>;
 			}
 
 			configureResend({