Skip to content

fix(app): open in powershell#15112

Open
neriousy wants to merge 3 commits intoanomalyco:devfrom
neriousy:fix/open-in-powershell
Open

fix(app): open in powershell#15112
neriousy wants to merge 3 commits intoanomalyco:devfrom
neriousy:fix/open-in-powershell

Conversation

@neriousy
Copy link
Contributor

@neriousy neriousy commented Feb 25, 2026

Issue for this PR

Closes #15111

Type of change

  • Bug fix
  • New feature
  • Refactor / code improvement
  • Documentation

What does this PR do?

fixes the path when trying open current project in powershell

If you paste a large clearly AI generated description here your PR may be IGNORED or CLOSED!

How did you verify your code works?

run desktop app on windows and try opening project in powershell

Screenshots / recordings

2026-02-25.20-39-01.mp4

If this is a UI change, please include a screenshot or recording.

Checklist

  • I have tested my changes locally
  • I have not included unrelated changes in this PR

If you do not follow this template your PR will be automatically rejected.

Copilot AI review requested due to automatic review settings February 25, 2026 19:40
@github-actions
Copy link
Contributor

Thanks for your contribution!

This PR doesn't have a linked issue. All PRs must reference an existing issue.

Please:

  1. Open an issue describing the bug/feature (if one doesn't exist)
  2. Add Fixes #<number> or Closes #<number> to this PR description

See CONTRIBUTING.md for details.

@github-actions github-actions bot added the needs:compliance This means the issue will auto-close after 2 hours. label Feb 25, 2026
@github-actions github-actions bot removed needs:issue needs:compliance This means the issue will auto-close after 2 hours. labels Feb 25, 2026
@github-actions
Copy link
Contributor

Thanks for updating your PR! It now meets our contributing guidelines. 👍

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR fixes an issue with opening projects in PowerShell on Windows. The problem was that the previous implementation used the generic openerOpenPath for PowerShell, which didn't properly handle the path when opening the terminal. The fix introduces a dedicated PowerShell handler that correctly sets the working directory before launching the shell.

Changes:

  • Moved resolvedApp calculation after resolvedPath in the openPath function to ensure paths are resolved before app detection
  • Added PowerShell detection logic to intercept PowerShell launches and use a custom handler
  • Created a new Tauri command open_in_powershell that spawns PowerShell with the correct working directory

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.

File Description
packages/desktop/src/index.tsx Moved resolvedApp after resolvedPath, added isPowershell detection function, and routes PowerShell opens to the new command
packages/desktop/src/bindings.ts Added TypeScript binding for the new openInPowershell Tauri command
packages/desktop/src-tauri/src/os/windows.rs Implemented open_in_powershell function that spawns PowerShell in the correct directory; also reformatted imports alphabetically
packages/desktop/src-tauri/src/lib.rs Added open_in_powershell Tauri command with platform-specific implementation and registered it in the specta builder

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

bug(desktop): open in powershell does not work

2 participants