vellum 0.2.13 → 0.3.2

package/src/tools/network/web-search.ts

@@ -1,319 +0,0 @@
-import { RiskLevel } from '../../permissions/types.js';
-import type { Tool, ToolContext, ToolExecutionResult } from '../types.js';
-import type { ToolDefinition } from '../../providers/types.js';
-import { registerTool } from '../registry.js';
-import { getConfig } from '../../config/loader.js';
-import { getSecureKey } from '../../security/secure-keys.js';
-import { getLogger } from '../../util/logger.js';
-
-const log = getLogger('web-search');
-
-const BRAVE_API_URL = 'https://api.search.brave.com/res/v1/web/search';
-const PERPLEXITY_API_URL = 'https://api.perplexity.ai/chat/completions';
-const RATE_LIMIT_MAX_RETRIES = 3;
-const RATE_LIMIT_BASE_DELAY_MS = 1000;
-
-/**
- * Parse a Retry-After header value into milliseconds.
- * RFC 7231 allows either delta-seconds (e.g. "120") or an HTTP-date
- * (e.g. "Tue, 17 Feb 2026 12:00:00 GMT"). Returns undefined if unparseable.
- */
-function parseRetryAfterMs(value: string): number | undefined {
-  const seconds = Number(value);
-  if (!isNaN(seconds)) {
-    return seconds * 1000;
-  }
-  // Try HTTP-date format — Date.parse handles RFC 2822 / IMF-fixdate
-  const dateMs = Date.parse(value);
-  if (!isNaN(dateMs)) {
-    return Math.max(0, dateMs - Date.now());
-  }
-  return undefined;
-}
-
-type WebSearchProvider = 'perplexity' | 'brave';
-
-interface BraveSearchResult {
-  title: string;
-  url: string;
-  description: string;
-  age?: string;
-  extra_snippets?: string[];
-}
-
-interface BraveSearchResponse {
-  query?: { original: string; more_results_available?: boolean };
-  web?: { results?: BraveSearchResult[] };
-}
-
-interface PerplexityChoice {
-  message?: { content?: string };
-}
-
-interface PerplexityResponse {
-  choices?: PerplexityChoice[];
-  citations?: string[];
-}
-
-function getWebSearchProvider(): WebSearchProvider {
-  const config = getConfig();
-  const configured = config.webSearchProvider ?? 'perplexity';
-  // 'anthropic-native' is handled by the Anthropic client directly;
-  // fall back to perplexity for other providers.
-  if (configured === 'anthropic-native') return 'perplexity';
-  return configured as WebSearchProvider;
-}
-
-function getApiKey(provider: WebSearchProvider): string | undefined {
-  if (provider === 'brave') {
-    if (process.env.BRAVE_API_KEY) return process.env.BRAVE_API_KEY;
-    const secureKey = getSecureKey('brave');
-    if (secureKey) return secureKey;
-    const config = getConfig();
-    return config.apiKeys.brave;
-  }
-
-  // Perplexity
-  if (process.env.PERPLEXITY_API_KEY) return process.env.PERPLEXITY_API_KEY;
-  const secureKey = getSecureKey('perplexity');
-  if (secureKey) return secureKey;
-  const config = getConfig();
-  return config.apiKeys.perplexity;
-}
-
-function formatBraveResults(results: BraveSearchResult[], query: string): string {
-  if (results.length === 0) {
-    return `No results found for "${query}".`;
-  }
-
-  const lines: string[] = [`Web search results for "${query}":\n`];
-
-  for (let i = 0; i < results.length; i++) {
-    const r = results[i];
-    lines.push(`${i + 1}. ${r.title}`);
-    lines.push(`   URL: ${r.url}`);
-    if (r.description) {
-      lines.push(`   ${r.description}`);
-    }
-    if (r.age) {
-      lines.push(`   Age: ${r.age}`);
-    }
-    if (r.extra_snippets && r.extra_snippets.length > 0) {
-      for (const snippet of r.extra_snippets) {
-        lines.push(`   > ${snippet}`);
-      }
-    }
-    lines.push('');
-  }
-
-  return lines.join('\n');
-}
-
-function formatPerplexityResults(data: PerplexityResponse, query: string): string {
-  const content = data.choices?.[0]?.message?.content;
-  if (!content) {
-    return `No results found for "${query}".`;
-  }
-
-  const lines: string[] = [`Web search results for "${query}":\n`];
-  lines.push(content);
-
-  if (data.citations && data.citations.length > 0) {
-    lines.push('\nSources:');
-    for (let i = 0; i < data.citations.length; i++) {
-      lines.push(`  [${i + 1}] ${data.citations[i]}`);
-    }
-  }
-
-  return lines.join('\n');
-}
-
-async function executeBraveSearch(
-  query: string,
-  count: number,
-  offset: number,
-  freshness: string | undefined,
-  apiKey: string,
-): Promise<ToolExecutionResult> {
-  const params = new URLSearchParams({
-    q: query,
-    count: String(count),
-    offset: String(offset),
-  });
-
-  const validFreshness = ['pd', 'pw', 'pm', 'py'];
-  if (freshness && validFreshness.includes(freshness)) {
-    params.set('freshness', freshness);
-  }
-
-  const url = `${BRAVE_API_URL}?${params.toString()}`;
-
-  for (let attempt = 0; attempt <= RATE_LIMIT_MAX_RETRIES; attempt++) {
-    const response = await fetch(url, {
-      headers: {
-        'Accept': 'application/json',
-        'Accept-Encoding': 'gzip',
-        'X-Subscription-Token': apiKey,
-      },
-    });
-
-    if (response.ok) {
-      const data = await response.json() as BraveSearchResponse;
-      const results = data.web?.results ?? [];
-      return { content: formatBraveResults(results, query), isError: false };
-    }
-
-    await response.text();
-
-    if (response.status === 401 || response.status === 403) {
-      return { content: 'Error: Invalid or expired Brave Search API key', isError: true };
-    }
-
-    if (response.status === 429 && attempt < RATE_LIMIT_MAX_RETRIES) {
-      const retryAfter = response.headers.get('retry-after');
-      const parsed = retryAfter ? parseRetryAfterMs(retryAfter) : undefined;
-      const delayMs = parsed ?? RATE_LIMIT_BASE_DELAY_MS * Math.pow(2, attempt);
-      log.warn({ attempt: attempt + 1, delayMs }, 'Brave Search rate limited, retrying');
-      await new Promise(resolve => setTimeout(resolve, delayMs));
-      continue;
-    }
-
-    log.warn({ status: response.status }, 'Brave Search API error');
-    if (response.status === 429) {
-      return { content: 'Error: Brave Search rate limit exceeded after retries. Try again shortly.', isError: true };
-    }
-    return { content: `Error: Brave Search API returned status ${response.status}`, isError: true };
-  }
-
-  return { content: 'Error: Brave Search rate limit exceeded after retries. Try again shortly.', isError: true };
-}
-
-async function executePerplexitySearch(
-  query: string,
-  apiKey: string,
-): Promise<ToolExecutionResult> {
-  for (let attempt = 0; attempt <= RATE_LIMIT_MAX_RETRIES; attempt++) {
-    const response = await fetch(PERPLEXITY_API_URL, {
-      method: 'POST',
-      headers: {
-        'Content-Type': 'application/json',
-        'Authorization': `Bearer ${apiKey}`,
-      },
-      body: JSON.stringify({
-        model: 'sonar',
-        messages: [
-          { role: 'user', content: query },
-        ],
-      }),
-    });
-
-    if (response.ok) {
-      const data = await response.json() as PerplexityResponse;
-      return { content: formatPerplexityResults(data, query), isError: false };
-    }
-
-    await response.text();
-
-    if (response.status === 401 || response.status === 403) {
-      return { content: 'Error: Invalid or expired Perplexity API key', isError: true };
-    }
-
-    if (response.status === 429 && attempt < RATE_LIMIT_MAX_RETRIES) {
-      const retryAfter = response.headers.get('retry-after');
-      const parsed = retryAfter ? parseRetryAfterMs(retryAfter) : undefined;
-      const delayMs = parsed ?? RATE_LIMIT_BASE_DELAY_MS * Math.pow(2, attempt);
-      log.warn({ attempt: attempt + 1, delayMs }, 'Perplexity rate limited, retrying');
-      await new Promise(resolve => setTimeout(resolve, delayMs));
-      continue;
-    }
-
-    log.warn({ status: response.status }, 'Perplexity API error');
-    if (response.status === 429) {
-      return { content: 'Error: Perplexity rate limit exceeded after retries. Try again shortly.', isError: true };
-    }
-    return { content: `Error: Perplexity API returned status ${response.status}`, isError: true };
-  }
-
-  return { content: 'Error: Perplexity rate limit exceeded after retries. Try again shortly.', isError: true };
-}
-
-class WebSearchTool implements Tool {
-  name = 'web_search';
-  description = 'Search the web and return results. Useful for looking up current information, documentation, or anything the assistant doesn\'t know.';
-  category = 'network';
-  defaultRiskLevel = RiskLevel.Low;
-
-  getDefinition(): ToolDefinition {
-    return {
-      name: this.name,
-      description: this.description,
-      input_schema: {
-        type: 'object',
-        properties: {
-          query: {
-            type: 'string',
-            description: 'The search query string',
-          },
-          count: {
-            type: 'number',
-            description: 'Number of results to return (1-20, default 10). Only used with Brave provider.',
-          },
-          offset: {
-            type: 'number',
-            description: 'Pagination offset (0-9, default 0). Only used with Brave provider.',
-          },
-          freshness: {
-            type: 'string',
-            description: 'Filter by recency: "pd" (past day), "pw" (past week), "pm" (past month), "py" (past year). Only used with Brave provider.',
-          },
-        },
-        required: ['query'],
-      },
-    };
-  }
-
-  async execute(input: Record<string, unknown>, _context: ToolContext): Promise<ToolExecutionResult> {
-    const query = input.query;
-    if (!query || typeof query !== 'string') {
-      return { content: 'Error: query is required and must be a string', isError: true };
-    }
-
-    let provider = getWebSearchProvider();
-    let apiKey = getApiKey(provider);
-
-    // Fallback: if the configured provider has no key, try the other provider
-    if (!apiKey) {
-      const fallback: WebSearchProvider = provider === 'perplexity' ? 'brave' : 'perplexity';
-      const fallbackKey = getApiKey(fallback);
-      if (fallbackKey) {
-        log.info({ from: provider, to: fallback }, 'Configured web search provider has no API key, falling back');
-        provider = fallback;
-        apiKey = fallbackKey;
-      } else {
-        return {
-          content: 'Error: No web search API key configured. Set PERPLEXITY_API_KEY or BRAVE_API_KEY environment variable, or configure a key in settings.',
-          isError: true,
-        };
-      }
-    }
-
-    try {
-      log.debug({ query, provider }, 'Executing web search');
-
-      if (provider === 'brave') {
-        const count = typeof input.count === 'number' ? Math.min(20, Math.max(1, Math.round(input.count))) : 10;
-        const offset = typeof input.offset === 'number' ? Math.min(9, Math.max(0, Math.round(input.offset))) : 0;
-        const freshness = typeof input.freshness === 'string' ? input.freshness : undefined;
-        return await executeBraveSearch(query, count, offset, freshness, apiKey);
-      }
-
-      return await executePerplexitySearch(query, apiKey);
-    } catch (err) {
-      const msg = err instanceof Error ? err.message : String(err);
-      log.error({ err }, 'Web search failed');
-      return { content: `Error: Web search failed: ${msg}`, isError: true };
-    }
-  }
-}
-
-registerTool(new WebSearchTool());

package/src/tools/playbooks/index.ts

@@ -1,4 +0,0 @@
-export { executePlaybookCreate } from './playbook-create.js';
-export { executePlaybookList } from './playbook-list.js';
-export { executePlaybookUpdate } from './playbook-update.js';
-export { executePlaybookDelete } from './playbook-delete.js';

package/src/tools/playbooks/playbook-create.ts

@@ -1,96 +0,0 @@
-import { and, eq } from 'drizzle-orm';
-import { v4 as uuid } from 'uuid';
-import type { ToolContext, ToolExecutionResult } from '../types.js';
-import { getDb } from '../../memory/db.js';
-import { computeMemoryFingerprint } from '../../memory/fingerprint.js';
-import { memoryItems } from '../../memory/schema.js';
-import { enqueueMemoryJob } from '../../memory/jobs-store.js';
-import type { Playbook, PlaybookAutonomyLevel } from '../../playbooks/types.js';
-import { truncate } from '../../util/truncate.js';
-
-const VALID_AUTONOMY_LEVELS = new Set<string>(['auto', 'draft', 'notify']);
-
-export async function executePlaybookCreate(input: Record<string, unknown>, context: ToolContext): Promise<ToolExecutionResult> {
-  const trigger = input.trigger as string;
-  const action = input.action as string;
-
-  if (!trigger || typeof trigger !== 'string') {
-    return { content: 'Error: trigger is required and must be a string', isError: true };
-  }
-  if (!action || typeof action !== 'string') {
-    return { content: 'Error: action is required and must be a string', isError: true };
-  }
-
-  const channel = typeof input.channel === 'string' ? input.channel : '*';
-  const category = typeof input.category === 'string' ? input.category : 'general';
-  const autonomyLevel: PlaybookAutonomyLevel =
-    typeof input.autonomy_level === 'string' && VALID_AUTONOMY_LEVELS.has(input.autonomy_level)
-      ? (input.autonomy_level as PlaybookAutonomyLevel)
-      : 'draft';
-  const priority = typeof input.priority === 'number' ? input.priority : 0;
-
-  const playbook: Playbook = { trigger, channel, category, action, autonomyLevel, priority };
-  const statement = JSON.stringify(playbook);
-  const subject = truncate(`Playbook: ${trigger}`, 80, '');
-  const scopeId = context.memoryScopeId ?? 'default';
-
-  const fingerprint = computeMemoryFingerprint(scopeId, 'playbook', subject, statement);
-
-  try {
-    const db = getDb();
-
-    const existing = db
-      .select()
-      .from(memoryItems)
-      .where(and(eq(memoryItems.fingerprint, fingerprint), eq(memoryItems.scopeId, scopeId)))
-      .get();
-
-    if (existing) {
-      return {
-        content: `A playbook with this exact configuration already exists (ID: ${existing.id}).`,
-        isError: false,
-      };
-    }
-
-    const id = uuid();
-    const now = Date.now();
-
-    db.insert(memoryItems).values({
-      id,
-      kind: 'playbook',
-      subject,
-      statement,
-      status: 'active',
-      confidence: 0.95,
-      importance: 0.8,
-      fingerprint,
-      verificationState: 'user_confirmed',
-      scopeId,
-      firstSeenAt: now,
-      lastSeenAt: now,
-      lastUsedAt: null,
-    }).run();
-
-    enqueueMemoryJob('embed_item', { itemId: id });
-
-    const autonomyLabel = autonomyLevel === 'auto' ? 'execute automatically'
-      : autonomyLevel === 'draft' ? 'draft for review' : 'notify only';
-
-    return {
-      content: [
-        'Playbook created successfully.',
-        `  ID: ${id}`,
-        `  Trigger: ${trigger}`,
-        `  Channel: ${channel}`,
-        `  Category: ${category}`,
-        `  Action: ${action}`,
-        `  Autonomy: ${autonomyLabel}`,
-        `  Priority: ${priority}`,
-      ].join('\n'),
-      isError: false,
-    };
-  } catch (err) {
-    const msg = err instanceof Error ? err.message : String(err);
-    return { content: `Error creating playbook: ${msg}`, isError: true };
-  }
-}

package/src/tools/playbooks/playbook-delete.ts

@@ -1,52 +0,0 @@
-import { and, eq } from 'drizzle-orm';
-import type { ToolContext, ToolExecutionResult } from '../types.js';
-import { getDb } from '../../memory/db.js';
-import { memoryItems } from '../../memory/schema.js';
-import { parsePlaybookStatement } from '../../playbooks/types.js';
-
-export async function executePlaybookDelete(input: Record<string, unknown>, context: ToolContext): Promise<ToolExecutionResult> {
-  const playbookId = input.playbook_id as string;
-  if (!playbookId || typeof playbookId !== 'string') {
-    return { content: 'Error: playbook_id is required and must be a string', isError: true };
-  }
-
-  const scopeId = context.memoryScopeId ?? 'default';
-
-  try {
-    const db = getDb();
-
-    const existing = db
-      .select()
-      .from(memoryItems)
-      .where(and(
-        eq(memoryItems.id, playbookId),
-        eq(memoryItems.kind, 'playbook'),
-        eq(memoryItems.scopeId, scopeId),
-      ))
-      .get();
-
-    if (!existing) {
-      return { content: `Error: Playbook with ID "${playbookId}" not found`, isError: true };
-    }
-
-    const playbook = parsePlaybookStatement(existing.statement);
-    const triggerLabel = playbook?.trigger ?? existing.subject;
-
-    // Soft-delete by marking as superseded rather than hard-deleting,
-    // consistent with how other memory items are retired.
-    // Setting invalidAt so the cleanup job can eventually hard-delete it.
-    const now = Date.now();
-    db.update(memoryItems)
-      .set({ status: 'superseded', invalidAt: now })
-      .where(eq(memoryItems.id, existing.id))
-      .run();
-
-    return {
-      content: `Playbook deleted (ID: ${existing.id}, trigger: "${triggerLabel}").`,
-      isError: false,
-    };
-  } catch (err) {
-    const msg = err instanceof Error ? err.message : String(err);
-    return { content: `Error deleting playbook: ${msg}`, isError: true };
-  }
-}

package/src/tools/playbooks/playbook-list.ts

@@ -1,74 +0,0 @@
-import { and, desc, eq, isNull } from 'drizzle-orm';
-import type { ToolContext, ToolExecutionResult } from '../types.js';
-import { getDb } from '../../memory/db.js';
-import { memoryItems } from '../../memory/schema.js';
-import { parsePlaybookStatement } from '../../playbooks/types.js';
-
-export async function executePlaybookList(input: Record<string, unknown>, context: ToolContext): Promise<ToolExecutionResult> {
-  const scopeId = context.memoryScopeId ?? 'default';
-  const channelFilter = typeof input.channel === 'string' ? input.channel : null;
-  const categoryFilter = typeof input.category === 'string' ? input.category : null;
-
-  try {
-    const db = getDb();
-
-    const rows = db
-      .select({
-        id: memoryItems.id,
-        subject: memoryItems.subject,
-        statement: memoryItems.statement,
-        importance: memoryItems.importance,
-        lastSeenAt: memoryItems.lastSeenAt,
-      })
-      .from(memoryItems)
-      .where(and(
-        eq(memoryItems.kind, 'playbook'),
-        eq(memoryItems.status, 'active'),
-        eq(memoryItems.scopeId, scopeId),
-        isNull(memoryItems.invalidAt),
-      ))
-      .orderBy(desc(memoryItems.importance))
-      .all();
-
-    if (rows.length === 0) {
-      return { content: 'No playbooks found.', isError: false };
-    }
-
-    const entries: Array<{ id: string; subject: string; statement: string; playbook: NonNullable<ReturnType<typeof parsePlaybookStatement>> }> = [];
-    for (const row of rows) {
-      const playbook = parsePlaybookStatement(row.statement);
-      if (!playbook) continue;
-
-      // Apply filters
-      if (channelFilter && playbook.channel !== channelFilter && playbook.channel !== '*') continue;
-      if (categoryFilter && playbook.category !== categoryFilter) continue;
-
-      entries.push({ id: row.id, subject: row.subject, statement: row.statement, playbook });
-    }
-
-    if (entries.length === 0) {
-      const filters = [
-        channelFilter ? `channel="${channelFilter}"` : null,
-        categoryFilter ? `category="${categoryFilter}"` : null,
-      ].filter(Boolean).join(', ');
-      return { content: `No playbooks found matching ${filters}.`, isError: false };
-    }
-
-    // Sort by priority descending
-    entries.sort((a, b) => b.playbook.priority - a.playbook.priority);
-
-    const lines: string[] = [`Found ${entries.length} playbook(s):\n`];
-    for (const { id, playbook } of entries) {
-      const channelLabel = playbook.channel === '*' ? 'all channels' : playbook.channel;
-      const autonomyLabel = playbook.autonomyLevel === 'auto' ? 'auto'
-        : playbook.autonomyLevel === 'draft' ? 'draft' : 'notify';
-      lines.push(`- **${playbook.trigger}** (${channelLabel}) → ${playbook.action}`);
-      lines.push(`  _ID: ${id} | category: ${playbook.category} | autonomy: ${autonomyLabel} | priority: ${playbook.priority}_`);
-    }
-
-    return { content: lines.join('\n'), isError: false };
-  } catch (err) {
-    const msg = err instanceof Error ? err.message : String(err);
-    return { content: `Error listing playbooks: ${msg}`, isError: true };
-  }
-}

package/src/tools/playbooks/playbook-update.ts

@@ -1,111 +0,0 @@
-import { and, eq } from 'drizzle-orm';
-import type { ToolContext, ToolExecutionResult } from '../types.js';
-import { getDb } from '../../memory/db.js';
-import { computeMemoryFingerprint } from '../../memory/fingerprint.js';
-import { memoryItems } from '../../memory/schema.js';
-import { enqueueMemoryJob } from '../../memory/jobs-store.js';
-import { parsePlaybookStatement } from '../../playbooks/types.js';
-import type { Playbook, PlaybookAutonomyLevel } from '../../playbooks/types.js';
-import { truncate } from '../../util/truncate.js';
-
-const VALID_AUTONOMY_LEVELS = new Set<string>(['auto', 'draft', 'notify']);
-
-export async function executePlaybookUpdate(input: Record<string, unknown>, context: ToolContext): Promise<ToolExecutionResult> {
-  const playbookId = input.playbook_id as string;
-  if (!playbookId || typeof playbookId !== 'string') {
-    return { content: 'Error: playbook_id is required and must be a string', isError: true };
-  }
-
-  const scopeId = context.memoryScopeId ?? 'default';
-
-  try {
-    const db = getDb();
-
-    const existing = db
-      .select()
-      .from(memoryItems)
-      .where(and(
-        eq(memoryItems.id, playbookId),
-        eq(memoryItems.kind, 'playbook'),
-        eq(memoryItems.scopeId, scopeId),
-      ))
-      .get();
-
-    if (!existing) {
-      return { content: `Error: Playbook with ID "${playbookId}" not found`, isError: true };
-    }
-
-    const currentPlaybook = parsePlaybookStatement(existing.statement);
-    if (!currentPlaybook) {
-      return { content: `Error: Playbook data is corrupted for ID "${playbookId}"`, isError: true };
-    }
-
-    // Merge updates onto existing playbook
-    const updated: Playbook = {
-      trigger: typeof input.trigger === 'string' ? input.trigger : currentPlaybook.trigger,
-      channel: typeof input.channel === 'string' ? input.channel : currentPlaybook.channel,
-      category: typeof input.category === 'string' ? input.category : currentPlaybook.category,
-      action: typeof input.action === 'string' ? input.action : currentPlaybook.action,
-      autonomyLevel:
-        typeof input.autonomy_level === 'string' && VALID_AUTONOMY_LEVELS.has(input.autonomy_level)
-          ? (input.autonomy_level as PlaybookAutonomyLevel)
-          : currentPlaybook.autonomyLevel,
-      priority: typeof input.priority === 'number' ? input.priority : currentPlaybook.priority,
-    };
-
-    const statement = JSON.stringify(updated);
-    const subject = truncate(`Playbook: ${updated.trigger}`, 80, '');
-    const now = Date.now();
-
-    const fingerprint = computeMemoryFingerprint(scopeId, 'playbook', subject, statement);
-
-    // Check if another playbook already has this fingerprint
-    const collision = db
-      .select({ id: memoryItems.id })
-      .from(memoryItems)
-      .where(and(
-        eq(memoryItems.fingerprint, fingerprint),
-        eq(memoryItems.scopeId, scopeId),
-      ))
-      .get();
-    if (collision && collision.id !== existing.id) {
-      return {
-        content: `Error: Another playbook with this exact configuration already exists (ID: ${collision.id}).`,
-        isError: true,
-      };
-    }
-
-    db.update(memoryItems)
-      .set({
-        subject,
-        statement,
-        fingerprint,
-        lastSeenAt: now,
-        verificationState: 'user_confirmed',
-      })
-      .where(eq(memoryItems.id, existing.id))
-      .run();
-
-    enqueueMemoryJob('embed_item', { itemId: existing.id });
-
-    const autonomyLabel = updated.autonomyLevel === 'auto' ? 'execute automatically'
-      : updated.autonomyLevel === 'draft' ? 'draft for review' : 'notify only';
-
-    return {
-      content: [
-        'Playbook updated successfully.',
-        `  ID: ${existing.id}`,
-        `  Trigger: ${updated.trigger}`,
-        `  Channel: ${updated.channel}`,
-        `  Category: ${updated.category}`,
-        `  Action: ${updated.action}`,
-        `  Autonomy: ${autonomyLabel}`,
-        `  Priority: ${updated.priority}`,
-      ].join('\n'),
-      isError: false,
-    };
-  } catch (err) {
-    const msg = err instanceof Error ? err.message : String(err);
-    return { content: `Error updating playbook: ${msg}`, isError: true };
-  }
-}