@leadbay/mcp 0.42.0 → 0.43.0

package/CHANGELOG.md

@@ -1,5 +1,21 @@
 # Changelog — @leadbay/mcp
 
+## 0.43.0 — 2026-09-30 — a misnamed argument is read or named, never dropped
+
+- **`lead_id` and `leadId` both work.** Leadbay tools spell the lead id
+  differently: researching a lead and adding a note take `leadId`, setting a
+  CRM status takes `lead_ids`. An assistant that used the other spelling was
+  told the id was missing, and a scheduled run lost a night's notes and
+  statuses this way. An argument that differs only by capitals or underscores
+  is now read as the one the tool expects, and setting a status also takes a
+  single lead id.
+- **Any other unknown argument is named.** It used to be dropped silently. It
+  now comes back as an error that names it, suggests the argument it probably
+  meant, and lists the ones the tool takes. A call rejected this way writes
+  nothing.
+- **Team prospecting setup names the right arguments.** It told the assistant
+  to pass `user_prompt` to two tools that take `prompt` and `base`.
+
 ## 0.42.0 — 2026-09-28 — every rep gets the same boards
 
 - **The route planner and the lead triage board are now finished pages.**

package/dist/bin.js

@@ -30318,7 +30318,7 @@ single-country cohort along an axis that does not exist here, and PHASE 3 will p
 those campaigns without ever looking again. The three cases do NOT get the same
 treatment:
 
-- **This workspace's own country** ("plumbers across the US" on a US workspace) \u2192 drop only that clause and keep everything else. Say you dropped it, then continue: the lens covers the whole workspace anyway. **Unless dropping it leaves NOTHING** \u2014 \`audience: "France"\` on an FR workspace is entirely that clause, and what remains is an empty string. Do NOT continue into PHASE 1 with it: \`leadbay_refine_lead_targeting({user_prompt: ""})\` would overwrite my refinement prompt with nothing and \`leadbay_create_lens\` + \`leadbay_promote_lens\` would then persist and ACTIVATE a scopeless lens, to express something this workspace already is. Write nothing at all: tell me the workspace already covers exactly that, and ask for a real sector, size, or sub-country criterion before anything is created. Same for \`rep_split\` \u2014 if the sanitized split is empty, there is no split to make.
+- **This workspace's own country** ("plumbers across the US" on a US workspace) \u2192 drop only that clause and keep everything else. Say you dropped it, then continue: the lens covers the whole workspace anyway. **Unless dropping it leaves NOTHING** \u2014 \`audience: "France"\` on an FR workspace is entirely that clause, and what remains is an empty string. Do NOT continue into PHASE 1 with it: \`leadbay_refine_lead_targeting({prompt: ""})\` would overwrite my refinement prompt with nothing and \`leadbay_create_lens\` + \`leadbay_promote_lens\` would then persist and ACTIVATE a scopeless lens, to express something this workspace already is. Write nothing at all: tell me the workspace already covers exactly that, and ask for a real sector, size, or sub-country criterion before anything is created. Same for \`rep_split\` \u2014 if the sanitized split is empty, there is no split to make.
 - **A different country** ("plumbers across France" on a US workspace) \u2192 **STOP. Create nothing.** Do NOT drop the country and build a lens for this workspace instead \u2014 that would hand me a US lens, plus campaigns, presented as the answer to a France request. Say this workspace holds only its own country's companies, so the ask cannot be filled here, and end your turn.
 - **A supra-national scope** ("plumbers across EMEA") \u2192 also stop: name what the workspace covers and ask whether I want that instead, rather than assuming it.
 
@@ -30333,9 +30333,9 @@ that rep: stop rather than silently handing them an empty campaign or, worse, a
 the home country's leads labelled with another country's name. Carry only the sanitized
 split into PHASE 3.
 
-Call \`leadbay_refine_lead_targeting({user_prompt: "<my audience with the home-country clause removed>"})\` \u2014 pass the SANITIZED text, not the raw argument, or the country label reaches the lens anyway and fences it to a same-named village. This handles the clarification protocol natively \u2014 if the system needs more info (e.g. industry disambiguation, geography precision), it returns \`status: "clarification_needed"\` with options. Surface those to me; on my answer, re-call \`leadbay_refine_lead_targeting\` until the prompt converges.
+Call \`leadbay_refine_lead_targeting({prompt: "<my audience with the home-country clause removed>"})\` \u2014 pass the SANITIZED text, not the raw argument, or the country label reaches the lens anyway and fences it to a same-named village. This handles the clarification protocol natively \u2014 if the system needs more info (e.g. industry disambiguation, geography precision), it returns \`status: "clarification_needed"\` with options. Surface those to me; on my answer, re-call \`leadbay_refine_lead_targeting\` until the prompt converges.
 
-When the prompt has converged, call \`leadbay_create_lens({user_prompt: <refined>, name: "<short descriptive name>"})\` to create a draft lens, then \`leadbay_promote_lens({lensId})\` to make it the active lens.
+When the prompt has converged, call \`leadbay_create_lens({base: <the active lens id>, name: "<short descriptive name>"})\` to create a draft lens, then \`leadbay_promote_lens({lensId})\` to make it the active lens.
 
 # PHASE 2 \u2014 PULL + VALIDATE CANDIDATES
 
@@ -32511,6 +32511,70 @@ function findShapeMismatch(tool, args) {
   }
   return void 0;
 }
+var argKeyNorm = (k) => k.replace(/_/g, "").toLowerCase();
+function editDistance(a, b) {
+  const row = Array.from({ length: b.length + 1 }, (_, j) => j);
+  for (let i = 1; i <= a.length; i++) {
+    let prev = row[0];
+    row[0] = i;
+    for (let j = 1; j <= b.length; j++) {
+      const cur = row[j];
+      row[j] = Math.min(row[j] + 1, row[j - 1] + 1, prev + (a[i - 1] === b[j - 1] ? 0 : 1));
+      prev = cur;
+    }
+  }
+  return row[b.length];
+}
+function normalizeArgKeys(tool, args) {
+  const schema = tool.inputSchema;
+  const props = schema?.properties;
+  if (!props || typeof props !== "object") return { args };
+  const declared = Object.keys(props).filter((k) => k !== ORIGIN_FIELD);
+  const out = {};
+  for (const [key, value] of Object.entries(args)) {
+    if (Object.prototype.hasOwnProperty.call(props, key)) out[key] = value;
+  }
+  const unknown = [];
+  for (const [key, value] of Object.entries(args)) {
+    if (Object.prototype.hasOwnProperty.call(props, key)) continue;
+    const n = argKeyNorm(key);
+    const same = declared.find((d) => argKeyNorm(d) === n);
+    const plural = declared.find(
+      (d) => props[d]?.type === "array" && argKeyNorm(d) === `${n}s`
+    );
+    if (same && !(same in out)) {
+      out[same] = value;
+    } else if (!same && plural && !(plural in out)) {
+      out[plural] = typeof value === "string" && !value.trim().startsWith("[") ? [value] : value;
+    } else {
+      unknown.push(key);
+    }
+  }
+  if (unknown.length === 0) return { args: out };
+  if (schema?.additionalProperties !== false) {
+    for (const key of unknown) out[key] = args[key];
+    return { args: out };
+  }
+  const required = Array.isArray(schema.required) ? schema.required : [];
+  const missing = required.filter((r) => r !== TRIGGERED_BY_FIELD && !(r in out));
+  const suggest = (key) => {
+    const n = argKeyNorm(key);
+    const close2 = declared.map((d) => ({ d, dist: editDistance(n, argKeyNorm(d)) })).filter(({ dist }) => dist <= Math.max(2, Math.floor(n.length / 3))).sort((a, b) => a.dist - b.dist)[0]?.d;
+    return close2 ?? (missing.length === 1 ? missing[0] : void 0);
+  };
+  const named = unknown.map((k) => {
+    const s = suggest(k);
+    return s ? `\`${k}\` (did you mean \`${s}\`?)` : `\`${k}\``;
+  });
+  return {
+    error: {
+      error: true,
+      code: "INVALID_PARAMS",
+      message: `${tool.name} has no argument ${named.join(", ")}`,
+      hint: `Accepted arguments: ${declared.map((d) => `\`${d}\``).join(", ")}. Re-call with those names. Nothing was written.`
+    }
+  };
+}
 function toolsListPayload(tools2) {
   return tools2.map((t) => {
     const out = {
@@ -32774,6 +32838,7 @@ function buildServer(client, opts = {}) {
     }
     const rawArgs = req.params.arguments ?? {};
     const { triggered_by: rawTriggeredBy, origin, cleaned: args } = extractTriggeredBy(rawArgs);
+    const arg_keys = Object.keys(args).slice(0, 20).map((k) => k.slice(0, 64));
     const triggered_by = name === "leadbay_report_friction" ? void 0 : rawTriggeredBy;
     const progressToken = req.params?._meta?.progressToken;
     const progress = progressToken !== void 0 ? (params) => {
@@ -32899,8 +32964,10 @@ ${url}
           isError: true
         };
       }
-      const shapeError = findShapeMismatch(tool, args);
-      const result = shapeError ?? await runWithRequestSignal(extra.signal, () => tool.execute(client, args, {
+      const keyed = normalizeArgKeys(tool, args);
+      const callArgs = keyed.error ? args : keyed.args;
+      const shapeError = keyed.error ?? findShapeMismatch(tool, callArgs);
+      const result = shapeError ?? await runWithRequestSignal(extra.signal, () => tool.execute(client, callArgs, {
         logger: opts.logger,
         notificationsInbox: opts.notificationsInbox,
         signal: extra.signal,
@@ -32990,6 +33057,7 @@ ${url}
             format: "error-envelope",
             bytes: envText.length,
             error_code: envCode,
+            arg_keys,
             triggered_by,
             origin
           });
@@ -33130,6 +33198,7 @@ ${url}
             bytes: errText.length,
             error_code: code,
             ...typeof httpStatus2 === "number" ? { http_status: httpStatus2 } : {},
+            arg_keys,
             triggered_by,
             origin
           });
@@ -33161,6 +33230,7 @@ ${url}
             format: "error-envelope",
             bytes: errText.length,
             error_code: code,
+            arg_keys,
             triggered_by,
             origin
           });
@@ -34646,7 +34716,7 @@ var OAUTH_BASE_URLS = {
     fr: "https://staging.api.leadbay.app"
   }
 };
-var VERSION = "0.42.0";
+var VERSION = "0.43.0";
 var HELP = `
 leadbay-mcp ${VERSION} \u2014 Leadbay Model Context Protocol server
 

package/dist/http-server.js

@@ -2439,7 +2439,7 @@ single-country cohort along an axis that does not exist here, and PHASE 3 will p
 those campaigns without ever looking again. The three cases do NOT get the same
 treatment:
 
-- **This workspace's own country** ("plumbers across the US" on a US workspace) \u2192 drop only that clause and keep everything else. Say you dropped it, then continue: the lens covers the whole workspace anyway. **Unless dropping it leaves NOTHING** \u2014 \`audience: "France"\` on an FR workspace is entirely that clause, and what remains is an empty string. Do NOT continue into PHASE 1 with it: \`leadbay_refine_lead_targeting({user_prompt: ""})\` would overwrite my refinement prompt with nothing and \`leadbay_create_lens\` + \`leadbay_promote_lens\` would then persist and ACTIVATE a scopeless lens, to express something this workspace already is. Write nothing at all: tell me the workspace already covers exactly that, and ask for a real sector, size, or sub-country criterion before anything is created. Same for \`rep_split\` \u2014 if the sanitized split is empty, there is no split to make.
+- **This workspace's own country** ("plumbers across the US" on a US workspace) \u2192 drop only that clause and keep everything else. Say you dropped it, then continue: the lens covers the whole workspace anyway. **Unless dropping it leaves NOTHING** \u2014 \`audience: "France"\` on an FR workspace is entirely that clause, and what remains is an empty string. Do NOT continue into PHASE 1 with it: \`leadbay_refine_lead_targeting({prompt: ""})\` would overwrite my refinement prompt with nothing and \`leadbay_create_lens\` + \`leadbay_promote_lens\` would then persist and ACTIVATE a scopeless lens, to express something this workspace already is. Write nothing at all: tell me the workspace already covers exactly that, and ask for a real sector, size, or sub-country criterion before anything is created. Same for \`rep_split\` \u2014 if the sanitized split is empty, there is no split to make.
 - **A different country** ("plumbers across France" on a US workspace) \u2192 **STOP. Create nothing.** Do NOT drop the country and build a lens for this workspace instead \u2014 that would hand me a US lens, plus campaigns, presented as the answer to a France request. Say this workspace holds only its own country's companies, so the ask cannot be filled here, and end your turn.
 - **A supra-national scope** ("plumbers across EMEA") \u2192 also stop: name what the workspace covers and ask whether I want that instead, rather than assuming it.
 
@@ -2454,9 +2454,9 @@ that rep: stop rather than silently handing them an empty campaign or, worse, a
 the home country's leads labelled with another country's name. Carry only the sanitized
 split into PHASE 3.
 
-Call \`leadbay_refine_lead_targeting({user_prompt: "<my audience with the home-country clause removed>"})\` \u2014 pass the SANITIZED text, not the raw argument, or the country label reaches the lens anyway and fences it to a same-named village. This handles the clarification protocol natively \u2014 if the system needs more info (e.g. industry disambiguation, geography precision), it returns \`status: "clarification_needed"\` with options. Surface those to me; on my answer, re-call \`leadbay_refine_lead_targeting\` until the prompt converges.
+Call \`leadbay_refine_lead_targeting({prompt: "<my audience with the home-country clause removed>"})\` \u2014 pass the SANITIZED text, not the raw argument, or the country label reaches the lens anyway and fences it to a same-named village. This handles the clarification protocol natively \u2014 if the system needs more info (e.g. industry disambiguation, geography precision), it returns \`status: "clarification_needed"\` with options. Surface those to me; on my answer, re-call \`leadbay_refine_lead_targeting\` until the prompt converges.
 
-When the prompt has converged, call \`leadbay_create_lens({user_prompt: <refined>, name: "<short descriptive name>"})\` to create a draft lens, then \`leadbay_promote_lens({lensId})\` to make it the active lens.
+When the prompt has converged, call \`leadbay_create_lens({base: <the active lens id>, name: "<short descriptive name>"})\` to create a draft lens, then \`leadbay_promote_lens({lensId})\` to make it the active lens.
 
 # PHASE 2 \u2014 PULL + VALIDATE CANDIDATES
 
@@ -31013,6 +31013,70 @@ function findShapeMismatch(tool, args) {
   }
   return void 0;
 }
+var argKeyNorm = (k) => k.replace(/_/g, "").toLowerCase();
+function editDistance(a, b) {
+  const row = Array.from({ length: b.length + 1 }, (_, j) => j);
+  for (let i = 1; i <= a.length; i++) {
+    let prev = row[0];
+    row[0] = i;
+    for (let j = 1; j <= b.length; j++) {
+      const cur = row[j];
+      row[j] = Math.min(row[j] + 1, row[j - 1] + 1, prev + (a[i - 1] === b[j - 1] ? 0 : 1));
+      prev = cur;
+    }
+  }
+  return row[b.length];
+}
+function normalizeArgKeys(tool, args) {
+  const schema = tool.inputSchema;
+  const props = schema?.properties;
+  if (!props || typeof props !== "object") return { args };
+  const declared = Object.keys(props).filter((k) => k !== ORIGIN_FIELD);
+  const out = {};
+  for (const [key, value] of Object.entries(args)) {
+    if (Object.prototype.hasOwnProperty.call(props, key)) out[key] = value;
+  }
+  const unknown = [];
+  for (const [key, value] of Object.entries(args)) {
+    if (Object.prototype.hasOwnProperty.call(props, key)) continue;
+    const n = argKeyNorm(key);
+    const same = declared.find((d) => argKeyNorm(d) === n);
+    const plural = declared.find(
+      (d) => props[d]?.type === "array" && argKeyNorm(d) === `${n}s`
+    );
+    if (same && !(same in out)) {
+      out[same] = value;
+    } else if (!same && plural && !(plural in out)) {
+      out[plural] = typeof value === "string" && !value.trim().startsWith("[") ? [value] : value;
+    } else {
+      unknown.push(key);
+    }
+  }
+  if (unknown.length === 0) return { args: out };
+  if (schema?.additionalProperties !== false) {
+    for (const key of unknown) out[key] = args[key];
+    return { args: out };
+  }
+  const required = Array.isArray(schema.required) ? schema.required : [];
+  const missing = required.filter((r) => r !== TRIGGERED_BY_FIELD && !(r in out));
+  const suggest = (key) => {
+    const n = argKeyNorm(key);
+    const close2 = declared.map((d) => ({ d, dist: editDistance(n, argKeyNorm(d)) })).filter(({ dist }) => dist <= Math.max(2, Math.floor(n.length / 3))).sort((a, b) => a.dist - b.dist)[0]?.d;
+    return close2 ?? (missing.length === 1 ? missing[0] : void 0);
+  };
+  const named = unknown.map((k) => {
+    const s = suggest(k);
+    return s ? `\`${k}\` (did you mean \`${s}\`?)` : `\`${k}\``;
+  });
+  return {
+    error: {
+      error: true,
+      code: "INVALID_PARAMS",
+      message: `${tool.name} has no argument ${named.join(", ")}`,
+      hint: `Accepted arguments: ${declared.map((d) => `\`${d}\``).join(", ")}. Re-call with those names. Nothing was written.`
+    }
+  };
+}
 function toolsListPayload(tools2) {
   return tools2.map((t) => {
     const out = {
@@ -31276,6 +31340,7 @@ function buildServer(client, opts = {}) {
     }
     const rawArgs = req.params.arguments ?? {};
     const { triggered_by: rawTriggeredBy, origin, cleaned: args } = extractTriggeredBy(rawArgs);
+    const arg_keys = Object.keys(args).slice(0, 20).map((k) => k.slice(0, 64));
     const triggered_by = name === "leadbay_report_friction" ? void 0 : rawTriggeredBy;
     const progressToken = req.params?._meta?.progressToken;
     const progress = progressToken !== void 0 ? (params) => {
@@ -31401,8 +31466,10 @@ ${url}
           isError: true
         };
       }
-      const shapeError = findShapeMismatch(tool, args);
-      const result = shapeError ?? await runWithRequestSignal(extra.signal, () => tool.execute(client, args, {
+      const keyed = normalizeArgKeys(tool, args);
+      const callArgs = keyed.error ? args : keyed.args;
+      const shapeError = keyed.error ?? findShapeMismatch(tool, callArgs);
+      const result = shapeError ?? await runWithRequestSignal(extra.signal, () => tool.execute(client, callArgs, {
         logger: opts.logger,
         notificationsInbox: opts.notificationsInbox,
         signal: extra.signal,
@@ -31492,6 +31559,7 @@ ${url}
             format: "error-envelope",
             bytes: envText.length,
             error_code: envCode,
+            arg_keys,
             triggered_by,
             origin
           });
@@ -31632,6 +31700,7 @@ ${url}
             bytes: errText.length,
             error_code: code,
             ...typeof httpStatus2 === "number" ? { http_status: httpStatus2 } : {},
+            arg_keys,
             triggered_by,
             origin
           });
@@ -31663,6 +31732,7 @@ ${url}
             format: "error-envelope",
             bytes: errText.length,
             error_code: code,
+            arg_keys,
             triggered_by,
             origin
           });
@@ -31880,7 +31950,7 @@ function parseWriteEnv(env = process.env) {
 }
 
 // src/http-server.ts
-var VERSION = true ? "0.42.0" : "0.0.0-dev";
+var VERSION = true ? "0.43.0" : "0.0.0-dev";
 var PORT = Number(process.env.PORT ?? 8080);
 var HOST = process.env.HOST ?? "0.0.0.0";
 var logger = {

package/dist/installer-electron.js

@@ -1805,7 +1805,7 @@ var init_installer_gui = __esm({
     init_install_dxt();
     init_install_shared();
     init_oauth();
-    VERSION = true ? "0.42.0" : "0.0.0-dev";
+    VERSION = true ? "0.43.0" : "0.0.0-dev";
     MESSAGES = {
       en: {
         installer: {

package/dist/installer-gui.js

@@ -1068,7 +1068,7 @@ async function oauthLogin(opts) {
 }
 
 // installer/installer-gui.ts
-var VERSION = true ? "0.42.0" : "0.0.0-dev";
+var VERSION = true ? "0.43.0" : "0.0.0-dev";
 var MESSAGES = {
   en: {
     installer: {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@leadbay/mcp",
-  "version": "0.42.0",
+  "version": "0.43.0",
   "mcpName": "io.github.leadbay/leadbay-mcp",
   "description": "Model Context Protocol (MCP) server for Leadbay — AI lead discovery, qualification, and enrichment for Claude Desktop, Cursor, and Claude Code.",
   "type": "module",