修复导出预览独立窗口在 neko-pc 中最小化、最大化/恢复按钮无效的问题 (#1555)

* 修复导出预览独立窗口在 neko-pc 中最小化、最大化/恢复按钮无效的问题

* 检查 nekoWindowControl 是否已存在，并通过 waitForExportPreviewRewriteGate 做两段等待：先等 1.5s，失败后再等 6.5s；只有 shell 已 ready 或窗口控制 API 已注入时，才会 stop()/document.write()。如果仍然没满足条件，会关闭这次预览窗口并返回，不再抢断 preload 导航

Author: MingTianSang

static/app-chat-export.js

@@ -122,6 +122,93 @@
         }
     }
 
+    function isExportPreviewShellReady(previewWindow, targetUrl) {
+        if (!previewWindow || previewWindow.closed) return false;
+        try {
+            var href = previewWindow.location && previewWindow.location.href;
+            if (!href || href === 'about:blank') return false;
+            var current = new URL(href, window.location.href);
+            var target = new URL(targetUrl, window.location.href);
+            if (current.origin !== target.origin || current.pathname !== target.pathname) {
+                return false;
+            }
+            var doc = previewWindow.document;
+            return !!(doc && (doc.readyState === 'interactive' || doc.readyState === 'complete'));
+        } catch (_) {
+            return false;
+        }
+    }
+
+    function hasExportPreviewWindowControlApi(previewWindow) {
+        if (!previewWindow || previewWindow.closed) return false;
+        try {
+            var api = previewWindow.nekoWindowControl;
+            return !!(api && typeof api.minimize === 'function' && typeof api.maximize === 'function');
+        } catch (_) {
+            return false;
+        }
+    }
+
+    function waitForExportPreviewShell(previewWindow, targetUrl, timeoutMs) {
+        return new Promise(function (resolve) {
+            if (!previewWindow || previewWindow.closed) {
+                resolve(false);
+                return;
+            }
+
+            var waitMs = Number(timeoutMs);
+            if (!Number.isFinite(waitMs) || waitMs <= 0) waitMs = 1500;
+            var settled = false;
+            var pollTimer = null;
+            var timeoutTimer = null;
+
+            function cleanup() {
+                if (pollTimer) {
+                    window.clearInterval(pollTimer);
+                    pollTimer = null;
+                }
+                if (timeoutTimer) {
+                    window.clearTimeout(timeoutTimer);
+                    timeoutTimer = null;
+                }
+                try {
+                    previewWindow.removeEventListener('load', checkReady);
+                } catch (_) {}
+            }
+
+            function finish(ok) {
+                if (settled) return;
+                settled = true;
+                cleanup();
+                resolve(!!ok);
+            }
+
+            function checkReady() {
+                if (!previewWindow || previewWindow.closed) {
+                    finish(false);
+                    return;
+                }
+                if (isExportPreviewShellReady(previewWindow, targetUrl)) {
+                    finish(true);
+                }
+            }
+
+            try {
+                previewWindow.addEventListener('load', checkReady);
+            } catch (_) {}
+            pollTimer = window.setInterval(checkReady, 40);
+            timeoutTimer = window.setTimeout(function () { finish(false); }, waitMs);
+            checkReady();
+        });
+    }
+
+    async function waitForExportPreviewRewriteGate(previewWindow, targetUrl) {
+        var shellReady = await waitForExportPreviewShell(previewWindow, targetUrl, 1500);
+        if (shellReady || hasExportPreviewWindowControlApi(previewWindow)) return true;
+        shellReady = await waitForExportPreviewShell(previewWindow, targetUrl, 6500);
+        return !!(shellReady || hasExportPreviewWindowControlApi(previewWindow));
+    }
+
     function showToast(key, fallback, duration) {
         if (typeof window.showStatusToast !== 'function') return;
         window.showStatusToast(translateLabel(key, fallback), duration || 3000);
@@ -2465,10 +2552,21 @@
         if (isExistingWindow) {
             disposePreviewModal(false);
         }
+        state.previewWindow = previewWindow;
+        if (!isExistingWindow) {
+            var canRewritePreview = await waitForExportPreviewRewriteGate(previewWindow, getExportPreviewShellUrl());
+            if (!previewWindow || previewWindow.closed) return null;
+            if (!canRewritePreview) {
+                if (state.previewWindow === previewWindow) state.previewWindow = null;
+                try {
+                    previewWindow.close();
+                } catch (_) {}
+                return null;
+            }
+        }
         try {
             if (typeof previewWindow.stop === 'function') previewWindow.stop();
         } catch (_) {}
-        state.previewWindow = previewWindow;
         var doc = previewWindow.document;
         doc.open();
         doc.write('<!DOCTYPE html><html lang="' + escapeHtml(document.documentElement.lang || 'en') + '"' + getPreviewThemeAttributesHtml() + '><head><meta charset="utf-8">'

tests/unit/test_chat_export_static_contracts.py

@@ -0,0 +1,24 @@
+from pathlib import Path
+
+
+PROJECT_ROOT = Path(__file__).resolve().parents[2]
+CHAT_EXPORT_JS = PROJECT_ROOT / "static" / "app-chat-export.js"
+
+
+def test_export_preview_waits_for_shell_before_rewriting_document():
+    script = CHAT_EXPORT_JS.read_text(encoding="utf-8")
+
+    assert "function waitForExportPreviewShell(previewWindow, targetUrl, timeoutMs)" in script
+    assert "function waitForExportPreviewRewriteGate(previewWindow, targetUrl)" in script
+    assert "function hasExportPreviewWindowControlApi(previewWindow)" in script
+    assert "function isExportPreviewShellReady(previewWindow, targetUrl)" in script
+    assert "href === 'about:blank'" in script
+    assert "previewWindow.addEventListener('load', checkReady)" in script
+    assert "waitForExportPreviewShell(previewWindow, targetUrl, 6500)" in script
+    assert "shellReady || hasExportPreviewWindowControlApi(previewWindow)" in script
+
+    gate_index = script.index("await waitForExportPreviewRewriteGate(previewWindow, getExportPreviewShellUrl());")
+    guard_index = script.index("if (!canRewritePreview) {", gate_index)
+    stop_index = script.index("if (typeof previewWindow.stop === 'function') previewWindow.stop();", gate_index)
+    doc_open_index = script.index("var doc = previewWindow.document;", gate_index)
+    assert gate_index < guard_index < stop_index < doc_open_index