feature: @putout/variables: apply-declarations-order: add

Author: coderaiser

packages/plugin-destructuring/README.md

@@ -19,7 +19,6 @@ npm i @putout/plugin-destructuring
 
 - ✅ [apply-array](#apply-array);
 - ✅ [apply-object](#apply-object);
-- ✅ [apply-declarations-order](#apply-declarations-order);
 - ✅ [convert-object-to-array](#convert-object-to-array);
 - ✅ [extract-properties](#extract-properties);
 - ✅ [remove-useless-object](#remove-useless-object);
@@ -36,7 +35,6 @@ npm i @putout/plugin-destructuring
     "rules": {
         "destructuring/apply-array": "on",
         "destructuring/apply-object": "on",
-        "destructuring/apply-declarations-order": "on",
         "destructuring/convert-object-to-array": "on",
         "destructuring/extract-properties": "on",
         "destructuring/remove-useless-object": "on",
@@ -81,24 +79,6 @@ const {name} = user;
 ({hello} = world);
 ```
 
-## apply-declarations-order
-
-Helps to [extract-properties](#extract-properties'). Checkout in 🐊[**Putout Editor**](https://putout.vercel.app/#/gist/b70ff926b36e1e97ec7129aa0e0458a7/ece0a706de2fd24a66b4671284f7f75017f3c268).
-
-### ❌ Example of incorrect code
-
-```js
-const {env} = require('node:process');
-const process = require('node:process');
-```
-
-### ✅ Example of correct code
-
-```js
-const process = require('node:process');
-const {env} = process;
-```
-
 ## remove-useless-object
 
 Check out in 🐊[**Putout Editor**](https://putout.cloudcmd.io/#/gist/c9ed04b421d75ae39e58038fa6e14630/4c097e3173990ec7e5ebabbe2cedf8e952092ebf).

packages/plugin-destructuring/lib/apply-declarations-order/index.spec.js

@@ -1,4 +1,3 @@
-import * as applyDeclarationsOrder from './apply-declarations-order/index.js';
 import * as applyObject from './apply-object/index.js';
 import * as applyArray from './apply-array/index.js';
 import * as extractPropertiesEqualDeep from './extract-properties-equal-deep/index.js';
@@ -14,7 +13,6 @@ import * as mergeProperties from './merge-properties/index.js';
 export const rules = {
     'apply-array': applyArray,
     'apply-object': applyObject,
-    'apply-declarations-order': applyDeclarationsOrder,
     'convert-object-to-array': convertObjectToArray,
     'extract-properties-equal-deep': extractPropertiesEqualDeep,
     'extract-properties-not-equal-deep': extractPropertiesNotEqualDeep,

packages/plugin-destructuring/lib/index.js

@@ -17,6 +17,7 @@ npm i @putout/plugin-variables -D
 
 ## Rules
 
+- ✅ [apply-declarations-order](#apply-declarations-order);
 - ✅ [convert-const-to-let](#convert-const-to-let');
 - ✅ [extract-keywords](#extract-keywords');
 - ✅ [remove-useless-assignment](#remove-useless-assignmenn);
@@ -32,6 +33,7 @@ npm i @putout/plugin-variables -D
 ```json
 {
     "rules": {
+        "variables/apply-declarations-order": "on",
         "variables/convert-const-to-let": "on",
         "variables/extract-keywords": "on",
         "variables/remove-useless-assignment": "on",
@@ -47,6 +49,24 @@ npm i @putout/plugin-variables -D
 }
 ```
 
+## apply-declarations-order
+
+Helps to [reuse duplicate init](#reuse-duplicate-init'). Checkout in 🐊[**Putout Editor**](https://putout.vercel.app/#/gist/b70ff926b36e1e97ec7129aa0e0458a7/ece0a706de2fd24a66b4671284f7f75017f3c268).
+
+### ❌ Example of incorrect code
+
+```js
+const {env} = require('node:process');
+const process = require('node:process');
+```
+
+### ✅ Example of correct code
+
+```js
+const process = require('node:process');
+const {env} = process;
+```
+
 ## assignment
 
 Checkout in 🐊[**Putout Editor**](https://putout.cloudcmd.io/#/gist/a2f7fe5e2c294443576f95dce6fde67e/0699ccb4f7335e8e3f80de891913a8e3ad4f35e3).

packages/plugin-destructuring/test/fixture/apply-declarations-order-fix.js

@@ -0,0 +1,6 @@
+const {declarations} = 5;
+
+if (parentPath.isExportDeclaration())
+    return;
+
+const init = 5;

packages/plugin-variables/README.md

@@ -10,6 +10,7 @@ const {
 const {
     isVariableDeclaration,
     isIdentifier,
+    isCallExpression,
 } = types;
 
 export const report = () => `Apply declarations order`;
@@ -27,6 +28,9 @@ export const traverse = ({push}) => ({
         if (!isIdentifier(__a))
             return;
 
+        if (!isCallExpression(__b))
+            return;
+        
         const prev = path.getAllPrevSiblings();
 
         for (const current of prev.filter(isVariableDeclaration)) {

…/fixture/apply-declarations-order-fix.js …/fixture/apply-declarations-order-fix.jspackages/plugin-destructuring/lib/apply-declarations-order/fixture/apply-declarations-order-fix.js renamed to packages/plugin-variables/lib/apply-declarations-order/fixture/apply-declarations-order-fix.js packages/plugin-destructuring/lib/apply-declarations-order/fixture/apply-declarations-order-fix.js renamed to packages/plugin-variables/lib/apply-declarations-order/fixture/apply-declarations-order-fix.js

@@ -0,0 +1,29 @@
+import {createTest} from '@putout/test';
+import * as plugin from './index.js';
+
+const test = createTest(import.meta.url, {
+    plugins: [
+        ['variables/apply-declarations-order', plugin],
+    ],
+});
+
+test('variables: apply-declarations-order: report', (t) => {
+    t.report('apply-declarations-order', `Apply declarations order`);
+    t.end();
+});
+
+test('variables: apply-declarations-order: transform', (t) => {
+    t.transform('apply-declarations-order');
+    t.end();
+});
+
+test('variables: apply-declarations-order: no report: identifier', (t) => {
+    t.noReport('identifier');
+    t.end();
+});
+
+test('variables: apply-declarations-order: no report: literal', (t) => {
+    t.noReport('literal');
+    t.end();
+});
+