Updateable<T> allow undefined?

Why cannot Updateable<T> allow undefined?

I’m using zod for input validation. For an update, all properties besides an id should be optional. However, this seems to not allow me to use .set within Kysely because it does not allow undefined.

Use case:

import { Kysely, KyselyConfig, Updateable } from "kysely";
import { z } from "zod";

interface UserTable {
  id: number;
  firstName: string | null;
  lastName: string | null;
}
interface Database {
  user: UserTable;
}
const db = new Kysely<Database>({} as KyselyConfig);

const userSchema = z.object({
  id: z.number(),
  firstName: z.string().optional(),
  lastName: z.string().optional(),
});

const user = userSchema.parse({
  id: 1,
  firstName: "Ben",
});

// doesn't work
db.updateTable("user").set(user).where("id", "=", 1).execute();
// works
db.updateTable("user")
  .set(user as Updateable<UserTable>)
  .where("id", "=", 1)
  .execute();

Error:

[koskimas]

undefined and missing property are both allowed unless exactOptionalPropertyTypes is true, which treats these two as different cases. Missing/undefined values is a common case, but people using settings this strict is not.

Let’s fix this unless we need hacks.

[bestickley]

awesome. thanks, @igalklebanov! This seems like a very common use case. I hope your PR gets merged, but if it cannot maybe Kysely could export a helper type like type NullableColumn<T> = ColumnType<T | null, T | undefined | null, T | undefined | null>.