@wisewandtools/mcp-server 2.2.0 → 2.2.3

package/dist/index.js

@@ -1169,6 +1169,63 @@ var HealthChecker = class {
 
 // src/handlers/tools/ArticleToolHandler.ts
 import { z as z2 } from "zod";
+
+// src/utils/tool-helpers.ts
+var UUID_REGEX = /^[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}$/i;
+var ToolInputError = class extends Error {
+  static {
+    __name(this, "ToolInputError");
+  }
+  constructor(message) {
+    super(message);
+    this.name = "ToolInputError";
+  }
+};
+function resolveId(args, ...fallbackKeys) {
+  const value = args.id ?? args.Id ?? args.ID;
+  if (value !== void 0 && value !== null) {
+    return validateUuid(value);
+  }
+  for (const key of fallbackKeys) {
+    if (args[key] !== void 0 && args[key] !== null) {
+      return validateUuid(args[key]);
+    }
+  }
+  const tried = ["id", ...fallbackKeys].join(", ");
+  throw new ToolInputError(`Missing required ID. Provide one of: ${tried}`);
+}
+__name(resolveId, "resolveId");
+function validateUuid(value) {
+  const str = String(value);
+  if (!UUID_REGEX.test(str)) {
+    throw new ToolInputError(`Invalid ID format: "${str}" is not a valid UUID`);
+  }
+  return str;
+}
+__name(validateUuid, "validateUuid");
+function resolveArray(args, key) {
+  let value = args[key];
+  if (value === void 0 || value === null) {
+    throw new ToolInputError(`Missing required parameter: ${key}`);
+  }
+  if (typeof value === "string") {
+    try {
+      value = JSON.parse(value);
+    } catch {
+      throw new ToolInputError(`Invalid ${key} format: expected JSON array`);
+    }
+  }
+  if (!Array.isArray(value)) {
+    throw new ToolInputError(`Invalid ${key} format: expected array, got ${typeof value}`);
+  }
+  if (value.length === 0) {
+    throw new ToolInputError(`${key} must not be empty`);
+  }
+  return value;
+}
+__name(resolveArray, "resolveArray");
+
+// src/handlers/tools/ArticleToolHandler.ts
 var boolField = z2.union([z2.boolean(), z2.string()]).transform(
   (val) => typeof val === "string" ? val === "true" : val
 ).optional().default(false);
@@ -1465,7 +1522,7 @@ var ArticleToolHandler = class {
       publishwordpress_date: { type: "string", description: "WordPress publish date (ISO 8601)" },
       publishwordpress_status: { type: "string", enum: ["draft", "publish", "future"], description: "WordPress post status", default: "draft" },
       publishwordpress_category: { type: "number", description: "WordPress category ID" },
-      publishwordpress_author: { type: "number", description: "WordPress author ID" },
+      publishwordpress_author: { type: "number", description: "WordPress author ID. CAUTION: If the WP connection user lacks edit_others_posts capability, setting this will cause publishing to fail with rest_cannot_edit_others. Omit unless the WP user has admin/editor role." },
       publishwordpress_connection: { type: "string", format: "uuid", description: "WordPress connection ID" },
       // Shopify Publishing
       use_publishshopify: { type: "boolean", description: "Automatically publish to Shopify" },
@@ -1537,21 +1594,38 @@ var ArticleToolHandler = class {
           const validated = CreateArticleSchema.parse(args);
           const apiInput = this.buildApiInput(validated);
           const article = await this.apiClient.createArticle(apiInput);
-          await this.cache.set(`article:${article.id}`, article, 300);
           this.metrics.recordAPICall("create_article", "success");
+          const warnings = [];
+          if (validated.use_publishwordpress && !validated.publishwordpress_connection) {
+            warnings.push("Warning: use_publishwordpress is true but publishwordpress_connection is not set. The article may not be published to WordPress. Provide a publishwordpress_connection UUID.");
+          }
+          if (validated.publishwordpress_author) {
+            warnings.push(
+              'Warning: publishwordpress_author is set. If the WP connection user lacks edit_others_posts capability, auto-publish will silently fail with "rest_cannot_edit_others". Omit publishwordpress_author unless the WP user has admin/editor role.'
+            );
+          }
+          const hasAutoPublish = validated.use_publishwordpress || validated.use_publishshopify || validated.use_publishprestashop || validated.use_publishwoocommerce;
+          const next_steps = hasAutoPublish ? [
+            `Generation auto-triggered. Use 'get_article' with id: ${article.id} to check progress`,
+            `Use 'get_article_output' with id: ${article.id} to get content once ready`
+          ] : [
+            `Use 'generate_article' with id: ${article.id} to start content generation`,
+            `Use 'get_article' with id: ${article.id} to check status`
+          ];
+          const response = {
+            success: true,
+            article_id: article.id,
+            title: article.title || "Article created",
+            message: `Article "${article.title || validated.subject}" created successfully.`,
+            next_steps
+          };
+          if (warnings.length > 0) {
+            response.warnings = warnings;
+          }
           return {
             content: [{
               type: "text",
-              text: JSON.stringify({
-                success: true,
-                article_id: article.id,
-                title: article.title || "Article created",
-                message: `Article "${article.title || validated.subject}" created successfully.`,
-                next_steps: [
-                  `Use 'generate_article' with article_id: ${article.id} to start content generation`,
-                  `Use 'get_article' with article_id: ${article.id} to check status`
-                ]
-              }, null, 2)
+              text: JSON.stringify(response, null, 2)
             }]
           };
         } catch (error) {
@@ -1572,15 +1646,16 @@ var ArticleToolHandler = class {
       inputSchema: {
         type: "object",
         properties: {
-          article_id: { type: "string", format: "uuid", description: "ID of the article to generate" },
+          id: { type: "string", format: "uuid", description: "ID of the article to generate" },
           wait_for_completion: { type: "boolean", description: "Wait for generation to complete (may take several minutes)", default: true },
           max_wait_time: { type: "number", description: "Maximum time to wait in seconds", default: 300, maximum: 600 }
         },
-        required: ["article_id"]
+        required: ["id"]
       },
       handler: /* @__PURE__ */ __name(async (args) => {
         try {
-          const { article_id, wait_for_completion = true, max_wait_time = 300 } = args;
+          const article_id = resolveId(args, "article_id");
+          const { wait_for_completion = true, max_wait_time = 300 } = args;
           await this.apiClient.runArticleGeneration(article_id);
           if (!wait_for_completion) {
             return { content: [{ type: "text", text: JSON.stringify({ success: true, message: "Article generation started", article_id, next_step: `Use 'get_article' or 'get_article_output' to check status` }, null, 2) }] };
@@ -1618,6 +1693,22 @@ var ArticleToolHandler = class {
           }
           throw new Error(`Generation timeout after ${max_wait_time} seconds`);
         } catch (error) {
+          const errorMsg = (error.message || "").toLowerCase();
+          if (errorMsg.includes("already running") || errorMsg.includes("in queue") || errorMsg.includes("already in progress")) {
+            const article_id = args.id || args.article_id;
+            return {
+              content: [{
+                type: "text",
+                text: JSON.stringify({
+                  success: true,
+                  article_id,
+                  status: "already_running",
+                  message: "Article generation is already in progress. Use get_article to check status.",
+                  next_steps: [`Use 'get_article' with id: ${article_id} to check progress`]
+                }, null, 2)
+              }]
+            };
+          }
           logger.error("Failed to generate article", { error: error.message });
           this.metrics.recordAPICall("generate_article", "error");
           return { content: [{ type: "text", text: JSON.stringify({ error: true, message: error.message || "Failed to generate article" }, null, 2) }], isError: true };
@@ -1631,12 +1722,12 @@ var ArticleToolHandler = class {
       description: "Get article details and status",
       inputSchema: {
         type: "object",
-        properties: { article_id: { type: "string", format: "uuid", description: "ID of the article to retrieve" } },
-        required: ["article_id"]
+        properties: { id: { type: "string", format: "uuid", description: "ID of the article to retrieve" } },
+        required: ["id"]
       },
       handler: /* @__PURE__ */ __name(async (args) => {
         try {
-          const { article_id } = args;
+          const article_id = resolveId(args, "article_id");
           const cached = await this.cache.get(`article:${article_id}`);
           if (cached) {
             return { content: [{ type: "text", text: JSON.stringify({ success: true, article: cached, from_cache: true }, null, 2) }] };
@@ -1662,8 +1753,7 @@ var ArticleToolHandler = class {
         properties: {
           search: { type: "string", description: "Search term to filter articles" },
           status: { type: "string", description: "Filter by status" },
-          projectId: { type: "string", format: "uuid", description: "Filter by project (camelCase as per API)" },
-          project_id: { type: "string", format: "uuid", description: "Filter by project (alias)" },
+          project_id: { type: "string", format: "uuid", description: "Filter by project" },
           maker: { type: "string", description: "Filter by maker" },
           persona: { type: "string", description: "Filter by persona" },
           author: { type: "string", description: "Filter by author" },
@@ -1677,7 +1767,12 @@ var ArticleToolHandler = class {
       },
       handler: /* @__PURE__ */ __name(async (args) => {
         try {
-          const result2 = await this.apiClient.listArticles(args);
+          const apiParams = { ...args };
+          if (apiParams.project_id) {
+            apiParams.projectId = apiParams.project_id;
+            delete apiParams.project_id;
+          }
+          const result2 = await this.apiClient.listArticles(apiParams);
           this.metrics.recordAPICall("list_articles", "success");
           return {
             content: [{
@@ -1686,7 +1781,17 @@ var ArticleToolHandler = class {
                 success: true,
                 total: result2.total,
                 count: result2.items.length,
-                articles: result2.items.map((a) => ({ id: a.id, title: a.title, status: a.status, created_at: a.created_at, type: a.data?.input?.type })),
+                articles: result2.items.map((a) => ({
+                  id: a.id,
+                  title: a.title,
+                  status: a.status,
+                  created_at: a.created_at,
+                  type: a.data?.input?.type,
+                  project_id: a.data?.input?.project_id,
+                  target_keyword: a.data?.input?.target_keyword,
+                  lang: a.data?.input?.lang,
+                  subject: a.data?.input?.subject
+                })),
                 pagination: { skip: result2.skip, take: result2.take, has_more: result2.skip + result2.take < result2.total }
               }, null, 2)
             }]
@@ -1706,18 +1811,19 @@ var ArticleToolHandler = class {
       inputSchema: {
         type: "object",
         properties: {
-          article_id: { type: "string", format: "uuid", description: "ID of the article to update" },
+          id: { type: "string", format: "uuid", description: "ID of the article to update" },
           updates: {
             type: "object",
             description: "Fields to update (supports all create_article parameters)",
             properties: this.buildArticleInputSchema()
           }
         },
-        required: ["article_id", "updates"]
+        required: ["id", "updates"]
       },
       handler: /* @__PURE__ */ __name(async (args) => {
         try {
-          const { article_id, updates } = args;
+          const article_id = resolveId(args, "article_id");
+          const { updates } = args;
           await this.apiClient.updateArticle(article_id, updates);
           await this.cache.delete(`article:${article_id}`);
           this.metrics.recordAPICall("update_article", "success");
@@ -1739,14 +1845,15 @@ var ArticleToolHandler = class {
       inputSchema: {
         type: "object",
         properties: {
-          article_id: { type: "string", format: "uuid", description: "ID of the article" },
+          id: { type: "string", format: "uuid", description: "ID of the article" },
           format: { type: "string", enum: ["full", "html", "markdown", "summary"], description: "Output format", default: "full" }
         },
-        required: ["article_id"]
+        required: ["id"]
       },
       handler: /* @__PURE__ */ __name(async (args) => {
         try {
-          const { article_id, format = "full" } = args;
+          const article_id = resolveId(args, "article_id");
+          const { format = "full" } = args;
           const cacheKey = `article:output:${article_id}`;
           const cached = await this.cache.get(cacheKey);
           const output = cached || await this.apiClient.getArticleOutput(article_id);
@@ -1754,19 +1861,30 @@ var ArticleToolHandler = class {
             await this.cache.set(cacheKey, output, 600);
           }
           this.metrics.recordAPICall("get_article_output", "success");
+          const rawOutput = output;
+          const normalizedOutput = rawOutput?.data?.output ?? rawOutput?.data ?? rawOutput?.output ?? rawOutput;
+          logger.debug("get_article_output raw response keys", { keys: Object.keys(rawOutput || {}), normalizedKeys: Object.keys(normalizedOutput || {}) });
           let formattedOutput;
           switch (format) {
-            case "html":
-              formattedOutput = { html: output.html_content, title: output.title };
+            case "html": {
+              const html = normalizedOutput?.html_content ?? normalizedOutput?.content ?? null;
+              const title = normalizedOutput?.title ?? null;
+              formattedOutput = html || title ? { html, title } : { ...normalizedOutput, _note: "Fields not found at expected paths (html_content, title)" };
               break;
+            }
             case "markdown":
-              formattedOutput = { content: output.content, title: output.title };
+              formattedOutput = { content: normalizedOutput?.content ?? normalizedOutput?.markdown ?? null, title: normalizedOutput?.title ?? null };
               break;
-            case "summary":
-              formattedOutput = { title: output.title, excerpt: output.excerpt, word_count: output.word_count, seo_score: output.seo_score };
+            case "summary": {
+              const title = normalizedOutput?.title ?? null;
+              const excerpt = normalizedOutput?.excerpt ?? null;
+              const word_count = normalizedOutput?.word_count ?? null;
+              const seo_score = normalizedOutput?.seo_score ?? null;
+              formattedOutput = title || excerpt ? { title, excerpt, word_count, seo_score } : { ...normalizedOutput, _note: "Fields not found at expected paths (title, excerpt)" };
               break;
+            }
             default:
-              formattedOutput = output;
+              formattedOutput = normalizedOutput;
           }
           return { content: [{ type: "text", text: JSON.stringify({ success: true, article_id, format, output: formattedOutput, from_cache: !!cached }, null, 2) }] };
         } catch (error) {
@@ -1784,15 +1902,16 @@ var ArticleToolHandler = class {
       inputSchema: {
         type: "object",
         properties: {
-          article_id: { type: "string", format: "uuid", description: "ID of the article" },
+          id: { type: "string", format: "uuid", description: "ID of the article" },
           output_id: { type: "string", format: "uuid", description: "ID of the output to update" },
           updates: { type: "object", description: "Fields to update in the output" }
         },
-        required: ["article_id", "output_id", "updates"]
+        required: ["id", "output_id", "updates"]
       },
       handler: /* @__PURE__ */ __name(async (args) => {
         try {
-          const { article_id, output_id, updates } = args;
+          const article_id = resolveId(args, "article_id");
+          const { output_id, updates } = args;
           const result2 = await this.apiClient.updateArticleOutput(article_id, output_id, updates);
           await this.cache.delete(`article:output:${article_id}`);
           this.metrics.recordAPICall("update_article_output", "success");
@@ -1947,18 +2066,18 @@ var ProjectToolHandler = class {
       inputSchema: {
         type: "object",
         properties: {
-          project_id: {
+          id: {
             type: "string",
             format: "uuid",
             description: "Project ID"
           }
         },
-        required: ["project_id"]
+        required: ["id"]
       },
       handler: /* @__PURE__ */ __name(async (args) => {
         try {
-          const { project_id } = args;
-          const cached = await this.cache.get(`project:${project_id}`);
+          const id = resolveId(args, "project_id");
+          const cached = await this.cache.get(`project:${id}`);
           if (cached) {
             return {
               content: [
@@ -1973,8 +2092,8 @@ var ProjectToolHandler = class {
               ]
             };
           }
-          const project = await this.apiClient.getProject(project_id);
-          await this.cache.set(`project:${project_id}`, project, 600);
+          const project = await this.apiClient.getProject(id);
+          await this.cache.set(`project:${id}`, project, 600);
           this.metrics.recordAPICall("get_project", "success");
           return {
             content: [
@@ -2042,12 +2161,7 @@ var ProjectToolHandler = class {
                   success: true,
                   total: result2.total,
                   count: result2.items.length,
-                  projects: result2.items.map((p) => ({
-                    id: p.id,
-                    name: p.name,
-                    website_url: p.website_url,
-                    created_at: p.created_at
-                  }))
+                  projects: result2.items
                 }, null, 2)
               }
             ]
@@ -2078,7 +2192,7 @@ var ProjectToolHandler = class {
       inputSchema: {
         type: "object",
         properties: {
-          project_id: {
+          id: {
             type: "string",
             format: "uuid",
             description: "Project ID"
@@ -2095,13 +2209,14 @@ var ProjectToolHandler = class {
             }
           }
         },
-        required: ["project_id", "updates"]
+        required: ["id", "updates"]
       },
       handler: /* @__PURE__ */ __name(async (args) => {
         try {
-          const { project_id, updates } = args;
-          await this.apiClient.updateProject(project_id, updates);
-          await this.cache.delete(`project:${project_id}`);
+          const id = resolveId(args, "project_id");
+          const { updates } = args;
+          await this.apiClient.updateProject(id, updates);
+          await this.cache.delete(`project:${id}`);
           this.metrics.recordAPICall("update_project", "success");
           return {
             content: [
@@ -2110,7 +2225,7 @@ var ProjectToolHandler = class {
                 text: JSON.stringify({
                   success: true,
                   message: "Project updated successfully",
-                  project_id,
+                  project_id: id,
                   updated_fields: Object.keys(updates)
                 }, null, 2)
               }
@@ -2142,7 +2257,7 @@ var ProjectToolHandler = class {
       inputSchema: {
         type: "object",
         properties: {
-          project_id: {
+          id: {
             type: "string",
             format: "uuid",
             description: "Project ID to delete"
@@ -2153,11 +2268,12 @@ var ProjectToolHandler = class {
             const: true
           }
         },
-        required: ["project_id", "confirm"]
+        required: ["id", "confirm"]
       },
       handler: /* @__PURE__ */ __name(async (args) => {
         try {
-          const { project_id, confirm } = args;
+          const id = resolveId(args, "project_id");
+          const { confirm } = args;
           if (!confirm) {
             return {
               content: [
@@ -2172,8 +2288,8 @@ var ProjectToolHandler = class {
               isError: true
             };
           }
-          await this.apiClient.deleteProject(project_id);
-          await this.cache.delete(`project:${project_id}`);
+          await this.apiClient.deleteProject(id);
+          await this.cache.delete(`project:${id}`);
           this.metrics.recordAPICall("delete_project", "success");
           return {
             content: [
@@ -2182,7 +2298,7 @@ var ProjectToolHandler = class {
                 text: JSON.stringify({
                   success: true,
                   message: "Project deleted successfully",
-                  project_id
+                  project_id: id
                 }, null, 2)
               }
             ]
@@ -2229,33 +2345,104 @@ var PublishingToolHandler = class {
   publishToWordPressTool() {
     return {
       name: "publish_to_wordpress",
-      description: "Publish content to WordPress",
+      description: "Publish content to WordPress. connection_id must reference a WordPress connection with sufficient permissions.",
       inputSchema: {
         type: "object",
         properties: {
           entity_id: { type: "string", format: "uuid", description: "ID of the article/content to publish" },
           status: { type: "string", enum: ["draft", "publish", "future"], description: "Publishing status", default: "draft" },
-          schedule_date: { type: "string", format: "date-time", description: "Schedule date for future publishing (ISO 8601)" },
+          date: { type: "string", format: "date-time", description: 'Publication date (ISO 8601). Required when status is "future".' },
           category_id: { type: "number", description: "WordPress category ID" },
-          author_id: { type: "number", description: "WordPress author ID" },
-          connection_id: { type: "string", format: "uuid", description: "WordPress connection ID" }
+          author_id: { type: "number", description: "WordPress author ID. CAUTION: If the WP connection user lacks edit_others_posts capability, setting this will cause publishing to fail with rest_cannot_edit_others. Omit unless the WP user has admin/editor role." },
+          connection_id: { type: "string", format: "uuid", description: "WordPress connection ID (must have sufficient WP permissions)" },
+          clear_author: { type: "boolean", description: 'Clear the stored publishwordpress_author from article config before publishing. Use this to prevent "rest_cannot_edit_others" errors.', default: false }
         },
-        required: ["entity_id"]
+        required: ["entity_id", "connection_id"]
       },
       handler: /* @__PURE__ */ __name(async (args) => {
         try {
-          const { entity_id, ...options } = args;
+          const entity_id = resolveId(args, "entity_id");
+          const { connection_id, status, date, category_id, author_id } = args;
+          const clear_author = args.clear_author ?? false;
+          const warnings = [];
+          try {
+            const article = await this.apiClient.getArticle(entity_id);
+            const articleStatus = article.status || "unknown";
+            if (articleStatus !== "completed" && articleStatus !== "success") {
+              const statusMessages = {
+                draft: "Article has not been generated yet.",
+                pending: "Article generation is still pending.",
+                processing: "Article is currently being generated.",
+                failed: "Article generation failed previously.",
+                error: "Article encountered an error during generation."
+              };
+              return {
+                content: [{
+                  type: "text",
+                  text: JSON.stringify({
+                    error: true,
+                    message: statusMessages[articleStatus] || `Article status is "${articleStatus}" \u2014 content must be generated before publishing.`,
+                    article_status: articleStatus,
+                    entity_id,
+                    next_steps: [
+                      `Use 'generate_article' with id: ${entity_id} to generate content first`,
+                      `Use 'get_article' with id: ${entity_id} to check current status`
+                    ]
+                  }, null, 2)
+                }],
+                isError: true
+              };
+            }
+            const storedAuthor = article.data?.input?.publishwordpress_author;
+            if (storedAuthor && !author_id) {
+              if (clear_author) {
+                try {
+                  await this.apiClient.updateArticle(entity_id, { publishwordpress_author: void 0 });
+                  logger.info("Cleared stored publishwordpress_author before publishing", { entity_id, cleared_author: storedAuthor });
+                } catch (clearError) {
+                  logger.warn("Failed to clear publishwordpress_author", { error: clearError.message });
+                  warnings.push(`Could not clear stored publishwordpress_author (${storedAuthor}). Publishing may fail if the WP connection user lacks edit_others_posts.`);
+                }
+              } else {
+                warnings.push(
+                  `Article has stored publishwordpress_author=${storedAuthor}. If the WP connection user lacks edit_others_posts capability, publishing will fail with "rest_cannot_edit_others". Use clear_author: true to clear it before publishing.`
+                );
+              }
+            }
+          } catch (fetchError) {
+            logger.warn("Could not pre-validate article before publishing", { entity_id, error: fetchError.message });
+          }
+          const options = { connection_id };
+          if (status) options.status = status;
+          if (date) options.date = date;
+          if (category_id) options.category_id = category_id;
+          if (author_id) options.author_id = author_id;
           const result2 = await this.apiClient.publishToWordPress(entity_id, options);
           this.metrics.recordAPICall("publish_wordpress", result2.success ? "success" : "error");
           if (result2.success) {
-            return { content: [{ type: "text", text: JSON.stringify({ success: true, message: "Content published to WordPress successfully", entity_id, wordpress_url: result2.url, status: options.status || "draft" }, null, 2) }] };
+            const response = {
+              success: true,
+              message: "Content published to WordPress successfully",
+              entity_id,
+              wordpress_url: result2.url,
+              status: status || "draft"
+            };
+            if (warnings.length > 0) response.warnings = warnings;
+            return { content: [{ type: "text", text: JSON.stringify(response, null, 2) }] };
           } else {
             throw new Error(result2.error || "Publishing failed");
           }
         } catch (error) {
           logger.error("Failed to publish to WordPress", { error: error.message });
           this.metrics.recordAPICall("publish_wordpress", "error");
-          return { content: [{ type: "text", text: JSON.stringify({ error: true, message: error.message || "Failed to publish to WordPress" }, null, 2) }], isError: true };
+          const errorMsg = (error.message || "").toLowerCase();
+          const hints = [];
+          if (errorMsg.includes("rest_cannot_edit_others") || errorMsg.includes("403") || errorMsg.includes("cannot edit others")) {
+            hints.push("This error is likely caused by a stored publishwordpress_author value. Try again with clear_author: true to clear it before publishing.");
+          }
+          const response = { error: true, message: error.message || "Failed to publish to WordPress" };
+          if (hints.length > 0) response.hints = hints;
+          return { content: [{ type: "text", text: JSON.stringify(response, null, 2) }], isError: true };
         }
       }, "handler")
     };
@@ -2496,18 +2683,18 @@ var PersonaToolHandler = class {
       inputSchema: {
         type: "object",
         properties: {
-          persona_id: {
+          id: {
             type: "string",
             format: "uuid",
             description: "Persona ID"
           }
         },
-        required: ["persona_id"]
+        required: ["id"]
       },
       handler: /* @__PURE__ */ __name(async (args) => {
         try {
-          const { persona_id } = args;
-          const cached = await this.cache.get(`persona:${persona_id}`);
+          const id = resolveId(args, "persona_id");
+          const cached = await this.cache.get(`persona:${id}`);
           if (cached) {
             return {
               content: [
@@ -2522,8 +2709,8 @@ var PersonaToolHandler = class {
               ]
             };
           }
-          const persona = await this.apiClient.getPersona(persona_id);
-          await this.cache.set(`persona:${persona_id}`, persona, 600);
+          const persona = await this.apiClient.getPersona(id);
+          await this.cache.set(`persona:${id}`, persona, 600);
           this.metrics.recordAPICall("get_persona", "success");
           return {
             content: [
@@ -2628,7 +2815,7 @@ var PersonaToolHandler = class {
       inputSchema: {
         type: "object",
         properties: {
-          persona_id: {
+          id: {
             type: "string",
             format: "uuid",
             description: "Persona ID"
@@ -2649,13 +2836,14 @@ var PersonaToolHandler = class {
             }
           }
         },
-        required: ["persona_id", "updates"]
+        required: ["id", "updates"]
       },
       handler: /* @__PURE__ */ __name(async (args) => {
         try {
-          const { persona_id, updates } = args;
-          await this.apiClient.updatePersona(persona_id, updates);
-          await this.cache.delete(`persona:${persona_id}`);
+          const id = resolveId(args, "persona_id");
+          const { updates } = args;
+          await this.apiClient.updatePersona(id, updates);
+          await this.cache.delete(`persona:${id}`);
           this.metrics.recordAPICall("update_persona", "success");
           return {
             content: [
@@ -2664,7 +2852,7 @@ var PersonaToolHandler = class {
                 text: JSON.stringify({
                   success: true,
                   message: "Persona updated successfully",
-                  persona_id,
+                  persona_id: id,
                   updated_fields: Object.keys(updates)
                 }, null, 2)
               }
@@ -2696,7 +2884,7 @@ var PersonaToolHandler = class {
       inputSchema: {
         type: "object",
         properties: {
-          persona_id: {
+          id: {
             type: "string",
             format: "uuid",
             description: "Persona ID to delete"
@@ -2707,11 +2895,12 @@ var PersonaToolHandler = class {
             const: true
           }
         },
-        required: ["persona_id", "confirm"]
+        required: ["id", "confirm"]
       },
       handler: /* @__PURE__ */ __name(async (args) => {
         try {
-          const { persona_id, confirm } = args;
+          const id = resolveId(args, "persona_id");
+          const { confirm } = args;
           if (!confirm) {
             return {
               content: [
@@ -2726,8 +2915,8 @@ var PersonaToolHandler = class {
               isError: true
             };
           }
-          await this.apiClient.deletePersona(persona_id);
-          await this.cache.delete(`persona:${persona_id}`);
+          await this.apiClient.deletePersona(id);
+          await this.cache.delete(`persona:${id}`);
           this.metrics.recordAPICall("delete_persona", "success");
           return {
             content: [
@@ -2736,7 +2925,7 @@ var PersonaToolHandler = class {
                 text: JSON.stringify({
                   success: true,
                   message: "Persona deleted successfully",
-                  persona_id
+                  persona_id: id
                 }, null, 2)
               }
             ]
@@ -2818,16 +3007,8 @@ var BulkOperationsToolHandler = class {
       },
       handler: /* @__PURE__ */ __name(async (args) => {
         try {
-          let processedArgs = { ...args };
-          if (typeof args.articles === "string") {
-            logger.info("Converting articles from string to array");
-            try {
-              processedArgs.articles = JSON.parse(args.articles);
-            } catch (error) {
-              throw new Error("Invalid articles format: expected array or JSON string");
-            }
-          }
-          const { articles, auto_generate = false } = processedArgs;
+          const articles = resolveArray(args, "articles");
+          const auto_generate = args.auto_generate ?? false;
           logger.info("Validating and transforming article data types", { count: articles.length });
           const validatedArticles = articles.map((article, index) => {
             try {
@@ -2941,17 +3122,24 @@ var BulkOperationsToolHandler = class {
       },
       handler: /* @__PURE__ */ __name(async (args) => {
         try {
-          let processedArgs = { ...args };
-          if (typeof args.articles === "string") {
-            logger.info("Converting articles from string to array (bulk cost estimate)");
-            try {
-              processedArgs.articles = JSON.parse(args.articles);
-            } catch (error) {
-              throw new Error("Invalid articles format: expected array or JSON string");
-            }
-          }
-          const { articles } = processedArgs;
+          const articles = resolveArray(args, "articles");
           const result2 = await this.apiClient.calculateBulkCost(articles);
+          logger.debug("bulk_estimate_cost raw API response keys", { keys: Object.keys(result2 || {}) });
+          const breakdown = result2?.breakdown ?? result2?.items ?? result2?.estimates ?? [];
+          let total;
+          if (typeof result2?.total === "number" && !isNaN(result2.total)) {
+            total = result2.total;
+          } else if (typeof result2?.credits === "number" && !isNaN(result2.credits)) {
+            total = result2.credits;
+          } else if (typeof result2?.cost === "number" && !isNaN(result2.cost)) {
+            total = result2.cost;
+          } else if (Array.isArray(breakdown) && breakdown.length > 0) {
+            total = breakdown.reduce((sum, item) => sum + (item?.credits ?? item?.cost ?? 0), 0);
+            logger.info("bulk_estimate_cost: total derived from breakdown sum", { total });
+          } else {
+            total = 0;
+            logger.warn("bulk_estimate_cost: could not extract total from API response, defaulting to 0", { resultKeys: Object.keys(result2 || {}) });
+          }
           this.metrics.recordAPICall("bulk_estimate_cost", "success");
           return {
             content: [
@@ -2960,16 +3148,15 @@ var BulkOperationsToolHandler = class {
                 text: JSON.stringify({
                   success: true,
                   total_cost: {
-                    credits: result2.total,
-                    usd: (result2.total * 0.01).toFixed(2)
-                    // Assuming 1 credit = $0.01
+                    credits: total,
+                    usd: (total * 0.01).toFixed(2)
                   },
                   article_count: articles.length,
                   average_cost_per_article: {
-                    credits: Math.round(result2.total / articles.length),
-                    usd: (result2.total / articles.length * 0.01).toFixed(2)
+                    credits: articles.length > 0 ? Math.round(total / articles.length) : 0,
+                    usd: articles.length > 0 ? (total / articles.length * 0.01).toFixed(2) : "0.00"
                   },
-                  breakdown: result2.breakdown
+                  breakdown
                 }, null, 2)
               }
             ]
@@ -3093,18 +3280,18 @@ var ConnectionsToolHandler = class {
       inputSchema: {
         type: "object",
         properties: {
-          connection_id: {
+          id: {
             type: "string",
             format: "uuid",
             description: "Connection ID"
           }
         },
-        required: ["connection_id"]
+        required: ["id"]
       },
       handler: /* @__PURE__ */ __name(async (args) => {
         try {
-          const { connection_id } = args;
-          const cached = await this.cache.get(`connection:${connection_id}`);
+          const id = resolveId(args, "connection_id");
+          const cached = await this.cache.get(`connection:${id}`);
           if (cached) {
             return {
               content: [
@@ -3119,8 +3306,8 @@ var ConnectionsToolHandler = class {
               ]
             };
           }
-          const connection = await this.apiClient.getConnection(connection_id);
-          await this.cache.set(`connection:${connection_id}`, connection, 600);
+          const connection = await this.apiClient.getConnection(id);
+          await this.cache.set(`connection:${id}`, connection, 600);
           this.metrics.recordAPICall("get_connection", "success");
           return {
             content: [
@@ -3223,7 +3410,7 @@ var ConnectionsToolHandler = class {
       inputSchema: {
         type: "object",
         properties: {
-          connection_id: {
+          id: {
             type: "string",
             format: "uuid",
             description: "Connection ID"
@@ -3236,13 +3423,14 @@ var ConnectionsToolHandler = class {
             }
           }
         },
-        required: ["connection_id", "updates"]
+        required: ["id", "updates"]
       },
       handler: /* @__PURE__ */ __name(async (args) => {
         try {
-          const { connection_id, updates } = args;
-          await this.apiClient.updateConnection(connection_id, updates);
-          await this.cache.delete(`connection:${connection_id}`);
+          const id = resolveId(args, "connection_id");
+          const { updates } = args;
+          await this.apiClient.updateConnection(id, updates);
+          await this.cache.delete(`connection:${id}`);
           this.metrics.recordAPICall("update_connection", "success");
           return {
             content: [
@@ -3251,7 +3439,7 @@ var ConnectionsToolHandler = class {
                 text: JSON.stringify({
                   success: true,
                   message: "Connection updated successfully",
-                  connection_id
+                  connection_id: id
                 }, null, 2)
               }
             ]
@@ -3282,7 +3470,7 @@ var ConnectionsToolHandler = class {
       inputSchema: {
         type: "object",
         properties: {
-          connection_id: {
+          id: {
             type: "string",
             format: "uuid",
             description: "Connection ID to delete"
@@ -3293,11 +3481,12 @@ var ConnectionsToolHandler = class {
             const: true
           }
         },
-        required: ["connection_id", "confirm"]
+        required: ["id", "confirm"]
       },
       handler: /* @__PURE__ */ __name(async (args) => {
         try {
-          const { connection_id, confirm } = args;
+          const id = resolveId(args, "connection_id");
+          const { confirm } = args;
           if (!confirm) {
             return {
               content: [
@@ -3312,8 +3501,8 @@ var ConnectionsToolHandler = class {
               isError: true
             };
           }
-          await this.apiClient.deleteConnection(connection_id);
-          await this.cache.delete(`connection:${connection_id}`);
+          await this.apiClient.deleteConnection(id);
+          await this.cache.delete(`connection:${id}`);
           this.metrics.recordAPICall("delete_connection", "success");
           return {
             content: [
@@ -3322,7 +3511,7 @@ var ConnectionsToolHandler = class {
                 text: JSON.stringify({
                   success: true,
                   message: "Connection deleted successfully",
-                  connection_id
+                  connection_id: id
                 }, null, 2)
               }
             ]
@@ -3592,7 +3781,7 @@ var CategoryPagesToolHandler = class {
           country: { type: "string", enum: [...COUNTRY_ENUM2], description: "Target country for localization" },
           project_id: { type: "string", format: "uuid", description: "Project UUID" },
           persona_id: { type: "string", format: "uuid", description: "Persona UUID" },
-          apply_project_brief_config: { type: "boolean", description: "Apply project configuration", default: false },
+          apply_project_brief_config: { type: "boolean", description: "Apply project configuration. When true with a project_id, lang/country will be inherited from the project if not explicitly provided.", default: false },
           use_faq: { type: "boolean", description: "Generate FAQ section", default: false },
           use_toc: { type: "boolean", description: "Generate table of contents", default: false },
           use_image: { type: "boolean", description: "Generate featured image", default: false },
@@ -3605,7 +3794,7 @@ var CategoryPagesToolHandler = class {
           publishwordpress_date: { type: "string", description: "Publication date (ISO 8601 or WordPress format)" },
           publishwordpress_status: { type: "string", enum: ["draft", "publish", "future"], description: "WordPress post status" },
           publishwordpress_category: { type: "number", description: "WordPress category ID" },
-          publishwordpress_author: { type: "number", description: "WordPress author ID" },
+          publishwordpress_author: { type: "number", description: "WordPress author ID. CAUTION: If the WP connection user lacks edit_others_posts capability, setting this will cause publishing to fail with rest_cannot_edit_others. Omit unless the WP user has admin/editor role." },
           publishwordpress_connection: { type: "string", format: "uuid", description: "WordPress connection UUID" }
         },
         required: ["name", "subject"]
@@ -3613,9 +3802,21 @@ var CategoryPagesToolHandler = class {
       handler: /* @__PURE__ */ __name(async (args) => {
         try {
           const validated = CreateCategoryPageSchema.parse(args);
+          if (validated.apply_project_brief_config && validated.project_id && (!validated.lang || !validated.country)) {
+            try {
+              const project = await this.apiClient.getProject(validated.project_id);
+              if (!validated.lang && project.default_lang) {
+                validated.lang = project.default_lang;
+              }
+              if (!validated.country && project.default_country) {
+                validated.country = project.default_country;
+              }
+            } catch (e) {
+              logger.warn("Could not fetch project for lang/country defaults", { error: e.message });
+            }
+          }
           const input = this.buildApiInput(validated);
           const categoryPage = await this.apiClient.createCategoryPage(input);
-          await this.cache.set(`category_page:${categoryPage.id}`, categoryPage, 300);
           this.metrics.recordAPICall("create_category_page", "success");
           const displayName = categoryPage.name || validated.name || args.name || "Unnamed";
           return {
@@ -3636,7 +3837,7 @@ var CategoryPagesToolHandler = class {
       inputSchema: { type: "object", properties: { id: { type: "string", format: "uuid", description: "ID of the category page" } }, required: ["id"] },
       handler: /* @__PURE__ */ __name(async (args) => {
         try {
-          const { id } = args;
+          const id = resolveId(args);
           const cached = await this.cache.get(`category_page:${id}`);
           if (cached) {
             return { content: [{ type: "text", text: JSON.stringify({ success: true, category_page: cached, from_cache: true }, null, 2) }] };
@@ -3696,7 +3897,8 @@ var CategoryPagesToolHandler = class {
       },
       handler: /* @__PURE__ */ __name(async (args) => {
         try {
-          const { id, updates } = args;
+          const id = resolveId(args);
+          const { updates } = args;
           await this.apiClient.updateCategoryPage(id, updates);
           await this.cache.delete(`category_page:${id}`);
           this.metrics.recordAPICall("update_category_page", "success");
@@ -3724,7 +3926,8 @@ var CategoryPagesToolHandler = class {
       },
       handler: /* @__PURE__ */ __name(async (args) => {
         try {
-          const { id, wait_for_completion = true, max_wait_time = 300 } = args;
+          const id = resolveId(args);
+          const { wait_for_completion = true, max_wait_time = 300 } = args;
           await this.apiClient.runCategoryPageGeneration(id);
           if (!wait_for_completion) {
             return { content: [{ type: "text", text: JSON.stringify({ success: true, message: "Category page generation started", category_page_id: id }, null, 2) }] };
@@ -3760,7 +3963,7 @@ var CategoryPagesToolHandler = class {
       inputSchema: { type: "object", properties: { id: { type: "string", format: "uuid", description: "ID of the category page" } }, required: ["id"] },
       handler: /* @__PURE__ */ __name(async (args) => {
         try {
-          const { id } = args;
+          const id = resolveId(args);
           const cacheKey = `category_page:output:${id}`;
           const cached = await this.cache.get(cacheKey);
           const output = cached || await this.apiClient.getCategoryPageOutput(id);
@@ -3781,10 +3984,19 @@ var CategoryPagesToolHandler = class {
     return {
       name: "estimate_category_page_cost",
       description: "Estimate the cost of generating a category page",
-      inputSchema: { type: "object", properties: { data: { type: "object", description: "Category page configuration for cost estimation" } }, required: ["data"] },
+      inputSchema: {
+        type: "object",
+        properties: {
+          subject: { type: "string", description: "Category page subject", minLength: 1 },
+          length: { oneOf: [{ type: "number", maximum: 5e3 }, { type: "string", const: "auto" }], default: "auto" },
+          use_image: { type: "boolean", default: false }
+        },
+        required: ["subject"]
+      },
       handler: /* @__PURE__ */ __name(async (args) => {
         try {
-          const cost = await this.apiClient.calculateCategoryPageCost(args);
+          const costInput = args.data || args;
+          const cost = await this.apiClient.calculateCategoryPageCost(costInput);
           this.metrics.recordAPICall("estimate_category_page_cost", "success");
           return { content: [{ type: "text", text: JSON.stringify({ success: true, cost_estimate: cost }, null, 2) }] };
         } catch (error) {
@@ -3802,7 +4014,7 @@ var CategoryPagesToolHandler = class {
       inputSchema: { type: "object", properties: { items: { type: "array", items: { type: "object" }, description: "Array of category page configurations" } }, required: ["items"] },
       handler: /* @__PURE__ */ __name(async (args) => {
         try {
-          const { items } = args;
+          const items = resolveArray(args, "items");
           const result2 = await this.apiClient.createBulkCategoryPages(items);
           this.metrics.recordAPICall("bulk_create_category_pages", "success");
           return { content: [{ type: "text", text: JSON.stringify({ success: true, message: `Bulk created ${items.length} category pages`, result: result2 }, null, 2) }] };
@@ -3906,7 +4118,7 @@ var ProductPagesToolHandler = class {
           country: { type: "string", enum: [...COUNTRY_ENUM3], description: "Target country for localization" },
           project_id: { type: "string", format: "uuid", description: "Project UUID" },
           persona_id: { type: "string", format: "uuid", description: "Persona UUID" },
-          apply_project_brief_config: { type: "boolean", description: "Apply project configuration", default: false },
+          apply_project_brief_config: { type: "boolean", description: "Apply project configuration. When true with a project_id, lang/country will be inherited from the project if not explicitly provided.", default: false },
           use_faq: { type: "boolean", description: "Generate FAQ section", default: false },
           use_toc: { type: "boolean", description: "Generate table of contents", default: false },
           use_image: { type: "boolean", description: "Generate featured image", default: false },
@@ -3919,7 +4131,7 @@ var ProductPagesToolHandler = class {
           publishwordpress_date: { type: "string", description: "Publication date (ISO 8601 or WordPress format)" },
           publishwordpress_status: { type: "string", enum: ["draft", "publish", "future"], description: "WordPress post status" },
           publishwordpress_category: { type: "number", description: "WordPress category ID" },
-          publishwordpress_author: { type: "number", description: "WordPress author ID" },
+          publishwordpress_author: { type: "number", description: "WordPress author ID. CAUTION: If the WP connection user lacks edit_others_posts capability, setting this will cause publishing to fail with rest_cannot_edit_others. Omit unless the WP user has admin/editor role." },
           publishwordpress_connection: { type: "string", format: "uuid", description: "WordPress connection UUID" }
         },
         required: ["name", "subject"]
@@ -3927,9 +4139,21 @@ var ProductPagesToolHandler = class {
       handler: /* @__PURE__ */ __name(async (args) => {
         try {
           const validated = CreateProductPageSchema.parse(args);
+          if (validated.apply_project_brief_config && validated.project_id && (!validated.lang || !validated.country)) {
+            try {
+              const project = await this.apiClient.getProject(validated.project_id);
+              if (!validated.lang && project.default_lang) {
+                validated.lang = project.default_lang;
+              }
+              if (!validated.country && project.default_country) {
+                validated.country = project.default_country;
+              }
+            } catch (e) {
+              logger.warn("Could not fetch project for lang/country defaults", { error: e.message });
+            }
+          }
           const input = this.buildApiInput(validated);
           const productPage = await this.apiClient.createProductPage(input);
-          await this.cache.set(`product_page:${productPage.id}`, productPage, 300);
           this.metrics.recordAPICall("create_product_page", "success");
           const displayName = productPage.name || validated.name || args.name || "Unnamed";
           return { content: [{ type: "text", text: JSON.stringify({ success: true, product_page_id: productPage.id, name: displayName, message: `Product page "${displayName}" created successfully`, next_steps: [`Use 'generate_product_page' with id: ${productPage.id}`, `Use 'get_product_page' with id: ${productPage.id}`] }, null, 2) }] };
@@ -3948,7 +4172,7 @@ var ProductPagesToolHandler = class {
       inputSchema: { type: "object", properties: { id: { type: "string", format: "uuid", description: "ID of the product page" } }, required: ["id"] },
       handler: /* @__PURE__ */ __name(async (args) => {
         try {
-          const { id } = args;
+          const id = resolveId(args);
           const cached = await this.cache.get(`product_page:${id}`);
           if (cached) {
             return { content: [{ type: "text", text: JSON.stringify({ success: true, product_page: cached, from_cache: true }, null, 2) }] };
@@ -3999,7 +4223,8 @@ var ProductPagesToolHandler = class {
       inputSchema: { type: "object", properties: { id: { type: "string", format: "uuid", description: "ID of the product page to update" }, updates: { type: "object", description: "Fields to update" } }, required: ["id", "updates"] },
       handler: /* @__PURE__ */ __name(async (args) => {
         try {
-          const { id, updates } = args;
+          const id = resolveId(args);
+          const { updates } = args;
           await this.apiClient.updateProductPage(id, updates);
           await this.cache.delete(`product_page:${id}`);
           this.metrics.recordAPICall("update_product_page", "success");
@@ -4019,7 +4244,8 @@ var ProductPagesToolHandler = class {
       inputSchema: { type: "object", properties: { id: { type: "string", format: "uuid" }, wait_for_completion: { type: "boolean", default: true }, max_wait_time: { type: "number", default: 300, maximum: 600 } }, required: ["id"] },
       handler: /* @__PURE__ */ __name(async (args) => {
         try {
-          const { id, wait_for_completion = true, max_wait_time = 300 } = args;
+          const id = resolveId(args);
+          const { wait_for_completion = true, max_wait_time = 300 } = args;
           await this.apiClient.runProductPageGeneration(id);
           if (!wait_for_completion) {
             return { content: [{ type: "text", text: JSON.stringify({ success: true, message: "Product page generation started", product_page_id: id }, null, 2) }] };
@@ -4055,7 +4281,7 @@ var ProductPagesToolHandler = class {
       inputSchema: { type: "object", properties: { id: { type: "string", format: "uuid" } }, required: ["id"] },
       handler: /* @__PURE__ */ __name(async (args) => {
         try {
-          const { id } = args;
+          const id = resolveId(args);
           const cacheKey = `product_page:output:${id}`;
           const cached = await this.cache.get(cacheKey);
           const output = cached || await this.apiClient.getProductPageOutput(id);
@@ -4076,10 +4302,19 @@ var ProductPagesToolHandler = class {
     return {
       name: "estimate_product_page_cost",
       description: "Estimate the cost of generating a product page",
-      inputSchema: { type: "object", properties: { data: { type: "object", description: "Product page configuration for cost estimation" } }, required: ["data"] },
+      inputSchema: {
+        type: "object",
+        properties: {
+          subject: { type: "string", description: "Product page subject", minLength: 1 },
+          length: { oneOf: [{ type: "number", maximum: 5e3 }, { type: "string", const: "auto" }], default: "auto" },
+          use_image: { type: "boolean", default: false }
+        },
+        required: ["subject"]
+      },
       handler: /* @__PURE__ */ __name(async (args) => {
         try {
-          const cost = await this.apiClient.calculateProductPageCost(args);
+          const costInput = args.data || args;
+          const cost = await this.apiClient.calculateProductPageCost(costInput);
           this.metrics.recordAPICall("estimate_product_page_cost", "success");
           return { content: [{ type: "text", text: JSON.stringify({ success: true, cost_estimate: cost }, null, 2) }] };
         } catch (error) {
@@ -4097,7 +4332,7 @@ var ProductPagesToolHandler = class {
       inputSchema: { type: "object", properties: { items: { type: "array", items: { type: "object" }, description: "Array of product page configurations" } }, required: ["items"] },
       handler: /* @__PURE__ */ __name(async (args) => {
         try {
-          const { items } = args;
+          const items = resolveArray(args, "items");
           const result2 = await this.apiClient.createBulkProductPages(items);
           this.metrics.recordAPICall("bulk_create_product_pages", "success");
           return { content: [{ type: "text", text: JSON.stringify({ success: true, message: `Bulk created ${items.length} product pages`, result: result2 }, null, 2) }] };
@@ -4218,7 +4453,7 @@ var DiscoverToolHandler = class {
           keywords_secondary: { type: "string", description: "Secondary keywords" },
           project_id: { type: "string", format: "uuid", description: "Project UUID" },
           persona_id: { type: "string", format: "uuid", description: "Persona UUID" },
-          apply_project_brief_config: { type: "boolean", description: "Apply project brief configuration settings. When true, inherits all feature flags and settings from the project brief (use_faq, use_toc, use_inline_images, internal links, etc.). Recommended for consistency.", default: false },
+          apply_project_brief_config: { type: "boolean", description: "Apply project brief configuration settings. When true, inherits all feature flags and settings from the project brief (use_faq, use_toc, use_inline_images, internal links, etc.). When true with a project_id, lang/country will be inherited from the project if not explicitly provided. Recommended for consistency.", default: false },
           lang: { type: "string", enum: ["fr", "en"], description: "Language code" },
           country: { type: "string", enum: ["fr", "be", "ch", "ca", "us", "gb"], description: "Country code" },
           length: { description: 'Number (max 2000) or "auto"' },
@@ -4260,7 +4495,7 @@ var DiscoverToolHandler = class {
           publishwordpress_date: { type: "string", description: "WordPress publish date" },
           publishwordpress_status: { type: "string", enum: ["draft", "publish", "future"], description: "WordPress post status" },
           publishwordpress_category: { type: "number", description: "WordPress category ID" },
-          publishwordpress_author: { type: "number", description: "WordPress author ID" },
+          publishwordpress_author: { type: "number", description: "WordPress author ID. CAUTION: If the WP connection user lacks edit_others_posts capability, setting this will cause publishing to fail with rest_cannot_edit_others. Omit unless the WP user has admin/editor role." },
           publishwordpress_connection: { type: "string", description: "WordPress connection ID" },
           // Additional features
           use_infotable: { type: "boolean", description: "Add info table" },
@@ -4285,6 +4520,19 @@ var DiscoverToolHandler = class {
             apply_project_brief_config_raw_type: typeof args.apply_project_brief_config
           });
           const validated = DiscoverContentSchema.parse(args);
+          if (validated.apply_project_brief_config && validated.project_id && (!validated.lang || !validated.country)) {
+            try {
+              const project = await this.apiClient.getProject(validated.project_id);
+              if (!validated.lang && project.default_lang) {
+                validated.lang = project.default_lang;
+              }
+              if (!validated.country && project.default_country) {
+                validated.country = project.default_country;
+              }
+            } catch (e) {
+              logger.warn("Could not fetch project for lang/country defaults", { error: e.message });
+            }
+          }
           logger.info("discover_content VALIDATED (after boolField transform)", {
             apply_project_brief_config: validated.apply_project_brief_config,
             apply_project_brief_config_type: typeof validated.apply_project_brief_config,
@@ -4304,7 +4552,6 @@ var DiscoverToolHandler = class {
             apply_project_brief_config_value: cleanedInput.apply_project_brief_config
           });
           const discovery = await this.apiClient.discoverArticles(cleanedInput);
-          await this.cache.set(`discover:${discovery.id}`, discovery, 300);
           this.metrics.recordAPICall("discover_content", "success");
           return {
             content: [{
@@ -4335,7 +4582,7 @@ var DiscoverToolHandler = class {
       inputSchema: { type: "object", properties: { id: { type: "string", format: "uuid" } }, required: ["id"] },
       handler: /* @__PURE__ */ __name(async (args) => {
         try {
-          const { id } = args;
+          const id = resolveId(args);
           const cached = await this.cache.get(`discover:${id}`);
           if (cached) {
             return { content: [{ type: "text", text: JSON.stringify({ success: true, discovery: cached, from_cache: true }, null, 2) }] };
@@ -4386,7 +4633,8 @@ var DiscoverToolHandler = class {
       inputSchema: { type: "object", properties: { id: { type: "string", format: "uuid" }, updates: { type: "object", description: "Fields to update" } }, required: ["id", "updates"] },
       handler: /* @__PURE__ */ __name(async (args) => {
         try {
-          const { id, updates } = args;
+          const id = resolveId(args);
+          const { updates } = args;
           await this.apiClient.updateDiscoverArticle(id, updates);
           await this.cache.delete(`discover:${id}`);
           this.metrics.recordAPICall("update_discover_article", "success");
@@ -4406,7 +4654,7 @@ var DiscoverToolHandler = class {
       inputSchema: { type: "object", properties: { id: { type: "string", format: "uuid" } }, required: ["id"] },
       handler: /* @__PURE__ */ __name(async (args) => {
         try {
-          const { id } = args;
+          const id = resolveId(args);
           await this.apiClient.runDiscovery(id);
           await this.cache.delete(`discover:${id}`);
           this.metrics.recordAPICall("run_discovery", "success");
@@ -4426,7 +4674,7 @@ var DiscoverToolHandler = class {
       inputSchema: { type: "object", properties: { id: { type: "string", format: "uuid" } }, required: ["id"] },
       handler: /* @__PURE__ */ __name(async (args) => {
         try {
-          const { id } = args;
+          const id = resolveId(args);
           const cacheKey = `discover:output:${id}`;
           const cached = await this.cache.get(cacheKey);
           const output = cached || await this.apiClient.getDiscoverArticleOutput(id);
@@ -4447,10 +4695,19 @@ var DiscoverToolHandler = class {
     return {
       name: "estimate_discover_cost",
       description: "Estimate the cost of a discover article",
-      inputSchema: { type: "object", properties: { data: { type: "object", description: "Discovery configuration for cost estimation" } }, required: ["data"] },
+      inputSchema: {
+        type: "object",
+        properties: {
+          subject: { type: "string", description: "Discover article subject", minLength: 1 },
+          length: { oneOf: [{ type: "number", maximum: 2e3 }, { type: "string", const: "auto" }], default: "auto" },
+          use_image: { type: "boolean", default: false }
+        },
+        required: ["subject"]
+      },
       handler: /* @__PURE__ */ __name(async (args) => {
         try {
-          const cost = await this.apiClient.calculateDiscoverCost(args);
+          const costInput = args.data || args;
+          const cost = await this.apiClient.calculateDiscoverCost(costInput);
           this.metrics.recordAPICall("estimate_discover_cost", "success");
           return { content: [{ type: "text", text: JSON.stringify({ success: true, cost_estimate: cost }, null, 2) }] };
         } catch (error) {
@@ -4468,7 +4725,7 @@ var DiscoverToolHandler = class {
       inputSchema: { type: "object", properties: { items: { type: "array", items: { type: "object" }, description: "Array of discover article configurations" } }, required: ["items"] },
       handler: /* @__PURE__ */ __name(async (args) => {
         try {
-          const { items } = args;
+          const items = resolveArray(args, "items");
           const result2 = await this.apiClient.createBulkDiscoverArticles(items);
           this.metrics.recordAPICall("bulk_create_discover_articles", "success");
           return { content: [{ type: "text", text: JSON.stringify({ success: true, message: `Bulk created ${items.length} discover articles`, result: result2 }, null, 2) }] };
@@ -4561,8 +4818,10 @@ var RSSToolHandler = class {
         } catch (error) {
           logger.error("Failed to create feed", { error: error.message });
           this.metrics.recordAPICall("create_feed", "error");
+          const errorMessage = error.message || "Failed to create feed";
+          const guidance = "Please verify the feed URL is a valid RSS/Atom feed and is accessible. If the URL is correct, this may be a temporary Wisewand service issue \u2014 try again later.";
           return {
-            content: [{ type: "text", text: JSON.stringify({ error: true, message: error.message || "Failed to create feed", details: error.issues || void 0 }, null, 2) }],
+            content: [{ type: "text", text: JSON.stringify({ error: true, message: errorMessage, guidance, details: error.issues || void 0 }, null, 2) }],
             isError: true
           };
         }
@@ -4582,7 +4841,7 @@ var RSSToolHandler = class {
       },
       handler: /* @__PURE__ */ __name(async (args) => {
         try {
-          const { id } = args;
+          const id = resolveId(args);
           const cached = await this.cache.get(`feed:${id}`);
           if (cached) {
             return { content: [{ type: "text", text: JSON.stringify({ success: true, feed: cached, from_cache: true }, null, 2) }] };
@@ -4672,7 +4931,8 @@ var RSSToolHandler = class {
       },
       handler: /* @__PURE__ */ __name(async (args) => {
         try {
-          const { id, updates } = args;
+          const id = resolveId(args);
+          const { updates } = args;
           await this.apiClient.updateFeed(id, updates);
           await this.cache.delete(`feed:${id}`);
           this.metrics.recordAPICall("update_feed", "success");
@@ -4698,7 +4958,7 @@ var RSSToolHandler = class {
       },
       handler: /* @__PURE__ */ __name(async (args) => {
         try {
-          const { id } = args;
+          const id = resolveId(args);
           await this.apiClient.deleteFeed(id);
           await this.cache.delete(`feed:${id}`);
           this.metrics.recordAPICall("delete_feed", "success");
@@ -4820,7 +5080,7 @@ var JobsToolHandler = class {
       },
       handler: /* @__PURE__ */ __name(async (args) => {
         try {
-          const { id } = args;
+          const id = resolveId(args);
           const job = await this.apiClient.getJob(id);
           this.metrics.recordAPICall("get_job", "success");
           return {