Add actions field to wp cron event list (#124)

Co-authored-by: swissspidy <841956+swissspidy@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: Pascal Birchler <pascalb@google.com>

Author: 3 people

features/cron-event.feature

@@ -159,6 +159,77 @@ Feature: Manage WP Cron events
       Debug: Arguments:
       """
 
+  Scenario: List cron events with actions field
+    Given a wp-content/mu-plugins/test-cron-actions.php file:
+      """
+      <?php
+      add_action( 'wp_cli_test_hook', 'wp_cli_test_function' );
+      add_action( 'wp_cli_test_hook', array( 'MyTestClass', 'my_method' ) );
+      add_action( 'wp_cli_test_hook_closure', function() {
+        // Test closure
+      } );
+
+      function wp_cli_test_function() {
+        // Test function
+      }
+
+      class MyTestClass {
+        public static function my_method() {
+          // Test method
+        }
+      }
+      """
+
+    When I run `wp cron event schedule wp_cli_test_hook now`
+    Then STDOUT should contain:
+      """
+      Success: Scheduled event with hook 'wp_cli_test_hook'
+      """
+
+    When I run `wp cron event list --fields=hook,actions --format=csv`
+    Then STDOUT should contain:
+      """
+      wp_cli_test_function
+      """
+    And STDOUT should contain:
+      """
+      MyTestClass::my_method
+      """
+
+    When I run `wp cron event list --hook=wp_cli_test_hook --fields=hook,actions --format=json`
+    Then STDOUT should be JSON containing:
+      """
+      [{"hook":"wp_cli_test_hook"}]
+      """
+    And STDOUT should contain:
+      """
+      wp_cli_test_function
+      """
+
+    When I run `wp cron event schedule wp_cli_test_hook_closure now`
+    Then STDOUT should contain:
+      """
+      Success: Scheduled event with hook 'wp_cli_test_hook_closure'
+      """
+
+    When I run `wp cron event list --hook=wp_cli_test_hook_closure --fields=hook,actions --format=csv`
+    Then STDOUT should contain:
+      """
+      Closure
+      """
+
+    When I run `wp cron event schedule wp_cli_test_hook_no_actions now`
+    Then STDOUT should contain:
+      """
+      Success: Scheduled event with hook 'wp_cli_test_hook_no_actions'
+      """
+
+    When I run `wp cron event list --hook=wp_cli_test_hook_no_actions --fields=hook,actions --format=csv`
+    Then STDOUT should contain:
+      """
+      None
+      """
+
   Scenario: Confirm that cron event run in debug mode shows the start of events
     When I try `wp cron event run wp_version_check --debug=cron`
     Then STDOUT should contain:

src/Cron_Event_Command.php

@@ -85,6 +85,7 @@ class Cron_Event_Command extends WP_CLI_Command {
 	 * * schedule
 	 * * interval
 	 * * next_run
+	 * * actions
 	 *
 	 * ## EXAMPLES
 	 *
@@ -113,6 +114,14 @@ public function list_( $args, $assoc_args ) {
 			$events = array();
 		}
 
+		// Populate actions field only if requested
+		$requested_fields = $formatter->fields;
+		if ( ! empty( $requested_fields ) && in_array( 'actions', $requested_fields, true ) ) {
+			foreach ( $events as $event ) {
+				$event->actions = self::get_hook_actions( $event->hook );
+			}
+		}
+
 		foreach ( $events as $key => $event ) {
 			foreach ( $this->fields as $field ) {
 				if ( ! empty( $assoc_args[ $field ] ) && $event->{$field} !== $assoc_args[ $field ] ) {
@@ -399,6 +408,7 @@ protected static function format_event( stdClass $event ) {
 		$event->next_run          = get_date_from_gmt( date( 'Y-m-d H:i:s', $event->time ), self::$time_format ); //phpcs:ignore WordPress.DateTime.RestrictedFunctions.date_date
 		$event->next_run_gmt      = date( self::$time_format, $event->time ); //phpcs:ignore WordPress.DateTime.RestrictedFunctions.date_date
 		$event->next_run_relative = self::interval( $event->time - time() );
+		$event->actions           = '';
 
 		return $event;
 	}
@@ -607,4 +617,92 @@ private static function interval( $since ) {
 	private function get_formatter( &$assoc_args ) {
 		return new \WP_CLI\Formatter( $assoc_args, $this->fields, 'event' );
 	}
+
+	/**
+	 * Gets the actions (callbacks) registered for a specific hook.
+	 *
+	 * @param string $hook_name The name of the hook.
+	 * @return string A comma-separated list of action callbacks, or 'None' if no actions are registered.
+	 */
+	protected static function get_hook_actions( $hook_name ) {
+		static $cache = array();
+
+		if ( isset( $cache[ $hook_name ] ) ) {
+			return $cache[ $hook_name ];
+		}
+
+		global $wp_filter;
+
+		if ( ! isset( $wp_filter[ $hook_name ] ) ) {
+			$cache[ $hook_name ] = 'None';
+			return $cache[ $hook_name ];
+		}
+
+		$hook = $wp_filter[ $hook_name ];
+
+		// Get callbacks from the WP_Hook object (WordPress 4.7+)
+		if ( $hook instanceof \WP_Hook ) {
+			$callbacks = $hook->callbacks;
+		} else {
+			// Fallback for older WordPress versions
+			$callbacks = $hook;
+		}
+
+		if ( empty( $callbacks ) ) {
+			$cache[ $hook_name ] = 'None';
+			return $cache[ $hook_name ];
+		}
+
+		ksort( $callbacks );
+
+		$actions = array();
+
+		// Iterate through all priorities
+		foreach ( $callbacks as $priority => $priority_callbacks ) {
+			foreach ( $priority_callbacks as $callback_info ) {
+				if ( ! isset( $callback_info['function'] ) ) {
+					continue;
+				}
+				$callback  = $callback_info['function'];
+				$actions[] = self::format_callback( $callback );
+			}
+		}
+
+		$result              = empty( $actions ) ? 'None' : implode( ', ', $actions );
+		$cache[ $hook_name ] = $result;
+		return $cache[ $hook_name ];
+	}
+
+	/**
+	 * Formats a callback into a readable string.
+	 *
+	 * @param callable $callback The callback to format.
+	 * @return string A formatted string representing the callback.
+	 */
+	protected static function format_callback( $callback ) {
+		if ( is_string( $callback ) ) {
+			return $callback;
+		} elseif ( is_array( $callback ) && count( $callback ) === 2 ) {
+			/**
+			 * @var array{0: string, 1: string} $callback
+			 */
+			$class  = $callback[0];
+			$method = $callback[1];
+
+			if ( is_object( $class ) ) {
+				$class_name = get_class( $class );
+			} else {
+				$class_name = $class;
+			}
+
+			return $class_name . '::' . $method;
+		} elseif ( $callback instanceof \Closure ) {
+			return 'Closure';
+		} elseif ( is_object( $callback ) ) {
+			return get_class( $callback ) . '::__invoke';
+		}
+
+		// Fallback for unknown callback types
+		return 'Unknown';
+	}
 }