@diviops/mcp-server 1.5.13 → 1.5.16

package/dist/preset-cli/write-path.d.ts

@@ -20,6 +20,7 @@ import type { DiviopsResponse } from "../envelope.js";
 import type { ButtonPresetEntry } from "./button-emitter.js";
 import type { HeadingFontPresetEntry } from "./heading-font-emitter.js";
 import type { TextBodyFontPresetEntry } from "./text-body-font-emitter.js";
+import type { SpacingPresetEntry } from "./spacing-emitter.js";
 /** The plugin capability the storage-path capability contract ships. */
 export declare const STORAGE_CAPABILITY = "storage_multipath_probe_v1";
 /** The REST route the CLI posts to — same route `diviops_preset_create` uses. */
@@ -88,3 +89,18 @@ export declare function applyTextBodyFontPreset(client: PresetWriteClient, entry
     serverVersion: string;
     dry_run?: boolean;
 }): Promise<DiviopsResponse<unknown>>;
+/**
+ * Apply a `divi/spacing` section group preset: capability-gate, then POST
+ * to `/preset/create`. Mirrors `applyTextBodyFontPreset`'s sequence — the
+ * capability check runs BEFORE the write, and the write reuses the
+ * existing storage-routed route (no plugin route is added).
+ *
+ * Unlike the font emitters, the spacing entry carries `primary_attr_name`
+ * (`"module"` for the section cell per the canonical capture), which IS
+ * sent on the wire — the `/preset/create` route accepts it as an optional
+ * snake_case param and stores it as `primaryAttrName` in the preset.
+ */
+export declare function applySpacingPreset(client: PresetWriteClient, entry: SpacingPresetEntry, opts: {
+    serverVersion: string;
+    dry_run?: boolean;
+}): Promise<DiviopsResponse<unknown>>;

package/dist/preset-cli/write-path.js

@@ -18,6 +18,7 @@ import { WPClient } from "../wp-client.js";
 import { buildPresetCreateBody } from "./button-emitter.js";
 import { buildHeadingFontPresetCreateBody } from "./heading-font-emitter.js";
 import { buildTextBodyFontPresetCreateBody } from "./text-body-font-emitter.js";
+import { buildSpacingPresetCreateBody } from "./spacing-emitter.js";
 /** The plugin capability the storage-path capability contract ships. */
 export const STORAGE_CAPABILITY = "storage_multipath_probe_v1";
 /** The REST route the CLI posts to — same route `diviops_preset_create` uses. */
@@ -130,3 +131,24 @@ export async function applyTextBodyFontPreset(client, entry, opts) {
         body,
     });
 }
+/**
+ * Apply a `divi/spacing` section group preset: capability-gate, then POST
+ * to `/preset/create`. Mirrors `applyTextBodyFontPreset`'s sequence — the
+ * capability check runs BEFORE the write, and the write reuses the
+ * existing storage-routed route (no plugin route is added).
+ *
+ * Unlike the font emitters, the spacing entry carries `primary_attr_name`
+ * (`"module"` for the section cell per the canonical capture), which IS
+ * sent on the wire — the `/preset/create` route accepts it as an optional
+ * snake_case param and stores it as `primaryAttrName` in the preset.
+ */
+export async function applySpacingPreset(client, entry, opts) {
+    await assertStorageCapability(client, opts.serverVersion);
+    const body = buildSpacingPresetCreateBody(entry, {
+        dry_run: opts.dry_run,
+    });
+    return client.requestEnveloped(PRESET_CREATE_ROUTE, {
+        method: "POST",
+        body,
+    });
+}

package/dist/wp-cli-fs-validator.d.ts

@@ -11,7 +11,7 @@
  * This module adds a second-pass validator specifically for DEFAULT-tier
  * FS commands. EXTENDED-tier FS commands (`import`, `eval-file`) are
  * already opt-in via DIVIOPS_WP_CLI_ALLOW; their validation is tracked
- * separately (see dev-repo #271 scope discussion).
+ * separately as a future scope expansion.
  *
  * Guarantees:
  *   - All path arguments must resolve under SAFE_FS_ROOT after canonicalization

package/dist/wp-cli-fs-validator.js

@@ -11,7 +11,7 @@
  * This module adds a second-pass validator specifically for DEFAULT-tier
  * FS commands. EXTENDED-tier FS commands (`import`, `eval-file`) are
  * already opt-in via DIVIOPS_WP_CLI_ALLOW; their validation is tracked
- * separately (see dev-repo #271 scope discussion).
+ * separately as a future scope expansion.
  *
  * Guarantees:
  *   - All path arguments must resolve under SAFE_FS_ROOT after canonicalization

package/dist/wp-cli.js

@@ -83,8 +83,7 @@ const DEFAULT_COMMANDS = [
     'core verify-checksums',
     'core language list',
     // DB (read-only introspection only — `db query` stays out; it's arbitrary SQL with no scoping
-    // and belongs in a higher-risk tier, tracked separately). See dev-repo #361 Chunk B for the
-    // opt-in design discussion.
+    // and belongs in a higher-risk tier, tracked separately).
     'db columns',
     'db size',
     'db tables',

package/dist/wp-client.js

@@ -361,6 +361,23 @@ export class WPClient {
             typeof result.capabilities !== 'object') {
             result.capabilities = {};
         }
+        // ADR-003 / ADR-007 Pro-extension fields. Free-only sites omit
+        // them entirely; pre-V1 Pro plugins might emit partial shapes.
+        // Normalize defensively so downstream gates can read uniform
+        // types without per-call shape checks.
+        if (typeof result.pro_active !== 'boolean') {
+            result.pro_active = false;
+        }
+        if (result.available_targets === null ||
+            typeof result.available_targets !== 'object' ||
+            Array.isArray(result.available_targets)) {
+            result.available_targets = {};
+        }
+        if (result.active_modules === null ||
+            typeof result.active_modules !== 'object' ||
+            Array.isArray(result.active_modules)) {
+            result.active_modules = {};
+        }
         return result;
     }
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@diviops/mcp-server",
-  "version": "1.5.13",
+  "version": "1.5.16",
   "description": "MCP server exposing Divi 5 Visual Builder as tools for Claude",
   "type": "module",
   "main": "dist/index.js",
@@ -9,7 +9,8 @@
     "diviops-preset": "dist/preset-cli/bin.js"
   },
   "files": [
-    "dist",
+    "dist/**/*",
+    "!dist/**/__tests__/**",
     "templates",
     "data/verified-attrs.json",
     "data/verified-attrs-backlog.json",

package/dist/preset-cli/__tests__/button-emitter.test.d.ts

@@ -1,8 +0,0 @@
-/**
- * `divi/button` emitter shape coverage:
- *  - fixture-based shape assertion against round-3-button-canonical-complete.json
- *  - emit-on-specification discriminator (with vs without font.weight)
- *  - hover shape (NO `value` wrapper), radius vocabulary, variable tokens,
- *    the do-not-emit list, and the opt-in bypass corner.
- */
-export {};

package/dist/preset-cli/__tests__/button-emitter.test.js

@@ -1,188 +0,0 @@
-/**
- * `divi/button` emitter shape coverage:
- *  - fixture-based shape assertion against round-3-button-canonical-complete.json
- *  - emit-on-specification discriminator (with vs without font.weight)
- *  - hover shape (NO `value` wrapper), radius vocabulary, variable tokens,
- *    the do-not-emit list, and the opt-in bypass corner.
- */
-import { test } from "node:test";
-import assert from "node:assert/strict";
-import { readFileSync } from "node:fs";
-import { fileURLToPath } from "node:url";
-import { dirname, join } from "node:path";
-import { emitButtonGroupPreset, composeButtonAttrs, buildPresetCreateBody, } from "../button-emitter.js";
-import { loadRegistry } from "../registry.js";
-const __dirname = dirname(fileURLToPath(import.meta.url));
-const REPO_ROOT = join(__dirname, "..", "..", "..", "..");
-const FIXTURE = join(REPO_ROOT, "docs/verification/evidence/canonical-shape-dumps-2026-05-18/round-3-button-canonical-complete.json");
-const registry = loadRegistry();
-test("emitter byte-matches the round-3 canonical button fixture attrs", () => {
-    const fixture = JSON.parse(readFileSync(FIXTURE, "utf8"));
-    const canonicalAttrs = fixture.canonical_button_primary_preset.attrs;
-    const entry = emitButtonGroupPreset({
-        name: "canonical-button-primary-vb-2026-05-18",
-        bg_color: "gcid-primary-color",
-        bg_color_hover: "gcid-secondary-color",
-        radius: {
-            topLeft: "8px",
-            topRight: "8px",
-            bottomLeft: "8px",
-            bottomRight: "8px",
-        },
-        font: {
-            family: "Inter",
-            weight: "600",
-            color: "gcid-body-color",
-        },
-    }, registry);
-    assert.deepEqual(entry.attrs, canonicalAttrs, "emitted attrs must byte-match the canonical capture");
-    assert.equal(entry.type, "group");
-    assert.equal(entry.group_name, "divi/button");
-    assert.equal(entry.group_id, "button");
-    assert.equal(entry.module_name, "divi/button");
-});
-test("hover bg color emits at desktop.hover.color with NO value wrapper", () => {
-    const attrs = composeButtonAttrs({
-        name: "hover-only",
-        bg_color: "#111111",
-        bg_color_hover: "#222222",
-    });
-    const bg = attrs.button.decoration.background;
-    assert.equal(bg.desktop.value.color, "#111111", "desktop is 3-level (value.color)");
-    assert.equal(bg.desktop.hover.color, "#222222", "hover is 2-level (hover.color)");
-    assert.equal("value" in bg.desktop.hover, false, "hover must NOT carry a `value` wrapper");
-});
-test("emit-on-specification discriminator: font.weight present vs absent", () => {
-    const withWeight = composeButtonAttrs({
-        name: "with-weight",
-        font: { family: "Inter", weight: "600" },
-    });
-    const withoutWeight = composeButtonAttrs({
-        name: "without-weight",
-        font: { family: "Inter" },
-    });
-    const wv = withWeight.button.decoration.font.font.desktop.value;
-    const wov = withoutWeight.button.decoration.font.font.desktop.value;
-    assert.equal(wv.weight, "600", "weight emitted when specified");
-    assert.equal("weight" in wv, true);
-    assert.equal("weight" in wov, false, "weight absent when not specified");
-    assert.deepEqual(Object.keys(wov), ["family"], "ONLY the touched key emitted");
-});
-test("radius vocabulary is {topLeft,topRight,bottomLeft,bottomRight,sync}", () => {
-    const attrs = composeButtonAttrs({
-        name: "r",
-        radius: { topLeft: "8px", topRight: "8px", bottomLeft: "8px", bottomRight: "8px" },
-    });
-    const radius = attrs.button.decoration.border.desktop.value.radius;
-    assert.deepEqual(Object.keys(radius).sort(), [
-        "bottomLeft",
-        "bottomRight",
-        "sync",
-        "topLeft",
-        "topRight",
-    ]);
-    assert.equal(radius.sync, "on", "equal corners derive sync=on");
-});
-test("radius sync derives off when corners differ; partial corners stay absent", () => {
-    const attrs = composeButtonAttrs({
-        name: "r",
-        radius: { topLeft: "8px", bottomRight: "4px" },
-    });
-    const radius = attrs.button.decoration.border.desktop.value.radius;
-    assert.equal(radius.sync, "off");
-    assert.equal("topRight" in radius, false, "untouched corner absent");
-    assert.equal("bottomLeft" in radius, false, "untouched corner absent");
-});
-test("radius sync derives off for a single specified corner (Rule 1a)", () => {
-    // Per Rule 1a in canonical-preset-shapes: a partial corner set must derive
-    // sync="off". A 1-element corner array is trivially "all equal" — auto-
-    // deriving "on" is only correct when all 4 corners are present and identical.
-    const attrs = composeButtonAttrs({
-        name: "r",
-        radius: { topLeft: "8px" },
-    });
-    const radius = attrs.button.decoration.border.desktop.value.radius;
-    assert.equal(radius.sync, "off", "single corner must NOT derive sync=on");
-    assert.equal("topRight" in radius, false, "untouched corner absent");
-});
-test("radius sync stays off for 3 identical corners; on only with all 4", () => {
-    const three = composeButtonAttrs({
-        name: "r",
-        radius: { topLeft: "8px", topRight: "8px", bottomLeft: "8px" },
-    });
-    assert.equal(three.button.decoration.border.desktop.value.radius.sync, "off", "3 identical corners is still a partial set — sync=off");
-    const all = composeButtonAttrs({
-        name: "r",
-        radius: { topLeft: "8px", topRight: "8px", bottomLeft: "8px", bottomRight: "8px" },
-    });
-    assert.equal(all.button.decoration.border.desktop.value.radius.sync, "on", "all 4 identical corners derive sync=on");
-});
-test("explicit radius.sync wins over derivation", () => {
-    const attrs = composeButtonAttrs({
-        name: "r",
-        radius: { topLeft: "8px", sync: "on" },
-    });
-    assert.equal(attrs.button.decoration.border.desktop.value.radius.sync, "on", "an explicitly-passed sync flag is honored regardless of corner count");
-});
-test("variable tokens get the {name,settings} object shape + trailing )$", () => {
-    const attrs = composeButtonAttrs({
-        name: "v",
-        bg_color: "gcid-primary-color",
-    });
-    const color = attrs.button.decoration.background.desktop.value.color;
-    assert.ok(color.endsWith(")$"), "token must end with )$");
-    assert.match(color, /^\$variable\(/);
-    const payload = JSON.parse(color.slice("$variable(".length, -2));
-    assert.deepEqual(payload, {
-        type: "color",
-        value: { name: "gcid-primary-color", settings: {} },
-    });
-});
-test("literal hex color is emitted verbatim, not wrapped", () => {
-    const attrs = composeButtonAttrs({ name: "h", bg_color: "#2563eb" });
-    assert.equal(attrs.button.decoration.background.desktop.value.color, "#2563eb");
-});
-test("do-not-emit list: no attrs.font/spacing top-level, no renderAttrs, no button slot", () => {
-    const entry = emitButtonGroupPreset({
-        name: "clean",
-        bg_color: "#111",
-        font: { family: "Inter" },
-    }, registry);
-    assert.equal("font" in entry.attrs, false, "no top-level attrs.font");
-    assert.equal("spacing" in entry.attrs, false, "no top-level attrs.spacing");
-    assert.equal("renderAttrs" in entry, false, "no renderAttrs");
-    const decoration = entry.attrs.button.decoration;
-    assert.equal("button" in decoration, false, "no button.decoration.button slot by default");
-});
-test("bypass_hover_padding_gate emits ONLY the padding.top corner", () => {
-    const attrs = composeButtonAttrs({
-        name: "bypass",
-        bg_color: "#111",
-        bypass_hover_padding_gate: true,
-    });
-    const buttonSlot = attrs.button.decoration.button;
-    assert.deepEqual(buttonSlot, {
-        desktop: { value: { padding: { top: "0px" } } },
-    });
-});
-test("emitter rejects an empty preset (no styling specified)", () => {
-    assert.throws(() => emitButtonGroupPreset({ name: "empty" }, registry), /empty preset/);
-});
-test("emitter rejects a missing name", () => {
-    assert.throws(() => emitButtonGroupPreset({ name: "" }, registry), /requires a non-empty `name`/);
-});
-test("buildPresetCreateBody mirrors the diviops_preset_create body shape", () => {
-    const entry = emitButtonGroupPreset({ name: "Primary", bg_color: "#111" }, registry);
-    const body = buildPresetCreateBody(entry, { dry_run: true });
-    assert.deepEqual(body, {
-        module_name: "divi/button",
-        name: "Primary",
-        attrs: entry.attrs,
-        type: "group",
-        group_name: "divi/button",
-        group_id: "button",
-        dry_run: true,
-    });
-    const noDry = buildPresetCreateBody(entry);
-    assert.equal("dry_run" in noDry, false, "dry_run omitted when not requested");
-});

package/dist/preset-cli/__tests__/cli.test.d.ts

@@ -1,9 +0,0 @@
-/**
- * CLI integration coverage: --help, arg parsing, structured exit codes,
- * dry-run output, evidence-gate exit, capability-missing exit.
- *
- * Apply-mode here is exercised through the credential-missing path only
- * (no live write — #725 AC #8). The mocked write-path proper lives in
- * write-path.test.ts.
- */
-export {};