Skip to content

Conversation

@initstring
Copy link
Owner

Prisma v7 Upgrade

  • Architecture: Adopted the Minimal Adapter Pattern to support Prisma v7 without database migrations.
  • Configuration: Moved database connection logic from schema.prisma (datasource.url is deprecated) to a new type-safe prisma.config.ts.
  • Runtime: Updated src/server/db.ts and all maintenance scripts (init, seed, etc.) to use @prisma/adapter-pg.

Verification

  • Test Suite: All 49 test suites passed (npm run test).
  • Prisma: prisma generate and prisma migrate deploy verified working.

@initstring initstring self-assigned this Dec 26, 2025
@initstring
Copy link
Owner Author

@codex please review

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines 8 to 10
datasource db {
provider = "postgresql"
url = env("DATABASE_URL")
}

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Restore datasource url for Prisma schema validity

The Prisma schema now omits the url field in the db datasource, which causes validation to fail (P1012: Argument "url" is missing in data source block "db"). The new prisma.config.ts does not satisfy the schema requirement, so prisma validate/migrate/generate and any CI steps that invoke the CLI will error before the app can run. Please keep a datasource url (or equivalent) in the schema so the client and migrations can be produced.

Useful? React with 👍 / 👎.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants