@dimina-kit/devtools 0.4.0-dev.20260618090552 → 0.4.0-dev.20260624040247

package/README.md

@@ -670,9 +670,13 @@ pnpm build:preload      # 仅构建 preload TypeScript
 pnpm build:renderer     # Vite 构建渲染层
 pnpm dev                # Watch 模式 + Electron
 pnpm check-types        # 类型检查
-pnpm test               # 运行测试
+pnpm test               # 运行单元测试（vitest）
+pnpm test:coverage      # 单元测试 + 覆盖率报告（终端表格 + coverage/index.html）
+pnpm test:e2e           # 运行 Playwright e2e 测试
 ```
 
+> 覆盖率：`pnpm test:coverage`（单包）或仓库根 `pnpm test:coverage`（全部包），报告输出到各包 `coverage/`，浏览器打开 `coverage/index.html` 看逐行高亮。
+
 ---
 
 ## 安全说明

package/dist/main/app/app.js

@@ -240,6 +240,20 @@ function wireAppWindowEvents(config, instance) {
             // project is open.
             if (isAppQuitting())
                 return;
+            // A close arriving while a project teardown is already in flight (the
+            // user rapid-double-clicked the close button) MUST keep the window open.
+            // This guard runs BEFORE hasActiveSession() on purpose: `closeProject()`
+            // → `disposeSession()` synchronously nulls the active session before it
+            // finishes awaiting `session.close()`, so by the time the second close
+            // arrives `hasActiveSession()` is already false. With the old guard order
+            // that second close fell straight through the hasActiveSession() check
+            // WITHOUT `preventDefault()`, so Electron destroyed the last window →
+            // `window-all-closed` → `app.quit()`, quitting the whole app on a
+            // double-click. Swallow the re-entrant close and keep the window.
+            if (closing) {
+                e.preventDefault();
+                return;
+            }
             if (!context.workspace.hasActiveSession())
                 return;
             // Close button while a project session is open: stay in the workbench
@@ -249,8 +263,6 @@ function wireAppWindowEvents(config, instance) {
             // unable to invoke anything, so subsequent clicks on Import/etc. would
             // raise `No handler registered for ...`.
             e.preventDefault();
-            if (closing)
-                return;
             closing = true;
             try {
                 if (config.onBeforeClose) {