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

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

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

Conversation

@jiashengguo

@jiashengguojiashengguo commented Mar 31, 2026

Copy link
Copy Markdown
Member

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

CopilotAI review requested due to automatic review settings March 31, 2026 04:13
@coderabbitai

coderabbitaiBot commented Mar 31, 2026

Copy link
Copy Markdown
Contributor

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 nameStatusExplanation
Description Check✅ PassedCheck skipped - CodeRabbit’s high-level summary is enabled.
Title check✅ PassedThe 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✅ PassedDocstring 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.

CopilotAI left a comment

Copy link
Copy Markdown

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.

FileDescription
src/server.tsMakes SQLite adapter loading resilient to export name changes by resolving multiple possible exports.
package.jsonUpdates dependency ranges for Prisma adapters and bumps package version.
pnpm-lock.yamlUpdates 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.

Comment threadsrc/server.ts
Comment threadsrc/server.ts
Comment threadpackage.json
Comment threadpnpm-lock.yaml

@coderabbitaicoderabbitaiBot left a comment

Copy link
Copy Markdown
Contributor

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

Comment threadpackage.json
@jiashengguo
jiashengguo merged commit a531a7f into devMar 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 freeto 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

@jiashengguo
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Add copy buttons to all
 blocks\n(function() {\n function addCopyButtons() {\n document.querySelectorAll('pre code').forEach(function(codeBlock) {\n if (codeBlock.parentElement.hasAttribute('data-copy-added')) return;\n codeBlock.parentElement.setAttribute('data-copy-added', 'true');\n \n var btn = document.createElement('button');\n btn.textContent = 'Copy';\n btn.style.cssText = 'position:absolute;top:4px;right:4px;padding:2px 8px;font-size:11px;background:#4ecdc4;border:none;border-radius:4px;color:#1a1a2e;cursor:pointer;opacity:0.7;transition:opacity 0.2s;';\n btn.onmouseover = function() { this.style.opacity = '1'; };\n btn.onmouseout = function() { this.style.opacity = '0.7'; };\n btn.onclick = function() {\n navigator.clipboard.writeText(codeBlock.textContent).then(function() {\n btn.textContent = 'Copied!';\n setTimeout(function() { btn.textContent = 'Copy'; }, 1500);\n });\n };\n codeBlock.parentElement.style.position = 'relative';\n codeBlock.parentElement.appendChild(btn);\n });\n }\n \n addCopyButtons();\n \n // Re-run on dynamic content\n var observer = new MutationObserver(addCopyButtons);\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Add Copy Buttons to Code Blocks");
}
} catch(__e) { console.warn('[Userscript:Add Copy Buttons to Code Blocks]', __e); }
})();
(function(){
try {
var __m = "github.com";
var __re = new RegExp('^' + "github\\.com" + '
Skip to content

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

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

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

Conversation

@jiashengguo

@jiashengguojiashengguo commented Mar 31, 2026

Copy link
Copy Markdown
Member

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

CopilotAI review requested due to automatic review settings March 31, 2026 04:13
@coderabbitai

coderabbitaiBot commented Mar 31, 2026

Copy link
Copy Markdown
Contributor

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 nameStatusExplanation
Description Check✅ PassedCheck skipped - CodeRabbit’s high-level summary is enabled.
Title check✅ PassedThe 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✅ PassedDocstring 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.

CopilotAI left a comment

Copy link
Copy Markdown

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.

FileDescription
src/server.tsMakes SQLite adapter loading resilient to export name changes by resolving multiple possible exports.
package.jsonUpdates dependency ranges for Prisma adapters and bumps package version.
pnpm-lock.yamlUpdates 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.

Comment threadsrc/server.ts
Comment threadsrc/server.ts
Comment threadpackage.json
Comment threadpnpm-lock.yaml

@coderabbitaicoderabbitaiBot left a comment

Copy link
Copy Markdown
Contributor

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

Comment threadpackage.json
@jiashengguo
jiashengguo merged commit a531a7f into devMar 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 freeto 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

@jiashengguo
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Force GitHub README to respect dark mode\n(function() {\n var style = document.createElement('style');\n style.textContent = '\n .markdown-body {\n color-scheme: dark light;\n }\n .markdown-body pre { background: #161b22 !important; }\n .markdown-body code { background: rgba(110, 118, 129, 0.4) !important; }\n .markdown-body table th, .markdown-body table td { border-color: #30363d !important; }\n .markdown-body img { background: #0d1117; }\n .markdown-body blockquote { border-left-color: #8b949e; }\n .markdown-body hr { border-color: #30363d; }\n ';\n document.head.appendChild(style);\n})();", "GitHub Dark Mode README Fix"); } } catch(__e) { console.warn('[Userscript:GitHub Dark Mode README Fix]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content

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

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

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

Conversation

@jiashengguo

@jiashengguojiashengguo commented Mar 31, 2026

Copy link
Copy Markdown
Member

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

CopilotAI review requested due to automatic review settings March 31, 2026 04:13
@coderabbitai

coderabbitaiBot commented Mar 31, 2026

Copy link
Copy Markdown
Contributor

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 nameStatusExplanation
Description Check✅ PassedCheck skipped - CodeRabbit’s high-level summary is enabled.
Title check✅ PassedThe 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✅ PassedDocstring 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.

CopilotAI left a comment

Copy link
Copy Markdown

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.

FileDescription
src/server.tsMakes SQLite adapter loading resilient to export name changes by resolving multiple possible exports.
package.jsonUpdates dependency ranges for Prisma adapters and bumps package version.
pnpm-lock.yamlUpdates 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.

Comment threadsrc/server.ts
Comment threadsrc/server.ts
Comment threadpackage.json
Comment threadpnpm-lock.yaml

@coderabbitaicoderabbitaiBot left a comment

Copy link
Copy Markdown
Contributor

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

Comment threadpackage.json
@jiashengguo
jiashengguo merged commit a531a7f into devMar 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 freeto 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

@jiashengguo
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Highlight search terms from Google/DuckDuckGo/Bing referrer\n(function() {\n var ref = document.referrer;\n var terms = [];\n \n if (ref.includes('google.com') || ref.includes('duckduckgo.com') || ref.includes('bing.com')) {\n var url = new URL(ref);\n var q = url.searchParams.get('q') || url.searchParams.get('p');\n if (q) {\n terms = q.split(/\\s+/).filter(function(t) { return t.length > 2; });\n }\n }\n \n if (terms.length === 0) return;\n \n var style = document.createElement('style');\n style.textContent = '.userscript-highlight { background: #fbbf24; color: #1a1a2e; padding: 1px 3px; border-radius: 2px; }';\n document.head.appendChild(style);\n \n function highlight(node) {\n if (node.nodeType === 3) { // text node\n var text = node.textContent;\n var found = false;\n terms.forEach(function(term) {\n var regex = new RegExp('(' + term.replace(/[.*+?^${}()|[\\]\\\\]/g, '\\\\') + ')', 'gi');\n if (regex.test(text)) {\n found = true;\n var frag = document.createDocumentFragment();\n var parts = text.split(regex);\n parts.forEach(function(part, i) {\n if (i % 2 === 0) {\n frag.appendChild(document.createTextNode(part));\n } else {\n var span = document.createElement('span');\n span.className = 'userscript-highlight';\n span.textContent = part;\n frag.appendChild(span);\n }\n });\n node.parentNode.replaceChild(frag, node);\n }\n });\n } else if (node.nodeType === 1 && node.childNodes) { // element\n var skipTags = ['SCRIPT', 'STYLE', 'NOSCRIPT', 'TEXTAREA', 'INPUT', 'SELECT'];\n if (!skipTags.includes(node.tagName)) {\n Array.from(node.childNodes).forEach(highlight);\n }\n }\n }\n \n highlight(document.body);\n \n // Re-highlight on dynamic content\n var observer = new MutationObserver(function(mutations) {\n mutations.forEach(function(m) {\n m.addedNodes.forEach(function(node) {\n if (node.nodeType === 1 || node.nodeType === 3) highlight(node);\n });\n });\n });\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Highlight Search Terms"); } } catch(__e) { console.warn('[Userscript:Highlight Search Terms]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content

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

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

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

Conversation

@jiashengguo

@jiashengguojiashengguo commented Mar 31, 2026

Copy link
Copy Markdown
Member

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

CopilotAI review requested due to automatic review settings March 31, 2026 04:13
@coderabbitai

coderabbitaiBot commented Mar 31, 2026

Copy link
Copy Markdown
Contributor

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 nameStatusExplanation
Description Check✅ PassedCheck skipped - CodeRabbit’s high-level summary is enabled.
Title check✅ PassedThe 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✅ PassedDocstring 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.

CopilotAI left a comment

Copy link
Copy Markdown

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.

FileDescription
src/server.tsMakes SQLite adapter loading resilient to export name changes by resolving multiple possible exports.
package.jsonUpdates dependency ranges for Prisma adapters and bumps package version.
pnpm-lock.yamlUpdates 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.

Comment threadsrc/server.ts
Comment threadsrc/server.ts
Comment threadpackage.json
Comment threadpnpm-lock.yaml

@coderabbitaicoderabbitaiBot left a comment

Copy link
Copy Markdown
Contributor

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

Comment threadpackage.json
@jiashengguo
jiashengguo merged commit a531a7f into devMar 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 freeto 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

@jiashengguo
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Strip utm_, fbclid, gclid, etc. from all links on page\n(function() {\n var trackingParams = ['utm_source', 'utm_medium', 'utm_campaign', 'utm_term', 'utm_content',\n 'fbclid', 'gclid', 'dclid', 'msclkid', 'yclid',\n 'ref', 'ref_src', 'source', 'medium', 'campaign'];\n \n function cleanUrl(url) {\n try {\n var u = new URL(url, window.location.origin);\n var changed = false;\n trackingParams.forEach(function(p) {\n if (u.searchParams.has(p)) {\n u.searchParams.delete(p);\n changed = true;\n }\n });\n return changed ? u.toString() : url;\n } catch (e) {\n return url;\n }\n }\n \n function cleanLinks() {\n document.querySelectorAll('a[href]').forEach(function(a) {\n var clean = cleanUrl(a.href);\n if (clean !== a.href) a.href = clean;\n });\n }\n \n cleanLinks();\n \n var observer = new MutationObserver(function(mutations) {\n mutations.forEach(function(m) {\n m.addedNodes.forEach(function(node) {\n if (node.nodeType === 1) {\n if (node.tagName === 'A') cleanLinks();\n node.querySelectorAll('a[href]').forEach(function(a) {\n var clean = cleanUrl(a.href);\n if (clean !== a.href) a.href = clean;\n });\n }\n });\n });\n });\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Remove Tracking Parameters from Links"); } } catch(__e) { console.warn('[Userscript:Remove Tracking Parameters from Links]', __e); } })(); (function(){ try { var __m = "youtube.com"; var __re = new RegExp('^' + "youtube\\.com" + '
Skip to content

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

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

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

Conversation

@jiashengguo

@jiashengguojiashengguo commented Mar 31, 2026

Copy link
Copy Markdown
Member

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

CopilotAI review requested due to automatic review settings March 31, 2026 04:13
@coderabbitai

coderabbitaiBot commented Mar 31, 2026

Copy link
Copy Markdown
Contributor

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 nameStatusExplanation
Description Check✅ PassedCheck skipped - CodeRabbit’s high-level summary is enabled.
Title check✅ PassedThe 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✅ PassedDocstring 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.

CopilotAI left a comment

Copy link
Copy Markdown

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.

FileDescription
src/server.tsMakes SQLite adapter loading resilient to export name changes by resolving multiple possible exports.
package.jsonUpdates dependency ranges for Prisma adapters and bumps package version.
pnpm-lock.yamlUpdates 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.

Comment threadsrc/server.ts
Comment threadsrc/server.ts
Comment threadpackage.json
Comment threadpnpm-lock.yaml

@coderabbitaicoderabbitaiBot left a comment

Copy link
Copy Markdown
Contributor

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

Comment threadpackage.json
@jiashengguo
jiashengguo merged commit a531a7f into devMar 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 freeto 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

@jiashengguo
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Auto-enable theater mode on YouTube\n(function() {\n function tryTheater() {\n var btn = document.querySelector('button[aria-label=\"Theater mode\"], ytd-player #player button[title=\"Theater mode\"]');\n if (btn && !btn.classList.contains('activated')) {\n btn.click();\n }\n }\n \n // Try immediately\n tryTheater();\n \n // Try after navigation (SPA)\n var lastUrl = location.href;\n setInterval(function() {\n if (location.href !== lastUrl) {\n lastUrl = location.href;\n setTimeout(tryTheater, 500);\n }\n }, 1000);\n \n // Also try on player load\n var observer = new MutationObserver(tryTheater);\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "YouTube Theater Mode Default"); } } catch(__e) { console.warn('[Userscript:YouTube Theater Mode Default]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content

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

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

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

Conversation

@jiashengguo

@jiashengguojiashengguo commented Mar 31, 2026

Copy link
Copy Markdown
Member

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

CopilotAI review requested due to automatic review settings March 31, 2026 04:13
@coderabbitai

coderabbitaiBot commented Mar 31, 2026

Copy link
Copy Markdown
Contributor

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 nameStatusExplanation
Description Check✅ PassedCheck skipped - CodeRabbit’s high-level summary is enabled.
Title check✅ PassedThe 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✅ PassedDocstring 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.

CopilotAI left a comment

Copy link
Copy Markdown

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.

FileDescription
src/server.tsMakes SQLite adapter loading resilient to export name changes by resolving multiple possible exports.
package.jsonUpdates dependency ranges for Prisma adapters and bumps package version.
pnpm-lock.yamlUpdates 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.

Comment threadsrc/server.ts
Comment threadsrc/server.ts
Comment threadpackage.json
Comment threadpnpm-lock.yaml

@coderabbitaicoderabbitaiBot left a comment

Copy link
Copy Markdown
Contributor

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

Comment threadpackage.json
@jiashengguo
jiashengguo merged commit a531a7f into devMar 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 freeto 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

@jiashengguo
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Remove or un-stick sticky/fixed headers that block content\n(function() {\n function unstick() {\n document.querySelectorAll('header, nav, [role=\"banner\"], .header, .navbar, .sticky, .fixed-top, [style*=\"position: fixed\"], [style*=\"position:sticky\"]').forEach(function(el) {\n if (el.style.position === 'fixed' || el.style.position === 'sticky' || \n getComputedStyle(el).position === 'fixed' || getComputedStyle(el).position === 'sticky') {\n el.style.position = 'static';\n el.style.top = 'auto';\n el.style.zIndex = 'auto';\n }\n });\n }\n \n unstick();\n \n var observer = new MutationObserver(unstick);\n observer.observe(document.body, { childList: true, subtree: true, attributes: true, attributeFilter: ['style', 'class'] });\n})();", "Kill Sticky Headers"); } } catch(__e) { console.warn('[Userscript:Kill Sticky Headers]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content

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

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

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

Conversation

@jiashengguo

@jiashengguojiashengguo commented Mar 31, 2026

Copy link
Copy Markdown
Member

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

CopilotAI review requested due to automatic review settings March 31, 2026 04:13
@coderabbitai

coderabbitaiBot commented Mar 31, 2026

Copy link
Copy Markdown
Contributor

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 nameStatusExplanation
Description Check✅ PassedCheck skipped - CodeRabbit’s high-level summary is enabled.
Title check✅ PassedThe 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✅ PassedDocstring 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.

CopilotAI left a comment

Copy link
Copy Markdown

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.

FileDescription
src/server.tsMakes SQLite adapter loading resilient to export name changes by resolving multiple possible exports.
package.jsonUpdates dependency ranges for Prisma adapters and bumps package version.
pnpm-lock.yamlUpdates 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.

Comment threadsrc/server.ts
Comment threadsrc/server.ts
Comment threadpackage.json
Comment threadpnpm-lock.yaml

@coderabbitaicoderabbitaiBot left a comment

Copy link
Copy Markdown
Contributor

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

Comment threadpackage.json
@jiashengguo
jiashengguo merged commit a531a7f into devMar 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 freeto 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

@jiashengguo
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Universal Dark Mode - works on any site\n(function() {\n var enabled = true;\n \n function applyDarkMode() {\n if (!enabled) return;\n \n // Create style element if it doesn't exist\n var style = document.getElementById('universal-dark-mode-style');\n if (!style) {\n style = document.createElement('style');\n style.id = 'universal-dark-mode-style';\n document.head.appendChild(style);\n }\n \n // Dark mode CSS - inverts colors but preserves images/video\n style.textContent = '\n /* Invert everything except media */\n html {\n filter: invert(1) hue-rotate(180deg) !important;\n background: #1a1a2e !important;\n }\n \n /* Restore images, videos, iframes, canvas */\n img, video, iframe, canvas, svg, picture, [style*=\"background-image\"] {\n filter: invert(1) hue-rotate(180deg) !important;\n }\n \n /* Preserve specific elements that should not be inverted */\n .no-dark-mode, .no-dark-mode *,\n [data-theme=\"light\"], [data-theme=\"light\"],\n .ace_editor, .ace_editor *,\n .CodeMirror, .CodeMirror *,\n .monaco-editor, .monaco-editor *,\n .markdown-body pre, .markdown-body pre *,\n .highlight, .highlight *,\n pre code, pre code * {\n filter: none !important;\n }\n \n /* Fix common UI elements */\n .modal, .popup, .dropdown-menu, .tooltip, .popover {\n filter: invert(1) hue-rotate(180deg) !important;\n background: #2d2d44 !important;\n border-color: #444 !important;\n }\n \n /* Scrollbars */\n ::-webkit-scrollbar { background: #1a1a2e !important; }\n ::-webkit-scrollbar-thumb { background: #444 !important; }\n ::-webkit-scrollbar-thumb:hover { background: #555 !important; }\n \n /* Selection */\n ::selection { background: #4ecdc4 !important; color: #1a1a2e !important; }\n ::-moz-selection { background: #4ecdc4 !important; color: #1a1a2e !important; }\n ';\n }\n \n function removeDarkMode() {\n var style = document.getElementById('universal-dark-mode-style');\n if (style) style.remove();\n }\n \n // Toggle with Alt+Shift+D\n document.addEventListener('keydown', function(e) {\n if (e.altKey && e.shiftKey && e.key === 'D') {\n e.preventDefault();\n enabled = !enabled;\n if (enabled) {\n applyDarkMode();\n console.log('[Universal Dark Mode] Enabled');\n } else {\n removeDarkMode();\n console.log('[Universal Dark Mode] Disabled');\n }\n }\n });\n \n // Apply on load\n applyDarkMode();\n \n // Re-apply on dynamic content\n var observer = new MutationObserver(function(mutations) {\n if (enabled && !document.getElementById('universal-dark-mode-style')) {\n applyDarkMode();\n }\n });\n observer.observe(document.head, { childList: true });\n \n console.log('[Universal Dark Mode] Loaded - Press Alt+Shift+D to toggle');\n})();", "Universal Dark Mode"); } } catch(__e) { console.warn('[Userscript:Universal Dark Mode]', __e); } })(); })();
Skip to content

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

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

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

Conversation

@jiashengguo

@jiashengguojiashengguo commented Mar 31, 2026

Copy link
Copy Markdown
Member

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

CopilotAI review requested due to automatic review settings March 31, 2026 04:13
@coderabbitai

coderabbitaiBot commented Mar 31, 2026

Copy link
Copy Markdown
Contributor

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 nameStatusExplanation
Description Check✅ PassedCheck skipped - CodeRabbit’s high-level summary is enabled.
Title check✅ PassedThe 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✅ PassedDocstring 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.

CopilotAI left a comment

Copy link
Copy Markdown

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.

FileDescription
src/server.tsMakes SQLite adapter loading resilient to export name changes by resolving multiple possible exports.
package.jsonUpdates dependency ranges for Prisma adapters and bumps package version.
pnpm-lock.yamlUpdates 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.

Comment threadsrc/server.ts
Comment threadsrc/server.ts
Comment threadpackage.json
Comment threadpnpm-lock.yaml

@coderabbitaicoderabbitaiBot left a comment

Copy link
Copy Markdown
Contributor

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

Comment threadpackage.json
@jiashengguo
jiashengguo merged commit a531a7f into devMar 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 freeto 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

@jiashengguo