niahere 0.2.42 → 0.2.44

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "niahere",
-  "version": "0.2.42",
+  "version": "0.2.44",
   "description": "A personal AI assistant daemon — scheduled jobs, chat across Telegram and Slack, persona system, and visual identity.",
   "type": "module",
   "scripts": {

package/src/channels/slack.ts

@@ -7,7 +7,7 @@ import type { Channel, ChatState, Attachment, AttachmentType } from "../types";
 import { getConfig, updateRawConfig, resetConfig } from "../utils/config";
 import { relativeTime } from "../utils/format";
 import { runMigrations } from "../db/migrate";
-import { Session } from "../db/models";
+import { Session, Message } from "../db/models";
 import { log } from "../utils/log";
 import { getMcpServers } from "../mcp";
 import { getNiaHome, getPaths } from "../utils/paths";
@@ -493,7 +493,7 @@ class SlackChannel implements Channel {
           .catch((err) => log.debug({ err, channel: msg.channel }, "slack: failed to add thinking reaction"));
 
         try {
-          const { result } = await state.engine.send(text, {
+          const { result, messageId } = await state.engine.send(text, {
             onActivity(status) {
               log.debug({ status }, "slack engine activity");
             },
@@ -504,22 +504,27 @@ class SlackChannel implements Channel {
           // [NO_REPLY] or empty = agent chose not to respond (thread judgement)
           if (!reply || cleanSentinel(reply) === "[NO_REPLY]") {
             log.info({ channel: msg.channel, key }, "slack: agent chose not to reply");
+            if (messageId) await Message.updateDeliveryStatus(messageId, "sent").catch(() => {});
             return;
           }
 
-          if (replyThreadTs) {
-            await client.chat.postMessage({
-              channel: msg.channel,
-              text: reply,
-              thread_ts: replyThreadTs,
-            });
-          } else {
-            await say(reply);
+          try {
+            if (replyThreadTs) {
+              await client.chat.postMessage({
+                channel: msg.channel,
+                text: reply,
+                thread_ts: replyThreadTs,
+              });
+            } else {
+              await say(reply);
+            }
+            if (messageId) await Message.updateDeliveryStatus(messageId, "sent").catch(() => {});
+            log.info({ channel: msg.channel, key, chars: reply.length }, "slack reply sent");
+          } catch (sendErr) {
+            if (messageId) await Message.updateDeliveryStatus(messageId, "failed").catch(() => {});
+            throw sendErr;
           }
-
-          log.info({ channel: msg.channel, key, chars: reply.length }, "slack reply sent");
         } catch (err) {
-
           const errText = err instanceof Error ? err.message : String(err);
           log.error({ err, channel: msg.channel }, "slack message processing failed");
 
@@ -530,7 +535,7 @@ class SlackChannel implements Channel {
               thread_ts: replyThreadTs,
             }).catch(() => {});
           } else {
-            await say(`[error] ${errText}`);
+            await say(`[error] ${errText}`).catch(() => {});
           }
         } finally {
           await client.reactions.remove({ channel: msg.channel, timestamp: msg.ts, name: "thinking_face" })

package/src/channels/telegram.ts

@@ -3,7 +3,7 @@ import { createChatEngine } from "../chat/engine";
 import type { Channel, ChatState, Attachment } from "../types";
 import { getConfig, updateRawConfig } from "../utils/config";
 import { runMigrations } from "../db/migrate";
-import { Session } from "../db/models";
+import { Session, Message } from "../db/models";
 import { log } from "../utils/log";
 import { getMcpServers } from "../mcp";
 import { classifyMime, validateAttachment, prepareImage } from "../utils/attachment";
@@ -136,16 +136,21 @@ class TelegramChannel implements Channel {
       bot.api.sendChatAction(chatId, "typing").catch(() => {});
 
       try {
-        const { result } = await state.engine.send(text, {}, attachments);
+        const { result, messageId } = await state.engine.send(text, {}, attachments);
 
         const reply = result.trim() || "(no response)";
         try {
-          await bot.api.sendMessage(chatId, reply, { parse_mode: "MarkdownV2" });
-        } catch {
-          await bot.api.sendMessage(chatId, reply).catch(() => {});
+          try {
+            await bot.api.sendMessage(chatId, reply, { parse_mode: "MarkdownV2" });
+          } catch {
+            await bot.api.sendMessage(chatId, reply);
+          }
+          if (messageId) await Message.updateDeliveryStatus(messageId, "sent").catch(() => {});
+          log.info({ chatId, chars: result.length }, "telegram reply sent");
+        } catch (sendErr) {
+          if (messageId) await Message.updateDeliveryStatus(messageId, "failed").catch(() => {});
+          throw sendErr;
         }
-
-        log.info({ chatId, chars: result.length }, "telegram reply sent");
       } catch (err) {
         const errText = err instanceof Error ? err.message : String(err);
         log.error({ err, chatId }, "telegram message processing failed");

package/src/chat/engine.ts

@@ -10,8 +10,10 @@ import { getAgentDefinitions } from "../core/agents";
 import { Session, Message, ActiveEngine } from "../db/models";
 import type { Attachment, SendResult, StreamCallback, ActivityCallback, SendCallbacks, ChatEngine, EngineOptions } from "../types";
 import { truncate, formatToolUse } from "../utils/format-activity";
+import { log } from "../utils/log";
 
 const IDLE_TIMEOUT = 10 * 60 * 1000; // 10 minutes
+const LONG_RUNNING_WARN = 30 * 60 * 1000; // 30 minutes
 
 interface SDKUserMessage {
   type: "user";
@@ -128,20 +130,50 @@ export async function createChatEngine(opts: EngineOptions): Promise<ChatEngine>
   let queryHandle: Query | null = null;
   let pending: PendingResult | null = null;
   let idleTimer: ReturnType<typeof setTimeout> | null = null;
+  let longRunningTimer: ReturnType<typeof setTimeout> | null = null;
+  let longRunningWarned = false;
   let alive = false;
 
+  function clearIdleTimer() {
+    if (idleTimer) {
+      clearTimeout(idleTimer);
+      idleTimer = null;
+    }
+  }
+
   function resetIdleTimer() {
-    if (idleTimer) clearTimeout(idleTimer);
+    clearIdleTimer();
     idleTimer = setTimeout(() => {
+      if (pending) {
+        // Don't tear down while a request is in flight
+        log.warn({ room }, "idle timer fired while request pending, skipping teardown");
+        return;
+      }
       teardown();
     }, IDLE_TIMEOUT);
   }
 
-  function teardown() {
-    if (idleTimer) {
-      clearTimeout(idleTimer);
-      idleTimer = null;
+  function clearLongRunningTimer() {
+    if (longRunningTimer) {
+      clearTimeout(longRunningTimer);
+      longRunningTimer = null;
     }
+    longRunningWarned = false;
+  }
+
+  function startLongRunningTimer() {
+    clearLongRunningTimer();
+    longRunningTimer = setTimeout(() => {
+      if (pending) {
+        longRunningWarned = true;
+        log.warn({ room, elapsed: LONG_RUNNING_WARN / 1000 }, "engine request running for 30+ minutes");
+      }
+    }, LONG_RUNNING_WARN);
+  }
+
+  function teardown() {
+    clearIdleTimer();
+    clearLongRunningTimer();
     if (stream) {
       stream.end();
       stream = null;
@@ -285,31 +317,44 @@ export async function createChatEngine(opts: EngineOptions): Promise<ChatEngine>
               const costUsd = (message as any).total_cost_usd as number;
               const turns = (message as any).num_turns as number;
 
+              let messageId: number | undefined;
               if (sessionId && resultText) {
-                await Message.save({
+                messageId = await Message.save({
                   sessionId,
                   room,
                   sender: "nia",
                   content: resultText,
                   isFromAgent: true,
+                  deliveryStatus: "pending",
                 });
                 await Session.touch(sessionId);
               }
 
               await ActiveEngine.unregister(room);
-              pending.resolve({ result: resultText, costUsd, turns });
+              clearLongRunningTimer();
+              pending.resolve({ result: resultText, costUsd, turns, messageId });
               pending = null;
               resetIdleTimer();
             } else {
               const errors = (message as any).errors;
               const errorText = `[error] ${errors?.join(", ") || "unknown error"}`;
               await ActiveEngine.unregister(room);
+              clearLongRunningTimer();
               pending.resolve({ result: errorText, costUsd: 0, turns: 0 });
               pending = null;
               resetIdleTimer();
             }
           }
         }
+
+        // Stream ended without a result — subprocess exited or was killed
+        if (pending) {
+          const partial = pending.accumulatedText;
+          log.error({ room, partialChars: partial.length }, "query stream ended without result, rejecting pending request");
+          await ActiveEngine.unregister(room).catch(() => {});
+          pending.reject(new Error(`stream ended without result (${partial.length} chars accumulated)`));
+          pending = null;
+        }
       } catch (err) {
         if (pending) {
           await ActiveEngine.unregister(room).catch(() => {});
@@ -317,6 +362,7 @@ export async function createChatEngine(opts: EngineOptions): Promise<ChatEngine>
           pending = null;
         }
       } finally {
+        clearLongRunningTimer();
         alive = false;
         stream = null;
         queryHandle = null;
@@ -334,6 +380,10 @@ export async function createChatEngine(opts: EngineOptions): Promise<ChatEngine>
     },
 
     async send(userMessage: string, callbacks?: SendCallbacks, attachments?: Attachment[]) {
+      // Clear idle timer — engine is not idle while processing a request
+      clearIdleTimer();
+      startLongRunningTimer();
+
       await ActiveEngine.register(room, channel);
 
       if (!alive || !stream) {

package/src/chat/identity.ts

@@ -17,7 +17,7 @@ function loadFile(dir: string, name: string): string {
 
 export function loadIdentity(): string {
   const { selfDir } = getPaths();
-  const files = ["identity.md", "owner.md", "soul.md", "rules.md"];
+  const files = ["identity.md", "owner.md", "soul.md", "rules.md", "memory.md"];
   return files.map((f) => loadFile(selfDir, f)).filter(Boolean).join("\n\n");
 }
 

package/src/cli/index.ts

@@ -433,6 +433,12 @@ switch (command) {
     process.exit(exitCode);
   }
 
+  case "backup": {
+    const { backupCommand } = await import("../commands/backup");
+    await backupCommand();
+    break;
+  }
+
   case "validate": {
     const { validateConfig } = await import("../commands/validate");
     const result = validateConfig();
@@ -444,6 +450,15 @@ switch (command) {
   case "update": {
     const { version: currentVersion } = await import("../../package.json");
     console.log(`Current: v${currentVersion}`);
+    // Auto-backup before update
+    try {
+      const { createBackup } = await import("../commands/backup");
+      console.log("Backing up...");
+      await createBackup(true);
+      console.log("✓ pre-update backup created");
+    } catch (err) {
+      console.log(`⚠ backup skipped: ${errMsg(err)}`);
+    }
     console.log("Updating...");
     const install = Bun.spawn(["npm", "i", "-g", "niahere@latest"], { stdio: ["ignore", "inherit", "inherit"] });
     const installExit = await install.exited;

package/src/commands/backup.ts

@@ -0,0 +1,146 @@
+import { existsSync, mkdirSync, readdirSync, statSync, unlinkSync } from "fs";
+import { join, basename } from "path";
+import { getNiaHome } from "../utils/paths";
+import { getConfig } from "../utils/config";
+
+const MAX_BACKUPS = 10;
+
+function getBackupDir(): string {
+  const dir = join(getNiaHome(), "backups");
+  mkdirSync(dir, { recursive: true });
+  return dir;
+}
+
+function humanDate(): string {
+  const now = new Date();
+  const y = now.getFullYear();
+  const m = String(now.getMonth() + 1).padStart(2, "0");
+  const d = String(now.getDate()).padStart(2, "0");
+  return `${y}-${m}-${d}`;
+}
+
+function formatSize(bytes: number): string {
+  if (bytes < 1024) return `${bytes}B`;
+  if (bytes < 1024 * 1024) return `${(bytes / 1024).toFixed(1)}KB`;
+  return `${(bytes / (1024 * 1024)).toFixed(1)}MB`;
+}
+
+function pruneOldBackups(dir: string): void {
+  const files = readdirSync(dir)
+    .filter((f) => f.startsWith("niahere-") && f.endsWith(".tar.gz"))
+    .map((f) => ({ name: f, mtime: statSync(join(dir, f)).mtimeMs }))
+    .sort((a, b) => b.mtime - a.mtime);
+
+  for (const file of files.slice(MAX_BACKUPS)) {
+    unlinkSync(join(dir, file.name));
+    console.log(`  pruned old backup: ${file.name}`);
+  }
+}
+
+export async function createBackup(silent = false): Promise<string> {
+  const home = getNiaHome();
+  const backupDir = getBackupDir();
+  const filename = `niahere-${humanDate()}-${Math.floor(Date.now() / 1000)}.tar.gz`;
+  const outPath = join(backupDir, filename);
+
+  // Directories/files to include (relative to home)
+  const includes: string[] = [];
+  if (existsSync(join(home, "config.yaml"))) includes.push("config.yaml");
+  if (existsSync(join(home, "self"))) includes.push("self");
+  if (existsSync(join(home, "agents"))) includes.push("agents");
+  if (existsSync(join(home, "skills"))) includes.push("skills");
+
+  // Database dump
+  const config = getConfig();
+  const dbUrl = config.database_url;
+  let dbDumped = false;
+  if (dbUrl) {
+    const dumpPath = join(home, "tmp", "db-backup.sql");
+    mkdirSync(join(home, "tmp"), { recursive: true });
+    const pg = Bun.spawn(["pg_dump", dbUrl, "-f", dumpPath], {
+      stdout: "pipe",
+      stderr: "pipe",
+    });
+    const exitCode = await pg.exited;
+    if (exitCode === 0 && existsSync(dumpPath)) {
+      // Copy to a relative path for tar
+      const relDump = "db-backup.sql";
+      const { copyFileSync } = await import("fs");
+      copyFileSync(dumpPath, join(home, relDump));
+      includes.push(relDump);
+      dbDumped = true;
+    } else if (!silent) {
+      const stderr = await new Response(pg.stderr).text();
+      console.log(`  ⚠ db dump skipped: ${stderr.trim() || `exit ${exitCode}`}`);
+    }
+  }
+
+  if (includes.length === 0) {
+    console.log("Nothing to back up.");
+    return "";
+  }
+
+  // Create tar.gz
+  const tar = Bun.spawn(["tar", "czf", outPath, ...includes], {
+    cwd: home,
+    stdout: "pipe",
+    stderr: "pipe",
+  });
+  const tarExit = await tar.exited;
+  if (tarExit !== 0) {
+    const stderr = await new Response(tar.stderr).text();
+    throw new Error(`tar failed: ${stderr.trim()}`);
+  }
+
+  // Clean up temp db dump
+  if (dbDumped) {
+    try { unlinkSync(join(home, "db-backup.sql")); } catch {}
+    try { unlinkSync(join(home, "tmp", "db-backup.sql")); } catch {}
+  }
+
+  const size = statSync(outPath).size;
+  if (!silent) {
+    console.log(`✓ backup created: ${filename} (${formatSize(size)})`);
+    if (dbDumped) console.log("  includes: files + database");
+    else console.log("  includes: files only (no database)");
+  }
+
+  pruneOldBackups(backupDir);
+
+  return outPath;
+}
+
+export function listBackups(): void {
+  const dir = getBackupDir();
+  const files = readdirSync(dir)
+    .filter((f) => f.startsWith("niahere-") && f.endsWith(".tar.gz"))
+    .map((f) => {
+      const stat = statSync(join(dir, f));
+      return { name: f, size: stat.size, mtime: stat.mtimeMs };
+    })
+    .sort((a, b) => b.mtime - a.mtime);
+
+  if (files.length === 0) {
+    console.log("No backups found.");
+    return;
+  }
+
+  console.log(`${files.length} backup(s) in ${dir}:\n`);
+  for (const f of files) {
+    const date = new Date(f.mtime).toLocaleString();
+    console.log(`  ${f.name}  ${formatSize(f.size)}  ${date}`);
+  }
+}
+
+export async function backupCommand(): Promise<void> {
+  const sub = process.argv[3];
+  if (sub === "list") {
+    listBackups();
+  } else if (!sub) {
+    await createBackup();
+  } else {
+    console.log("Usage:");
+    console.log("  nia backup        — create a backup");
+    console.log("  nia backup list   — list existing backups");
+  }
+}

package/src/db/migrations/008_message_delivery_status.ts

@@ -0,0 +1,10 @@
+import type postgres from "postgres";
+
+export const name = "008_message_delivery_status";
+
+export async function up(sql: postgres.Sql): Promise<void> {
+  await sql`
+    ALTER TABLE messages
+    ADD COLUMN IF NOT EXISTS delivery_status TEXT DEFAULT 'sent'
+  `;
+}

package/src/db/models/message.ts

@@ -1,12 +1,45 @@
 import { getSql } from "../connection";
 import type { SaveMessageParams, RoomStats, RecentMessage, SearchResult, SessionMessage } from "../../types";
 
-export async function save(params: SaveMessageParams): Promise<void> {
+export type DeliveryStatus = "pending" | "sent" | "failed";
+
+export async function save(params: SaveMessageParams & { deliveryStatus?: DeliveryStatus }): Promise<number> {
   const sql = getSql();
-  await sql`
-    INSERT INTO messages (session_id, room, sender, content, is_from_agent)
-    VALUES (${params.sessionId}, ${params.room}, ${params.sender}, ${params.content}, ${params.isFromAgent})
+  const status = params.deliveryStatus || "sent";
+  const rows = await sql`
+    INSERT INTO messages (session_id, room, sender, content, is_from_agent, delivery_status)
+    VALUES (${params.sessionId}, ${params.room}, ${params.sender}, ${params.content}, ${params.isFromAgent}, ${status})
+    RETURNING id
   `;
+  return rows[0].id;
+}
+
+export async function updateDeliveryStatus(id: number, status: DeliveryStatus): Promise<void> {
+  const sql = getSql();
+  await sql`UPDATE messages SET delivery_status = ${status} WHERE id = ${id}`;
+}
+
+export async function getUndelivered(room?: string): Promise<Array<{ id: number; room: string; content: string; createdAt: string }>> {
+  const sql = getSql();
+  const rows = room
+    ? await sql`
+        SELECT id, room, content, created_at
+        FROM messages
+        WHERE delivery_status = 'failed' AND is_from_agent = true AND room = ${room}
+        ORDER BY created_at ASC
+      `
+    : await sql`
+        SELECT id, room, content, created_at
+        FROM messages
+        WHERE delivery_status = 'failed' AND is_from_agent = true
+        ORDER BY created_at ASC
+      `;
+  return rows.map((r) => ({
+    id: r.id,
+    room: r.room,
+    content: r.content,
+    createdAt: String(r.created_at),
+  }));
 }
 
 export async function getRecent(limit = 20, room?: string): Promise<RecentMessage[]> {

package/src/prompts/environment.md

@@ -92,31 +92,102 @@ Your persona files live in {{selfDir}}/:
 - `identity.md` — your personality and voice
 - `owner.md` — info about who runs you
 - `soul.md` — how you work
-- `rules.md` — behavioral overrides and custom instructions (loaded into every session, hot-reloads without restart)
-- `memory.md` — persistent learnings (read on demand, not loaded automatically)
+- `rules.md` — behavioral instructions (loaded into every session automatically)
+- `memory.md` — facts and context (loaded into every session automatically)
 
 ### Rules vs Memory
 
-**Rules** (`rules.md`) = instructions for how to behave. Loaded into every session automatically.
-- "stamp updates should be 1-2 lines max"
-- "never send long messages in #tech"
-- Use `add_rule` tool to add new rules, or edit the file directly.
-
-**Memory** (`memory.md`) = facts and context. Read on demand when relevant.
-- "2026-03-13: DB was down, Telegram send failed"
-- "Aman prefers terminal over Slack for debugging"
-- Use `read_memory` to recall what you know. Use `add_memory` to save new memories.
-
-**Which to use?**
-- "From now on, do X" → rule
-- "Remember that X happened" / "I prefer X" → memory
-
-### When to save (proactive)
-Don't wait for the user to say "remember this." Proactively save when you learn:
-- Personal facts: travel plans, location, schedule, preferences
-- Work context: project decisions, team changes, deadlines
-- Corrections: user corrected you on something worth remembering
-- Patterns: recurring requests, preferred communication style
-
-Example: if the owner says "I'm going home on the 21st, early morning flight" — save it as a memory without being asked. These are facts future sessions need.
-- If unsure, ask.
+The difference is simple: **rules are instructions, memories are facts.**
+
+**Rules** = verbs. They change your behavior. They tell you to do or not do something.
+- Start with: do / don't / always / never / keep / avoid / when X then Y
+- Test: "If I ignore this, my response is **wrong**"
+- Tool: `add_rule`
+- Loaded: every session, always
+
+**Memory** = nouns. They give you context. They tell you something is true.
+- Start with: a name, date, or factual statement
+- Test: "If I don't know this, my response is **uninformed** but not wrong"
+- Tool: `add_memory`
+- Loaded: every session, always
+
+### The decision flowchart
+
+Ask yourself one question: **"Is this telling me HOW to act, or WHAT is true?"**
+
+| Signal | → | Where |
+|--------|---|-------|
+| "From now on..." / "Always..." / "Never..." / "Stop doing..." | → | **Rule** |
+| "I prefer..." / "I like when you..." / "Do it like this..." | → | **Rule** (it's a behavioral preference = instruction) |
+| "I'm traveling to Delhi on the 21st" | → | **Memory** |
+| "We use Postgres, not MySQL" / "The deploy is on Friday" | → | **Memory** |
+| "Last time X broke because of Y" | → | **Memory** (fact about past) |
+| "Don't do X again, it broke last time" | → | **Rule** (instruction) + **Memory** (the incident) |
+| User corrects your formatting/tone/length | → | **Rule** (you need to change behavior) |
+| User mentions a person, project, deadline | → | **Memory** |
+
+### Good vs bad entries
+
+**Good rules** — specific, actionable, earns its token cost every session:
+- "Stamp/standup job output: 1-2 lines max, no preamble"
+- "In Slack channels, keep replies under 3 paragraphs"
+- "Never send code blocks in Telegram — they render badly"
+- "When Aman says 'ship it', commit and push without asking"
+
+**Bad rules** — vague, redundant, or one-time:
+- "Be helpful" (already in your identity)
+- "Use good formatting" (too vague to act on)
+- "Send the report to #general today" (one-time task, not a rule)
+
+**Good memories** — dated, one fact, useful across sessions:
+- "2026-03-21: Aman traveling to Delhi, back 2026-03-28"
+- "Kay.ai is the main work project — ask.kay.ai is the product URL"
+- "Aman prefers debugging via terminal, not Slack"
+- "2026-03-13: Postgres went down, Telegram sends failed — DNS issue"
+
+**Bad memories** — raw logs, transient state, duplicates:
+- Pasting full error logs or stack traces
+- "Currently working on X" (stale by next session)
+- Anything already in rules.md or identity.md
+
+### When to save (be proactive)
+
+Rules and memories don't only come from the user telling you things. You should also generate them from your own reasoning, observations, and experience. **Think of yourself as learning, not just recording.**
+
+#### From the user (explicit)
+
+| You notice... | Save as |
+|---------------|---------|
+| User says "from now on" / "always" / "stop doing X" | **Rule** |
+| User corrects your tone, format, length, or approach | **Rule** |
+| User mentions a preference about how you communicate | **Rule** |
+| User shares travel plans, schedule, personal facts | **Memory** |
+| User mentions people, projects, deadlines, decisions | **Memory** |
+| User corrects a factual misunderstanding | **Memory** |
+| Both behavior change AND a fact behind it | **Rule** + **Memory** |
+
+#### From your own thinking (self-generated)
+
+You are not a passive recorder. Reflect on your own experience and save learnings:
+
+| You realize... | Save as |
+|----------------|---------|
+| A tool or approach failed — you should avoid it next time | **Rule** ("Don't use X for Y — it fails because Z") |
+| You found a better way to do something after trial and error | **Rule** ("For X, use Y approach instead of Z") |
+| A job keeps erroring the same way — there's a pattern | **Rule** (the workaround) + **Memory** (the incident pattern) |
+| You notice the user always ignores or rejects a certain kind of response | **Rule** (stop doing that) |
+| You discover how a system works (API quirk, config gotcha, infra detail) | **Memory** |
+| You learn who someone is, what team they're on, what they work on | **Memory** |
+| You notice a pattern in when/how the user communicates | **Memory** |
+| A job succeeded in an unusual way worth remembering | **Memory** |
+| You figure out the relationship between projects, services, or people | **Memory** |
+
+**The key principle:** if you'd want to know this at the start of your next session, save it now. Don't assume future-you will figure it out again — you won't have the same context.
+
+### Hygiene
+
+- **Before adding:** call `read_memory` / check rules.md — don't duplicate
+- **Update > add:** if a memory or rule already covers the topic, update it instead
+- **Date memories:** always include the date so stale entries are obvious
+- **Remove stale entries:** travel plans that passed, deadlines that shipped, incidents that are resolved
+- **Keep rules lean:** every rule costs tokens in every session — max ~20 rules, each must earn its place

package/src/types/engine.ts

@@ -2,6 +2,8 @@ export interface SendResult {
   result: string;
   costUsd: number;
   turns: number;
+  /** DB message ID for delivery status tracking (only set for agent replies) */
+  messageId?: number;
 }
 
 export type StreamCallback = (textSoFar: string) => void;