[DeadCode] Skip used via compact() on RemoveUnusedClosureVariableUseRector (#7431)

* [DeadCode] Skip used via compact() on RemoveUnusedClosureVariableUseRector

* Fix

Author: samsonasik

rules-tests/DeadCode/Rector/Closure/RemoveUnusedClosureVariableUseRector/Fixture/skip_used_via_func_get_arg.php.inc

@@ -0,0 +1,18 @@
+<?php
+
+namespace Rector\Tests\DeadCode\Rector\Closure\RemoveUnusedClosureVariableUseRector\Fixture;
+
+/**
+ * @see https://3v4l.org/9vhce
+ */
+final class SkipUsedViaCompact
+{
+    public function run($value)
+    {
+        return function () use ($value) {
+            $v = compact('value');
+
+            return $v;
+        };
+    }
+}

rules/DeadCode/Rector/Concat/RemoveUnusedClosureVariableUseRector.php

@@ -6,6 +6,7 @@
 
 use PhpParser\Node;
 use PhpParser\Node\Expr\Closure;
+use Rector\DeadCode\NodeAnalyzer\ExprUsedInNodeAnalyzer;
 use Rector\PhpParser\Node\BetterNodeFinder;
 use Rector\Rector\AbstractRector;
 use Symplify\RuleDocGenerator\ValueObject\CodeSample\CodeSample;
@@ -17,7 +18,8 @@
 final class RemoveUnusedClosureVariableUseRector extends AbstractRector
 {
     public function __construct(
-        private readonly BetterNodeFinder $betterNodeFinder
+        private readonly BetterNodeFinder $betterNodeFinder,
+        private readonly ExprUsedInNodeAnalyzer $exprUsedInNodeAnalyzer
     ) {
     }
 
@@ -72,7 +74,10 @@ public function refactor(Node $node): ?Node
                 continue;
             }
 
-            $isUseUsed = (bool) $this->betterNodeFinder->findVariableOfName($node->stmts, $useVariableName);
+            $isUseUsed = (bool) $this->betterNodeFinder->findFirst(
+                $node->stmts,
+                fn (Node $subNode): bool => $this->exprUsedInNodeAnalyzer->isUsed($subNode, $useVariable->var)
+            );
             if ($isUseUsed) {
                 continue;
             }