@trops/dash-core 0.1.595 → 0.1.596

package/dist/index.esm.js

@@ -60497,24 +60497,40 @@ var DashboardStageInner = function DashboardStageInner(_ref3) {
 
   // ─── Unsaved-changes guard (Phase 2B) ────────────────────────────
   // Single source of truth for "is the user mid-edit with un-saved
-  // work?". Edit mode (`previewMode === false`) is necessary; a
-  // populated `currentWorkspaceRef` is sufficient — the layout
-  // builder mutates that ref on every drag/drop/widget change.
+  // work?". A state variable — NOT a derived ref check — because the
+  // useEffect that mirrors this to `globalThis.__dashboardIsDirty`
+  // must re-run when the value changes, and `useRef` mutations don't
+  // trigger re-renders.
   //
-  // The flag is mirrored to `globalThis.__dashboardIsDirty` so the
-  // Electron main process can poll it synchronously during window
-  // close + app-quit handlers without an IPC round-trip.
+  // Mutations that flip it true:
+  //   - LayoutBuilder.onWorkspaceChange (widget drag/drop/swap/delete)
+  //     via `handleWorkspaceChange`
+  //   - Header title rename via `handleWorkspaceNameChange`
+  //   - Folder / theme changes via `handleWorkspaceFolderChange`,
+  //     `handleWorkspaceThemeChange`
+  //
+  // Mutations that reset to false:
+  //   - Successful save (`handleSaveWorkspaceComplete`)
+  //   - Cancel/discard (`performCancelEdit` + the discard modal path)
+  //   - Re-entering edit mode (`handleToggleEditMode` enter branch)
+  //
+  // Mirrored to `globalThis.__dashboardIsDirty` so the Electron main
+  // process can poll synchronously during window close + app-quit
+  // handlers without an IPC round-trip.
   //
   // A `pendingNavigation` object captures a navigation request that
   // arrived while dirty so the user can choose to discard or keep
   // editing. Shapes:
   //   { kind: "open-workspace", workspace }       — sidebar switch
   //   { kind: "cancel-edit" }                     — Cancel button
-  var _useState57 = useState(null),
+  var _useState57 = useState(false),
     _useState58 = _slicedToArray(_useState57, 2),
-    pendingNavigation = _useState58[0],
-    setPendingNavigation = _useState58[1];
-  var isDirty = previewMode === false && currentWorkspaceRef.current !== null;
+    isDirty = _useState58[0],
+    setIsDirty = _useState58[1];
+  var _useState59 = useState(null),
+    _useState60 = _slicedToArray(_useState59, 2),
+    pendingNavigation = _useState60[0],
+    setPendingNavigation = _useState60[1];
   useEffect(function () {
     if (typeof globalThis !== "undefined") {
       globalThis.__dashboardIsDirty = isDirty;
@@ -60612,10 +60628,10 @@ var DashboardStageInner = function DashboardStageInner(_ref3) {
   // We record the requested ID and open it once it appears in
   // workspaceConfig — handles the case where the workspace was just
   // created and the config reload is still in flight.
-  var _useState59 = useState(null),
-    _useState60 = _slicedToArray(_useState59, 2),
-    pendingOpenWorkspaceId = _useState60[0],
-    setPendingOpenWorkspaceId = _useState60[1];
+  var _useState61 = useState(null),
+    _useState62 = _slicedToArray(_useState61, 2),
+    pendingOpenWorkspaceId = _useState62[0],
+    setPendingOpenWorkspaceId = _useState62[1];
   useEffect(function () {
     var handler = function handler(e) {
       var _e$detail2;
@@ -61036,6 +61052,9 @@ var DashboardStageInner = function DashboardStageInner(_ref3) {
       setPreviewMode(function () {
         return false;
       });
+      // LayoutBuilder fired a mutation (widget drag/drop/swap/delete).
+      // Mark the workspace dirty so the navigation + close guards fire.
+      setIsDirty(true);
 
       // Update the tab's workspace reference
       if (activeTabId) {
@@ -61239,10 +61258,10 @@ var DashboardStageInner = function DashboardStageInner(_ref3) {
   }
 
   // ─── Page State ──────────────────────────────────────────────────
-  var _useState61 = useState(null),
-    _useState62 = _slicedToArray(_useState61, 2),
-    activePageId = _useState62[0],
-    setActivePageId = _useState62[1];
+  var _useState63 = useState(null),
+    _useState64 = _slicedToArray(_useState63, 2),
+    activePageId = _useState64[0],
+    setActivePageId = _useState64[1];
 
   // Page history stack for goBack() — pushes the previous page id
   // whenever a navigation happens through navigateToPage().
@@ -61638,12 +61657,15 @@ var DashboardStageInner = function DashboardStageInner(_ref3) {
     }
     currentWorkspaceRef.current = null;
     originalWorkspaceRef.current = null;
+    setIsDirty(false);
     setPreviewMode(true);
   }
   function handleToggleEditMode() {
     if (previewMode) {
-      // Entering edit mode — snapshot the current workspace
+      // Entering edit mode — snapshot the current workspace and
+      // reset the dirty flag so we start from a clean slate.
       originalWorkspaceRef.current = deepCopy(workspaceSelected);
+      setIsDirty(false);
       setPreviewMode(false);
       return;
     }
@@ -61665,6 +61687,9 @@ var DashboardStageInner = function DashboardStageInner(_ref3) {
 
     // Update the tab name and workspace reference
     updateTabWorkspace(tempWorkspace);
+    // Header rename counts as a workspace mutation; mark dirty so
+    // navigation + close guards fire.
+    setIsDirty(true);
   }
   function handleWorkspaceFolderChange(menuId) {
     if (!workspaceSelected) return;
@@ -61828,6 +61853,7 @@ var DashboardStageInner = function DashboardStageInner(_ref3) {
     // Clear edit-mode refs — edits are now persisted
     currentWorkspaceRef.current = null;
     originalWorkspaceRef.current = null;
+    setIsDirty(false);
     setPreviewMode(function () {
       return true;
     });
@@ -62344,10 +62370,11 @@ var DashboardStageInner = function DashboardStageInner(_ref3) {
         if (pending.kind === "cancel-edit") {
           performCancelEdit();
         } else if (pending.kind === "open-workspace") {
-          // Clear edit refs before navigating so the new workspace
-          // mount doesn't inherit dirty state.
+          // Clear edit refs + dirty flag before navigating so the
+          // new workspace mount doesn't inherit dirty state.
           currentWorkspaceRef.current = null;
           originalWorkspaceRef.current = null;
+          setIsDirty(false);
           setPreviewMode(true);
           handleOpenTab(pending.workspace);
         }