Release v1.0.5: crash on widget delete, window leaks, color and palette fixes

- Fix crash (EXC_BAD_ACCESS) when deleting/hiding/reloading a widget:
  isReleasedWhenClosed = false (close() was double-releasing under ARC)
- Fix widget window/terminal leak: close() instead of orderOut, deferred
  past the context-menu action dispatch
- Fix MilkDrop visualizer background thread: timer now stops correctly
  (CFRunLoopWakeUp + join on its own run loop), no more leaked 24fps threads
- Fix wallpaper palette inconsistency across displays: whole color pipeline
  is now sRGB-tagged (no more oversaturation on P3 displays), deterministic
  palette ordering, per-screen wallpaper URL resolution
- MilkDrop: smooth palette/color cross-fade on wallpaper change
- Fix widget position drifting on restart (grid snap applied to restored frame)
- Fix snap guides stuck on screen after mouse-up (final snap re-showed them)
- Fix "show on all Spaces" not applying until restart (collectionBehavior)
- Apply interface language before config load (localized default names)
- Config.save: atomic write, log errors instead of silently swallowing
- Remove Telegram item from the status menu

Co-authored-by: factory-droid[bot] <138933559+factory-droid[bot]@users.noreply.github.com>

Author: titovcode

MilkDropResources/milkdrop.html

@@ -56,6 +56,9 @@
   // --- Rainbow state ---
   // Палитра обоев [[r,g,b], ...]. Пусто — режим выключен.
   let palette = [];
+  // Отображаемая палитра: каждый кадр плавно тянется к `palette`, чтобы
+  // смена обоев не переключала цвета резко (как cross-fade фона виджета).
+  let displayedPalette = [];
   let rainbowEnabled = false;
   let rainbowWaveIndex = 0;
   // Beat-детектор низа (кик).
@@ -79,12 +82,14 @@
   let glowPulse = 0;
 
   // Цвет палитры в непрерывной позиции t (циклично, с интерполяцией).
+  // Берём отображаемую палитру, чтобы переход между палитрами был плавным.
   function samplePalette(t) {
-    const N = palette.length;
+    const src = displayedPalette.length > 0 ? displayedPalette : palette;
+    const N = src.length;
     const f = (((t % 1) + 1) % 1) * N;
     const i = Math.floor(f);
     const frac = f - i;
-    const a = palette[i % N], b = palette[(i + 1) % N];
+    const a = src[i % N], b = src[(i + 1) % N];
     return [a[0] + (b[0] - a[0]) * frac,
             a[1] + (b[1] - a[1]) * frac,
             a[2] + (b[2] - a[2]) * frac];
@@ -127,7 +132,10 @@
   function advanceRainbow() {
     if (palette.length === 0) return;
     rainbowWaveIndex = (rainbowWaveIndex + 1) % palette.length;
-    targetColor = palette[rainbowWaveIndex].slice();
+    // Цель берём из отображаемой палитры: при смене обоев волна плавно
+    // перетечёт в новые оттенки, а не прыгнет в ещё не переехавший цвет.
+    const src = (displayedPalette.length === palette.length) ? displayedPalette : palette;
+    targetColor = src[rainbowWaveIndex].slice();
   }
 
 
@@ -442,6 +450,25 @@
       glowColor[2] = waveColor[2];
     }
 
+    // Плавный переход палитры к новой (смена обоев): тянем каждый канал
+    // каждого цвета. Темп 0.05/кадр ≈ то же ~1 с переливание, что и у фона.
+    if (palette.length > 0) {
+      if (displayedPalette.length !== palette.length) {
+        // Стартуем от текущего цвета волны — переход без вспышки.
+        displayedPalette = palette.map(() => waveColor.slice());
+      }
+      for (let i = 0; i < palette.length; i++) {
+        for (let ch = 0; ch < 3; ch++) {
+          const d = palette[i][ch] - displayedPalette[i][ch];
+          displayedPalette[i][ch] = Math.abs(d) < 0.5
+            ? palette[i][ch]
+            : displayedPalette[i][ch] + d * 0.05;
+        }
+      }
+    } else if (displayedPalette.length > 0) {
+      displayedPalette = [];
+    }
+
     // «Панч»: масштабируем всю картинку от центра на время удара.
     const scaled = punch > 0.0005;
     if (scaled) {
@@ -497,11 +524,15 @@
       ? colors.filter(c => Array.isArray(c) && c.length === 3)
               .map(c => vivify([c[0] | 0, c[1] | 0, c[2] | 0]))
       : [];
+    const wasEnabled = rainbowEnabled;
     rainbowEnabled = !!enabled && palette.length > 0;
-    if (rainbowEnabled) {
-      // Стартуем с первого цвета палитры.
+    if (rainbowEnabled && !wasEnabled) {
+      // Сбрасываем позицию ТОЛЬКО при включении режима. При обычной смене
+      // обоев позицию по битам не трогаем — цвета сами плавно перетекут
+      // через displayedPalette и интерполяцию волны.
       rainbowWaveIndex = 0;
-      targetColor = palette[0].slice();
+      const src = (displayedPalette.length === palette.length) ? displayedPalette : palette;
+      targetColor = src[0].slice();
       beatCooldown = 0;
       idleFrames = 0;
     }

Sources/TermJ/AppDelegate.swift

@@ -52,6 +52,11 @@ final class AppDelegate: NSObject, NSApplicationDelegate, NSWindowDelegate {
     var editorCleanModeCheckbox: NSButton?
 
     func applicationDidFinishLaunching(_ notification: Notification) {
+        // Язык применяем ДО загрузки конфига: локализованные дефолтные
+        // имена виджетов вычисляются в момент декода/создания конфига.
+        if let lang = AppConfig.peekLanguage() {
+            TermJPreferredLanguage = lang == "auto" ? nil : lang
+        }
         appConfig = AppConfig.load()
         common = appConfig.common
 
@@ -176,7 +181,15 @@ final class AppDelegate: NSObject, NSApplicationDelegate, NSWindowDelegate {
         if let term = inst.contentView as? TermWidgetView {
             term.terminateProcessGroup()
         }
-        inst.window.orderOut(nil)
+        // Прячем сразу, а close() откладываем на следующий цикл runloop:
+        // удаление приходит из контекстного меню окна, и AppKit ещё
+        // работает с этим меню после возврата из action — немедленный
+        // close() освобождал окно и меню прямо во время диспетча → краш.
+        // При этом окно обязательно закрываем (не только orderOut), иначе
+        // оно вместе с наблюдателями/терминалом живёт до конца сессии.
+        let closingWindow = inst.window
+        closingWindow.orderOut(nil)
+        DispatchQueue.main.async { closingWindow.close() }
         appConfig.widgets.removeAll { $0.id == id }
         appConfig.save()
         updateOtherWidgets()
@@ -536,7 +549,12 @@ final class AppDelegate: NSObject, NSApplicationDelegate, NSWindowDelegate {
                 if let term = inst.contentView as? TermWidgetView {
                     term.terminateProcessGroup()
                 }
-                inst.window.orderOut(nil)
+                // close() обязательно (иначе окно утекает вместе с
+                // наблюдателями), но отложенно — action может прийти из
+                // контекстного меню этого же окна (см. destroyWidget).
+                let closingWindow = inst.window
+                closingWindow.orderOut(nil)
+                DispatchQueue.main.async { closingWindow.close() }
                 widgets.removeValue(forKey: id)
                 updateOtherWidgets()
             }
@@ -817,7 +835,11 @@ final class AppDelegate: NSObject, NSApplicationDelegate, NSWindowDelegate {
                 term.terminateProcessGroup()
             }
             saveWidgetConfig(id: id, from: inst.window)
-            inst.window.orderOut(nil)
+            // close() обязательно, но отложенно — action может прийти из
+            // контекстного меню этого же окна (см. destroyWidget).
+            let closingWindow = inst.window
+            closingWindow.orderOut(nil)
+            DispatchQueue.main.async { closingWindow.close() }
             widgets.removeValue(forKey: id)
         }
 

Sources/TermJ/Audio/AudioCaptureManager.swift

@@ -96,7 +96,7 @@ final class AudioCaptureManager: NSObject, SCStreamOutput, SCStreamDelegate {
     // MARK: - Таймер отображения
 
     private func startDisplayTimerIfNeeded() {
-        guard backgroundTimer == nil else { return }
+        guard backgroundTimerThread == nil else { return }
         startBackgroundTimer()
         // Fallback-таймер на main нужен только на случай, если background thread
         // по какой-то причине остановится.
@@ -114,7 +114,11 @@ final class AudioCaptureManager: NSObject, SCStreamOutput, SCStreamDelegate {
     /// Запускает основной рендеринг на background RunLoop.
     /// NSMenu в статус-баре блокирует main RunLoop, но не этот поток.
     private func startBackgroundTimer() {
-        guard backgroundTimer == nil else { return }
+        // Проверяем именно поток, а не таймер: таймер присваивается
+        // асинхронно на фоновом потоке, и проверка `backgroundTimer == nil`
+        // давала гонку при быстром вкл/выкл виджета.
+        guard backgroundTimerThread == nil else { return }
+        shouldStopBackgroundTimer = false
 
         let thread = Thread { [weak self] in
             guard let self = self else { return }
@@ -128,8 +132,11 @@ final class AudioCaptureManager: NSObject, SCStreamOutput, SCStreamDelegate {
             while !self.shouldStopBackgroundTimer {
                 runLoop.run(mode: .default, before: Date(timeIntervalSinceNow: 0.1))
             }
-            self.backgroundTimer?.invalidate()
+            // Инвалидируем таймер на его собственном потоке — invalidate()
+            // с другого потока для RunLoop-таймера недопустим.
+            timer.invalidate()
             self.backgroundTimer = nil
+            self.backgroundRunLoop = nil
         }
         thread.name = "termj.visualizer.renderer"
         thread.start()
@@ -139,11 +146,21 @@ final class AudioCaptureManager: NSObject, SCStreamOutput, SCStreamDelegate {
     private var shouldStopBackgroundTimer = false
 
     private func stopBackgroundTimer() {
+        guard let thread = backgroundTimerThread else { return }
         shouldStopBackgroundTimer = true
-        backgroundRunLoop = nil
-        backgroundTimer?.invalidate()
-        backgroundTimer = nil
+        // Будим RunLoop, чтобы цикл сразу увидел флаг и вышел.
+        if let runLoop = backgroundRunLoop {
+            CFRunLoopWakeUp(runLoop.getCFRunLoop())
+        }
+        // Ждём завершения потока: он сам инвалидирует таймер на своём RunLoop.
+        var waited: TimeInterval = 0
+        while !thread.isFinished, waited < 1.0 {
+            Thread.sleep(forTimeInterval: 0.01)
+            waited += 0.01
+        }
         backgroundTimerThread = nil
+        backgroundTimer = nil
+        backgroundRunLoop = nil
         shouldStopBackgroundTimer = false
     }
 

Sources/TermJ/Config.swift

@@ -319,6 +319,19 @@ struct AppConfig: Codable {
         return dir.appendingPathComponent("config.json")
     }
 
+    /// Читает только `common.language` из файла конфига, не декодируя всё.
+    /// Нужно ДО полной загрузки: локализованные дефолты (имена виджетов)
+    /// вычисляются в момент декода и должны подхватить выбранный язык.
+    static func peekLanguage() -> String? {
+        struct LangPeek: Decodable {
+            struct Common: Decodable { let language: String? }
+            let common: Common?
+        }
+        guard let data = try? Data(contentsOf: fileURL),
+              let peek = try? JSONDecoder().decode(LangPeek.self, from: data) else { return nil }
+        return peek.common?.language
+    }
+
     /// Загружает конфиг с диска, при отсутствии — создаёт дефолтный.
     static func load() -> AppConfig {
         let url = fileURL
@@ -344,8 +357,13 @@ struct AppConfig: Codable {
     func save() {
         let enc = JSONEncoder()
         enc.outputFormatting = [.prettyPrinted, .sortedKeys, .withoutEscapingSlashes]
-        if let data = try? enc.encode(self) {
-            try? data.write(to: AppConfig.fileURL)
+        do {
+            let data = try enc.encode(self)
+            try data.write(to: AppConfig.fileURL, options: .atomic)
+        } catch {
+            // Не глотаем ошибку молча: иначе пользователь теряет layout
+            // при следующем запуске без какой-либо диагностики.
+            print("[TermJ] Failed to save config to \(AppConfig.fileURL.path): \(error)")
         }
     }
 }

Sources/TermJ/StatusMenu.swift

@@ -213,12 +213,6 @@ extension AppDelegate {
         githubItem.image = githubMenuIcon()
         menu.addItem(githubItem)
 
-        let supportItem = NSMenuItem(title: L("menu.support"),
-                                     action: #selector(openSupport), keyEquivalent: "")
-        supportItem.target = self
-        supportItem.image = NSImage(systemSymbolName: "paperplane.fill", accessibilityDescription: nil)
-        menu.addItem(supportItem)
-
         menu.addItem(NSMenuItem.separator())
 
         let quitItem = NSMenuItem(title: L("menu.quit"),

Sources/TermJ/TermWidgetView.swift

@@ -280,11 +280,12 @@ final class TermWidgetView: LocalProcessTerminalView, LocalProcessTerminalViewDe
     }
 
     private func interpolateColor(_ a: NSColor, _ b: NSColor, _ t: CGFloat) -> NSColor {
-        let a1 = a.usingColorSpace(.deviceRGB)
-        let b1 = b.usingColorSpace(.deviceRGB)
+        let a1 = a.usingColorSpace(.sRGB)
+        let b1 = b.usingColorSpace(.sRGB)
         if let a1, let b1 {
+            // sRGB, чтобы промежуточные кадры анимации не перенасыщались на P3.
             return NSColor(
-                red: a1.redComponent + (b1.redComponent - a1.redComponent) * t,
+                srgbRed: a1.redComponent + (b1.redComponent - a1.redComponent) * t,
                 green: a1.greenComponent + (b1.greenComponent - a1.greenComponent) * t,
                 blue: a1.blueComponent + (b1.blueComponent - a1.blueComponent) * t,
                 alpha: a1.alphaComponent + (b1.alphaComponent - a1.alphaComponent) * t