feature: variables: remove-unused: ArrayPattern inside ArrayPattern: add support

Author: coderaiser

packages/engine-runner/lib/progress.spec.js

@@ -16,10 +16,7 @@ test('putout: runner: progress: start', async (t) => {
         ]);
     `;
 
-    const [[{
-        pluginsIndex,
-        pluginsCount,
-    }]] = await Promise.all([
+    const [[{pluginsCount}]] = await Promise.all([
         once(progress, 'start'),
         putout(source, {
             progress,
@@ -45,10 +42,7 @@ test('putout: runner: progress: end', async (t) => {
         ]);
     `;
 
-    const [[{
-        pluginsIndex,
-        pluginsCount,
-    }]] = await Promise.all([
+    const [[{pluginsCount}]] = await Promise.all([
         once(progress, 'end'),
         putout(source, {
             progress,
@@ -74,10 +68,7 @@ test('putout: runner: progress: push', async (t) => {
         ]);
     `;
 
-    const [[{
-        pluginsIndex,
-        pluginsCount,
-    }]] = await Promise.all([
+    const [[{pluginsCount}]] = await Promise.all([
         once(progress, 'push'),
         putout(source, {
             runPlugins,

packages/operate/lib/properties/traverse-properties.spec.js

@@ -203,10 +203,7 @@ test('operate: traverse-properties: filesystem', async (t) => {
         ]);
     `;
 
-    const [[{
-        pluginsIndex,
-        pluginsCount,
-    }]] = await Promise.all([
+    const [[{pluginsIndex}]] = await Promise.all([
         once(progress, 'push'),
         putout(source, {
             progress,

packages/plugin-variables/lib/remove-unused/fixture/destr-nested-vars.js

@@ -14,3 +14,7 @@ const defaultProcessors = [];
 const [error, {
     processors = defaultProcessors,
 }] = getOptions();
+
+const [[{
+    pluginsCount,
+}]] = getOptions();

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

@@ -737,6 +737,7 @@ test('remove-unused-variables: get-vars: destr nested vars', (t) => {
         processors: d_,
         obj: du,
         world: d_,
+        pluginsCount: d_,
     }];
 
     t.deepEqual(result, expected);

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

@@ -40,6 +40,7 @@ export default ({use, declare, addParams}) => {
 
     const traverseTmpl = traverseTemplateLiteral(use);
     const traverseArray = traverseArrayExpression(use);
+    const declareArray = traverseArrayExpression(declare);
 
     return {
         'ObjectExpression'(path) {
@@ -144,6 +145,11 @@ export default ({use, declare, addParams}) => {
                         continue;
                     }
 
+                    if (elPath.isArrayPattern()) {
+                        declareArray(elPath.get('elements'));
+                        continue;
+                    }
+                    
                     if (elPath.isAssignmentPattern()) {
                         const leftPath = elPath.get('left');
                         const rightPath = elPath.get('right');

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

@@ -125,7 +125,7 @@ export const traverseArrayExpression = (use) => {
         for (const elementPath of elementsPaths) {
             const {node} = elementPath;
 
-            if (isObjectExpression(node))
+            if (node.properties)
                 traverseObjExpression(elementPath.get('properties'));
         }
     };

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

@@ -1,5 +1,6 @@
 import test from 'supertape';
 import _putout from 'putout';
+import montag from 'montag';
 import {readFixtures} from './fixture.js';
 import * as removeUnusedVariables from './index.js';
 
@@ -110,7 +111,12 @@ test('remove-unused-variables: putout: destr vars', (t) => {
 
 test('remove-unused-variables: putout: destr nested vars', (t) => {
     const {code} = putout(fixture.destrNestedVars);
-    const expected = 'const [, {}] = getOptions();\n';
+    const expected = montag`
+        const [, {}] = getOptions();
+        
+        const [[{}]] = getOptions();
+   
+   `;
 
     t.equal(code, expected);
     t.end();

packages/plugin-variables/package.json

@@ -49,6 +49,7 @@
     "eslint-plugin-putout": "^29.0.0",
     "just-camel-case": "^6.2.0",
     "madrun": "^11.0.0",
+    "montag": "^1.2.1",
     "nodemon": "^3.0.1",
     "supertape": "^11.3.1",
     "try-catch": "^3.0.1"

packages/putout/test/putout.js

@@ -829,10 +829,7 @@ test('putout: progress', async (t) => {
         ]);
     `;
 
-    const [[{
-        pluginsIndex,
-        pluginsCount,
-    }]] = await Promise.all([
+    const [[{pluginsCount}]] = await Promise.all([
         once(progress, 'push'),
         putout(source, {
             progress,