[mortenaslo]

Summary

Fixes from the v0.10.0 pre-release pass. Each one was reproduced before it was fixed.

1. "Open in Explorer" opened Documents instead of the folder for forward-slash paths (the form git prints, so every session opened in a sibling worktree) and for names containing a comma.
2. Restore and Wake permanently deleted a session whose launch failed. This bug predates v0.9.0.
3. Worktree remove (feat(git): worktrees in the git menu — where am I, switch-aware, go to, prune, remove #180) could:
   • delete a worktree nested inside the one being removed, uncommitted work included;
   • delete sleeping sessions even when git then refused the remove.
   • WSL paths were also matched the wrong way.

Why

1. Explorer

The folder was passed to explorer.exe as its raw command line. I measured the result by asking the Shell which folder each new Explorer window showed:

Path	Before	After (FolderOpener)
plain, spaces, &, %, trailing \, 8.3	✓	✓
C:/forward/slashes	Documents	✓
a,b (quoting doesn't help)	Documents	✓

Services/FolderOpener opens the folder itself through the shell (ShellExecute "open" on the full Windows path). It refuses anything that isn't an existing directory first: ShellExecute on a file runs it, and folder paths come from state.json and imports. It runs off the UI thread. New sibling-worktree sessions now store the Windows path form in the first place.

2. Restore and Wake

Both called LaunchSessionAsync with the default removeOnFailure: true. A refused launch (for example a WSL distro that's gone) or a PTY that failed to start therefore removed the session, and the next save made that permanent. Wake's own catch never ran, because the PTY-failure path returns rather than throws.

Both now pass removeOnFailure: false and end in FallBackToDormantIfNotLive. That also covers every WebView2 failure on restore; #175 only covered the two batched ones.

Before/after, run against a real state.json via CSM_STATE_PATH:

	Before (main 448aca7)	After
sidebar row	gone	kept, asleep
saved in state.json	deleted	kept, IsDormant=true

3. Worktree remove

• Nested worktree. git's clean check skips ignored folders. Reproduced with git 2.54: worktree B in A's .gitignored .worktrees/, with uncommitted work. A reports clean, and git worktree remove A exits 0 and deletes B. The menu now refuses while GitWorktrees.NestedInside finds any, checked against a list re-read at click time.
• Sleeping sessions are deleted only after git has removed the folder. They never go through Recently closed, so a refusal used to lose them for good.
• WSL paths:
  • \\wsl.localhost\ is folded into \\wsl$\. Before, such a session matched no worktree at all.
  • Below the distro name, paths compare case-sensitively: /home/u/WT is not /home/u/wt.

Implementation notes

• Not fixed here: a mixed-DPI window-bounds restore issue and two low-severity git items from the same review. Tracked in a follow-up issue.
• CLAUDE.md: new "Opening a folder in Explorer" section; Wake/restore failure behaviour under Session Lifecycle; the nested / sleeping / WSL rules under "Worktrees in the git menu".

Test plan

• FolderOpenerTests: forward slashes, comma, &, %, spaces, drive root; a file, a missing path, a relative path, a URL and calc.exe are all refused.
• GitWorktreeTests: WSL case rules and both share spellings; NestedInside; and the real-git reproduction of the nested layout (A reported clean, B detected).
• Restore failure, before/after against the real app (table above).
• Full suite: 762/762.
• Manual: 📁 on a session opened via the Worktrees menu.

🤖 Generated with Claude Code

https://claude.ai/code/session_01NsYm9iLc2aRnRDfV2uZRQX