feature: @putout/plugin-apply-arrow: different scopes

Author: coderaiser

packages/plugin-apply-arrow/lib/apply-arrow.js

@@ -12,10 +12,15 @@ module.exports.match = () => ({
         const {name} = __a;
         const binding = path.parentPath.scope.bindings[name];
 
+        if (!binding.referenced)
+            return true;
+        
         if (path.parentPath.isExportNamedDeclaration())
             return binding.references === 1;
 
-        return !binding.referenced;
+        const [ref] = binding.referencePaths;
+        
+        return ref.scope.uid !== binding.scope.uid;
     },
 });
 

packages/plugin-apply-arrow/test/apply-arrow.js

@@ -22,6 +22,11 @@ test('putout: apply-arrow: transform', (t) => {
     t.end();
 });
 
+test('putout: apply-arrow: transform: other-scope-reference', (t) => {
+    t.transform('other-scope-reference');
+    t.end();
+});
+
 test('putout: apply-arrow: no report: long', (t) => {
     t.noReport('long');
     t.end();

packages/plugin-apply-arrow/test/fixture/other-scope-reference-fix.js

@@ -0,0 +1,15 @@
+export const createPath = ({tokens, start, end}) => ({
+    isNextPunctuator: createIsNextPunctuator({
+        tokens,
+        end,
+    }),
+});
+
+const createIsNextPunctuator = ({tokens, end}) => () => {
+    const current = tokens[end];
+    
+    if (!current)
+        return false;
+    
+    return isPunctuator(current, ';');
+};

packages/plugin-apply-arrow/test/fixture/other-scope-reference.js

@@ -0,0 +1,17 @@
+export const createPath = ({tokens, start, end}) => ({
+    isNextPunctuator: createIsNextPunctuator({
+        tokens,
+        end,
+    }),
+});
+
+function createIsNextPunctuator({tokens, end}) {
+    return () => {
+        const current = tokens[end];
+        
+        if (!current)
+            return false;
+        
+        return isPunctuator(current, ';');
+    };
+}