@dreamboard-games/workspace-codegen 0.1.1 → 0.1.3

package/dist/manifest-contract.js

@@ -2235,7 +2235,7 @@ function renderCardPropertiesSchemaByCardSetId(cardSets) {
             `  ${quote(cardSet.id)}: ${toPascalCase(cardSet.id)}CardPropertiesSchema,`,
         ];
     });
-    return `const cardPropertiesSchemaByCardSetId: Record<string, z.ZodTypeAny> = {\n${entries.join("\n")}\n};`;
+    return `const cardPropertiesSchemaByCardSetId: Record<string, z.ZodType<unknown>> = {\n${entries.join("\n")}\n};`;
 }
 function renderObjectSchemaSection(typeName, schemaName, schema) {
     return renderBlocks([
@@ -2373,7 +2373,7 @@ function renderCardStateSchemaById(analysis) {
     return `z.object(
   Object.fromEntries(
     literals.cardIds.map((cardId) => [cardId, createCardStateSchema(cardId)]),
-  ) as Record<CardId, z.ZodTypeAny>,
+  ) as Record<CardId, z.ZodType<unknown>>,
 )`;
 }
 function renderCardStateSchemaFactory(analysis) {
@@ -2459,8 +2459,17 @@ function renderGenericBoardStateSchema(board, runtimeBoardId) {
         : "ids.playerId.nullable().optional()"},
     templateId: z.string().nullable().optional(),
     fields: ${`${boardFieldsTypeName(board.board.id)}Schema`},
+    // T220: per-board state.spaces is loose-keyed by string. The
+    // canonical narrow id (\`ids.spaceId\`) is a Zod enum of EVERY
+    // space id across every board, which makes a strict
+    // \`z.record(enum, …)\` reject any board whose spaces are a
+    // proper subset of that enum. Loose keying matches the JSON wire
+    // shape (\`additionalProperties\`); the inner \`id: ids.spaceId\`
+    // narrows the value's spaceId at parse time. A future per-board
+    // branded-id refactor (option B) can re-tighten this without a
+    // wire change.
     spaces: z.record(
-      ids.spaceId,
+      z.string(),
       z.object({
         id: ids.spaceId,
         name: z.string().nullable().optional(),
@@ -2512,8 +2521,9 @@ function renderHexBoardStateSchema(board, runtimeBoardId) {
         : "ids.playerId.nullable().optional()"},
     templateId: z.string().nullable().optional(),
     fields: ${`${boardFieldsTypeName(board.board.id)}Schema`},
+    // T220: see generic-board comment on the loose-keying choice.
     spaces: z.record(
-      ids.spaceId,
+      z.string(),
       z.object({
         id: ids.spaceId,
         name: z.string().nullable().optional(),
@@ -2575,8 +2585,9 @@ function renderSquareBoardStateSchema(board, runtimeBoardId) {
         : "ids.playerId.nullable().optional()"},
     templateId: z.string().nullable().optional(),
     fields: ${`${boardFieldsTypeName(board.board.id)}Schema`},
+    // T220: see generic-board comment on the loose-keying choice.
     spaces: z.record(
-      ids.spaceId,
+      z.string(),
       z.object({
         id: ids.spaceId,
         name: z.string().nullable().optional(),
@@ -4160,7 +4171,11 @@ const runtimeGenericBoardStateSchema = z.object({
   playerId: ids.playerId.nullable().optional(),
   templateId: z.string().nullable().optional(),
   fields: unknownRecordSchema,
-  spaces: z.record(ids.spaceId, boardSpaceStateSchema),
+  // T220: per-board state.spaces is loose-keyed by string. See the
+  // codegen-template comment in renderGenericBoardStateSchema for
+  // the rationale; the wire shape is unchanged (additionalProperties
+  // JSON), and the inner id field narrows at parse time.
+  spaces: z.record(z.string(), boardSpaceStateSchema),
   relations: z.array(boardRelationStateSchema),
   containers: z.record(ids.boardContainerId, boardContainerStateSchema),
 });
@@ -4190,7 +4205,8 @@ const squareVertexStateSchema = z.object({
 });
 const runtimeHexBoardStateSchema = runtimeGenericBoardStateSchema.extend({
   layout: z.literal("hex"),
-  spaces: z.record(ids.spaceId, hexSpaceStateSchema),
+  // T220: loose-keyed by string — see comment above.
+  spaces: z.record(z.string(), hexSpaceStateSchema),
   relations: z.array(boardRelationStateSchema),
   containers: z.object({}),
   orientation: z.enum(["pointy-top", "flat-top"]),
@@ -4199,7 +4215,8 @@ const runtimeHexBoardStateSchema = runtimeGenericBoardStateSchema.extend({
 });
 const runtimeSquareBoardStateSchema = runtimeGenericBoardStateSchema.extend({
   layout: z.literal("square"),
-  spaces: z.record(ids.spaceId, squareSpaceStateSchema),
+  // T220: loose-keyed by string — see comment above.
+  spaces: z.record(z.string(), squareSpaceStateSchema),
   relations: z.array(boardRelationStateSchema),
   containers: z.record(ids.boardContainerId, boardContainerStateSchema),
   edges: z.array(hexEdgeStateSchema),
@@ -4397,11 +4414,11 @@ export const schemas = {
 } as const;
 
 export function createGameStateSchema<
-  PhaseNameSchema extends z.ZodTypeAny,
-  PublicSchema extends z.ZodTypeAny,
-  PrivateSchema extends z.ZodTypeAny,
-  HiddenSchema extends z.ZodTypeAny,
-  PhasesSchema extends z.ZodTypeAny,
+  PhaseNameSchema extends z.ZodType<unknown>,
+  PublicSchema extends z.ZodType<unknown>,
+  PrivateSchema extends z.ZodType<unknown>,
+  HiddenSchema extends z.ZodType<unknown>,
+  PhasesSchema extends z.ZodType<unknown>,
 >({
   phaseNameSchema,
   publicSchema,
@@ -4575,9 +4592,16 @@ function renderManifestRuntimeSource(legacySource) {
     return withoutLiterals
         .replace(generatedFileBanner, `${generatedFileBanner}\n// @ts-nocheck`)
         .replace(`} from "@dreamboard/app-sdk/reducer";\n\nconst unknownRecordSchema`, `} from "@dreamboard/app-sdk/reducer";\nimport { literals } from "./manifest-literals";\nimport type { PlayerId as PublicPlayerId, TableState as PublicTableState } from "./manifest-types";\n\nconst unknownRecordSchema`)
+        .replaceAll("literals.playerIds as unknown as readonly PlayerId[]", "literals.playerIds")
+        .replaceAll("cardId: cardIdSchema as unknown as z.ZodType<CardId>,", "cardId: assumeManifestSchema<CardId>(cardIdSchema),")
+        .replaceAll("deckId: deckIdSchema as unknown as z.ZodType<DeckId>,", "deckId: assumeManifestSchema<DeckId>(deckIdSchema),")
+        .replaceAll("handId: handIdSchema as unknown as z.ZodType<HandId>,", "handId: assumeManifestSchema<HandId>(handIdSchema),")
+        .replace(/export const staticBoards = ([\s\S]*?) as const;\n\nconst baseInitialTable = /, "export const staticBoards = $1 as const satisfies StaticBoards<PublicTableState>;\n\nconst baseInitialTable = ")
+        .replace(/const baseInitialTable = ([\s\S]*?) as const as unknown as TableState;\nconst baseDeckCardsByZoneId:/, "const baseInitialTable = cloneManifestDefault<PublicTableState>($1);\nconst baseDeckCardsByZoneId:")
+        .replace("staticBoards: staticBoards as unknown as StaticBoards<TableState>,", "staticBoards,")
         .replace(`export const manifestContract: ReducerManifestContract<\n  RuntimeTableRecord,`, `export const manifestContract: ReducerManifestContract<\n  PublicTableState,`)
         .replace(`export const manifestContract: ReducerManifestContract<\n  PublicTableState,\n  string,\n  PlayerId,`, `export const manifestContract: ReducerManifestContract<\n  PublicTableState,\n  string,\n  PublicPlayerId,`)
-        .replace(`const playerIdSchema = markManifestScopedSchema(\n  z\n    .string()\n    .min(1)\n    .transform((value) => asPlayerId(value)),\n);`, `const playerIdSchema = markManifestScopedSchema(\n  z\n    .string()\n    .min(1)\n    .transform((value) => asPlayerId(value)),\n) as unknown as z.ZodType<PublicPlayerId>;`);
+        .replace(`const playerIdSchema = markManifestScopedSchema(\n  z\n    .string()\n    .min(1)\n    .transform((value) => asPlayerId(value)),\n);`, `const playerIdSchema = assumeManifestSchema<PublicPlayerId>(\n  markManifestScopedSchema(\n    z\n      .string()\n      .min(1)\n      .transform((value) => asPlayerId(value)),\n  ),\n);`);
 }
 function renderCardSetTypeSections(analysis) {
     return analysis.cardSets