autoRefetch & enabled is current behaviour right?

[husayt]

I have been struggling to get controlled autoRefetch behavior working(to be able to pause autorefetching), as I was expecting that if enabled is false then autorefetch woudln't have an effect. But apparently this is not the case. So to pause the query it is not enough to set enabled false, but also need to set autoRefetch to false as well. Also, I finally managed to get it working by passing as extra params

Or maybe I am missing something?
Here are some of my failed attempts and finally working configuration.

const latestPriceQuery = defineQueryOptions(
  ({
    enabled,
    symbol = "",
    autoRefetch = false,
  }: {
    enabled: boolean
    symbol?: string
    autoRefetch?: number | boolean
  }) => ({
      key: QUERY_KEYS.price(symbol),
      query: () => $fetch(`/api/assets/${symbol}/last`),
      enabled: !!symbol && enabled,
      autoRefetch,
    }),
)

//doesn't work --- keeps querying even when `state.enabled` becomes false
useQuery(latestCryptoPriceQuery, () => ({
    enabled: state.enabled,
    symbol: state.symbol,
    autoRefetch: state.enabled ? 15000 : false,
  }))

//doesn't work --- keeps querying even when `state.enabled` becomes false
useQuery(() => latestCryptoPriceQuery({
    enabled: state.enabled,
    symbol: state.symbol,
    autoRefetch: state.enabled ? 15000 : false,
  }))

//doesn't work --- keeps querying even when `state.enabled` becomes false
useQuery(() => ({...latestCryptoPriceQuery({
    enabled: state.enabled,
    symbol: state.symbol,
  }),
    autoRefetch: state.enabled ? 15000 : false,
)

//finally works --- fetch pasuses once state.enabled becomes false
useQuery(() => ({...latestCryptoPriceQuery({
   enabled: state.enabled,
   symbol: state.symbol,
 }),
   autoRefetch: ()=>state.enabled ? 15000 : false,
)

Am I missing something or autoRefetch is not supported as dynamic parameter inside defineQueryOptions

Or am I passing dynamic parameters in wrong way?

Stack:
"@pinia/colada-nuxt": "0.2.4",
"@pinia/colada-plugin-auto-refetch": "0.2.4",

[posva]

enabled should block the auto refetch. I thought I had unit tests for this 🤔

[posva]

I verified and there are tests for both things, so something else is off. Maybe try creating a PR with a unit test or a boiled down reproduction ( no nuxt)

[husayt]

also, @posva seems something wrong with my query. It doesn't recognise dynamic parameters, even without autoRefetch. Here state is reactive . state.enabled changes but that doesn't affect the query. Is this not the way to pass dynamic query params? Or is it because enabled is not part of query's key?

useQuery(latestCryptoPriceQuery, () => ({
    enabled: state.enabled,
    symbol: state.symbol,
  }))

[posva]

Remember to add the language to code fences: https://docs.github.com/en/get-started/writing-on-github/working-with-advanced-formatting/creating-and-highlighting-code-blocks#syntax-highlighting