Merge pull request #13 from wp-forge/codex-drop-error-log-path-tool

[codex] Drop error log path tool

Author: wpscholar

src/Tools/ErrorLogTools.php

@@ -23,10 +23,6 @@ trait ErrorLogTools {
 	 * @return void
 	 */
 	private function add_error_log_abilities() {
-		$this->add_ability( self::INTERNAL_PREFIX . 'get-error-log-path', 'Get Error Log Path', 'Get the WordPress debug log path used by this site', $this->schema(), function () {
-			return array( 'path' => $this->get_debug_log_path() );
-		}, true, 'manage_options' );
-
 		$this->add_ability( self::INTERNAL_PREFIX . 'read-error-log', 'Read Error Log', 'Read the tail of the WordPress debug log', $this->schema(
 			array(
 				'lines' => $this->int_prop( 'Number of log lines to return.', 200 ),

tests/integration/mcp-endpoint.mjs

@@ -218,7 +218,6 @@ const expectedTools = [
   'wp-forge-spam-comment',
   'wp-forge-get-site-health-info',
   'wp-forge-list-site-health-tests',
-  'wp-forge-get-error-log-path',
   'wp-forge-read-error-log',
   'wp-forge-run-wp-cli-command',
   'wp-forge-get-global-styles',
@@ -405,7 +404,6 @@ await expectSuccess('wp-forge-delete-comment', { id: commentId });
 const siteHealthInfo = await expectSuccess('wp-forge-get-site-health-info');
 assert(siteHealthInfo['wp-core'], 'wp-forge-get-site-health-info did not include wp-core debug data');
 await expectSuccess('wp-forge-list-site-health-tests');
-await expectSuccess('wp-forge-get-error-log-path');
 await expectSuccess('wp-forge-read-error-log', { lines: 5 });
 await expectError('wp-forge-run-wp-cli-command', { args: ['plugin', 'list'] }, 403);
 

tests/run.php

@@ -171,7 +171,7 @@ function assert_same( $expected, $actual, $message ) {
 $all       = $abilities->list_abilities();
 $names     = array_column( $all, 'name' );
 
-assert_same( 56, count( $all ), 'Expected the WordPress ability catalog.' );
+assert_same( 55, count( $all ), 'Expected the WordPress ability catalog.' );
 assert_true( in_array( 'wp-forge-search-content', $names, true ), 'Expected content search ability.' );
 assert_true( in_array( 'wp-forge-get-site-info', $names, true ), 'Expected site info ability.' );
 assert_true( in_array( 'wp-forge-run-api-function', $names, true ), 'Expected REST runner ability.' );
@@ -223,7 +223,6 @@ function assert_same( $expected, $actual, $message ) {
 	'wp-forge-spam-comment',
 	'wp-forge-get-site-health-info',
 	'wp-forge-list-site-health-tests',
-	'wp-forge-get-error-log-path',
 	'wp-forge-read-error-log',
 	'wp-forge-run-wp-cli-command',
 	'wp-forge-get-global-styles',
@@ -254,7 +253,7 @@ function assert_same( $expected, $actual, $message ) {
 
 $direct_tools = $abilities->list_tools();
 $direct_tool_names = array_column( $direct_tools, 'name' );
-assert_same( 56, count( $direct_tools ), 'Expected all abilities to be exposed as direct MCP tools.' );
+assert_same( 55, count( $direct_tools ), 'Expected all abilities to be exposed as direct MCP tools.' );
 assert_true( in_array( 'wp-forge-search-content', $direct_tool_names, true ), 'Direct tool list should include content search.' );
 assert_true( in_array( 'wp-forge-get-active-theme', $direct_tool_names, true ), 'Direct tool list should include active theme.' );
 assert_true( ! in_array( 'wp-forge-list-abilities', $direct_tool_names, true ), 'Gateway list tool should not be exposed.' );
@@ -296,11 +295,11 @@ function assert_same( $expected, $actual, $message ) {
 assert_same( array( 'page' ), array_column( $public_hierarchical_post_types['message']['post_types'], 'slug' ), 'Post type list should pass filters through to get_post_types().' );
 
 $wp_ability_names = $abilities->get_wordpress_ability_names();
-assert_same( 56, count( $wp_ability_names ), 'Expected all abilities to be available for the MCP adapter.' );
+assert_same( 55, count( $wp_ability_names ), 'Expected all abilities to be available for the MCP adapter.' );
 assert_true( in_array( 'wp-forge/search-content', $wp_ability_names, true ), 'Adapter ability list should use WordPress ability names.' );
 
 $abilities->register_wordpress_abilities();
-assert_same( 56, count( $registered_abilities ), 'Expected every ability to be registered with the WordPress Abilities API.' );
+assert_same( 55, count( $registered_abilities ), 'Expected every ability to be registered with the WordPress Abilities API.' );
 assert_true( isset( $registered_abilities['wp-forge/search-content'] ), 'Content search should be registered with the WordPress Abilities API.' );
 assert_same( 'Search and filter content for any registered post type', $registered_abilities['wp-forge/search-content']['description'], 'Registered ability should preserve descriptions.' );
 assert_same( true, $registered_abilities['wp-forge/search-content']['meta']['show_in_rest'], 'Registered abilities should be exposed through the Abilities REST API.' );
@@ -331,7 +330,7 @@ public function create_server() {
 assert_same( 'mcp', $adapter->args[1], 'Adapter server should keep the existing REST namespace.' );
 assert_same( 'wp-forge', $adapter->args[2], 'Adapter server should keep the existing REST route.' );
 assert_same( 'WordPress MCP', $adapter->args[3], 'Adapter server should preserve the server name.' );
-assert_same( 56, count( $adapter->args[9] ), 'Adapter server should expose every registered ability.' );
+assert_same( 55, count( $adapter->args[9] ), 'Adapter server should expose every registered ability.' );
 assert_true( in_array( 'wp-forge/search-content', $adapter->args[9], true ), 'Adapter server should expose content search.' );
 
 echo 'Tests passed: ' . $tests_run . PHP_EOL;