Circular dependencies?

[Carlos-err406]

i get many circular dependencies from zod-to-json-schema when building with rollup

see image:

[jporre]

This happens to me as well after upgrading to version 3.23.5 of ZOD
Sveltekit Project,
zod version -> zod3.23.5 on build :

Circular dependency: node_modules/zod-to-json-schema/dist/esm/parseDef.js -> node_modules/zod-to-json-schema/dist/esm/parsers/array.js -> node_modules/zod-to-json-schema/dist/esm/parseDef.js
Circular dependency: node_modules/zod-to-json-schema/dist/esm/parseDef.js -> node_modules/zod-to-json-schema/dist/esm/parsers/branded.js -> node_modules/zod-to-json-schema/dist/esm/parseDef.js
Circular dependency: node_modules/zod-to-json-schema/dist/esm/parseDef.js -> node_modules/zod-to-json-schema/dist/esm/parsers/catch.js -> node_modules/zod-to-json-schema/dist/esm/parseDef.js
Circular dependency: node_modules/zod-to-json-schema/dist/esm/parseDef.js -> node_modules/zod-to-json-schema/dist/esm/parsers/default.js -> node_modules/zod-to-json-schema/dist/esm/parseDef.js
Circular dependency: node_modules/zod-to-json-schema/dist/esm/parseDef.js -> node_modules/zod-to-json-schema/dist/esm/parsers/effects.js -> node_modules/zod-to-json-schema/dist/esm/parseDef.js
Circular dependency: node_modules/zod-to-json-schema/dist/esm/parseDef.js -> node_modules/zod-to-json-schema/dist/esm/parsers/intersection.js -> node_modules/zod-to-json-schema/dist/esm/parseDef.js
Circular dependency: node_modules/zod-to-json-schema/dist/esm/parseDef.js -> node_modules/zod-to-json-schema/dist/esm/parsers/map.js -> node_modules/zod-to-json-schema/dist/esm/parseDef.js
Circular dependency: node_modules/zod-to-json-schema/dist/esm/parseDef.js -> node_modules/zod-to-json-schema/dist/esm/parsers/map.js -> node_modules/zod-to-json-schema/dist/esm/parsers/record.js -> node_modules/zod-to-json-schema/dist/esm/parseDef.js
Circular dependency: node_modules/zod-to-json-schema/dist/esm/parseDef.js -> node_modules/zod-to-json-schema/dist/esm/parsers/nullable.js -> node_modules/zod-to-json-schema/dist/esm/parseDef.js
Circular dependency: node_modules/zod-to-json-schema/dist/esm/parseDef.js -> node_modules/zod-to-json-schema/dist/esm/parsers/nullable.js -> node_modules/zod-to-json-schema/dist/esm/parsers/union.js -> node_modules/zod-to-json-schema/dist/esm/parseDef.js
Circular dependency: node_modules/zod-to-json-schema/dist/esm/parseDef.js -> node_modules/zod-to-json-schema/dist/esm/parsers/object.js -> node_modules/zod-to-json-schema/dist/esm/parseDef.js
Circular dependency: node_modules/zod-to-json-schema/dist/esm/parseDef.js -> node_modules/zod-to-json-schema/dist/esm/parsers/optional.js -> node_modules/zod-to-json-schema/dist/esm/parseDef.js
Circular dependency: node_modules/zod-to-json-schema/dist/esm/parseDef.js -> node_modules/zod-to-json-schema/dist/esm/parsers/pipeline.js -> node_modules/zod-to-json-schema/dist/esm/parseDef.js
Circular dependency: node_modules/zod-to-json-schema/dist/esm/parseDef.js -> node_modules/zod-to-json-schema/dist/esm/parsers/promise.js -> node_modules/zod-to-json-schema/dist/esm/parseDef.js
Circular dependency: node_modules/zod-to-json-schema/dist/esm/parseDef.js -> node_modules/zod-to-json-schema/dist/esm/parsers/set.js -> node_modules/zod-to-json-schema/dist/esm/parseDef.js
Circular dependency: node_modules/zod-to-json-schema/dist/esm/parseDef.js -> node_modules/zod-to-json-schema/dist/esm/parsers/tuple.js -> node_modules/zod-to-json-schema/dist/esm/parseDef.js
Circular dependency: node_modules/zod-to-json-schema/dist/esm/parseDef.js -> node_modules/zod-to-json-schema/dist/esm/parsers/readonly.js -> node_modules/zod-to-json-schema/dist/esm/parseDef.js

[StefanTerdell]

Thanks for opening an issue. Is this an actual problem?

[Carlos-err406]

Thanks for opening an issue. Is this an actual problem?

more of an unconformity since it apparently does not break the app, just nasty logs at build

[willgottschalk]

Seems like a duplicate of #113

We're seeing this issue with v3.22.5. While I can't tell any noticeable issues, I'd prefer not to see these logs when building my app

[willgottschalk]

Also, this doesn't seem like a hard fix either.

zod-to-json-schema/src/parsers/optional.ts

Line 2 in b96cd60

import { JsonSchema7Type, parseDef } from "../parseDef.js";

The first import is a type and the second import could be made into an argument of the function to break the cycle. That's probably the least invasive way

import { ZodDefaultDef } from "zod";
import type { JsonSchema7Type, ParseDefFunc } from "../parseDef.js";
import { Refs } from "../Refs.js";

export function parseDefaultDef(
  _def: ZodDefaultDef,
  refs: Refs,
  parseDef: ParseDefFunc,
): JsonSchema7Type & { default: any } {
  return {
    ...parseDef(_def.innerType._def, refs),
    default: _def.defaultValue(),
  };
}

There are other ways to break cycles but this seems to require the least amount of rework to the code

[StefanTerdell]

I'm hesitant to change the structure of the code to accommodate weird bundler behavior when there's nothing wrong with it.

I'd prefer not to see these logs when building my app

Maybe there are some log level settings you can try?

Not closing the issue, mind. It's just not super high on the list of priorities ATM. :)

[pekeler]

I got the exact same problem in my project. Noticed it's been reported over at ciscoheat/sveltekit-superforms#350 as well.

[Rar9]

Any fix in sight?

[NanezX]

Same thing here! Glad that I found this issue.

I would love to get rid of those warning logs, even someone does not find any relatable runtime error, I don't like to have those logs on building time

[NanezX]

Hey guys, I submitted a PR with a possible solution #164 and I think it could work

Hopefully @StefanTerdell will like this solution, let's wait for him :)

[StefanTerdell]

Hey @NanezX, your contributions are released now as part of 3.24.2. Would you (or anyone else) mind confirming this fixes the logging issue?