niahere 0.2.22 → 0.2.23

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "niahere",
-  "version": "0.2.22",
+  "version": "0.2.23",
   "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/cli/index.ts

@@ -412,6 +412,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 +442,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/commands/validate.ts

@@ -0,0 +1,153 @@
+import { existsSync, readFileSync } from "fs";
+import yaml from "js-yaml";
+import { getPaths } from "../utils/paths";
+
+const PASS = "\u2713";
+const FAIL = "\u2717";
+const WARN = "\u26A0";
+
+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