fix(restoreStateFile): add migration for v5 to 6 volview.zip json

Author: PaulHax

src/io/import/processors/restoreStateFile.ts

@@ -146,12 +146,127 @@ const migrate210To300 = (inputManifest: any) => {
   return manifest;
 };
 
+const migrate501To600 = (inputManifest: any) => {
+  const manifest = JSON.parse(JSON.stringify(inputManifest));
+  
+  // Convert views array to viewByID object
+  if (manifest.views && Array.isArray(manifest.views)) {
+    manifest.viewByID = {};
+    manifest.views.forEach((view: any) => {
+      const migratedView = { ...view };
+      
+      // Add required 'name' field if missing
+      if (!migratedView.name) {
+        migratedView.name = migratedView.id;
+      }
+      
+      // Convert 'props' to 'options' if present
+      if (migratedView.props) {
+        // Convert any non-string values in props to strings for options
+        migratedView.options = {};
+        Object.entries(migratedView.props).forEach(([key, value]) => {
+          if (typeof value === 'string') {
+            migratedView.options[key] = value;
+          } else {
+            // Convert arrays and objects to JSON strings
+            migratedView.options[key] = JSON.stringify(value);
+          }
+        });
+        delete migratedView.props;
+      }
+      
+      // Add orientation for 2D views based on the view ID
+      if (migratedView.type === '2D' && !migratedView.options) {
+        migratedView.options = {};
+      }
+      if (migratedView.type === '2D') {
+        // Set orientation based on view ID (Coronal, Sagittal, Axial)
+        if (['Coronal', 'Sagittal', 'Axial'].includes(migratedView.id)) {
+          migratedView.options.orientation = migratedView.id;
+        }
+      }
+      
+      // Handle type conversion for Oblique views
+      if (migratedView.type === 'Oblique3D') {
+        migratedView.type = 'Oblique';
+      }
+      
+      // Set dataID based on whether the view has config with data
+      // In 5.0.1, if a view has config entries, it means it's associated with that data
+      if (migratedView.config && Object.keys(migratedView.config).length > 0) {
+        // Use the first (and typically only) key from config as the dataID
+        migratedView.dataID = Object.keys(migratedView.config)[0];
+      } else {
+        migratedView.dataID = null;
+      }
+      
+      manifest.viewByID[migratedView.id] = migratedView;
+    });
+    delete manifest.views;
+  }
+  
+  // Add missing fields with proper defaults
+  if (manifest.isActiveViewMaximized === undefined) {
+    manifest.isActiveViewMaximized = false;
+  }
+  
+  if (manifest.activeView === undefined) {
+    manifest.activeView = null;
+  }
+  
+  // Convert layout to layoutSlots and update layout structure
+  if (manifest.layout && !manifest.layoutSlots) {
+    const slots: string[] = [];
+    
+    // Extract all slot names and convert layout to new format
+    const convertLayoutItem = (item: any): any => {
+      if (typeof item === 'string') {
+        // This is a view name like "Coronal", "3D", etc.
+        const slotIndex = slots.length;
+        slots.push(item);
+        return {
+          type: 'slot',
+          slotIndex
+        };
+      }
+      if (item.direction && item.items) {
+        // This is a nested layout
+        return {
+          type: 'layout',
+          direction: item.direction,
+          items: item.items.map(convertLayoutItem)
+        };
+      }
+      return item;
+    };
+    
+    // Convert the root layout
+    if (manifest.layout.direction && manifest.layout.items) {
+      manifest.layout = {
+        direction: manifest.layout.direction,
+        items: manifest.layout.items.map(convertLayoutItem)
+      };
+    }
+    
+    manifest.layoutSlots = slots;
+  }
+  
+  // Ensure parentToLayers exists as an array
+  if (!manifest.parentToLayers) {
+    manifest.parentToLayers = [];
+  }
+  
+  manifest.version = '6.0.0';
+  return manifest;
+};
+
 const migrateManifest = (manifestString: string) => {
   const inputManifest = JSON.parse(manifestString);
   return pipe(
     inputManifest,
     migrateOrPass(['1.1.0', '1.0.0', '0.5.0'], migrateBefore210),
-    migrateOrPass(['2.1.0'], migrate210To300)
+    migrateOrPass(['2.1.0'], migrate210To300),
+    migrateOrPass(['5.0.1'], migrate501To600)
   );
 };
 

src/io/state-file/index.ts

@@ -12,7 +12,7 @@ import { ARCHIVE_FILE_TYPES } from '@/src/io/mimeTypes';
 import { useViewConfigStore } from '@/src/store/view-configs';
 
 export const MANIFEST = 'manifest.json';
-export const MANIFEST_VERSION = '5.0.1';
+export const MANIFEST_VERSION = '6.0.0';
 
 export async function serialize() {
   const datasetStore = useDatasetStore();