feature: @putout/plugin-variables: remove-unused: ArrayPattern: filter before check

Author: coderaiser

packages/plugin-variables/lib/remove-unused/fixture/array-pattern-fix.js

@@ -5,3 +5,5 @@ const [c] = array;
 console.log(c);
 
 const [] = array;
+
+filter(([, v]) => v);

packages/plugin-variables/lib/remove-unused/fixture/array-pattern.js

@@ -6,3 +6,4 @@ console.log(c);
 
 const [z] = array;
 
+filter(([k, v]) => v);

packages/plugin-variables/lib/remove-unused/get-vars.spec.js

@@ -252,6 +252,10 @@ test('remove-unused-variables: get-vars: array pattern', (t) => {
         z: d_,
         array: _u,
         console: _u,
+        filter: _u,
+    }, {
+        v: du,
+        k: d_,
     }];
 
     t.deepEqual(result, expected);

packages/plugin-variables/lib/remove-unused/get-vars/traverse.js

@@ -118,11 +118,13 @@ export const traverseObjectExpression = (use) => {
     };
 };
 
+const exists = ({node}) => Boolean(node);
+
 export const traverseArrayExpression = (use) => {
     const traverseObjExpression = traverseObjectExpression(use);
 
     return (elementsPaths) => {
-        for (const elementPath of elementsPaths) {
+        for (const elementPath of elementsPaths.filter(exists)) {
             const {node} = elementPath;
 
             if (isIdentifier(node))

packages/plugin-variables/lib/remove-unused/index.spec.js

@@ -11,3 +11,8 @@ test('putout: plugin-variables: remove-unused: report', (t) => {
     t.reportCode('const a = 5', `'a' is defined but never used`);
     t.end();
 });
+
+test('putout: plugin-variables: remove-unused: transform: array-pattern', (t) => {
+    t.transform('array-pattern');
+    t.end();
+});