Updating a date with the node-postgres adapter

I am trying to update a date value like this:

        await db
          .update(schema.order)
          .set({ completedDate: new Date() }) // same error with sql`now()`
          .where(eq(schema.order.id, order.id));

        await db
          .update(schema.order)
          .set({ completedDate: new Date() }) // same error with sql`now()`
          .where(eq(schema.order.id, order.id));

But I get an error

value.toISOString is not a function

value.toISOString is not a function

, and searching that up, I came across this: https://github.com/drizzle-team/drizzle-orm/issues/2388

In the schema it is defined like this:

completedDate: timestamp({ withTimezone: true }),

completedDate: timestamp({ withTimezone: true }),

I am not really sure how to solve it, I'd prefer to solve it without using type: 'string', because there's other code depending on it being a Date object returned..
Am I doing something wrong or is it just broken?

TOSL•2/11/25, 4:09 PM

All the db drivers by default always returns strings for dates. You have to set the on you column to tell Drizzle how to handle it from there.

TTOSL All the db drivers by default always returns strings for dates. You have to set ...

AlveOP•2/11/25, 8:19 PM

even when setting mode: "date" I get the exact same error

AAlve even when setting mode: "date" I get the exact same error

TOSL•2/11/25, 9:35 PM

Assuming you haven't had it set from the very beginning and you've seeded your db with some things already, maybe try dropping your db.

francis•2/12/25, 3:09 PM

dropping the db won't help here, this isn't a sql datatype problem

francis•2/12/25, 3:10 PM

check with a select to see what actual type you get out the other end, to make sure it's operating in date mode properly

TTOSL Assuming you haven't had it set from the very beginning and you've seeded your d...

AlveOP•2/12/25, 11:11 PM

Yeah it pruned my local db when I changed it so it should be fine

Ffrancis check with a select to see what actual type you get out the other end, to make s...

AlveOP•2/12/25, 11:15 PM

I get out a date if I'm querying it, and that date object has toISOString. This happens somewhere internally in drizzle

francis•2/12/25, 11:16 PM

if you pass in the result of the query to an update, what happens?

Ffrancis if you pass in the result of the query to an update, what happens?

AlveOP•2/12/25, 11:24 PM

that works fine apparently

AlveOP•2/12/25, 11:25 PM

but setting completedDate: queryResult.completedDate fails

Ffrancis if you pass in the result of the query to an update, what happens?

AlveOP•2/12/25, 11:28 PM

I think I figured it out, I have a updatedAt that I forgot, that is exactly like what's in the issue I linked on the table

If I remove the $onUpdate it works, so I guess the date works, but I need to provide the old date from updatedAt...