@indemn/cli 1.7.0 → 1.7.2

package/src/mcp/registry.ts

@@ -20,8 +20,9 @@ type Args = Record<string, unknown>;
 /** `confirmedByPerson` is true when the host asked the person and they agreed. */
 type Handler = (args: any, meta: { confirmedByPerson: boolean }) => Promise<CallToolResult>;
 
+// Compact JSON: the host is a model, and pretty-printing a result costs it about a fifth more tokens.
 function text(data: unknown, isError = false): CallToolResult {
-  return { content: [{ type: 'text', text: JSON.stringify(data, null, 2) }], ...(isError ? { isError: true } : {}) };
+  return { content: [{ type: 'text', text: JSON.stringify(data) }], ...(isError ? { isError: true } : {}) };
 }
 
 /**
@@ -34,14 +35,14 @@ export const MAX_RESULT_CHARS = 80_000;
 
 /** Keep a result inside the size limit: trim a list and say so, or cut other text with a note. */
 function fit(body: { context: ResultContext; data: unknown }): CallToolResult {
-  const full = JSON.stringify(body, null, 2);
+  const full = JSON.stringify(body);
   if (full.length <= MAX_RESULT_CHARS) return text(body);
   const hint = 'Result was too large. Ask for less: use limit/page, a filter, or response_format "concise".';
   const list = Array.isArray(body.data) ? body.data : (body.data as { items?: unknown[] } | null)?.items;
   if (Array.isArray(list)) {
     const shaped = (keep: number) => ({ context: body.context, data: { items: list.slice(0, keep), returned: keep, total: list.length, truncated: true, hint } });
     let keep = list.length;
-    while (keep > 0 && JSON.stringify(shaped(keep), null, 2).length > MAX_RESULT_CHARS) keep = Math.floor(keep * 0.7);
+    while (keep > 0 && JSON.stringify(shaped(keep)).length > MAX_RESULT_CHARS) keep = Math.floor(keep * 0.7);
     return text(shaped(keep));
   }
   return { content: [{ type: 'text', text: `${full.slice(0, MAX_RESULT_CHARS)}\n... [truncated: ${full.length - MAX_RESULT_CHARS} more characters. ${hint}]` }] };
@@ -202,7 +203,11 @@ export function createRegistry(
                   return { content: [{ type: 'text', text: `Error: Not found: ${found.missing.join(', ')}. Nothing was changed and nothing was asked. Look it up again (list tools give the ids) and retry.` }], isError: true };
                 }
                 const names: Record<string, string> = found.names;
-                const named = (s: string) => Object.entries(names).reduce((t, [id, label]) => t.split(id).join(`"${label}"`), s);
+                // Whole ids only, and not inside a link: an org id in a url must stay an id.
+                const named = (s: string) => Object.entries(names).reduce(
+                  (t, [id, label]) => t.replace(new RegExp(`(?<![\\w/=.-])${id.replace(/[.*+?^${}()|[\]\\]/g, '\\$&')}(?![\\w/-])`, 'g'), `"${label}"`),
+                  s,
+                );
                 const base = named(extra.preview ? await extra.preview(args) : meta.describe ? meta.describe(args) : defaultPreview(name, meta, args));
                 const ctx = await opts.context();
                 const orgLabel = org ? org.name ?? names[org.id] ?? org.id : undefined;

package/src/mcp/tool-meta.ts

@@ -47,7 +47,7 @@ export const TOOL_META: Record<string, ToolMeta> = {
   get_agent: read('Get agent'),
   create_agent: write('Create agent'),
   update_agent: { ...write('Update agent', { idempotentHint: true }), guard: true, describe: (a) => `This will change agent ${a.agent_id}.` },
-  delete_agent: { ...destroy('Delete agent'), guard: true, describe: (a) => `This will delete agent ${a.agent_id}. It cannot be restored from the CLI.` },
+  delete_agent: { ...destroy('Delete agent'), guard: true, describe: (a) => `This will delete agent ${a.agent_id}. It cannot be restored.` },
   agent_summary: read('Summarize agent'),
   agent_stats: read('Agent conversation metrics'),
   set_agent_lifecycle: {
@@ -93,8 +93,9 @@ export const TOOL_META: Record<string, ToolMeta> = {
     describe: (a) => `This will create ${((a.functions as unknown[] | undefined) ?? []).length} skill(s) on agent ${a.agent_id}.`,
   },
   list_function_parameters: read('List skill parameters'),
-  add_function_parameter: write('Add skill parameter'),
-  update_function_parameter: write('Update skill parameter', { idempotentHint: true }),
+  // Guarded like update_function: asked only when the agent may be live (liveChange in tools.ts).
+  add_function_parameter: { ...write('Add skill parameter'), guard: true },
+  update_function_parameter: { ...write('Update skill parameter', { idempotentHint: true }), guard: true },
   delete_function_parameter: { ...destroy('Delete skill parameter'), guard: true, describe: (a) => `This will delete parameter ${a.parameter_id} of skill ${a.function_id} on agent ${a.agent_id}.` },
   list_master_functions: { ...read('List skill templates'), paginate: true },
   list_llm_models: { ...read('List LLM models'), paginate: true },
@@ -109,12 +110,28 @@ export const TOOL_META: Record<string, ToolMeta> = {
   add_kb_data: write('Add knowledge base entry'),
   scrape_url: write('Scrape a website into a knowledge base', { openWorldHint: true }),
   upload_kb_file: write('Upload file to knowledge base'),
-  update_kb_data: write('Update knowledge base entry', { idempotentHint: true }),
+  update_kb_data: {
+    ...write('Update knowledge base entry', { idempotentHint: true }),
+    guard: true,
+    describe: (a) => `This will change entry ${a.source_id} of knowledge base ${a.kb_id}${a.answer ? ` to the answer: "${String(a.answer)}"` : ''}.`,
+  },
   delete_kb_data: { ...destroy('Delete knowledge base entry'), guard: true, describe: (a) => `This will delete source ${a.source_id} from knowledge base ${a.kb_id}, with its vectors.` },
-  link_kb: write('Link knowledge bases to agent', { idempotentHint: true }),
+  link_kb: {
+    ...write('Link knowledge bases to agent', { idempotentHint: true }),
+    guard: true,
+    describe: (a) => `This will let agent ${a.agent_id} answer from knowledge base(s) ${((a.kb_ids as string[] | undefined) ?? []).join(', ')}; customers get answers from them right away.`,
+  },
   unlink_kb: { ...destroy('Unlink knowledge base from agent'), guard: true, describe: (a) => `This will unlink knowledge base ${a.kb_id} from agent ${a.agent_id}; the agent stops using it immediately.` },
   export_kb: read('Export knowledge base'),
-  import_kb: { ...destroy('Import into knowledge base'), guard: true, describe: (a) => `This will import new content into knowledge base ${a.kb_id}.` },
+  import_kb: {
+    ...destroy('Import into knowledge base'),
+    guard: true,
+    describe: (a) => {
+      const sources = ((a.data as { sources?: Array<{ name?: string; url?: string }> } | undefined)?.sources ?? []);
+      const files = sources.map((s) => s.name ?? s.url?.split('?')[0].split('/').pop() ?? 'a file');
+      return `This will import ${files.length ? files.map((f) => `"${f}"`).join(', ') : 'new content'} into knowledge base ${a.kb_id}.`;
+    },
+  },
   kb_chunks: read('Show what the agent learned from a source'),
   reimport_kb_source: {
     ...write('Re-crawl a website source', { openWorldHint: true }),
@@ -144,7 +161,8 @@ export const TOOL_META: Record<string, ToolMeta> = {
   attach_phone_number: { ...write('Attach phone number'), guard: true, describe: (a) => `This will attach ${a.phone_number} to agent ${a.agent_id}; its voice settings reset to the defaults. Numbers the org does not own are refused (never bought).` },
   search_phone_numbers: read('Search phone numbers to buy'),
   get_phone_number: read('Get phone number details'),
-  buy_phone_number: { ...write('Buy phone number'), guard: true, describe: (a) => `This will BUY ${a.phone_number} from Twilio and attach it to agent ${a.agent_id}. The org pays for it every month, and it cannot be undone from here.` },
+  // Money leaves the org and nothing here undoes it: shown like a delete.
+  buy_phone_number: { ...destroy('Buy phone number'), guard: true, describe: (a) => `This will BUY ${a.phone_number} from Twilio and attach it to agent ${a.agent_id}. The org pays for it every month, and it cannot be undone from here.` },
   detach_phone_number: { ...destroy('Detach phone number'), guard: true, describe: (a) => `This will detach ${a.phone_number} from agent ${a.agent_id}; calls to it stop reaching the agent.` },
   voice_preview: write('Start a test voice call'),
 
@@ -166,8 +184,9 @@ export const TOOL_META: Record<string, ToolMeta> = {
   get_custom_chart: read('Get custom chart data'),
   get_org_settings: read('Get org settings'),
   set_org_timezone: { ...write('Set org timezone', { idempotentHint: true }), guard: true, describe: (a) => `This will set the org timezone to ${a.timezone}.` },
-  set_business_hours: { ...destroy('Replace business hours', { idempotentHint: true }), guard: true, describe: (a) => `This will replace the org business hours (enabled: ${a.enabled}).` },
+  set_business_hours: { ...destroy('Replace business hours', { idempotentHint: true }), guard: true },
   add_holiday: { ...write('Add holiday'), guard: true, describe: (a) => `This will close the org on ${a.date} (${a.name}).` },
+  remove_holiday: { ...destroy('Remove holiday'), guard: true, describe: (a) => `This will remove the holiday on ${a.date}; the org is open that day again.` },
   set_org_admin_settings: { ...write('Change org admin settings', { idempotentHint: true }), guard: true },
   list_audit_logs: read('List audit log'),
 
@@ -178,7 +197,14 @@ export const TOOL_META: Record<string, ToolMeta> = {
   update_ticket: write('Update, assign or close a ticket'),
   archive_ticket: { ...destroy('Archive ticket'), guard: true, describe: (a) => `This will archive ticket ${a.ticket_id}.` },
   list_ticket_rules: read('List ticket rules'),
-  save_ticket_rule: write('Create or change a ticket rule'),
+  // A standing rule: agents open tickets on their own from now on.
+  save_ticket_rule: {
+    ...write('Create or change a ticket rule'),
+    guard: true,
+    describe: (a) => a.rule_id
+      ? `This will change ticket rule ${a.rule_id}${a.prompt ? ` to open a ticket when: ${a.prompt}` : ''}${a.enabled === false ? ' and turn it off' : ''}.`
+      : `This will add ticket rule "${a.name ?? 'untitled'}": agents open a ticket whenever ${a.prompt}, for ${(a.agent_ids as string[] | undefined)?.length ? 'the chosen agents' : 'every agent'}, from now on.`,
+  },
   delete_ticket_rule: { ...destroy('Delete ticket rule'), guard: true, describe: (a) => `This will delete ticket rule ${a.rule_id}; agents stop opening tickets for it.` },
 
   // Billing (read-only)