Skip to content

Conversation

@mhartington
Copy link
Contributor

@mhartington mhartington commented Nov 19, 2025

Tanstack start recently updated their prisma integration, this updates the guide to use the latest changes

Summary by CodeRabbit

  • Documentation
    • TanStack Start guide completion time reduced from 10 to 5 minutes.
    • Updated Node.js minimum version requirement to 20+.
    • Streamlined project setup process with simplified scaffolding.
    • Refreshed guide examples and database model references for improved clarity.

@github-actions
Copy link
Contributor

Dangerous URL check

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

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Nov 19, 2025

Walkthrough

This PR comprehensively rewrites the TanStack Start guide, reducing setup time from 10 to 5 minutes by replacing manual scaffolding with npm create @tanstack/start@latest, updating Node.js requirements to 20+, refactoring Prisma client from local to shared/global pattern, and restructuring all data examples from User/Post to Todo model.

Changes

Cohort / File(s) Summary
TanStack Start Guide Rewrite
content/800-guides/160-tanstack-start.mdx
Major content overhaul: modernized setup flow with npm create command, Node.js 20+ requirement bump, Prisma schema redesign (User/Post → Todo model), client instantiation refactored to shared/global pattern, all data access/server functions/loaders/UI components updated to reflect Todo-centric approach, estimated completion time halved.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

  • Prisma schema consistency: verify the Todo model definition is correctly integrated throughout all code examples and that no orphaned User/Post references remain
  • Client pattern verification: ensure the shared/global Prisma client instantiation is properly applied across all server functions and loaders with correct import paths
  • Data access layer alignment: cross-check that server functions, loaders, and UI components all reference Todo data consistently with matching variable names and types
  • Prerequisites accuracy: confirm Node.js 20+ requirement is appropriate for the example and doesn't conflict with other stated dependencies
  • Code example validity: all provided snippets should be syntactically correct and semantically aligned with the revised architecture

Possibly related PRs

  • feat: add chatGPT page #7173: Modifies the same guide file with updates to server-side data fetching, loader/server-function wiring, and Prisma usage patterns
  • DC-5242 Astro Better-Auth Guide #7215: Shifts Prisma client setup to shared/global pattern with module path relocation, mirroring the architectural change in this PR
  • clerk updates #7073: Adjusts Prisma client placement and import paths for root-level/shared client instantiation, aligning with the client pattern restructuring here

Pre-merge checks

❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Title check ❓ Inconclusive The title 'docs(): update tanstack start guide' is vague and generic, using non-descriptive language that doesn't convey the specific nature of the updates. Consider a more specific title that highlights the key changes, such as 'docs(): update TanStack Start guide to use npm create and Todo schema' or 'docs(): modernize TanStack Start guide with latest Prisma integration flow'.
✅ 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.

Tip

📝 Customizable high-level summaries are now available in beta!

You can now customize how CodeRabbit generates the high-level summary in your pull requests — including its content, structure, tone, and formatting.

  • Provide your own instructions using the high_level_summary_instructions setting.
  • Format the summary however you like (bullet lists, tables, multi-section layouts, contributor stats, etc.).
  • Use high_level_summary_in_walkthrough to move the summary from the description to the walkthrough section.

Example instruction:

"Divide the high-level summary into five sections:

  1. 📝 Description — Summarize the main change in 50–60 words, explaining what was done.
  2. 📓 References — List relevant issues, discussions, documentation, or related PRs.
  3. 📦 Dependencies & Requirements — Mention any new/updated dependencies, environment variable changes, or configuration updates.
  4. 📊 Contributor Summary — Include a Markdown table showing contributions:
    | Contributor | Lines Added | Lines Removed | Files Changed |
  5. ✔️ Additional Notes — Add any extra reviewer context.
    Keep each section concise (under 200 words) and use bullet or numbered lists for clarity."

Note: This feature is currently in beta for Pro-tier users, and pricing will be announced later.


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.

@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.

@cloudflare-workers-and-pages
Copy link

Deploying docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: 8b219dc
Status: ✅  Deploy successful!
Preview URL: https://b788f792.docs-51g.pages.dev
Branch Preview URL: https://docs-tanstack-updates.docs-51g.pages.dev

View logs

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: 0

🧹 Nitpick comments (2)
content/800-guides/160-tanstack-start.mdx (2)

59-59: Align headings with sentence case per documentation guidelines.

Per the repository's style guide, headings should use sentence case (capitalizing only the first word and product names), not title case. Adjust:

  • Line 59: "Define your Prisma Schema" → "Define your Prisma schema"
  • Line 82: "Configure the Prisma Client generator" → Consider if "Client" should be lowercase here (compare to other headings)
  • Line 111: "The Prisma Client" → "The Prisma client" (unless treating "Prisma Client" as a full product name exception)

For consistency, review whether "Prisma Client" is treated as a unified product name (like "Prisma ORM") or if "client" should be lowercase in these contexts.

Based on learnings

Also applies to: 82-82, 111-111


35-35: Fix typo in database provider name.

"Prisma PostgresSQL" should be "Prisma PostgreSQL" (double-S is incorrect). This appears in the interactive prompt example.

📜 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 ce8bb47 and 8b219dc.

⛔ Files ignored due to path filters (1)
  • package-lock.json is excluded by !**/package-lock.json
📒 Files selected for processing (1)
  • content/800-guides/160-tanstack-start.mdx (6 hunks)
🧰 Additional context used
🧠 Learnings (2)
📚 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/800-guides/160-tanstack-start.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/800-guides/160-tanstack-start.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). (3)
  • GitHub Check: Check links
  • GitHub Check: Check internal links
  • GitHub Check: Cloudflare Pages
🔇 Additional comments (1)
content/800-guides/160-tanstack-start.mdx (1)

82-107: Verify Prisma command sequence and npm script naming.

The command sequence in this section is confusing:

  • Line 87 (section 2.2): npm run db:seed -- --name init is described as creating database tables, but db:seed typically adds data, not creates schema.
  • Section 2.3 title says "Seed the database" but contains both db:generate (line 94) and db:seed (line 100).
  • The section titles don't align clearly with what each command does.

This suggests either the commands, section titles, or both need correction. TanStack Start's npm create scaffolding might use non-standard npm script names, so I need to verify this sequence is accurate.

Can you confirm that:

  1. npm run db:seed -- --name init is the correct command to create/push the database schema (not npx prisma db push or npx prisma migrate dev --name init)?
  2. The section titles accurately describe what each command does?
  3. The ordering (seed-with-init → generate → seed) is the intended workflow for TanStack Start's Prisma setup?

If these are custom TanStack Start npm scripts with non-obvious names, consider adding inline comments in the code blocks to clarify (e.g., # Creates database schema before the db:seed line).

@github-actions
Copy link
Contributor

🍈 Lychee Link Check Report

Note: Links are cached for 5 minutes. Failed links (timeouts, rate limits) are retried in a second run with longer timeout.

📊 Results Overview

Status Count
🔍 Total 2233
✅ Successful 2194
⏳ Timeouts 0
🔀 Redirected 5
👻 Excluded 23
❓ Unknown 0
🚫 Errors 10
⛔ Unsupported 1

Errors per input

Errors in 200-orm/500-reference/250-error-reference.mdx

Errors in 800-guides/050-migrate-from-mongoose.mdx

Errors in 800-guides/370-bun.mdx

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