Fix DuckDB migration ordering after archive-and-reset

Lite/Database/DuckDbInitializer.cs

@@ -101,20 +101,20 @@
     /// </summary>
     public async Task InitializeAsync()
     {
         _logger?.LogInformation("Initializing DuckDB database at {Path}", _databasePath);
 
         var directory = Path.GetDirectoryName(_databasePath);
         if (!string.IsNullOrEmpty(directory) && !Directory.Exists(directory))
         {
             Directory.CreateDirectory(directory);
             _logger?.LogInformation("Created database directory: {Directory}", directory);
         }
 
         var archivePath = Path.Combine(directory ?? ".", "archive");
         if (!Directory.Exists(archivePath))
         {
             Directory.CreateDirectory(archivePath);
             _logger?.LogInformation("Created archive directory: {ArchivePath}", archivePath);
         }
 
         /* Try to open the database. If the DuckDB storage version has changed,
@@ -141,9 +141,15 @@
 
             var existingVersion = await GetSchemaVersionAsync(connection);
 
-            /* Create tables first (IF NOT EXISTS) so migrations can ALTER them safely.
-               After an archive-and-reset the database is empty — running ALTER TABLE
-               before CREATE TABLE would fail on nonexistent tables. */
+            /* On a fresh/reset database (v0), skip migrations entirely — they DROP tables
+               expecting CREATE TABLE to follow, which is destructive on a blank DB.
+               Just create tables with the current schema and stamp the version. */
+            if (existingVersion > 0 && existingVersion < CurrentSchemaVersion)
+            {
+                _logger?.LogInformation("Schema upgrade needed: v{Old} -> v{New}", existingVersion, CurrentSchemaVersion);
+                await RunMigrationsAsync(connection, existingVersion);
+            }
+
             foreach (var tableStatement in Schema.GetAllTableStatements())
             {
                 await ExecuteNonQueryAsync(connection, tableStatement);
@@ -156,12 +162,10 @@
 
             if (existingVersion < CurrentSchemaVersion)
             {
-                _logger?.LogInformation("Schema upgrade needed: v{Old} -> v{New}", existingVersion, CurrentSchemaVersion);
-                await RunMigrationsAsync(connection, existingVersion);
                 await SetSchemaVersionAsync(connection, CurrentSchemaVersion);
             }
 
             _logger?.LogInformation("Database initialization complete. Schema version: {Version}", CurrentSchemaVersion);
         }
 
         await CreateArchiveViewsAsync();
@@ -214,7 +218,7 @@
                 cmd.CommandText = $"EXPORT DATABASE '{exportDir.Replace("'", "''")}' (FORMAT PARQUET)";
                 await cmd.ExecuteNonQueryAsync();
                 exported = true;
                 _logger?.LogInformation("Exported old database to {ExportDir}", exportDir);
             }
         }
         catch (Exception ex)
@@ -228,7 +232,7 @@
         {
             /* DuckDB may have .wal files too */
             File.Move(_databasePath, backupPath);
             _logger?.LogInformation("Backed up old database to {BackupPath}", backupPath);
 
             var walPath = _databasePath + ".wal";
             if (File.Exists(walPath))
@@ -522,12 +526,12 @@
                 }
 
                 if (serverNames.Count > 0)
                     _logger?.LogInformation("Fixed server_id in {Table} for {Count} server(s)", table, serverNames.Count);
             }
             catch (Exception ex)
             {
                 /* Table might not exist yet — that's fine, it will be created with correct IDs */
                 _logger?.LogDebug(ex, "Skipped server_id fix for {Table} (may not exist yet)", table);
             }
         }
     }
@@ -589,7 +593,7 @@
             }
         }
 
         _logger?.LogDebug("Archive views created/refreshed for {Count} tables", ArchivableTables.Length);
     }
 
     /// <summary>