@gabrielbryk/json-schema-to-zod 2.9.0 → 2.10.1

package/CHANGELOG.md

@@ -1,5 +1,12 @@
 # @gabrielbryk/json-schema-to-zod
 
+## 2.10.1
+
+### Patch Changes
+
+- 1c318a0: Ensure bundled outputs remain type-safe by grouping strongly connected defs, emitting lazy getters for recursive object refs, and reordering bundle members to avoid duplicate exports.
+- 87526f0: Preserve precise types for recursive schemas by emitting typed lazy wrappers instead of erasing types, and add a workflow regression type-check to guard against inference blowups.
+
 ## 2.9.0
 
 ### Minor Changes
@@ -11,6 +18,11 @@
 ### Patch Changes
 
 - b8f6248: - Commit remaining utility, ref resolution test, and config updates.
+- 04b6c6b: Fix bundle output by emitting z.lazy for cyclical refs outside object properties, use ZodError.issues in generated conditionals, and make nested type extraction array-aware to avoid invalid indexers.
+- b8f7b29: Fix type errors in CI by replacing symbol-based allOf indexing, guarding invalid refs, and tightening string content schema parsing types.
+- 691cc5b: - Added ESLint (recommended + no-require-imports) and cleaned all lint issues across src/tests; tightened types and removed unused vars.
+  - Ensured ESM-friendly test evals and ref/anchor resolver code without require usage.
+- 4d127fe: Remove fallback to the non-existent ZodError.errors property in generated conditional schemas; rely on ZodError.issues to avoid TypeScript errors.
 - 7d257dd: - Ensure dist/esm emits real ESM with NodeNext settings and type:module, and update tests to run under ESM by providing createRequire shims.
 
 ## 2.8.0

package/dist/cjs/core/emitZod.js

@@ -73,6 +73,16 @@ const emitZod = (analysis) => {
     const { module, name, type, noImport, exportRefs, withMeta, ...rest } = options;
     const declarations = new Map();
     const dependencies = new Map();
+    const reserveName = (base) => {
+        let candidate = base;
+        let i = 1;
+        while (usedNames.has(candidate) || declarations.has(candidate)) {
+            candidate = `${base}${i}`;
+            i += 1;
+        }
+        usedNames.add(candidate);
+        return candidate;
+    };
     const parsedSchema = (0, parseSchema_js_1.parseSchema)(schema, {
         module,
         name,
@@ -94,15 +104,21 @@ const emitZod = (analysis) => {
     });
     const declarationBlock = declarations.size
         ? orderDeclarations(Array.from(declarations.entries()), dependencies)
-            .map(([refName, value]) => {
+            .flatMap(([refName, value]) => {
             const shouldExport = exportRefs && module === "esm";
-            const decl = `${shouldExport ? "export " : ""}const ${refName} = ${value}`;
-            return decl;
+            const isCycle = cycleRefNames.has(refName);
+            if (!isCycle) {
+                return [`${shouldExport ? "export " : ""}const ${refName} = ${value}`];
+            }
+            const baseName = `${refName}Def`;
+            const lines = [`const ${baseName} = ${value}`];
+            lines.push(`${shouldExport ? "export " : ""}const ${refName} = ${baseName}`);
+            return lines;
         })
             .join("\n")
         : "";
-    const jsdocs = rest.withJsdocs && typeof schema !== "boolean" && schema.description
-        ? (0, jsdocs_js_1.expandJsdocs)(schema.description)
+    const jsdocs = rest.withJsdocs && typeof schema === "object" && schema !== null && "description" in schema
+        ? (0, jsdocs_js_1.expandJsdocs)(String(schema.description ?? ""))
         : "";
     const lines = [];
     if (module === "cjs" && !noImport) {

package/dist/cjs/generators/generateBundle.js

@@ -11,23 +11,27 @@ const generateSchemaBundle = (schema, options = {}) => {
     const defs = schema.$defs || {};
     const definitions = schema.definitions || {};
     const defNames = Object.keys(defs);
-    const { rootName, rootTypeName, defInfoMap } = buildBundleContext(defNames, defs, options);
+    const { rootName, rootTypeName, defInfoMap, groups } = buildBundleContext(defNames, defs, options);
     const files = [];
-    const targets = planBundleTargets(schema, defs, definitions, defNames, options, rootName, rootTypeName);
+    const targets = planBundleTargets(schema, defs, definitions, defNames, options, rootName, defInfoMap, rootTypeName);
     for (const target of targets) {
         const usedRefs = target.usedRefs;
-        const analysis = (0, analyzeSchema_js_1.analyzeSchema)(target.schemaWithDefs, {
-            ...options,
-            module,
-            name: target.schemaName,
-            type: target.typeName,
-            parserOverride: createRefHandler(target.defName, defInfoMap, usedRefs, {
-                ...(target.schemaWithDefs.$defs || {}),
-                ...(target.schemaWithDefs.definitions || {}),
-            }, options),
-        });
-        const zodSchema = (0, emitZod_js_1.emitZod)(analysis);
-        const finalSchema = buildSchemaFile(zodSchema, usedRefs, defInfoMap, module);
+        const zodParts = [];
+        for (const member of target.members) {
+            const analysis = (0, analyzeSchema_js_1.analyzeSchema)(member.schemaWithDefs, {
+                ...options,
+                module,
+                name: member.schemaName,
+                type: member.typeName,
+                parserOverride: createRefHandler(member.defName, defInfoMap, usedRefs, {
+                    ...(member.schemaWithDefs.$defs || {}),
+                    ...(member.schemaWithDefs.definitions || {}),
+                }, options, target.groupId),
+            });
+            const zodSchema = (0, emitZod_js_1.emitZod)(analysis);
+            zodParts.push(zodSchema);
+        }
+        const finalSchema = buildSchemaFile(zodParts, usedRefs, defInfoMap, module, target);
         files.push({ fileName: target.fileName, contents: finalSchema });
     }
     // Nested types extraction (optional)
@@ -57,13 +61,16 @@ const buildDefInfoMap = (defNames, defs, options) => {
         const pascalName = toPascalCase(defName);
         const schemaName = options.splitDefs?.schemaName?.(defName, { isRoot: false }) ?? `${pascalName}Schema`;
         const typeName = options.splitDefs?.typeName?.(defName, { isRoot: false }) ?? pascalName;
+        const fileName = options.splitDefs?.fileName?.(defName, { isRoot: false }) ?? `${defName}.schema.ts`;
         map.set(defName, {
             name: defName,
             pascalName,
             schemaName,
             typeName,
+            fileName,
             dependencies,
             hasCycle: false,
+            groupId: "",
         });
     }
     return map;
@@ -76,13 +83,22 @@ const buildBundleContext = (defNames, defs, options) => {
         if (info)
             info.hasCycle = true;
     }
+    const groups = buildSccGroups(defInfoMap);
+    for (const [groupId, members] of groups) {
+        for (const defName of members) {
+            const info = defInfoMap.get(defName);
+            if (info)
+                info.groupId = groupId;
+        }
+    }
     const rootName = options.splitDefs?.rootName ?? options.name ?? "RootSchema";
     const rootTypeName = typeof options.type === "string"
         ? options.type
         : options.splitDefs?.rootTypeName ?? (typeof options.type === "boolean" && options.type ? rootName : undefined);
-    return { defInfoMap, rootName, rootTypeName };
+    return { defInfoMap, rootName, rootTypeName, groups };
 };
-const createRefHandler = (currentDefName, defInfoMap, usedRefs, allDefs, options) => {
+const createRefHandler = (currentDefName, defInfoMap, usedRefs, allDefs, options, currentGroupId) => {
+    const useLazyCrossRefs = options.refResolution?.lazyCrossRefs ?? true;
     return (schema, refs) => {
         if (typeof schema["$ref"] === "string") {
             const refPath = schema["$ref"];
@@ -96,7 +112,7 @@ const createRefHandler = (currentDefName, defInfoMap, usedRefs, allDefs, options
                 const refInfo = defInfoMap.get(refName);
                 if (refInfo) {
                     // Track imports when referencing other defs
-                    if (refName !== currentDefName) {
+                    if (refName !== currentDefName && refInfo.groupId !== currentGroupId) {
                         usedRefs.add(refName);
                     }
                     const isCycle = refName === currentDefName || (refInfo.hasCycle && !!currentDefName);
@@ -109,7 +125,14 @@ const createRefHandler = (currentDefName, defInfoMap, usedRefs, allDefs, options
                     });
                     if (resolved)
                         return resolved;
-                    if (isCycle && options.refResolution?.lazyCrossRefs) {
+                    if (isCycle && useLazyCrossRefs) {
+                        const inObjectProperty = refs.path.includes("properties") ||
+                            refs.path.includes("patternProperties") ||
+                            refs.path.includes("additionalProperties");
+                        if (inObjectProperty && refName === currentDefName) {
+                            // Self-recursion inside object getters can safely reference the schema name
+                            return refInfo.schemaName;
+                        }
                         return `z.lazy(() => ${refInfo.schemaName})`;
                     }
                     return refInfo.schemaName;
@@ -128,42 +151,73 @@ const createRefHandler = (currentDefName, defInfoMap, usedRefs, allDefs, options
         return undefined;
     };
 };
-const buildSchemaFile = (zodCode, usedRefs, defInfoMap, module) => {
+const buildSchemaFile = (zodCodeParts, usedRefs, defInfoMap, module, target) => {
     if (module !== "esm")
-        return zodCode;
-    const imports = [];
+        return zodCodeParts.join("\n");
+    const groupFileById = new Map();
+    for (const info of defInfoMap.values()) {
+        if (!groupFileById.has(info.groupId)) {
+            groupFileById.set(info.groupId, info.fileName.replace(/\.ts$/, ".js"));
+        }
+    }
+    const importsByFile = new Map();
     for (const refName of [...usedRefs].sort()) {
         const refInfo = defInfoMap.get(refName);
         if (refInfo) {
-            imports.push(`import { ${refInfo.schemaName} } from './${refName}.schema.js';`);
+            const groupFile = groupFileById.get(refInfo.groupId) ?? refInfo.fileName.replace(/\.ts$/, ".js");
+            const path = `./${groupFile}`;
+            const set = importsByFile.get(path) ?? new Set();
+            set.add(refInfo.schemaName);
+            importsByFile.set(path, set);
         }
     }
-    if (!imports.length)
-        return zodCode;
-    return zodCode.replace('import { z } from "zod"', `import { z } from "zod"\n${imports.join("\n")}`);
+    const imports = [];
+    for (const [path, names] of [...importsByFile.entries()].sort(([a], [b]) => a.localeCompare(b))) {
+        imports.push(`import { ${[...names].sort().join(", ")} } from '${path}';`);
+    }
+    const body = zodCodeParts
+        .map((code, idx) => {
+        if (idx === 0)
+            return code;
+        return code.replace(/^import \{ z \} from "zod"\n?/, "");
+    })
+        .join("\n");
+    return imports.length
+        ? body.replace('import { z } from "zod"', `import { z } from "zod"\n${imports.join("\n")}`)
+        : body;
 };
-const planBundleTargets = (rootSchema, defs, definitions, defNames, options, rootName, rootTypeName) => {
+const planBundleTargets = (rootSchema, defs, definitions, defNames, options, rootName, defInfoMap, rootTypeName) => {
     const targets = [];
+    const groupById = new Map();
     for (const defName of defNames) {
-        const defSchema = defs[defName];
-        const defSchemaWithDefs = {
-            ...defSchema,
-            $defs: { ...defs, ...defSchema?.$defs },
-            definitions: {
-                ...defSchema.definitions,
-                ...definitions,
-            },
-        };
-        const pascalName = toPascalCase(defName);
-        const schemaName = options.splitDefs?.schemaName?.(defName, { isRoot: false }) ?? `${pascalName}Schema`;
-        const typeName = options.splitDefs?.typeName?.(defName, { isRoot: false }) ?? pascalName;
-        const fileName = options.splitDefs?.fileName?.(defName, { isRoot: false }) ?? `${defName}.schema.ts`;
+        const info = defInfoMap.get(defName);
+        const gid = info?.groupId || defName;
+        if (!groupById.has(gid))
+            groupById.set(gid, []);
+        groupById.get(gid).push(defName);
+    }
+    for (const [groupId, memberDefs] of groupById.entries()) {
+        const orderedDefs = orderGroupMembers(memberDefs, defInfoMap);
+        const members = orderedDefs.map((defName) => {
+            const defSchema = defs[defName];
+            const defSchemaWithDefs = {
+                ...defSchema,
+                $defs: { ...defs, ...defSchema?.$defs },
+                definitions: {
+                    ...defSchema.definitions,
+                    ...definitions,
+                },
+            };
+            const pascalName = toPascalCase(defName);
+            const schemaName = options.splitDefs?.schemaName?.(defName, { isRoot: false }) ?? `${pascalName}Schema`;
+            const typeName = options.splitDefs?.typeName?.(defName, { isRoot: false }) ?? pascalName;
+            return { defName, schemaWithDefs: defSchemaWithDefs, schemaName, typeName };
+        });
+        const fileName = defInfoMap.get(memberDefs[0])?.fileName ?? `${memberDefs[0]}.schema.ts`;
         targets.push({
-            defName,
-            schemaWithDefs: defSchemaWithDefs,
-            schemaName,
-            typeName,
+            groupId,
             fileName,
+            members,
             usedRefs: new Set(),
             isRoot: false,
         });
@@ -171,17 +225,22 @@ const planBundleTargets = (rootSchema, defs, definitions, defNames, options, roo
     if (options.splitDefs?.includeRoot ?? true) {
         const rootFile = options.splitDefs?.fileName?.("root", { isRoot: true }) ?? "workflow.schema.ts";
         targets.push({
-            defName: null,
-            schemaWithDefs: {
-                ...rootSchema,
-                definitions: {
-                    ...rootSchema.definitions,
-                    ...definitions,
-                },
-            },
-            schemaName: rootName,
-            typeName: rootTypeName,
+            groupId: "root",
             fileName: rootFile,
+            members: [
+                {
+                    defName: null,
+                    schemaWithDefs: {
+                        ...rootSchema,
+                        definitions: {
+                            ...rootSchema.definitions,
+                            ...definitions,
+                        },
+                    },
+                    schemaName: rootName,
+                    typeName: rootTypeName,
+                },
+            ],
             usedRefs: new Set(),
             isRoot: true,
         });
@@ -212,6 +271,35 @@ const findRefDependencies = (schema, validDefNames) => {
     traverse(schema);
     return deps;
 };
+const orderGroupMembers = (defs, defInfoMap) => {
+    const inGroup = new Set(defs);
+    const visited = new Set();
+    const temp = new Set();
+    const result = [];
+    const visit = (name) => {
+        if (visited.has(name))
+            return;
+        if (temp.has(name)) {
+            return;
+        }
+        temp.add(name);
+        const info = defInfoMap.get(name);
+        if (info) {
+            for (const dep of info.dependencies) {
+                if (inGroup.has(dep)) {
+                    visit(dep);
+                }
+            }
+        }
+        temp.delete(name);
+        visited.add(name);
+        result.push(name);
+    };
+    for (const name of defs) {
+        visit(name);
+    }
+    return result;
+};
 const detectCycles = (defInfoMap) => {
     const cycleNodes = new Set();
     const visited = new Set();
@@ -245,6 +333,52 @@ const detectCycles = (defInfoMap) => {
     }
     return cycleNodes;
 };
+const buildSccGroups = (defInfoMap) => {
+    const indexMap = new Map();
+    const lowLink = new Map();
+    const onStack = new Set();
+    const stack = [];
+    let index = 0;
+    const groups = new Map();
+    const strongConnect = (node) => {
+        indexMap.set(node, index);
+        lowLink.set(node, index);
+        index += 1;
+        stack.push(node);
+        onStack.add(node);
+        const info = defInfoMap.get(node);
+        if (info) {
+            for (const dep of info.dependencies) {
+                if (!indexMap.has(dep)) {
+                    strongConnect(dep);
+                    lowLink.set(node, Math.min(lowLink.get(node), lowLink.get(dep)));
+                }
+                else if (onStack.has(dep)) {
+                    lowLink.set(node, Math.min(lowLink.get(node), indexMap.get(dep)));
+                }
+            }
+        }
+        if (lowLink.get(node) === indexMap.get(node)) {
+            const members = [];
+            let w;
+            do {
+                w = stack.pop();
+                if (w) {
+                    onStack.delete(w);
+                    members.push(w);
+                }
+            } while (w && w !== node);
+            const groupId = members.sort().join("__");
+            groups.set(groupId, members);
+        }
+    };
+    for (const name of defInfoMap.keys()) {
+        if (!indexMap.has(name)) {
+            strongConnect(name);
+        }
+    }
+    return groups;
+};
 const collectNestedTypes = (rootSchema, defs, defNames, rootTypeName) => {
     const allNestedTypes = [];
     for (const defName of defNames) {
@@ -289,7 +423,7 @@ const findNestedTypesInSchema = (schema, parentTypeName, defNames, currentPath =
     }
     // inline $defs
     if (record.$defs && typeof record.$defs === "object") {
-        for (const [_defName, defSchema] of Object.entries(record.$defs)) {
+        for (const [, defSchema] of Object.entries(record.$defs)) {
             nestedTypes.push(...findNestedTypesInSchema(defSchema, parentTypeName, defNames, currentPath));
         }
     }
@@ -320,6 +454,26 @@ const generateNestedTypesFile = (nestedTypes) => {
         " * They are extracted using TypeScript indexed access types.",
         " */",
         "",
+        "type Access<T, P extends readonly (string | number)[]> =",
+        "  P extends []",
+        "    ? NonNullable<T>",
+        "    : P extends readonly [infer H, ...infer R]",
+        "      ? H extends \"items\"",
+        "        ? Access<NonNullable<T> extends Array<infer U> ? U : unknown, Extract<R, (string | number)[]>>",
+        "        : H extends \"additionalProperties\"",
+        "          ? Access<NonNullable<T> extends Record<string, infer V> ? V : unknown, Extract<R, (string | number)[]>>",
+        "          : H extends number",
+        "            ? Access<NonNullable<T> extends Array<infer U> ? U : unknown, Extract<R, (string | number)[]>>",
+        "            : H extends string",
+        "              ? Access<",
+        "                  H extends keyof NonNullable<T>",
+        "                    ? NonNullable<NonNullable<T>[H]>",
+        "                    : unknown,",
+        "                  Extract<R, (string | number)[]>",
+        "                >",
+        "              : unknown",
+        "      : unknown;",
+        "",
     ];
     const byParent = new Map();
     for (const info of nestedTypes) {
@@ -338,14 +492,15 @@ const generateNestedTypesFile = (nestedTypes) => {
         lines.push(`import type { ${typeName} } from './${file}.schema.js';`);
     }
     lines.push("");
+    const buildAccessExpr = (parentType, propertyPath) => {
+        const path = propertyPath.map((prop) => (typeof prop === "number" ? prop : JSON.stringify(prop))).join(", ");
+        return `Access<${parentType}, [${path}]>`;
+    };
     for (const [parentType, types] of [...byParent.entries()].sort()) {
         lines.push(`// From ${parentType}`);
         for (const info of types.sort((a, b) => a.typeName.localeCompare(b.typeName))) {
             if (info.propertyPath.length > 0) {
-                let accessExpr = parentType;
-                for (const prop of info.propertyPath) {
-                    accessExpr = `NonNullable<${accessExpr}['${prop}']>`;
-                }
+                const accessExpr = buildAccessExpr(parentType, info.propertyPath);
                 lines.push(`export type ${info.typeName} = ${accessExpr};`);
             }
         }

package/dist/cjs/parsers/parseAllOf.js

@@ -3,19 +3,21 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.parseAllOf = parseAllOf;
 const parseSchema_js_1 = require("./parseSchema.js");
 const half_js_1 = require("../utils/half.js");
-const originalIndex = Symbol("Original index");
+const originalIndexKey = "__originalIndex";
 const ensureOriginalIndex = (arr) => {
-    let newArr = [];
+    const newArr = [];
     for (let i = 0; i < arr.length; i++) {
         const item = arr[i];
         if (typeof item === "boolean") {
-            newArr.push(item ? { [originalIndex]: i } : { [originalIndex]: i, not: {} });
+            newArr.push(item ? { [originalIndexKey]: i } : { [originalIndexKey]: i, not: {} });
         }
-        else if (originalIndex in item) {
+        else if (typeof item === "object" &&
+            item !== null &&
+            originalIndexKey in item) {
             return arr;
         }
         else {
-            newArr.push({ ...item, [originalIndex]: i });
+            newArr.push({ ...item, [originalIndexKey]: i });
         }
     }
     return newArr;
@@ -28,7 +30,11 @@ function parseAllOf(schema, refs) {
         const item = schema.allOf[0];
         return (0, parseSchema_js_1.parseSchema)(item, {
             ...refs,
-            path: [...refs.path, "allOf", item[originalIndex]],
+            path: [
+                ...refs.path,
+                "allOf",
+                item[originalIndexKey] ?? 0,
+            ],
         });
     }
     else {

package/dist/cjs/parsers/parseBoolean.js

@@ -1,7 +1,5 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.parseBoolean = void 0;
-const parseBoolean = (_schema) => {
-    return "z.boolean()";
-};
+const parseBoolean = () => "z.boolean()";
 exports.parseBoolean = parseBoolean;

package/dist/cjs/parsers/parseIfThenElse.js

@@ -17,8 +17,8 @@ const parseIfThenElse = (schema, refs) => {
     ? ${$then}.safeParse(value)
     : ${$else}.safeParse(value);
   if (!result.success) {
-    const issues = result.error.issues ?? result.error.errors ?? [];
-    issues.forEach((issue) => ctx.addIssue(issue))
+    const issues = result.error.issues;
+    issues.forEach((issue) => ctx.addIssue({ ...issue }))
   }
 })`;
     // Store original if/then/else for JSON Schema round-trip

package/dist/cjs/parsers/parseNull.js

@@ -1,7 +1,5 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.parseNull = void 0;
-const parseNull = (_schema) => {
-    return "z.null()";
-};
+const parseNull = () => "z.null()";
 exports.parseNull = parseNull;

package/dist/cjs/parsers/parseObject.js

@@ -200,6 +200,7 @@ function parseObject(objectSchema, refs) {
         output += `.and(${(0, parseAnyOf_js_1.parseAnyOf)({
             ...objectSchema,
             anyOf: objectSchema.anyOf.map((x) => typeof x === "object" &&
+                x !== null &&
                 !x.type &&
                 (x.properties || x.additionalProperties || x.patternProperties)
                 ? { ...x, type: "object" }
@@ -210,6 +211,7 @@ function parseObject(objectSchema, refs) {
         output += `.and(${(0, parseOneOf_js_1.parseOneOf)({
             ...objectSchema,
             oneOf: objectSchema.oneOf.map((x) => typeof x === "object" &&
+                x !== null &&
                 !x.type &&
                 (x.properties || x.additionalProperties || x.patternProperties)
                 ? { ...x, type: "object" }
@@ -220,6 +222,7 @@ function parseObject(objectSchema, refs) {
         output += `.and(${(0, parseAllOf_js_1.parseAllOf)({
             ...objectSchema,
             allOf: objectSchema.allOf.map((x) => typeof x === "object" &&
+                x !== null &&
                 !x.type &&
                 (x.properties || x.additionalProperties || x.patternProperties)
                 ? { ...x, type: "object" }
@@ -233,6 +236,7 @@ function parseObject(objectSchema, refs) {
     // propertyNames
     if (objectSchema.propertyNames) {
         const normalizedPropNames = typeof objectSchema.propertyNames === "object" &&
+            objectSchema.propertyNames !== null &&
             !objectSchema.propertyNames.type &&
             objectSchema.propertyNames.pattern
             ? { ...objectSchema.propertyNames, type: "string" }
@@ -261,12 +265,12 @@ function parseObject(objectSchema, refs) {
         if (entries.length) {
             output += `.superRefine((obj, ctx) => {
   ${entries
-                .map(([key, schema], idx) => {
+                .map(([key, schema]) => {
                 const parsed = (0, parseSchema_js_1.parseSchema)(schema, { ...refs, path: [...refs.path, "dependentSchemas", key] });
                 return `if (Object.prototype.hasOwnProperty.call(obj, ${JSON.stringify(key)})) {
     const result = ${parsed}.safeParse(obj);
     if (!result.success) {
-      ctx.addIssue({ code: "custom", message: "Dependent schema failed", path: [], params: { issues: result.error.issues } });
+      ctx.addIssue({ code: "custom", message: ${objectSchema.errorMessage?.dependentSchemas ?? JSON.stringify("Dependent schema failed")}, path: [], params: { issues: result.error.issues } });
     }
   }`;
             })
@@ -278,7 +282,8 @@ function parseObject(objectSchema, refs) {
     if (objectSchema.dependentRequired && typeof objectSchema.dependentRequired === "object") {
         const entries = Object.entries(objectSchema.dependentRequired);
         if (entries.length) {
-            const depRequiredMessage = objectSchema.errorMessage?.dependentRequired ?? "Dependent required properties missing";
+            const depRequiredMessage = objectSchema.errorMessage?.dependentRequired ??
+                "Dependent required properties missing";
             output += `.superRefine((obj, ctx) => {
   ${entries
                 .map(([prop, deps]) => {
@@ -306,5 +311,8 @@ const shouldUseGetter = (parsed, refs) => {
     if (refs.currentSchemaName && parsed.includes(refs.currentSchemaName)) {
         return true;
     }
+    if (refs.cycleRefNames?.has(parsed)) {
+        return true;
+    }
     return Boolean(refs.inProgress && refs.inProgress.has(parsed));
 };

package/dist/cjs/parsers/parseSchema.js

@@ -33,9 +33,7 @@ const parseSchema = (schema, refs = { seen: new Map(), path: [] }, blockMeta) =>
     if (typeof schema !== "object")
         return schema ? (0, anyOrUnknown_js_1.anyOrUnknown)(refs) : "z.never()";
     const parentBase = refs.currentBaseUri ?? refs.rootBaseUri ?? "root:///";
-    const baseUri = typeof schema.$id === "string"
-        ? (0, resolveUri_js_1.resolveUri)(parentBase, schema.$id)
-        : parentBase;
+    const baseUri = typeof schema.$id === "string" ? (0, resolveUri_js_1.resolveUri)(parentBase, schema.$id) : parentBase;
     const dynamicAnchors = Array.isArray(refs.dynamicAnchors) ? [...refs.dynamicAnchors] : [];
     if (typeof schema.$dynamicAnchor === "string") {
         dynamicAnchors.push({
@@ -85,6 +83,9 @@ const parseSchema = (schema, refs = { seen: new Map(), path: [] }, blockMeta) =>
 exports.parseSchema = parseSchema;
 const parseRef = (schema, refs) => {
     const refValue = schema.$dynamicRef ?? schema.$ref;
+    if (typeof refValue !== "string") {
+        return (0, anyOrUnknown_js_1.anyOrUnknown)(refs);
+    }
     const resolved = resolveRef(schema, refValue, refs);
     if (!resolved) {
         refs.onUnresolvedRef?.(refValue, refs.path);
@@ -123,6 +124,14 @@ const parseRef = (schema, refs) => {
     // (or is currently being resolved). This avoids TDZ on true cycles while
     // letting ordered, acyclic refs stay direct.
     if (isSameCycle || refs.inProgress.has(refName)) {
+        const inObjectProperty = refs.path.includes("properties") ||
+            refs.path.includes("patternProperties") ||
+            refs.path.includes("additionalProperties");
+        if (inObjectProperty && refName === refs.currentSchemaName) {
+            // Getter properties defer evaluation, so a direct reference avoids extra lazies
+            // for self-recursion.
+            return refName;
+        }
         return `z.lazy(() => ${refName})`;
     }
     return refName;
@@ -194,7 +203,10 @@ const resolveRef = (schemaNode, ref, refs) => {
         const loaded = refs.resolveExternalRef(extBase);
         if (loaded) {
             // If async resolver is used synchronously here, it will be ignored; keep simple sync for now
-            const schema = loaded.then ? undefined : loaded;
+            const maybePromise = loaded;
+            const schema = typeof maybePromise.then === "function"
+                ? undefined
+                : loaded;
             if (schema) {
                 const { registry } = (0, buildRefRegistry_js_1.buildRefRegistry)(schema, extBase);
                 registry.forEach((entry, k) => refs.refRegistry?.set(k, entry));
@@ -342,10 +354,10 @@ const selectParser = (schema, refs) => {
         return (0, parseNumber_js_1.parseNumber)(schema);
     }
     else if (exports.its.a.primitive(schema, "boolean")) {
-        return (0, parseBoolean_js_1.parseBoolean)(schema);
+        return (0, parseBoolean_js_1.parseBoolean)();
     }
     else if (exports.its.a.primitive(schema, "null")) {
-        return (0, parseNull_js_1.parseNull)(schema);
+        return (0, parseNull_js_1.parseNull)();
     }
     else if (exports.its.a.conditional(schema)) {
         return (0, parseIfThenElse_js_1.parseIfThenElse)(schema, refs);

package/dist/cjs/parsers/parseString.js

@@ -289,7 +289,7 @@ const parseString = (schema, refs) => {
     if (contentMediaType != "") {
         r += contentMediaType;
         r += (0, withMessage_js_1.withMessage)(schema, "contentSchema", ({ value }) => {
-            if (value && value instanceof Object) {
+            if (value && typeof value === "object") {
                 return {
                     opener: `.pipe(${(0, parseSchema_js_1.parseSchema)(value, refContext)}`,
                     closer: ")",