niahere 0.2.73 → 0.2.75

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "niahere",
-  "version": "0.2.73",
+  "version": "0.2.75",
   "description": "A personal AI assistant daemon — chat, scheduled jobs, persona system, extensible via skills.",
   "type": "module",
   "scripts": {
@@ -44,7 +44,7 @@
   "license": "MIT",
   "private": false,
   "dependencies": {
-    "@anthropic-ai/claude-agent-sdk": "^0.2.97",
+    "@anthropic-ai/claude-agent-sdk": "^0.2.119",
     "@anthropic-ai/sdk": "^0.88.0",
     "@modelcontextprotocol/sdk": "^1.27.1",
     "@slack/bolt": "^4.6.0",

package/src/channels/slack.ts

@@ -22,15 +22,14 @@ function cleanSentinel(text: string): string {
 class SlackChannel implements Channel {
   name = "slack";
   private app: App | null = null;
-  private defaultChannelId: string | null = null;
   private dmUserId: string | null = null;
   /** Timestamps of messages Nia posted proactively (used to detect replies to our own messages) */
   private outboundTs = new Set<string>();
 
   async sendMessage(text: string): Promise<void> {
     if (!this.app) throw new Error("Slack not started");
-    const target = this.defaultChannelId || this.dmUserId;
-    if (!target) throw new Error("No Slack recipient — DM the bot first, or set slack_channel_id in config");
+    const target = this.dmUserId;
+    if (!target) throw new Error("No Slack recipient — set dm_user_id in config");
     const result = await this.app.client.chat.postMessage({ channel: target, text });
     if (result.ts) this.outboundTs.add(result.ts);
   }
@@ -45,8 +44,8 @@ class SlackChannel implements Channel {
 
   async sendMedia(data: Buffer, mimeType: string, filename?: string): Promise<void> {
     if (!this.app) throw new Error("Slack not started");
-    const target = this.defaultChannelId || this.dmUserId;
-    if (!target) throw new Error("No Slack recipient — DM the bot first, or set slack_channel_id in config");
+    const target = this.dmUserId;
+    if (!target) throw new Error("No Slack recipient — set dm_user_id in config");
     await this.app.client.filesUploadV2({
       channel_id: target,
       file: data,
@@ -61,7 +60,6 @@ class SlackChannel implements Channel {
 
     await runMigrations();
 
-    this.defaultChannelId = config.channels.slack.channel_id;
     this.dmUserId = config.channels.slack.dm_user_id;
 
     const chats = new Map<string, ChatState>();

package/src/cli/channels.ts

@@ -7,21 +7,38 @@ import { log } from "../utils/log";
 export async function sendCommand(): Promise<void> {
   const args = process.argv.slice(3);
   let channel: string | undefined;
+  let toChannelId: string | undefined;
+  let threadTs: string | undefined;
   const msgParts: string[] = [];
   for (let i = 0; i < args.length; i++) {
     if ((args[i] === "--channel" || args[i] === "-c") && args[i + 1]) {
       channel = args[++i];
+    } else if (args[i] === "--to" && args[i + 1]) {
+      toChannelId = args[++i];
+    } else if (args[i] === "--thread" && args[i + 1]) {
+      threadTs = args[++i];
     } else {
       msgParts.push(args[i]);
     }
   }
   const message = msgParts.join(" ");
-  if (!message) fail("Usage: nia send [-c channel] <message>");
+  if (!message) fail("Usage: nia send [-c channel] [--to <slack-channel-id>] [--thread <ts>] <message>");
+
+  // --to implies slack channel
+  if (toChannelId) channel = channel || "slack";
+  // --thread requires --to
+  if (threadTs && !toChannelId) fail("--thread requires --to <slack-channel-id>");
 
   const { sendMessage } = await import("../mcp/tools");
 
+  // Build sourceCtx for targeted channel/thread sends
+  const sourceCtx = toChannelId
+    ? { slackChannelId: toChannelId, slackThreadTs: threadTs, channel: "slack" as const }
+    : undefined;
+  const target = toChannelId ? "thread" as const : "auto" as const;
+
   try {
-    const result = await sendMessage(message, channel);
+    const result = await sendMessage(message, channel, undefined, sourceCtx, target);
     console.log(result);
   } catch (err) {
     fail(`Failed to send: ${errMsg(err)}`);
@@ -29,27 +46,43 @@ export async function sendCommand(): Promise<void> {
 }
 
 export function telegramCommand(): void {
-  const token = process.argv[3];
-  const chatId = process.argv[4];
+  const sub = process.argv[3];
 
-  if (!token) {
-    const config = getConfig();
-    if (config.channels.telegram.bot_token) {
-      console.log(`Telegram: configured (...${config.channels.telegram.bot_token.slice(-6)})`);
-    } else {
-      console.log("Telegram: not configured");
+  if (sub === "setup") {
+    const args = process.argv.slice(4);
+    let token: string | undefined;
+    let chatId: string | undefined;
+
+    for (const arg of args) {
+      if (arg.startsWith("--bot-token=")) token = arg.slice("--bot-token=".length);
+      else if (arg.startsWith("--chat-id=")) chatId = arg.slice("--chat-id=".length);
     }
-    console.log("\nUsage: nia telegram <bot-token> [chat-id]");
+
+    if (!token) {
+      fail("Usage: nia telegram setup --bot-token=<token> [--chat-id=<id>]");
+    }
+
+    const tg: Record<string, unknown> = { bot_token: token };
+    if (chatId) tg.chat_id = Number(chatId);
+    updateRawConfig({ channels: { telegram: tg } });
+
+    console.log(`Telegram bot token saved to ${getPaths().config}`);
+    if (chatId) console.log(`Chat ID: ${chatId}`);
+    console.log("Run `nia restart` to activate.");
     return;
   }
 
-  const tg: Record<string, unknown> = { bot_token: token };
-  if (chatId) tg.chat_id = Number(chatId);
-  updateRawConfig({ channels: { telegram: tg } });
-
-  console.log(`Telegram bot token saved to ${getPaths().config}`);
-  if (chatId) console.log(`Chat ID: ${chatId}`);
-  console.log("Run `nia restart` to activate.");
+  // Default: show status
+  const config = getConfig();
+  if (config.channels.telegram.bot_token) {
+    console.log(`Telegram: configured (...${config.channels.telegram.bot_token.slice(-6)})`);
+    if (config.channels.telegram.chat_id) {
+      console.log(`  Chat ID: ${config.channels.telegram.chat_id}`);
+    }
+  } else {
+    console.log("Telegram: not configured");
+  }
+  console.log("\nUsage: nia telegram setup --bot-token=<token> [--chat-id=<id>]");
 }
 
 /** Call Slack auth.test to enrich config with workspace/bot info. */
@@ -80,60 +113,79 @@ export async function enrichSlackConfig(botToken: string): Promise<Record<string
 }
 
 export async function slackCommand(): Promise<void> {
-  const botToken = process.argv[3];
-  const appToken = process.argv[4];
-
-  if (!botToken) {
-    const config = getConfig();
-    if (config.channels.slack.bot_token) {
-      console.log(`Slack: configured (...${config.channels.slack.bot_token.slice(-6)})`);
-      if (config.channels.slack.workspace) {
-        console.log(`  Workspace: ${config.channels.slack.workspace} (${config.channels.slack.workspace_url})`);
-        console.log(`  Bot: @${config.channels.slack.bot_name} (${config.channels.slack.bot_user_id})`);
-      }
-      // Verify auth is working
-      try {
-        const resp = await fetch("https://slack.com/api/auth.test", {
-          headers: { Authorization: `Bearer ${config.channels.slack.bot_token}` },
-        });
-        const data = (await resp.json()) as Record<string, unknown>;
-        if (data.ok) {
-          console.log(`  Auth: ${ICON_PASS} valid`);
-          // Backfill workspace info if missing
-          if (!config.channels.slack.workspace) {
-            const enriched = await enrichSlackConfig(config.channels.slack.bot_token);
-            if (Object.keys(enriched).length > 0) {
-              updateRawConfig({ channels: { slack: enriched } });
-              console.log("  (workspace info backfilled)");
-            }
-          }
-        } else {
-          console.log(`  Auth: ${ICON_FAIL} ${data.error}`);
-        }
-      } catch (err) {
-        console.log(`  Auth: ${ICON_FAIL} could not reach Slack API`);
-      }
-    } else {
-      console.log("Slack: not configured");
+  const sub = process.argv[3];
+
+  if (sub === "setup") {
+    const args = process.argv.slice(4);
+    let botToken: string | undefined;
+    let appToken: string | undefined;
+    let dmUserId: string | undefined;
+
+    for (const arg of args) {
+      if (arg.startsWith("--bot-token=")) botToken = arg.slice("--bot-token=".length);
+      else if (arg.startsWith("--app-token=")) appToken = arg.slice("--app-token=".length);
+      else if (arg.startsWith("--dm-user-id=")) dmUserId = arg.slice("--dm-user-id=".length);
     }
-    console.log("\nUsage: nia slack <bot-token> <app-token> [channel-id]");
-    console.log("\nCreate a Slack app: https://api.slack.com/apps (use defaults/channels/slack-manifest.json)");
-    return;
-  }
 
-  if (!appToken) fail("App token required. Usage: nia slack <bot-token> <app-token> [channel-id]");
+    if (!botToken || !appToken) {
+      fail("Usage: nia slack setup --bot-token=xoxb-... --app-token=xapp-... [--dm-user-id=U...]");
+    }
 
-  const sl: Record<string, unknown> = { bot_token: botToken, app_token: appToken };
-  const channelId = process.argv[5];
-  if (channelId) sl.channel_id = channelId;
+    if (!botToken.startsWith("xoxb-")) {
+      fail(`Invalid bot token — must start with "xoxb-" (got "${botToken.slice(0, 10)}...")`);
+    }
+    if (!appToken.startsWith("xapp-")) {
+      fail(`Invalid app token — must start with "xapp-" (got "${appToken.slice(0, 10)}...")`);
+    }
+
+    const sl: Record<string, unknown> = { bot_token: botToken, app_token: appToken };
+    if (dmUserId) sl.dm_user_id = dmUserId;
 
-  // Enrich with workspace/bot info from auth.test
-  const enriched = await enrichSlackConfig(botToken);
-  Object.assign(sl, enriched);
+    const enriched = await enrichSlackConfig(botToken);
+    Object.assign(sl, enriched);
 
-  updateRawConfig({ channels: { slack: sl } });
+    updateRawConfig({ channels: { slack: sl } });
 
-  console.log(`Slack tokens saved to ${getPaths().config}`);
-  if (channelId) console.log(`Channel ID: ${channelId}`);
-  console.log("Run `nia restart` to activate.");
+    console.log(`Slack tokens saved to ${getPaths().config}`);
+    if (dmUserId) console.log(`DM user: ${dmUserId}`);
+    console.log("Run `nia restart` to activate.");
+    return;
+  }
+
+  // Default: show status
+  const config = getConfig();
+  if (config.channels.slack.bot_token) {
+    console.log(`Slack: configured (...${config.channels.slack.bot_token.slice(-6)})`);
+    if (config.channels.slack.workspace) {
+      console.log(`  Workspace: ${config.channels.slack.workspace} (${config.channels.slack.workspace_url})`);
+      console.log(`  Bot: @${config.channels.slack.bot_name} (${config.channels.slack.bot_user_id})`);
+    }
+    if (config.channels.slack.dm_user_id) {
+      console.log(`  DM user: ${config.channels.slack.dm_user_id}`);
+    }
+    try {
+      const resp = await fetch("https://slack.com/api/auth.test", {
+        headers: { Authorization: `Bearer ${config.channels.slack.bot_token}` },
+      });
+      const data = (await resp.json()) as Record<string, unknown>;
+      if (data.ok) {
+        console.log(`  Auth: ${ICON_PASS} valid`);
+        if (!config.channels.slack.workspace) {
+          const enriched = await enrichSlackConfig(config.channels.slack.bot_token);
+          if (Object.keys(enriched).length > 0) {
+            updateRawConfig({ channels: { slack: enriched } });
+            console.log("  (workspace info backfilled)");
+          }
+        }
+      } else {
+        console.log(`  Auth: ${ICON_FAIL} ${data.error}`);
+      }
+    } catch (err) {
+      console.log(`  Auth: ${ICON_FAIL} could not reach Slack API`);
+    }
+  } else {
+    console.log("Slack: not configured");
+  }
+  console.log("\nUsage: nia slack setup --bot-token=xoxb-... --app-token=xapp-... [--dm-user-id=U...]");
+  console.log("\nCreate a Slack app: https://api.slack.com/apps (use defaults/channels/slack-manifest.json)");
 }

package/src/cli/index.ts

@@ -572,7 +572,7 @@ Chat:
   chat [-c] [-r] [--employee|--agent|--job name]  Interactive chat
   run <prompt>                    One-shot execution
   history [room]                  Recent messages
-  send [-c ch] <msg>              Send a message via channel
+  send [-c ch] [--to C --thread T] <msg>  Send message (DM, channel, or thread)
 
 Jobs:
   job <sub>                       Manage jobs (list|add|update|remove|run|...)
@@ -587,8 +587,8 @@ Persona:
 Channels:
   channels [on|off]               Toggle channels
   watch <sub>                     Manage Slack watch channels
-  telegram <token>                Configure telegram
-  slack <bot> <app>               Configure slack
+  telegram [setup]                Configure telegram
+  slack [setup]                   Configure slack
 
 System:
   config <set|get|list>           Manage config values

package/src/commands/init.ts

@@ -168,7 +168,7 @@ export async function runInit(): Promise<void> {
     // Slack
     let slackBotToken = "";
     let slackAppToken = "";
-    let slackChannelId = (exSl.channel_id as string) || "";
+    let slackDmUserId = (exSl.dm_user_id as string) || "";
 
     const existingSlackBot = (exSl.bot_token as string) || "";
 
@@ -182,12 +182,12 @@ export async function runInit(): Promise<void> {
         const appInput = await ask(rl, "App token (xapp-...)", "");
         slackAppToken = appInput || existingSlackApp;
         if (slackBotToken && slackAppToken) {
-          slackChannelId = await ask(rl, "Default channel ID for outbound messages (optional)", slackChannelId);
+          slackDmUserId = await ask(rl, "DM user ID for outbound messages (U...)", slackDmUserId);
         }
       } else {
         slackBotToken = existingSlackBot;
         slackAppToken = (exSl.app_token as string) || "";
-        slackChannelId = (exSl.channel_id as string) || "";
+        slackDmUserId = (exSl.dm_user_id as string) || "";
       }
     } else {
       const setupSlack = await ask(rl, "\nSet up Slack? (y/n)", "n");
@@ -210,7 +210,7 @@ export async function runInit(): Promise<void> {
         slackAppToken = await ask(rl, "App token (xapp-...)", "");
 
         if (slackBotToken && slackAppToken) {
-          slackChannelId = await ask(rl, "Default channel ID for outbound messages (optional)", "");
+          slackDmUserId = await ask(rl, "DM user ID for outbound messages (U...)", "");
         }
       }
     }
@@ -421,7 +421,7 @@ export async function runInit(): Promise<void> {
     }
     if (slackBotToken && slackAppToken) {
       const sl: Record<string, unknown> = { bot_token: slackBotToken, app_token: slackAppToken };
-      if (slackChannelId) sl.channel_id = slackChannelId;
+      if (slackDmUserId) sl.dm_user_id = slackDmUserId;
       // Enrich with workspace/bot info from auth.test
       const enriched = await enrichSlackConfig(slackBotToken);
       Object.assign(sl, enriched);

package/src/core/alive.ts

@@ -100,7 +100,7 @@ async function notifyUser(message: string): Promise<void> {
   }
 
   const slToken = config.channels.slack.bot_token;
-  const slRecipient = config.channels.slack.dm_user_id || config.channels.slack.channel_id;
+  const slRecipient = config.channels.slack.dm_user_id;
   if (slToken && slRecipient) {
     try {
       const resp = await fetch("https://slack.com/api/chat.postMessage", {

package/src/mcp/tools.ts

@@ -171,9 +171,9 @@ async function sendDirect(target: string, text: string): Promise<void> {
 
   if (target === "slack") {
     const token = config.channels.slack.bot_token;
-    const recipient = config.channels.slack.channel_id || config.channels.slack.dm_user_id;
+    const recipient = config.channels.slack.dm_user_id;
     if (!token) throw new Error("Slack not configured (no bot token)");
-    if (!recipient) throw new Error("No Slack recipient — DM the bot first, or set slack_channel_id in config");
+    if (!recipient) throw new Error("No Slack recipient — set dm_user_id in config");
     const { App } = await import("@slack/bolt");
     const app = new App({ token, signingSecret: "unused" });
     await app.client.chat.postMessage({ token, channel: recipient, text });
@@ -205,9 +205,9 @@ async function sendMediaDirect(target: string, data: Buffer, mimeType: string, f
 
   if (target === "slack") {
     const token = config.channels.slack.bot_token;
-    const recipient = config.channels.slack.channel_id || config.channels.slack.dm_user_id;
+    const recipient = config.channels.slack.dm_user_id;
     if (!token) throw new Error("Slack not configured (no bot token)");
-    if (!recipient) throw new Error("No Slack recipient — DM the bot first, or set slack_channel_id in config");
+    if (!recipient) throw new Error("No Slack recipient — set dm_user_id in config");
     const { App } = await import("@slack/bolt");
     const app = new App({ token, signingSecret: "unused" });
     await app.client.filesUploadV2({
@@ -245,11 +245,8 @@ export async function sendMessage(text: string, channelName?: string, mediaPath?
       // Replying in-thread: use the source session's room prefix
       roomPrefix = sourceCtx.room.replace(/-\d+$/, "");
     } else {
-      const channelId = config.channels.slack.channel_id;
       const dmUserId = config.channels.slack.dm_user_id;
-      if (channelId) {
-        roomPrefix = `slack-${channelId}`;
-      } else if (dmUserId) {
+      if (dmUserId) {
         roomPrefix = `slack-dm-${dmUserId}`;
       }
     }

package/src/prompts/environment.md

@@ -100,8 +100,7 @@ Config reference:
 - `channels.telegram.open` — if true, anyone can message the bot
 - `channels.slack.bot_token` — Slack bot token (xoxb-...)
 - `channels.slack.app_token` — Slack app token (xapp-...)
-- `channels.slack.channel_id` — default Slack channel for outbound
-- `channels.slack.dm_user_id` — auto-registered DM user
+- `channels.slack.dm_user_id` — Slack user ID for DM-based outbound messages
 - `channels.slack.watch` — per-channel proactive monitoring. Keys use `channel_id#channel_name` format. The `behavior` field is optional and has three forms: (1) omitted — loads `~/.niahere/watches/<channel_name>/behavior.md`; (2) single word like `deal-monitor` — loads `~/.niahere/watches/deal-monitor/behavior.md` (dir-per-watch, like agents); (3) inline prose. File-backed watches hot-reload via mtime tracking, no restart needed.
   {{slackWatch}}
 

package/src/types/config.ts

@@ -19,7 +19,6 @@ export interface SlackWatchChannel {
 export interface SlackConfig {
   bot_token: string | null;
   app_token: string | null;
-  channel_id: string | null;
   dm_user_id: string | null;
   bot_user_id: string | null;
   bot_name: string | null;

package/src/utils/config.ts

@@ -23,7 +23,6 @@ const DEFAULTS: Config = {
     slack: {
       bot_token: null,
       app_token: null,
-      channel_id: null,
       dm_user_id: null,
       bot_user_id: null,
       bot_name: null,
@@ -124,9 +123,12 @@ export function loadConfig(): Config {
 
   const slAppToken = process.env.SLACK_APP_TOKEN || (typeof chSl.app_token === "string" ? chSl.app_token : null);
 
-  const slChannelId = process.env.SLACK_CHANNEL_ID || (typeof chSl.channel_id === "string" ? chSl.channel_id : null);
-
-  const slDmUserId = typeof chSl.dm_user_id === "string" ? chSl.dm_user_id : null;
+  // Legacy: channel_id was removed in favor of dm_user_id. Fall back to channel_id if dm_user_id is not set.
+  const legacyChannelId = process.env.SLACK_CHANNEL_ID || (typeof chSl.channel_id === "string" ? chSl.channel_id : null);
+  const slDmUserId =
+    process.env.SLACK_DM_USER_ID ||
+    (typeof chSl.dm_user_id === "string" ? chSl.dm_user_id : null) ||
+    legacyChannelId;
 
   const slBotUserId = typeof chSl.bot_user_id === "string" ? chSl.bot_user_id : null;
   const slBotName = typeof chSl.bot_name === "string" ? chSl.bot_name : null;
@@ -164,7 +166,6 @@ export function loadConfig(): Config {
       slack: {
         bot_token: slBotToken,
         app_token: slAppToken,
-        channel_id: slChannelId,
         dm_user_id: slDmUserId,
         bot_user_id: slBotUserId,
         bot_name: slBotName,

package/src/utils/pid.ts

@@ -3,23 +3,50 @@ import { dirname } from "path";
 import { getPaths } from "./paths";
 import { log } from "./log";
 
+type PidEntry = { pid: number; lstart: string };
+
+function getLstart(pid: number): string {
+  try {
+    const result = Bun.spawnSync(["ps", "-o", "lstart=", "-p", String(pid)]);
+    return new TextDecoder().decode(result.stdout).trim();
+  } catch {
+    return "";
+  }
+}
+
 export function writePid(pid: number): void {
   const { pid: pidPath } = getPaths();
+  const lstart = getLstart(pid);
+  if (!lstart) {
+    log.warn({ pid }, "could not capture pid identity (ps returned nothing)");
+  }
   mkdirSync(dirname(pidPath), { recursive: true });
-  writeFileSync(pidPath, String(pid));
+  writeFileSync(pidPath, JSON.stringify({ pid, lstart }));
 }
 
-export function readPid(): number | null {
+function readEntry(): PidEntry | null {
   const { pid: pidPath } = getPaths();
   if (!existsSync(pidPath)) return null;
 
   try {
-    return parseInt(readFileSync(pidPath, "utf8").trim(), 10);
+    const raw = readFileSync(pidPath, "utf8").trim();
+    if (/^\d+$/.test(raw)) {
+      return { pid: parseInt(raw, 10), lstart: "" };
+    }
+    const parsed = JSON.parse(raw);
+    if (typeof parsed?.pid === "number" && typeof parsed?.lstart === "string") {
+      return parsed as PidEntry;
+    }
+    return null;
   } catch {
     return null;
   }
 }
 
+export function readPid(): number | null {
+  return readEntry()?.pid ?? null;
+}
+
 export function removePid(): void {
   const { pid: pidPath } = getPaths();
   try {
@@ -30,15 +57,22 @@ export function removePid(): void {
 }
 
 export function isRunning(): boolean {
-  const pid = readPid();
-  if (pid === null) return false;
+  const entry = readEntry();
+  if (entry === null) return false;
 
-  try {
-    process.kill(pid, 0);
-    return true;
-  } catch {
-    log.warn({ stalePid: pid }, "removing stale pid file (process not running)");
+  const currentLstart = getLstart(entry.pid);
+  if (!currentLstart) {
+    log.warn({ stalePid: entry.pid }, "removing stale pid file (process not running)");
+    removePid();
+    return false;
+  }
+  if (entry.lstart && currentLstart !== entry.lstart) {
+    log.warn(
+      { stalePid: entry.pid, recorded: entry.lstart, current: currentLstart },
+      "removing stale pid file (process identity mismatch)",
+    );
     removePid();
     return false;
   }
+  return true;
 }