Enable noUnusedImports and noUnusedVariables in Biome

Author: wojtekmaj

biome.json

@@ -12,6 +12,10 @@
       "complexity": {
         "noUselessSwitchCase": "off"
       },
+      "correctness": {
+        "noUnusedImports": "warn",
+        "noUnusedVariables": "warn"
+      },
       "suspicious": {
         "noConsoleLog": "warn"
       }

src/filter.ts

@@ -27,7 +27,7 @@ function asyncFilter<T>(
           })
           .catch(reject);
       }),
-  ).then(() => result.filter((cur, idx) => idx in result));
+  ).then(() => result.filter((_cur, idx) => idx in result));
 }
 
 export default asyncFilter;

src/filter_strict.ts

@@ -27,7 +27,7 @@ function asyncFilterStrict<T>(
           })
           .catch(reject);
       }),
-  ).then(() => result.filter((cur, idx) => idx in result));
+  ).then(() => result.filter((_cur, idx) => idx in result));
 }
 
 export default asyncFilterStrict;

src/forEach.spec.ts

@@ -43,7 +43,7 @@ describe('asyncForEach()', () => {
   });
 
   it.skip('assertions below are valid for synchronous .forEach()', () => {
-    const [arr, pushDuplicate] = makePushDuplicate();
+    const [_arr, pushDuplicate] = makePushDuplicate();
     const mapper = vi.fn().mockImplementation(pushDuplicate);
 
     inputArr.forEach(mapper);
@@ -57,7 +57,7 @@ describe('asyncForEach()', () => {
   });
 
   it('iterates through an array properly', async () => {
-    const [arr, pushDuplicate] = makePushDuplicateInRandomTime();
+    const [_arr, pushDuplicate] = makePushDuplicateInRandomTime();
     const mapper = vi.fn().mockImplementation(pushDuplicate);
 
     await asyncForEach(inputArr, mapper);

src/forEach_strict.spec.ts

@@ -47,7 +47,7 @@ describe('asyncForEachStrict()', () => {
   });
 
   it.skip('assertions below are valid for synchronous .forEach()', () => {
-    const [arr, pushDuplicate] = makePushDuplicate();
+    const [_arr, pushDuplicate] = makePushDuplicate();
     const mapper = vi.fn().mockImplementation(pushDuplicate);
 
     inputArr.forEach(mapper);
@@ -61,7 +61,7 @@ describe('asyncForEachStrict()', () => {
   });
 
   it('iterates through an array properly', async () => {
-    const [arr, pushDuplicate] = makePushDuplicateInRandomTime();
+    const [_arr, pushDuplicate] = makePushDuplicateInRandomTime();
     const mapper = vi.fn().mockImplementation(pushDuplicate);
 
     await asyncForEachStrict(inputArr, mapper);

src/reduce.spec.ts

@@ -60,14 +60,14 @@ describe('asyncReduce()', () => {
   });
 
   it('returns type string given function that returns Promise<string> and no initial value', async () => {
-    const result = await asyncReduce(['a', 'b', 'c'], async (temp, cur) => cur);
+    const result = await asyncReduce(['a', 'b', 'c'], async (_temp, cur) => cur);
 
     assertType<string>(result);
 
     expect(result).toBe('c');
 
     // Sanity check
-    const resultSync = ['a', 'b', 'c'].reduce((temp, cur) => cur);
+    const resultSync = ['a', 'b', 'c'].reduce((_temp, cur) => cur);
 
     assertType<string>(resultSync);
 
@@ -99,7 +99,7 @@ describe('asyncReduce()', () => {
   it('returns type number[] given function that returns Promise<number> and initial value of type number', async () => {
     const result = await asyncReduce(
       ['a', 'b', 'c'],
-      async (temp, cur, idx) => [...temp, idx],
+      async (temp, _cur, idx) => [...temp, idx],
       [0],
     );
 
@@ -109,7 +109,7 @@ describe('asyncReduce()', () => {
 
     // Sanity check
     const resultSync = ['a', 'b', 'c'].reduce(
-      (temp, cur, idx) => {
+      (temp, _cur, idx) => {
         temp.push(idx);
         return temp;
       },