-
Notifications
You must be signed in to change notification settings - Fork 1.4k
fix(orm/typedsql): bump Prisma to 6.15.0 and refresh dev deps to resolve config type breakage #8293
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: latest
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -16,11 +16,12 @@ | |
"author": "", | ||
"license": "ISC", | ||
"dependencies": { | ||
"@prisma/client": "6.9.0" | ||
"@prisma/client": "6.15.0" | ||
}, | ||
"devDependencies": { | ||
"@types/node": "22.15.32", | ||
"prisma": "6.9.0", | ||
"tsx": "4.20.3" | ||
"@types/node": "^24.3.0", | ||
"dotenv": "^17.2.2", | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 💡 Verification agent 🧩 Analysis chaindotenv is added but not wired; ensure .env loading is actually invoked. Right now scripts won’t load If you prefer script-based loading (recommended for examples), update scripts (outside changed lines) like: {
"prisma": { "seed": "dotenv -e .env -- tsx prisma/seed.ts" },
"scripts": {
"start": "dotenv -e .env -- tsx src/index.ts",
"dev": "dotenv -e .env -- tsx src/index.ts --watch",
"test": "dotenv -e .env -- tsx src/index.ts",
"debug": "dotenv -e .env -- tsx src/index.ts"
}
} Or add at the top of both entrypoints: import 'dotenv/config' Verification script: 🏁 Script executed: #!/bin/bash
# Check if dotenv is imported or used via CLI
rg -nP 'dotenv/(config| -e )' -C2 orm/typedsql || true Length of output: 55 🏁 Script executed: #!/bin/bash
# Show the top of package.json to inspect scripts and dependencies
sed -n '1,200p' orm/typedsql/package.json
# Search for dotenv imports in TypeScript files
rg -nP "import\s+['\"]dotenv/config['\"]" -C2 orm/typedsql
# Search for dotenv requires in CommonJS-style files
rg -nP "require\(['\"]dotenv/config['\"]\)" -C2 orm/typedsql
# Broad search for any dotenv usage
rg -n 'dotenv' -C2 orm/typedsql Length of output: 1039 Wire dotenv initialization
🤖 Prompt for AI Agents
|
||
"prisma": "6.15.0", | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 💡 Verification agent 🧩 Analysis chainPrisma CLI 6.15.0 bump is consistent; validate config typing is resolved. Since this PR targets a TypedSQL config type break, confirm Run: If 🏁 Script executed: #!/bin/bash
# Locate prisma.config and seed file
fd -a 'prisma.config.*' orm/typedsql || true
fd -a 'seed.ts' orm/typedsql || true
# Type-check if a tsconfig exists for the example
fd -a 'tsconfig*.json' orm/typedsql || true Length of output: 152 Add Prisma TS config and tsconfig.json for TypeScript seed 🤖 Prompt for AI Agents
|
||
"tsx": "4.20.5" | ||
Comment on lines
+22
to
+25
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🧹 Nitpick (assertive) Dev deps bumps look fine; declare supported Node version to prevent type/runtime drift.
Proposed addition (outside changed lines): {
"engines": { "node": ">=18.18 <=26" }
} Adjust the range to your CI/runtime. 🤖 Prompt for AI Agents
|
||
} | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💡 Verification agent
🧩 Analysis chain
Prisma Client bump to 6.15.0 looks good; ensure regeneration happens.
This aligns the example with current Prisma. Make sure
@prisma/client
postinstall runs or that CI executesprisma generate
so TS types reflect 6.15.0.Run:
🏁 Script executed:
Length of output: 1499
Ensure Prisma Client is regenerated after the version bump
Add a
postinstall
script in orm/typedsql/package.json (or ensure your CI runsnpx prisma generate
) so that TypeScript types are updated to 6.15.0, for example:🤖 Prompt for AI Agents