Skip to content

Conversation

@mhartington
Copy link
Member

@mhartington mhartington commented Nov 20, 2025

Summary by CodeRabbit

  • Documentation
    • Reworked the "Get started with Prisma Migrate from scratch" guide into tabbed Prisma 7 vs Prisma 6 views for side-by-side comparison.
    • Added version-specific configuration examples, interactive migration walkthroughs showing commands and SQL results, and explicit CLI prompts/notes.
    • Expanded narrative around initial migrations and adjusted the "Going further" troubleshooting link and wording.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Nov 20, 2025

Walkthrough

Reformats the "Get started with Prisma Migrate from scratch" doc into parallel Prisma 7 and Prisma 6 tabs, adds a Prisma 7 prisma.config.ts, updates versioned schema.prisma snippets, and converts migration examples into interactive CodeWithResult/cmd/cmdResult blocks; small wording edits in "Going further."

Changes

Cohort / File(s) Summary
Documentation restructuring
content/200-orm/300-prisma-migrate/050-getting-started.mdx
Replaced single schema example with a <TabbedContent> containing Prisma 7 and Prisma 6 <TabItem>s. Added Prisma 7 prisma.config.ts, version-specific schema.prisma examples, refactored migration steps into CodeWithResult, cmd, and cmdResult interactive blocks, renumbered migration steps, and adjusted "Going further" wording.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

  • Verify Prisma 7 vs Prisma 6 schema/provider syntax and DB URL differences match targeted versions.
  • Confirm prisma.config.ts is valid TypeScript and aligns with Prisma 7 API.
  • Check CodeWithResult / cmd / cmdResult blocks render and that CLI prompts/results match the shown commands.
  • Ensure migration SQL output corresponds to the provided schemas (tables, columns, foreign keys).

Pre-merge checks

❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Title check ❓ Inconclusive The title is vague and generic, using non-descriptive phrasing ('update') that doesn't convey the specific nature of the substantial changes made. Revise the title to be more specific about the primary change, such as: 'docs: add Prisma 6/7 dual-version migration guide with interactive examples' or 'docs: restructure migration getting started with version-specific tabs and interactive blocks'.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

📜 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 b71abeb and fb1406e.

📒 Files selected for processing (1)
  • content/200-orm/300-prisma-migrate/050-getting-started.mdx (5 hunks)
🧰 Additional context used
🧠 Learnings (3)
📓 Common learnings
Learnt from: aidankmcalister
Repo: prisma/docs PR: 7301
File: content/800-guides/410-cloudflare-workers.mdx:87-107
Timestamp: 2025-11-20T21:00:02.521Z
Learning: Do not suggest changes to `prisma.config.ts` files in the prisma/docs repository, as the code examples are intentional for documentation purposes.
Learnt from: ankur-arch
Repo: prisma/docs PR: 7066
File: content/200-orm/200-prisma-client/700-debugging-and-troubleshooting/245-troubleshooting-binary-size-issues.mdx:8-22
Timestamp: 2025-08-11T09:40:55.237Z
Learning: When the queryCompiler preview feature is enabled in Prisma ORM (v6.7.0+), it does not require Rust engines for CLI tools like `prisma migrate` or `prisma db pull`. The previous understanding that CLI tools would still need Rust binaries even with queryCompiler enabled is incorrect.
Learnt from: aidankmcalister
Repo: prisma/docs PR: 7165
File: content/800-guides/550-test-guide.mdx:85-90
Timestamp: 2025-10-08T16:23:00.388Z
Learning: For .mdx files in the prisma/docs repository: All headings and titles should use sentence case (e.g., "Getting started with Prisma ORM", "Best practices for authentication"), not title case. Exception: Always preserve exact casing for product names including "Prisma Postgres", "Prisma", "Prisma ORM", and "Prisma Data Platform".
📚 Learning: 2025-11-20T21:00:02.521Z
Learnt from: aidankmcalister
Repo: prisma/docs PR: 7301
File: content/800-guides/410-cloudflare-workers.mdx:87-107
Timestamp: 2025-11-20T21:00:02.521Z
Learning: Do not suggest changes to `prisma.config.ts` files in the prisma/docs repository, as the code examples are intentional for documentation purposes.

Applied to files:

  • content/200-orm/300-prisma-migrate/050-getting-started.mdx
📚 Learning: 2025-10-09T21:32:50.340Z
Learnt from: aidankmcalister
Repo: prisma/docs PR: 7167
File: content/900-ai/prompts/astro.mdx:84-84
Timestamp: 2025-10-09T21:32:50.340Z
Learning: The `npx prisma init` command supports the following flags: `--db` (shorthand for `--datasource-provider prisma+postgres`), `--output` (specifies output location for generated client), `--generator-provider` (defines the generator provider), `--datasource-provider`, `--url`, `--preview-feature`, and `--with-model`. These are documented valid CLI options for Prisma init command.

Applied to files:

  • content/200-orm/300-prisma-migrate/050-getting-started.mdx
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)
  • GitHub Check: Check internal links
  • GitHub Check: Cloudflare Pages

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

🧹 Nitpick comments (1)
content/200-orm/300-prisma-migrate/050-getting-started.mdx (1)

75-89: Clarify Prisma 7 configuration requirement and file location.

The text at Line 75 uses "be sure to have," which is ambiguous about whether prisma.config.ts is mandatory. For Prisma 7, is this a required configuration file for all projects, or only when using non-default paths? Additionally, the schema path in the config (Line 81, "prisma/schema.prisma") differs from the earlier schema example (file=schema.prisma). This could confuse readers about project structure expectations.

Consider strengthening the language to clarify: (1) whether prisma.config.ts is required for Prisma 7, and (2) whether the paths shown are defaults, recommendations, or project-specific examples.

📜 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 8526167 and 3abdbc4.

📒 Files selected for processing (1)
  • content/200-orm/300-prisma-migrate/050-getting-started.mdx (1 hunks)
🧰 Additional context used
🧠 Learnings (3)
📓 Common learnings
Learnt from: aidankmcalister
Repo: prisma/docs PR: 7165
File: content/800-guides/550-test-guide.mdx:85-90
Timestamp: 2025-10-08T16:23:00.388Z
Learning: For .mdx files in the prisma/docs repository: All headings and titles should use sentence case (e.g., "Getting started with Prisma ORM", "Best practices for authentication"), not title case. Exception: Always preserve exact casing for product names including "Prisma Postgres", "Prisma", "Prisma ORM", and "Prisma Data Platform".
Learnt from: ankur-arch
Repo: prisma/docs PR: 7066
File: content/200-orm/200-prisma-client/700-debugging-and-troubleshooting/245-troubleshooting-binary-size-issues.mdx:8-22
Timestamp: 2025-08-11T09:40:55.237Z
Learning: When the queryCompiler preview feature is enabled in Prisma ORM (v6.7.0+), it does not require Rust engines for CLI tools like `prisma migrate` or `prisma db pull`. The previous understanding that CLI tools would still need Rust binaries even with queryCompiler enabled is incorrect.
📚 Learning: 2025-10-08T16:23:00.388Z
Learnt from: aidankmcalister
Repo: prisma/docs PR: 7165
File: content/800-guides/550-test-guide.mdx:85-90
Timestamp: 2025-10-08T16:23:00.388Z
Learning: For .mdx files in the prisma/docs repository: All headings and titles should use sentence case (e.g., "Getting started with Prisma ORM", "Best practices for authentication"), not title case. Exception: Always preserve exact casing for product names including "Prisma Postgres", "Prisma", "Prisma ORM", and "Prisma Data Platform".

Applied to files:

  • content/200-orm/300-prisma-migrate/050-getting-started.mdx
📚 Learning: 2025-10-09T21:32:50.340Z
Learnt from: aidankmcalister
Repo: prisma/docs PR: 7167
File: content/900-ai/prompts/astro.mdx:84-84
Timestamp: 2025-10-09T21:32:50.340Z
Learning: The `npx prisma init` command supports the following flags: `--db` (shorthand for `--datasource-provider prisma+postgres`), `--output` (specifies output location for generated client), `--generator-provider` (defines the generator provider), `--datasource-provider`, `--url`, `--preview-feature`, and `--with-model`. These are documented valid CLI options for Prisma init command.

Applied to files:

  • content/200-orm/300-prisma-migrate/050-getting-started.mdx
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
  • GitHub Check: Cloudflare Pages

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Nov 20, 2025

Deploying docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: fb1406e
Status: ✅  Deploy successful!
Preview URL: https://53af1673.docs-51g.pages.dev
Branch Preview URL: https://getting-started-migration-fe.docs-51g.pages.dev

View logs

@mhartington mhartington force-pushed the getting-started-migration-feedback branch from 3abdbc4 to b71abeb Compare November 20, 2025 20:53
@github-actions
Copy link
Contributor

Dangerous URL check

No absolute URLs to prisma.io/docs found.
No local URLs found.

@github-actions
Copy link
Contributor

@github-actions
Copy link
Contributor

Redirect check

This PR probably requires the following redirects to be added to static/_redirects:

  • This PR does not change any pages in a way that would require a redirect.

@mhartington mhartington merged commit 2cf9b86 into main Nov 24, 2025
8 checks passed
@mhartington mhartington deleted the getting-started-migration-feedback branch November 24, 2025 20:58
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.

2 participants