botholomew 0.15.3 → 0.15.5

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "botholomew",
-  "version": "0.15.3",
+  "version": "0.15.5",
   "description": "An autonomous AI agent for knowledge work — works your task queue while you sleep.",
   "type": "module",
   "bin": {

package/src/chat/agent.ts

@@ -35,11 +35,13 @@ import {
 
 registerAllTools();
 
-/** Tools available in chat mode — no worker terminal tools (complete/fail/wait), no bulk-destructive file tools (delete, copy/move, dir ops) */
+/** Tools available in chat mode — no worker terminal tools (complete/fail/wait), no bulk-destructive file tools (copy/move, dir ops) */
 const CHAT_TOOL_NAMES = new Set([
   "create_task",
   "list_tasks",
   "view_task",
+  "update_task",
+  "delete_task",
   "context_info",
   "context_tree",
   "context_read",
@@ -50,9 +52,11 @@ const CHAT_TOOL_NAMES = new Set([
   "view_thread",
   "search_threads",
   "create_schedule",
+  "schedule_edit",
   "list_schedules",
-  "update_beliefs",
-  "update_goals",
+  "prompt_read",
+  "prompt_edit",
+  "task_edit",
   "capabilities_refresh",
   "mcp_list_tools",
   "mcp_search",
@@ -98,7 +102,7 @@ You do NOT execute long-running work directly — enqueue tasks for a background
 Use the available tools to look up tasks, threads, schedules, and context when the user asks about them. Files the agent can read and write live under \`context/\` as project-relative paths (e.g. \`notes/foo.md\`). Use \`context_tree\` to see what's there, \`search\` (hybrid regexp + semantic) to find content, then \`context_read\` / \`context_info\` to drill in.
 Past conversations live in CSV files under \`threads/\`; use \`list_threads\`, \`search_threads\`, and \`view_thread\` to find and page through them.
 When multiple tool calls are independent of each other (i.e., one does not depend on the result of another), call them all in a single response. They will be executed in parallel, which is faster than calling them one at a time.
-You can update the agent's beliefs and goals files when the user asks you to.
+You can read and edit the agent's prompt files (beliefs, goals, capabilities, soul) under \`prompts/\` via \`prompt_read\` and \`prompt_edit\` (line-range patches). Files marked \`agent-modification: false\` (e.g. soul.md) are read-only.
 You can author and refine slash-command skills (reusable prompt templates stored in \`skills/\`) via \`skill_list\`, \`skill_search\`, \`skill_read\`, \`skill_write\`, \`skill_edit\`, and \`skill_delete\`. New or edited skills are usable as \`/<name>\` on the user's next message.
 Format your responses using Markdown. Use headings, bold, italic, lists, and code blocks to make your responses clear and well-structured.
 `;
@@ -156,6 +160,11 @@ export interface ChatTurnCallbacks {
     isError: boolean,
     meta?: ToolEndMeta,
   ) => void;
+  /** Side-effect notification from inside a tool ("Created subtask: …"). The
+   *  TUI renders these inside the tool-call card so they stay anchored to the
+   *  tool that produced them. Workers don't supply this; tools fall back to
+   *  `logger.info`. */
+  onToolNotify?: (toolUseId: string, message: string) => void;
   /** Called between LLM turns. The TUI returns any queued user messages so
    *  the agent can inject them into the running turn instead of waiting for
    *  the entire tool loop to finish. Each returned message is logged + pushed
@@ -245,7 +254,7 @@ export async function runChatTurn(input: {
     // Rebuild the system prompt every iteration so that:
     //   (1) `loading: contextual` files get matched against the latest user
     //       message, and
-    //   (2) any update_beliefs / update_goals tool call in the previous
+    //   (2) any prompt_edit tool call in the previous
     //       iteration is reflected in the next LLM call.
     const keywordSource = findLastUserText(messages);
     const systemPrompt = await buildChatSystemPrompt(projectDir, {
@@ -406,6 +415,9 @@ export async function runChatTurn(input: {
           config,
           mcpxClient,
           shouldAbort: session ? () => session.aborted : undefined,
+          notify: callbacks.onToolNotify
+            ? (msg) => callbacks.onToolNotify?.(toolUse.id, msg)
+            : undefined,
         });
         const durationMs = Date.now() - start;
         const stored = maybeStoreResult(toolUse.name, result.output);
@@ -454,6 +466,7 @@ interface ChatToolCallCtx {
   config: Required<BotholomewConfig>;
   mcpxClient: McpxClient | null;
   shouldAbort?: () => boolean;
+  notify?: (message: string) => void;
 }
 
 async function executeChatToolCall(

package/src/commands/chat.ts

@@ -8,8 +8,9 @@ export function registerChatCommand(program: Command) {
       "Open the interactive chat TUI\n\n" +
         "  Tab navigation (Ctrl+<letter> from any tab):\n" +
         "    Ctrl+a  Chat        Ctrl+t  Tasks       Ctrl+w  Workers\n" +
-        "    Ctrl+o  Tools       Ctrl+r  Threads     Ctrl+g  Help\n" +
+        "    Ctrl+o  Tools       Ctrl+e  Threads     Ctrl+g  Help\n" +
         "    Ctrl+n  Context     Ctrl+s  Schedules   Esc     Return to Chat\n\n" +
+        "  Refresh: Ctrl+R refreshes Context · Tasks · Threads · Schedules · Workers\n\n" +
         "  Chat input:\n" +
         "    Enter          Send message\n" +
         "    ⌥+Enter        Insert newline\n" +

package/src/context/store.ts

@@ -13,6 +13,7 @@ import {
 import { dirname, join, posix, relative, sep } from "node:path";
 import { CONTEXT_DIR, PROTECTED_AREAS } from "../constants.ts";
 import { atomicWrite } from "../fs/atomic.ts";
+import { applyLinePatches, type LinePatch } from "../fs/patches.ts";
 import {
   getCanonicalRoot,
   PathEscapeError,
@@ -59,11 +60,7 @@ export class PathConflictError extends Error {
   }
 }
 
-export interface Patch {
-  start_line: number;
-  end_line: number;
-  content: string;
-}
+export type Patch = LinePatch;
 
 export interface ContextEntry {
   /** Project-relative path under context/, e.g. "notes/foo.md". Forward-slashes. */
@@ -775,26 +772,11 @@ export async function applyPatches(
   patches: Patch[],
 ): Promise<{ applied: number; lines: number }> {
   const content = await readContextFile(projectDir, path);
-  const lines = content.split("\n");
-
-  const sorted = [...patches].sort((a, b) => b.start_line - a.start_line);
-
-  for (const patch of sorted) {
-    if (patch.end_line === 0) {
-      const insertLines = patch.content === "" ? [] : patch.content.split("\n");
-      lines.splice(patch.start_line - 1, 0, ...insertLines);
-    } else {
-      const deleteCount = patch.end_line - patch.start_line + 1;
-      const insertLines = patch.content === "" ? [] : patch.content.split("\n");
-      lines.splice(patch.start_line - 1, deleteCount, ...insertLines);
-    }
-  }
-
-  const newContent = lines.join("\n");
+  const newContent = applyLinePatches(content, patches);
   await writeContextFile(projectDir, path, newContent, {
     onConflict: "overwrite",
   });
-  return { applied: patches.length, lines: lines.length };
+  return { applied: patches.length, lines: newContent.split("\n").length };
 }
 
 /**

package/src/fs/patches.ts

@@ -0,0 +1,39 @@
+import { z } from "zod";
+
+export interface LinePatch {
+  start_line: number;
+  end_line: number;
+  content: string;
+}
+
+export const LinePatchSchema = z.object({
+  start_line: z.number().describe("1-based inclusive start line"),
+  end_line: z
+    .number()
+    .describe("1-based inclusive end line (0 to insert without replacing)"),
+  content: z
+    .string()
+    .describe("Replacement text (empty string to delete lines)"),
+});
+
+/**
+ * Apply git-style line-range patches to a string. Patches are applied
+ * bottom-up so earlier line numbers stay stable. `end_line === 0` is an
+ * insert that doesn't replace; an empty `content` deletes.
+ */
+export function applyLinePatches(raw: string, patches: LinePatch[]): string {
+  const lines = raw.split("\n");
+  const sorted = [...patches].sort((a, b) => b.start_line - a.start_line);
+
+  for (const patch of sorted) {
+    const insertLines = patch.content === "" ? [] : patch.content.split("\n");
+    if (patch.end_line === 0) {
+      lines.splice(patch.start_line - 1, 0, ...insertLines);
+    } else {
+      const deleteCount = patch.end_line - patch.start_line + 1;
+      lines.splice(patch.start_line - 1, deleteCount, ...insertLines);
+    }
+  }
+
+  return lines.join("\n");
+}

package/src/schedules/store.ts

@@ -19,7 +19,7 @@ import {
   ScheduleFrontmatterSchema,
 } from "./schema.ts";
 
-function scheduleFilePath(projectDir: string, id: string): string {
+export function scheduleFilePath(projectDir: string, id: string): string {
   return join(getSchedulesDir(projectDir), `${id}.md`);
 }
 
@@ -27,20 +27,26 @@ function scheduleLockPath(projectDir: string, id: string): string {
   return join(getSchedulesLockDir(projectDir), `${id}.lock`);
 }
 
-function serializeSchedule(fm: ScheduleFrontmatter, body: string): string {
+export function serializeSchedule(
+  fm: ScheduleFrontmatter,
+  body: string,
+): string {
   return matter.stringify(`\n${body.trim()}\n`, fm as Record<string, unknown>);
 }
 
-interface ParseOk {
+export interface ScheduleParseOk {
   ok: true;
   schedule: Schedule;
 }
-interface ParseFail {
+export interface ScheduleParseFail {
   ok: false;
   reason: string;
 }
 
-function parseScheduleFile(raw: string, mtimeMs: number): ParseOk | ParseFail {
+export function parseScheduleFile(
+  raw: string,
+  mtimeMs: number,
+): ScheduleParseOk | ScheduleParseFail {
   let parsed: matter.GrayMatterFile<string>;
   try {
     parsed = matter(raw);

package/src/tasks/store.ts

@@ -21,7 +21,7 @@ import {
   type TaskStatus,
 } from "./schema.ts";
 
-function taskFilePath(projectDir: string, id: string): string {
+export function taskFilePath(projectDir: string, id: string): string {
   return join(getTasksDir(projectDir), `${id}.md`);
 }
 
@@ -33,23 +33,23 @@ function taskLockPath(projectDir: string, id: string): string {
  * Render a Task to its on-disk markdown form. Frontmatter contains every
  * field; the body is preserved as-is. Trailing newline keeps line count sane.
  */
-function serializeTask(fm: TaskFrontmatter, body: string): string {
+export function serializeTask(fm: TaskFrontmatter, body: string): string {
   return matter.stringify(`\n${body.trim()}\n`, fm as Record<string, unknown>);
 }
 
-interface ParseResult {
+export interface TaskParseOk {
   ok: true;
   task: Task;
 }
-interface ParseFailure {
+export interface TaskParseFail {
   ok: false;
   reason: string;
 }
 
-function parseTaskFile(
+export function parseTaskFile(
   raw: string,
   mtimeMs: number,
-): ParseResult | ParseFailure {
+): TaskParseOk | TaskParseFail {
   let parsed: matter.GrayMatterFile<string>;
   try {
     parsed = matter(raw);

package/src/tools/file/edit.ts

@@ -5,21 +5,12 @@ import {
   NotFoundError,
   readContextFile,
 } from "../../context/store.ts";
+import { LinePatchSchema } from "../../fs/patches.ts";
 import type { ToolDefinition } from "../tool.ts";
 
-const PatchSchema = z.object({
-  start_line: z.number().describe("1-based inclusive start line"),
-  end_line: z
-    .number()
-    .describe("1-based inclusive end line (0 to insert without replacing)"),
-  content: z
-    .string()
-    .describe("Replacement text (empty string to delete lines)"),
-});
-
 const inputSchema = z.object({
   path: z.string().describe("Project-relative path under context/"),
-  patches: z.array(PatchSchema).describe("Patches to apply"),
+  patches: z.array(LinePatchSchema).describe("Patches to apply"),
 });
 
 const outputSchema = z.object({

package/src/tools/prompt/edit.ts

@@ -0,0 +1,148 @@
+import { join } from "node:path";
+import { z } from "zod";
+import { getPromptsDir } from "../../constants.ts";
+import {
+  atomicWriteIfUnchanged,
+  MtimeConflictError,
+  readWithMtime,
+} from "../../fs/atomic.ts";
+import { applyLinePatches, LinePatchSchema } from "../../fs/patches.ts";
+import {
+  parseContextFile,
+  serializeContextFile,
+} from "../../utils/frontmatter.ts";
+import type { ToolDefinition } from "../tool.ts";
+
+const inputSchema = z.object({
+  name: z
+    .string()
+    .describe(
+      "Prompt name without extension (e.g. 'beliefs', 'goals', 'capabilities'). Resolves to prompts/<name>.md.",
+    ),
+  patches: z.array(LinePatchSchema).describe("Patches to apply"),
+});
+
+const outputSchema = z.object({
+  name: z.string(),
+  path: z.string().nullable(),
+  applied: z.number(),
+  content: z.string(),
+  is_error: z.boolean(),
+  error_type: z.string().optional(),
+  message: z.string().optional(),
+  next_action_hint: z.string().optional(),
+});
+
+export const promptEditTool = {
+  name: "prompt_edit",
+  description:
+    "[[ bash equivalent command: patch ]] Apply git-style line-range patches to a prompt file under prompts/. Operates on the whole file (frontmatter + body). Files marked `agent-modification: false` (e.g. soul.md) are protected. Use prompt_read first to inspect current line numbers.",
+  group: "context",
+  inputSchema,
+  outputSchema,
+  execute: async (input, ctx) => {
+    if (input.name.includes("/") || input.name.includes("..")) {
+      return {
+        name: input.name,
+        path: null,
+        applied: 0,
+        content: "",
+        is_error: true,
+        error_type: "invalid_name",
+        message: `Invalid prompt name: ${input.name}`,
+        next_action_hint:
+          "Use a basename without slashes or dots (e.g. 'beliefs').",
+      };
+    }
+    const filePath = join(getPromptsDir(ctx.projectDir), `${input.name}.md`);
+    const file = await readWithMtime(filePath);
+    if (!file) {
+      return {
+        name: input.name,
+        path: null,
+        applied: 0,
+        content: "",
+        is_error: true,
+        error_type: "not_found",
+        message: `Prompt not found: prompts/${input.name}.md`,
+      };
+    }
+
+    const original = file.content;
+    const preParsed = parseContextFile(original);
+    if (!preParsed.meta["agent-modification"]) {
+      return {
+        name: input.name,
+        path: filePath,
+        applied: 0,
+        content: original,
+        is_error: true,
+        error_type: "agent_modification_disabled",
+        message: `Agent modification not allowed for prompts/${input.name}.md`,
+      };
+    }
+
+    const updated = applyLinePatches(original, input.patches);
+    let postParsed: { meta: Record<string, unknown>; content: string };
+    try {
+      postParsed = parseContextFile(updated);
+    } catch (err) {
+      return {
+        name: input.name,
+        path: filePath,
+        applied: 0,
+        content: original,
+        is_error: true,
+        error_type: "invalid_frontmatter",
+        message: `Patched content failed to parse: ${err instanceof Error ? err.message : String(err)}`,
+        next_action_hint:
+          "Check that the frontmatter delimiters and YAML stay valid.",
+      };
+    }
+    if (!postParsed.meta["agent-modification"]) {
+      return {
+        name: input.name,
+        path: filePath,
+        applied: 0,
+        content: original,
+        is_error: true,
+        error_type: "agent_modification_disabled",
+        message: `Patch would clear agent-modification on prompts/${input.name}.md`,
+        next_action_hint:
+          "Don't change the agent-modification frontmatter flag.",
+      };
+    }
+
+    const serialized = serializeContextFile(
+      postParsed.meta,
+      postParsed.content,
+    );
+
+    try {
+      await atomicWriteIfUnchanged(filePath, serialized, file.mtimeMs);
+    } catch (err) {
+      if (err instanceof MtimeConflictError) {
+        return {
+          name: input.name,
+          path: filePath,
+          applied: 0,
+          content: original,
+          is_error: true,
+          error_type: "mtime_conflict",
+          message: `Prompt was modified concurrently: ${err.message}`,
+          next_action_hint:
+            "Re-read with prompt_read and recompute your patch line numbers before retrying.",
+        };
+      }
+      throw err;
+    }
+
+    return {
+      name: input.name,
+      path: filePath,
+      applied: input.patches.length,
+      content: serialized,
+      is_error: false,
+    };
+  },
+} satisfies ToolDefinition<typeof inputSchema, typeof outputSchema>;

package/src/tools/prompt/read.ts

@@ -0,0 +1,70 @@
+import { join } from "node:path";
+import { z } from "zod";
+import { getPromptsDir } from "../../constants.ts";
+import type { ToolDefinition } from "../tool.ts";
+
+const inputSchema = z.object({
+  name: z
+    .string()
+    .describe(
+      "Prompt name without extension (e.g. 'beliefs', 'goals', 'capabilities'). Resolves to prompts/<name>.md.",
+    ),
+});
+
+const outputSchema = z.object({
+  name: z.string(),
+  path: z.string().nullable(),
+  content: z.string(),
+  agent_modification: z.boolean(),
+  is_error: z.boolean(),
+  error_type: z.string().optional(),
+  message: z.string().optional(),
+});
+
+export const promptReadTool = {
+  name: "prompt_read",
+  description:
+    "[[ bash equivalent command: cat ]] Read a prompt file under prompts/ (e.g. beliefs, goals, capabilities, soul). Returns the whole file (frontmatter + body) for use with prompt_edit.",
+  group: "context",
+  inputSchema,
+  outputSchema,
+  execute: async (input, ctx) => {
+    if (input.name.includes("/") || input.name.includes("..")) {
+      return {
+        name: input.name,
+        path: null,
+        content: "",
+        agent_modification: false,
+        is_error: true,
+        error_type: "invalid_name",
+        message: `Invalid prompt name: ${input.name}`,
+      };
+    }
+    const filePath = join(getPromptsDir(ctx.projectDir), `${input.name}.md`);
+    const file = Bun.file(filePath);
+    if (!(await file.exists())) {
+      return {
+        name: input.name,
+        path: null,
+        content: "",
+        agent_modification: false,
+        is_error: true,
+        error_type: "not_found",
+        message: `Prompt not found: prompts/${input.name}.md`,
+      };
+    }
+    const content = await file.text();
+    // Cheap header sniff so the agent knows whether prompt_edit will be
+    // accepted before it constructs patches.
+    const agent_modification = /agent-modification:\s*true/.test(
+      content.split("---", 3).slice(0, 3).join("---"),
+    );
+    return {
+      name: input.name,
+      path: filePath,
+      content,
+      agent_modification,
+      is_error: false,
+    };
+  },
+} satisfies ToolDefinition<typeof inputSchema, typeof outputSchema>;

package/src/tools/registry.ts

@@ -3,8 +3,6 @@ import { capabilitiesRefreshTool } from "./capabilities/refresh.ts";
 // Context tools
 import { pipeToContextTool } from "./context/pipe.ts";
 import { readLargeResultTool } from "./context/read-large-result.ts";
-import { updateBeliefsTool } from "./context/update-beliefs.ts";
-import { updateGoalsTool } from "./context/update-goals.ts";
 // Context — directory operations
 import { contextCreateDirTool } from "./dir/create.ts";
 import { contextDirSizeTool } from "./dir/size.ts";
@@ -24,8 +22,12 @@ import { mcpExecTool } from "./mcp/exec.ts";
 import { mcpInfoTool } from "./mcp/info.ts";
 import { mcpListToolsTool } from "./mcp/list-tools.ts";
 import { mcpSearchTool } from "./mcp/search.ts";
+// Prompt tools
+import { promptEditTool } from "./prompt/edit.ts";
+import { promptReadTool } from "./prompt/read.ts";
 // Schedule tools
 import { createScheduleTool } from "./schedule/create.ts";
+import { scheduleEditTool } from "./schedule/edit.ts";
 import { listSchedulesTool } from "./schedule/list.ts";
 // Search tools
 import { searchTool } from "./search/index.ts";
@@ -40,6 +42,7 @@ import { skillWriteTool } from "./skill/write.ts";
 import { completeTaskTool } from "./task/complete.ts";
 import { createTaskTool } from "./task/create.ts";
 import { deleteTaskTool } from "./task/delete.ts";
+import { taskEditTool } from "./task/edit.ts";
 import { failTaskTool } from "./task/fail.ts";
 import { listTasksTool } from "./task/list.ts";
 import { updateTaskTool } from "./task/update.ts";
@@ -62,6 +65,7 @@ export function registerAllTools(): void {
   registerTool(waitTaskTool);
   registerTool(createTaskTool);
   registerTool(updateTaskTool);
+  registerTool(taskEditTool);
   registerTool(deleteTaskTool);
   registerTool(listTasksTool);
   registerTool(viewTaskTool);
@@ -79,8 +83,8 @@ export function registerAllTools(): void {
   registerTool(contextInfoTool);
   registerTool(contextExistsTool);
   registerTool(contextCountLinesTool);
-  registerTool(updateBeliefsTool);
-  registerTool(updateGoalsTool);
+  registerTool(promptReadTool);
+  registerTool(promptEditTool);
   registerTool(readLargeResultTool);
   registerTool(pipeToContextTool);
 
@@ -89,6 +93,7 @@ export function registerAllTools(): void {
 
   // Schedule
   registerTool(createScheduleTool);
+  registerTool(scheduleEditTool);
   registerTool(listSchedulesTool);
 
   // Search

package/src/tools/schedule/create.ts

@@ -33,7 +33,9 @@ export const createScheduleTool = {
       description: input.description,
       frequency: input.frequency,
     });
-    logger.info(`Created schedule: ${schedule.name} (${schedule.id})`);
+    const msg = `Created schedule: ${schedule.name} (${schedule.id})`;
+    if (ctx.notify) ctx.notify(msg);
+    else logger.info(msg);
     return {
       id: schedule.id,
       name: schedule.name,