talon-agent 1.0.0 → 1.1.0

package/LICENSE

@@ -0,0 +1,21 @@
+MIT License
+
+Copyright (c) 2025 Dylan Neve
+
+Permission is hereby granted, free of charge, to any person obtaining a copy
+of this software and associated documentation files (the "Software"), to deal
+in the Software without restriction, including without limitation the rights
+to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+copies of the Software, and to permit persons to whom the Software is
+furnished to do so, subject to the following conditions:
+
+The above copyright notice and this permission notice shall be included in all
+copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+SOFTWARE.

package/README.md

@@ -4,6 +4,7 @@
 [![TypeScript](https://img.shields.io/badge/TypeScript-5.9-3178C6?logo=typescript&logoColor=white)](https://www.typescriptlang.org/)
 [![Claude](https://img.shields.io/badge/Claude_Agent_SDK-Anthropic-D97706)](https://github.com/anthropics/claude-agent-sdk-typescript)
 [![License: MIT](https://img.shields.io/badge/license-MIT-blue.svg)](LICENSE)
+[![CI](https://github.com/dylanneve1/talon/actions/workflows/ci.yml/badge.svg)](https://github.com/dylanneve1/talon/actions/workflows/ci.yml)
 
 Multi-platform agentic AI harness powered by Claude. Runs on Telegram, Teams, and Terminal with full tool access through MCP.
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "talon-agent",
-  "version": "1.0.0",
+  "version": "1.1.0",
   "description": "Multi-frontend AI agent with full tool access, streaming, cron jobs, and plugin system",
   "author": "Dylan Neve",
   "license": "MIT",
@@ -36,6 +36,7 @@
     "tsconfig.json"
   ],
   "scripts": {
+    "build:binary": "bun build --compile --minify src/index.ts --outfile talon-bun",
     "start": "tsx src/index.ts",
     "cli": "tsx src/cli.ts",
     "setup": "tsx src/cli.ts setup",
@@ -45,28 +46,30 @@
     "test:coverage": "vitest run --coverage",
     "test:mutation": "stryker run",
     "typecheck": "tsc --noEmit",
-    "lint": "oxlint src/ --ignore-pattern '**/__tests__/**'",
+    "lint": "oxlint src/",
+    "knip": "knip",
     "format": "prettier --write src/ prompts/",
     "format:check": "prettier --check src/ prompts/"
   },
   "dependencies": {
-    "@anthropic-ai/claude-agent-sdk": "^0.2.89",
+    "@anthropic-ai/claude-agent-sdk": "^0.2.96",
     "@clack/prompts": "^1.2.0",
     "@grammyjs/auto-retry": "^2.0.2",
     "@grammyjs/transformer-throttler": "^1.2.1",
     "@modelcontextprotocol/sdk": "^1.29.0",
-    "@opencode-ai/sdk": "^1.3.13",
+    "@opencode-ai/sdk": "^1.4.0",
     "big-integer": "^1.6.52",
     "cheerio": "^1.2.0",
     "croner": "^10.0.1",
-    "grammy": "^1.41.1",
-    "marked": "^17.0.5",
+    "grammy": "^1.42.0",
+    "marked": "^18.0.0",
+    "p-retry": "^8.0.0",
     "picocolors": "^1.1.1",
     "pino": "^10.3.1",
     "pino-pretty": "^13.1.3",
     "telegram": "^2.26.22",
     "tsx": "^4.21.0",
-    "undici": "^7.24.7",
+    "undici": "^8.0.2",
     "write-file-atomic": "^7.0.1",
     "zod": "^4.3.6"
   },
@@ -74,13 +77,14 @@
     "@stryker-mutator/core": "^9.6.0",
     "@stryker-mutator/typescript-checker": "^9.6.0",
     "@stryker-mutator/vitest-runner": "^9.6.0",
-    "@types/node": "^25.5.0",
+    "@types/node": "^25.5.2",
     "@types/write-file-atomic": "^4.0.3",
-    "@vitest/coverage-v8": "^4.1.2",
+    "@vitest/coverage-v8": "^4.1.3",
     "fast-check": "^4.6.0",
-    "oxlint": "^1.58.0",
+    "knip": "^6.3.1",
+    "oxlint": "^1.59.0",
     "prettier": "^3.8.1",
     "typescript": "^6.0.2",
-    "vitest": "^4.1.2"
+    "vitest": "^4.1.3"
   }
 }

package/prompts/dream.md

@@ -5,11 +5,12 @@ You have access ONLY to filesystem tools (Read, Write, Edit, Bash, Glob, Grep).
 ## Your 4-stage task
 
 ### Stage 1 — Orient
-- Read `{{dreamStateFile}}` to confirm `last_run` timestamp
+
 - List log files in `{{logsDir}}/` that are dated on or after `{{lastRunIso}}`
-- If there are no new log files, update dream_state.json status to "idle" and stop
+- If there are no new log files, stop — the system will handle state updates
 
 ### Stage 2 — Gather
+
 - Read each new log file
 - Each log file uses this format:
   - User messages appear as `## HH:MM -- [Username]` followed by the full message text
@@ -25,17 +26,20 @@ You have access ONLY to filesystem tools (Read, Write, Edit, Bash, Glob, Grep).
 - Be selective — only extract genuinely new or updated information
 
 ### Stage 3 — Consolidate
+
 - Read the current memory file at `{{memoryFile}}`
 - Merge new information into the appropriate sections
 - Update existing entries if new info contradicts or extends them
 - Add new entries where appropriate
 - Keep entries concise and factual — no padding, no narrative
 - Preserve all existing structure and sections
+- Also write daily memory summaries to `{{dailyMemoryDir}}/YYYY-MM-DD.md` for each day of logs you processed. Include key learnings, conversation summaries, and follow-ups. Keep these concise — the bot reads them on demand for context.
 
 ### Stage 4 — Prune
+
 - Remove entries that have been explicitly contradicted
 - Remove entries that are clearly stale or irrelevant
 - Do NOT remove entries just because they're old — only remove if wrong or superseded
 - Write the updated memory.md back to `{{memoryFile}}`
 
-When done, your final action is to write `{ "last_run": <current_unix_ms>, "status": "idle" }` to `{{dreamStateFile}}`.
+When done with memory consolidation, stop. The system handles all dream_state.json updates.

package/prompts/heartbeat.md

@@ -0,0 +1,30 @@
+You are Talon's background heartbeat agent. You run periodically (every {{intervalMinutes}} minutes) to perform maintenance tasks defined by the user.
+
+You have access ONLY to filesystem tools (Read, Write, Edit, Bash, Glob, Grep). Do NOT attempt to use any Telegram, MCP, or messaging tools.
+
+## Context
+
+- Workspace: `{{workspace}}`
+- Memory file: `{{memoryFile}}`
+- Logs directory: `{{logsDir}}`
+- Last heartbeat: `{{lastRunIso}}`
+- Run number: #{{runCount}}
+- Today's daily memory: `{{dailyMemoryFile}}`
+
+## Instructions
+
+Read the user-defined instructions file at `{{instructionsFile}}`. Follow whatever tasks are defined there.
+
+If the instructions file does not exist or is empty, perform these default tasks:
+
+1. **Review recent logs** — Check `{{logsDir}}/` for log files dated after `{{lastRunIso}}`. If `{{lastRunIso}}` is `never`, treat it as the beginning of time and review all available logs. Extract any new facts, preferences, or notable events.
+2. **Update memory** — Merge any new information into `{{memoryFile}}`, keeping entries concise and factual.
+3. **Update daily notes** — Write today's learnings, observations, corrections, and follow-ups to `{{dailyMemoryFile}}`. Keep entries concise — the bot reads this file on demand for context.
+4. **Workspace hygiene** — Note any issues but do not delete files unless the instructions explicitly say to.
+
+## Rules
+
+- Be surgical and precise. Do not rewrite files unnecessarily.
+- Do not modify files outside the workspace unless the instructions explicitly allow it.
+- Keep your work focused and efficient — you have a 10-minute time limit.
+- When done, stop. The system handles all state tracking.

package/prompts/identity.md

@@ -17,6 +17,7 @@
 Your identity is defined in `~/.talon/workspace/identity.md`. Read it to know who you are.
 
 If the identity file is empty or only contains the template comments, you MUST ask the user during your first interaction:
+
 - What should I be called?
 - Who are you / who created me?
 - What will I be used for?

package/prompts/teams.md

@@ -32,6 +32,7 @@ Webhook-based integration — no reactions, media uploads, message editing, typi
 Messages render as Adaptive Cards. The formatting engine is NOT standard Markdown.
 
 What WORKS:
+
 - **bold** and _italic_
 - [links](https://example.com)
 - Fenced code blocks (triple backticks) — render as monospace in a grey box
@@ -39,11 +40,13 @@ What WORKS:
 - Numbered and bulleted lists
 
 What does NOT work:
+
 - Inline code with backticks — do NOT use `code` style, just write the text plain
 - Headings with # — use **bold** text instead
 - Images/media — not supported via webhook
 
 Style:
+
 - Concise. No filler.
 - Use **bold** for emphasis, _italic_ for secondary emphasis.
 - Use markdown tables for structured/tabular data — they render as proper grid tables.

package/prompts/telegram.md

@@ -75,6 +75,7 @@ When a user presses a callback button, you'll receive "[Button pressed]" with th
 ### Stickers
 
 Use stickers like a human would — they're part of Telegram culture:
+
 - When users send stickers, their set_name is captured. Use `save_sticker_pack` to save packs you like.
 - Once saved, read `~/.talon/workspace/stickers/<set_name>.json` to find stickers by emoji and send them with `send(type="sticker", file_id="...")`.
 - Send stickers to express emotions, reactions, or just for fun. Don't overuse them.

package/src/__tests__/chat-settings.test.ts

@@ -16,6 +16,11 @@ vi.mock("node:fs", () => ({
   mkdirSync: vi.fn(),
 }));
 
+// Mock write-file-atomic to prevent writes to the real production file
+vi.mock("write-file-atomic", () => ({
+  default: { sync: vi.fn() },
+}));
+
 const {
   getChatSettings,
   setChatModel,
@@ -252,7 +257,9 @@ describe("chat-settings", () => {
       const settings = getChatSettings("migrate-1");
       expect(settings.effort).toBe("off");
       // maxThinkingTokens should be removed
-      expect((settings as Record<string, unknown>).maxThinkingTokens).toBeUndefined();
+      expect(
+        (settings as Record<string, unknown>).maxThinkingTokens,
+      ).toBeUndefined();
     });
 
     it("migrates maxThinkingTokens=1000 to effort=low", () => {
@@ -320,7 +327,9 @@ describe("chat-settings", () => {
       const settings = getChatSettings("migrate-6");
       // Should keep existing effort, just clean up old field
       expect(settings.effort).toBe("high");
-      expect((settings as Record<string, unknown>).maxThinkingTokens).toBeUndefined();
+      expect(
+        (settings as Record<string, unknown>).maxThinkingTokens,
+      ).toBeUndefined();
     });
 
     it("handles missing store file gracefully", () => {
@@ -335,3 +344,100 @@ describe("chat-settings", () => {
     });
   });
 });
+
+describe("chat-settings — setPulseLastCheckMsgId", () => {
+  it("sets pulseLastCheckMsgId when msgId is provided", async () => {
+    const { setPulseLastCheckMsgId, getChatSettings } =
+      await import("../storage/chat-settings.js");
+    setPulseLastCheckMsgId("pulse-check-1", 42);
+    expect(getChatSettings("pulse-check-1").pulseLastCheckMsgId).toBe(42);
+  });
+
+  it("clears pulseLastCheckMsgId when undefined is passed", async () => {
+    const { setPulseLastCheckMsgId, getChatSettings } =
+      await import("../storage/chat-settings.js");
+    setPulseLastCheckMsgId("pulse-check-2", 100);
+    setPulseLastCheckMsgId("pulse-check-2", undefined);
+    expect(
+      getChatSettings("pulse-check-2").pulseLastCheckMsgId,
+    ).toBeUndefined();
+  });
+
+  it("removes empty settings object after all fields cleared", async () => {
+    const { setPulseLastCheckMsgId, getChatSettings } =
+      await import("../storage/chat-settings.js");
+    // Set only pulseLastCheckMsgId (no model, effort, pulse, pulseIntervalMs)
+    setPulseLastCheckMsgId("pulse-cleanup-1", 99);
+    setPulseLastCheckMsgId("pulse-cleanup-1", undefined);
+    // cleanupEmpty should have removed the settings object
+    expect(getChatSettings("pulse-cleanup-1")).toEqual({});
+  });
+});
+
+describe("chat-settings — migration of has-effort + maxThinkingTokens", () => {
+  it("cleans up maxThinkingTokens when effort already set", async () => {
+    const { loadChatSettings, getChatSettings } =
+      await import("../storage/chat-settings.js");
+    const { existsSync, readFileSync } = await import("node:fs");
+    vi.mocked(existsSync).mockReturnValueOnce(true);
+    vi.mocked(readFileSync).mockReturnValueOnce(
+      JSON.stringify({
+        "migrate-has-effort": { effort: "high", maxThinkingTokens: 16000 },
+      }),
+    );
+    loadChatSettings();
+    const s = getChatSettings("migrate-has-effort");
+    expect(s.effort).toBe("high");
+    expect((s as Record<string, unknown>).maxThinkingTokens).toBeUndefined();
+  });
+});
+
+describe("chat-settings — flushChatSettings", () => {
+  it("does not throw when called", async () => {
+    const { flushChatSettings, setChatModel } =
+      await import("../storage/chat-settings.js");
+    // Make dirty first so save() runs
+    setChatModel("flush-test", "claude-opus-4-6");
+    expect(() => flushChatSettings()).not.toThrow();
+  });
+});
+
+describe("chat-settings — cleanupEmpty keeps entry when other fields remain (line 115 FALSE branch)", () => {
+  it("does not delete entry when effort is still set after clearing model", () => {
+    const chatId = `cleanup-keep-entry-${Date.now()}`;
+    // Set both model and effort
+    setChatModel(chatId, "claude-sonnet-4-6");
+    setChatEffort(chatId, "high");
+    expect(getChatSettings(chatId).model).toBe("claude-sonnet-4-6");
+    expect(getChatSettings(chatId).effort).toBe("high");
+
+    // Clear model only — effort still set → cleanupEmpty condition is FALSE → entry kept
+    setChatModel(chatId, undefined);
+    expect(getChatSettings(chatId).effort).toBe("high");
+    expect(getChatSettings(chatId).model).toBeUndefined();
+  });
+});
+
+describe("chat-settings — backup recovery on corrupt primary", () => {
+  it("loads from backup when primary JSON is corrupt", async () => {
+    const { loadChatSettings, getChatSettings } =
+      await import("../storage/chat-settings.js");
+    vi.mocked(existsSync)
+      .mockReturnValueOnce(true) // primary exists
+      .mockReturnValueOnce(true); // backup exists
+    vi.mocked(readFileSync)
+      .mockReturnValueOnce("{ INVALID JSON") // primary corrupt
+      .mockReturnValueOnce(
+        JSON.stringify({
+          "backup-settings-chat": {
+            model: "claude-sonnet-4-6",
+            effort: "medium",
+          },
+        }),
+      );
+    loadChatSettings();
+    const s = getChatSettings("backup-settings-chat");
+    expect(s.model).toBe("claude-sonnet-4-6");
+    expect(s.effort).toBe("medium");
+  });
+});

package/src/__tests__/cleanup-registry.test.ts

@@ -0,0 +1,58 @@
+/**
+ * Tests for src/util/cleanup-registry.ts
+ */
+import { describe, it, expect } from "vitest";
+import { registerCleanup } from "../util/cleanup-registry.js";
+
+describe("cleanup-registry", () => {
+  it("registered handler is called when the exit event fires", () => {
+    let called = false;
+    registerCleanup(() => {
+      called = true;
+    });
+    process.emit("exit", 0);
+    expect(called).toBe(true);
+  });
+
+  it("multiple handlers are all called on exit", () => {
+    const results: number[] = [];
+    registerCleanup(() => results.push(1));
+    registerCleanup(() => results.push(2));
+    registerCleanup(() => results.push(3));
+    process.emit("exit", 0);
+    expect(results).toContain(1);
+    expect(results).toContain(2);
+    expect(results).toContain(3);
+  });
+
+  it("handler that throws does not prevent subsequent handlers from running", () => {
+    let afterCalled = false;
+    registerCleanup(() => {
+      throw new Error("boom");
+    });
+    registerCleanup(() => {
+      afterCalled = true;
+    });
+    expect(() => process.emit("exit", 0)).not.toThrow();
+    expect(afterCalled).toBe(true);
+  });
+
+  it("registering the same function object twice calls it twice", () => {
+    let count = 0;
+    const fn = () => count++;
+    registerCleanup(fn);
+    registerCleanup(fn);
+    process.emit("exit", 0);
+    expect(count).toBe(2);
+  });
+
+  it("does not add a new process exit listener on each registerCleanup call", () => {
+    const before = process.listenerCount("exit");
+    registerCleanup(() => {});
+    registerCleanup(() => {});
+    registerCleanup(() => {});
+    const after = process.listenerCount("exit");
+    // Listener count must not grow — one listener handles all handlers
+    expect(after).toBe(before);
+  });
+});