@dimina-kit/devtools 0.4.0-dev.20260616024534 → 0.4.0-dev.20260616085026

package/dist/main/index.bundle.js

@@ -885,15 +885,13 @@ function createMainWindow(opts) {
     }
   });
   mainWindow.once("ready-to-show", () => {
-    if (process.env.NODE_ENV === "test") {
-      mainWindow.showInactive();
-    } else {
-      if (opts.autoShow !== false) {
-        mainWindow.show();
-      }
-      if (!app3.isPackaged && process.env.DIMINA_DEVTOOLS_MAIN_INSPECTOR === "1") {
-        mainWindow.webContents.openDevTools({ mode: "detach" });
-      }
+    const isTest = process.env.NODE_ENV === "test";
+    if (opts.autoShow !== false) {
+      if (isTest) mainWindow.showInactive();
+      else mainWindow.show();
+    }
+    if (!isTest && !app3.isPackaged && process.env.DIMINA_DEVTOOLS_MAIN_INSPECTOR === "1") {
+      mainWindow.webContents.openDevTools({ mode: "detach" });
     }
   });
   const rendererDir2 = path5.dirname(opts.indexHtml);
@@ -2950,7 +2948,9 @@ function createWorkspaceService(ctx) {
         try {
           await ctx.onBeforeOpenProject(projectPath);
         } catch (err) {
-          return { success: false, error: err instanceof Error ? err.message : String(err) };
+          const error = err instanceof Error ? err.message : String(err);
+          sendStatus("error", error);
+          return { success: false, error };
         }
       }
       clearSimulatorServicewechatReferer();
@@ -9151,9 +9151,11 @@ function createDevtoolsBackend(config = {}) {
     // NOTE: `ownsWindows` is NOT structurally required — the framework's
     // `mainWindowWebPreferences()` + `onMainWindowCreated()` hooks can supply both,
     // and the project close→back lifecycle maps onto the Window facade's
-    // `onClose`/`newSession` (probe-validated). Retained for now; migrating to
-    // `runtime.windows.main` is the documented next step. (The `persist:simulator`
-    // partition is a fixed session used by child WCVs, not main-window state.)
+    // `onClose`/`newSession` (probe-validated). It is RETAINED BY DECISION, not
+    // inertia: migrating to `runtime.windows.main` buys no user value at high
+    // regression cost (see docs/deck-adoption-decision.md, F1). (The
+    // `persist:simulator` partition is a fixed session used by child WCVs, not
+    // main-window state.)
     ownsWindows: true,
     // Pre-ready: app name / CDP port / CSP / privileged scheme — must run before
     // the framework awaits app.whenReady().

package/dist/main/runtime/devtools-backend.d.ts

@@ -12,6 +12,11 @@ import type { WorkbenchAppConfig } from '../../shared/types.js';
  * backend is present), so `assemble` ignores the framework `runtime` and builds
  * the real devtools main window itself. Trust/frame unification and surfacing
  * the runtime facade are deferred (see framework-extraction-v2.md §7).
+ *
+ * NOT migrating onto deck's high-level host API (Window facade / runtime.view /
+ * DeckSession / grants) is a DELIBERATE, reviewed decision — not a backlog item.
+ * The ROI matrix (every candidate NO-GO) + revisit conditions live in
+ * docs/deck-adoption-decision.md. Read it before re-proposing such a migration.
  */
 export declare function createDevtoolsBackend(config?: WorkbenchAppConfig): RuntimeBackend;
 //# sourceMappingURL=devtools-backend.d.ts.map

package/dist/main/runtime/devtools-backend.js

@@ -12,6 +12,11 @@ import { registerAppLifecycle } from '../app/lifecycle.js';
  * backend is present), so `assemble` ignores the framework `runtime` and builds
  * the real devtools main window itself. Trust/frame unification and surfacing
  * the runtime facade are deferred (see framework-extraction-v2.md §7).
+ *
+ * NOT migrating onto deck's high-level host API (Window facade / runtime.view /
+ * DeckSession / grants) is a DELIBERATE, reviewed decision — not a backlog item.
+ * The ROI matrix (every candidate NO-GO) + revisit conditions live in
+ * docs/deck-adoption-decision.md. Read it before re-proposing such a migration.
  */
 export function createDevtoolsBackend(config = {}) {
     // Hoisted so `onShutdown` can reach the assembled context (assigned by `assemble`).
@@ -23,9 +28,11 @@ export function createDevtoolsBackend(config = {}) {
         // NOTE: `ownsWindows` is NOT structurally required — the framework's
         // `mainWindowWebPreferences()` + `onMainWindowCreated()` hooks can supply both,
         // and the project close→back lifecycle maps onto the Window facade's
-        // `onClose`/`newSession` (probe-validated). Retained for now; migrating to
-        // `runtime.windows.main` is the documented next step. (The `persist:simulator`
-        // partition is a fixed session used by child WCVs, not main-window state.)
+        // `onClose`/`newSession` (probe-validated). It is RETAINED BY DECISION, not
+        // inertia: migrating to `runtime.windows.main` buys no user value at high
+        // regression cost (see docs/deck-adoption-decision.md, F1). (The
+        // `persist:simulator` partition is a fixed session used by child WCVs, not
+        // main-window state.)
         ownsWindows: true,
         // Pre-ready: app name / CDP port / CSP / privileged scheme — must run before
         // the framework awaits app.whenReady().

package/dist/main/services/workspace/workspace-service.js

@@ -90,7 +90,15 @@ export function createWorkspaceService(ctx) {
                     await ctx.onBeforeOpenProject(projectPath);
                 }
                 catch (err) {
-                    return { success: false, error: err instanceof Error ? err.message : String(err) };
+                    // Surface the veto to the status bar — symmetric with the
+                    // validateProjectDir rejection below (which already emits an error
+                    // status). The framework caught the error, so it owns the minimal
+                    // user-visible feedback; a host gate must not have to thread
+                    // `notify` through a singleton just to report why the open was
+                    // denied. The host can still layer richer UX (e.g. a dialog).
+                    const error = err instanceof Error ? err.message : String(err);
+                    sendStatus('error', error);
+                    return { success: false, error };
                 }
             }
             clearSimulatorServicewechatReferer();

package/dist/main/windows/main-window/create.js

@@ -22,21 +22,24 @@ export function createMainWindow(opts) {
         },
     });
     mainWindow.once('ready-to-show', () => {
-        if (process.env.NODE_ENV === 'test') {
-            mainWindow.showInactive();
-        }
-        else {
-            // A login-gating host opts out via `autoShow: false` and shows the
-            // window itself once auth passes — don't flash an un-authed window.
-            if (opts.autoShow !== false) {
+        const isTest = process.env.NODE_ENV === 'test';
+        // Visibility is governed by `autoShow` in BOTH envs. A login-gating host
+        // opts out via `autoShow: false` and reveals the window itself once auth
+        // passes — don't flash an un-authed window. The env only chooses HOW to
+        // show: test uses showInactive() so e2e windows don't steal focus, prod
+        // uses show(). The framework must never force-show when the host opted
+        // out, in test env either (that would fight the host's own reveal handler).
+        if (opts.autoShow !== false) {
+            if (isTest)
+                mainWindow.showInactive();
+            else
                 mainWindow.show();
-            }
-            // Don't auto-open a detached DevTools for the devtools UI shell itself —
-            // it's noise for normal use (the mini-app's Console lives in the embedded
-            // right-panel DevTools, not here). Opt in via env for debugging the shell.
-            if (!app.isPackaged && process.env.DIMINA_DEVTOOLS_MAIN_INSPECTOR === '1') {
-                mainWindow.webContents.openDevTools({ mode: 'detach' });
-            }
+        }
+        // Don't auto-open a detached DevTools for the devtools UI shell itself —
+        // it's noise for normal use (the mini-app's Console lives in the embedded
+        // right-panel DevTools, not here). Opt in via env for debugging the shell.
+        if (!isTest && !app.isPackaged && process.env.DIMINA_DEVTOOLS_MAIN_INSPECTOR === '1') {
+            mainWindow.webContents.openDevTools({ mode: 'detach' });
         }
     });
     const rendererDir = path.dirname(opts.indexHtml);