Skip to content

fix: downgrade the driver of sqlite and mariadb to 6.18.0#32

Merged
jiashengguo merged 2 commits intodevfrom
jiasheng-dev
Mar 31, 2026
Merged

fix: downgrade the driver of sqlite and mariadb to 6.18.0#32
jiashengguo merged 2 commits intodevfrom
jiasheng-dev

Conversation

@jiashengguo
Copy link
Copy Markdown
Member

@jiashengguo jiashengguo commented Mar 31, 2026

Summary by CodeRabbit

  • Chores

    • Bumped package version to 0.4.1
    • Updated Prisma adapter dependency ranges
  • Improvements

    • Enhanced compatibility with SQLite and MariaDB adapters, reducing adapter-related installation/runtime issues

Copilot AI review requested due to automatic review settings March 31, 2026 04:13
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 31, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 4b5c0ecc-55ff-40a1-8467-25ce82d039bf

📥 Commits

Reviewing files that changed from the base of the PR and between 1ccf008 and 6613caf.

📒 Files selected for processing (1)
  • src/server.ts
🚧 Files skipped from review as they are similar to previous changes (1)
  • src/server.ts

📝 Walkthrough

Walkthrough

Bumps package version to 0.4.1 and updates Prisma adapter ranges to ^6.18.0. Refactors SQLite adapter loading to require the module object and select constructor via multiple export fallbacks; changes MariaDB adapter constructor name used during instantiation.

Changes

Cohort / File(s) Summary
Dependency updates
package.json
Package version bumped to 0.4.1. Prisma adapter dependency ranges changed to ^6.18.0 (was ^7.x).
Adapter loading / constructors
src/server.ts
SQLite path now requires the module object and selects constructor from _mod.PrismaBetterSQLite3 ?? _mod.PrismaBetterSqlite3 ?? _mod.default. MariaDB/MySQL path now instantiates the PrismaMariaDb export (renamed from prior PrismaMariaDB).

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Poem

🐇 I hopped through versions, tiny paws in tow,
I swapped a name, let fallbacks glow,
From module crates I pick the right key,
So adapters wake and run with glee,

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: downgrading Prisma adapter drivers for SQLite and MariaDB to version 6.18.0, which is reflected in both package.json and src/server.ts modifications.
Docstring Coverage ✅ Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch jiasheng-dev

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the proxy’s Prisma driver adapter dependencies (SQLite + MariaDB) to the 6.x line and adjusts the SQLite adapter loading logic in createAdapter to be compatible with adapter export name differences across Prisma/adapter versions.

Changes:

  • Downgrade @prisma/adapter-better-sqlite3 and @prisma/adapter-mariadb dependency ranges to ^6.18.0 (lockfile resolves to 6.19.2).
  • Update SQLite adapter require/constructor resolution to support multiple export names.
  • Bump package version from 0.4.0 to 0.4.1.

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated 4 comments.

File Description
src/server.ts Makes SQLite adapter loading resilient to export name changes by resolving multiple possible exports.
package.json Updates dependency ranges for Prisma adapters and bumps package version.
pnpm-lock.yaml Updates resolved Prisma adapter and transitive dependency versions consistent with the new ranges.
Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Copy Markdown

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@package.json`:
- Line 33: The code destructures the wrong export name (PrismaMariaDB) from the
MariaDB adapter causing a runtime error; update the destructured import in
src/server.ts to the correct export name PrismaMariaDb (replace any occurrences
of PrismaMariaDB with PrismaMariaDb where you require or import from
'@prisma/adapter-mariadb') so the adapter resolves correctly at runtime.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 620b04c3-8e34-4ffc-8f55-9caa84130c36

📥 Commits

Reviewing files that changed from the base of the PR and between d4841a3 and 1ccf008.

⛔ Files ignored due to path filters (1)
  • pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
📒 Files selected for processing (2)
  • package.json
  • src/server.ts

@jiashengguo jiashengguo merged commit a531a7f into dev Mar 31, 2026
3 checks passed
@jiashengguo jiashengguo deleted the jiasheng-dev branch March 31, 2026 04:33
jiashengguo added a commit that referenced this pull request Mar 31, 2026
fix: downgrade the driver of sqlite and mariadb to 6.18.0 (#32)
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