bricks-mcp-server 0.3.0 → 0.5.0

package/README.md

@@ -66,7 +66,13 @@ WP_URL=https://yoursite.tld WP_USER=you WP_APP_PASSWORD="…" npx -y bricks-mcp-
 
 ## Tools
 
-`bricks_ping`, `bricks_list_pages`, `bricks_get_page`, `bricks_update_page`, `bricks_list_templates`, `bricks_create_template`, `bricks_get_global_classes`, `bricks_update_global_classes`, `bricks_get_theme_styles`.
+`bricks_ping`, `bricks_list_pages`, `bricks_get_page`, `bricks_update_page`, `bricks_create_page`, `bricks_list_templates`, `bricks_create_template`, `bricks_get_global_classes`, `bricks_update_global_classes`, `bricks_get_theme_styles`, `bricks_list_media`, `bricks_reindex_query_filters`.
+
+Highlights (server ≥0.5.0, plugin ≥0.7.0):
+
+- **Optimistic locking** — `bricks_get_page` returns `modified`; pass it as `expected_modified` on `bricks_update_page` to get a 409 instead of overwriting edits made in the builder since you read the page.
+- **`bricks_create_page`** — create pages/posts/CPT entries (draft by default), optionally with Bricks content.
+- **`bricks_ping`** echoes the effective `WP_URL`/user and warns when the configured host doesn't match the site that answered (stale config, redirect, proxy).
 
 ## License
 

package/dist/index.js

@@ -3,7 +3,16 @@ import { Server } from "@modelcontextprotocol/sdk/server/index.js";
 import { StdioServerTransport } from "@modelcontextprotocol/sdk/server/stdio.js";
 import { CallToolRequestSchema, ListToolsRequestSchema, } from "@modelcontextprotocol/sdk/types.js";
 import { z } from "zod";
-import { wpRequest, diagnose, formatDiagnosis } from "./wp-client.js";
+import { wpRequest, loadConfig, diagnose, formatDiagnosis } from "./wp-client.js";
+/** Extract the hostname from a URL, or null if it can't be parsed. */
+function hostOf(url) {
+    try {
+        return new URL(url).host.toLowerCase();
+    }
+    catch {
+        return null;
+    }
+}
 // Subcommand dispatch: `setup` runs the interactive wizard, `doctor` runs the
 // diagnostic ladder once and exits. No subcommand → start the MCP stdio server.
 const subcommand = process.argv[2];
@@ -18,14 +27,41 @@ if (subcommand === "doctor" || subcommand === "diagnose") {
     console.log(formatDiagnosis(d));
     process.exit(d.problem ? 1 : 0);
 }
-const server = new Server({ name: "bricks-mcp-server", version: "0.3.0" }, { capabilities: { tools: {} } });
+// Announce the effective target on startup. stdout is reserved for the MCP
+// protocol on stdio, so this goes to stderr — it shows up in Claude Code's MCP
+// logs and makes a wrong/stale WP_URL visible before any tool is even called.
+if (process.env.WP_URL) {
+    console.error(`[bricks-mcp] target WP_URL=${process.env.WP_URL} WP_USER=${process.env.WP_USER ?? "?"}`);
+}
+const server = new Server({ name: "bricks-mcp-server", version: "0.5.0" }, { capabilities: { tools: {} } });
 const tools = [
     {
         name: "bricks_ping",
-        description: "Verify the WordPress connection and report Bricks + WP version.",
+        description: "Verify the WordPress connection and report which site you're connected to (wp_url + site that answered), Bricks + WP version.",
         inputSchema: { type: "object", properties: {}, additionalProperties: false },
         schema: z.object({}),
-        handler: async () => wpRequest("/ping"),
+        handler: async () => {
+            // Echo the config this *process* was launched with, up front. If Claude
+            // Code is still holding a stale config (old process not restarted after a
+            // site switch), wp_url makes it obvious without guessing from permalinks.
+            const { baseUrl, user } = loadConfig();
+            const res = (await wpRequest("/ping"));
+            const configHost = hostOf(baseUrl);
+            const siteHost = typeof res.site_url === "string" ? hostOf(res.site_url) : null;
+            const mismatch = configHost !== null && siteHost !== null && configHost !== siteHost;
+            return {
+                wp_url: baseUrl,
+                wp_user_config: user,
+                ...res,
+                ...(mismatch
+                    ? {
+                        warning: `⚠️ El host configurado (WP_URL=${configHost}) NO coincide con el sitio que respondió (${siteHost}). ` +
+                            `Puede ser un redirect/proxy, o una configuración MCP antigua todavía activa. ` +
+                            `Si acabas de cambiar de sitio, reinicia Claude Code por completo para que tome la nueva config.`,
+                    }
+                    : {}),
+            };
+        },
     },
     {
         name: "bricks_get_schema",
@@ -67,7 +103,7 @@ const tools = [
     },
     {
         name: "bricks_get_page",
-        description: "Get a page/post/template by ID, including its Bricks content/header/footer/settings JSON.",
+        description: "Get a page/post/template by ID, including its Bricks content/header/footer/settings JSON. The response includes `modified` — pass it as expected_modified on bricks_update_page to avoid clobbering edits made in the builder since you read it.",
         inputSchema: {
             type: "object",
             properties: { id: { type: "number" } },
@@ -79,11 +115,15 @@ const tools = [
     },
     {
         name: "bricks_update_page",
-        description: "Update Bricks content/header/footer/settings for a page/post/template. Pass any subset of fields. The 'content' field must be an array of Bricks element objects.",
+        description: "Update Bricks content/header/footer/settings for a page/post/template. Pass any subset of fields. The 'content' field must be an array of Bricks element objects. ALWAYS pass expected_modified (the `modified` value from bricks_get_page) — the server returns 409 if the page changed since, so you never overwrite the user's builder edits.",
         inputSchema: {
             type: "object",
             properties: {
                 id: { type: "number" },
+                expected_modified: {
+                    type: "string",
+                    description: "The `modified` value you got from bricks_get_page. If the page changed since, the update is rejected (409) — re-fetch, merge, retry.",
+                },
                 title: { type: "string" },
                 content: { type: "array", description: "Bricks elements array" },
                 header: { type: "array" },
@@ -95,6 +135,7 @@ const tools = [
         },
         schema: z.object({
             id: z.number().int().positive(),
+            expected_modified: z.string().optional(),
             title: z.string().optional(),
             content: z.array(z.any()).optional(),
             header: z.array(z.any()).optional(),
@@ -106,6 +147,67 @@ const tools = [
             return wpRequest(`/pages/${id}`, { method: "PUT", body });
         },
     },
+    {
+        name: "bricks_create_page",
+        description: "Create a new page, post, or CPT entry (e.g. publicacion, noticia, evento), optionally with Bricks content. Defaults to draft status so nothing goes live without human review. Not for Bricks templates — use bricks_create_template for those.",
+        inputSchema: {
+            type: "object",
+            properties: {
+                title: { type: "string" },
+                post_type: {
+                    type: "string",
+                    description: "Registered post type slug (default 'page'). CPTs allowed.",
+                },
+                status: {
+                    type: "string",
+                    description: "draft (default) | publish | private | pending",
+                },
+                slug: { type: "string" },
+                content: { type: "array", description: "Bricks elements array" },
+                settings: { type: "object" },
+            },
+            required: ["title"],
+            additionalProperties: false,
+        },
+        schema: z.object({
+            title: z.string(),
+            post_type: z.string().optional(),
+            status: z.string().optional(),
+            slug: z.string().optional(),
+            content: z.array(z.any()).optional(),
+            settings: z.record(z.any()).optional(),
+        }),
+        handler: async (args) => wpRequest("/pages", { method: "POST", body: args }),
+    },
+    {
+        name: "bricks_list_media",
+        description: "Search the WordPress media library (read-only). Returns attachment id, url, alt, mime, dimensions — use the id/url in Bricks image settings.",
+        inputSchema: {
+            type: "object",
+            properties: {
+                search: { type: "string" },
+                mime: {
+                    type: "string",
+                    description: "MIME filter, e.g. 'image' (default), 'image/svg+xml', 'video'.",
+                },
+                per_page: { type: "number", description: "1-100, default 30" },
+            },
+            additionalProperties: false,
+        },
+        schema: z.object({
+            search: z.string().optional(),
+            mime: z.string().optional(),
+            per_page: z.number().int().min(1).max(100).optional(),
+        }),
+        handler: async (args) => wpRequest("/media", { query: args }),
+    },
+    {
+        name: "bricks_reindex_query_filters",
+        description: "Regenerate the Bricks Query Filters index. Call this after writing/editing any filter-* element via MCP — filters don't render until the index is rebuilt.",
+        inputSchema: { type: "object", properties: {}, additionalProperties: false },
+        schema: z.object({}),
+        handler: async () => wpRequest("/query-filters/reindex", { method: "POST" }),
+    },
     {
         name: "bricks_list_templates",
         description: "List Bricks templates. Optional filter by template type (section, header, footer, popup, content, etc.).",

package/dist/setup.js

@@ -157,6 +157,14 @@ export async function runSetup() {
             "Elige 1, 2 o 3: ")).trim();
         if (client === "1") {
             if (hasClaudeCli()) {
+                // `claude mcp add` refuses to overwrite an existing name (silent no-op),
+                // which leaves a stale config pointing at the wrong site. Remove first so
+                // the add always takes effect. Ignore its exit code (nothing to remove is
+                // fine).
+                spawnSync("claude", ["mcp", "remove", "bricks", "-s", "user"], {
+                    stdio: "ignore",
+                    shell: process.platform === "win32",
+                });
                 const args = [
                     "mcp",
                     "add",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "bricks-mcp-server",
-  "version": "0.3.0",
+  "version": "0.5.0",
   "description": "Provider-agnostic MCP server that exposes Bricks Builder pages, templates, global classes and theme styles as tools. Works with any MCP-compatible client (Claude Code, Codex CLI, etc.).",
   "type": "module",
   "bin": {