Skip to content

fix(deps): Make the @prisma/client dep explicit#2057

Merged
Tobbe merged 1 commit into
mainfrom
tobbe-fix-deps-explicit-prisma-client
Jul 8, 2026
Merged

fix(deps): Make the @prisma/client dep explicit#2057
Tobbe merged 1 commit into
mainfrom
tobbe-fix-deps-explicit-prisma-client

Conversation

@Tobbe

@Tobbe Tobbe commented Jul 7, 2026

Copy link
Copy Markdown
Member

pnpm, and yarn in stricter modes than with its node_modules linker, needs @prisma/client to be listed as a direct dependency because they don't hoist it from @cedarjs/api as npm does

@netlify

netlify Bot commented Jul 7, 2026

Copy link
Copy Markdown

Deploy Preview for cedarjs canceled.

Name Link
🔨 Latest commit 28348d7
🔍 Latest deploy log https://app.netlify.com/projects/cedarjs/deploys/6a4d8e7d6c94c1000872ee39

@github-actions github-actions Bot added this to the next-release-patch milestone Jul 7, 2026
@greptile-apps

greptile-apps Bot commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR makes the Prisma client dependency explicit in API projects. The main changes are:

  • Adds @prisma/client@7.8.0 to both API test fixtures.
  • Adds @prisma/client@7.8.0 to all create-cedar-app API templates.
  • Keeps the Prisma client version aligned with the existing Prisma adapter version.

Confidence Score: 5/5

This looks safe to merge.

  • No blocking issues found in the changed code.
  • The added package version matches the existing Prisma adapter version.
  • The project overview doc still describes Prisma and package-manager support accurately.

Important Files Changed

Filename Overview
fixtures/test-project-esm/api/package.json Adds @prisma/client@7.8.0 to the ESM API fixture dependencies.
fixtures/test-project/api/package.json Adds @prisma/client@7.8.0 to the CJS API fixture dependencies.
packages/create-cedar-app/templates/esm-js/api/package.json Adds @prisma/client@7.8.0 to the ESM JavaScript API template.
packages/create-cedar-app/templates/esm-ts/api/package.json Adds @prisma/client@7.8.0 to the ESM TypeScript API template.
packages/create-cedar-app/templates/js/api/package.json Adds @prisma/client@7.8.0 to the JavaScript API template.
packages/create-cedar-app/templates/ts/api/package.json Adds @prisma/client@7.8.0 to the TypeScript API template.

Reviews (1): Last reviewed commit: "fix(deps): Make the `@prisma/client` dep..." | Re-trigger Greptile

@nx-cloud

nx-cloud Bot commented Jul 7, 2026

Copy link
Copy Markdown

🤖 Nx Cloud AI Fix

Ensure the fix-ci command is configured to always run in your CI pipeline to get automatic fixes in future runs. For more information, please see https://nx.dev/ci/features/self-healing-ci


View your CI Pipeline Execution ↗ for commit 28348d7

Command Status Duration Result
nx run-many -t build ✅ Succeeded 3m 14s View ↗

💡 Verify your cache is correct by running tasks in a sandbox. Read docs ↗


☁️ Nx Cloud last updated this comment at 2026-07-08 06:07:57 UTC

@Tobbe Tobbe merged commit 922d445 into main Jul 8, 2026
79 of 81 checks passed
@Tobbe Tobbe deleted the tobbe-fix-deps-explicit-prisma-client branch July 8, 2026 06:12
@github-actions

github-actions Bot commented Jul 8, 2026

Copy link
Copy Markdown

The changes in this PR are now available on npm.

Try them out by running yarn cedar upgrade -t 5.0.0-canary.2601

Or try it in a new app with yarn dlx create-cedar-app@5.0.0-canary.2601

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.

1 participant