Skip to content

feat: enhance Tauri opener service for Linux file exploration#7827

Merged
Vanalite merged 3 commits intojanhq:mainfrom
Dexterity104:fix/linux-open-containing-folder-crash
Mar 27, 2026
Merged

feat: enhance Tauri opener service for Linux file exploration#7827
Vanalite merged 3 commits intojanhq:mainfrom
Dexterity104:fix/linux-open-containing-folder-crash

Conversation

@Dexterity104
Copy link
Copy Markdown
Contributor

Describe Your Changes

  • Fix a Linux-only crash when clicking Open Containing Folder (File Explorer) in Settings -> General -> Data Folder.
  • Route Linux folder reveal through the existing Tauri open_file_explorer command (xdg-open) instead of @tauri-apps/plugin-opener revealItemInDir, which can panic with zbus/Tokio runtime nesting.
  • Keep existing behavior unchanged for macOS and Windows.

Fixes Issues

Self Checklist

  • Added relevant comments, esp in complex areas
  • Updated docs (for bug fixes / features)
  • Created issues for follow-up changes or refactoring needed

Before / After Evidence

  • Before: Clicking "Open Containing Folder" on Linux crashes Jan with:
    • Cannot start a runtime from within a runtime
    • panic from zbus runtime path
  • After: Clicking the same action opens the target folder via system file manager without crashing.

Screenshots / Logs

image

@Dexterity104
Copy link
Copy Markdown
Contributor Author

@Vanalite
Could you please review this PR and share your feedback?

@Vanalite
Copy link
Copy Markdown
Contributor

I'm concerning this workaround is avoiding using the tauri-plugin-opener. If this is fixed upstream, better to have the universal fix than treating Linux differently.

@Dexterity104
Copy link
Copy Markdown
Contributor Author

Hi @Vanalite,
You are right. I updated the code.
Could you please review this PR again?

Comment thread web-app/src/services/opener/tauri.ts
@Vanalite
Copy link
Copy Markdown
Contributor

@Dexterity104 I'm ok with unify all platforms to use tauri command. However, there are other places still invoke revealItemInDir - open logs location for instance. Can you verify if it's also broken on Linux?

On the other hands, we need to test if this change break window or mac because instead of treating Linux exclusively, we are now change all platforms to use tauri command

@Dexterity104
Copy link
Copy Markdown
Contributor Author

Hi @Vanalite

Thanks for reply.
I tested on Mac and it worked correctly. I think it will work on Windows too.
photo_2026-03-27_06-55-12

@Vanalite
Copy link
Copy Markdown
Contributor

Screenshot 2026-03-27 at 12 05 51 PM It's opening a wrong path on Window @Dexterity104

@Vanalite
Copy link
Copy Markdown
Contributor

Sweet, all platform works. Thanks @Dexterity104. Approve and will merge later.

@Dexterity104 Dexterity104 force-pushed the fix/linux-open-containing-folder-crash branch from ccb60d1 to 6a9100b Compare March 27, 2026 14:45
@Vanalite Vanalite merged commit 3fa510c into janhq:main Mar 27, 2026
@github-project-automation github-project-automation Bot moved this to QA in Jan Mar 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: QA

Development

Successfully merging this pull request may close these issues.

bug: linux crash when clicking "Open Containing Folder" in Settings → General

3 participants