arguments get reset if you call debounced function within itself

[archive64]

Summary

👋 Hi there! I've noticed that calling a debounced function from itself sets its arguments to undefined.

Minimal working example

debounce = require('debounce');

const func = debounce((x) => {
  console.log(x);
  func(2);
}, 1000);

func(1);

Actual output

1
undefined
undefined
undefined
// ...

Expected output

1
2
2
2
// ...

Workaround

As a temporary workaround, I can defer the call:

  setTimeout(() => func(2));

Cause

https://github.com/component/debounce/blob/master/index.js#L27-L28

result = func.apply(context, args); // calls debounced function, which sets args
context = args = null; // args gets reset here

[aaronbeall]

I just ran into (as the programming gods would have it, a couple hours chasing red herrings until I finally found this issue). You can reproduce this on NPM RunKit:

const debounce = require("debounce")

const fn = debounce(n => {
  console.log("fn", n)
  --n && fn(n);
});

fn(3)

Expect:

3
2
1

Actual:

3
undefined