Merge pull request #27 from akirk/fix/clean-array-param-output

akirk · web-flow · commit 63dd3e9dbc2b · 2026-01-30T18:57:33.000+01:00
diff --git a/class-wphookextractor.php b/class-wphookextractor.php
@@ -145,7 +145,7 @@ public function extract_vars( $params, $tokens, $i ) {
 		$vars = array( '' );
 		$signature = $tokens[ $i ][1];
 		$line = $tokens[ $i ][2];
-		$search_window = 50;
+		$search_window = 100;
 		for ( $j = $i + 1; $j < $i + $search_window; $j++ ) {
 			if ( ! isset( $tokens[ $j ] ) ) {
 				break;
@@ -686,19 +686,24 @@ public function create_documentation_content( $hooks, $github_blob_url ) {
 					// Determine if this parameter should be optional (not used consistently across all files).
 					$is_optional = $i >= $consistent_param_count;
 
+					// Strip trailing { from array parameter descriptions (WordPress documentation style).
+					if ( isset( $p[2] ) ) {
+						$p[2] = rtrim( $p[2], ' {' );
+					}
+
 					if ( 'unknown' === $p[0] ) {
 						$params .= "\n- `{$p[1]}`";
 						if ( $is_optional ) {
 							$signature_params[] = $p[1] . ' = null';
 						} else {
 							$signature_params[] = $p[1];
 						}
-						if ( isset( $p[2] ) ) {
+						if ( ! empty( $p[2] ) ) {
 							$params .= ' ' . $p[2];
 						}
 					} else {
 						$params .= "\n- *`{$p[0]}`* `{$p[1]}`";
-						if ( isset( $p[2] ) ) {
+						if ( ! empty( $p[2] ) ) {
 							$params .= ' ' . $p[2];
 						}
 						if ( substr( $p[0], -5 ) === '|null' || $is_optional ) {
