Skip to content

feat: implement global command palette and dynamic OG images - #41

Merged
Yash-pluto merged 6 commits into
mainfrom
feat/roadmaps-and-resources
Aug 10, 2026
Merged

Yash-pluto merged 6 commits into
mainfrom
feat/roadmaps-and-resources

Conversation

@Yash-pluto

@Yash-pluto Yash-pluto commented Aug 10, 2026 •

Copy link
Copy Markdown
Owner

Summary by CodeRabbit

  • Bug Fixes
    • Improved compatibility with search result data while preserving existing text highlighting behavior.

- Redesigned landing page with a new hero section and categorized topic grids
- Built mobile-responsive navigation including a slide-out drawer and inline TOC
- Upgraded MDX components with advanced code blocks and callouts/admonitions
- Integrated Fuse.js for fuzzy search with exact-match text highlighting
- Added breadcrumbs, a feedback widget, and 'Edit on GitHub' links
@Yash-pluto Yash-pluto self-assigned this Aug 10, 2026
@vercel

vercel Bot commented Aug 10, 2026 •

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
refme Ready Ready Preview Aug 10, 2026 7:03am

@coderabbitai

coderabbitai Bot commented Aug 10, 2026 •

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

The HighlightText props contract now accepts readonly matches arrays. Existing highlighting and matching behavior remains unchanged.

Changes

HighlightText type contract

Layer / File(s) Summary
Readonly matches typing
frontend/app/page.tsx
HighlightText now types matches as readonly any[], matching readonly Fuse.js result data without changing runtime behavior.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

  • Yash-pluto/refme#40: Both changes update frontend/app/page.tsx and relate to HighlightText search-result highlighting.

Poem

Readonly matches now flow,
Highlighting keeps its glow.
Types align with data’s flight,
Runtime logic stays just right.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Title check ⚠️ Warning The title describes a global command palette and dynamic OG images, but the changeset only updates the HighlightText matches type. Use a title that describes the HighlightText prop update, such as "fix: accept readonly Fuse.js matches in HighlightText".
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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.
✨ 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 feat/roadmaps-and-resources

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

@Yash-pluto Yash-pluto left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

fixed it

@Yash-pluto Yash-pluto added enhancement checked The provided code works. labels Aug 10, 2026
@Yash-pluto
Yash-pluto merged commit 7cbc588 into main Aug 10, 2026
3 of 4 checks passed

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🧹 Nitpick comments (1)
frontend/app/page.tsx (1)

141-142: 📐 Maintainability & Code Quality | 🔵 Trivial | ⚡ Quick win

Use FuseResultMatch for the matches prop.

The lockfile pins fuse.js@7.5.0, which exports FuseResultMatch. Replace readonly any[] with readonly FuseResultMatch[] and add a type-only import.

🤖 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 `@frontend/app/page.tsx` around lines 141 - 142, Update the matches prop type
in the relevant component to use readonly FuseResultMatch[] instead of readonly
any[], and add a type-only import for FuseResultMatch from fuse.js. Preserve the
existing optional prop behavior.
🤖 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 `@frontend/app/page.tsx`:
- Around line 141-142: Update the matches prop type in the relevant component to
use readonly FuseResultMatch[] instead of readonly any[], and add a type-only
import for FuseResultMatch from fuse.js. Preserve the existing optional prop
behavior.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 5b27eb76-e03b-4767-9640-0bef7c71baed

📥 Commits

Reviewing files that changed from the base of the PR and between b0855dd and 5e359bf.

📒 Files selected for processing (1)
  • frontend/app/page.tsx

@Yash-pluto
Yash-pluto deleted the feat/roadmaps-and-resources branch August 10, 2026 07:11

This branch was successfully deployed

1 active deployment
Preview — 5e359bfb Deployed Aug 10, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

checked The provided code works.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant