feature: @putout/plugin-remove-useless-variables: declaration: no loc support

Author: coderaiser

packages/plugin-remove-useless-variables/lib/declaration/fixture/no-loc-fix.js

@@ -0,0 +1,9 @@
+import a from 'a';
+
+const noop = () => {};
+const b = 0;
+
+export const fn = (a, b) => {
+    if (c)
+        return noop;
+};

packages/plugin-remove-useless-variables/lib/declaration/fixture/no-loc.js

@@ -0,0 +1,7 @@
+import a from 'a';
+const b = 0;
+
+export const fn = (a, b) => {
+    if (c)
+        return () => {};
+};

packages/plugin-remove-useless-variables/lib/declaration/index.js

@@ -119,7 +119,12 @@ function isTakesOneLine(binding) {
     if (!binding.path.node.init)
         return false;
 
-    const {start, end} = binding.path.node.init.loc;
+    const {loc} = binding.path.node.init;
+    
+    if (!loc)
+        return false;
+    
+    const {start, end} = loc;
 
     return start.line === end.line;
 }

packages/plugin-remove-useless-variables/lib/declaration/index.spec.js

@@ -1,4 +1,6 @@
 import {createTest} from '@putout/test';
+import * as declare from '@putout/plugin-declare';
+import * as maybe from '@putout/plugin-maybe';
 import * as plugin from './index.js';
 
 const test = createTest(import.meta.url, {
@@ -103,3 +105,11 @@ test('remove useless variables: declaration: no transform: re-assign-member', (t
     t.noTransform('re-assign-member');
     t.end();
 });
+
+test('remove useless variables: declaration: transform: no-loc', (t) => {
+    t.transform('no-loc', {
+        declare,
+        maybe,
+    });
+    t.end();
+});

packages/plugin-remove-useless-variables/package.json

@@ -36,6 +36,8 @@
   ],
   "devDependencies": {
     "@putout/eslint-flat": "^3.0.0",
+    "@putout/plugin-declare": "*",
+    "@putout/plugin-maybe": "*",
     "@putout/plugin-remove-useless-array-from": "*",
     "@putout/plugin-reuse-duplicate-init": "*",
     "@putout/test": "^14.0.0",