Support `cancel` to `throttle` about es-toolkit

[raon0211]

Hi, thanks for your suggestion!

I’ve thought about adding a cancel feature to throttle, but the way throttle works makes it tricky. Since throttle doesn't actually queue any asynchronous tasks, it’s hard to define what exactly would be canceled. That’s why we haven’t included a cancel option in throttle.

What are your thoughts?

[Hanna922]

I simply thought that the cancel feature would cancel the pending function and make it immediately callable by setting lastCallTime to null. For example, it should be able to pass the following test code.

it('should call the function immediately after cancel is called', async () => {
  const func = vi.fn();
  const throttledFunc = throttle(func, 100);

  throttledFunc();
  expect(func).toHaveBeenCalledTimes(1);

  throttledFunc.cancel();
  throttledFunc();
  expect(func).toHaveBeenCalledTimes(2);
});

How do you feel about providing this kind of cancel?

[raon0211]

As you mentioned, providing those kinds of cancel options might be helpful.

However, I'm not sure if it is really that beneficial. For debounce, we have to cancel the debounced function when the component is cleaned up, but for throttle, there's no need to do so.

Additionally, since we can only abort AbortControllers once, it might not align well with the definition of cancel for throttle.

So, because it is not a major use case, I think we might not support cancel for throttle in es-toolkit. What do you think?

[Hanna922]

Yes, I agree that it is not a major use case. I'll close this issue. Thank you for opinion 🙇🏻‍♀️