Cannot stop polling by setting refreshInterval 0

SWR version 0.3.1

I have a demo reproducing this issue here.

In this demo I have a flag variable controlling whether swr should poll by conditionally setting refreshInterval value.

{
  refreshInterval: shouldPoll ? 1000 : 0
}

In my swr config I also have an onSuccess callback in which I change my flag value

{
  refreshInterval: shouldPoll ? 1000 : 0,
  onSuccess() {
    setShouldPoll(false);
  }
}

When the flag is set to false as the first fetch finishes, the polling stops normally as expected. Everything is fine so far.

However, if I set the flag value in the callback not of the first fetch but the revalidation fetches, polling is not stopping.

{
  refreshInterval: intValue < 2 ? 1000 : 0, // intValue is initially 0
  onSuccess() {
    setIntValue(value => value + 1); 
  }
}

After the first fetch, intValue is set to 1, and polling is supposed to continue. After the 2nd fetch, intValue is set to 2, and polling is supposed to stop because refreshInterval is now 0. But the polling continued.

[SevenOutman]

I think it’s the naming of onSuccess options that is tricky. I understand it as ‘when fetches finish’, while it’s actually not.

A workaround can be moving my ‘when fetches finish’ logic from onSuccess into a customer fetcher.

async () => {
  await generalFetcher();
  setIntValue(value => value + 1); 
},
{
  refreshInterval: intValue < 2 ? 1000 : 0, // intValue is initially 0
  // onSuccess() {
  //   setIntValue(value => value + 1); 
  // }
}

[promer94]

A simplified revalidation steps

1. revalidate start
2. newData = await CONCURRENT_PROMISES[key]
   the fetcher has finished and we have newData
3. eventsRef.current.emit('onSuccess', newData, key, config) onSucuess called
4. newData is used to update stateRef.
5. revalidate is over

So onSuccess is called after we successfully get the newData.