From 705515e104844611b97f04bc0b39d64bec40fadb Mon Sep 17 00:00:00 2001
From: Abdul Malik Ikhsan <samsonasik@gmail.com>
Date: Thu, 16 Jul 2026 18:33:48 +0700
Subject: [PATCH] [CodeQuality] Skip super global variables on
 AssertIssetToSpecificMethodRector

---
 .../Fixture/skip_superglobals.php.inc         | 21 +++++++++++++++++++
 .../AssertIssetToSpecificMethodRector.php     | 21 +++++++++++++++++++
 2 files changed, 42 insertions(+)
 create mode 100644 rules-tests/CodeQuality/Rector/MethodCall/AssertIssetToSpecificMethodRector/Fixture/skip_superglobals.php.inc

diff --git a/rules-tests/CodeQuality/Rector/MethodCall/AssertIssetToSpecificMethodRector/Fixture/skip_superglobals.php.inc b/rules-tests/CodeQuality/Rector/MethodCall/AssertIssetToSpecificMethodRector/Fixture/skip_superglobals.php.inc
new file mode 100644
index 00000000..085cc2c2
--- /dev/null
+++ b/rules-tests/CodeQuality/Rector/MethodCall/AssertIssetToSpecificMethodRector/Fixture/skip_superglobals.php.inc
@@ -0,0 +1,21 @@
+<?php
+
+namespace Rector\PHPUnit\Tests\CodeQuality\Rector\MethodCall\AssertIssetToSpecificMethodRector\Fixture;
+
+use PHPUnit\Framework\TestCase;
+
+final class SkipSuperglobals extends TestCase
+{
+    public function test(): void
+    {
+        self::assertFalse(isset($GLOBALS['userconfig']));
+        self::assertFalse(isset($_SERVER['userconfig']));
+        self::assertFalse(isset($_GET['userconfig']));
+        self::assertFalse(isset($_POST['userconfig']));
+        self::assertFalse(isset($_FILES['userconfig']));
+        self::assertFalse(isset($_COOKIE['userconfig']));
+        self::assertFalse(isset($_SESSION['userconfig']));
+        self::assertFalse(isset($_REQUEST['userconfig']));
+        self::assertFalse(isset($_ENV['userconfig']));
+    }
+}
diff --git a/rules/CodeQuality/Rector/MethodCall/AssertIssetToSpecificMethodRector.php b/rules/CodeQuality/Rector/MethodCall/AssertIssetToSpecificMethodRector.php
index 7b753d42..6ef3bf39 100644
--- a/rules/CodeQuality/Rector/MethodCall/AssertIssetToSpecificMethodRector.php
+++ b/rules/CodeQuality/Rector/MethodCall/AssertIssetToSpecificMethodRector.php
@@ -22,6 +22,21 @@
  */
 final class AssertIssetToSpecificMethodRector extends AbstractRector
 {
+    /**
+     * @var string[]
+     */
+    private const array SUPER_GLOBAL_VARIABLE_NAMES = [
+        'GLOBALS',
+        '_SERVER',
+        '_GET',
+        '_POST',
+        '_FILES',
+        '_COOKIE',
+        '_SESSION',
+        '_REQUEST',
+        '_ENV',
+    ];
+
     public function __construct(
         private readonly IdentifierManipulator $identifierManipulator,
         private readonly TestsNodeAnalyzer $testsNodeAnalyzer,
@@ -78,6 +93,12 @@ public function refactor(Node $node): ?Node
             return null;
         }
 
+        // Keep the non-evaluating behavior of isset() for global state. A superglobal
+        // may be unavailable or unset, while passing it directly evaluates the variable.
+        if ($this->isNames($issetExpr->var, self::SUPER_GLOBAL_VARIABLE_NAMES)) {
+            return null;
+        }
+
         // isset() on an ArrayAccess object is not equivalent to assertArrayHasKey():
         // the key may be any type (assertArrayHasKey() requires int|string) and offsetExists()
         // semantics can differ from array_key_exists()