z.discriminatordUnion can't handle Unions

[WORMSS]

z.discriminatordUnion() can't handle z.union()

const baseAttribute = const baseAttribute = z.object({
  name: nameSchema,
  code: z.string(),
});
const linkAttribute = baseAttribute.extend({
  type: z.literal('Link'),
  options: z.object({
    code: z.string(),
  }),
});
const otherAttribute = baseAttribute.extend({
  type: z.union([
    z.literal('Time'),
    z.literal('GeoJson'),
    z.literal('Boolean'),
  ]),
});

export const attributeSchema = z.discriminatedUnion('type', [linkAttribute, otherAttribute]);

I've just seen that you want to deprecate discriminatedUnion from #2109

But I was hoping this would be a super quick fix.

else if (type instanceof ZodUnion) {
  const accumulator = [];
  for ( const option of type.options ) {
    const types = getDiscriminator(option);
    if (types === null) {
      return null; // nested is of unknown type.
    }
    accumulator.push(...types);
  }
  return accumulator;
}

[JacobWeisenburger]

considering discriminatedUnion is going to be deprecated. I'm not sure any more work should be done on it.

@colinhacks thoughts?

[JacobWeisenburger]

@WORMSS, I'm going to close this issue. Feel free to reopen if you have more to say.

[stdedos]

Is there a plan for that depreciation? It seems that #2106 was last updated last month.

Would there be a "quick fix" that would address this instead?

[dplantera]

Is there a plan for that depreciation? It seems that #2106 was last updated last month.

Would there be a "quick fix" that would address this instead?

You might wanna checkout #1845 (comment) where teunmooij introduced a dedicated fix with z.enum support which could handle your usecase. z.enum works fine for primitive homogeneous discriminator properties like in your example.

zod.discriminatedunion.