docs(compute): update Alchemy provider guide - #8121
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
WalkthroughAdded a complete Alchemy guide for Prisma Compute and Prisma Postgres. Updated documentation navigation and added a permanent redirect from the previous Postgres Alchemy route. ChangesAlchemy documentation
Estimated code review effort: 3 (Moderate) | ~20 minutes Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
🍈 Lychee Link Check Report18 links: ✅ All links are working!Full Statistics Table
|
|
The latest updates on your projects. Learn more about Argos notifications ↗︎
|
There was a problem hiding this comment.
Actionable comments posted: 3
🧹 Nitpick comments (1)
apps/docs/content/docs/compute/alchemy.mdx (1)
19-22: 📐 Maintainability & Code Quality | 🔵 Trivial | ⚡ Quick winUse the docs package-manager command tabs.
These terminal sequences use Bun-only
bashfences. Usenpmfenced blocks for package-manager-independent commands so the docs tooling renders Bun, npm, pnpm, and Yarn variants.
apps/docs/content/docs/compute/alchemy.mdx#L19-L22: Use annpmfence for installation.apps/docs/content/docs/compute/alchemy.mdx#L28-L30: Use annpmfence for authentication.apps/docs/content/docs/compute/alchemy.mdx#L88-L91: Use annpmfence for deployment.apps/docs/content/docs/compute/alchemy.mdx#L97-L100: Use annpmfence for subsequent deployments.apps/docs/content/docs/compute/alchemy.mdx#L169-L171: Use annpmfence for local development.apps/docs/content/docs/compute/alchemy.mdx#L208-L211: Use annpmfence for CI deployment and cleanup.Based on learnings: The docs tooling converts an
npmfenced block into package-manager tabs automatically.🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@apps/docs/content/docs/compute/alchemy.mdx` around lines 19 - 22, Replace the Bun-only bash fences with npm fences for the package-manager commands in apps/docs/content/docs/compute/alchemy.mdx at lines 19-22 (installation), 28-30 (authentication), 88-91 (deployment), 97-100 (subsequent deployments), 169-171 (local development), and 208-211 (CI deployment and cleanup), preserving each command sequence so the docs tooling generates Bun, npm, pnpm, and Yarn tabs.Source: Learnings
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@apps/docs/content/docs/compute/alchemy.mdx`:
- Around line 208-211: Update the CI command example in the Alchemy
documentation to deploy and destroy an ephemeral preview stage rather than prod.
Use the same non-production stage consistently for both commands, and explicitly
state that cleanup must never target the prod stage.
- Line 90: Remove the hard-coded https:// prefix from both curl commands in
apps/docs/content/docs/compute/alchemy.mdx at lines 90 and 99, using the full
URL from the stack output directly because app.url already includes the scheme.
- Around line 82-84: Update the migration guidance near the Compute database
provisioning note to explicitly state that Prisma CLI migrations should map
connection.directConnectionString to datasource.url in prisma.config.ts, while
connection.databaseUrl remains the application runtime URL; note that Prisma ORM
7 reads the migration URL from prisma.config.ts.
---
Nitpick comments:
In `@apps/docs/content/docs/compute/alchemy.mdx`:
- Around line 19-22: Replace the Bun-only bash fences with npm fences for the
package-manager commands in apps/docs/content/docs/compute/alchemy.mdx at lines
19-22 (installation), 28-30 (authentication), 88-91 (deployment), 97-100
(subsequent deployments), 169-171 (local development), and 208-211 (CI
deployment and cleanup), preserving each command sequence so the docs tooling
generates Bun, npm, pnpm, and Yarn tabs.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Repository UI
Review profile: CHILL
Plan: Pro
Run ID: 6c484d96-fd70-458d-bdb0-319e5b3d759d
📒 Files selected for processing (5)
apps/docs/content/docs/compute/alchemy.mdxapps/docs/content/docs/compute/meta.jsonapps/docs/content/docs/postgres/iac/alchemy.mdxapps/docs/content/docs/postgres/iac/meta.jsonapps/docs/vercel.json
💤 Files with no reviewable changes (1)
- apps/docs/content/docs/postgres/iac/alchemy.mdx
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@apps/docs/content/docs/postgres/iac/meta.json`:
- Line 3: Restore Alchemy discoverability in the Postgres IaC navigation by
updating the pages array in the metadata object to include a
documentation-renderer-supported cross-link pointing to /docs/compute/alchemy,
while preserving the existing Terraform and Pulumi entries.
🪄 Autofix
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Repository UI
Review profile: CHILL
Plan: Pro
Run ID: fcbd34e8-296d-417e-9520-bc8a4336b31b
📒 Files selected for processing (1)
apps/docs/content/docs/postgres/iac/meta.json
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@apps/docs/content/docs/compute/alchemy.mdx`:
- Line 21: Update the command adding `.alchemy/` to `.gitignore` to ensure a
newline separates it from any existing final entry, including when the file
lacks a trailing newline, while preserving the intended ignore pattern.
🪄 Autofix
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Repository UI
Review profile: CHILL
Plan: Pro
Run ID: 543af027-ddcc-4c7e-984d-2c456ef4e4d3
📒 Files selected for processing (1)
apps/docs/content/docs/compute/alchemy.mdx
Summary
Validation
pnpm --filter docs types:checkpnpm --filter docs lint:linkspnpm --filter docs audit:redirectsSummary by CodeRabbit