fix(state): restore view configurations and fix dataID mapping

- Add deserializeAll method to restore view configs from state files
- Fix view dataID mapping to use correct field (was incorrectly using view.id)
- Preserve windowing defaults when applying configuration patches
- Include view config in state file schema

Author: PaulHax

src/io/import/processors/restoreStateFile.ts

@@ -30,6 +30,7 @@ import handleDicomStream from '@/src/io/import/processors/handleDicomStream';
 import downloadStream from '@/src/io/import/processors/downloadStream';
 import { FileEntry } from '@/src/io/types';
 import { useViewStore } from '@/src/store/views';
+import { useViewConfigStore } from '@/src/store/view-configs';
 
 const LABELMAP_PALETTE_2_1_0 = {
   '1': {
@@ -378,6 +379,9 @@ const restoreStateFile: ImportHandler = async (dataSource, context) => {
     // Restore the views
     useViewStore().deserialize(manifest, stateIDToStoreID);
 
+    // Restore view configs
+    useViewConfigStore().deserializeAll(manifest, stateIDToStoreID);
+
     // Restore the labelmaps
     const segmentGroupIDMap = await useSegmentGroupStore().deserialize(
       manifest,

src/io/state-file/schema.ts

@@ -281,6 +281,7 @@ const View = z.object({
   type: z.union([z.literal('2D'), z.literal('3D'), z.literal('Oblique')]),
   dataID: z.string().optional().nullable(),
   options: z.record(z.string()).optional(),
+  config: z.record(ViewConfig).optional(),
 });
 
 export type View = z.infer<typeof View>;

src/store/view-configs.ts

@@ -61,10 +61,22 @@ export const useViewConfigStore = defineStore('viewConfig', () => {
     volumeColoringStore.deserialize(viewID, updatedConfig);
   };
 
+  const deserializeAll = (
+    manifest: StateFile['manifest'],
+    dataIDMap: Record<string, string>
+  ) => {
+    Object.entries(manifest.viewByID).forEach(([viewID, view]) => {
+      if (view.config) {
+        deserialize(viewID, view.config, dataIDMap);
+      }
+    });
+  };
+
   return {
     removeView,
     removeData,
     serialize,
     deserialize,
+    deserializeAll,
   };
 });

src/store/view-configs/common.ts

@@ -13,16 +13,18 @@ const serializeViewConfig = <
   viewConfigStateKey: K
 ) => {
   const dataIDs = stateFile.manifest.datasets.map((dataset) => dataset.id);
-  const { views } = stateFile.manifest;
+  const views = Object.values(stateFile.manifest.viewByID);
 
   views.forEach((view) => {
     dataIDs.forEach((dataID) => {
-      const { config } = view;
-
       const viewConfig = viewConfigs[view.id]?.[dataID];
       if (viewConfig !== undefined) {
-        const configForData = ensureDefault(dataID, config, {} as ViewConfig);
-
+        // Initialize config if it doesn't exist
+        if (!view.config) {
+          // eslint-disable-next-line no-param-reassign
+          view.config = {};
+        }
+        const configForData = ensureDefault(dataID, view.config, {} as ViewConfig);
         configForData[viewConfigStateKey] = viewConfig as ViewConfig[K];
       }
     });

src/store/view-configs/windowing.ts

@@ -131,9 +131,14 @@ export const useWindowingStore = defineStore('windowing', () => {
       }
     }
 
+    // Ensure we always have required fields from defaults
+    const baseConfig = currentInternalConfig || defaults;
+    
     const newInternalConfig = {
+      ...baseConfig,
       ...widthLevelPatchOnSwitchingFromAuto,
       ...patch,
+      auto: patch.auto ?? currentInternalConfig?.auto ?? defaults.auto,
       useAuto: effectiveUseAuto,
       // one way from false to true
       userTriggered:

src/store/views.ts

@@ -213,7 +213,7 @@ export const useViewStore = defineStore('view', () => {
     Object.entries(manifest.viewByID).forEach(([id, view]) => {
       viewByID[id] = {
         ...view,
-        dataID: view.id ? dataIDMap[view.id] : null,
+        dataID: view.dataID ? dataIDMap[view.dataID] : null,
       } as unknown as ViewInfo;
     });
   }