feat(blog): Bun Rust rewrite on Prisma Compute post + Prisma Compute series#7943
Conversation
…series Adds the "We put Bun's Rust rewrite in production on Prisma Compute" post by Alexey Orlenko with hero/meta images and author photo. Creates the prisma-compute series (featured, linked to the Compute docs), adds the public beta launch post as part 1, and unfeatures postgres-features so the two featured series are Prisma Compute and Prisma Next. Series pages can now render an optional docs link via docsUrl/docsLabel in the registry. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
WalkthroughThis PR adds optional ChangesBlog series expansion and Prisma Compute launch content
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. Comment |
|
The latest updates on your projects. Learn more about Argos notifications ↗︎
|
Featured series now keep their registry order instead of ranking by part count, so prisma-compute (first in the registry) takes the home highlight card ahead of prisma-next. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
There was a problem hiding this comment.
🧹 Nitpick comments (1)
apps/blog/src/app/(blog)/series/[key]/page.tsx (1)
80-87: ⚡ Quick winAdd security attributes for external documentation link.
The external documentation link should include
rel="noopener noreferrer"to prevent potential security risks associated with thewindow.openerreference. Consider also addingtarget="_blank"to open documentation in a new tab, improving the user experience.🔒 Proposed fix
{meta.docsUrl ? ( <a href={meta.docsUrl} + target="_blank" + rel="noopener noreferrer" className="mt-3 inline-block text-sm font-medium text-foreground-ppg hover:underline" > {meta.docsLabel ?? "Read the docs"} → </a> ) : null}🤖 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/blog/src/app/`(blog)/series/[key]/page.tsx around lines 80 - 87, The external docs anchor rendering conditional on meta.docsUrl should include security and UX attributes; update the anchor created where meta.docsUrl is used (in page.tsx rendering the doc link) to add target="_blank" and rel="noopener noreferrer" alongside the existing href and className so the link opens in a new tab and prevents window.opener exploits while keeping meta.docsLabel as the link text.
🤖 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.
Nitpick comments:
In `@apps/blog/src/app/`(blog)/series/[key]/page.tsx:
- Around line 80-87: The external docs anchor rendering conditional on
meta.docsUrl should include security and UX attributes; update the anchor
created where meta.docsUrl is used (in page.tsx rendering the doc link) to add
target="_blank" and rel="noopener noreferrer" alongside the existing href and
className so the link opens in a new tab and prevents window.opener exploits
while keeping meta.docsLabel as the link text.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: 33e9a7e1-349d-47ed-8c08-86059760ecc5
⛔ Files ignored due to path filters (3)
apps/blog/public/authors/alexey-orlenko.pngis excluded by!**/*.pngapps/blog/public/bun-rust-rewrite-prisma-compute/imgs/hero.svgis excluded by!**/*.svgapps/blog/public/bun-rust-rewrite-prisma-compute/imgs/meta.pngis excluded by!**/*.png
📒 Files selected for processing (4)
apps/blog/content/blog/bun-rust-rewrite-prisma-compute/index.mdxapps/blog/content/blog/launching-prisma-compute-public-beta/index.mdxapps/blog/src/app/(blog)/series/[key]/page.tsxapps/blog/src/lib/series-registry.ts
What
Adds the "We put Bun's Rust rewrite in production on Prisma Compute" blog post (draft 4 from prisma-orm-messaging) and creates a new Prisma Compute blog series.
Post
apps/blog/content/blog/bun-rust-rewrite-prisma-compute/index.mdx, authored by Alexey Orlenko (new author photo atpublic/authors/alexey-orlenko.png)public/bun-rust-rewrite-prisma-compute/imgs/platform— the frontmatter schema enum doesn't allowbun/rust/computeSeries
prisma-computeseries in the registry: the public beta launch post is part 1, this post is part 2postgres-featuresunfeatured)docsUrl/docsLabel; the series page header renders it as a docs link, connecting/series/prisma-computeto the Prisma Compute docsVerification
pnpm types:checkinapps/blogpasses (includes fumadocs-mdx frontmatter validation)🤖 Generated with Claude Code
Summary by CodeRabbit