agent-tempo 1.3.1 → 1.4.0

package/dist/tools/broadcast.js

@@ -1,76 +1,81 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.registerBroadcastTool = registerBroadcastTool;
+exports.buildBroadcastTool = buildBroadcastTool;
 const zod_1 = require("zod");
 const crypto_1 = require("crypto");
 const signals_1 = require("../workflows/signals");
-const helpers_1 = require("./helpers");
+const descriptor_1 = require("./descriptor");
 const validation_1 = require("../utils/validation");
 const search_attributes_1 = require("../utils/search-attributes");
-function registerBroadcastTool(server, client, config, getPlayerId, handle) {
-    (0, helpers_1.defineTool)(server, 'broadcast', 'Send a message to all active players in the ensemble. Optionally filter by player type.', {
-        message: zod_1.z.string().max(validation_1.MESSAGE_MAX).describe('The message to broadcast'),
-        type: zod_1.z.string().optional().describe('Only send to players of this type (e.g., "tempo-soloist")'),
-        includeStale: zod_1.z.boolean().optional().describe('Include disconnected sessions (draining/detached phases; default: false). Argument name kept for backward compatibility.'),
-    }, async (args) => {
-        const { message, type: playerType, includeStale: rawIncludeStale } = args;
-        const includeDisconnected = rawIncludeStale === true;
-        try {
-            const query = `WorkflowType = "agentSessionWorkflow" AND ExecutionStatus = "Running"`;
-            const targets = [];
-            for await (const workflow of client.workflow.list({ query })) {
-                try {
-                    const wfHandle = client.workflow.getHandle(workflow.workflowId);
-                    const metadata = await wfHandle.query('getMetadata');
-                    // Filter by ensemble
-                    if (metadata.ensemble !== config.ensemble)
-                        continue;
-                    // Exclude sender
-                    if (metadata.playerId === getPlayerId())
-                        continue;
-                    // Filter by attachment phase (post-#176). Phase lives on the
-                    // `AgentTempoAttachmentState` search attribute.
-                    const phase = (0, search_attributes_1.getAttachmentPhase)(workflow);
-                    if (!(0, validation_1.shouldIncludeInBroadcast)(phase, includeDisconnected))
-                        continue;
-                    // Filter by player type if specified
-                    if (playerType && metadata.playerType !== playerType)
-                        continue;
-                    targets.push({
-                        playerId: metadata.playerId,
-                        playerType: metadata.playerType,
-                    });
+function buildBroadcastTool(client, config, getPlayerId, handle) {
+    return {
+        name: 'broadcast',
+        description: 'Send a message to all active players in the ensemble. Optionally filter by player type.',
+        params: {
+            message: zod_1.z.string().max(validation_1.MESSAGE_MAX).describe('The message to broadcast'),
+            type: zod_1.z.string().optional().describe('Only send to players of this type (e.g., "tempo-soloist")'),
+            includeStale: zod_1.z.boolean().optional().describe('Include disconnected sessions (draining/detached phases; default: false). Argument name kept for backward compatibility.'),
+        },
+        handler: async (args) => {
+            const { message, type: playerType, includeStale: rawIncludeStale } = args;
+            const includeDisconnected = rawIncludeStale === true;
+            try {
+                const query = `WorkflowType = "agentSessionWorkflow" AND ExecutionStatus = "Running"`;
+                const targets = [];
+                for await (const workflow of client.workflow.list({ query })) {
+                    try {
+                        const wfHandle = client.workflow.getHandle(workflow.workflowId);
+                        const metadata = await wfHandle.query('getMetadata');
+                        // Filter by ensemble
+                        if (metadata.ensemble !== config.ensemble)
+                            continue;
+                        // Exclude sender
+                        if (metadata.playerId === getPlayerId())
+                            continue;
+                        // Filter by attachment phase (post-#176). Phase lives on the
+                        // `AgentTempoAttachmentState` search attribute.
+                        const phase = (0, search_attributes_1.getAttachmentPhase)(workflow);
+                        if (!(0, validation_1.shouldIncludeInBroadcast)(phase, includeDisconnected))
+                            continue;
+                        // Filter by player type if specified
+                        if (playerType && metadata.playerType !== playerType)
+                            continue;
+                        targets.push({
+                            playerId: metadata.playerId,
+                            playerType: metadata.playerType,
+                        });
+                    }
+                    catch {
+                        // Workflow may have just completed — skip it
+                    }
                 }
-                catch {
-                    // Workflow may have just completed — skip it
+                if (targets.length === 0) {
+                    return (0, descriptor_1.ok)('No active players matched the broadcast filter.');
                 }
+                // #357: stamp every fan-out cue with the same `broadcastId` so the
+                // TUI can fold the N deliveries into one chat row. Generated ONCE
+                // here in the MCP-tool process — not inside the workflow — so
+                // workflow determinism is preserved (the workflow only sees the id
+                // as an opaque string on `OutboxEntryInput`).
+                const broadcastId = (0, crypto_1.randomUUID)();
+                // Fan out cue outbox entries for each target.
+                const entryIds = [];
+                for (const target of targets) {
+                    const entry = {
+                        type: 'cue',
+                        targetPlayerId: target.playerId,
+                        message,
+                        broadcastId,
+                    };
+                    const entryId = await handle.executeUpdate(signals_1.submitOutboxUpdate, { args: [entry] });
+                    entryIds.push(entryId);
+                }
+                const names = targets.map((t) => t.playerId);
+                return (0, descriptor_1.ok)(`Broadcast sent to ${targets.length} player${targets.length === 1 ? '' : 's'}: ${names.join(', ')}`);
             }
-            if (targets.length === 0) {
-                return (0, helpers_1.ok)('No active players matched the broadcast filter.');
-            }
-            // #357: stamp every fan-out cue with the same `broadcastId` so the
-            // TUI can fold the N deliveries into one chat row. Generated ONCE
-            // here in the MCP-tool process — not inside the workflow — so
-            // workflow determinism is preserved (the workflow only sees the id
-            // as an opaque string on `OutboxEntryInput`).
-            const broadcastId = (0, crypto_1.randomUUID)();
-            // Fan out cue outbox entries for each target.
-            const entryIds = [];
-            for (const target of targets) {
-                const entry = {
-                    type: 'cue',
-                    targetPlayerId: target.playerId,
-                    message,
-                    broadcastId,
-                };
-                const entryId = await handle.executeUpdate(signals_1.submitOutboxUpdate, { args: [entry] });
-                entryIds.push(entryId);
+            catch (err) {
+                return (0, descriptor_1.fail)(`Failed to broadcast: ${(0, descriptor_1.formatError)(err)}`);
             }
-            const names = targets.map((t) => t.playerId);
-            return (0, helpers_1.ok)(`Broadcast sent to ${targets.length} player${targets.length === 1 ? '' : 's'}: ${names.join(', ')}`);
-        }
-        catch (err) {
-            return (0, helpers_1.fail)(`Failed to broadcast: ${(0, helpers_1.formatError)(err)}`);
-        }
-    });
+        },
+    };
 }

package/dist/tools/cancel-stage.d.ts

@@ -1,3 +1,3 @@
-import { McpServer } from '@modelcontextprotocol/sdk/server/mcp.js';
 import { WorkflowHandle } from '@temporalio/client';
-export declare function registerCancelStageTool(server: McpServer, handle: WorkflowHandle): void;
+import { type TempoToolDescriptor } from './descriptor';
+export declare function buildCancelStageTool(handle: WorkflowHandle): TempoToolDescriptor;

package/dist/tools/cancel-stage.js

@@ -1,20 +1,25 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.registerCancelStageTool = registerCancelStageTool;
+exports.buildCancelStageTool = buildCancelStageTool;
 const zod_1 = require("zod");
-const helpers_1 = require("./helpers");
+const descriptor_1 = require("./descriptor");
 const validation_1 = require("../utils/validation");
-function registerCancelStageTool(server, handle) {
-    (0, helpers_1.defineTool)(server, 'cancel_stage', 'Cancel an active pipeline stage. Players are no longer tracked. Conductor only.', {
-        name: zod_1.z.string().max(validation_1.STAGE_NAME_MAX).describe('Name of the stage to cancel'),
-    }, async (args) => {
-        const { name } = args;
-        try {
-            await handle.signal('cancelStage', name);
-            return (0, helpers_1.ok)(`Stage **${name}** cancelled.`);
-        }
-        catch (err) {
-            return (0, helpers_1.fail)(`Failed to cancel stage: ${(0, helpers_1.formatError)(err)}`);
-        }
-    });
+function buildCancelStageTool(handle) {
+    return {
+        name: 'cancel_stage',
+        description: 'Cancel an active pipeline stage. Players are no longer tracked. Conductor only.',
+        params: {
+            name: zod_1.z.string().max(validation_1.STAGE_NAME_MAX).describe('Name of the stage to cancel'),
+        },
+        handler: async (args) => {
+            const { name } = args;
+            try {
+                await handle.signal('cancelStage', name);
+                return (0, descriptor_1.ok)(`Stage **${name}** cancelled.`);
+            }
+            catch (err) {
+                return (0, descriptor_1.fail)(`Failed to cancel stage: ${(0, descriptor_1.formatError)(err)}`);
+            }
+        },
+    };
 }

package/dist/tools/clear-state.d.ts

@@ -1,3 +1,3 @@
-import { McpServer } from '@modelcontextprotocol/sdk/server/mcp.js';
 import { WorkflowHandle } from '@temporalio/client';
-export declare function registerClearStateTool(server: McpServer, handle: WorkflowHandle): void;
+import { type TempoToolDescriptor } from './descriptor';
+export declare function buildClearStateTool(handle: WorkflowHandle): TempoToolDescriptor;

package/dist/tools/clear-state.js

@@ -1,6 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.registerClearStateTool = registerClearStateTool;
+exports.buildClearStateTool = buildClearStateTool;
 /**
  * `clear_state` — remove one of the calling player's saved-state slots
  * (#334 PR-1, ADR 0011).
@@ -14,24 +14,29 @@ exports.registerClearStateTool = registerClearStateTool;
  */
 const zod_1 = require("zod");
 const signals_1 = require("../workflows/signals");
-const helpers_1 = require("./helpers");
+const descriptor_1 = require("./descriptor");
 const validation_1 = require("../utils/validation");
-function registerClearStateTool(server, handle) {
-    (0, helpers_1.defineTool)(server, 'clear_state', `Clear one of your saved-state slots. Owner-only — you can only clear your own state. Idempotent (clearing an empty slot is a no-op). Returns whether the slot was non-empty before the clear.`, {
-        key: zod_1.z.string().regex(validation_1.PLAYER_STATE_KEY_REGEX).max(validation_1.PLAYER_STATE_KEY_MAX).optional().describe(`Slot name (default "${validation_1.PLAYER_STATE_DEFAULT_KEY}").`),
-    }, async (args) => {
-        const { key } = args;
-        const slotKey = key ?? validation_1.PLAYER_STATE_DEFAULT_KEY;
-        try {
-            const result = await handle.executeUpdate(signals_1.clearPlayerStateUpdate, {
-                args: [{ key: slotKey }],
-            });
-            return (0, helpers_1.ok)(result.cleared
-                ? `Cleared slot **"${slotKey}"**.`
-                : `Slot **"${slotKey}"** was already empty.`);
-        }
-        catch (err) {
-            return (0, helpers_1.fail)(`Failed to clear state: ${(0, helpers_1.formatError)(err)}`);
-        }
-    });
+function buildClearStateTool(handle) {
+    return {
+        name: 'clear_state',
+        description: `Clear one of your saved-state slots. Owner-only — you can only clear your own state. Idempotent (clearing an empty slot is a no-op). Returns whether the slot was non-empty before the clear.`,
+        params: {
+            key: zod_1.z.string().regex(validation_1.PLAYER_STATE_KEY_REGEX).max(validation_1.PLAYER_STATE_KEY_MAX).optional().describe(`Slot name (default "${validation_1.PLAYER_STATE_DEFAULT_KEY}").`),
+        },
+        handler: async (args) => {
+            const { key } = args;
+            const slotKey = key ?? validation_1.PLAYER_STATE_DEFAULT_KEY;
+            try {
+                const result = await handle.executeUpdate(signals_1.clearPlayerStateUpdate, {
+                    args: [{ key: slotKey }],
+                });
+                return (0, descriptor_1.ok)(result.cleared
+                    ? `Cleared slot **"${slotKey}"**.`
+                    : `Slot **"${slotKey}"** was already empty.`);
+            }
+            catch (err) {
+                return (0, descriptor_1.fail)(`Failed to clear state: ${(0, descriptor_1.formatError)(err)}`);
+            }
+        },
+    };
 }

package/dist/tools/coat-check-evict.d.ts

@@ -1,4 +1,4 @@
-import { McpServer } from '@modelcontextprotocol/sdk/server/mcp.js';
 import { Client } from '@temporalio/client';
 import { Config } from '../config';
-export declare function registerCoatCheckEvictTool(server: McpServer, client: Client, config: Config, getPlayerId: () => string): void;
+import { type TempoToolDescriptor } from './descriptor';
+export declare function buildCoatCheckEvictTool(client: Client, config: Config, getPlayerId: () => string): TempoToolDescriptor;

package/dist/tools/coat-check-evict.js

@@ -1,6 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.registerCoatCheckEvictTool = registerCoatCheckEvictTool;
+exports.buildCoatCheckEvictTool = buildCoatCheckEvictTool;
 /**
  * `coat_check_evict` — remove a coat-check entry (#318, ADR 0008) before
  * its TTL expires. Owner-or-conductor only: the workflow validator rejects
@@ -14,30 +14,35 @@ exports.registerCoatCheckEvictTool = registerCoatCheckEvictTool;
 const zod_1 = require("zod");
 const config_1 = require("../config");
 const maestro_signals_1 = require("../workflows/maestro-signals");
-const helpers_1 = require("./helpers");
+const descriptor_1 = require("./descriptor");
 const validation_1 = require("../utils/validation");
-function registerCoatCheckEvictTool(server, client, config, getPlayerId) {
-    (0, helpers_1.defineTool)(server, 'coat_check_evict', `Evict a coat-check entry (#318) before its TTL expires. Owner-or-conductor only — non-owners (and non-conductors) get a permission error.
+function buildCoatCheckEvictTool(client, config, getPlayerId) {
+    return {
+        name: 'coat_check_evict',
+        description: `Evict a coat-check entry (#318) before its TTL expires. Owner-or-conductor only — non-owners (and non-conductors) get a permission error.
 
-Use to free a slot when this ensemble is at the 20-entry cap and you want to make room. \`evicted: false\` means the ticket was already gone (TTL-expired or evicted by someone else).`, {
-        ticket: zod_1.z.string().regex(validation_1.COAT_CHECK_TICKET_REGEX).max(validation_1.COAT_CHECK_TICKET_MAX).describe(`The ticket id returned by an earlier \`coat_check_put\` (≤${validation_1.COAT_CHECK_TICKET_MAX} chars).`),
-    }, async (args) => {
-        const { ticket } = args;
-        const evictedBy = getPlayerId();
-        try {
-            const handle = client.workflow.getHandle((0, config_1.maestroWorkflowId)(config.ensemble));
-            const result = await handle.executeUpdate(maestro_signals_1.coatCheckEvictUpdate, {
-                args: [{ ticket, evictedBy }],
-            });
-            if (!result.evicted) {
-                return (0, helpers_1.ok)(`Ticket **${ticket}** was already gone (no-op).`);
+Use to free a slot when this ensemble is at the 20-entry cap and you want to make room. \`evicted: false\` means the ticket was already gone (TTL-expired or evicted by someone else).`,
+        params: {
+            ticket: zod_1.z.string().regex(validation_1.COAT_CHECK_TICKET_REGEX).max(validation_1.COAT_CHECK_TICKET_MAX).describe(`The ticket id returned by an earlier \`coat_check_put\` (≤${validation_1.COAT_CHECK_TICKET_MAX} chars).`),
+        },
+        handler: async (args) => {
+            const { ticket } = args;
+            const evictedBy = getPlayerId();
+            try {
+                const handle = client.workflow.getHandle((0, config_1.maestroWorkflowId)(config.ensemble));
+                const result = await handle.executeUpdate(maestro_signals_1.coatCheckEvictUpdate, {
+                    args: [{ ticket, evictedBy }],
+                });
+                if (!result.evicted) {
+                    return (0, descriptor_1.ok)(`Ticket **${ticket}** was already gone (no-op).`);
+                }
+                return (0, descriptor_1.ok)(`Evicted ticket **${ticket}**.`);
             }
-            return (0, helpers_1.ok)(`Evicted ticket **${ticket}**.`);
-        }
-        catch (err) {
-            // Surfaces `CoatCheckEvictPermissionDenied` ApplicationFailure with
-            // owner/conductor diagnostic from the workflow validator.
-            return (0, helpers_1.fail)(`Failed to evict ticket: ${(0, helpers_1.formatError)(err)}`);
-        }
-    });
+            catch (err) {
+                // Surfaces `CoatCheckEvictPermissionDenied` ApplicationFailure with
+                // owner/conductor diagnostic from the workflow validator.
+                return (0, descriptor_1.fail)(`Failed to evict ticket: ${(0, descriptor_1.formatError)(err)}`);
+            }
+        },
+    };
 }

package/dist/tools/coat-check-get.d.ts

@@ -1,4 +1,4 @@
-import { McpServer } from '@modelcontextprotocol/sdk/server/mcp.js';
 import { Client } from '@temporalio/client';
 import { Config } from '../config';
-export declare function registerCoatCheckGetTool(server: McpServer, client: Client, config: Config, getPlayerId: () => string): void;
+import { type TempoToolDescriptor } from './descriptor';
+export declare function buildCoatCheckGetTool(client: Client, config: Config, getPlayerId: () => string): TempoToolDescriptor;

package/dist/tools/coat-check-get.js

@@ -1,6 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.registerCoatCheckGetTool = registerCoatCheckGetTool;
+exports.buildCoatCheckGetTool = buildCoatCheckGetTool;
 /**
  * `coat_check_get` — redeem a coat-check ticket and pull the stashed content
  * (#318, ADR 0008). Returns the entry's summary, content body, and audit
@@ -18,39 +18,44 @@ exports.registerCoatCheckGetTool = registerCoatCheckGetTool;
 const zod_1 = require("zod");
 const config_1 = require("../config");
 const maestro_signals_1 = require("../workflows/maestro-signals");
-const helpers_1 = require("./helpers");
+const descriptor_1 = require("./descriptor");
 const validation_1 = require("../utils/validation");
-function registerCoatCheckGetTool(server, client, config, getPlayerId) {
-    (0, helpers_1.defineTool)(server, 'coat_check_get', `Redeem a coat-check ticket (#318) and pull the stashed content. Returns the entry's summary, content body, and audit info — or "not found" when the ticket is missing / expired / evicted (no error, just empty).
+function buildCoatCheckGetTool(client, config, getPlayerId) {
+    return {
+        name: 'coat_check_get',
+        description: `Redeem a coat-check ticket (#318) and pull the stashed content. Returns the entry's summary, content body, and audit info — or "not found" when the ticket is missing / expired / evicted (no error, just empty).
 
-Successful redemptions bump the entry's fetch-audit counters (\`lastFetchedAt\` / \`lastFetchedBy\` / \`fetchCount\`) so the putter can later see whether anyone has redeemed. \`coat_check_list\` won't bump these — only an actual redemption counts.`, {
-        ticket: zod_1.z.string().regex(validation_1.COAT_CHECK_TICKET_REGEX).max(validation_1.COAT_CHECK_TICKET_MAX).describe(`The ticket id returned by an earlier \`coat_check_put\` (≤${validation_1.COAT_CHECK_TICKET_MAX} chars).`),
-    }, async (args) => {
-        const { ticket } = args;
-        const fetchedBy = getPlayerId();
-        try {
-            const handle = client.workflow.getHandle((0, config_1.maestroWorkflowId)(config.ensemble));
-            const entry = await handle.executeUpdate(maestro_signals_1.coatCheckGetUpdate, {
-                args: [{ ticket, fetchedBy }],
-            });
-            if (!entry) {
-                return (0, helpers_1.ok)(`Ticket **${ticket}** is not found (missing, expired, or evicted).`);
+Successful redemptions bump the entry's fetch-audit counters (\`lastFetchedAt\` / \`lastFetchedBy\` / \`fetchCount\`) so the putter can later see whether anyone has redeemed. \`coat_check_list\` won't bump these — only an actual redemption counts.`,
+        params: {
+            ticket: zod_1.z.string().regex(validation_1.COAT_CHECK_TICKET_REGEX).max(validation_1.COAT_CHECK_TICKET_MAX).describe(`The ticket id returned by an earlier \`coat_check_put\` (≤${validation_1.COAT_CHECK_TICKET_MAX} chars).`),
+        },
+        handler: async (args) => {
+            const { ticket } = args;
+            const fetchedBy = getPlayerId();
+            try {
+                const handle = client.workflow.getHandle((0, config_1.maestroWorkflowId)(config.ensemble));
+                const entry = await handle.executeUpdate(maestro_signals_1.coatCheckGetUpdate, {
+                    args: [{ ticket, fetchedBy }],
+                });
+                if (!entry) {
+                    return (0, descriptor_1.ok)(`Ticket **${ticket}** is not found (missing, expired, or evicted).`);
+                }
+                const lines = [
+                    `**Ticket ${ticket}** — stashed by ${entry.putBy} at ${entry.putAt}, expires ${entry.expiresAt}`,
+                    `Summary: ${entry.summary}`,
+                ];
+                if (entry.contentType)
+                    lines.push(`Content-Type: ${entry.contentType}`);
+                lines.push(`Size: ${entry.size} bytes`);
+                lines.push(`Fetches: ${entry.fetchCount}${entry.lastFetchedBy ? ` (last by ${entry.lastFetchedBy} at ${entry.lastFetchedAt})` : ''}`);
+                lines.push('');
+                lines.push('---');
+                lines.push(entry.content);
+                return (0, descriptor_1.ok)(lines.join('\n'));
             }
-            const lines = [
-                `**Ticket ${ticket}** — stashed by ${entry.putBy} at ${entry.putAt}, expires ${entry.expiresAt}`,
-                `Summary: ${entry.summary}`,
-            ];
-            if (entry.contentType)
-                lines.push(`Content-Type: ${entry.contentType}`);
-            lines.push(`Size: ${entry.size} bytes`);
-            lines.push(`Fetches: ${entry.fetchCount}${entry.lastFetchedBy ? ` (last by ${entry.lastFetchedBy} at ${entry.lastFetchedAt})` : ''}`);
-            lines.push('');
-            lines.push('---');
-            lines.push(entry.content);
-            return (0, helpers_1.ok)(lines.join('\n'));
-        }
-        catch (err) {
-            return (0, helpers_1.fail)(`Failed to redeem ticket: ${(0, helpers_1.formatError)(err)}`);
-        }
-    });
+            catch (err) {
+                return (0, descriptor_1.fail)(`Failed to redeem ticket: ${(0, descriptor_1.formatError)(err)}`);
+            }
+        },
+    };
 }

package/dist/tools/coat-check-list.d.ts

@@ -1,4 +1,4 @@
-import { McpServer } from '@modelcontextprotocol/sdk/server/mcp.js';
 import { Client } from '@temporalio/client';
 import { Config } from '../config';
-export declare function registerCoatCheckListTool(server: McpServer, client: Client, config: Config): void;
+import { type TempoToolDescriptor } from './descriptor';
+export declare function buildCoatCheckListTool(client: Client, config: Config): TempoToolDescriptor;

package/dist/tools/coat-check-list.js

@@ -1,6 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.registerCoatCheckListTool = registerCoatCheckListTool;
+exports.buildCoatCheckListTool = buildCoatCheckListTool;
 /**
  * `coat_check_list` — list coat-check entry headers (#318, ADR 0008) for
  * the calling player's ensemble. Read-only — does NOT bump fetch-audit
@@ -12,49 +12,54 @@ exports.registerCoatCheckListTool = registerCoatCheckListTool;
 const zod_1 = require("zod");
 const config_1 = require("../config");
 const maestro_signals_1 = require("../workflows/maestro-signals");
-const helpers_1 = require("./helpers");
+const descriptor_1 = require("./descriptor");
 const validation_1 = require("../utils/validation");
-function registerCoatCheckListTool(server, client, config) {
-    (0, helpers_1.defineTool)(server, 'coat_check_list', `List coat-check entries (#318) for this ensemble. Read-only — does NOT bump fetch-audit counters; only \`coat_check_get\` does. Sorted newest-first. Pass \`unfetchedOnly: true\` to surface entries nobody has redeemed yet — useful for an owner cleaning up stale stashes.`, {
-        putBy: zod_1.z.string().max(validation_1.PLAYER_NAME_MAX).optional().describe('Optional filter — only entries stashed by this player.'),
-        prefix: zod_1.z.string().max(validation_1.COAT_CHECK_SUMMARY_MAX).optional().describe('Optional summary-prefix filter — narrows the listing to entries whose summary starts with this string.'),
-        unfetchedOnly: zod_1.z.boolean().optional().describe('When true, only return entries with fetchCount=0 (never redeemed). Default false.'),
-    }, async (args) => {
-        const { putBy, prefix, unfetchedOnly } = args;
-        try {
-            const handle = client.workflow.getHandle((0, config_1.maestroWorkflowId)(config.ensemble));
-            const filter = {
-                ...(putBy !== undefined ? { putBy } : {}),
-                ...(prefix !== undefined ? { prefix } : {}),
-                ...(unfetchedOnly !== undefined ? { unfetchedOnly } : {}),
-            };
-            const headers = await handle.query(maestro_signals_1.coatCheckListQuery, filter);
-            if (headers.length === 0) {
-                const filters = [];
-                if (putBy)
-                    filters.push(`putBy="${putBy}"`);
-                if (prefix)
-                    filters.push(`prefix="${prefix}"`);
-                if (unfetchedOnly)
-                    filters.push('unfetchedOnly=true');
-                const suffix = filters.length > 0 ? ` (filter: ${filters.join(', ')})` : '';
-                return (0, helpers_1.ok)(`No coat-check entries in this ensemble${suffix}.`);
+function buildCoatCheckListTool(client, config) {
+    return {
+        name: 'coat_check_list',
+        description: `List coat-check entries (#318) for this ensemble. Read-only — does NOT bump fetch-audit counters; only \`coat_check_get\` does. Sorted newest-first. Pass \`unfetchedOnly: true\` to surface entries nobody has redeemed yet — useful for an owner cleaning up stale stashes.`,
+        params: {
+            putBy: zod_1.z.string().max(validation_1.PLAYER_NAME_MAX).optional().describe('Optional filter — only entries stashed by this player.'),
+            prefix: zod_1.z.string().max(validation_1.COAT_CHECK_SUMMARY_MAX).optional().describe('Optional summary-prefix filter — narrows the listing to entries whose summary starts with this string.'),
+            unfetchedOnly: zod_1.z.boolean().optional().describe('When true, only return entries with fetchCount=0 (never redeemed). Default false.'),
+        },
+        handler: async (args) => {
+            const { putBy, prefix, unfetchedOnly } = args;
+            try {
+                const handle = client.workflow.getHandle((0, config_1.maestroWorkflowId)(config.ensemble));
+                const filter = {
+                    ...(putBy !== undefined ? { putBy } : {}),
+                    ...(prefix !== undefined ? { prefix } : {}),
+                    ...(unfetchedOnly !== undefined ? { unfetchedOnly } : {}),
+                };
+                const headers = await handle.query(maestro_signals_1.coatCheckListQuery, filter);
+                if (headers.length === 0) {
+                    const filters = [];
+                    if (putBy)
+                        filters.push(`putBy="${putBy}"`);
+                    if (prefix)
+                        filters.push(`prefix="${prefix}"`);
+                    if (unfetchedOnly)
+                        filters.push('unfetchedOnly=true');
+                    const suffix = filters.length > 0 ? ` (filter: ${filters.join(', ')})` : '';
+                    return (0, descriptor_1.ok)(`No coat-check entries in this ensemble${suffix}.`);
+                }
+                const lines = [];
+                lines.push(`${headers.length}/${validation_1.COAT_CHECK_SLOTS_MAX} coat-check ${headers.length === 1 ? 'entry' : 'entries'}:`);
+                lines.push('');
+                for (const h of headers) {
+                    const fetchTag = h.fetchCount === 0
+                        ? ' (unfetched)'
+                        : ` (fetched ${h.fetchCount}× by ${h.lastFetchedBy ?? '(unknown)'} at ${h.lastFetchedAt ?? '(unknown)'})`;
+                    const typeTag = h.contentType ? ` [${h.contentType}]` : '';
+                    lines.push(`- **${h.ticket}** ${typeTag}— ${h.summary}`);
+                    lines.push(`    putBy=${h.putBy} · putAt=${h.putAt} · expires=${h.expiresAt} · ${h.size} bytes${fetchTag}`);
+                }
+                return (0, descriptor_1.ok)(lines.join('\n'));
             }
-            const lines = [];
-            lines.push(`${headers.length}/${validation_1.COAT_CHECK_SLOTS_MAX} coat-check ${headers.length === 1 ? 'entry' : 'entries'}:`);
-            lines.push('');
-            for (const h of headers) {
-                const fetchTag = h.fetchCount === 0
-                    ? ' (unfetched)'
-                    : ` (fetched ${h.fetchCount}× by ${h.lastFetchedBy ?? '(unknown)'} at ${h.lastFetchedAt ?? '(unknown)'})`;
-                const typeTag = h.contentType ? ` [${h.contentType}]` : '';
-                lines.push(`- **${h.ticket}** ${typeTag}— ${h.summary}`);
-                lines.push(`    putBy=${h.putBy} · putAt=${h.putAt} · expires=${h.expiresAt} · ${h.size} bytes${fetchTag}`);
+            catch (err) {
+                return (0, descriptor_1.fail)(`Failed to list coat-check entries: ${(0, descriptor_1.formatError)(err)}`);
             }
-            return (0, helpers_1.ok)(lines.join('\n'));
-        }
-        catch (err) {
-            return (0, helpers_1.fail)(`Failed to list coat-check entries: ${(0, helpers_1.formatError)(err)}`);
-        }
-    });
+        },
+    };
 }

package/dist/tools/coat-check-put.d.ts

@@ -1,4 +1,4 @@
-import { McpServer } from '@modelcontextprotocol/sdk/server/mcp.js';
 import { Client } from '@temporalio/client';
 import { Config } from '../config';
-export declare function registerCoatCheckPutTool(server: McpServer, client: Client, config: Config, getPlayerId: () => string): void;
+import { type TempoToolDescriptor } from './descriptor';
+export declare function buildCoatCheckPutTool(client: Client, config: Config, getPlayerId: () => string): TempoToolDescriptor;