Restore graph preset after failed transition

Author: Coding-Dev-Tools

engraphis/dashboard_assets/ledger.js

@@ -18,6 +18,7 @@
     graphWorkspace: '',
     graphData: null,
     graphDataMode: 'overview',
+    graphDataPreset: 'galaxy',
     graphDataIncludeCode: false,
     graphDataShowUnlinked: false,
     graphDataAsOf: null,
@@ -1142,6 +1143,7 @@
     state.workspace = name;
     state.graphWorkspace = '';
     state.graphData = null;
+    state.graphDataPreset = 'galaxy';
     state.graphDataIncludeCode = false;
     state.graphDataShowUnlinked = false;
     state.graphDataRepo = '';
@@ -3490,6 +3492,7 @@
         state.graphData = data;
         state.graphWorkspace = targetWorkspace;
         state.graphDataMode = targetMode;
+        state.graphDataPreset = byId('graph-preset').value;
         state.graphDataIncludeCode = responseIncludeCode;
         state.graphDataShowUnlinked = targetShowUnlinked;
         state.graphDataAsOf = targetAsOf;
@@ -3531,9 +3534,14 @@
           : fullGraph && (error.status === 413 || error.code === 'GRAPH_CAPACITY')
             ? `All nodes exceed the server capacity. Enter an exact repository filter or reduce the workspace graph. (${error.message})`
           : `Graph unavailable: ${error.message}. Choose Reload data to try again.`;
-        if (state.graphData && state.graphDataMode !== targetMode) {
-          state.graphMode = state.graphDataMode;
+        if (state.graphData) {
+          if (state.graphDataMode !== targetMode) state.graphMode = state.graphDataMode;
+          /* The toolbar preset changes before a replacement request begins. Restore the
+             committed preset together with the committed renderer so a failed Every-node
+             transition cannot leave aria-pressed and the active engine disagreeing. */
+          byId('graph-preset').value = state.graphDataPreset || 'galaxy';
           updateGraphModeControls();
+          syncGraphChoices();
         }
         // Restore the committed renderer's freeze/overlay state. The old engine survived
         // because we never mutated state.graphEngine on the failure path.

tests/e2e/ledger.spec.js

@@ -483,7 +483,7 @@ test('Ledger enters Every node from a loaded overview without losing its scope',
   expect(allAssetRequests).toHaveLength(1);
 });
 
-test('Ledger keeps authored Galaxy coordinates in the Every-node renderer', async ({ page }) => {
+test('Ledger keeps authored Galaxy coordinates on the orbit renderer in Every-node view', async ({ page }) => {
   await mockApi(page, {
     graphScene: {
       nodes: [
@@ -518,7 +518,9 @@ test('Ledger keeps authored Galaxy coordinates in the Every-node renderer', asyn
 
   await page.locator('[data-graph-preset-choice="every"]').click();
   await expect(page.locator('#graph-canvas')).toHaveAttribute('aria-busy', 'false');
-  await expect(page.locator('.engraphis-all-canvas')).toHaveCount(1);
+  // Authored Galaxy scenes intentionally stay on the quality/orbit renderer; the dedicated
+  // Every-node WebGL canvas is reserved for non-Galaxy complete scenes.
+  await expect(page.locator('.engraphis-all-canvas')).toHaveCount(0);
   await expect(page.locator('.graph-spacetime-overlay')).toHaveCount(1);
 });