Conversation
…lur status in canvas-cursor
…ount in splash-cursor
…frame in gradient-grid-hero
…rability Sanitize dynamic slug parameters and handle missing files in documentation layout
…-leaks Fix memory and CPU leaks in interactive components
|
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: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (10)
📝 WalkthroughWalkthroughThis PR adds slug whitelist validation and safe ENOENT-based 404 handling for docs MDX file loading, and updates several nurui canvas components to track requestAnimationFrame handles for proper cancellation on cleanup, plus refactors inline event listeners into named handler functions. ChangesDocs Slug Validation
Canvas Animation Lifecycle Fixes
Estimated code review effort: 3 (Moderate) | ~25 minutes Sequence Diagram(s)sequenceDiagram
participant Client
participant Page
participant FileSystem
Client->>Page: request /docs/[slug]
Page->>Page: validate slug against whitelist
alt slug invalid
Page->>Client: notFound()
else slug valid
Page->>FileSystem: readFile(content/docs/slug.mdx)
alt file missing (ENOENT)
FileSystem-->>Page: ENOENT error
Page->>Client: notFound()
else file exists
FileSystem-->>Page: MDX content
Page->>Client: render content
end
end
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Warning There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure. 🔧 ESLint
ESLint install failed. For unrecoverable errors, disable the tool in CodeRabbit configuration. 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. Comment |
Summary by CodeRabbit