feat: Add file explorer open in browser actions#1843
Open
elashera wants to merge 1 commit into
Open
Conversation
Contributor
|
thx for the PR! will review and get this in today |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
With the growing use of HTML files for local text, diagrams, and generated artifacts, this adds a faster way to open local files in Orca's built-in browser.
file://URLs.Screenshots
Screen recording:
Screen.Recording.2026-05-14.at.13.07.36.mov
The recording shows:
Testing
pnpm lintpnpm exec oxlint src/renderer/src/components/browser-pane/BrowserPane.tsx src/renderer/src/components/right-sidebar/FileExplorerRow.tsx src/renderer/src/components/right-sidebar/FileExplorerBackgroundMenu.tsx src/renderer/src/lib/file-preview.ts src/renderer/src/lib/file- preview.test.tspnpm typecheckpnpm exec tsc --noEmit -p config/tsconfig.web.json --composite falsepnpm testenv -u OPENCODE_CONFIG_DIR -u ORCA_OPENCODE_CONFIG_DIR -u PI_CODING_AGENT_DIR -u ORCA_PI_CODING_AGENT_DIR pnpm testpnpm buildenv -u OPENCODE_CONFIG_DIR -u ORCA_OPENCODE_CONFIG_DIR -u PI_CODING_AGENT_DIR -u ORCA_PI_CODING_AGENT_DIR pnpm buildnot create
/usr/local/bin/orca-devdue permissions.file-preview.test.tsfor opening local file URLs in Orca browser tabs.pnpm exec vitest run --config config/vitest.config.ts src/renderer/src/lib/file-preview.test.tsAI Review Report
The AI coding agent reviewed the change for scope, interaction behavior, and cross-platform compatibility.
Checked risks:
file://URL conversion.Findings and results:
absolutePathToFileUrihelper instead of hardcoded separators.Cmd/Ctrlplatform differences.file://URLs resolve on the local machine, not on the remote host.Security Audit
The AI coding agent reviewed the security impact of the browser/file explorer changes.
Reviewed areas:
Summary:
file://navigation to avoid misleading or broken remote-file assumptions.Follow-up:
Notes
renderer helper.