laive-mcp 0.1.2 → 0.1.4

package/CHANGELOG.md

@@ -2,6 +2,14 @@
 
 ## Unreleased
 
+## v0.1.4 - 2026-03-22
+
+- Fixed MCP tool schema advertising so argument-bearing tools like `set_tempo`, `get_track_details`, `get_device_tree`, `create_clip`, and `set_parameter` now publish explicit JSON Schemas through `tools/list` instead of empty input objects, allowing Codex clients to send required parameters.
+
+## v0.1.3 - 2026-03-22
+
+- Fixed MCP `tools/call` responses to return proper `CallToolResult` envelopes with `content`, `structuredContent`, and `isError`, so Codex clients accept the responses instead of rejecting them as an unexpected type.
+
 ## v0.1.2 - 2026-03-22
 
 - Fixed an MCP transport crash when the Live bridge socket is unreachable by preventing the bridge client from raising an unhandled `error` event during lazy connection attempts. Tool calls now return structured MCP errors instead of closing the server process.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "laive-mcp",
-  "version": "0.1.2",
+  "version": "0.1.4",
   "description": "Local MCP, install tooling, and helper assets for controlling Ableton Live.",
   "license": "GPL-3.0-only",
   "type": "module",

package/packages/mcp-server/src/default-tools.js

@@ -1,5 +1,20 @@
 import { McpServerError } from "./errors.js";
 
+const EMPTY_OBJECT_SCHEMA = {
+  type: "object",
+  properties: {},
+  additionalProperties: false
+};
+
+function createObjectSchema({ properties = {}, required = [] } = {}) {
+  return {
+    type: "object",
+    properties,
+    required,
+    additionalProperties: false
+  };
+}
+
 function requireString(value, fieldName) {
   if (typeof value !== "string" || value.length === 0) {
     throw new McpServerError(
@@ -25,6 +40,7 @@ export function buildDefaultTools({ stateAdapter, bridgeAdapter, policyAdapter }
     {
       name: "get_project_summary",
       description: "Return a compact summary of the current Live set state.",
+      inputSchema: EMPTY_OBJECT_SCHEMA,
       async execute() {
         const summary = await stateAdapter.getProjectSummary();
         return {
@@ -41,6 +57,7 @@ export function buildDefaultTools({ stateAdapter, bridgeAdapter, policyAdapter }
     {
       name: "get_selected_context",
       description: "Return the selected track, scene, clip, and device context.",
+      inputSchema: EMPTY_OBJECT_SCHEMA,
       async execute() {
         const context = await stateAdapter.getSelectedContext();
         return {
@@ -59,6 +76,7 @@ export function buildDefaultTools({ stateAdapter, bridgeAdapter, policyAdapter }
     {
       name: "list_tracks",
       description: "List tracks in compact form.",
+      inputSchema: EMPTY_OBJECT_SCHEMA,
       async execute() {
         const tracks = await stateAdapter.listTracks();
         return {
@@ -75,6 +93,23 @@ export function buildDefaultTools({ stateAdapter, bridgeAdapter, policyAdapter }
     {
       name: "get_track_details",
       description: "Return detailed state for a track identified by ID, name, or index.",
+      inputSchema: createObjectSchema({
+        properties: {
+          id: {
+            type: "string",
+            description: "Track identifier, for example `track:7`."
+          },
+          name: {
+            type: "string",
+            description: "Exact track name."
+          },
+          index: {
+            type: "integer",
+            minimum: 0,
+            description: "Zero-based visible-track index."
+          }
+        }
+      }),
       async execute(args) {
         const target = args.id ?? args.name ?? args.index;
         if (target === undefined) {
@@ -99,6 +134,15 @@ export function buildDefaultTools({ stateAdapter, bridgeAdapter, policyAdapter }
     {
       name: "get_device_tree",
       description: "Return device state for a track.",
+      inputSchema: createObjectSchema({
+        properties: {
+          trackId: {
+            type: "string",
+            description: "Track identifier, for example `track:7`."
+          }
+        },
+        required: ["trackId"]
+      }),
       async execute(args) {
         const trackId = args.trackId ?? args.track ?? args.id;
         requireString(trackId, "trackId");
@@ -117,6 +161,20 @@ export function buildDefaultTools({ stateAdapter, bridgeAdapter, policyAdapter }
     {
       name: "set_tempo",
       description: "Update the current song tempo.",
+      inputSchema: createObjectSchema({
+        properties: {
+          tempo: {
+            type: "number",
+            exclusiveMinimum: 0,
+            description: "Target song tempo in BPM."
+          },
+          dryRun: {
+            type: "boolean",
+            description: "If true, preview the action without mutating Live."
+          }
+        },
+        required: ["tempo"]
+      }),
       async execute(args) {
         const nextTempo = Number(args.tempo);
         if (!Number.isFinite(nextTempo) || nextTempo <= 0) {
@@ -139,6 +197,19 @@ export function buildDefaultTools({ stateAdapter, bridgeAdapter, policyAdapter }
     {
       name: "create_track",
       description: "Create a new track.",
+      inputSchema: createObjectSchema({
+        properties: {
+          kind: {
+            type: "string",
+            enum: ["midi", "audio"],
+            description: "Track type to create."
+          },
+          dryRun: {
+            type: "boolean",
+            description: "If true, preview the action without mutating Live."
+          }
+        }
+      }),
       async execute(args) {
         const kind = args.kind ?? "midi";
         await policyAdapter.assertAllowed("create_track", args);
@@ -157,6 +228,33 @@ export function buildDefaultTools({ stateAdapter, bridgeAdapter, policyAdapter }
     {
       name: "create_clip",
       description: "Create a MIDI clip on a target track and slot.",
+      inputSchema: createObjectSchema({
+        properties: {
+          trackId: {
+            type: "string",
+            description: "Track identifier, for example `track:7`."
+          },
+          slotIndex: {
+            type: "integer",
+            minimum: 0,
+            description: "Zero-based session slot index on the target track."
+          },
+          lengthBeats: {
+            type: "number",
+            exclusiveMinimum: 0,
+            description: "Clip length in beats. Defaults to 4."
+          },
+          name: {
+            type: "string",
+            description: "Optional clip name."
+          },
+          dryRun: {
+            type: "boolean",
+            description: "If true, preview the action without mutating Live."
+          }
+        },
+        required: ["trackId", "slotIndex"]
+      }),
       async execute(args) {
         requireString(args.trackId, "trackId");
         if (!Number.isInteger(args.slotIndex) || args.slotIndex < 0) {
@@ -185,6 +283,31 @@ export function buildDefaultTools({ stateAdapter, bridgeAdapter, policyAdapter }
     {
       name: "set_parameter",
       description: "Set a device parameter by track/device/parameter identifiers.",
+      inputSchema: createObjectSchema({
+        properties: {
+          trackId: {
+            type: "string",
+            description: "Track identifier containing the target device."
+          },
+          deviceId: {
+            type: "string",
+            description: "Device identifier containing the target parameter."
+          },
+          parameterId: {
+            type: "string",
+            description: "Parameter identifier to update."
+          },
+          value: {
+            type: "number",
+            description: "Target numeric parameter value."
+          },
+          dryRun: {
+            type: "boolean",
+            description: "If true, preview the action without mutating Live."
+          }
+        },
+        required: ["trackId", "deviceId", "parameterId", "value"]
+      }),
       async execute(args) {
         requireString(args.trackId, "trackId");
         requireString(args.deviceId, "deviceId");
@@ -219,6 +342,14 @@ export function buildDefaultTools({ stateAdapter, bridgeAdapter, policyAdapter }
     {
       name: "refresh_state",
       description: "Force a state refresh for a target scope.",
+      inputSchema: createObjectSchema({
+        properties: {
+          target: {
+            type: "string",
+            description: "Refresh scope, for example `project`, `song`, or `track:7`."
+          }
+        }
+      }),
       async execute(args) {
         const target = args.target ?? "project";
         const refreshed = await stateAdapter.refreshState(target);
@@ -236,6 +367,7 @@ export function buildDefaultTools({ stateAdapter, bridgeAdapter, policyAdapter }
     {
       name: "get_capabilities",
       description: "Return bridge and server capabilities.",
+      inputSchema: EMPTY_OBJECT_SCHEMA,
       async execute() {
         const capabilities = await bridgeAdapter.getCapabilities();
         return {

package/packages/mcp-server/src/server.js

@@ -86,15 +86,23 @@ export class LaiveMcpServer {
 
     if (message.method === "tools/call") {
       const params = message.params ?? {};
-      const result = await this.invokeTool(params.name, params.arguments ?? {}, {
-        requestId: message.id ?? null
-      });
+      try {
+        const result = await this.invokeTool(params.name, params.arguments ?? {}, {
+          requestId: message.id ?? null
+        });
 
-      return {
-        jsonrpc: "2.0",
-        id: message.id ?? null,
-        result
-      };
+        return {
+          jsonrpc: "2.0",
+          id: message.id ?? null,
+          result: toToolResult(result)
+        };
+      } catch (error) {
+        return {
+          jsonrpc: "2.0",
+          id: message.id ?? null,
+          result: toToolErrorResult(error)
+        };
+      }
     }
 
     throw new McpServerError("method_not_found", `Unsupported method: ${message.method}`);
@@ -113,6 +121,38 @@ export class LaiveMcpServer {
   }
 }
 
+function toToolResult(result) {
+  return {
+    content: [
+      {
+        type: "text",
+        text:
+          typeof result?.summary === "string" && result.summary.length > 0
+            ? result.summary
+            : JSON.stringify(result, null, 2)
+      }
+    ],
+    structuredContent: result,
+    isError: false
+  };
+}
+
+function toToolErrorResult(error) {
+  const shape = toErrorShape(error);
+  return {
+    content: [
+      {
+        type: "text",
+        text: shape.message
+      }
+    ],
+    structuredContent: {
+      error: shape
+    },
+    isError: true
+  };
+}
+
 function createUnsupportedAdapter(name) {
   return new Proxy(
     {},