Skip to content

Conversation

@ankur-arch
Copy link
Contributor

@ankur-arch ankur-arch commented Nov 15, 2025

Summary by CodeRabbit

  • Documentation

    • Updated documentation for Prisma ORM v7 configuration changes, including datasource and environment variable setup in Prisma Config.
    • Removed middleware-related examples and guidance; directed users toward client extensions for custom query logic.
    • Marked datasource fields (url, directUrl, shadowDatabaseUrl) as deprecated in Prisma ORM v7.
  • Chores

    • Updated system requirements: Node.js 20.19.0+, TypeScript 5.4+.
    • Enhanced configuration tooling with expanded review options and automated workflow improvements.

aidankmcalister and others added 6 commits November 4, 2025 11:54
* `prisma-client-js` deprecated

* chore: empty commit

* broken link updated
* added env vars section to various pages

* minor coderabbit updates

* verbose removed

* test removed redirect list

* removed quickstart addition

* converted quickstarts back
* middleware deleted

* more middleware removed

* more middleware removed

* coderabbit changes
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Nov 15, 2025

Caution

Review failed

The pull request is closed.

Walkthrough

This PR deprecates Prisma middleware across the documentation, updates system requirements to Node.js 20.19.0+ and TypeScript 5.4+, marks datasource configuration fields as deprecated in favor of Prisma Config, introduces v7 configuration examples with dotenv environment variable loading, and removes middleware-related references from AI tool guides. Additionally, CodeRabbit and lychee workflow configurations are enhanced.

Changes

Cohort / File(s) Summary
Configuration & CI/CD
.coderabbit.yaml, .github/workflows/lychee.yml
Added CodeRabbit tone instructions and expanded review flags (high_level_summary, request_changes_workflow, poem, etc.); removed --verbose flags and trimmed lychee output at redirect marker
Middleware Documentation Removal
content/200-orm/200-prisma-client/300-client-extensions/500-middleware/index.mdx, content/200-orm/200-prisma-client/300-client-extensions/500-middleware/100-soft-delete-middleware.mdx, content/200-orm/200-prisma-client/300-client-extensions/500-middleware/200-logging-middleware.mdx, content/200-orm/200-prisma-client/300-client-extensions/500-middleware/300-session-data-middleware.mdx
Deleted entire middleware documentation pages including examples, FAQs, and implementation guidance
Middleware References Removal
content/200-orm/500-reference/050-prisma-client-reference.mdx, content/200-orm/200-prisma-client/300-client-extensions/index.mdx, content/200-orm/200-prisma-client/600-observability-and-logging/250-opentelemetry-tracing.mdx, content/200-orm/500-reference/500-preview-features/050-client-preview-features.mdx
Removed $use() method section, $on/$use limitations block, middleware tracing configuration, and middlewares preview feature entry
Generator & Schema Updates
content/200-orm/100-prisma-schema/10-overview/03-generators.mdx, content/200-orm/100-prisma-schema/10-overview/02-data-sources.mdx, content/200-orm/500-reference/100-prisma-schema-reference.mdx
Updated prisma-client as default generator (deprecated prisma-client-js in v7); added deprecation notes for url, directUrl, shadowDatabaseUrl datasource fields
Prisma Config Documentation
content/200-orm/500-reference/325-prisma-config-reference.mdx
Comprehensive v7 update: expanded PrismaConfig interface changes, added dotenv/config environment variable loading guidance, removed engine config from v7 examples, documented deprecated fields (adapter, studio, directUrl)
Environment Variables Setup
content/200-orm/200-prisma-client/000-setup-and-configuration/010-generating-prisma-client.mdx
Added "Loading environment variables" section with dotenv and prisma/config env helper implementation
System Requirements & Tooling
content/200-orm/500-reference/400-system-requirements.mdx, content/200-orm/800-more/350-ai-tools/100-cursor.mdx, content/200-orm/800-more/350-ai-tools/300-windsurf.mdx, content/200-orm/800-more/350-ai-tools/400-github-copilot.mdx
Updated Node.js minimum to 20.19.0+, TypeScript to 5.4+; removed middleware guidance from AI tool rules
Performance & Query Optimization
content/200-orm/200-prisma-client/100-queries/100-query-optimization-performance.mdx, content/250-postgres/400-query-optimization/100-setup.mdx, content/700-optimize/200-getting-started.mdx, content/300-accelerate/200-getting-started.mdx
Removed logging middleware references; refactored query resolver example to use fluent API with dataloader batching; removed middleware integration examples from Accelerate/Optimize guides
Query Extensions Documentation
content/200-orm/200-prisma-client/300-client-extensions/120-query.mdx
Clarified query extension behavior with type safety statement; removed "Query extensions versus middlewares" comparison section

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20–25 minutes

The changes are largely homogeneous (systematic middleware deprecation and deletions), but several areas warrant careful review:

  • content/200-orm/500-reference/325-prisma-config-reference.mdx — verify all v7 breaking changes are accurately documented, environment variable loading examples work correctly, and deprecated field references (adapter, studio, directUrl) are properly signposted
  • content/200-orm/200-prisma-client/100-queries/100-query-optimization-performance.mdx — confirm the fluent API example (findUnique(...).posts()) correctly illustrates dataloader batching behavior and improves upon the original findMany pattern
  • content/200-orm/100-prisma-schema/10-overview/03-generators.mdx — ensure deprecation messaging for prisma-client-js is unambiguous and migration guidance to prisma-client is clear

Possibly related PRs


📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between e25a5a7 and bfdbc16.

📒 Files selected for processing (24)
  • .coderabbit.yaml (2 hunks)
  • .github/workflows/lychee.yml (2 hunks)
  • content/200-orm/100-prisma-schema/10-overview/02-data-sources.mdx (1 hunks)
  • content/200-orm/100-prisma-schema/10-overview/03-generators.mdx (3 hunks)
  • content/200-orm/200-prisma-client/000-setup-and-configuration/010-generating-prisma-client.mdx (1 hunks)
  • content/200-orm/200-prisma-client/100-queries/100-query-optimization-performance.mdx (0 hunks)
  • content/200-orm/200-prisma-client/300-client-extensions/120-query.mdx (1 hunks)
  • content/200-orm/200-prisma-client/300-client-extensions/500-middleware/100-soft-delete-middleware.mdx (0 hunks)
  • content/200-orm/200-prisma-client/300-client-extensions/500-middleware/200-logging-middleware.mdx (0 hunks)
  • content/200-orm/200-prisma-client/300-client-extensions/500-middleware/300-session-data-middleware.mdx (0 hunks)
  • content/200-orm/200-prisma-client/300-client-extensions/500-middleware/index.mdx (0 hunks)
  • content/200-orm/200-prisma-client/300-client-extensions/index.mdx (0 hunks)
  • content/200-orm/200-prisma-client/600-observability-and-logging/250-opentelemetry-tracing.mdx (1 hunks)
  • content/200-orm/500-reference/050-prisma-client-reference.mdx (0 hunks)
  • content/200-orm/500-reference/100-prisma-schema-reference.mdx (2 hunks)
  • content/200-orm/500-reference/325-prisma-config-reference.mdx (10 hunks)
  • content/200-orm/500-reference/400-system-requirements.mdx (2 hunks)
  • content/200-orm/500-reference/500-preview-features/050-client-preview-features.mdx (1 hunks)
  • content/200-orm/800-more/350-ai-tools/100-cursor.mdx (0 hunks)
  • content/200-orm/800-more/350-ai-tools/300-windsurf.mdx (0 hunks)
  • content/200-orm/800-more/350-ai-tools/400-github-copilot.mdx (0 hunks)
  • content/250-postgres/400-query-optimization/100-setup.mdx (1 hunks)
  • content/300-accelerate/200-getting-started.mdx (1 hunks)
  • content/700-optimize/200-getting-started.mdx (1 hunks)

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

@ankur-arch ankur-arch closed this Nov 15, 2025
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.

3 participants