fix(HasSudoPassword): Refine sudo password application condition

guanguans · guanguans · commit 842a8242751c · 2025-12-03T18:29:45.000+08:00
- Enhance the condition for applying the sudo password to exclude specific PHP SAPI environments.
- This change prevents unnecessary sudo password application in CLI and embedded contexts.

Signed-off-by: guanguans &lt;ityaozm@gmail.com&gt;
diff --git a/src/Concerns/HasSudoPassword.php b/src/Concerns/HasSudoPassword.php
@@ -43,6 +43,15 @@ public function withSudoPassword(
 
     protected function shouldApplySudoPassword(): bool
     {
-        return $this->sudoPassword && OsHelper::isUnix();
+        return $this->sudoPassword && OsHelper::isUnix() && !\in_array(
+            \PHP_SAPI,
+            [
+                'cli',
+                // 'cli-server',
+                'embed',
+                'phpdbg',
+            ],
+            true
+        );
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -43,6 +43,15 @@ public function withSudoPassword(`
`43`	`43`
`44`	`44`	`protected function shouldApplySudoPassword(): bool`
`45`	`45`	`{`
`46`		`- return $this->sudoPassword && OsHelper::isUnix();`
	`46`	`+ return $this->sudoPassword && OsHelper::isUnix() && !\in_array(`
	`47`	`+ \PHP_SAPI,`
	`48`	`+ [`
	`49`	`+ 'cli',`
	`50`	`+ // 'cli-server',`
	`51`	`+ 'embed',`
	`52`	`+ 'phpdbg',`
	`53`	`+ ],`
	`54`	`+ true`
	`55`	`+ );`
`47`	`56`	`}`
`48`	`57`	`}`