Skip to content

Conversation

@atharvadeosthale
Copy link
Member

No description provided.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 26, 2026

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

  • 🔍 Trigger a full review

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Comment on lines +24 to +27
const frontmatterMatch = content.match(/^---\n([\s\S]*?)\n---/);
if (!frontmatterMatch) {
return;
}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

do we not have a frontmatter parser somewhere already? 👀

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would like to avoid adding a preprocessor. Only if it would affect 90% of the markdown files it would be suitable.

cant we just pass the routeId as frontmatter to

// Accept explicit routePath (like "/docs/quick-starts/react"). If not provided, use current page.
? that way we can use the existing file without having to copy it.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I tried having an implementation where it just uses a markdown (prompt.md) file at a route. The problem with that is that we lose all styling because it uses some other markdown processor (I think it-markdown). I checked how partials are processed, and it's through another pre-processor. Since I couldn't go around any other way, I decided to do this. Lmk ur thoughts @TorstenDittmann

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants