Skip to content

feat(desktop): add reveal-in-Finder button to file pane header - #6071

Merged
saddlepaddle merged 1 commit into
ParadevAI:mainfrom
kkjcheng:add-open-finder-option
Jul 31, 2026
Merged

saddlepaddle merged 1 commit into
ParadevAI:mainfrom
kkjcheng:add-open-finder-option

Conversation

@kkjcheng

@kkjcheng kkjcheng commented Jul 31, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • Adds a folder-icon button to the v2 file pane header that reveals the current file in Finder via the existing electronTrpc.external.openInFinder mutation (shell.showItemInFolder).
  • Sits between Copy path and Open in editor, matching the sibling buttons' styling and tooltip pattern.

Test plan

  • Open a file in the v2 workspace, click the new folder icon in the pane header, and confirm Finder opens with the file highlighted.
  • Confirm tooltip reads "Reveal in Finder".

Open in Stage

Summary by cubic

Add a “Reveal in Finder” button to the v2 file pane header that opens the current file in Finder via electronTrpc.external.openInFinder.

The button sits between Copy path and Open in editor, matches existing styles, and shows the tooltip “Reveal in Finder.”

Written for commit 0879bd0. Summary will update on new commits.

Review in cubic

Summary by CodeRabbit

  • New Features
    • Added a “Reveal in Finder” action to the file pane header.
    • Users can open the current file’s location directly in Finder.

@ghost

ghost commented Jul 31, 2026

Copy link
Copy Markdown

Ready to review this PR? Stage has broken it down into 1 individual chapter for you:

Title
1 Add Reveal in Finder button to FilePaneHeader
Open in Stage

Chapters generated by Stage for commit 0879bd0 on Jul 31, 2026 6:41pm UTC.

@coderabbitai

coderabbitai Bot commented Jul 31, 2026 •

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 87a7b0c5-eef0-4c76-8e91-077d1deae4d6

📥 Commits

Reviewing files that changed from the base of the PR and between 4a6c985 and 0879bd0.

📒 Files selected for processing (1)
  • apps/desktop/src/renderer/routes/_authenticated/_dashboard/v2-workspace/$workspaceId/hooks/usePaneRegistry/components/FilePane/components/FilePaneHeaderExtras/FilePaneHeaderExtras.tsx

📝 Walkthrough

Walkthrough

The file pane header adds a tooltip button that invokes Electron’s external.openInFinder mutation for the current file path.

Changes

Finder reveal

Layer / File(s) Summary
Add Finder reveal action
apps/desktop/.../FilePaneHeaderExtras.tsx
The header imports the Finder icon and Electron client, invokes external.openInFinder with the current file path, and renders a “Reveal in Finder” button.

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

Possibly related PRs

Suggested reviewers: avipeltz

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description explains the change and includes a test plan, but it omits the required Checklist section and reports no completed testing. Add the required Checklist section and record the completed lint, typecheck, and UI verification steps.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly summarizes the main change and follows the repository's conventional commit format.
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 unit tests (beta)
  • Create PR with unit tests

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.

@saddlepaddle
saddlepaddle merged commit 3b3f79f into ParadevAI:main Jul 31, 2026
12 checks passed
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