vellum 0.2.13 → 0.3.2

package/src/config/system-prompt.ts

@@ -1,675 +0,0 @@
-import { readFileSync, existsSync, copyFileSync } from 'node:fs';
-import { join } from 'node:path';
-import { getWorkspaceDir, getWorkspacePromptPath, isMacOS } from '../util/platform.js';
-import { getLogger } from '../util/logger.js';
-import { loadSkillCatalog, type SkillSummary } from './skills.js';
-import { getConfig } from './loader.js';
-import { listCredentialMetadata } from '../tools/credentials/metadata-store.js';
-
-const log = getLogger('system-prompt');
-
-const PROMPT_FILES = ['SOUL.md', 'IDENTITY.md', 'USER.md', 'LOOKS.md'] as const;
-
-/**
- * Copy template prompt files into the data directory if they don't already exist.
- * Called once during daemon startup so users always have discoverable files to edit.
- *
- * BOOTSTRAP.md is handled separately: it is only created when *none* of the core
- * prompt files existed beforehand (a truly fresh install).  This prevents the
- * daemon from recreating the file on every restart after the user deletes it to
- * signal that onboarding is complete.
- */
-export function ensurePromptFiles(): void {
-  const templatesDir = join(import.meta.dirname ?? __dirname, 'templates');
-
-  // Track whether this is a fresh workspace (no core prompt files exist yet).
-  const isFirstRun = PROMPT_FILES.every(
-    (file) => !existsSync(getWorkspacePromptPath(file)),
-  );
-
-  for (const file of PROMPT_FILES) {
-    const dest = getWorkspacePromptPath(file);
-    if (existsSync(dest)) continue;
-
-    const src = join(templatesDir, file);
-    try {
-      if (!existsSync(src)) {
-        log.warn({ src }, 'Prompt template not found, skipping');
-        continue;
-      }
-      copyFileSync(src, dest);
-      log.info({ file, dest }, 'Created prompt file from template');
-    } catch (err) {
-      log.warn({ err, file }, 'Failed to create prompt file from template');
-    }
-  }
-
-  // Only seed BOOTSTRAP.md on a truly fresh install so that deleting it
-  // reliably signals onboarding completion across daemon restarts.
-  if (isFirstRun) {
-    const bootstrapDest = getWorkspacePromptPath('BOOTSTRAP.md');
-    if (!existsSync(bootstrapDest)) {
-      const bootstrapSrc = join(templatesDir, 'BOOTSTRAP.md');
-      try {
-        if (existsSync(bootstrapSrc)) {
-          copyFileSync(bootstrapSrc, bootstrapDest);
-          log.info({ file: 'BOOTSTRAP.md', dest: bootstrapDest }, 'Created BOOTSTRAP.md for first-run onboarding');
-        }
-      } catch (err) {
-        log.warn({ err, file: 'BOOTSTRAP.md' }, 'Failed to create BOOTSTRAP.md from template');
-      }
-    }
-  }
-}
-
-/**
- * Returns true when BOOTSTRAP.md has been deleted from the workspace,
- * signalling the first-run ritual is complete.
- */
-export function isOnboardingComplete(): boolean {
-  const bootstrapPath = getWorkspacePromptPath('BOOTSTRAP.md');
-  return !existsSync(bootstrapPath);
-}
-
-/**
- * Build the system prompt from ~/.vellum prompt files,
- * then append a generated skills catalog (if any skills are available).
- *
- * Composition:
- *   1. Base prompt: IDENTITY.md + SOUL.md (guaranteed to exist after ensurePromptFiles)
- *   2. Append USER.md (user profile)
- *   3. If BOOTSTRAP.md exists, append first-run ritual instructions
- *   4. Append skills catalog from ~/.vellum/workspace/skills
- */
-export function buildSystemPrompt(): string {
-  const soulPath = getWorkspacePromptPath('SOUL.md');
-  const identityPath = getWorkspacePromptPath('IDENTITY.md');
-  const userPath = getWorkspacePromptPath('USER.md');
-  const bootstrapPath = getWorkspacePromptPath('BOOTSTRAP.md');
-
-  const looksPath = getWorkspacePromptPath('LOOKS.md');
-
-  const soul = readPromptFile(soulPath);
-  const identity = readPromptFile(identityPath);
-  const user = readPromptFile(userPath);
-  const looks = readPromptFile(looksPath);
-  const bootstrap = readPromptFile(bootstrapPath);
-
-  const parts: string[] = [];
-  if (identity) parts.push(identity);
-  if (soul) parts.push(soul);
-  if (user) parts.push(user);
-  if (looks) parts.push(looks);
-  if (bootstrap) {
-    parts.push(
-      '# First-Run Ritual\n\n'
-      + 'BOOTSTRAP.md is present — this is your first conversation. Follow its instructions.\n\n'
-      + bootstrap,
-    );
-  }
-  parts.push(buildConfigSection());
-  parts.push(buildTaskScheduleReminderRoutingSection());
-  parts.push(buildAttachmentSection());
-  if (!isOnboardingComplete()) {
-    parts.push(buildStarterTaskPlaybookSection());
-  }
-  parts.push(buildToolPermissionSection());
-  parts.push(buildSystemPermissionSection());
-  parts.push(buildChannelAwarenessSection());
-  parts.push(buildSwarmGuidanceSection());
-  parts.push(buildAccessPreferenceSection());
-  parts.push(buildIntegrationSection());
-  parts.push(buildWorkspaceReflectionSection());
-  parts.push(buildLearningMemorySection());
-  parts.push(buildPostToolResponseSection());
-
-  return appendSkillsCatalog(parts.join('\n\n'));
-}
-
-function buildTaskScheduleReminderRoutingSection(): string {
-  return [
-    '## Tool Routing: Tasks vs Schedules vs Reminders',
-    '',
-    'These three systems serve different purposes. Choose the right one based on user intent:',
-    '',
-    '### Task Queue (task_list_add / task_list_show / task_list_update / task_list_remove)',
-    'For tracking things the user wants to do or remember. Use when the user says:',
-    '- "Add to my tasks", "add to my queue", "put this on my task list"',
-    '- "Track this", "I need to do X", "queue this up"',
-    '- Any request to add a one-off item to their personal to-do list',
-    '',
-    'To modify an existing task, use `task_list_update`:',
-    '- "Bump the priority on X", "make X high priority", "move this up"',
-    '- "Change the status of X", "mark X as done"',
-    '- "Update the notes on X"',
-    'Do NOT use `task_list_add` for updates — it will detect duplicates and suggest using `task_list_update` instead.',
-    '',
-    'To remove a task from the queue, use `task_list_remove`:',
-    '- "Remove X from my tasks", "delete that task", "clean up the duplicate"',
-    '- "Take this off the list", "drop this task"',
-    '',
-    'You can create ad-hoc work items by providing just a `title` to `task_list_add` — no existing task template is needed. A lightweight template is auto-created behind the scenes. For reusable task definitions with templates and input schemas, use `task_save` first.',
-    '',
-    '**IMPORTANT:** When you call `task_list_show`, the Tasks window opens automatically on the client AND the tool returns the current task list. Present a brief summary of the tasks in your chat response so the user can see them inline. Do NOT also create a separate surface/UI (via `ui_show` or `app_create`) to display the task queue — that causes duplicate windows.',
-    '',
-    '### Schedules (schedule_create / schedule_list / schedule_update / schedule_delete)',
-    'For recurring automated jobs that run on a recurrence schedule (cron or RRULE). Use ONLY when the user explicitly wants:',
-    '- Recurring automation: "every day at 9am", "weekly on Mondays", "every hour"',
-    '- Complex recurrence patterns: "every other Tuesday", "last weekday of the month" (use RRULE)',
-    '- Bounded recurrence: "every day for 30 days", "weekly until March" (RRULE with COUNT or UNTIL)',
-    '- Periodic background tasks: "check my email every morning", "run this report weekly"',
-    '',
-    '#### RRULE Set Constructs',
-    'When building RRULE expressions, these set lines are supported:',
-    '- **RRULE** — one or more recurrence rules (multiple RRULE lines form a union of occurrences)',
-    '- **RDATE** — add one-off dates that are not covered by the RRULE pattern',
-    '- **EXDATE** — exclude specific dates from the recurrence set',
-    '- **EXRULE** — exclude an entire series of dates defined by a recurrence pattern',
-    '',
-    'Exclusions (EXDATE, EXRULE) take precedence over inclusions (RRULE, RDATE). All RRULE expressions must include a DTSTART line and at least one RRULE or RDATE inclusion.',
-    '',
-    '### Reminders (reminder_create / reminder_list / reminder_cancel)',
-    'For one-time time-triggered notifications. Use ONLY when the user wants:',
-    '- A notification at a specific future time: "remind me at 3pm", "remind me in 2 hours"',
-    '- A timed alert, not a tracked task',
-    '',
-    '### Common mistakes to avoid',
-    '- "Add this to my tasks" → task_list_add (NOT schedule_create or reminder_create)',
-    '- "What\'s on my task list?" → task_list_show (NOT schedule_list)',
-    '- "Remind me to buy groceries" without a time → task_list_add (it\'s a task, not a timed reminder)',
-    '- "Remind me at 5pm to buy groceries" → reminder_create (explicit time trigger)',
-    '- "Check my inbox every morning at 8am" → schedule_create (recurring automation, cron)',
-    '- "Every other Tuesday at 10am" → schedule_create (recurring automation, RRULE)',
-    '- "Every weekday except holidays" → schedule_create (RRULE with EXDATE for exclusions)',
-    '- "Daily for the next 30 days" → schedule_create (RRULE with COUNT=30)',
-    '- "Bump priority on X" → task_list_update (NOT task_list_add)',
-    '- "Move this up" / "change this task priority" → task_list_update (NOT task_list_add)',
-    '- "Mark X as done" → task_list_update (NOT task_list_add)',
-    '- "Remove X from my tasks" → task_list_remove (NOT task_list_update)',
-    '- "Delete that task" / "clean up the duplicate" → task_list_remove',
-    '',
-    '### Entity type routing: work items vs task templates',
-    '',
-    'There are two entity types with separate ID spaces:',
-    '- **Work items** (the user\'s task queue) — managed by task_list_add, task_list_show, task_list_update, task_list_remove',
-    '- **Task templates** (reusable definitions) — managed by task_save, task_list, task_run, task_delete',
-    '',
-    'Do NOT pass a work item ID to a task template tool or vice versa:',
-    '- Deleting a work item from the queue → task_list_remove (NOT task_delete)',
-    '- Deleting a task template → task_delete (NOT task_list_remove)',
-    '- Running a task template → task_run with task_id (NOT a work item ID)',
-    '- Updating a work item → task_list_update with work_item_id (NOT a task template ID)',
-    '',
-    'If an error says "entity mismatch", read the corrective action and selector fields it provides to pick the right tool.',
-  ].join('\n');
-}
-
-function buildAttachmentSection(): string {
-  return [
-    '## Sending Files to the User',
-    '',
-    'To deliver any file you create or download (images, videos, PDFs, audio, etc.) to the user, you MUST include a self-closing XML tag in your response text:',
-    '',
-    '```',
-    '<vellum-attachment source="sandbox" path="scratch/output.png" />',
-    '```',
-    '',
-    '**CRITICAL:** This tag is the ONLY way files reach the user. If you save a file to disk but do not include the tag, the user will NOT see it. Always emit the tag after creating or downloading a file.',
-    '',
-    '- `source`: `sandbox` (default, files inside the sandbox working directory) or `host` (absolute paths on the host filesystem — requires user approval).',
-    '- `path`: Required. Relative path for sandbox, absolute path for host.',
-    '- `filename`: Optional override for the delivered filename (defaults to the basename of the path).',
-    '- `mime_type`: Optional MIME type override (inferred from the file extension if omitted).',
-    '',
-    'Examples:',
-    '```',
-    '<vellum-attachment source="sandbox" path="scratch/chart.png" />',
-    '<vellum-attachment source="sandbox" path="scratch/video.mp4" mime_type="video/mp4" />',
-    '<vellum-attachment source="sandbox" path="scratch/report.pdf" />',
-    '```',
-    '',
-    'Limits: up to 5 attachments per turn, 20 MB each. Tool outputs that produce image or file content blocks are also automatically converted into attachments.',
-    '',
-    '### Inline Images and GIFs',
-    '',
-    'The chat natively renders images and animated GIFs inline in message bubbles. When you have an image or GIF URL (e.g. from Giphy, web search, or any tool), embed it directly in your response text using markdown image syntax:',
-    '',
-    '`![description](https://media.giphy.com/media/example/giphy.gif)`',
-    '',
-    'This renders the image/GIF visually inside the chat bubble with full animation. You can also use `ui_show`, `app_create`, or `vellum-attachment` for images when appropriate. Do NOT wrap image markdown in code fences or it will render as literal text.',
-  ].join('\n');
-}
-
-
-export function buildStarterTaskPlaybookSection(): string {
-  return [
-    '## Starter Task Playbooks',
-    '',
-    'When the user clicks a starter task card in the dashboard, you receive a deterministic kickoff message in the format `[STARTER_TASK:<task_id>]`. Follow the playbook for that task exactly.',
-    '',
-    '### Kickoff intent contract',
-    '- `[STARTER_TASK:make_it_yours]` — "Make it yours" color personalisation flow',
-    '- `[STARTER_TASK:research_topic]` — "Research something for me" flow',
-    '- `[STARTER_TASK:research_to_ui]` — "Turn it into a webpage or interactive UI" flow',
-    '',
-    '### Playbook: make_it_yours',
-    'Goal: Help the user choose an accent color for their dashboard.',
-    '',
-    '1. If the user\'s locale is missing or has `confidence: low` in USER.md, briefly confirm their location/language before proceeding.',
-    '2. Present a concise set of accent color options (e.g. 5-7 curated colors with names and hex codes). Keep it short and scannable.',
-    '3. Let the user pick one. Accept color names, hex values, or descriptions (e.g. "something warm").',
-    '4. Confirm the selection: "I\'ll set your accent color to **{label}** ({hex}). Sound good?"',
-    '5. On confirmation:',
-    '   - Update the `## Dashboard Color Preference` section in USER.md with `label`, `hex`, `source: "user_selected"`, and `applied: true`.',
-    '   - Update the `## Onboarding Tasks` section: set `make_it_yours` to `done`.',
-    '   - Apply the color to the Home Base dashboard using `app_file_edit` to update the theme styles in the Home Base HTML with the chosen accent color.',
-    '6. If the user declines or wants to skip, set `make_it_yours` to `deferred_to_dashboard` in USER.md and move on.',
-    '',
-    '### Playbook: research_topic',
-    'Goal: Research a topic the user is interested in and summarise findings.',
-    '',
-    '1. Ask the user what topic they\'d like researched. Be specific: "What would you like me to look into?"',
-    '2. Once given a topic, use available tools (web search, browser, etc.) to gather information.',
-    '3. Synthesise the findings into a clear, well-structured summary.',
-    '4. Update the `## Onboarding Tasks` section in USER.md: set `research_topic` to `done`.',
-    '',
-    '### Playbook: research_to_ui',
-    'Goal: Transform research (from a prior research_topic task or current conversation context) into a visual webpage or interactive UI.',
-    '',
-    '1. Check the conversation history for prior research content. If none exists, ask the user what content they\'d like visualised.',
-    '2. Synthesise the research into a polished, interactive HTML page using `app_create`.',
-    '3. Follow all Dynamic UI quality standards (anti-AI-slop rules, design tokens, hover states, etc.).',
-    '4. Update the `## Onboarding Tasks` section in USER.md: set `research_to_ui` to `done`.',
-    '',
-    '### General rules for all starter tasks',
-    '- Update the relevant task status in the `## Onboarding Tasks` section of USER.md as you progress (`in_progress` when starting, `done` when complete).',
-    '- Respect trust gating: do NOT ask for elevated permissions during any starter task flow. These are introductory experiences.',
-    '- Keep responses concise and action-oriented. Avoid lengthy explanations of what you\'re about to do.',
-    '- If the user deviates from the flow, adapt gracefully. Complete the task if possible, or mark it as `deferred_to_dashboard`.',
-  ].join('\n');
-}
-
-function buildToolPermissionSection(): string {
-  return [
-    '## Tool Permissions',
-    '',
-    'Some tools (host_bash, host_file_write, host_file_edit, host_file_read) require your user\'s approval before they run. When you call one of these tools, your user sees **Allow / Don\'t Allow** buttons in the chat directly below your message.',
-    '',
-    '**CRITICAL RULE:** You MUST ALWAYS output a text message BEFORE calling any tool that requires approval. NEVER call a permission-gated tool without preceding text. Your user needs context to decide whether to allow.',
-    '',
-    'Your text should follow this pattern:',
-    '1. **Acknowledge** the request conversationally.',
-    '2. **Explain what you need at a high level** (e.g. "I\'ll need to look through your Downloads folder"). Do NOT include raw terminal commands or backtick code. Keep it non-technical.',
-    '3. **State safety** in plain language. Is it read-only? Will it change anything?',
-    '4. **Ask for permission** explicitly at the end.',
-    '',
-    'Style rules:',
-    '- NEVER use em dashes (the long dash). Use commas, periods, or "and" instead.',
-    '- NEVER show raw commands in backticks like `ls -lt ~/Downloads`. Describe the action in plain English.',
-    '- Keep it conversational, like you\'re talking to a friend.',
-    '',
-    'Good examples:',
-    '- "Sure! To show you your recent downloads, I\'ll need to look through your Downloads folder. This is read-only, nothing gets moved or deleted. Can you allow this for me?"',
-    '- "Yes, I can help with that! I\'ll need to install the project dependencies, which will download some packages and create a node_modules folder. Hit Allow to proceed."',
-    '- "Absolutely! I\'ll need to read your shell configuration file to check your setup. I won\'t change anything. Can you allow this?"',
-    '- "I can look into that! I\'ll need to access your contacts database to pull up the info. This is just a read-only lookup, nothing gets modified. Can you allow this?"',
-    '',
-    'Bad examples (NEVER do this):',
-    '- "I\'ll run `ls -lt ~/Desktop/`" (raw command, too technical)',
-    '- "I\'ll list your most recent downloads for you." (doesn\'t ask for permission)',
-    '- Using em dashes anywhere in the response',
-    '- Calling a tool with no preceding text at all',
-    '',
-    'Be conversational and transparent. Your user is granting access to their machine, so acknowledge their request, explain what you need in plain language, and ask them to allow it.',
-    '',
-    '### Handling Permission Denials',
-    '',
-    'When your user denies a tool permission (clicks "Don\'t Allow"), you will receive an error indicating the denial. Follow these rules:',
-    '',
-    '1. **Do NOT immediately retry the tool call.** Retrying without waiting creates another permission prompt, which is annoying and disrespectful of the user\'s decision.',
-    '2. **Acknowledge the denial.** Tell the user that the action was not performed because they chose not to allow it.',
-    '3. **Ask before retrying.** Ask if they would like you to try again, or if they would prefer a different approach.',
-    '4. **Wait for an explicit response.** Only retry the tool call after the user explicitly confirms they want you to try again.',
-    '5. **Offer alternatives.** If possible, suggest alternative approaches that might not require the denied permission.',
-    '',
-    'Example:',
-    '- Tool denied → "No problem! I wasn\'t able to access your Downloads folder since you chose not to allow it. Would you like me to try again, or is there another way I can help?"',
-    '',
-    '### Always-Available Tools (No Approval Required)',
-    '',
-    '- **file_read** on your workspace directory — You can freely read any file under your `.vellum` workspace at any time. Use this proactively to check files, load context, and inform your responses without asking.',
-    '- **web_search** — You can search the web at any time without approval. Use this to look up documentation, current information, or anything you need.',
-  ].join('\n');
-}
-
-function buildSystemPermissionSection(): string {
-  return [
-    '## System Permissions',
-    '',
-    'When a tool execution fails with a permission/access error (e.g. "Operation not permitted", "EACCES", sandbox denial), use `request_system_permission` to ask your user to grant the required macOS permission through System Settings.',
-    '',
-    'Common cases:',
-    '- Reading files in ~/Documents, ~/Desktop, ~/Downloads → `full_disk_access`',
-    '- Screen capture / recording → `screen_recording`',
-    '- Accessibility / UI automation → `accessibility`',
-    '',
-    'Do NOT explain how to open System Settings manually — the tool handles it with a clickable button.',
-  ].join('\n');
-}
-
-export function buildChannelAwarenessSection(): string {
-  return [
-    '## Channel Awareness & Trust Gating',
-    '',
-    'Each turn may include a `<channel_capabilities>` block in the user message describing what the current channel supports. Use this to adapt your behaviour:',
-    '',
-    '### Channel-specific rules',
-    '- When `dashboard_capable` is `false`, never reference the dashboard UI, settings panels, dynamic pages, or visual pickers. Present data as formatted text.',
-    '- When `supports_dynamic_ui` is `false`, do not call `ui_show`, `ui_update`, or `app_create`.',
-    '- When `supports_voice_input` is `false`, do not ask the user to speak or use their microphone.',
-    '- Non-dashboard channels should defer dashboard-specific actions. Tell the user they can complete those steps later from the desktop app.',
-    '',
-    '### Permission ask trust gating',
-    '- Do NOT proactively ask for elevated permissions (microphone, computer control, file access) until the trust stage field `firstConversationComplete` in USER.md is `true`.',
-    '- Even after `firstConversationComplete`, only ask for permissions that are relevant to the current channel capabilities.',
-    '- Do not ask for microphone permissions on channels where `supports_voice_input` is `false`.',
-    '- Do not ask for computer-control permissions on non-dashboard channels.',
-    '- When you do request a permission, be transparent about what it enables and why you need it.',
-  ].join('\n');
-}
-
-export function buildSwarmGuidanceSection(): string {
-  return [
-    '## Parallel Task Orchestration',
-    '',
-    'Use `swarm_delegate` only when a task has **multiple independent parts** that benefit from parallel execution (e.g. "research X, implement Y, and review Z"). For single-focus tasks, work directly — do not decompose them into a swarm.',
-  ].join('\n');
-}
-
-function buildAccessPreferenceSection(): string {
-  return [
-    '## External Service Access Preference',
-    '',
-    'When interacting with external services (GitHub, Slack, Linear, Jira, cloud providers, etc.),',
-    'follow this priority order:',
-    '',
-    '1. **CLI tools via host_bash** — If a CLI is installed on the user\'s machine (gh, slack, linear,',
-    '   jira, aws, gcloud, etc.), use it. CLIs handle auth, pagination, and output formatting.',
-    '   Use --json or equivalent flags for structured output when available.',
-    '2. **Direct API calls via host_bash** — Use curl/httpie with API tokens from credential_store.',
-    '   Faster and more reliable than browser automation.',
-    '3. **web_fetch** — For public endpoints or simple API calls that don\'t need auth.',
-    '4. **Browser automation as last resort** — Only when the task genuinely requires a browser',
-    '   (e.g., no API exists, visual interaction needed, or OAuth consent screen).',
-    '',
-    'Before using browser tools for a business system, ask yourself:',
-    '- Is there a CLI installed? (Check with `cli_discover` or `which <tool>` via host_bash)',
-    '- Does the service have a public API I can call with curl?',
-    '- Can I get the data via web_fetch?',
-    '',
-    'If yes to any of these, use that path instead of the browser.',
-    ...(isMacOS() ? [
-      '',
-      'On macOS, also consider the `macos-automation` skill for interacting with native apps',
-      '(Messages, Contacts, Calendar, Mail, Reminders, Music, Finder, etc.) via osascript.',
-    ] : []),
-  ].join('\n');
-}
-
-function buildIntegrationSection(): string {
-  const allCreds = listCredentialMetadata();
-  // Show OAuth2-connected services (those with oauth2TokenUrl in metadata)
-  const oauthCreds = allCreds.filter((c) => c.oauth2TokenUrl && c.field === 'access_token');
-  if (oauthCreds.length === 0) return '';
-
-  const lines = ['## Connected Services', ''];
-  for (const cred of oauthCreds) {
-    const state = cred.accountInfo
-      ? `Connected (${cred.accountInfo})`
-      : 'Connected';
-    lines.push(`- **${cred.service}**: ${state}`);
-  }
-
-  return lines.join('\n');
-}
-
-function buildWorkspaceReflectionSection(): string {
-  return [
-    '## Workspace Reflection',
-    '',
-    'Before you finish responding to a conversation, pause and consider: did you learn anything worth saving?',
-    '',
-    '- Did your user share personal facts (name, role, timezone, preferences)?',
-    '- Did they correct your behavior or express a preference about how you communicate?',
-    '- Did they mention a project, tool, or workflow you should remember?',
-    '- Did you adapt your style in a way that worked well and should persist?',
-    '',
-    'If yes, briefly explain what you\'re updating, then update the relevant workspace file (USER.md, SOUL.md, or IDENTITY.md) as part of your response.',
-  ].join('\n');
-}
-
-function buildLearningMemorySection(): string {
-  return [
-    '## Learning from Mistakes',
-    '',
-    'When you make a mistake, hit a dead end, or discover something non-obvious, save it to memory so you don\'t repeat it.',
-    '',
-    'Use `memory_save` with `kind: "learning"` for:',
-    '- **Mistakes and corrections** — wrong assumptions, failed approaches, gotchas you ran into',
-    '- **Discoveries** — undocumented behaviors, surprising API quirks, things that weren\'t obvious',
-    '- **Working solutions** — the approach that actually worked after trial and error',
-    '- **Tool/service insights** — rate limits, auth flows, CLI flags that matter',
-    '',
-    'The statement should capture both what happened and the takeaway. Write it as advice to your future self.',
-    '',
-    'Examples:',
-    '- `memory_save({ kind: "learning", subject: "macOS Shortcuts CLI", statement: "shortcuts CLI requires full disk access to export shortcuts — if permission is denied, guide the user to grant it in System Settings rather than retrying." })`',
-    '- `memory_save({ kind: "learning", subject: "Gmail API pagination", statement: "Gmail search returns max 100 results per page. Always check nextPageToken and loop if the user asks for \'all\' messages." })`',
-    '',
-    'Don\'t overthink it. If you catch yourself thinking "I\'ll remember that for next time," save it.',
-  ].join('\n');
-}
-
-function buildPostToolResponseSection(): string {
-  return [
-    '## Tool Call Timing',
-    '',
-    '**Call tools FIRST, explain AFTER:**',
-    '- When a user request requires a tool, call it immediately at the start of your response',
-    '- After the tool call, provide a brief conversational explanation of what you did',
-    '- Do NOT provide conversational preamble before calling the tool',
-    '',
-    'Example (CORRECT):',
-    '  → Call document_create',
-    '  → Text: "I\'ve opened the editor for your blog post about pizza. Let me start writing..."',
-    '',
-    'Example (WRONG):',
-    '  → Text: "I\'ll create a blog post for you about pizza..."',
-    '  → Call document_create  ← Too late! Call tools first.',
-  ].join('\n');
-}
-
-function buildConfigSection(): string {
-  // Always use `file_edit` (not `host_file_edit`) for workspace files — file_edit
-  // handles sandbox path mapping internally, and host_file_edit is permission-gated
-  // which would trigger approval prompts for routine workspace updates.
-  const hostWorkspaceDir = getWorkspaceDir();
-
-  const config = getConfig();
-  const dockerSandboxActive =
-    config.sandbox.enabled && config.sandbox.backend === 'docker';
-  const localWorkspaceDir = dockerSandboxActive
-    ? '/workspace'
-    : hostWorkspaceDir;
-
-  // When Docker sandbox is active, shell commands run inside the container
-  // (use /workspace/) but file_edit/file_read/file_write run on the host
-  // (use the host path). Without Docker, both use the same path.
-  const configPreamble = dockerSandboxActive
-    ? `Your workspace is mounted at \`${localWorkspaceDir}/\` inside the Docker sandbox (host path: \`${hostWorkspaceDir}/\`). For **bash/shell commands** (which run inside Docker), use \`${localWorkspaceDir}/\`. For **file_edit, file_read, and file_write** tools (which run on the host), use the host path \`${hostWorkspaceDir}/\` or relative paths.`
-    : `Your configuration directory is \`${hostWorkspaceDir}/\`.`;
-
-  return [
-    '## Configuration',
-    `- **Active model**: \`${config.model}\` (provider: ${config.provider})`,
-    `${configPreamble} Key files you may read or edit include but are not limited to:`,
-    '',
-    '- `IDENTITY.md` — Your name, nature, personality, and emoji. Updated during the first-run ritual.',
-    '- `SOUL.md` — Core principles, personality, and evolution guidance. Your behavioral foundation.',
-    '- `USER.md` — Profile of your user. Update as you learn about them over time.',
-    '- `LOOKS.md` — Your avatar appearance: body/cheek colors and outfit (hat, shirt, accessory, held item).',
-    '- `BOOTSTRAP.md` — First-run ritual script (only present during onboarding; you delete it when done).',
-    '- `skills/` — Directory of installed skills (loaded automatically at startup).',
-    '',
-    '### Proactive Workspace Editing',
-    '',
-    `You MUST actively update your workspace files as you learn. You don't need to ask your user whether it's okay — just briefly explain what you're updating, then use \`file_edit\` to make targeted edits.`,
-    '',
-    '**USER.md** — update when you learn:',
-    '- Their name or what they prefer to be called',
-    '- Projects they\'re working on, tools they use, languages they code in',
-    '- Communication preferences (concise vs detailed, formal vs casual)',
-    '- Interests, hobbies, or context that helps you assist them better',
-    '- Anything else about your user that will help you serve them better',
-    '',
-    '**SOUL.md** — update when you notice:',
-    '- They prefer a different tone or interaction style (add to Personality or User-Specific Behavior)',
-    '- A behavioral pattern worth codifying (e.g. "always explain before acting", "skip preamble")',
-    '- You\'ve adapted in a way that\'s working well and should persist',
-    '- You decide to change your personality to better serve your user',
-    '',
-    '**IDENTITY.md** — update when:',
-    '- They rename you or change your role',
-    '',
-    '**LOOKS.md** — update when:',
-    '- They ask you to change your appearance, colors, or outfit',
-    '- You want to refresh your look',
-    '- Available body/cheek colors: violet, emerald, rose, amber, indigo, slate, cyan, blue, green, red, orange, pink',
-    '- Available hats: none, top_hat, crown, cap, beanie, wizard_hat, cowboy_hat',
-    '- Available shirts: none, tshirt, suit, hoodie, tank_top, sweater',
-    '- Available accessories: none, sunglasses, monocle, bowtie, necklace, scarf, cape',
-    '- Available held items: none, sword, staff, shield, balloon',
-    '- Available outfit colors: red, blue, yellow, purple, orange, pink, cyan, brown, black, white, gold, silver',
-    '',
-    'When updating, read the file first, then make a targeted edit. Include all useful information, but don\'t bloat the files over time',
-  ].join('\n');
-}
-
-/**
- * Strip lines starting with `_` (comment convention for prompt .md files)
- * and collapse any resulting consecutive blank lines.
- *
- * Lines inside fenced code blocks (``` or ~~~ delimiters per CommonMark)
- * are never stripped, so code examples with `_`-prefixed identifiers are preserved.
- */
-export function stripCommentLines(content: string): string {
-  const normalized = content.replace(/\r\n/g, '\n');
-  let openFenceChar: string | null = null;
-  const filtered = normalized.split('\n').filter((line) => {
-    const fenceMatch = line.match(/^ {0,3}(`{3,}|~{3,})/);
-    if (fenceMatch) {
-      const char = fenceMatch[1][0];
-      if (!openFenceChar) {
-        openFenceChar = char;
-      } else if (char === openFenceChar) {
-        openFenceChar = null;
-      }
-    }
-    if (openFenceChar) return true;
-    return !line.trimStart().startsWith('_');
-  });
-  return filtered
-    .join('\n')
-    .replace(/\n{3,}/g, '\n\n')
-    .trim();
-}
-
-function readPromptFile(path: string): string | null {
-  if (!existsSync(path)) return null;
-
-  try {
-    const content = stripCommentLines(readFileSync(path, 'utf-8'));
-    if (content.length === 0) return null;
-    log.debug({ path }, 'Loaded prompt file');
-    return content;
-  } catch (err) {
-    log.warn({ err, path }, 'Failed to read prompt file');
-    return null;
-  }
-}
-
-function appendSkillsCatalog(basePrompt: string): string {
-  const skills = loadSkillCatalog();
-
-  const sections: string[] = [basePrompt];
-
-  const catalog = formatSkillsCatalog(skills);
-  if (catalog) sections.push(catalog);
-
-  sections.push(buildDynamicSkillWorkflowSection());
-
-  return sections.join('\n\n');
-}
-
-function buildDynamicSkillWorkflowSection(): string {
-  return [
-    '## Dynamic Skill Authoring Workflow',
-    '',
-    'When your user requests a capability that no existing tool or skill can satisfy, follow this exact procedure:',
-    '',
-    '1. **Validate the gap.** Confirm no existing tool or installed skill covers the need.',
-    '2. **Draft a TypeScript snippet.** Write a self-contained snippet that exports a `default` or `run` function with signature `(input: unknown) => unknown | Promise<unknown>`.',
-    '3. **Test with `evaluate_typescript_code`.** Call the tool to run the snippet in a sandbox. Iterate until it passes.',
-    '4. **Persist with `scaffold_managed_skill`.** Only after successful evaluation and explicit user consent, call `scaffold_managed_skill` to write the skill to `~/.vellum/workspace/skills/<id>/`.',
-    '5. **Load and use.** Call `skill_load` with the new skill ID before invoking the skill-driven flow.',
-    '',
-    'Important constraints:',
-    '- **Never persist or delete skills without explicit user confirmation.** Both operations require user approval.',
-    '- If evaluation fails after 3 attempts, summarize the failure and ask your user for guidance instead of continuing to retry.',
-    '- After a skill is written or deleted, the next turn may run in a recreated session due to file-watcher eviction. Continue normally.',
-    '- To remove a managed skill, use `delete_managed_skill`.',
-    '',
-    '### Browser Skill Prerequisite',
-    'If you need browser capabilities (navigating web pages, clicking elements, extracting content) and `browser_*` tools are not available, load the "browser" skill first using `skill_load`.',
-    '',
-    '### X (Twitter) Skill',
-    'When the user asks to post, reply, or interact with X/Twitter, load the "twitter" skill using `skill_load`. Do NOT use computer-use or the browser skill for X — the X skill provides CLI commands (`vellum x post`, `vellum x reply`) that are faster and more reliable.',
-  ].join('\n');
-}
-
-function escapeXml(str: string): string {
-  return str
-    .replace(/&/g, '&amp;')
-    .replace(/</g, '&lt;')
-    .replace(/>/g, '&gt;')
-    .replace(/"/g, '&quot;')
-    .replace(/'/g, '&apos;');
-}
-
-function formatSkillsCatalog(skills: SkillSummary[]): string {
-  // Filter out skills with disableModelInvocation or unsupported OS
-  const visible = skills.filter(s => {
-    if (s.disableModelInvocation) return false;
-    const os = s.metadata?.os;
-    if (os && os.length > 0 && !os.includes(process.platform)) return false;
-    return true;
-  });
-  if (visible.length === 0) return '';
-
-  const lines = ['<available_skills>'];
-  for (const skill of visible) {
-    const idAttr = escapeXml(skill.id);
-    const nameAttr = escapeXml(skill.name);
-    const descAttr = escapeXml(skill.description);
-    const locAttr = escapeXml(skill.directoryPath);
-    lines.push(`<skill id="${idAttr}" name="${nameAttr}" description="${descAttr}" location="${locAttr}" />`);
-  }
-  lines.push('</available_skills>');
-
-  return [
-    '## Available Skills',
-    'The following skills are available. Before executing one, call the `skill_load` tool with its `id` to load the full instructions.',
-    '',
-    lines.join('\n'),
-  ].join('\n');
-}