fix: Refactor PrismaNeon initialization to use PoolConfig instead of Pool directly #644
+2
−3
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
This PR fixes a runtime error caused by passing a
Pool
instance into thePrismaNeon
adapter instead of aPoolConfig
object. The adapter expects a configuration object (with aconnectionString
), but the code constructed and passed aPool
, which lacks the expectedconnectionString
property. Prisma's client then fell back to default host/user values and produced the runtime error:“No database host or connection string was set, and key parameters have default values (host: localhost, user: ryan, db: ryan, password: null)...”
The change updates the PrismaNeon initialization to pass the expected
PoolConfig
object with theconnectionString
taken fromDATABASE_URL
.Related Issues
Closes #643
Checklist
Notes:
pnpm migrate
works and confirmed the root cause and fix in issue comments.Screenshots (if applicable)
N/A
Additional Notes
Root cause and reproduction details are available in the original issue: #643