Skip to content

Add rich file previews#153

Merged
ryota-murakami merged 4 commits into
mainfrom
codex/update-deps-info-file-preview
May 11, 2026
Merged

Add rich file previews#153
ryota-murakami merged 4 commits into
mainfrom
codex/update-deps-info-file-preview

Conversation

@ryota-murakami

Copy link
Copy Markdown
Contributor

Summary

  • add Shiki-backed source previews and Markdown Reading Mode for skill files
  • prevent Reading Mode from horizontally scrolling the whole pane
  • strip react-markdown AST props before DOM spread and harden Markdown/code preview tests
  • address local CodeRabbit CLI review feedback around skip links, preview language detection, and font fallbacks

Validation

  • coderabbit review --agent --base origin/main --type all
  • coderabbit review --agent --type uncommitted --base origin/main
  • pnpm exec vitest run src/renderer/src/components/skills/filePreviewLanguage.test.ts src/renderer/src/components/skills/FileContent.browser.test.tsx
  • pnpm validate
  • npx kill-port 9222
  • pnpm test:e2e

Note: the final extra CodeRabbit rerun hit the local CLI usage rate limit after all actionable findings from the last completed review were fixed.

@vercel

vercel Bot commented May 11, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
skills-desktop Ready Ready Preview, Comment May 11, 2026 3:13pm

Request Review

@coderabbitai

coderabbitai Bot commented May 11, 2026

Copy link
Copy Markdown
Contributor

Warning

Rate limit exceeded

@ryota-murakami has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 47 minutes and 59 seconds before requesting another review.

You’ve run out of usage credits. Purchase more in the billing tab.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: 35bafda4-6b7e-40ad-aaf8-d0452ad65e72

📥 Commits

Reviewing files that changed from the base of the PR and between 49beef4 and b5cf16b.

⛔ Files ignored due to path filters (1)
  • pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml, !pnpm-lock.yaml
📒 Files selected for processing (13)
  • package.json
  • src/renderer/src/components/SkipToMainContentLink.tsx
  • src/renderer/src/components/sidebar/BookmarkDetailModal.tsx
  • src/renderer/src/components/sidebar/SourceCard.tsx
  • src/renderer/src/components/skills/CodePreview.tsx
  • src/renderer/src/components/skills/FileContent.browser.test.tsx
  • src/renderer/src/components/skills/FileContent.tsx
  • src/renderer/src/components/skills/filePreviewLanguage.test.ts
  • src/renderer/src/components/skills/filePreviewLanguage.ts
  • src/renderer/src/components/skills/shikiPreview.ts
  • src/renderer/src/styles/globals.css
  • tailwind.config.ts
  • vitest.config.ts
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch codex/update-deps-info-file-preview

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.

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 58.99281% with 57 lines in your changes missing coverage. Please review.
✅ Project coverage is 52.43%. Comparing base (49beef4) to head (b5cf16b).

Files with missing lines Patch % Lines
src/renderer/src/components/skills/shikiPreview.ts 15.38% 33 Missing ⚠️
src/renderer/src/components/skills/FileContent.tsx 73.75% 18 Missing and 3 partials ⚠️
...derer/src/components/skills/filePreviewLanguage.ts 89.47% 0 Missing and 2 partials ⚠️
src/renderer/src/components/sidebar/SourceCard.tsx 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #153      +/-   ##
==========================================
+ Coverage   52.01%   52.43%   +0.42%     
==========================================
  Files         174      176       +2     
  Lines        4216     4350     +134     
  Branches      893      915      +22     
==========================================
+ Hits         2193     2281      +88     
- Misses       1831     1872      +41     
- Partials      192      197       +5     
Flag Coverage Δ
unittests 52.43% <58.99%> (+0.42%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@ryota-murakami ryota-murakami merged commit b5ea95b into main May 11, 2026
9 checks passed
@ryota-murakami ryota-murakami deleted the codex/update-deps-info-file-preview branch May 11, 2026 15:16
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.

2 participants