Skip to content

Conversation

@aidankmcalister
Copy link
Member

@aidankmcalister aidankmcalister commented Jan 8, 2026

Summary by CodeRabbit

  • Documentation
    • Added a warning about a known SQLite 3.51.0+ nested EXISTS bug affecting certain driver/adapters; recommends upgrading to adapter v7.3.0+ (which pins an unaffected SQLite), notes an alternative adapter is not impacted, and clarifies the issue is upstream with a fix pending.

✏️ Tip: You can customize this high-level summary in your review settings.

@github-actions
Copy link
Contributor

github-actions bot commented Jan 8, 2026

Dangerous URL check

No absolute URLs to prisma.io/docs found.
No local URLs found.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 8, 2026

Walkthrough

Added a warning to the SQLite docs noting a nested EXISTS query bug in SQLite 3.51.0+ affecting the better-sqlite3 adapter, with mitigation options (upgrade adapter to v7.3.0+, use libsql adapter, or await an upstream patched SQLite release). Surrounding content unchanged.

Changes

Cohort / File(s) Summary
SQLite documentation update
content/200-orm/050-overview/500-databases/500-sqlite.mdx
Inserted a warning block under "Using the better-sqlite3 driver" documenting a nested EXISTS bug in SQLite 3.51.0+ that affects the better-sqlite3 adapter; noted that upgrading the adapter to v7.3.0+ pins an unaffected SQLite, that the libsql adapter is not affected, and that upstream SQLite has a fix pending release (+13/-1).

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 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 clearly and specifically identifies the main change: adding a SQLite 3.51.1 bug warning to the documentation.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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


📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between dd35351 and 35c06a5.

📒 Files selected for processing (1)
  • content/200-orm/050-overview/500-databases/500-sqlite.mdx
🧰 Additional context used
🧠 Learnings (1)
📚 Learning: 2026-01-06T22:06:57.725Z
Learnt from: newclarityex
Repo: prisma/docs PR: 7425
File: content/200-orm/050-overview/500-databases/400-mysql.mdx:80-80
Timestamp: 2026-01-06T22:06:57.725Z
Learning: In Prisma docs, when showing examples of instantiating driver adapters with connection strings from environment variables, use the template literal pattern `const connectionString = `${process.env.DATABASE_URL}`` for consistency across all database adapter examples (PostgreSQL, MySQL/MariaDB, etc.).

Applied to files:

  • content/200-orm/050-overview/500-databases/500-sqlite.mdx
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)
  • GitHub Check: Check internal links
  • GitHub Check: Cloudflare Pages
🔇 Additional comments (1)
content/200-orm/050-overview/500-databases/500-sqlite.mdx (1)

43-55: SQLite forum link is accessible, but the v7.3.0 version claim requires verification.

The SQLite forum link resolves correctly (HTTP 200). The nested EXISTS bug in SQLite 3.51.0 is real and documented—it was fixed in SQLite 3.51.1 (released November 28, 2025).

However, the claim that @prisma/adapter-better-sqlite3 v7.3.0 "pins to an unaffected SQLite version" could not be verified from public Prisma documentation or release notes. Clarify:

  1. Whether v7.3.0 is the correct recommended version
  2. Whether it specifically pins to SQLite 3.51.1+ or an earlier unaffected version
  3. Alternatively, mention that updating SQLite to 3.51.1+ resolves the issue directly

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

@github-actions
Copy link
Contributor

github-actions bot commented Jan 8, 2026

Redirect check

This PR probably requires the following redirects to be added to static/_redirects:

  • This PR does not change any pages in a way that would require a redirect.

@github-actions
Copy link
Contributor

github-actions bot commented Jan 8, 2026

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Jan 8, 2026

Deploying docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: 35c06a5
Status: ✅  Deploy successful!
Preview URL: https://4dce469e.docs-51g.pages.dev
Branch Preview URL: https://dr-6767-sqlite-bug-warning.docs-51g.pages.dev

View logs

@github-actions
Copy link
Contributor

github-actions bot commented Jan 8, 2026

🍈 Lychee Link Check Report

Note: Links are cached for 5 minutes. Failed links (timeouts, rate limits) are retried in a second run with longer timeout.

📊 Results Overview

Status Count
🔍 Total 2299
✅ Successful 2269
⏳ Timeouts 0
🔀 Redirected 5
👻 Excluded 24
❓ Unknown 0
🚫 Errors 0
⛔ Unsupported 1

Copy link
Contributor

@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

🤖 Fix all issues with AI agents
In @content/200-orm/050-overview/500-databases/500-sqlite.mdx:
- Around line 45-59: Update the warning block to state that SQLite 3.51.1
(released Nov 28, 2025) fixes the nested EXISTS bug and that the issue is
resolved upstream; correct the affected better-sqlite3 range to "better-sqlite3
12.4.2 and later" (instead of "newer than 12.4.1") and explicitly note that
12.4.1 (which contains SQLite 3.50.4) is unaffected; adjust the Workarounds list
to recommend downgrading to 12.4.1, switching to libsql via
@prisma/adapter-libsql, or upgrading to a future better-sqlite3 release that
bundles SQLite 3.51.1+.
📜 Review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 500e472 and dd35351.

📒 Files selected for processing (1)
  • content/200-orm/050-overview/500-databases/500-sqlite.mdx
🧰 Additional context used
🧠 Learnings (1)
📚 Learning: 2026-01-06T22:06:57.725Z
Learnt from: newclarityex
Repo: prisma/docs PR: 7425
File: content/200-orm/050-overview/500-databases/400-mysql.mdx:80-80
Timestamp: 2026-01-06T22:06:57.725Z
Learning: In Prisma docs, when showing examples of instantiating driver adapters with connection strings from environment variables, use the template literal pattern `const connectionString = `${process.env.DATABASE_URL}`` for consistency across all database adapter examples (PostgreSQL, MySQL/MariaDB, etc.).

Applied to files:

  • content/200-orm/050-overview/500-databases/500-sqlite.mdx
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)
  • GitHub Check: Check links
  • GitHub Check: Check internal links

coderabbitai[bot]
coderabbitai bot previously approved these changes Jan 8, 2026
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