fix: fix once with arguments (#30)

* fix: fix `once` with args

* chore: changeset

* chore: pre

.changeset/cool-socks-kick.md

@@ -0,0 +1,5 @@
+---
+"@hiogawa/utils": patch
+---
+
+fix: fix `once` with arguments

packages/utils/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hiogawa/utils",
-  "version": "1.4.2-pre.4",
+  "version": "1.4.2-pre.5",
   "type": "module",
   "main": "./dist/index.js",
   "module": "./dist/index.js",

packages/utils/src/lodash.test.ts

@@ -342,4 +342,15 @@ describe("once", () => {
     expect(f()).toMatchInlineSnapshot("1");
     expect(count).toMatchInlineSnapshot("1");
   });
+
+  it("args", () => {
+    let count = 0;
+    const f = once((increment: number) => (count = count + increment));
+
+    expect(count).toMatchInlineSnapshot("0");
+    expect(f(2)).toMatchInlineSnapshot("2");
+    expect(count).toMatchInlineSnapshot("2");
+    expect(f(4)).toMatchInlineSnapshot("2");
+    expect(count).toMatchInlineSnapshot("2");
+  });
 });

packages/utils/src/lodash.ts

@@ -78,13 +78,13 @@ export function isNotNil<T>(value: T): value is NonNullable<T> {
 export function once<F extends (...args: any[]) => any>(f: F): F {
   let result: unknown;
   let called = false;
-  const wrapper = () => {
+  function wrapper(...args: any[]) {
     if (!called) {
-      result = f(...arguments);
+      result = f(...args);
       called = true;
     }
     return result;
-  };
+  }
   return wrapper as F;
 }