DupFileManager

DogmaDragon · October 27, 2026, 4:06pm

DupFileManager: harden detached process spawn with start_new_session

main ← phildapunk:fix/dupfilemanager-detach-start-new-session

opened 07:33AM - 26 Oct 26 UTC

### Context Split out of #802 at the maintainer's request to keep that PR scope…d to a single plugin. While fixing #801 (FileMonitor's background service never actually starting on macOS), I found that `executeProcess()`'s detach path — shared byte-for-byte across `FileMonitor`, `DupFileManager`, and `RenameFile`'s vendored `StashPluginHelper.py` copies — spawns the detached child with a plain `subprocess.Popen(args)`. The code already carried a comment flagging that macOS may need `start_new_session=True` so the child survives the launching task process exiting, but the flag was never applied. This PR applies the same fix to `DupFileManager`'s copy proactively. (FileMonitor's copy is fixed in #802, where the gap was actually reproduced and causing a filed bug; this plugin isn't known to be affected, but shares the identical code path.) ### What this does - `executeProcess()`: pass `start_new_session=True` when detaching. - Bump `DupFileManager` version 1.0.2 → 1.0.3. ### LLM-assisted contribution disclosure Per the repository's contribution policy: this change was prepared with LLM assistance (Claude). The diff has been reviewed by me and I take full responsibility for the change and its license compliance.

@David-Maisonave would you be interested in reviewing it?