feat(drizzle): update schemas & refactor

[realmikesolo]

I have updated schemas and types with email being nullable in users table. Also, I have bumped Drizzle ORM & Drizzle Kit versions and refactored tests.

☕️ Reasoning

As already discussed in discord channel, email column should be nullable.

[socket-security]

No dependency changes detected. Learn more about Socket for GitHub ↗︎

👍 No dependency changes detected in pull request

[realmikesolo]

I have added id to fixtures because NextAuth always pass value for id, so I think it should be in fixtures to provide correct test cases.

[ndom91]

@realmikesolo it looks like the pnpm-lock.yaml is from a much older commit. By just bumping two versions we shuoldn't get ~30k changed lines 😅 Would be great if you could fix that.

Maybe git checkout pnpm-lock.yaml origin/main to get the latest version fo that file from main, then run pnpm install again to update it with your bumped drizzle versions, and thennn you should have a normal up-to-date pnpm-lock.yaml to commit 🙏

[realmikesolo]

Hello @ndom91! I have fixed pnpm-lock.yaml

[ndom91]

Nice, looks much better!

Out of curiosity, do the non-PK columns really all need that isPrimaryKey: false? 🤔

[realmikesolo]

Hey @ndom91. I have updated type, so if we don't need isPrimaryKey: false for non-PK columns. But I have faced this issue Cannot find module '@auth/core/adapters' or its corresponding type declarations and it is a bit strange for me because I didn't change any dependencies related to auth/core. Could you please check it?

[ndom91]

Hey @ndom91. I have updated type, so if we don't need isPrimaryKey: false for non-PK columns. But I have faced this issue Cannot find module '@auth/core/adapters' or its corresponding type declarations and it is a bit strange for me because I didn't change any dependencies related to auth/core. Could you please check it?

Yeah I don't have time today, but this weekend I can definitely take a closer look and hopefully get this merged. The lock file definitely looks much better!

[realmikesolo]

Hey @ndom91. I have updated type, so if we don't need isPrimaryKey: false for non-PK columns. But I have faced this issue Cannot find module '@auth/core/adapters' or its corresponding type declarations and it is a bit strange for me because I didn't change any dependencies related to auth/core. Could you please check it?

Yeah I don't have time today, but this weekend I can definitely take a closer look and hopefully get this merged. The lock file definitely looks much better!

Ok, thank you very much!

[ndom91]

Hey @ndom91. I have updated type, so if we don't need isPrimaryKey: false for non-PK columns. But I have faced this issue Cannot find module '@auth/core/adapters' or its corresponding type declarations and it is a bit strange for me because I didn't change any dependencies related to auth/core. Could you please check it?

Where are you running into this @auth/core/adapters issue? I was abel to run all the tests successfully still

[realmikesolo]

Hey @ndom91. I have updated type, so if we don't need isPrimaryKey: false for non-PK columns. But I have faced this issue Cannot find module '@auth/core/adapters' or its corresponding type declarations and it is a bit strange for me because I didn't change any dependencies related to auth/core. Could you please check it?

Where are you running into this @auth/core/adapters issue? I was abel to run all the tests successfully still

I was facing this issue during the build. It might be problem on my local setup then. Thank you for your time