feat: add dynamicOptionsComparer query feature

Author: js2me

.changeset/some-tigers-drive.md

@@ -0,0 +1,5 @@
+---
+"mobx-tanstack-query": minor
+---
+
+added `dynamicOptionsComparer` query feature

package.json

@@ -9,7 +9,7 @@
     "check": "npm run lint:check && npm run ts:check",
     "prebuild": "npm run clean && npm run check",
     "build:watch": "pnpm build && nodemon --watch src --ext ts,tsx --exec \"pnpm build\"",
-    "build": "vite build && tsx scripts/prepare-dist",
+    "build": "vite build",
     "pub": "pnpm build && sborshik publish --useDistDir",
     "pub:patch": "PUBLISH_VERSION=patch pnpm pub",
     "pub:minor": "PUBLISH_VERSION=minor pnpm pub",
@@ -67,7 +67,7 @@
     "lefthook": "^1.11.13",
     "nodemon": "^3.1.0",
     "rimraf": "^6.0.1",
-    "sborshik": "^1.0.25",
+    "sborshik": "^1.0.31",
     "terser": "^5.44.0",
     "tsx": "^4.20.6",
     "typescript": "^5.8.3",

pnpm-lock.yaml

@@ -467,28 +467,17 @@ export class InfiniteQuery<
 
       config.abortSignal?.addEventListener('abort', cleanup);
     } else {
-      if (isQueryKeyDynamic) {
-        reaction(
-          queryKeyOrDynamicQueryKey,
-          (queryKey) => this.update({ queryKey }),
-          {
-            signal: this.abortController.signal,
-            delay: this.config.dynamicOptionsUpdateDelay,
-          },
-        );
-      }
-      if (getDynamicOptions) {
-        reaction(() => getDynamicOptions(this), this.update, {
-          signal: this.abortController.signal,
+      if (getAllDynamicOptions) {
+        reaction(getAllDynamicOptions, this.update, {
           delay: this.config.dynamicOptionsUpdateDelay,
+          signal: config.abortSignal,
+          fireImmediately: true,
+          equals: this.features.dynamicOptionsComparer,
         });
       }
       this._observerSubscription = this.queryObserver.subscribe(
         this.updateResult,
       );
-      this.abortController.signal.addEventListener('abort', () =>
-        this.handleDestroy(),
-      );
     }
 
     if (config.onDone) {

src/inifinite-query.ts

@@ -443,6 +443,7 @@ export class Query<
                 delay: this.features.dynamicOptionsUpdateDelay,
                 signal: config.abortSignal,
                 fireImmediately: true,
+                equals: this.features.dynamicOptionsComparer,
               });
             }
           }
@@ -459,20 +460,12 @@ export class Query<
 
       config.abortSignal?.addEventListener('abort', cleanup);
     } else {
-      if (isQueryKeyDynamic) {
-        reaction(
-          queryKeyOrDynamicQueryKey,
-          (queryKey) => this.update({ queryKey }),
-          {
-            signal: this.abortController.signal,
-            delay: this.features.dynamicOptionsUpdateDelay,
-          },
-        );
-      }
-      if (getDynamicOptions) {
-        reaction(() => getDynamicOptions(this), this.update, {
-          signal: this.abortController.signal,
+      if (getAllDynamicOptions) {
+        reaction(getAllDynamicOptions, this.update, {
           delay: this.features.dynamicOptionsUpdateDelay,
+          signal: config.abortSignal,
+          fireImmediately: true,
+          equals: this.features.dynamicOptionsComparer,
         });
       }
       this._observerSubscription = this.queryObserver.subscribe(

src/query.ts

@@ -6,7 +6,7 @@ import type {
   QueryKey,
   QueryObserverOptions,
 } from '@tanstack/query-core';
-
+import type { IReactionOptions } from 'mobx';
 import type { Query } from './query.js';
 import type { AnyQueryClient } from './query-client.types.js';
 
@@ -173,6 +173,8 @@ export interface QueryFeatures {
    * [**Documentation**](https://js2me.github.io/mobx-tanstack-query/api/Query.html#autoremovepreviousquery-queryfeature)
    */
   autoRemovePreviousQuery?: boolean;
+
+  dynamicOptionsComparer?: IReactionOptions<any, any>['equals'];
 }
 
 /**

src/query.types.ts

@@ -1,35 +1,4 @@
-import { defineConfig } from 'vite';
-import { resolve } from 'path';
-import dts from 'vite-plugin-dts';
-
-import tsconfig from "./tsconfig.json";
-import packageJson from "./package.json";
-import yummiesPackageJson from "./node_modules/yummies/package.json";
-
-const entries = Object.entries(tsconfig.compilerOptions.paths).map(([libName, paths]) => {
-  return {
-    libName,
-    entryName: paths[0].replace('/index.ts', '').replace('.ts', '').replace('./src/', ''),
-    entryPath: resolve(__dirname, paths[0]),
-  }
-});
-
-export default defineConfig({
-  appType: 'spa',
-  build: {
-    minify: 'terser',
-    sourcemap: true,
-    lib: {
-      entry:  Object.assign({}, ...entries.map(entry => ({ [entry.entryName === './src' ? 'index' : entry.entryName]: entry.entryPath }))), 
-      formats: ['es', 'cjs'],
-    },
-    rollupOptions :{
-      external: [...entries.map(entry => entry.libName), ...Object.keys(packageJson.peerDependencies), ...Object.keys(packageJson.dependencies), ...Object.keys(yummiesPackageJson.exports).map(key=> key.replace('./', 'yummies/'))], 
-      output: {
-        chunkFileNames: '~[name]-[hash].js',
-      }
-    }
-  },
-  resolve: {},
-  plugins: [dts()]
-})
+import { defineLibViteConfig } from "sborshik/vite";
+import { ConfigsManager } from "sborshik/utils";
+ 
+export default defineLibViteConfig(ConfigsManager.create()) 

vite.config.ts

@@ -1,42 +1,4 @@
-import { defineConfig } from "vitest/config";
-import react from "@vitejs/plugin-react-swc";
+import { defineLibVitestConfig } from "sborshik/vite";
+import { ConfigsManager } from "sborshik/utils";
 
-import tsconfig from "./tsconfig.json";
-import { resolve } from "path";
-
-const entries = Object.entries(tsconfig.compilerOptions.paths).map(([libName, paths]) => {
-  const entryName = paths[0].replace('/index.ts', '').replace('.ts', '').replace('./src/', '');
-
-  return {
-    libName,
-    entryName: entryName === './src' ? 'index' : entryName,
-    entryPath: resolve(__dirname, paths[0]),
-  }
-})
-
-export default defineConfig({
-  // @ts-ignore
-  plugins: [react()],
-  test: {
-    globals: true,
-    environment: "jsdom",
-    coverage: {
-      provider: 'istanbul', // or 'v8'
-      include: ['src'],
-      exclude: ['src/preset'],
-      reporter: [
-        'text',
-        'text-summary',
-        'html'
-      ],
-      reportsDirectory: './coverage'
-    },
-  },
-  resolve: {
-    alias: Object.assign({}, ...entries.map(entry => {
-      return {
-        [entry.libName]: entry.entryPath,
-      }
-    })),
-  },
-});
+export default defineLibVitestConfig(ConfigsManager.create());