nullish with id autoincrement

[vigyanhoon]

Right now zod-prisma will generate zod file without nullish for ids that are auto incremented. We dont pass ids from client where its auto incremented, so it should be optional/nullish when generated.

May be you guys need following code fix in your types.ts

[ic-dominic]

Seconded, I've ran into this issue multiple times already.

I think by default the @default(autoincrement()) should make the zod object ID nullish and maybe an opt-out can be provided to the generator for people that don't want this behaviour.

[sterba-tech]

My solution

Currently I am solving this issue by re-exporting the generated model and omitting fields I do not want to have there.

import { OrderModel as OrderModelGenerated } from "db/zod"

export const OrderModel = OrderModelGenerated.omit({
  id: true,
  createdAt: true,
  updatedAt: true,
})

Suggestions

I think the best way how to solve would be adding excludeFields to the generator configuration.

generator zod {
  provider                 = "zod-prisma"
  excludeFields            = ["id", "createdAt", "updatedAt"]
  // ... other options
}

Another way could be manually add some zod-omit annotation above the field itself in Prisma schema.

Just starting with this problematic, might get to creating a PR later when I will know more pros and cons of possible solutions.

[vladaman]

We got the same issue. We use Zod + Prisma + tRPC + fastify, which requires the client to provide an id when creating a new record.

So far we've solved it with following:

export const connectorsRouter = router({
  create: authedProcedure.input(UserModel.omit({id: true, updatedAt: true, createAt: true}))
    .mutation(async (opts) => {
      let server = opts.ctx.req.server;
      return await server.prisma.user.create({
        data: {
          ...opts.input, id: DbHelpers.getRandomArbitrary(100_000_000, 900_000_000) // generates random number
        }
      })
    }),

[andrewmartin]

@vladaman thanks for your suggestion, this seems like a worthy solution! Would be really good to have some simple decorator option to make them nullish though. I'm going to +1 this thread :)