@decocms/start 2.6.0 → 2.7.0

package/.agents/skills/deco-to-tanstack-migration/SKILL.md

@@ -136,7 +136,7 @@ See: `references/deco-framework/README.md`
 
 **Actions**:
 1. `from "apps/commerce/types.ts"` → `from "@decocms/apps/commerce/types"`
-2. `from "apps/admin/widgets.ts"` → `from "~/types/widgets"` (create local file)
+2. `from "apps/admin/widgets.ts"` → `from "@decocms/start/types/widgets"` (framework owns the string aliases — do **not** create a local `src/types/widgets.ts`)
 3. `from "apps/website/components/Image.tsx"` → `from "~/components/ui/Image"` (create local)
 4. SDK utilities: `~/sdk/useOffer` → `@decocms/apps/commerce/sdk/useOffer`, etc.
 

package/.agents/skills/deco-to-tanstack-migration/references/post-migration-cleanup.md

@@ -135,7 +135,38 @@ utility). Your runtime behavior gets MUCH better — segment cookies, IS
 cookies, VTEX session auth all start working again instead of being
 silently stubbed to `{}` / `null`.
 
-## 6. Search for orphan `TODO: move into framework` comments
+## 6. Drop `src/types/widgets.ts` — framework owns it
+
+Older migrations scaffold a local `src/types/widgets.ts` containing 8
+string-aliased widget types (`ImageWidget`, `HTMLWidget`, …). The
+framework now exports the same set (plus `TextArea`) at
+`@decocms/start/types/widgets`, and the schema generator detects the
+widgets via type-text matching, so the local file is purely
+duplicated boilerplate.
+
+```bash
+# Quick check
+rg -n "from ['\"]~/types/widgets['\"]" src/ | wc -l   # >0 → cleanup applies
+```
+
+Replace all imports in one pass:
+
+```bash
+# macOS / BSD sed: drop the empty quotes after -i
+rg -l "from ['\"]~/types/widgets['\"]" src/ \
+  | xargs sed -i '' "s|from ['\"]~/types/widgets['\"]|from \"@decocms/start/types/widgets\"|g"
+```
+
+Then delete the now-orphan local file:
+
+```bash
+rm src/types/widgets.ts
+```
+
+Confirm `tsc --noEmit` is still clean — the framework version is a
+strict superset of what the migration script generated.
+
+## 7. Search for orphan `TODO: move into framework` comments
 
 Real sites accumulate `TODO` comments like `// TODO: move into decoVitePlugin
 in next @decocms/start release`. These are roadmap items the framework
@@ -152,7 +183,7 @@ For each hit, decide:
 
 ## Verification checklist
 
-After completing 1-6:
+After completing 1-7:
 
 - [ ] `npm run typecheck` baseline matches pre-cleanup count (no new errors)
 - [ ] `npm run dev` starts and `/`, `/some-pdp/p`, `/s?q=foo` all render

package/.cursor/skills/deco-to-tanstack-migration/SKILL.md

@@ -218,7 +218,7 @@ See: `references/deco-framework/README.md`
 
 **Actions**:
 1. `from "apps/commerce/types.ts"` → `from "@decocms/apps/commerce/types"`
-2. `from "apps/admin/widgets.ts"` → `from "~/types/widgets"` (create local file with string aliases)
+2. `from "apps/admin/widgets.ts"` → `from "@decocms/start/types/widgets"` (framework owns the string aliases — `ImageWidget`, `HTMLWidget`, etc.; do **not** create a local `src/types/widgets.ts`)
 3. `from "apps/website/components/Image.tsx"` → `from "~/components/ui/Image"` (create local components)
 4. SDK utilities: `~/sdk/useOffer` → `@decocms/apps/commerce/sdk/useOffer`, `~/sdk/format` → `@decocms/apps/commerce/sdk/formatPrice`, etc.
 

package/.cursor/skills/deco-to-tanstack-migration/references/post-migration-cleanup.md

@@ -135,7 +135,38 @@ utility). Your runtime behavior gets MUCH better — segment cookies, IS
 cookies, VTEX session auth all start working again instead of being
 silently stubbed to `{}` / `null`.
 
-## 6. Search for orphan `TODO: move into framework` comments
+## 6. Drop `src/types/widgets.ts` — framework owns it
+
+Older migrations scaffold a local `src/types/widgets.ts` containing 8
+string-aliased widget types (`ImageWidget`, `HTMLWidget`, …). The
+framework now exports the same set (plus `TextArea`) at
+`@decocms/start/types/widgets`, and the schema generator detects the
+widgets via type-text matching, so the local file is purely
+duplicated boilerplate.
+
+```bash
+# Quick check
+rg -n "from ['\"]~/types/widgets['\"]" src/ | wc -l   # >0 → cleanup applies
+```
+
+Replace all imports in one pass:
+
+```bash
+# macOS / BSD sed: drop the empty quotes after -i
+rg -l "from ['\"]~/types/widgets['\"]" src/ \
+  | xargs sed -i '' "s|from ['\"]~/types/widgets['\"]|from \"@decocms/start/types/widgets\"|g"
+```
+
+Then delete the now-orphan local file:
+
+```bash
+rm src/types/widgets.ts
+```
+
+Confirm `tsc --noEmit` is still clean — the framework version is a
+strict superset of what the migration script generated.
+
+## 7. Search for orphan `TODO: move into framework` comments
 
 Real sites accumulate `TODO` comments like `// TODO: move into decoVitePlugin
 in next @decocms/start release`. These are roadmap items the framework
@@ -152,7 +183,7 @@ For each hit, decide:
 
 ## Verification checklist
 
-After completing 1-6:
+After completing 1-7:
 
 - [ ] `npm run typecheck` baseline matches pre-cleanup count (no new errors)
 - [ ] `npm run dev` starts and `/`, `/some-pdp/p`, `/s?q=foo` all render

package/package.json

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

package/scripts/migrate/phase-verify.ts

@@ -30,7 +30,9 @@ const REQUIRED_FILES = [
   "src/hooks/useCart.ts",
   "src/hooks/useUser.ts",
   "src/hooks/useWishlist.ts",
-  "src/types/widgets.ts",
+  // src/types/widgets.ts intentionally omitted — provided by the
+  // framework at `@decocms/start/types/widgets`; sites no longer
+  // shadow the file locally.
   "src/types/deco.ts",
   "src/types/commerce-app.ts",
   "src/components/ui/Image.tsx",
@@ -442,7 +444,8 @@ const checks: Check[] = [
     severity: "warning",
     fn: (ctx) => {
       const typeFiles = [
-        "src/types/widgets.ts",
+        // widgets.ts is provided by @decocms/start/types/widgets, not
+        // scaffolded locally.
         "src/types/deco.ts",
         "src/types/commerce-app.ts",
       ];

package/scripts/migrate/templates/types-gen.ts

@@ -3,15 +3,12 @@ import type { MigrationContext } from "../types";
 export function generateTypeFiles(ctx: MigrationContext): Record<string, string> {
   const files: Record<string, string> = {};
 
-  files["src/types/widgets.ts"] = `export type ImageWidget = string;
-export type HTMLWidget = string;
-export type VideoWidget = string;
-export type TextWidget = string;
-export type RichText = string;
-export type Secret = string;
-export type Color = string;
-export type ButtonWidget = string;
-`;
+  // src/types/widgets.ts is no longer generated — the framework owns these
+  // string aliases (`ImageWidget`, `HTMLWidget`, …) at
+  // `@decocms/start/types/widgets`, and `transforms/imports.ts` rewrites
+  // `apps/admin/widgets.ts` directly to that path. Schema generation
+  // works the same way: the generator matches by type *text*, not module
+  // identity (see scripts/generate-schema.ts:WIDGET_TYPE_FORMATS).
 
   files["src/types/deco.ts"] = `export type SectionProps<T extends (...args: any[]) => any> = Awaited<
   ReturnType<T>

package/scripts/migrate/transforms/imports.ts

@@ -29,7 +29,12 @@ const IMPORT_RULES: Array<[RegExp, string | null]> = [
   [/^"@deco\/deco"$/, `"~/types/deco"`],
 
   // Apps — widgets & components
-  [/^"apps\/admin\/widgets\.ts"$/, `"~/types/widgets"`],
+  // Widget aliases (ImageWidget, HTMLWidget, ...) are framework-owned —
+  // every site has the same type set, and the schema generator detects
+  // them via type-text matching, not module identity. Re-export from
+  // @decocms/start/types/widgets so we don't keep a duplicated 8-line
+  // file in every site.
+  [/^"apps\/admin\/widgets\.ts"$/, `"@decocms/start/types/widgets"`],
   [/^"apps\/website\/components\/Image\.tsx"$/, `"~/components/ui/Image"`],
   [/^"apps\/website\/components\/Picture\.tsx"$/, `"~/components/ui/Picture"`],
   [/^"apps\/website\/components\/Video\.tsx"$/, `"~/components/ui/Video"`],