Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Caution Review failedThe pull request is closed. WalkthroughAdds a "Compare" section to the landing footer, updates the marketing submodule pointer, enables GitHub Flavored Markdown via remark-gfm in Next.js MDX config, and bumps version.txt from v2.9.47 to v2.9.48. Changes
Sequence Diagram(s)sequenceDiagram
autonumber
participant User
participant NextJS as Next.js App
participant MDX as MDX Compiler
participant Remark as remark-gfm
User->>NextJS: Request MDX-backed page
NextJS->>MDX: Compile MDX content
MDX->>Remark: Apply GFM transformations (tables, task lists, autolinks)
Remark-->>MDX: Transformed AST
MDX-->>NextJS: Compiled React components
NextJS-->>User: Rendered page
Note over Remark,MDX: New: remark-gfm in the pipeline
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes Possibly related PRs
Poem
✨ Finishing touches
🧪 Generate unit tests
📜 Recent review detailsConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro 📒 Files selected for processing (4)
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 |
There was a problem hiding this comment.
2 issues found across 4 files
Prompt for AI agents (all 2 issues)
Understand the root cause of the following 2 issues and fix them.
<file name="apps/web/app/(landing)/home/Footer.tsx">
<violation number="1" location="apps/web/app/(landing)/home/Footer.tsx:27">
Broken internal link: "/best-fyxer-alternative" appears to have no page or redirect, likely leading to a 404. Create the route or update the href to a valid path.</violation>
</file>
<file name="apps/web/next.config.ts">
<violation number="1" location="apps/web/next.config.ts:10">
remarkPlugins should receive the plugin function, not require.resolve(...) which returns a path string; import and pass the remark-gfm plugin function instead.</violation>
</file>
React with 👍 or 👎 to teach cubic. Mention @cubic-dev-ai to give feedback, ask questions, or re-run the review.
| { name: "Customer Support", href: "/support" }, | ||
| { name: "E-commerce", href: "/ecommerce" }, | ||
| ], | ||
| compare: [{ name: "vs Fyxer.ai", href: "/best-fyxer-alternative" }], |
There was a problem hiding this comment.
Broken internal link: "/best-fyxer-alternative" appears to have no page or redirect, likely leading to a 404. Create the route or update the href to a valid path.
Prompt for AI agents
Address the following comment on apps/web/app/(landing)/home/Footer.tsx at line 27:
<comment>Broken internal link: "/best-fyxer-alternative" appears to have no page or redirect, likely leading to a 404. Create the route or update the href to a valid path.</comment>
<file context>
@@ -24,6 +24,7 @@ const navigation = {
{ name: "Customer Support", href: "/support" },
{ name: "E-commerce", href: "/ecommerce" },
],
+ compare: [{ name: "vs Fyxer.ai", href: "/best-fyxer-alternative" }],
support: [
{ name: "Pricing", href: "/#pricing" },
</file context>
| const withMDX = nextMdx(); | ||
| const withMDX = nextMdx({ | ||
| options: { | ||
| remarkPlugins: [[require.resolve("remark-gfm")]], |
There was a problem hiding this comment.
remarkPlugins should receive the plugin function, not require.resolve(...) which returns a path string; import and pass the remark-gfm plugin function instead.
Prompt for AI agents
Address the following comment on apps/web/next.config.ts at line 10:
<comment>remarkPlugins should receive the plugin function, not require.resolve(...) which returns a path string; import and pass the remark-gfm plugin function instead.</comment>
<file context>
@@ -5,7 +5,11 @@ import withSerwistInit from "@serwist/next";
-const withMDX = nextMdx();
+const withMDX = nextMdx({
+ options: {
+ remarkPlugins: [[require.resolve("remark-gfm")]],
+ },
+});
</file context>
Summary by CodeRabbit
New Features
Chores