Make conservative run and conservative batchRun consistent with eacho…

…ther

mathesar_ui/src/pages/import/preview/ImportPreviewContent.svelte

@@ -227,7 +227,7 @@
           on:retry={init}
           on:delete={cancel}
         />
-      {:else if !$previewRequest.hasInitialized}
+      {:else if !$previewRequest.hasSettled}
         <div class="loading"><Spinner /></div>
       {:else if $previewRequest.error}
         <ErrorInfo

mathesar_ui/src/stores/AsyncRpcApiStore.ts

@@ -81,7 +81,9 @@ export default class AsyncRpcApiStore<Props, T, U = T> extends AsyncStore<
    * that have already been initialized.
    */
   static async runBatchConservatively(runners: BatchRunner[]) {
-    const toRun = runners.filter((runner) => !runner.getValue().hasInitialized);
+    const toRun = runners.filter(
+      (runner) => runner.getValue().isIdleAndUnsettled,
+    );
     await AsyncRpcApiStore.runBatch(toRun);
   }
 }

mathesar_ui/src/stores/AsyncStore.ts

@@ -94,9 +94,13 @@ export class AsyncStoreValue<T, E> {
     return !this.isLoading && this.settlement?.state === 'rejected';
   }
 
-  get hasInitialized(): boolean {
+  get hasSettled(): boolean {
     return this.settlement !== undefined;
   }
+
+  get isIdleAndUnsettled(): boolean {
+    return !this.hasSettled && !this.isLoading;
+  }
 }
 
 export default class AsyncStore<Props = void, T = unknown>
@@ -152,18 +156,12 @@ export default class AsyncStore<Props = void, T = unknown>
     }
   }
 
-  /**
-   * Only runs if the current value is one of
-   * - not initialized
-   * - rejected
-   * - not loading
-   */
   async runConservatively(props: Props): Promise<AsyncStoreValue<T, string>> {
     const value = get(this.value);
-    if (value.isLoading || value.isOk) {
-      return value;
+    if (value.isIdleAndUnsettled) {
+      return this.run(props);
     }
-    return this.run(props);
+    return value;
   }
 
   /**