From 180397f10a0d52b398ba6406fb9b9db57025a079 Mon Sep 17 00:00:00 2001 From: Sucukdeluxe Date: Tue, 24 Mar 2026 08:58:07 +0100 Subject: [PATCH] Revert post-MKV-move auto-rename that corrupted other packages The post-MKV-move rename pass added in v1.7.107 ran on the shared mkvLibraryDir (Entpackt/), causing files from OTHER packages to be renamed to the current package's name. For example, Orange.Is.The.New.Black files were renamed to Ted.S02E13...SAUERKRAUT.mkv. Remove the post-MKV-move rename entirely. The original hybrid race condition (1 file per season not renamed) is far less damaging than cross-package corruption. Co-Authored-By: Claude Opus 4.6 (1M context) --- src/main/download-manager.ts | 16 ---------------- 1 file changed, 16 deletions(-) diff --git a/src/main/download-manager.ts b/src/main/download-manager.ts index 6df43f7..bc8af47 100644 --- a/src/main/download-manager.ts +++ b/src/main/download-manager.ts @@ -10745,22 +10745,6 @@ export class DownloadManager extends EventEmitter { pkg.postProcessLabel = "Verschiebe Videos..."; this.emitState(); await this.collectMkvFilesToLibrary(packageId, pkg, shouldAbort); - - // ── Post-MKV-move rename pass ── - // During hybrid extraction, files can finish extracting between the - // auto-rename scan and MKV-move, causing them to be moved with their - // original scene names (e.g. awa-diethundermans03e21hd.mkv). - // Run auto-rename on the MKV library dir to catch these stragglers. - if (this.settings.autoRename4sf4sj && this.settings.collectMkvToLibrary) { - const mkvDir = String(this.settings.mkvLibraryDir || "").trim(); - if (mkvDir && await this.existsAsync(mkvDir)) { - throwIfAborted(); - const postMkvRenamed = await this.autoRenameExtractedVideoFiles(mkvDir, pkg, shouldAbort); - if (postMkvRenamed > 0) { - logger.info(`Post-MKV-Move Auto-Rename: pkg=${pkg.name}, renamed=${postMkvRenamed} in ${mkvDir}`); - } - } - } } throwIfAborted();