fix(zod): address PR review + green CI for zod4 baseline

Resolve review feedback and unblock CI on #3642:

- resolveIsZodV4: treat "packageJson present but no detectable zod
  version" the same as "no packageJson" and default to Zod 4, matching
  the documented 'auto' intent (Copilot). Add regression coverage.
- normalizeOperationsAndTags: skip emitting a normalized zod object for
  operation/tag overrides that carry only unsupported (output-only)
  fields, so they are true no-ops and cannot inject default
  strict/generate/coerce values that override global override.zod.*
  during downstream merges (CodeRabbit). Add regression coverage.
- types: convert empty `extends` interfaces (OperationZodOptions,
  NormalizedOperationZodOptions) to type aliases to satisfy the
  no-empty-object-type lint rule.
- Run formatter on types.ts / options.test.ts (fixes the failing
  `vp fmt --check` CI step).
- Regenerate 3 stale petstore-tags-split-zod-schemas snapshots to the
  Zod 4 baseline (zod.email() instead of zod.string().email()).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

Author: the-ult

packages/core/src/types.ts

@@ -875,7 +875,7 @@ export interface ZodOptions extends BaseZodOptions {
  * schema-layout settings belong on `override.zod`, not `override.operations.*`
  * / `override.tags.*`.
  */
-export interface OperationZodOptions extends BaseZodOptions {}
+export type OperationZodOptions = BaseZodOptions;
 
 export interface EffectOptions {
   strict?: ZodOptions['strict'];
@@ -934,16 +934,10 @@ export interface NormalizedZodOptions {
   timeOptions: ZodTimeOptions;
 }
 
-export interface NormalizedOperationZodOptions
-  extends Pick<
-    NormalizedZodOptions,
-    | 'strict'
-    | 'generate'
-    | 'coerce'
-    | 'preprocess'
-    | 'params'
-    | 'useBrandedTypes'
-  > {}
+export type NormalizedOperationZodOptions = Pick<
+  NormalizedZodOptions,
+  'strict' | 'generate' | 'coerce' | 'preprocess' | 'params' | 'useBrandedTypes'
+>;
 
 export interface NormalizedEffectOptions {
   strict: NormalizedZodOptions['strict'];

packages/orval/src/utils/options.test.ts

@@ -1079,7 +1079,8 @@ describe('normalizeOptions', () => {
         expect.stringContaining('override.tags.Pets.zod'),
       );
       expect(
-        'version' in (normalized.output.override.operations.listPets?.zod ?? {}),
+        'version' in
+          (normalized.output.override.operations.listPets?.zod ?? {}),
       ).toBe(false);
       expect(
         'generateMeta' in (normalized.output.override.tags.Pets?.zod ?? {}),
@@ -1090,6 +1091,57 @@ describe('normalizeOptions', () => {
     }
   });
 
+  it('emits no zod object for operation/tag overrides that only carry unsupported fields', async () => {
+    const workspace = await createTempWorkspace();
+    logWarningSpy.mockClear();
+
+    try {
+      const normalized = await normalizeOptions(
+        {
+          input: {
+            target: {
+              openapi: '3.1.0',
+              info: { title: 'Test', version: '1.0.0' },
+              paths: {},
+            },
+          },
+          output: {
+            target: './generated.ts',
+            client: 'zod',
+            override: {
+              operations: {
+                listPets: {
+                  zod: {
+                    version: 3,
+                  } as never,
+                },
+              },
+              tags: {
+                Pets: {
+                  zod: {
+                    generateMeta: true,
+                  } as never,
+                },
+              },
+            },
+          },
+        },
+        workspace,
+      );
+
+      // An unsupported-only entry must be a true no-op: no normalized zod
+      // object is emitted, so default strict/generate/coerce values can't leak
+      // in and override global `override.zod.*` during downstream merges.
+      expect(
+        normalized.output.override.operations.listPets?.zod,
+      ).toBeUndefined();
+      expect(normalized.output.override.tags.Pets?.zod).toBeUndefined();
+    } finally {
+      await rm(workspace, { recursive: true, force: true });
+      logWarningSpy.mockClear();
+    }
+  });
+
   it('resolves global query mutators relative to the output workspace', async () => {
     const workspace = await createTempWorkspace();
 

packages/orval/src/utils/options.ts

@@ -954,6 +954,20 @@ function normalizeOperationsAndTags(
           );
         }
 
+        // Only emit a normalized zod object when the entry actually carries a
+        // supported operation-level field. Otherwise an unsupported-only entry
+        // (e.g. `{ version: 3 }`) would inject default strict/generate/coerce
+        // values that override global `override.zod.*` during downstream merges,
+        // contradicting the "ignored" warning above.
+        const hasSupportedOperationZodConfig =
+          !!zod &&
+          (zod.strict !== undefined ||
+            zod.generate !== undefined ||
+            zod.coerce !== undefined ||
+            zod.preprocess !== undefined ||
+            zod.params !== undefined ||
+            zod.useBrandedTypes !== undefined);
+
         return [
           key,
           {
@@ -976,7 +990,7 @@ function normalizeOperationsAndTags(
                   query: normalizeQueryOptions(query, workspace, global.query),
                 }
               : {}),
-            ...(zod
+            ...(hasSupportedOperationZodConfig && zod
               ? {
                   zod: {
                     strict: {

packages/zod/src/compatible-v4.test.ts

@@ -114,6 +114,14 @@ describe('resolveIsZodV4', () => {
     it('defaults to v4 when no packageJson is available', () => {
       expect(resolveIsZodV4('auto', undefined)).toBe(true);
     });
+
+    it('defaults to v4 when packageJson has no detectable zod version', () => {
+      expect(resolveIsZodV4('auto', {})).toBe(true);
+      expect(resolveIsZodV4('auto', { dependencies: {} })).toBe(true);
+      expect(
+        resolveIsZodV4('auto', { dependencies: { 'other-pkg': '1.0.0' } }),
+      ).toBe(true);
+    });
   });
 
   describe('when version is pinned explicitly', () => {

packages/zod/src/compatible-v4.ts

@@ -48,8 +48,15 @@ export const resolveIsZodV4 = (
   }
 
   // 'auto' (or unset) — infer from the installed/resolved zod version and
-  // default to Zod 4 when the target workspace has no detectable package yet.
-  return !packageJson || isZodVersionV4(packageJson);
+  // default to Zod 4 when the target workspace has no detectable zod version
+  // yet. Treat "no packageJson" and "packageJson without a detectable zod
+  // version" identically so partially-installed workspaces still get the
+  // modern baseline instead of silently falling back to Zod 3.
+  if (!packageJson || !getZodPackageVersion(packageJson)) {
+    return true;
+  }
+
+  return isZodVersionV4(packageJson);
 };
 
 export const getZodDateFormat = (isZodV4: boolean) => {

tests/__snapshots__/axios/petstore-tags-split-zod-schemas/model/pets/pet.zod.ts

@@ -36,7 +36,7 @@ export const Pet = zod
       id: zod.number(),
       name: zod.string(),
       tag: zod.string().optional(),
-      email: zod.string().email().optional(),
+      email: zod.email().optional(),
       callingCode: zod.enum(['+33', '+420']).optional(),
       country: zod.enum(["People's Republic of China", 'Uruguay']).optional(),
     }),

tests/__snapshots__/axios/petstore-tags-split-zod-schemas/model/pets/petWithTag.zod.ts

@@ -38,7 +38,7 @@ export const PetWithTag = zod.object({
         id: zod.number(),
         name: zod.string(),
         tag: zod.string().optional(),
-        email: zod.string().email().optional(),
+        email: zod.email().optional(),
         callingCode: zod.enum(['+33', '+420']).optional(),
         country: zod.enum(["People's Republic of China", 'Uruguay']).optional(),
       }),

tests/__snapshots__/axios/petstore-tags-split-zod-schemas/model/pets/pets.zod.ts

@@ -37,7 +37,7 @@ export const Pets = zod.array(
         id: zod.number(),
         name: zod.string(),
         tag: zod.string().optional(),
-        email: zod.string().email().optional(),
+        email: zod.email().optional(),
         callingCode: zod.enum(['+33', '+420']).optional(),
         country: zod.enum(["People's Republic of China", 'Uruguay']).optional(),
       }),