fix(codegen-ts): don't chain string validators onto a required jsonb open-bag

A `field.string` + `@dbColumnType: jsonb` open bag has a `z.unknown()` base (0.14.0).
A REQUIRED such field still got the required-non-empty `.min(1)` chained on, emitting
`z.unknown().min(1)` — a TS compile error (ZodUnknown has no .min), so an adopter's
generated schema stopped compiling after 0.14.0.

appendValidatorChain now skips the string .min/.max/.regex branch for a jsonb open bag
(a jsonb ARRAY still gets element-count bounds via the array branch); "required" for an
open bag means non-optional only, already handled by the optional() logic. Adds a
required-jsonb regression case to jsonb-open-bag-zod.test.ts. Full suite 869 green.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01DV7ad8tggzcpFJzXYvHwRU

Author: dmealing

CHANGELOG.md

@@ -7,6 +7,16 @@ this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.htm
 
 ## [Unreleased]
 
+### Fixed
+- **codegen-ts — required jsonb open-bag generated an uncompilable `z.unknown().min(1)`.**
+  The 0.14.0 jsonb open-bag change (`field.string` + `@dbColumnType: jsonb` → `z.unknown()`)
+  left the string character-count validators attached: a **required** jsonb field still got
+  the required-non-empty `.min(1)` chained onto its `z.unknown()` base, emitting
+  `z.unknown().min(1)` — a TS compile error (`ZodUnknown` has no `.min`). The validator
+  chain now skips the string `.min`/`.max`/`.regex` for a jsonb open bag (a jsonb array
+  still gets element-count bounds); "required" for an open bag means non-optional only.
+  Surfaced by an adopter whose generated schema stopped compiling after 0.14.0.
+
 ## [0.14.1] — 2026-06-29
 
 _npm `0.14.1` (full lockstep across all 13 `@metaobjectsdev/*` publish candidates)._

server/typescript/packages/codegen-ts/src/templates/zod-validators.ts

@@ -504,12 +504,19 @@ function appendValidatorChain(base: Code, field: MetaField): Code {
     }
   }
 
+  // A `field.string` + `@dbColumnType: jsonb` open bag has a `z.unknown()` base, so
+  // the string character-count validators (.min/.max/.regex) do NOT apply — chaining
+  // `.min(1)` for a required bag yields `z.unknown().min(1)`, a TS compile error.
+  // "Required" for an open bag means non-optional only (handled by the optional()
+  // logic below). A jsonb ARRAY still gets element-count bounds via the array branch.
+  const isJsonbBag = field.attr(FIELD_ATTR_DB_COLUMN_TYPE) === DB_COLUMN_TYPE_JSONB;
+
   let chain: Code = base;
   // Array element-count bounds apply to the z.array(...) wrapper regardless of element type.
   if (field.isArray) {
     if (arrMin !== undefined) chain = code`${chain}.min(${arrMin})`;
     if (arrMax !== undefined) chain = code`${chain}.max(${arrMax})`;
-  } else if (field.subType === FIELD_SUBTYPE_STRING) {
+  } else if (field.subType === FIELD_SUBTYPE_STRING && !isJsonbBag) {
     if (minLen !== undefined) chain = code`${chain}.min(${minLen})`;
     else if (isRequired) chain = code`${chain}.min(1)`;
     if (maxLen !== undefined) chain = code`${chain}.max(${maxLen})`;

server/typescript/packages/codegen-ts/test/jsonb-open-bag-zod.test.ts

@@ -29,6 +29,7 @@ const YAML = `metadata:
       - source.rdb: { table: doc }
       - field.long: { name: id }
       - field.string: { name: payload, dbColumnType: jsonb }
+      - field.string: { name: config, dbColumnType: jsonb, required: true }
       - field.string: { name: title }
       - identity.primary: { fields: [id], generation: increment }
 `;
@@ -68,6 +69,20 @@ describe("field.string + @dbColumnType: jsonb → Zod z.unknown() (open JSON bag
     expect(out).toContain("title: z.string()");
   });
 
+  // A REQUIRED jsonb open-bag must NOT get the string `.min(1)` (required-non-empty)
+  // chained onto z.unknown() — `z.unknown().min(1)` is a TS compile error (ZodUnknown
+  // has no .min). "Required" for an open bag means non-optional, full stop. Regression
+  // for the jsonb→z.unknown() change that left the string validators attached.
+  test("a REQUIRED jsonb field is z.unknown() with NO .min/.max chained", async () => {
+    const out = await loadDoc();
+    // Insert schema: required jsonb bag = bare z.unknown() (no .min, no .optional).
+    expect(out).toContain("config: z.unknown(),");
+    expect(out).not.toContain("config: z.unknown().min");
+    expect(out).not.toContain("config: z.unknown().max");
+    // Update schema still relaxes every field to optional — that's expected.
+    expect(out).toContain("config: z.unknown().optional()");
+  });
+
   // The TS-type emitter must stay in lock-step with the Zod emitter (the
   // inferred-types contract): a jsonb-bag field is `unknown`, not `string`,
   // so a contract-only (runtime:false) target's TS type matches its Zod.