niahere 0.2.22 → 0.2.24

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "niahere",
-  "version": "0.2.22",
+  "version": "0.2.24",
   "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

@@ -508,36 +508,45 @@ class SlackChannel implements Channel {
       log.warn({ err }, "could not get slack bot user ID");
     }
 
-    // Resolve watch channel names → IDs
-    const watchConfig = config.channels.slack.watch;
-    if (watchConfig) {
-      try {
-        const channelList: { id: string; name: string }[] = [];
-        let cursor: string | undefined;
-        do {
-          const resp = await app.client.conversations.list({
-            types: "public_channel,private_channel",
-            exclude_archived: true,
-            limit: 200,
-            cursor,
-          });
-          for (const ch of resp.channels || []) {
-            if (ch.id && ch.name) channelList.push({ id: ch.id, name: ch.name });
-          }
-          cursor = resp.response_metadata?.next_cursor || undefined;
-        } while (cursor);
-
-        for (const [name, cfg] of Object.entries(watchConfig)) {
-          const match = channelList.find((c) => c.name === name);
-          if (match) {
-            watchChannels.set(match.id, { name, behavior: cfg.behavior });
-            log.info({ channel: name, id: match.id }, "slack: watching channel");
-          } else {
-            log.warn({ channel: name }, "slack: watch channel not found");
+    // Parse watch channels — keys are "channel_id#channel_name" or just "channel_name"
+    const rawWatchConfig = config.channels.slack.watch;
+    if (rawWatchConfig) {
+      for (const [key, cfg] of Object.entries(rawWatchConfig)) {
+        const hashIdx = key.indexOf("#");
+        if (hashIdx !== -1) {
+          // channel_id#channel_name format — use ID directly, no API call needed
+          const id = key.slice(0, hashIdx);
+          const name = key.slice(hashIdx + 1);
+          watchChannels.set(id, { name, behavior: cfg.behavior });
+          log.info({ channel: name, id }, "slack: watching channel");
+        } else {
+          // Legacy: plain channel name — resolve via API
+          try {
+            const channelList: { id: string; name: string }[] = [];
+            let cursor: string | undefined;
+            do {
+              const resp = await app.client.conversations.list({
+                types: "public_channel,private_channel",
+                exclude_archived: true,
+                limit: 200,
+                cursor,
+              });
+              for (const ch of resp.channels || []) {
+                if (ch.id && ch.name) channelList.push({ id: ch.id, name: ch.name });
+              }
+              cursor = resp.response_metadata?.next_cursor || undefined;
+            } while (cursor);
+            const match = channelList.find((c) => c.name === key);
+            if (match) {
+              watchChannels.set(match.id, { name: key, behavior: cfg.behavior });
+              log.info({ channel: key, id: match.id }, "slack: watching channel (resolved by name)");
+            } else {
+              log.warn({ channel: key }, "slack: watch channel not found");
+            }
+          } catch (err) {
+            log.warn({ err, channel: key }, "slack: failed to resolve watch channel");
           }
         }
-      } catch (err) {
-        log.warn({ err }, "slack: failed to resolve watch channels");
       }
     }
 

package/src/chat/repl.ts

@@ -6,16 +6,7 @@ import { getMcpServers, setMcpServers } from "../mcp";
 import { createNiaMcpServer } from "../mcp/server";
 import { Session } from "../db/models";
 import { relativeTime } from "../utils/format";
-
-// ANSI helpers
-const DIM = "\x1b[2m";
-const BOLD = "\x1b[1m";
-const CYAN = "\x1b[36m";
-const RESET = "\x1b[0m";
-const CLEAR_LINE = "\x1b[2K\r";
-
-// Braille spinner frames
-const SPINNER = ["⠋", "⠙", "⠹", "⠸", "⠼", "⠴", "⠦", "⠧", "⠇", "⠏"];
+import { DIM, BOLD, CYAN, RESET, CLEAR_LINE, SPINNER } from "../utils/cli";
 
 class StatusLine {
   private frame = 0;

package/src/cli/channels.ts

@@ -1,7 +1,7 @@
 import { getConfig, updateRawConfig } from "../utils/config";
 import { getPaths } from "../utils/paths";
 import { errMsg } from "../utils/errors";
-import { fail } from "../utils/cli";
+import { fail, ICON_PASS, ICON_FAIL } from "../utils/cli";
 import { log } from "../utils/log";
 
 export async function sendCommand(): Promise<void> {
@@ -98,7 +98,7 @@ export async function slackCommand(): Promise<void> {
         });
         const data = (await resp.json()) as Record<string, unknown>;
         if (data.ok) {
-          console.log(`  Auth: \u2713 valid`);
+          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);
@@ -108,10 +108,10 @@ export async function slackCommand(): Promise<void> {
             }
           }
         } else {
-          console.log(`  Auth: \u2717 ${data.error}`);
+          console.log(`  Auth: ${ICON_FAIL} ${data.error}`);
         }
       } catch (err) {
-        console.log(`  Auth: \u2717 could not reach Slack API`);
+        console.log(`  Auth: ${ICON_FAIL} could not reach Slack API`);
       }
     } else {
       console.log("Slack: not configured");

package/src/cli/index.ts

@@ -8,7 +8,7 @@ import { Message } from "../db/models";
 import { withDb } from "../db/connection";
 import { getNiaHome, getPaths } from "../utils/paths";
 import { errMsg } from "../utils/errors";
-import { fail } from "../utils/cli";
+import { fail, ICON_PASS, ICON_WARN } from "../utils/cli";
 import { jobCommand } from "./job";
 import { statusCommand } from "./status";
 import { sendCommand, telegramCommand, slackCommand } from "./channels";
@@ -64,14 +64,14 @@ async function awaitStartup(timeout = 60_000): Promise<void> {
       if (ready.has(name)) continue;
       if (content.includes(STARTUP_MARKERS[name])) {
         ready.add(name);
-        console.log(`  \u2713 ${name}`);
+        console.log(`  ${ICON_PASS} ${name}`);
       }
     }
   }
 
   const pending = [...expecting].filter((e) => !ready.has(e));
   if (pending.length > 0) {
-    console.log(`  \u26A0 timed out waiting for: ${pending.join(", ")}`);
+    console.log(`  ${ICON_WARN} timed out waiting for: ${pending.join(", ")}`);
   }
 }
 
@@ -143,9 +143,7 @@ switch (command) {
     if (prompt) {
       const { createChatEngine } = await import("../chat/engine");
       const { getMcpServers } = await import("../mcp");
-      const DIM = "\x1b[2m";
-      const RST = "\x1b[0m";
-      const FRAMES = ["⠋", "⠙", "⠹", "⠸", "⠼", "⠴", "⠦", "⠧", "⠇", "⠏"];
+      const { DIM, RESET: RST, CLEAR_LINE, SPINNER: FRAMES } = await import("../utils/cli");
       let frame = 0;
       let statusText = "thinking";
       let spinTimer: ReturnType<typeof setInterval> | null = null;
@@ -153,7 +151,7 @@ switch (command) {
       let streaming = false;
 
       const renderSpinner = () => {
-        process.stderr.write(`\x1b[2K\r${DIM}  ${FRAMES[frame]} ${statusText}${RST}`);
+        process.stderr.write(`${CLEAR_LINE}${DIM}  ${FRAMES[frame]} ${statusText}${RST}`);
         frame = (frame + 1) % FRAMES.length;
       };
 
@@ -412,6 +410,14 @@ switch (command) {
     process.exit(exitCode);
   }
 
+  case "validate": {
+    const { validateConfig } = await import("../commands/validate");
+    const result = validateConfig();
+    for (const msg of result.messages) console.log(`  ${msg}`);
+    console.log(result.ok ? "\nConfig is valid." : "\nConfig has errors.");
+    process.exit(result.ok ? 0 : 1);
+  }
+
   case "init": {
     const { runInit } = await import("../commands/init");
     await runInit();
@@ -434,6 +440,7 @@ switch (command) {
     console.log("  memory [show|reset] — view or reset memory.md");
     console.log("  db <sub>            — database setup/status/migrate");
     console.log("  skills              — list available skills");
+    console.log("  validate            — validate config.yaml");
     console.log("  config <sub>        — get/set/list config values");
     console.log("  send [-c ch] <msg>  — send a message via channel");
     console.log("  telegram <token>    — configure telegram");

package/src/cli/job.ts

@@ -8,7 +8,7 @@ import { Job } from "../db/models";
 import { withDb } from "../db/connection";
 import type { ScheduleType } from "../types";
 import { errMsg } from "../utils/errors";
-import { fail, pickFromList } from "../utils/cli";
+import { fail, pickFromList, ICON_PASS, ICON_FAIL } from "../utils/cli";
 import { computeInitialNextRun } from "../core/scheduler";
 
 async function pickJob(prompt = "Pick a job"): Promise<string> {
@@ -164,7 +164,7 @@ export async function jobCommand(): Promise<void> {
             for (const e of entries) {
               const time = localTime(new Date(e.timestamp));
               const dur = `${e.duration_ms}ms`;
-              const icon = e.status === "ok" ? "\u2713" : "\u2717";
+              const icon = e.status === "ok" ? ICON_PASS : ICON_FAIL;
               const summary = e.error || e.result.slice(0, 60).replace(/\n/g, " ") || "-";
               console.log(`    ${icon} ${time}  ${dur.padStart(8)}  ${summary}`);
             }
@@ -230,7 +230,7 @@ export async function jobCommand(): Promise<void> {
       for (const e of entries) {
         const time = localTime(new Date(e.timestamp));
         const dur = `${e.duration_ms}ms`;
-        const status = e.status === "ok" ? "\u2713" : "\u2717";
+        const status = e.status === "ok" ? ICON_PASS : ICON_FAIL;
         const summary = e.error || e.result.slice(0, 80).replace(/\n/g, " ") || "-";
         console.log(`  ${status} ${time}  ${dur.padStart(8)}  ${e.job}  ${summary}`);
       }

package/src/cli/status.ts

@@ -8,6 +8,7 @@ import type { ScheduleType, JobStateStatus, RoomStats } from "../types";
 import { withDb } from "../db/connection";
 import { errMsg } from "../utils/errors";
 import { checkForUpdate } from "../utils/update";
+import { ICON_PASS, ICON_FAIL, ICON_RUNNING } from "../utils/cli";
 
 type StatusOptions = {
   json: boolean;
@@ -245,7 +246,7 @@ export async function statusCommand(argv: string[] = []): Promise<void> {
           safeDate(nextRun)!.getTime() <= now.getTime() &&
           !stateInfo;
 
-        const statusIcon = status === "ok" ? "\u2713" : status === "error" ? "\u2717" : status === "running" ? "\u21bb" : "\u2217";
+        const statusIcon = status === "ok" ? ICON_PASS : status === "error" ? ICON_FAIL : status === "running" ? ICON_RUNNING : "\u2217";
         const durationText = stateInfo?.duration_ms === undefined ? "n/a" : `${stateInfo.duration_ms}ms`;
         const nextText = nextRun ? formatTimeLine(nextRun, now) : "unknown";
         const lastText = lastRun ? formatTimeLine(lastRun, now) : "never";
@@ -292,7 +293,7 @@ export async function statusCommand(argv: string[] = []): Promise<void> {
       console.log("\nJobs (from state file):");
       for (const [name, info] of fallbackEntries) {
         const last = formatTimeLine(info.lastRun, now);
-        const icon = info.status === "ok" ? "\u2713" : info.status === "error" ? "\u2717" : "\u2217";
+        const icon = info.status === "ok" ? ICON_PASS : info.status === "error" ? ICON_FAIL : "\u2217";
         console.log(`  ${icon} ${name}: ${info.status} (last: ${last}, ${info.duration_ms}ms)`);
       }
     } else if (dbError) {

package/src/commands/health.ts

@@ -98,14 +98,11 @@ export async function healthCommand(): Promise<void> {
   push(checks, "bun", "ok", "v" + bunVersion);
 
   // Output
-  const GREEN = "\x1b[32m";
-  const YELLOW = "\x1b[33m";
-  const RED = "\x1b[31m";
-  const RST = "\x1b[0m";
+  const { GREEN, YELLOW, RED, RESET, ICON_PASS, ICON_FAIL, ICON_WARN } = await import("../utils/cli");
   const icons: Record<string, string> = {
-    ok: GREEN + "\u2713" + RST,
-    warn: YELLOW + "!" + RST,
-    fail: RED + "\u2717" + RST,
+    ok: GREEN + ICON_PASS + RESET,
+    warn: YELLOW + ICON_WARN + RESET,
+    fail: RED + ICON_FAIL + RESET,
   };
 
   console.log();

package/src/commands/validate.ts

@@ -0,0 +1,150 @@
+import { existsSync, readFileSync } from "fs";
+import yaml from "js-yaml";
+import { getPaths } from "../utils/paths";
+import { ICON_PASS as PASS, ICON_FAIL as FAIL, ICON_WARN as WARN } from "../utils/cli";
+
+interface Result {
+  ok: boolean;
+  messages: string[];
+}
+
+function check(label: string, fn: () => string | null): { icon: string; label: string; detail?: string } {
+  const err = fn();
+  if (err) return { icon: FAIL, label, detail: err };
+  return { icon: PASS, label };
+}
+
+function warn(label: string, detail: string): { icon: string; label: string; detail?: string } {
+  return { icon: WARN, label, detail };
+}
+
+export function validateConfig(): Result {
+  const { config: configPath } = getPaths();
+  const messages: string[] = [];
+  let ok = true;
+
+  // File exists
+  if (!existsSync(configPath)) {
+    return { ok: false, messages: [`${FAIL} config.yaml not found at ${configPath}`] };
+  }
+
+  // Valid YAML
+  let raw: Record<string, unknown>;
+  try {
+    const parsed = yaml.load(readFileSync(configPath, "utf8"));
+    if (!parsed || typeof parsed !== "object") {
+      return { ok: false, messages: [`${FAIL} config.yaml is empty or not an object`] };
+    }
+    raw = parsed as Record<string, unknown>;
+    messages.push(`${PASS} valid YAML`);
+  } catch (err) {
+    return { ok: false, messages: [`${FAIL} invalid YAML: ${(err as Error).message}`] };
+  }
+
+  // Timezone
+  if (raw.timezone) {
+    try {
+      Intl.DateTimeFormat(undefined, { timeZone: raw.timezone as string });
+      messages.push(`${PASS} timezone: ${raw.timezone}`);
+    } catch {
+      messages.push(`${FAIL} invalid timezone: ${raw.timezone}`);
+      ok = false;
+    }
+  }
+
+  // Active hours
+  const ah = raw.active_hours as Record<string, string> | undefined;
+  if (ah) {
+    const timeRe = /^\d{2}:\d{2}$/;
+    if (ah.start && !timeRe.test(ah.start)) {
+      messages.push(`${FAIL} active_hours.start invalid: "${ah.start}" (expected HH:MM)`);
+      ok = false;
+    } else if (ah.start) {
+      messages.push(`${PASS} active_hours: ${ah.start}–${ah.end || "?"}`);
+    }
+    if (ah.end && !timeRe.test(ah.end)) {
+      messages.push(`${FAIL} active_hours.end invalid: "${ah.end}" (expected HH:MM)`);
+      ok = false;
+    }
+  }
+
+  // Database URL
+  const dbUrl = (process.env.DATABASE_URL || raw.database_url) as string | undefined;
+  if (dbUrl && dbUrl.startsWith("postgres")) {
+    messages.push(`${PASS} database_url set`);
+  } else if (!dbUrl) {
+    messages.push(`${WARN} database_url not set (will use default)`);
+  }
+
+  // Runner
+  const runner = raw.runner as string | undefined;
+  if (runner && runner !== "claude" && runner !== "codex") {
+    messages.push(`${FAIL} runner must be "claude" or "codex", got "${runner}"`);
+    ok = false;
+  } else if (runner) {
+    messages.push(`${PASS} runner: ${runner}`);
+  }
+
+  // Channels
+  const ch = raw.channels as Record<string, unknown> | undefined;
+  if (ch) {
+    // Telegram
+    const tg = ch.telegram as Record<string, unknown> | undefined;
+    if (tg) {
+      if (tg.bot_token) {
+        messages.push(`${PASS} telegram.bot_token set`);
+      } else {
+        messages.push(`${WARN} telegram.bot_token missing — telegram won't start`);
+      }
+    }
+
+    // Slack
+    const sl = ch.slack as Record<string, unknown> | undefined;
+    if (sl) {
+      if (!sl.bot_token) {
+        messages.push(`${WARN} slack.bot_token missing — slack won't start`);
+      } else {
+        messages.push(`${PASS} slack.bot_token set`);
+      }
+      if (!sl.app_token) {
+        messages.push(`${WARN} slack.app_token missing — slack won't start (Socket Mode requires app_token)`);
+      } else {
+        messages.push(`${PASS} slack.app_token set`);
+      }
+
+      // Watch channels
+      const watch = sl.watch as Record<string, unknown> | undefined;
+      if (watch) {
+        for (const [key, val] of Object.entries(watch)) {
+          if (!val || typeof val !== "object") {
+            messages.push(`${FAIL} slack.watch.${key}: must be an object with "behavior" field`);
+            ok = false;
+            continue;
+          }
+          const behavior = (val as Record<string, unknown>).behavior;
+          if (typeof behavior !== "string" || !behavior.trim()) {
+            messages.push(`${FAIL} slack.watch.${key}: missing "behavior" string`);
+            ok = false;
+            continue;
+          }
+          const hasId = key.includes("#");
+          if (hasId) {
+            messages.push(`${PASS} slack.watch: ${key}`);
+          } else {
+            messages.push(`${WARN} slack.watch.${key}: using channel name — prefer "channel_id#${key}" format for reliability`);
+          }
+        }
+      }
+    }
+
+    // Unknown channel keys
+    const knownChannelKeys = new Set(["enabled", "default", "telegram", "slack"]);
+    for (const key of Object.keys(ch)) {
+      if (!knownChannelKeys.has(key)) {
+        messages.push(`${WARN} unknown channel key: "channels.${key}" — did you mean "channels.slack"?`);
+      }
+    }
+  }
+
+  return { ok, messages };
+}

package/src/mcp/server.ts

@@ -88,7 +88,7 @@ export function createNiaMcpServer() {
         "add_watch_channel",
         "Add or update a Slack watch channel. Watch channels receive ALL messages (not just @mentions) and act based on the behavior prompt. Requires daemon restart to take effect.",
         {
-          name: z.string().describe("Slack channel name (without #), e.g. 'ask-kay-thread-notifications'"),
+          name: z.string().describe("Slack channel key as 'channel_id#channel_name', e.g. 'C1234567890#ask-kay-thread-notifications'"),
           behavior: z.string().describe("What to monitor and how to respond, e.g. 'Monitor thread notifications. Flag failures to #tech.'"),
         },
         async (args) => ({
@@ -99,7 +99,7 @@ export function createNiaMcpServer() {
         "remove_watch_channel",
         "Remove a Slack watch channel. Requires daemon restart to take effect.",
         {
-          name: z.string().describe("Slack channel name to stop watching"),
+          name: z.string().describe("Slack channel key to stop watching (e.g. 'C1234567890#ask-kay-thread-notifications')"),
         },
         async (args) => ({
           content: [{ type: "text" as const, text: handlers.removeWatchChannel(args.name) }],

package/src/prompts/channel-slack.md

@@ -28,6 +28,7 @@
 
 ### Watch mode
 - Some channels are configured for proactive monitoring via `channels.slack.watch` in config.
+- Watch channel keys use the format `channel_id#channel_name` (e.g. `C1234567890#ask-kay-thread-notifications`). The ID is used for matching; the name is for readability.
 - In watch channels, you receive ALL messages — not just @mentions. Messages are prefixed with `[Watch mode — #channel-name]` and a behavior prompt.
 - Follow the behavior prompt to decide what to do: flag issues, escalate, or stay quiet.
 - Use `[NO_REPLY]` for messages that don't need action. Most watch messages will be `[NO_REPLY]`.

package/src/prompts/environment.md

@@ -59,7 +59,7 @@ Config reference:
 - `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.watch` — per-channel proactive monitoring (see Watch mode in Slack docs)
+- `channels.slack.watch` — per-channel proactive monitoring. Keys are `channel_id#channel_name` format.
 {{slackWatch}}
 
 ## Persona & Memory

package/src/utils/cli.ts

@@ -1,5 +1,24 @@
 import * as readline from "readline";
 
+// ANSI colors
+export const DIM = "\x1b[2m";
+export const BOLD = "\x1b[1m";
+export const RESET = "\x1b[0m";
+export const RED = "\x1b[31m";
+export const GREEN = "\x1b[32m";
+export const YELLOW = "\x1b[33m";
+export const CYAN = "\x1b[36m";
+export const CLEAR_LINE = "\x1b[2K\r";
+
+// Icons
+export const ICON_PASS = "\u2713";
+export const ICON_FAIL = "\u2717";
+export const ICON_WARN = "\u26A0";
+export const ICON_RUNNING = "\u21bb";
+
+// Spinner frames
+export const SPINNER = ["⠋", "⠙", "⠹", "⠸", "⠼", "⠴", "⠦", "⠧", "⠇", "⠏"];
+
 export function fail(msg: string): never {
   console.log(msg);
   process.exit(1);