Merge branch 'main' into copilot/add-package-version-command

Author: swissspidy

.github/workflows/code-quality.yml

@@ -6,6 +6,8 @@ on:
     branches:
       - main
       - master
+  schedule:
+     - cron: '17 2 * * *' # Run every day on a seemly random time.
 
 jobs:
   code-quality:

.github/workflows/copilot-setup-steps.yml

@@ -17,7 +17,7 @@ jobs:
 
     steps:
       - name: Checkout code
-        uses: actions/checkout@8e8c483db84b4bee98b60c0593521ed34d9990e8 # v6
+        uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6
 
       - name: Check existence of composer.json file
         id: check_composer_file

.typos.toml

@@ -0,0 +1,11 @@
+[default]
+extend-ignore-re = [
+	"(?Rm)^.*(#|//)\\s*spellchecker:disable-line$",
+	"(?s)(#|//)\\s*spellchecker:off.*?\\n\\s*(#|//)\\s*spellchecker:on",
+	"(#|//)\\s*spellchecker:ignore-next-line\\n.*"
+]
+
+[files]
+extend-exclude = [
+	"tests/phpunit/JsonManipulatorTest.php"
+]

README.md

@@ -312,14 +312,20 @@ These fields are optionally available:
 
 ### wp package update
 
-Updates all installed WP-CLI packages to their latest version.
+Updates installed WP-CLI packages to their latest version.
 
 ~~~
-wp package update 
+wp package update [<package-name>...]
 ~~~
 
+**OPTIONS**
+
+	[<package-name>...]
+		One or more package names to update. If not specified, all packages will be updated.
+
 **EXAMPLES**
 
+    # Update all packages.
     $ wp package update
     Using Composer to update packages...
     ---
@@ -334,6 +340,17 @@ wp package update
     ---
     Success: Packages updated.
 
+    # Update a specific package.
+    $ wp package update wp-cli/server-command
+    Using Composer to update packages...
+    ---
+    Loading composer repositories with package information
+    Updating dependencies
+    Writing lock file
+    Generating autoload files
+    ---
+    Success: Package updated successfully.
+
 
 
 ### wp package uninstall

composer.json

@@ -13,8 +13,8 @@
   ],
   "require": {
     "ext-json": "*",
-    "composer/composer": "^2.2.25",
-    "wp-cli/wp-cli": "^2.12"
+    "composer/composer": "^2.9.5",
+    "wp-cli/wp-cli": "^2.13"
   },
   "require-dev": {
     "wp-cli/scaffold-command": "^1 || ^2",

phpcs.xml.dist

@@ -18,7 +18,7 @@
 	<exclude-pattern>*/src/WP_CLI/JsonManipulator\.php$</exclude-pattern>
 	<exclude-pattern>*/src/WP_CLI/Package/Compat/Min_Composer_1_10/NullIOMethodsTrait\.php$</exclude-pattern>
 	<exclude-pattern>*/src/WP_CLI/Package/Compat/Min_Composer_2_3/NullIOMethodsTrait\.php$</exclude-pattern>
-	<exclude-pattern>*/tests/JsonManipulatorTest\.php$</exclude-pattern>
+	<exclude-pattern>*/tests/phpunit/JsonManipulatorTest\.php$</exclude-pattern>
 
 	<!-- Show progress. -->
 	<arg value="p"/>
@@ -63,4 +63,5 @@
 		<exclude-pattern>*/src/Package_Command\.php$</exclude-pattern>
 	</rule>
 
+	<exclude-pattern>*/tests/phpstan/scan-files\.php$</exclude-pattern>
 </ruleset>

phpstan.neon.dist

@@ -1,12 +1,13 @@
 parameters:
-  level: 1
+  level: 5
   paths:
     - src
     - package-command.php
   scanDirectories:
     - vendor/wp-cli/wp-cli/php
   scanFiles:
     - vendor/php-stubs/wordpress-stubs/wordpress-stubs.php
+    - tests/phpstan/scan-files.php
   treatPhpDocTypesAsCertain: false
   ignoreErrors:
     # WP_CLI_VERSION is defined in wp-cli core at runtime

phpunit.xml.dist

@@ -13,7 +13,7 @@
 		 colors="true"
 		 verbose="true">
 	<testsuite name="wp-cli/package-command tests">
-		<directory suffix="Test.php">tests</directory>
+		<directory suffix="Test.php">tests/phpunit</directory>
 	</testsuite>
 
 	<filter>

src/Package_Command.php

@@ -3,7 +3,6 @@
 use Composer\Composer;
 use Composer\Config;
 use Composer\Config\JsonConfigSource;
-use Composer\DependencyResolver\Pool;
 use Composer\Factory;
 use Composer\IO\NullIO;
 use Composer\Installer;
@@ -223,9 +222,7 @@ public function install( $args, $assoc_args ) {
 		$dir_package = false;
 		$version     = '';
 		if ( $this->is_git_repository( $package_name ) ) {
-			if ( '' === $version ) {
-				$version = "dev-{$this->get_github_default_branch( $package_name, $insecure )}";
-			}
+			$version     = "dev-{$this->get_github_default_branch( $package_name, $insecure )}";
 			$git_package = $package_name;
 			$matches     = [];
 			if ( preg_match( '#([^:\/]+\/[^\/]+)\.git#', $package_name, $matches ) ) {
@@ -247,7 +244,7 @@ public function install( $args, $assoc_args ) {
 				$gitlab_token = getenv( 'GITLAB_TOKEN' ); // Use GITLAB_TOKEN if available to avoid authorization failures or rate-limiting.
 				$headers      = $gitlab_token && strpos( $package_name, '://gitlab.com/' ) !== false ? [ 'PRIVATE-TOKEN' => $gitlab_token ] : [];
 				$response     = Utils\http_request( 'GET', $package_name, null, $headers, $options );
-				if ( 20 !== (int) substr( $response->status_code, 0, 2 ) ) {
+				if ( 20 !== (int) substr( (string) $response->status_code, 0, 2 ) ) {
 					@unlink( $temp ); // @codingStandardsIgnoreLine
 					WP_CLI::error( sprintf( "Couldn't download package from '%s' (HTTP code %d).", $package_name, $response->status_code ) );
 				}
@@ -316,11 +313,9 @@ public function install( $args, $assoc_args ) {
 			}
 		}
 
-		if ( $this->is_composer_v2() ) {
-			$package_name = function_exists( 'mb_strtolower' )
-				? mb_strtolower( $package_name )
-				: strtolower( $package_name );
-		}
+		$package_name = function_exists( 'mb_strtolower' )
+			? mb_strtolower( $package_name )
+			: strtolower( $package_name );
 
 		if ( '' === $version ) {
 			$version = self::DEFAULT_DEV_BRANCH_CONSTRAINTS;
@@ -932,8 +927,8 @@ private function package_index() {
 	 * Displays a set of packages
 	 *
 	 * @param string $context
-	 * @param array
-	 * @param array
+	 * @param array $packages
+	 * @param array $assoc_args
 	 */
 	private function show_packages( $context, $packages, $assoc_args ) {
 		$default_fields = [];
@@ -976,7 +971,7 @@ private function show_packages( $context, $packages, $assoc_args ) {
 				$update_version                = '';
 				if ( 'list' === $context && ! $skip_update_check ) {
 					try {
-						$latest = $this->find_latest_package( $package, $composer, null );
+						$latest = $this->find_latest_package( $package, $composer );
 						if ( $latest && $latest->getFullPrettyVersion() !== $package->getFullPrettyVersion() ) {
 							$update         = 'available';
 							$update_version = $latest->getPrettyVersion();
@@ -1039,7 +1034,7 @@ private function get_package_by_shortened_identifier( $package_name, $insecure =
 		// Check if the package exists on Packagist.
 		$url      = "https://repo.packagist.org/p2/{$package_name}.json";
 		$response = Utils\http_request( 'GET', $url, null, [], $options );
-		if ( 20 === (int) substr( $response->status_code, 0, 2 ) ) {
+		if ( 20 === (int) substr( (string) $response->status_code, 0, 2 ) ) {
 			return $package_name;
 		}
 
@@ -1048,7 +1043,7 @@ private function get_package_by_shortened_identifier( $package_name, $insecure =
 		$github_token = getenv( 'GITHUB_TOKEN' ); // Use GITHUB_TOKEN if available to avoid authorization failures or rate-limiting.
 		$headers      = $github_token ? [ 'Authorization' => 'token ' . $github_token ] : [];
 		$response     = Utils\http_request( 'GET', $url, null /*data*/, $headers, $options );
-		if ( 20 === (int) substr( $response->status_code, 0, 2 ) ) {
+		if ( 20 === (int) substr( (string) $response->status_code, 0, 2 ) ) {
 			return $url;
 		}
 
@@ -1058,7 +1053,7 @@ private function get_package_by_shortened_identifier( $package_name, $insecure =
 		$headers      = $github_token ? [ 'Authorization' => 'token ' . $github_token ] : [];
 		$headers      = $gitlab_token && strpos( $package_name, '://gitlab.com/' ) !== false ? [ 'PRIVATE-TOKEN' => $gitlab_token ] : [];
 		$response     = Utils\http_request( 'GET', $url, null /*data*/, $headers, $options );
-		if ( 20 === (int) substr( $response->status_code, 0, 2 ) ) {
+		if ( 20 === (int) substr( (string) $response->status_code, 0, 2 ) ) {
 			return $url;
 		}
 
@@ -1086,9 +1081,6 @@ private function get_installed_packages() {
 			if ( in_array( $package->getPrettyName(), $installed_package_keys, true ) ) {
 				$installed_packages[] = $package;
 			} elseif ( false !== $idx ) { // Legacy incorrect name check.
-				if ( ! $this->is_composer_v2() ) {
-					WP_CLI::warning( sprintf( "Found package '%s' misnamed '%s' in '%s'.", $package->getPrettyName(), $installed_package_keys[ $idx ], $this->get_composer_json_path() ) );
-				}
 				$installed_packages[] = $package;
 			}
 		}
@@ -1240,12 +1232,11 @@ private function create_default_composer_json( $composer_path ) {
 	 *
 	 * @param  PackageInterface $package
 	 * @param  Composer         $composer
-	 * @param  string           $phpVersion
-	 * @param  bool             $minorOnly
+	 * @param  bool             $minor_only
 	 *
-	 * @return PackageInterface|null
+	 * @return PackageInterface|false
 	 */
-	private function find_latest_package( PackageInterface $package, Composer $composer, $php_version, $minor_only = false ) {
+	private function find_latest_package( PackageInterface $package, Composer $composer, $minor_only = false ) {
 		// Find the latest version allowed in this pool/repository set.
 		$name             = $package->getPrettyName();
 		$version_selector = $this->get_version_selector( $composer );
@@ -1266,30 +1257,20 @@ private function find_latest_package( PackageInterface $package, Composer $compo
 			$target_version = '^' . $package->getVersion();
 		}
 
-		if ( $this->is_composer_v2() ) {
-			return $version_selector->findBestCandidate( $name, $target_version, $best_stability );
-		}
-
-		return $version_selector->findBestCandidate( $name, $target_version, $php_version, $best_stability );
+		return $version_selector->findBestCandidate( $name, $target_version, $best_stability );
 	}
 
 	/**
 	 * @return VersionSelector
 	 */
 	private function get_version_selector( Composer $composer ) {
 		if ( ! $this->version_selector ) {
-			if ( $this->is_composer_v2() ) {
-				$repository_set = new Repository\RepositorySet(
-					$composer->getPackage()->getMinimumStability(),
-					$composer->getPackage()->getStabilityFlags()
-				);
-				$repository_set->addRepository( new CompositeRepository( $composer->getRepositoryManager()->getRepositories() ) );
-				$this->version_selector = new VersionSelector( $repository_set );
-			} else {
-				$pool = new Pool( $composer->getPackage()->getMinimumStability(), $composer->getPackage()->getStabilityFlags() );
-				$pool->addRepository( new CompositeRepository( $composer->getRepositoryManager()->getRepositories() ) );
-				$this->version_selector = new VersionSelector( $pool );
-			}
+			$repository_set = new Repository\RepositorySet(
+				$composer->getPackage()->getMinimumStability(),
+				$composer->getPackage()->getStabilityFlags()
+			);
+			$repository_set->addRepository( new CompositeRepository( $composer->getRepositoryManager()->getRepositories() ) );
+			$this->version_selector = new VersionSelector( $repository_set );
 		}
 
 		return $this->version_selector;
@@ -1323,7 +1304,7 @@ private function check_github_package_name( $package_name, $version = '', $insec
 		$raw_content_url = "https://raw.githubusercontent.com/{$package_name}/{$this->get_raw_git_version( $version )}/composer.json";
 
 		$response = Utils\http_request( 'GET', $raw_content_url, null /*data*/, $headers, $options );
-		if ( 20 !== (int) substr( $response->status_code, 0, 2 ) ) {
+		if ( 20 !== (int) substr( (string) $response->status_code, 0, 2 ) ) {
 			// Could not get composer.json. Possibly private so warn and return best guess from input (always xxx/xxx).
 			WP_CLI::warning(
 				sprintf(
@@ -1470,7 +1451,7 @@ private function get_github_latest_release_tag( $package_name, $insecure ) {
 		$url      = "https://api.github.com/repos/{$package_name}/releases/latest";
 		$options  = [ 'insecure' => $insecure ];
 		$response = Utils\http_request( 'GET', $url, null, [], $options );
-		if ( 20 !== (int) substr( $response->status_code, 0, 2 ) ) {
+		if ( 20 !== (int) substr( (string) $response->status_code, 0, 2 ) ) {
 			WP_CLI::warning( 'Could not guess stable version from GitHub repository, falling back to master branch' );
 			return 'master';
 		}
@@ -1610,15 +1591,6 @@ static function () use (
 		);
 	}
 
-	/**
-	 * Check whether we are dealing with Composer version 2.0.0+.
-	 *
-	 * @return bool
-	 */
-	private function is_composer_v2() {
-		return version_compare( Composer::getVersion(), '2.0.0', '>=' );
-	}
-
 	/**
 	 * Try to retrieve default branch via GitHub API.
 	 *
@@ -1639,7 +1611,7 @@ private function get_github_default_branch( $package_name, $insecure = false ) {
 
 		$github_api_repo_url = "https://api.github.com/repos/{$package_name}";
 		$response            = Utils\http_request( 'GET', $github_api_repo_url, null /*data*/, $headers, $options );
-		if ( 20 !== (int) substr( $response->status_code, 0, 2 ) ) {
+		if ( 20 !== (int) substr( (string) $response->status_code, 0, 2 ) ) {
 			WP_CLI::warning(
 				sprintf(
 					"Couldn't fetch default branch for package '%s' (HTTP code %d). Presuming default branch is 'master'.",