fix: focus window on single-instance activation

[TimeToBuildBob]

Summary

• Add window.set_focus() after window.show() in the single-instance file watcher handler, matching the existing tray "Open Dashboard" handler pattern
• Without this, launching a second instance shows the existing window but leaves it behind other windows

Fixes #208

---

Important

Add window.set_focus() in listen_for_lockfile() to focus the main window on second instance launch.

• Behavior:
  • Add window.set_focus() after window.show() in listen_for_lockfile() in lib.rs to focus the main window when a second instance is launched.
  • Matches the existing behavior in the tray "Open Dashboard" handler.
• Fixes:
  • Fixes issue bug: single-instance handler shows window but doesn't focus it #208 where the main window would appear behind other windows when a second instance is launched.

^{This description was created by for 6afc5b1. You can customize this summary. It will automatically update as commits are pushed.}

[ellipsis-dev]

Important

Looks good to me! 👍

Reviewed everything up to 6afc5b1 in 16 seconds. Click for details.

• Reviewed 12 lines of code in 1 files
• Skipped 0 files when reviewing.
• Skipped posting 0 draft comments. View those below.
• Modify your settings and rules to customize what types of comments Ellipsis leaves. And don't forget to react with 👍 or 👎 to teach Ellipsis.

Workflow ID: wflow_aLfZrU0h25ZvgBQR

^{You can customize by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.}

[greptile-apps]

Greptile Summary

Adds window.set_focus() after window.show() in listen_for_lockfile() so that when a second instance is launched, the existing window is both shown and brought to the foreground. This matches the existing pattern used in the tray "Open Dashboard" menu handler.

• Adds window.set_focus().expect("Failed to focus main window") at line 215 in listen_for_lockfile()
• Fixes bug: single-instance handler shows window but doesn't focus it #208 where the main window would appear behind other windows on second instance activation
• No issues found; the change is minimal, correct, and consistent with the codebase

Confidence Score: 5/5

• This PR is safe to merge with no risk — it adds a single well-understood Tauri API call that matches an existing pattern in the same file.
• The change is a single line addition calling set_focus() after show(), directly mirroring the existing tray handler pattern at line 607-608. The Tauri set_focus() API is well-established. No logic changes, no new dependencies, no edge cases.
• No files require special attention.

Important Files Changed

Filename	Overview
src-tauri/src/lib.rs	Adds window.set_focus() after window.show() in listen_for_lockfile(), matching the existing tray "Open Dashboard" handler pattern. Minimal, correct change.

Sequence Diagram

sequenceDiagram
    participant User as User (2nd Instance)
    participant Plugin as single_instance plugin
    participant Lockfile as single_instance.lock
    participant Watcher as listen_for_lockfile()
    participant Window as Main Window

    User->>Plugin: Launch 2nd instance
    Plugin->>Lockfile: Create/truncate lock file
    Plugin->>Plugin: Log & quit 2nd instance
    Lockfile-->>Watcher: File change detected
    Watcher->>Lockfile: Remove lock file
    Watcher->>Window: window.show()
    Watcher->>Window: window.set_focus() ✨ NEW

Loading

_{Last reviewed commit: 6afc5b1}

[greptile-apps]

_{1 file reviewed, no comments}

_{Edit Code Review Agent Settings | Greptile}

[TimeToBuildBob]

@greptileai review

[greptile-apps]

_{1 file reviewed, no comments}

_{Edit Code Review Agent Settings | Greptile}