Address open review comments: add debug logging for range expansions and feature tests for site, comment, user range operations

Co-authored-by: swissspidy <841956+swissspidy@users.noreply.github.com>

Author: Copilot

features/comment.feature

@@ -501,3 +501,24 @@ Feature: Manage WordPress comments
       """
       Deleted comment {COMMENT_ID_3}.
       """
+
+  Scenario: Trash comments using ID ranges
+    Given a WP install
+
+    When I run `wp comment create --comment_post_ID=1 --comment_content='Comment D' --comment_author='D' --porcelain`
+    Then STDOUT should be a number
+    And save STDOUT as {COMMENT_ID_1}
+
+    When I run `wp comment create --comment_post_ID=1 --comment_content='Comment E' --comment_author='E' --porcelain`
+    Then STDOUT should be a number
+    And save STDOUT as {COMMENT_ID_2}
+
+    When I run `wp comment trash {COMMENT_ID_1}-{COMMENT_ID_2}`
+    Then STDOUT should contain:
+      """
+      Trashed comment {COMMENT_ID_1}.
+      """
+    And STDOUT should contain:
+      """
+      Trashed comment {COMMENT_ID_2}.
+      """

features/site.feature

@@ -891,3 +891,26 @@ Feature: Manage sites in a multisite installation
       """
       1
       """
+
+  Scenario: Archive sites using ID ranges
+    Given a WP multisite install
+    And I run `wp site create --slug=rangesite1 --porcelain`
+    And save STDOUT as {SITE_ID_1}
+    And I run `wp site create --slug=rangesite2 --porcelain`
+    And save STDOUT as {SITE_ID_2}
+    And I run `wp site create --slug=rangesite3 --porcelain`
+    And save STDOUT as {SITE_ID_3}
+
+    When I run `wp site archive {SITE_ID_1}-{SITE_ID_3}`
+    Then STDOUT should contain:
+      """
+      Success: Site {SITE_ID_1} archived.
+      """
+    And STDOUT should contain:
+      """
+      Success: Site {SITE_ID_2} archived.
+      """
+    And STDOUT should contain:
+      """
+      Success: Site {SITE_ID_3} archived.
+      """

features/user.feature

@@ -803,3 +803,28 @@ Feature: Manage WordPress users
       """
       Removed user {USER_ID_3}
       """
+
+  Scenario: Reset passwords using ID ranges
+    Given a WP install
+
+    When I run `wp user create resetrange1 resetrange1@example.com --role=subscriber --porcelain`
+    Then STDOUT should be a number
+    And save STDOUT as {USER_ID_1}
+
+    When I run `wp user create resetrange2 resetrange2@example.com --role=subscriber --porcelain`
+    Then STDOUT should be a number
+    And save STDOUT as {USER_ID_2}
+
+    When I run `wp user reset-password {USER_ID_1}-{USER_ID_2} --skip-email`
+    Then STDOUT should contain:
+      """
+      Reset password for resetrange1.
+      """
+    And STDOUT should contain:
+      """
+      Reset password for resetrange2.
+      """
+    And STDOUT should contain:
+      """
+      Passwords reset for 2 users.
+      """

src/WP_CLI/ExpandsIdRanges.php

@@ -2,6 +2,8 @@
 
 namespace WP_CLI;
 
+use WP_CLI;
+
 /**
  * Trait that provides ID range expansion for WP-CLI commands.
  *
@@ -39,13 +41,19 @@ protected static function expand_id_ranges( array $args, callable $get_ids_in_ra
 					$start = $end;
 					$end   = $temp;
 				}
-				$ids = array_merge( $ids, $get_ids_in_range( $start, $end ) );
+				$range_ids = $get_ids_in_range( $start, $end );
+				WP_CLI::debug( sprintf( "Expanded range '%s' to %d IDs.", $arg, count( $range_ids ) ), 'range-expansion' );
+				$ids = array_merge( $ids, $range_ids );
 			} elseif ( preg_match( '/^(\d+)-$/', $arg, $matches ) ) {
 				// Open-ended range: "34-"
-				$ids = array_merge( $ids, $get_ids_in_range( (int) $matches[1], null ) );
+				$range_ids = $get_ids_in_range( (int) $matches[1], null );
+				WP_CLI::debug( sprintf( "Expanded range '%s' to %d IDs.", $arg, count( $range_ids ) ), 'range-expansion' );
+				$ids = array_merge( $ids, $range_ids );
 			} elseif ( preg_match( '/^-(\d+)$/', $arg, $matches ) ) {
 				// Lower-bounded range: "-35"
-				$ids = array_merge( $ids, $get_ids_in_range( 1, (int) $matches[1] ) );
+				$range_ids = $get_ids_in_range( 1, (int) $matches[1] );
+				WP_CLI::debug( sprintf( "Expanded range '%s' to %d IDs.", $arg, count( $range_ids ) ), 'range-expansion' );
+				$ids = array_merge( $ids, $range_ids );
 			} else {
 				$ids[] = $arg;
 			}