Merge branch 'main' into copilot/add-wp-profile-queries

Author: swissspidy

.gitattributes

@@ -0,0 +1,14 @@
+/.actrc export-ignore
+/.distignore export-ignore
+/.editorconfig export-ignore
+/.github export-ignore
+/.gitignore export-ignore
+/.typos.toml export-ignore
+/AGENTS.md export-ignore
+/behat.yml export-ignore
+/features export-ignore
+/phpcs.xml.dist export-ignore
+/phpstan.neon.dist export-ignore
+/phpunit.xml.dist export-ignore
+/tests export-ignore
+/wp-cli.yml export-ignore

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

@@ -21,13 +21,11 @@ jobs:
 
       - name: Check existence of composer.json file
         id: check_composer_file
-        uses: andstor/file-existence-action@076e0072799f4942c8bc574a82233e1e4d13e9d6 # v3
-        with:
-          files: "composer.json"
+        run: echo "files_exists=$(test -f composer.json && echo true || echo false)" >> "$GITHUB_OUTPUT"
 
       - name: Set up PHP environment
         if: steps.check_composer_file.outputs.files_exists == 'true'
-        uses: shivammathur/setup-php@44454db4f0199b8b9685a5d763dc37cbf79108e1 # v2
+        uses: shivammathur/setup-php@accd6127cb78bee3e8082180cb391013d204ef9f # v2
         with:
           php-version: 'latest'
           ini-values: zend.assertions=1, error_reporting=-1, display_errors=On
@@ -38,7 +36,7 @@ jobs:
 
       - name: Install Composer dependencies & cache dependencies
         if: steps.check_composer_file.outputs.files_exists == 'true'
-        uses: ramsey/composer-install@3cf229dc2919194e9e36783941438d17239e8520 # v3
+        uses: ramsey/composer-install@65e4f84970763564f46a70b8a54b90d033b3bdda # v3
         env:
           COMPOSER_ROOT_VERSION: dev-${{ github.event.repository.default_branch }}
         with:

README.md

@@ -115,7 +115,7 @@ render based on the main query, and renders it.
 Profile key metrics for WordPress hooks (actions and filters).
 
 ~~~
-wp profile hook [<hook>] [--all] [--spotlight] [--url=<url>] [--fields=<fields>] [--format=<format>] [--order=<order>] [--orderby=<fields>]
+wp profile hook [<hook>] [--all] [--spotlight] [--url=<url>] [--fields=<fields>] [--format=<format>] [--order=<order>] [--orderby=<fields>] [--search=<pattern>]
 ~~~
 
 In order to profile callbacks on a specific hook, the action or filter
@@ -161,6 +161,9 @@ will need to execute during the course of the request.
 	[--orderby=<fields>]
 		Set orderby which field.
 
+	[--search=<pattern>]
+		Filter callbacks to those matching the given search pattern (case-insensitive).
+
 **EXAMPLES**
 
     # Profile a hook.
@@ -382,7 +385,7 @@ current theme.
 
 ## Installing
 
-Installing this package requires WP-CLI v2.12 or greater. Update to the latest stable release with `wp cli update`.
+Installing this package requires WP-CLI v2.13 or greater. Update to the latest stable release with `wp cli update`.
 
 Once you've done so, you can install the latest stable version of this package with:
 

composer.json

@@ -7,7 +7,7 @@
   "authors": [],
   "require": {
     "php": ">=7.2.24",
-    "wp-cli/wp-cli": "^2.12"
+    "wp-cli/wp-cli": "^2.13"
   },
   "require-dev": {
     "wp-cli/wp-cli-tests": "^5"

features/profile-hook.feature

@@ -11,7 +11,6 @@ Feature: Profile a specific hook
       | template_redirect |
     And STDERR should be empty
 
-  @require-wp-4.4
   Scenario: Profile all callbacks when --all flag is used
     Given a WP install
 
@@ -109,7 +108,50 @@ Feature: Profile a specific hook
       | total (1)                   |                                           | 0          | 1            |
     And STDERR should be empty
 
-  @require-wp-4.4
+  Scenario: Search for callbacks by name pattern on a specific hook
+    Given a WP install
+    And a wp-content/mu-plugins/search-test.php file:
+      """
+      <?php
+      function wp_cli_search_hook_one() {}
+      function wp_cli_search_hook_two() {}
+      function unrelated_callback() {}
+      add_action( 'init', 'wp_cli_search_hook_one' );
+      add_action( 'init', 'wp_cli_search_hook_two' );
+      add_action( 'init', 'unrelated_callback' );
+      """
+
+    When I run `wp profile hook init --fields=callback --search=wp_cli_search_hook`
+    Then STDOUT should contain:
+      """
+      wp_cli_search_hook_one()
+      """
+    And STDOUT should contain:
+      """
+      wp_cli_search_hook_two()
+      """
+    And STDOUT should not contain:
+      """
+      unrelated_callback()
+      """
+    And STDERR should be empty
+
+  Scenario: Search for callbacks by name pattern across all hooks
+    Given a WP install
+    And a wp-content/mu-plugins/search-all-test.php file:
+      """
+      <?php
+      function wp_cli_search_all_hook() {}
+      add_action( 'init', 'wp_cli_search_all_hook' );
+      """
+
+    When I run `wp profile hook --all --fields=callback --search=wp_cli_search_all_hook`
+    Then STDOUT should contain:
+      """
+      wp_cli_search_all_hook()
+      """
+    And STDERR should be empty
+
   Scenario: Hooks should only be called once
     Given a WP install
     And a wp-content/mu-plugins/action-test.php file:

features/profile.feature

@@ -8,7 +8,7 @@ Feature: Basic profile usage
       """
       usage: wp profile eval <php-code> [--hook[=<hook>]] [--fields=<fields>] [--format=<format>] [--order=<order>] [--orderby=<fields>]
          or: wp profile eval-file <file> [--hook[=<hook>]] [--fields=<fields>] [--format=<format>] [--order=<order>] [--orderby=<fields>]
-         or: wp profile hook [<hook>] [--all] [--spotlight] [--url=<url>] [--fields=<fields>] [--format=<format>] [--order=<order>] [--orderby=<fields>]
+         or: wp profile hook [<hook>] [--all] [--spotlight] [--url=<url>] [--fields=<fields>] [--format=<format>] [--order=<order>] [--orderby=<fields>] [--search=<pattern>]
          or: wp profile queries [--url=<url>] [--hook=<hook>] [--callback=<callback>] [--fields=<fields>] [--format=<format>] [--order=<order>] [--orderby=<fields>]
          or: wp profile stage [<stage>] [--all] [--spotlight] [--url=<url>] [--fields=<fields>] [--format=<format>] [--order=<order>] [--orderby=<fields>]
 

src/Command.php

@@ -127,6 +127,7 @@ class Command {
 	 *     | total (7)                | 1.0335s | 77.42%      |
 	 *     +--------------------------+---------+-------------+
 	 *
+	 * @skipglobalargcheck
 	 * @when before_wp_load
 	 */
 	public function stage( $args, $assoc_args ) {
@@ -233,6 +234,9 @@ public function stage( $args, $assoc_args ) {
 	 * [--orderby=<fields>]
 	 * : Set orderby which field.
 	 *
+	 * [--search=<pattern>]
+	 * : Filter callbacks to those matching the given search pattern (case-insensitive).
+	 *
 	 * ## EXAMPLES
 	 *
 	 *     # Profile a hook.
@@ -251,6 +255,7 @@ public function stage( $args, $assoc_args ) {
 	 *     | total (7)                      | 8          | 0            |
 	 *     +--------------------------------+------------+--------------+
 	 *
+	 * @skipglobalargcheck
 	 * @when before_wp_load
 	 */
 	public function hook( $args, $assoc_args ) {
@@ -291,6 +296,13 @@ public function hook( $args, $assoc_args ) {
 		if ( Utils\get_flag_value( $assoc_args, 'spotlight' ) ) {
 			$loggers = self::shine_spotlight( $loggers, $metrics );
 		}
+		$search = Utils\get_flag_value( $assoc_args, 'search', false );
+		if ( false !== $search && '' !== $search ) {
+			if ( ! $focus ) {
+				WP_CLI::error( '--search requires --all or a specific hook.' );
+			}
+			$loggers = self::filter_by_callback( $loggers, $search );
+		}
 		$formatter->display_items( $loggers, true, $order, $orderby );
 	}
 
@@ -700,4 +712,20 @@ private static function shine_spotlight( $loggers, $metrics ) {
 
 		return $loggers;
 	}
+
+	/**
+	 * Filter loggers to only those whose callback name matches a pattern.
+	 *
+	 * @param array  $loggers
+	 * @param string $pattern
+	 * @return array
+	 */
+	private static function filter_by_callback( $loggers, $pattern ) {
+		return array_filter(
+			$loggers,
+			function ( $logger ) use ( $pattern ) {
+				return isset( $logger->callback ) && false !== stripos( $logger->callback, $pattern );
+			}
+		);
+	}
 }