Skip to content

feat: small enhancements#2234

Merged
novusnota merged 1 commit into
mainfrom
novusnota/ideas
Jun 24, 2026
Merged

feat: small enhancements#2234
novusnota merged 1 commit into
mainfrom
novusnota/ideas

Conversation

@novusnota

@novusnota novusnota commented Jun 24, 2026

Copy link
Copy Markdown
Collaborator
  • Up-right link for Acton URL on the landing page
  • Different light theme for code blocks
  • Process and remove {/* ... */} MDX comments from sources

Closes #2235.

Summary by CodeRabbit

  • New Features
    • Updated the code block theme for light mode and added cleaner handling of MDX comments, which are now removed from rendered content.
    • Improved home page quick links with clearer external-link iconography for outbound items.
  • Style
    • Refined link visuals on the home page to better distinguish internal and external destinations.

* Up-right link for Acton URL on the landing page
* Different light theme for code blocks
* Process and remove `{/* ... */}` MDX comments from sources
@vercel

vercel Bot commented Jun 24, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
docs Error Error Jun 24, 2026 4:44pm

Request Review

@coderabbitai

coderabbitai Bot commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

📝 Walkthrough

Walkthrough

Three small updates: source.config.ts switches the MDX code-block light theme to one-light and adds a remarkRemoveMdxComments plugin that strips block-comment MDX expression nodes from the AST. The home page QuickLink type gains an optional external flag that controls whether QuickLinkRow renders ArrowUpRight or ArrowRight. Non-functional comments are added to withBaseUrl in shared.ts.

Changes

MDX build pipeline: theme and comment-stripping plugin

Layer / File(s) Summary
Code theme and remarkRemoveMdxComments plugin
source.config.ts
Light theme changed from github-light-default to one-light. New remarkRemoveMdxComments plugin recursively removes mdxFlowExpression/mdxTextExpression nodes whose value is solely a /* ... */ block comment.

Home page external-link icon

Layer / File(s) Summary
QuickLink external icon variant
src/app/(home)/page.tsx, src/lib/shared.ts
ArrowUpRight added to imports; QuickLink type extended with external?: boolean; Acton toolchain entry marked external: true; QuickLinkRow selects ArrowUpRight vs ArrowRight based on that flag. Non-functional comments added inside withBaseUrl.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Poem

🐇 Hop hop, the light theme changed today,
one-light now brightens the coders' way.
Comments stripped clean from the MDX tree,
And external links get an arrow, you see—
↗️ Up and to the right, oh how fancy!
This bunny approves, most romancey. 🌟

🚥 Pre-merge checks | ✅ 3 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
Title check ❓ Inconclusive The title is generic and does not meaningfully describe the actual changes. Rename it to reflect the main updates, such as MDX comment removal, code theme changes, and the quick-link icon update.
✅ Passed checks (3 passed)
Check name Status Explanation
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch novusnota/ideas

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.

@novusnota novusnota merged commit 4b7d45e into main Jun 24, 2026
16 of 17 checks passed
@novusnota novusnota deleted the novusnota/ideas branch June 24, 2026 17:00
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.

Do not show MDX comments in LLM contents

1 participant