refactor: remove the unused SUDO_USER from the user helper function

- Remove the unused `SUDO_USER` from the `user` helper function since Windows doesn't set `SUDO_USER` environment variable at all, so this is redundant code.

- Refactored the `user` helper function to have the `USERNAME` variable return by default, and only return the `USER` if it's set. I'm not convinced `USER` is ever used on Windows either, but for now we'll keep it.

- Add URLs to the Laravel docs for `resolve`, `retry`, and `tap` helper functions where their functionality was based off.

Author: yCodeTech

cli/includes/helpers.php

@@ -245,6 +245,8 @@ function addTableSeparator($rows) {
 	/**
 	 * Resolve the given class from the container.
 	 *
+	 * https://laravel.com/docs/12.x/helpers#method-resolve
+	 *
 	 * @param string $class
 	 * @return mixed
 	 */
@@ -268,6 +270,8 @@ function swap($class, $instance) {
 	/**
 	 * Retry the given function N times.
 	 *
+	 * https://laravel.com/docs/12.x/helpers#method-retry
+	 *
 	 * @param int $retries
 	 * @param callable $fn
 	 * @param int $sleep
@@ -298,6 +302,8 @@ function retry($retries, $fn, $sleep = 0) {
 	/**
 	 * Tap the given value.
 	 *
+	 * https://laravel.com/docs/12.x/helpers#method-tap
+	 *
 	 * @param mixed $value
 	 * @param callable $callback
 	 * @return mixed
@@ -315,15 +321,11 @@ function tap($value, callable $callback) {
  * @return string
  */
 function user() {
-	if (!isset($_SERVER['SUDO_USER'])) {
-		if (!isset($_SERVER['USER'])) {
-			return $_SERVER['USERNAME'];
-		}
-
+	if (isset($_SERVER['USER'])) {
 		return $_SERVER['USER'];
 	}
 
-	return $_SERVER['SUDO_USER'];
+	return $_SERVER['USERNAME'];
 }
 
 /**