@setluca/mcp 0.5.0 → 0.5.2

package/dist/task-tools/inbox.js

@@ -5,9 +5,12 @@ import * as Schema from "effect/Schema";
 import { formatError } from "../errors.js";
 import { WORKSPACE_INPUT_FIELDS } from "../fields.js";
 import { logAgentSurfaceEvent } from "../observability.js";
-import { isPartialFailure, listingIds, numberInput, operationResult, resultOrError, taskTool, } from "./runtime.js";
+import { operationById } from "../operations.js";
+import { recordIdsOf } from "../pagination.js";
+import { isPartialFailure, numberInput, operationResult, resultOrError, taskTool, } from "./runtime.js";
 const DEFAULT_QUEUE_LIMIT = 20;
 const DEFAULT_EXPLAIN_TOP = 3;
+const queueIds = recordIdsOf(operationById("reviewQueue.list"));
 const TRIAGE_INBOX_TOOL_NAME = "luca_triage_inbox";
 /**
  * One explanation, with a failed lookup carried in the result instead of
@@ -79,7 +82,7 @@ const triageInbox = taskTool({
         const queue = yield* call("reviewQueue.list", {
             query: { limit: numberInput(input.limit, DEFAULT_QUEUE_LIMIT) },
         });
-        const ids = listingIds(queue, numberInput(input.explainTop, DEFAULT_EXPLAIN_TOP));
+        const ids = queueIds(queue, numberInput(input.explainTop, DEFAULT_EXPLAIN_TOP));
         const attempts = yield* Effect.forEach(ids, (id) => explainOne(call, id), 
         // Effect.forEach defaults to concurrency 1 (sequential). `explainTop`
         // caps this fan-out at 5 independent lookups, so unbounded is fine.

package/dist/task-tools/leads.js

@@ -48,7 +48,7 @@ const flagForHuman = taskTool({
 const draftReply = taskTool({
     name: "luca_draft_reply",
     title: "Draft a reply for a lead",
-    description: 'Draft a reply for a lead, written the way the coach writes, and put it in the SetLuca review queue, where it waits for the coach to approve it. It never sends, whatever the channel\'s auto-send setting. Optional guidance steers tone or content. Example: call with { leadId, guidance: "keep it short and warm" }.',
+    description: "Draft a reply for a lead, written the way the coach writes, and put it in the SetLuca review queue, where it waits for the coach to approve it. Drafting again replaces this lead's earlier unapproved draft from this tool. It never sends, whatever the channel's auto-send setting. Optional guidance steers tone or content. Example: call with { leadId, guidance: \"keep it short and warm\" }.",
     inputSchema: {
         ...WORKSPACE_INPUT_FIELDS,
         leadId: NonEmptyString.annotate({

package/dist/task-tools/runtime.js

@@ -63,38 +63,6 @@ export function taskRequest(operation, input, parts = {}) {
         ...optionalField("workspace", workspaceOverride(input)),
     };
 }
-/**
- * The only part of a listing page a composing tool reads for itself: the ids of
- * the items it then fans out over. Decoding that alone leaves the raw page
- * intact for the caller, returned exactly as the API sent it, while the ids the
- * tool acts on are validated rather than asserted.
- */
-const ListingIds = Schema.Struct({
-    items: Schema.optional(Schema.Array(Schema.Struct({ id: Schema.optional(Schema.String) }))),
-});
-const decodeListingIds = Schema.decodeUnknownOption(ListingIds);
-/**
- * Ids of the first `limit` items on a listing page. A page whose fields we
- * cannot read, or whose items carry no id, contributes nothing. The tool still
- * returns the page itself rather than failing over a fan-out it cannot do.
- */
-export function listingIds(page, limit) {
-    // Both fallbacks below are empty arrays, and substituting a one-item array
-    // for either is an equivalent mutation rather than an untested branch: a
-    // substituted item carries no `id`, so the `filter` at the end of this
-    // function drops it and the result is the same empty id list. Substituting
-    // `undefined` for the outer fallback is a real change and is covered by the
-    // undecodable-page test.
-    const items = decodeListingIds(page).pipe(
-    // Stryker disable next-line ArrayDeclaration: a substituted item has no id and is filtered out below
-    Option.map((decoded) => decoded.items ?? []), 
-    // Stryker disable next-line ArrayDeclaration: a substituted item has no id and is filtered out below
-    Option.getOrElse(() => []));
-    return items
-        .slice(0, limit)
-        .map((item) => item.id)
-        .filter((id) => id !== undefined);
-}
 /**
  * A request body carrying a single text field, or no body at all when the input
  * is absent or blank. Sending an empty string would mean something different to

package/dist/tool.js

@@ -95,6 +95,9 @@ function buildFromTaskTool(tool) {
         // A task tool overwrites or removes existing state when any write it
         // composes does.
         mutatesExisting: composed.some((operation) => operation.mutatesExisting),
+        // Repeating a task tool has no further effect only when that holds for
+        // every operation it composes.
+        idempotent: composed.every((operation) => operation.idempotent),
         ...optionalField("confirm", confirm),
         untrustedContent,
         // A task tool reaches as far as the furthest operation it composes.

package/dist/toolsets.js

@@ -1,8 +1,8 @@
 import { LUCA_TASK_TOOLS } from "./task-tools.js";
 /**
- * The tools the `claude` toolset exposes. This list is the contract with the
- * connector directory submission, and a test pins it name for name. Adding or
- * removing a tool here changes what the directory lists.
+ * The tools the `claude` toolset exposes, and a test pins it name for name.
+ * Coaches who chose `?toolset=claude` rely on it staying read and draft, so a
+ * tool that sends, books, or changes settings does not belong here.
  */
 export const CLAUDE_TOOL_NAMES = [
     "luca_triage_inbox",

package/dist/version.js

@@ -9,6 +9,6 @@
  *
  * Bump this in the same commit as package.json and server.json.
  */
-export const LUCA_MCP_VERSION = "0.5.0";
+export const LUCA_MCP_VERSION = "0.5.2";
 /** The user-agent every outbound Luca API call identifies itself with. */
 export const LUCA_MCP_USER_AGENT = `luca-mcp/${LUCA_MCP_VERSION}`;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@setluca/mcp",
-  "version": "0.5.0",
+  "version": "0.5.2",
   "private": false,
   "description": "Connect SetLuca to Claude, ChatGPT, Cursor, or any MCP client to triage the review queue, look up leads, draft replies, and check campaigns.",
   "keywords": [
@@ -68,15 +68,17 @@
     "registry:check": "bun scripts/check-registry.ts",
     "docs:generate": "bun scripts/generate-docs.ts --write",
     "docs:check": "bun scripts/generate-docs.ts --check",
-    "clean": "rm -rf .turbo dist coverage node_modules"
+    "clean": "rm -rf .turbo dist coverage node_modules",
+    "prepare": "effect-tsgo patch --typescript --no-oxlint"
   },
   "dependencies": {
     "@modelcontextprotocol/server": "^2.0.0",
-    "effect": "4.0.0"
+    "effect": "4.0.6"
   },
   "devDependencies": {
-    "@effect/platform-bun": "4.0.0",
-    "@effect/vitest": "4.0.0",
+    "@effect/platform-bun": "4.0.6",
+    "@effect/tsgo": "0.56.0",
+    "@effect/vitest": "4.0.6",
     "@modelcontextprotocol/client": "^2.0.0",
     "@modelcontextprotocol/inspector": "^2.5.0",
     "@mpsuesser/oxlint-plugin-effect": "0.6.0",