fix: install the project-owned local Postgres runtime - #102
Conversation
Signed-off-by: Aman Varshney <amanvarshney.work@gmail.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Essentials Run ID: 📒 Files selected for processing (4)
Included review availability: 1 review is currently available. Your included PR review attempts over the past 7 days set your current allowance at 3 reviews per hour. Summary by CodeRabbit
WalkthroughThe installer now includes Priority: ⬇️ Low Merge Risk: ⚪ Minimal · up to The installer places the Prisma development runtime in the intended project package without affecting unsupported project types. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
✨ Simplify code
Comment |
PR preview published
|
Fix
Declare
@prisma/dev@0.25.2in generated PostgreSQL projects' devDependencies so Composer can start local Postgres. Prisma 8 does not include that runtime itself.prisma: latest. No runtime download, global lookup, generated source patch, or CLI packaging change.Related PRs
prisma/composer#296 fixes Composer's CLI-relative lookup and its misleading instruction to install
prismawhen@prisma/devis missing. This scaffolding fix works with the currently published Composer release and does not need to wait for that release.prisma/prisma-cli#270 is not required. That alternative packages the runtime inside every CLI installation; this PR follows the project-owned optional-dependency direction recorded in the CLI design instead.
Verified
787a5d8, including the merged Hello World monorepo template.prisma@8.0.0-rc.15, app@prisma/composer-prisma-cloud@0.19.0, and the generated direct@prisma/dev@0.25.2dependency. No dependency or source edits after generation.dev:composercommands started local Postgres, applied the baseline migration, and served Alice/Bob/Carol over HTTP 200. Repeated requests returned the same three IDs. The monorepo/also returnedHello World!.