mcp-wordpress 3.1.11 → 3.1.13

package/src/tools/pages.ts

@@ -1,4 +1,5 @@
 import { WordPressClient } from "@/client/api.js";
+import type { MCPToolSchema } from "@/types/mcp.js";
 import { CreatePageRequest, PostQueryParams as PageQueryParams, UpdatePageRequest } from "@/types/wordpress.js";
 import { getErrorMessage } from "@/utils/error.js";
 import { toolParams } from "./params.js";
@@ -15,140 +16,137 @@ export class PageTools {
   public getTools(): Array<{
     name: string;
     description: string;
-    parameters?: Array<{
-      name: string;
-      type?: string;
-      description?: string;
-      required?: boolean;
-      enum?: string[];
-      items?: unknown;
-    }>;
+    inputSchema?: MCPToolSchema;
     handler: (client: WordPressClient, params: Record<string, unknown>) => Promise<unknown>;
   }> {
     return [
       {
         name: "wp_list_pages",
         description: "Lists pages from a WordPress site, with filters.",
-        parameters: [
-          {
-            name: "per_page",
-            type: "number",
-            description: "Number of items to return per page (max 100).",
+        inputSchema: {
+          type: "object",
+          properties: {
+            per_page: {
+              type: "number",
+              description: "Number of items to return per page (max 100).",
+            },
+            search: {
+              type: "string",
+              description: "Limit results to those matching a search term.",
+            },
+            status: {
+              type: "string",
+              description: "Filter by page status.",
+              enum: ["publish", "future", "draft", "pending", "private"],
+            },
           },
-          {
-            name: "search",
-            type: "string",
-            description: "Limit results to those matching a search term.",
-          },
-          {
-            name: "status",
-            type: "string",
-            description: "Filter by page status.",
-            enum: ["publish", "future", "draft", "pending", "private"],
-          },
-        ],
+          required: [],
+        },
         handler: this.handleListPages.bind(this),
       },
       {
         name: "wp_get_page",
         description: "Retrieves a single page by its ID, optionally including full content for editing.",
-        parameters: [
-          {
-            name: "id",
-            type: "number",
-            required: true,
-            description: "The unique identifier for the page.",
+        inputSchema: {
+          type: "object",
+          properties: {
+            id: {
+              type: "number",
+              description: "The unique identifier for the page.",
+            },
+            include_content: {
+              type: "boolean",
+              description: "If true, includes the full HTML content of the page. Default: false",
+            },
           },
-          {
-            name: "include_content",
-            type: "boolean",
-            description: "If true, includes the full HTML content of the page. Default: false",
-          },
-        ],
+          required: ["id"],
+        },
         handler: this.handleGetPage.bind(this),
       },
       {
         name: "wp_create_page",
         description: "Creates a new page.",
-        parameters: [
-          {
-            name: "title",
-            type: "string",
-            required: true,
-            description: "The title for the page.",
-          },
-          {
-            name: "content",
-            type: "string",
-            description: "The content for the page, in HTML format.",
+        inputSchema: {
+          type: "object",
+          properties: {
+            title: {
+              type: "string",
+              description: "The title for the page.",
+            },
+            content: {
+              type: "string",
+              description: "The content for the page, in HTML format.",
+            },
+            status: {
+              type: "string",
+              description: "The publishing status for the page.",
+              enum: ["publish", "draft", "pending", "private"],
+            },
           },
-          {
-            name: "status",
-            type: "string",
-            description: "The publishing status for the page.",
-            enum: ["publish", "draft", "pending", "private"],
-          },
-        ],
+          required: ["title"],
+        },
         handler: this.handleCreatePage.bind(this),
       },
       {
         name: "wp_update_page",
         description: "Updates an existing page.",
-        parameters: [
-          {
-            name: "id",
-            type: "number",
-            required: true,
-            description: "The ID of the page to update.",
-          },
-          {
-            name: "title",
-            type: "string",
-            description: "The new title for the page.",
-          },
-          {
-            name: "content",
-            type: "string",
-            description: "The new content for the page, in HTML format.",
+        inputSchema: {
+          type: "object",
+          properties: {
+            id: {
+              type: "number",
+              description: "The ID of the page to update.",
+            },
+            title: {
+              type: "string",
+              description: "The new title for the page.",
+            },
+            content: {
+              type: "string",
+              description: "The new content for the page, in HTML format.",
+            },
+            status: {
+              type: "string",
+              description: "The new status for the page.",
+              enum: ["publish", "draft", "pending", "private"],
+            },
           },
-          {
-            name: "status",
-            type: "string",
-            description: "The new status for the page.",
-            enum: ["publish", "draft", "pending", "private"],
-          },
-        ],
+          required: ["id"],
+        },
         handler: this.handleUpdatePage.bind(this),
       },
       {
         name: "wp_delete_page",
         description: "Deletes a page.",
-        parameters: [
-          {
-            name: "id",
-            type: "number",
-            required: true,
-            description: "The ID of the page to delete.",
-          },
-          {
-            name: "force",
-            type: "boolean",
-            description: "If true, permanently delete. If false, move to trash. Defaults to false.",
+        inputSchema: {
+          type: "object",
+          properties: {
+            id: {
+              type: "number",
+              description: "The ID of the page to delete.",
+            },
+            force: {
+              type: "boolean",
+              description: "If true, permanently delete. If false, move to trash. Defaults to false.",
+            },
           },
-        ],
+          required: ["id"],
+        },
         handler: this.handleDeletePage.bind(this),
       },
       {
         name: "wp_get_page_revisions",
         description: "Retrieves revisions for a specific page.",
-        parameters: [
-          {
-            name: "id",
-            type: "number",
-            required: true,
-            description: "The ID of the page to get revisions for.",
+        inputSchema: {
+          type: "object",
+          properties: {
+            id: {
+              type: "number",
+              description: "The ID of the page to get revisions for.",
+            },
           },
-        ],
+          required: ["id"],
+        },
         handler: this.handleGetPageRevisions.bind(this),
       },
     ];
@@ -214,8 +212,21 @@ export class PageTools {
   public async handleDeletePage(client: WordPressClient, params: Record<string, unknown>): Promise<unknown> {
     const { id, force } = params as { id: number; force?: boolean };
     try {
-      await client.deletePage(id, force);
-      const action = params.force ? "permanently deleted" : "moved to trash";
+      const result = await client.deletePage(id, force);
+      const action = force ? "permanently deleted" : "moved to trash";
+
+      if (result?.deleted === false) {
+        throw new Error(
+          `WordPress refused to delete page ${id}. The page may be protected or the operation was rejected.`,
+        );
+      }
+
+      if (result?.deleted) {
+        const title = result.previous?.title?.rendered;
+        return title ? `✅ Page "${title}" has been ${action}.` : `✅ Page ${id} has been ${action}.`;
+      }
+
+      // Some WordPress installations return empty/null responses on successful deletion
       return `✅ Page ${id} has been ${action}.`;
     } catch (_error) {
       throw new Error(`Failed to delete page: ${getErrorMessage(_error)}`);

package/src/tools/performance/PerformanceTools.ts

@@ -99,7 +99,10 @@ export default class PerformanceTools {
     return [
       {
         name: "wp_performance_stats",
-        description: "Get real-time performance statistics and metrics",
+        description:
+          "Get real-time performance statistics and metrics. " +
+          "Note: Top-level metrics (totalRequests, averageResponseTime, errorRate) are session-wide aggregates across all sites. " +
+          "Per-site cache and client stats are shown in the siteSpecific section when a site parameter is provided.",
         parameters: [
           {
             name: "site",
@@ -156,7 +159,9 @@ export default class PerformanceTools {
       },
       {
         name: "wp_performance_benchmark",
-        description: "Compare current performance against industry benchmarks",
+        description:
+          "Compare current performance against industry benchmarks. " +
+          "Note: Benchmarks are based on session-wide aggregated metrics across all sites, not per-site metrics.",
         parameters: [
           {
             name: "site",
@@ -318,6 +323,7 @@ export default class PerformanceTools {
 
       if (category === "overview" || category === "all") {
         result.overview = {
+          scope: site ? "session-wide (all sites combined)" : "session-wide",
           overallHealth: calculateHealthStatus(metrics),
           performanceScore: calculatePerformanceScore(metrics),
           totalRequests: metrics.requests.total,

package/src/tools/seo/auditors/SiteAuditor.ts

@@ -237,8 +237,8 @@ export class SiteAuditor {
     const posts = await this.client.getPosts({ per_page: this.config.maxPagesForContentAudit, status: ["publish"] });
     const pages = await this.client.getPages({ per_page: this.config.maxPagesForContentAudit, status: ["publish"] });
 
-    // Get site info (mock for now)
-    const siteUrl = "https://example.com"; // Would come from WordPress REST API
+    // Get site URL from the WordPress client configuration
+    const siteUrl = this.client.getSiteUrl();
 
     return {
       siteUrl,
@@ -599,9 +599,22 @@ export class SiteAuditor {
     }
 
     // Check for external dependencies (basic analysis)
+    let siteHostname: string;
+    try {
+      siteHostname = new URL(siteData.siteUrl).hostname;
+    } catch {
+      siteHostname = siteData.siteUrl.replace(/^https?:\/\//, "").replace(/[/:].*/g, "");
+    }
     const externalDependencies = [...siteData.posts, ...siteData.pages].reduce((count, item) => {
       const content = item.content?.rendered || "";
-      const externalLinks = content.match(/https?:\/\/(?!example\.com)[^"'\s>]*/gi) || [];
+      const externalLinks =
+        content.match(/https?:\/\/[^"'\s>]*/gi)?.filter((url) => {
+          try {
+            return new URL(url).hostname !== siteHostname;
+          } catch {
+            return true;
+          }
+        }) || [];
       return count + externalLinks.length;
     }, 0);
 

package/src/tools/site.ts

@@ -1,4 +1,5 @@
 import { WordPressClient } from "@/client/api.js";
+import type { MCPToolSchema } from "@/types/mcp.js";
 import { WordPressApplicationPassword } from "@/types/wordpress.js";
 import { getErrorMessage } from "@/utils/error.js";
 
@@ -14,43 +15,41 @@ export class SiteTools {
   public getTools(): Array<{
     name: string;
     description: string;
-    parameters?: Array<{
-      name: string;
-      type?: string;
-      description?: string;
-      required?: boolean;
-      enum?: string[];
-      items?: unknown;
-    }>;
+    inputSchema?: MCPToolSchema;
     handler: (client: WordPressClient, params: Record<string, unknown>) => Promise<unknown>;
   }> {
     return [
       {
         name: "wp_get_site_settings",
-        description: "Retrieves the general settings for a WordPress site.",
-        parameters: [],
+        description:
+          "Retrieves the general settings for a WordPress site. Requires administrator role (manage_options capability).",
+        inputSchema: {
+          type: "object",
+          properties: {},
+        },
         handler: this.handleGetSiteSettings.bind(this),
       },
       {
         name: "wp_update_site_settings",
-        description: "Updates one or more general settings for a WordPress site.",
-        parameters: [
-          {
-            name: "title",
-            type: "string",
-            description: "The title of the site.",
-          },
-          {
-            name: "description",
-            type: "string",
-            description: "The tagline or description of the site.",
-          },
-          {
-            name: "timezone",
-            type: "string",
-            description: "A city in the same timezone, e.g., 'America/New_York'.",
+        description:
+          "Updates one or more general settings for a WordPress site. Requires administrator role (manage_options capability).",
+        inputSchema: {
+          type: "object",
+          properties: {
+            title: {
+              type: "string",
+              description: "The title of the site.",
+            },
+            description: {
+              type: "string",
+              description: "The tagline or description of the site.",
+            },
+            timezone: {
+              type: "string",
+              description: "A city in the same timezone, e.g., 'America/New_York'.",
+            },
           },
-        ],
+        },
         handler: this.handleUpdateSiteSettings.bind(this),
       },
       {
@@ -63,71 +62,74 @@ export class SiteTools {
           '• Search pages: `wp_search_site --term="about" --type="pages"`\n' +
           '• Search media: `wp_search_site --term="logo" --type="media"`\n' +
           '• Find specific content: `wp_search_site --term="contact form"`',
-        parameters: [
-          {
-            name: "term",
-            type: "string",
-            required: true,
-            description: "The search term to look for.",
+        inputSchema: {
+          type: "object",
+          properties: {
+            term: {
+              type: "string",
+              description: "The search term to look for.",
+            },
+            type: {
+              type: "string",
+              description: "The type of content to search.",
+              enum: ["posts", "pages", "media"],
+            },
           },
-          {
-            name: "type",
-            type: "string",
-            description: "The type of content to search.",
-            enum: ["posts", "pages", "media"],
-          },
-        ],
+          required: ["term"],
+        },
         handler: this.handleSearchSite.bind(this),
       },
       {
         name: "wp_get_application_passwords",
         description: "Lists application passwords for a specific user.",
-        parameters: [
-          {
-            name: "user_id",
-            type: "number",
-            required: true,
-            description: "The ID of the user to get application passwords for.",
+        inputSchema: {
+          type: "object",
+          properties: {
+            user_id: {
+              type: "number",
+              description: "The ID of the user to get application passwords for.",
+            },
           },
-        ],
+          required: ["user_id"],
+        },
         handler: this.handleGetApplicationPasswords.bind(this),
       },
       {
         name: "wp_create_application_password",
         description: "Creates a new application password for a user.",
-        parameters: [
-          {
-            name: "user_id",
-            type: "number",
-            required: true,
-            description: "The ID of the user to create the password for.",
-          },
-          {
-            name: "app_name",
-            type: "string",
-            required: true,
-            description: "The name of the application this password is for.",
+        inputSchema: {
+          type: "object",
+          properties: {
+            user_id: {
+              type: "number",
+              description: "The ID of the user to create the password for.",
+            },
+            app_name: {
+              type: "string",
+              description: "The name of the application this password is for.",
+            },
           },
-        ],
+          required: ["user_id", "app_name"],
+        },
         handler: this.handleCreateApplicationPassword.bind(this),
       },
       {
         name: "wp_delete_application_password",
         description: "Revokes an existing application password.",
-        parameters: [
-          {
-            name: "user_id",
-            type: "number",
-            required: true,
-            description: "The ID of the user who owns the password.",
-          },
-          {
-            name: "uuid",
-            type: "string",
-            required: true,
-            description: "The UUID of the application password to revoke.",
+        inputSchema: {
+          type: "object",
+          properties: {
+            user_id: {
+              type: "number",
+              description: "The ID of the user who owns the password.",
+            },
+            uuid: {
+              type: "string",
+              description: "The UUID of the application password to revoke.",
+            },
           },
-        ],
+          required: ["user_id", "uuid"],
+        },
         handler: this.handleDeleteApplicationPassword.bind(this),
       },
     ];