fix(desktop): complete symlink support implementation and enable CI for PRs

[jerome-benoit]

Summary

• Completes the symlink support fix from PR fix(desktop): use current_binary() to support symlinked executables #7102 which was merged with compilation errors
• Fixes get_sidecar_path() to use tauri::process::current_binary(&app.env())
• Enables PR CI checks for nix-desktop workflow

Changes

• packages/desktop/src-tauri/src/cli.rs:

  • Add use tauri::Manager; import for .env() access
  • Change get_sidecar_path() to accept app: &tauri::AppHandle parameter
  • Use tauri::process::current_binary(&app.env()) instead of current_exe()
  • Update install_cli() to pass through AppHandle

• packages/desktop/src-tauri/src/lib.rs:

  • Pass app parameter to get_sidecar_path(app) call

• .github/workflows/nix-desktop.yml:

  • Add pull_request trigger to enable CI on PRs

Context

PR #7102 was merged but had a compilation error because tauri::process::current_binary() requires a parameter. This PR fixes that issue by properly passing the app environment through the call chain.

Testing

• CI builds will validate the fix compiles on Ubuntu and macOS

Completes #7102

[github-actions]

The following comment was made by an LLM, it may be inaccurate:

No duplicate PRs found

[Copilot]

Pull request overview

This PR completes the symlink support implementation from PR #7102, which was merged with compilation errors. It updates the get_sidecar_path() function to use tauri::process::current_binary() with proper parameter passing through the call chain, and enables PR CI checks for the nix-desktop workflow.

Key Changes

• Modified get_sidecar_path() to accept AppHandle parameter and use tauri::process::current_binary(app.env()) for symlink resolution
• Updated install_cli() to accept and pass through AppHandle to get_sidecar_path()
• Added pull_request trigger to nix-desktop workflow to enable CI on PRs

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File	Description
packages/desktop/src-tauri/src/cli.rs	Updated function signatures to accept and pass AppHandle for proper symlink resolution using tauri::process::current_binary()
packages/desktop/src-tauri/src/lib.rs	Updated call to get_sidecar_path() to pass the app parameter
.github/workflows/nix-desktop.yml	Added pull_request trigger with path filters to enable CI checks on PRs

---

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

[Marve10s]

Did you trigger the checks by yourself? Just curious

[github-actions]

Hey! Your PR title fix(desktop): complete symlink support implementation and enable CI for PRs doesn't follow conventional commit format.

Please update it to start with one of:

• feat: new feature
• fix: bug fix
• docs: documentation changes
• chore: maintenance tasks
• refactor: code refactoring
• test: adding or updating tests

See CONTRIBUTING.md for details.

[github-actions]

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.