mcp-wordpress 1.2.2 → 1.3.0

package/src/types/wordpress.ts

@@ -1,6 +1,6 @@
 /**
  * Core WordPress REST API Types
- * 
+ *
  * Comprehensive TypeScript definitions for WordPress REST API v2 responses
  */
 
@@ -22,19 +22,37 @@ export interface WordPressLinks {
   about?: WordPressLink[];
   author?: WordPressLink[];
   replies?: WordPressLink[];
-  'version-history'?: WordPressLink[];
-  'predecessor-version'?: WordPressLink[];
-  'wp:attachment'?: WordPressLink[];
-  'wp:term'?: WordPressLink[];
-  'wp:featuredmedia'?: WordPressLink[];
+  "version-history"?: WordPressLink[];
+  "predecessor-version"?: WordPressLink[];
+  "wp:attachment"?: WordPressLink[];
+  "wp:term"?: WordPressLink[];
+  "wp:featuredmedia"?: WordPressLink[];
   curies?: WordPressLink[];
 }
 
 // Post Types
-export type PostStatus = 'publish' | 'future' | 'draft' | 'pending' | 'private' | 'trash' | 'auto-draft' | 'inherit';
-export type PostFormat = 'standard' | 'aside' | 'chat' | 'gallery' | 'link' | 'image' | 'quote' | 'status' | 'video' | 'audio';
-export type CommentStatus = 'open' | 'closed';
-export type PingStatus = 'open' | 'closed';
+export type PostStatus =
+  | "publish"
+  | "future"
+  | "draft"
+  | "pending"
+  | "private"
+  | "trash"
+  | "auto-draft"
+  | "inherit";
+export type PostFormat =
+  | "standard"
+  | "aside"
+  | "chat"
+  | "gallery"
+  | "link"
+  | "image"
+  | "quote"
+  | "status"
+  | "video"
+  | "audio";
+export type CommentStatus = "open" | "closed";
+export type PingStatus = "open" | "closed";
 
 export interface WordPressPost {
   id: number;
@@ -89,20 +107,23 @@ export interface WordPressPage {
 }
 
 // Media Types
-export type MediaType = 'image' | 'video' | 'text' | 'application' | 'audio';
+export type MediaType = "image" | "video" | "text" | "application" | "audio";
 
 export interface MediaDetails {
   width?: number;
   height?: number;
   file?: string;
   filesize?: number;
-  sizes?: Record<string, {
-    file: string;
-    width: number;
-    height: number;
-    mime_type: string;
-    source_url: string;
-  }>;
+  sizes?: Record<
+    string,
+    {
+      file: string;
+      width: number;
+      height: number;
+      mime_type: string;
+      source_url: string;
+    }
+  >;
   image_meta?: {
     aperture?: string;
     credit?: string;
@@ -148,7 +169,12 @@ export interface WordPressMedia {
 }
 
 // User Types
-export type UserRole = 'administrator' | 'editor' | 'author' | 'contributor' | 'subscriber';
+export type UserRole =
+  | "administrator"
+  | "editor"
+  | "author"
+  | "contributor"
+  | "subscriber";
 
 export interface WordPressUser {
   id: number;
@@ -173,7 +199,7 @@ export interface WordPressUser {
 }
 
 // Comment Types
-export type CommentType = 'comment' | 'trackback' | 'pingback';
+export type CommentType = "comment" | "trackback" | "pingback";
 
 export interface WordPressComment {
   id: number;
@@ -189,7 +215,7 @@ export interface WordPressComment {
   date_gmt: string;
   content: WordPressRendered;
   link: string;
-  status: 'approved' | 'unapproved' | 'spam' | 'trash';
+  status: "approved" | "unapproved" | "spam" | "trash";
   type: CommentType;
   author_avatar_urls: Record<string, string>;
   meta: Record<string, any>;
@@ -288,7 +314,7 @@ export interface PaginationParams {
   page?: number;
   per_page?: number;
   search?: string;
-  order?: 'asc' | 'desc';
+  order?: "asc" | "desc";
   orderby?: string;
 }
 
@@ -326,19 +352,19 @@ export interface MediaQueryParams extends PaginationParams {
 }
 
 export interface UserQueryParams extends PaginationParams {
-  context?: 'view' | 'embed' | 'edit';
+  context?: "view" | "embed" | "edit";
   exclude?: number[];
   include?: number[];
   offset?: number;
   slug?: string[];
   roles?: UserRole[];
   capabilities?: string[];
-  who?: 'authors';
+  who?: "authors";
   has_published_posts?: string[];
 }
 
 export interface CommentQueryParams extends PaginationParams {
-  context?: 'view' | 'embed' | 'edit';
+  context?: "view" | "embed" | "edit";
   after?: string;
   author?: number[];
   author_exclude?: number[];
@@ -420,7 +446,8 @@ export interface CreateUserRequest {
   meta?: Record<string, any>;
 }
 
-export interface UpdateUserRequest extends Partial<Omit<CreateUserRequest, 'username'>> {
+export interface UpdateUserRequest
+  extends Partial<Omit<CreateUserRequest, "username">> {
   id: number;
 }
 
@@ -434,13 +461,14 @@ export interface CreateCommentRequest {
   author_url?: string;
   date?: string;
   date_gmt?: string;
-  status?: 'approved' | 'unapproved';
+  status?: "approved" | "unapproved";
   meta?: Record<string, any>;
 }
 
-export interface UpdateCommentRequest extends Partial<Omit<CreateCommentRequest, 'status'>> {
+export interface UpdateCommentRequest
+  extends Partial<Omit<CreateCommentRequest, "status">> {
   id: number;
-  status?: 'approved' | 'unapproved' | 'spam' | 'trash';
+  status?: "approved" | "unapproved" | "spam" | "trash";
 }
 
 export interface CreateCategoryRequest {

package/src/utils/debug.ts

@@ -5,10 +5,10 @@
  * This prevents console.log from interfering with MCP STDIO communication.
  */
 
-import type { DebugInfo } from '../types/index.js';
+import type { DebugInfo } from "../types/index.js";
 
 // Log levels
-export type LogLevel = 'debug' | 'info' | 'warn' | 'error';
+export type LogLevel = "debug" | "info" | "warn" | "error";
 
 // Logger interface
 export interface Logger {
@@ -26,8 +26,8 @@ export interface StructuredLogger extends Logger {
 
 // Check if debug mode is enabled
 const isDebugMode = (): boolean =>
-  (process.env.DEBUG === 'true' || process.env.NODE_ENV === 'development') &&
-  process.env.NODE_ENV !== 'test';
+  (process.env.DEBUG === "true" || process.env.NODE_ENV === "development") &&
+  process.env.NODE_ENV !== "test";
 
 // Get current timestamp
 const getTimestamp = (): string => new Date().toISOString();
@@ -38,9 +38,9 @@ const formatMessage = (level: LogLevel, args: any[]): string => {
   const prefix = `[${timestamp}] [${level.toUpperCase()}]`;
   return `${prefix} ${args
     .map((arg) =>
-      typeof arg === 'object' ? JSON.stringify(arg, null, 2) : String(arg)
+      typeof arg === "object" ? JSON.stringify(arg, null, 2) : String(arg),
     )
-    .join(' ')}`;
+    .join(" ")}`;
 };
 
 // Handle circular references in objects
@@ -53,15 +53,15 @@ const safeStringify = (obj: any): string => {
           return {
             name: value.name,
             message: value.message,
-            stack: value.stack
+            stack: value.stack,
           };
         }
         return value;
       },
-      2
+      2,
     );
   } catch (_error) {
-    return '[Object with circular reference]';
+    return "[Object with circular reference]";
   }
 };
 
@@ -71,27 +71,27 @@ const safeStringify = (obj: any): string => {
 export const debug: Logger = {
   log: (...args: any[]): void => {
     if (isDebugMode()) {
-      console.error(formatMessage('debug', args)); // Use stderr for debug to avoid STDIO interference
+      console.error(formatMessage("debug", args)); // Use stderr for debug to avoid STDIO interference
     }
   },
 
   info: (...args: any[]): void => {
     if (isDebugMode()) {
-      console.error(formatMessage('info', args));
+      console.error(formatMessage("info", args));
     }
   },
 
   warn: (...args: any[]): void => {
     if (isDebugMode()) {
-      console.error(formatMessage('warn', args));
+      console.error(formatMessage("warn", args));
     }
   },
 
   error: (...args: any[]): void => {
     if (isDebugMode()) {
-      console.error(formatMessage('error', args));
+      console.error(formatMessage("error", args));
     }
-  }
+  },
 };
 
 /**
@@ -101,7 +101,7 @@ export const silent: Logger = {
   log: (): void => {},
   warn: (): void => {},
   error: (): void => {},
-  info: (): void => {}
+  info: (): void => {},
 };
 
 /**
@@ -124,29 +124,29 @@ class StructuredLoggerImpl implements StructuredLogger {
       level,
       message: args
         .map((arg) =>
-          typeof arg === 'object' ? safeStringify(arg) : String(arg)
+          typeof arg === "object" ? safeStringify(arg) : String(arg),
         )
-        .join(' '),
-      ...(Object.keys(this.context).length > 0 && { context: this.context })
+        .join(" "),
+      ...(Object.keys(this.context).length > 0 && { context: this.context }),
     };
 
     console.error(safeStringify(debugInfo));
   }
 
   log(...args: any[]): void {
-    this.output('debug', args);
+    this.output("debug", args);
   }
 
   info(...args: any[]): void {
-    this.output('info', args);
+    this.output("info", args);
   }
 
   warn(...args: any[]): void {
-    this.output('warn', args);
+    this.output("warn", args);
   }
 
   error(...args: any[]): void {
-    this.output('error', args);
+    this.output("error", args);
   }
 
   logStructured(info: DebugInfo): void {
@@ -154,7 +154,7 @@ class StructuredLoggerImpl implements StructuredLogger {
 
     const enhancedInfo: DebugInfo = {
       ...info,
-      context: { ...this.context, ...info.context }
+      context: { ...this.context, ...info.context },
     };
 
     console.error(safeStringify(enhancedInfo));
@@ -163,7 +163,7 @@ class StructuredLoggerImpl implements StructuredLogger {
   child(context: Record<string, any>): StructuredLogger {
     return new StructuredLoggerImpl(
       { ...this.context, ...context },
-      this.enabled
+      this.enabled,
     );
   }
 }
@@ -172,7 +172,7 @@ class StructuredLoggerImpl implements StructuredLogger {
  * Create a structured logger instance
  */
 export const createStructuredLogger = (
-  context: Record<string, any> = {}
+  context: Record<string, any> = {},
 ): StructuredLogger => {
   return new StructuredLoggerImpl(context);
 };
@@ -186,7 +186,7 @@ export const logger: Logger = debug;
  * Create a logger with context
  */
 export const createLogger = (
-  context: Record<string, any> = {}
+  context: Record<string, any> = {},
 ): StructuredLogger => {
   return createStructuredLogger(context);
 };
@@ -207,13 +207,13 @@ export const startTimer = (label?: string): PerformanceTimer => {
       return Date.now() - start;
     },
 
-    endWithLog(message = 'Operation'): number {
+    endWithLog(message = "Operation"): number {
       const duration = Date.now() - start;
       debug.info(
-        `${message} completed in ${duration}ms${label ? ` [${label}]` : ''}`
+        `${message} completed in ${duration}ms${label ? ` [${label}]` : ""}`,
       );
       return duration;
-    }
+    },
   };
 };
 
@@ -222,14 +222,14 @@ export const startTimer = (label?: string): PerformanceTimer => {
  */
 export const logError = (
   error: Error | string,
-  context?: Record<string, any>
+  context?: Record<string, any>,
 ): void => {
-  if (typeof error === 'string') {
+  if (typeof error === "string") {
     debug.error(error, context);
   } else {
     debug.error(`${error.name}: ${error.message}`, {
       stack: error.stack,
-      ...context
+      ...context,
     });
   }
 };
@@ -237,14 +237,14 @@ export const logError = (
 /**
  * Conditional logging
  */
-export const logIf = (condition: boolean, level: LogLevel = 'debug') => {
+export const logIf = (condition: boolean, level: LogLevel = "debug") => {
   if (!condition) return silent;
 
   const loggers = {
     log: debug.log,
     info: debug.info,
     warn: debug.warn,
-    error: debug.error
+    error: debug.error,
   };
 
   return loggers[level as keyof typeof loggers] || debug.log;
@@ -255,12 +255,12 @@ export const logIf = (condition: boolean, level: LogLevel = 'debug') => {
  */
 export const getEnvVar = (
   key: string,
-  defaultValue?: string
+  defaultValue?: string,
 ): string | undefined => {
   const value = process.env[key];
   if (value === undefined && defaultValue !== undefined) {
     debug.warn(
-      `Environment variable ${key} not found, using default: ${defaultValue}`
+      `Environment variable ${key} not found, using default: ${defaultValue}`,
     );
     return defaultValue;
   }
@@ -274,7 +274,7 @@ export const validateEnvVars = (required: string[]): void => {
   const missing = required.filter((key) => !process.env[key]);
   if (missing.length > 0) {
     const error = new Error(
-      `Missing required environment variables: ${missing.join(', ')}`
+      `Missing required environment variables: ${missing.join(", ")}`,
     );
     logError(error);
     throw error;

package/src/utils/error.ts

@@ -6,16 +6,16 @@ export function getErrorMessage(error: unknown): string {
   if (error instanceof Error) {
     return error.message;
   }
-  
-  if (typeof error === 'string') {
+
+  if (typeof error === "string") {
     return error;
   }
-  
-  if (error && typeof error === 'object' && 'message' in error) {
+
+  if (error && typeof error === "object" && "message" in error) {
     return String(error.message);
   }
-  
-  return 'Unknown error occurred';
+
+  return "Unknown error occurred";
 }
 
 export function isError(error: unknown): error is Error {
@@ -23,37 +23,49 @@ export function isError(error: unknown): error is Error {
 }
 
 export function logAndReturn<T>(error: unknown, defaultValue: T): T {
-  console.error('Error occurred:', getErrorMessage(error));
+  console.error("Error occurred:", getErrorMessage(error));
   return defaultValue;
 }
 
 /**
  * Enhanced error handler for consistent tool error handling
  */
-export function handleToolError(error: unknown, operation: string, context?: Record<string, unknown>): never {
+export function handleToolError(
+  error: unknown,
+  operation: string,
+  context?: Record<string, unknown>,
+): never {
   console.error(`Error in ${operation}:`, error);
-  
+
   if (context) {
-    console.error('Context:', context);
+    console.error("Context:", context);
   }
 
   const message = getErrorMessage(error);
-  
+
   // Provide more specific error messages based on error content
-  if (message.includes('ECONNREFUSED') || message.includes('ENOTFOUND')) {
-    throw new Error(`Connection failed during ${operation}. Please check your WordPress site URL and network connection.`);
+  if (message.includes("ECONNREFUSED") || message.includes("ENOTFOUND")) {
+    throw new Error(
+      `Connection failed during ${operation}. Please check your WordPress site URL and network connection.`,
+    );
   }
 
-  if (message.includes('401') || message.includes('Unauthorized')) {
-    throw new Error(`Authentication failed during ${operation}. Please check your WordPress credentials.`);
+  if (message.includes("401") || message.includes("Unauthorized")) {
+    throw new Error(
+      `Authentication failed during ${operation}. Please check your WordPress credentials.`,
+    );
   }
 
-  if (message.includes('403') || message.includes('Forbidden')) {
-    throw new Error(`Permission denied during ${operation}. Please check your user permissions.`);
+  if (message.includes("403") || message.includes("Forbidden")) {
+    throw new Error(
+      `Permission denied during ${operation}. Please check your user permissions.`,
+    );
   }
 
-  if (message.includes('429') || message.includes('Too Many Requests')) {
-    throw new Error(`Rate limit exceeded during ${operation}. Please try again later.`);
+  if (message.includes("429") || message.includes("Too Many Requests")) {
+    throw new Error(
+      `Rate limit exceeded during ${operation}. Please try again later.`,
+    );
   }
 
   throw new Error(`Failed to ${operation}: ${message}`);
@@ -62,26 +74,36 @@ export function handleToolError(error: unknown, operation: string, context?: Rec
 /**
  * Validates required parameters
  */
-export function validateRequired(params: Record<string, unknown>, required: string[]): void {
-  const missing = required.filter(key => !params[key]);
+export function validateRequired(
+  params: Record<string, unknown>,
+  required: string[],
+): void {
+  const missing = required.filter((key) => !params[key]);
   if (missing.length > 0) {
-    throw new Error(`Missing required parameters: ${missing.join(', ')}`);
+    throw new Error(`Missing required parameters: ${missing.join(", ")}`);
   }
 }
 
 /**
  * Validates site parameter for multi-site configurations
  */
-export function validateSite(site: string | undefined, availableSites: string[]): string {
+export function validateSite(
+  site: string | undefined,
+  availableSites: string[],
+): string {
   if (!site) {
     if (availableSites.length === 1) {
       return availableSites[0];
     }
-    throw new Error(`Site parameter is required when multiple sites are configured. Available sites: ${availableSites.join(', ')}`);
+    throw new Error(
+      `Site parameter is required when multiple sites are configured. Available sites: ${availableSites.join(", ")}`,
+    );
   }
 
   if (!availableSites.includes(site)) {
-    throw new Error(`Site '${site}' not found. Available sites: ${availableSites.join(', ')}`);
+    throw new Error(
+      `Site '${site}' not found. Available sites: ${availableSites.join(", ")}`,
+    );
   }
 
   return site;

package/src/utils/toolWrapper.ts

@@ -3,14 +3,14 @@
  * Standardizes error handling and reduces repetitive try-catch blocks
  */
 
-import { getErrorMessage } from './error.js';
+import { getErrorMessage } from "./error.js";
 
 /**
  * Wrapper for tool methods that standardizes error handling
  */
 export function withErrorHandling<T extends any[], R>(
   operation: string,
-  fn: (...args: T) => Promise<R>
+  fn: (...args: T) => Promise<R>,
 ): (...args: T) => Promise<R> {
   return async (...args: T): Promise<R> => {
     try {
@@ -27,7 +27,7 @@ export function withErrorHandling<T extends any[], R>(
 export function withValidation<T extends any[], R>(
   operation: string,
   validator: (...args: T) => void,
-  fn: (...args: T) => Promise<R>
+  fn: (...args: T) => Promise<R>,
 ): (...args: T) => Promise<R> {
   return async (...args: T): Promise<R> => {
     try {
@@ -45,18 +45,18 @@ export function withValidation<T extends any[], R>(
 export const validators = {
   requireSite: (client: any, params: any) => {
     if (!client) {
-      throw new Error('WordPress client is required');
+      throw new Error("WordPress client is required");
     }
   },
 
   requireId: (params: { id?: number | string }) => {
     if (!params.id) {
-      throw new Error('ID parameter is required');
+      throw new Error("ID parameter is required");
     }
   },
 
   requireNonEmpty: (value: any, fieldName: string) => {
-    if (!value || (typeof value === 'string' && value.trim() === '')) {
+    if (!value || (typeof value === "string" && value.trim() === "")) {
       throw new Error(`${fieldName} cannot be empty`);
     }
   },
@@ -67,14 +67,18 @@ export const validators = {
         throw new Error(`${field} is required`);
       }
     }
-  }
+  },
 };
 
 /**
  * Decorator for class methods to add error handling
  */
 export function errorHandler(operation: string) {
-  return function (target: any, propertyKey: string, descriptor: PropertyDescriptor) {
+  return function (
+    target: any,
+    propertyKey: string,
+    descriptor: PropertyDescriptor,
+  ) {
     const originalMethod = descriptor.value;
 
     descriptor.value = async function (...args: any[]) {