Type safety on .set() method?

bennettcohen · 2024-07-11T14:39:13.694Z

Hey folks - I was wondering if there is a way to get type safety when updating a row using the .set method? In the example below, `updated_at` is the table name, not the property name `updatedAt`. This code runs with no errors, but of course never updates the updated at because it doesn't exist! ```typescript const data = await db .update(verifications) .set({ status: 'inactive', updated_at: sql`NOW()`, }) .where(and(eq(sql`${persons.id}::text`, personId))) .returning(); ``` Any tips for getting this to be type-safe ?

R

rphlmr ⚡•7/11/24, 3:13 PM

Hey

Can you share your tsconfig?

B

bennettcohenOP•7/12/24, 4:04 PM

oh hmm

B

bennettcohenOP•7/12/24, 4:05 PM

{
  "compilerOptions": {
    "target": "ESNext",
    "module": "CommonJS",
    "lib": ["es6"],
    "allowJs": true,
    "outDir": "build",
    "rootDir": "src",
    "strict": true,
    "noImplicitAny": true,
    "esModuleInterop": true,
    "moduleResolution": "node",
    "resolveJsonModule": true,
    "skipLibCheck": true,
    "baseUrl": "./src",
    "paths": {
      "@routes/*": ["routes/*"],
      "@v1/*": ["routes/v1/*"],
      "@utils/*": ["lib/utils/*"],
      "@middleware/*": ["lib/middleware/*"],
      "@schemas/*": ["schemas/*"],
      "@db": ["db"],
      "@db/*": ["db/*"],
      "@services/*": ["lib/services/*"],
      "@jobs/*": ["jobs/*"],
    }
  },
  "include": [
    "src",
    "src/**/*",
    "src/*",
    "src/*/*/",
    "src/**/*",
    "global.d.ts"
  ]
}

{
  "compilerOptions": {
    "target": "ESNext",
    "module": "CommonJS",
    "lib": ["es6"],
    "allowJs": true,
    "outDir": "build",
    "rootDir": "src",
    "strict": true,
    "noImplicitAny": true,
    "esModuleInterop": true,
    "moduleResolution": "node",
    "resolveJsonModule": true,
    "skipLibCheck": true,
    "baseUrl": "./src",
    "paths": {
      "@routes/*": ["routes/*"],
      "@v1/*": ["routes/v1/*"],
      "@utils/*": ["lib/utils/*"],
      "@middleware/*": ["lib/middleware/*"],
      "@schemas/*": ["schemas/*"],
      "@db": ["db"],
      "@db/*": ["db/*"],
      "@services/*": ["lib/services/*"],
      "@jobs/*": ["jobs/*"],
    }
  },
  "include": [
    "src",
    "src/**/*",
    "src/*",
    "src/*/*/",
    "src/**/*",
    "global.d.ts"
  ]
}

{
  "compilerOptions": {
    "target": "ESNext",
    "module": "CommonJS",
    "lib": ["es6"],
    "allowJs": true,
    "outDir": "build",
    "rootDir": "src",
    "strict": true,
    "noImplicitAny": true,
    "esModuleInterop": true,
    "moduleResolution": "node",
    "resolveJsonModule": true,
    "skipLibCheck": true,
    "baseUrl": "./src",
    "paths": {
      "@routes/*": ["routes/*"],
      "@v1/*": ["routes/v1/*"],
      "@utils/*": ["lib/utils/*"],
      "@middleware/*": ["lib/middleware/*"],
      "@schemas/*": ["schemas/*"],
      "@db": ["db"],
      "@db/*": ["db/*"],
      "@services/*": ["lib/services/*"],
      "@jobs/*": ["jobs/*"],
    }
  },
  "include": [
    "src",
    "src/**/*",
    "src/*",
    "src/*/*/",
    "src/**/*",
    "global.d.ts"
  ]
}

{
  "compilerOptions": {
    "target": "ESNext",
    "module": "CommonJS",
    "lib": ["es6"],
    "allowJs": true,
    "outDir": "build",
    "rootDir": "src",
    "strict": true,
    "noImplicitAny": true,
    "esModuleInterop": true,
    "moduleResolution": "node",
    "resolveJsonModule": true,
    "skipLibCheck": true,
    "baseUrl": "./src",
    "paths": {
      "@routes/*": ["routes/*"],
      "@v1/*": ["routes/v1/*"],
      "@utils/*": ["lib/utils/*"],
      "@middleware/*": ["lib/middleware/*"],
      "@schemas/*": ["schemas/*"],
      "@db": ["db"],
      "@db/*": ["db/*"],
      "@services/*": ["lib/services/*"],
      "@jobs/*": ["jobs/*"],
    }
  },
  "include": [
    "src",
    "src/**/*",
    "src/*",
    "src/*/*/",
    "src/**/*",
    "global.d.ts"
  ]
}

Anything you see?

B

bennettcohenOP•7/12/24, 4:05 PM

Pretty new to the TS world

B

bennettcohenOP•7/12/24, 4:05 PM

Thanks @Raphaël M (@rphlmr)

R

rphlmr ⚡•7/12/24, 4:28 PM

is your schema inside

src

src

?

R

rphlmr ⚡•7/12/24, 4:30 PM

your

include

include

"doesn’t look common" (but there is so many possibilities haha)

"**/*.ts","**/*.tsx", "global.d.ts"

"**/*.ts","**/*.tsx", "global.d.ts"

"**/*.ts","**/*.tsx", "global.d.ts"

"**/*.ts","**/*.tsx", "global.d.ts" should be enough to tell TypeScript that it has to check all of your files

B

bennettcohenOP•7/12/24, 4:33 PM

Ah was able to get it working...i think. The culprit was how i was initializing the db client. There is some weird hacky stuff around the SSL certificate when connecting to my prod db

B

bennettcohenOP•7/12/24, 4:33 PM

Thank u!

B

bennettcohenOP•7/12/24, 4:37 PM

Btw, would you know if there a nice way to do optional singles here? Often I'm retrieving one row and can be annoying to have to do

data?.[0]

data?.[0]

data?.[0]

data?.[0] @Raphaël M (@rphlmr)

R

rphlmr ⚡•7/12/24, 4:45 PM

update always returns an array this should be enough

const [updated] = await db
    .update(verifications)
    .set({
      status: 'inactive',
      updated_at: sql`NOW()`,
    })
    .where(and(eq(sql`${persons.id}::text`, personId)))
    .returning();

const [updated] = await db
    .update(verifications)
    .set({
      status: 'inactive',
      updated_at: sql`NOW()`,
    })
    .where(and(eq(sql`${persons.id}::text`, personId)))
    .returning();

const [updated] = await db
    .update(verifications)
    .set({
      status: 'inactive',
      updated_at: sql`NOW()`,
    })
    .where(and(eq(sql`${persons.id}::text`, personId)))
    .returning();

const [updated] = await db
    .update(verifications)
    .set({
      status: 'inactive',
      updated_at: sql`NOW()`,
    })
    .where(and(eq(sql`${persons.id}::text`, personId)))
    .returning();

Type safety on .set() method?

Similar Threads

Similar Threads

Similar Threads