fix: protect checksum workers and GUI garbage collection

[lukisch]

Summary

• Keep the checksum dialog and its native worker alive until the worker has actually stopped.
• Run cyclic Qt-object cleanup on the GUI thread; initialize the timer before disabling automatic collection and restore the original GC state even when shutdown cleanup raises.
• Close the GUI collector if any application setup after installation fails.
• Preserve the current master changelog entry alongside the checksum and GUI-GC entry.

Validation

• Focused checksum and GUI-GC tests: 25 passed.
• Full local suite: 416 passed, 2 skipped. Release smoke needs a locally built EXE; screenshot smoke needs a native Qt platform and was skipped with the offscreen platform.
• Translation check: 356 strings complete in all six languages.
• Ruff, compileall, and diff checks passed.
• GitHub CI and CodeQL for this updated head are pending.

[github-actions]

Welcome! Thanks for your first pull request in this repository.

A maintainer will review it soon. Please make sure:

• Your changes are tested
• Documentation is updated if needed
• The PR description explains what changed and why

Thanks for contributing.

[lukisch]

Zweitmodell-Review: CI grün (12/12), derselbe Sicherheitsbefund wie in #14.

1. Befehlsinjektion beim Terminal-Start (src/core/platform_utils.py, get_terminal_command)
Der Verzeichnisname wird in powershell -Command "Set-Location -LiteralPath '{directory}'" und cmd /K "cd /d {directory}" interpoliert. Ein Ordnername mit ' oder & (unter Windows zulässig) führt zu Codeausführung. Popen(..., cwd=target_dir) setzt das Arbeitsverzeichnis bereits, die Interpolation kann entfallen:

if shutil.which("powershell"):
    return ["powershell", "-NoExit"]
return ["cmd", "/K"]

Dazu ein Test mit einem Ordner a'b&c.

2. Überlappung
Der Inhalt ist (bis auf die Laufwerkskapazität) in #14 enthalten, batch_rename_service, checksum_dialog, gui_gc und properties_dialog sind in beiden PRs identisch oder fast identisch. Es sollte nur eine der beiden Varianten gemergt werden, sonst Konflikte.

Sonst: Tests für Batch-Rename-Rollback, Properties-Dialog und Terminal vorhanden. Keine Credentials oder Nutzerpfade im Diff.

---

Generated by Claude Code

[lukisch]

Korrektur zu meinem Review vom 3.10.2026

Frühere Aussage: Der Review nennt „derselbe Sicherheitsbefund wie in #14" als Befund dieses PR.

Korrektur: Die betroffene Funktion in src/core/platform_utils.py steht bereits auf dem Hauptzweig (master, zum Prüfzeitpunkt 8a3989c) und wird von diesem PR nicht geändert: Die Datei steht nicht in der Dateiliste des PR. Geprüft gegen den Stand 2d8120c306b2b3c733be0a0a3a5f1abf69402d99. Es ist kein Befund dieses PR. Siehe auch die Korrektur auf #14.

Bestätigt bleibt: #15 hat gegenüber #14 keinen eigenen Inhalt (#15 ist Git-Vorfahre von #14).

Hinweis zu Details: Weitere technische Einzelheiten bitte über einen dafür geeigneten privaten Kanal klären, nicht in diesem Thread. Entscheidungen bleiben offen.

---

Generated by Claude Code