fix(views): keep active view selected after data binding

Tools and rendering panels resolve currentImageID through activeView and viewByID.

Select the first startup view directly, then enforce a visible active view when data binds.

Session restore reuses that binding path when manifest data IDs are rebound.

Added store coverage for startup, normal data load, and session rebinding.

Verified with focused Vitest, lint, and full Chrome e2e.

Author: PaulHax

src/store/__tests__/views.spec.ts

@@ -0,0 +1,58 @@
+import { describe, it, beforeEach, expect } from 'vitest';
+import { setActivePinia, createPinia } from 'pinia';
+import { useViewStore } from '@/src/store/views';
+import type { Manifest } from '@/src/io/state-file/schema';
+
+describe('View store', () => {
+  beforeEach(() => {
+    setActivePinia(createPinia());
+  });
+
+  it('selects the first initial visible view', () => {
+    const store = useViewStore();
+
+    expect(store.activeView).toBe(store.visibleViews[0].id);
+  });
+
+  it('selects a visible view when data is attached to the default layout', () => {
+    const store = useViewStore();
+
+    store.setActiveView(null);
+    store.setDataForAllViews('image-1');
+
+    expect(store.activeView).toBe(store.visibleViews[0].id);
+  });
+
+  it('selects a visible view when session data IDs are rebound', () => {
+    const store = useViewStore();
+    const manifest: Manifest = {
+      version: '6.1.0',
+      dataSources: [],
+      activeView: null,
+      layout: {
+        direction: 'column',
+        items: [{ type: 'slot', slotIndex: 0 }],
+      },
+      layoutSlots: ['view-1'],
+      viewByID: {
+        'view-1': {
+          id: 'view-1',
+          type: '2D',
+          dataID: 'state-image',
+          name: 'Axial',
+          options: {
+            orientation: 'Axial',
+          },
+        },
+      },
+    };
+
+    store.deserializeLayout(manifest);
+    expect(store.activeView).toBeNull();
+
+    store.bindViewsToData('state-image', 'loaded-image', manifest);
+
+    expect(store.activeView).toBe('view-1');
+    expect(store.viewByID['view-1'].dataID).toBe('loaded-image');
+  });
+});

src/store/views.ts

@@ -147,13 +147,14 @@ export const useViewStore = defineStore('view', () => {
   }
 
   function ensureActiveViewIsVisible() {
-    if (!visibleViews.value.length) {
+    const views = visibleViews.value;
+    if (!views.length) {
       setActiveView(null);
       return;
     }
 
-    if (!visibleViews.value.find((view) => view.id === activeView.value)) {
-      setActiveView(visibleViews.value[0].id);
+    if (!views.find((view) => view.id === activeView.value)) {
+      setActiveView(views[0].id);
     }
   }
 
@@ -263,6 +264,8 @@ export const useViewStore = defineStore('view', () => {
     if (!(viewID in viewByID)) return;
     viewByID[viewID].dataID = dataID;
     ViewDataChangeEvent.trigger(viewID, dataID);
+    // Global tools resolve their image through activeView.
+    ensureActiveViewIsVisible();
   }
 
   function setDataForActiveView(dataID: Maybe<string>) {
@@ -342,16 +345,19 @@ export const useViewStore = defineStore('view', () => {
 
     Object.entries(manifest.viewByID).forEach(([id, view]) => {
       if (view.dataID === stateID && viewByID[id]) {
-        viewByID[id].dataID = storeID;
-        ViewDataChangeEvent.trigger(id, storeID);
+        setDataForView(id, storeID);
       }
     });
   }
 
   // initialization
 
   firstLayout.views.forEach((viewInit) => {
-    layoutSlots.value.push(addView(viewInit));
+    const viewId = addView(viewInit);
+    layoutSlots.value.push(viewId);
+    if (!activeView.value) {
+      setActiveView(viewId);
+    }
   });
 
   watch(disabledViewTypes, () => {