Update SQL console query logic and clean up DynamicSchemaManager migration comments. Removed unnecessary runQueryOnce option and legacy migration code for clarity.

Author: bean1352

tools/diagnostics-app/src/app/views/sql-console.tsx

@@ -40,11 +40,10 @@ function QueryHistoryContent({
 }: QueryHistoryDropdownProps) {
   const hasRestoredRef = React.useRef(false);
 
-  // useQuery reads from localStateDb (nested PowerSyncContext.Provider). runQueryOnce for local-only table.
+  // useQuery reads from localStateDb (nested PowerSyncContext.Provider)
   const { data: historyData, isLoading } = useQuery<QueryHistoryEntry>(
     `SELECT id, query, executed_at FROM query_history ORDER BY executed_at DESC LIMIT ?`,
-    [MAX_HISTORY],
-    { runQueryOnce: true }
+    [MAX_HISTORY]
   );
   const queryHistory = historyData ?? [];
 

tools/diagnostics-app/src/library/powersync/DynamicSchemaManager.ts

@@ -25,7 +25,6 @@ export class DynamicSchemaManager {
 
   /**
    * Load dynamic schema from local DB. Call after localStateDb is initialized (e.g. before connect).
-   * One-time migration: if no row in app_settings, try localStorage (legacy) and persist to DB.
    */
   async loadFromDb(): Promise<void> {
     const rows = await localStateDb.getAll<{ value: string }>(
@@ -36,19 +35,6 @@ export class DynamicSchemaManager {
     if (row?.value) {
       this.tables = JSON.parse(row.value);
       this.dirty = true;
-      return;
-    }
-    // One-time migration from localStorage (schema was stored there before)
-    try {
-      const legacy = typeof localStorage !== 'undefined' && localStorage.getItem('powersync_dynamic_schema');
-      if (legacy) {
-        this.tables = JSON.parse(legacy);
-        this.dirty = true;
-        await this.persistSchema();
-        localStorage.removeItem('powersync_dynamic_schema');
-      }
-    } catch {
-      // ignore
     }
   }