Skip to content

Conversation

@ArthurGamby
Copy link
Contributor

@ArthurGamby ArthurGamby commented Feb 12, 2026

Summary by CodeRabbit

  • Documentation
    • Added a comprehensive guide for integrating Prisma with Deno, covering project setup, configuration, schema management, migrations, seeding, and server development.

@vercel
Copy link

vercel bot commented Feb 12, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
docs Error Error Feb 12, 2026 5:27pm

Request Review

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 12, 2026

Walkthrough

A new Deno integration guide is introduced to the documentation, providing step-by-step instructions for using Prisma ORM with Deno, covering initialization, configuration, schema setup, migrations, seeding, and server creation. The guide is registered in the navigation sidebar.

Changes

Cohort / File(s) Summary
Documentation: Deno Guide
content/800-guides/375-deno.mdx
New comprehensive guide for integrating Prisma with Deno, including project setup, Deno configuration, schema updates for Deno runtime, environment handling, migrations, seed setup, and HTTP server example.
Navigation Configuration
sidebars.ts
Added "guides/deno" entry to the Web Frameworks/Development Tools section under Guides.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 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 describes the main change: creating a new Deno guide for Prisma, which aligns with the addition of a comprehensive 372-line Deno documentation file and sidebar update.
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.

Tip

Issue Planner is now in beta. Read the docs and try it out! Share your feedback on Discord.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

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/800-guides/375-deno.mdx`:
- Around line 93-97: The info block ends with the quoted phrase "My Deno
Project" but lacks terminal punctuation; update the info block by adding a
period at the end of the sentence (i.e., change the ending to "My Deno
Project".) Locate the info block containing the quoted string and append the
missing period so the sentence is properly terminated.
🧹 Nitpick comments (4)
content/800-guides/375-deno.mdx (4)

78-81: Redundant deno install invocation.

The first deno install (line 79) is superseded by deno install --allow-scripts (line 80), which does the same thing plus allows lifecycle scripts (needed for Prisma's postinstall). Running both is unnecessary — the second command alone is sufficient.

Suggested fix
-```terminal
-deno install
-deno install --allow-scripts
-```
+```terminal
+deno install --allow-scripts
+```

299-327: Consider combining findMany and count into a single query or noting the extra round-trip.

The handler makes two sequential database calls (findMany + count) when users.length would give the same result for an unpaginated query. For a tutorial this is fine functionally, but it may teach a pattern that's wasteful in practice. A brief inline comment noting this would help readers understand the trade-off.


208-219: Duplicated PrismaClient + adapter setup in seed.ts vs db.ts.

The seed script (lines 226–235) recreates the exact same PrismaClient + PrismaPg adapter wiring that db.ts (lines 208–219) already exports. While seed scripts sometimes need standalone setup, here both files live in the same project and share the same env var. Importing from db.ts would reduce duplication and keep the connection config in one place.

That said, if the intent is for the seed script to be fully self-contained (common in many Prisma guides), this is a reasonable trade-off — just worth being intentional about.

Also applies to: 225-235


129-148: Clarify why a direct connection is needed instead of the default Prisma Postgres URL.

Section 2.3 instructs users to replace the DATABASE_URL with a direct connection string but doesn't explain why this is necessary for Deno specifically (vs. the default connection URL that prisma init --db provides). A one-liner explaining that the Deno runtime with @prisma/adapter-pg requires a direct PostgreSQL TCP connection string would help readers understand the reasoning rather than just following steps.

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