preserve compatibility. Hooks should not report isLoading if the query changed.

Author: stevensJourney

packages/common/src/client/watched/processors/AbstractQueryProcessor.ts

@@ -85,10 +85,9 @@ export abstract class AbstractQueryProcessor<
   async updateSettings(settings: Settings) {
     await this.initialized;
 
-    if (!this.state.isLoading) {
+    if (!this.state.isFetching && this.reportFetching) {
       await this.updateState({
-        isLoading: true,
-        isFetching: this.reportFetching ? true : false
+        isFetching: true
       });
     }
 

packages/web/tests/watch.test.ts

@@ -521,11 +521,12 @@ describe('Watch Tests', { sequential: true }, () => {
       })
     });
 
-    expect(watch.state.isLoading).true;
+    expect(watch.state.isLoading).false;
+    expect(watch.state.isFetching).true;
 
     await vi.waitFor(
       () => {
-        expect(watch.state.isLoading).false;
+        expect(watch.state.isFetching).false;
       },
       { timeout: 1000 }
     );