fix: resolve TypeError in CLIRequest::parseCommand when argv is missing (#10415)

Author: gr8man

system/HTTP/CLIRequest.php

@@ -185,7 +185,7 @@ public function getOptionString(bool $useLongOpts = false): string
      */
     protected function parseCommand()
     {
-        $args = $this->getServer('argv');
+        $args = $this->getServer('argv') ?? [];
         array_shift($args); // Scrap index.php
 
         $optionValue = false;

tests/system/HTTP/CLIRequestTest.php

@@ -574,4 +574,15 @@ public function testIs(): void
     {
         $this->assertFalse($this->request->is('get'));
     }
+
+    public function testParseCommandWithMissingArgv(): void
+    {
+        Services::injectMock('superglobals', new Superglobals([], [], [], [], []));
+
+        $request = new CLIRequest(new App());
+
+        $this->assertSame('', $request->getPath());
+        $this->assertSame([], $request->getSegments());
+        $this->assertSame('', $request->getOptionString());
+    }
 }

user_guide_src/source/changelogs/v4.7.5.rst

@@ -30,6 +30,7 @@ Deprecations
 Bugs Fixed
 **********
 
+- **CLIRequest:** Fixed a bug where ``parseCommand()`` could throw a TypeError when ``argv`` is missing.
 - **Content Security Policy:** Fixed a bug where empty ``Content-Security-Policy``, ``Content-Security-Policy-Report-Only``, and ``Reporting-Endpoints`` response headers were generated when no corresponding values existed.
 - **Logger:** Fixed a bug where interpolating a log message with array or non-stringable context values could raise PHP warnings or errors.