RenameFile

DogmaDragon · October 27, 2026, 4:05pm

RenameFile: harden detached process spawn with start_new_session

main ← phildapunk:fix/renamefile-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 `RenameFile`'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 `RenameFile` version 1.0.4 → 1.0.5, with a version_history entry. ### 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?