niahere 0.2.41 → 0.2.42

package/package.json

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

package/src/channels/index.ts

@@ -4,7 +4,7 @@ import { log } from "../utils/log";
 import { createTelegramChannel } from "./telegram";
 import { createSlackChannel } from "./slack";
 
-export { getChannel } from "./registry";
+export { getChannel, getStarted } from "./registry";
 
 /** Register all built-in channel factories. Call once at startup. */
 export function registerAllChannels(): void {

package/src/channels/registry.ts

@@ -19,6 +19,10 @@ export function getChannel(name: string): Channel | undefined {
   return started.get(name);
 }
 
+export function getStarted(): Channel[] {
+  return [...started.values()];
+}
+
 export function clearStarted(): void {
   started.clear();
 }

package/src/chat/repl.ts

@@ -2,7 +2,7 @@ import * as readline from "readline";
 import { createChatEngine } from "./engine";
 import { runMigrations } from "../db/migrate";
 import { closeDb } from "../db/connection";
-import { getMcpServers, setMcpServers } from "../mcp";
+import { getMcpServers, setMcpFactory } from "../mcp";
 import { createNiaMcpServer } from "../mcp/server";
 import { Session } from "../db/models";
 import { relativeTime } from "../utils/format";
@@ -113,12 +113,9 @@ export async function startRepl(mode: ChatMode = "continue", simulateChannel?: s
     process.exit(1);
   }
 
-  // Initialize MCP server if not already set (standalone chat mode)
+  // Initialize MCP server factory if not already set (standalone chat mode)
   if (!getMcpServers()) {
-    try {
-      const mcpConfig = createNiaMcpServer();
-      setMcpServers({ nia: mcpConfig });
-    } catch {}
+    setMcpFactory(() => ({ nia: createNiaMcpServer() }));
   }
 
   // Determine session to use

package/src/cli/index.ts

@@ -381,12 +381,16 @@ switch (command) {
   case "channels": {
     const sub = process.argv[3];
     const { updateRawConfig } = await import("../utils/config");
-    if (sub === "on") {
-      updateRawConfig({ channels: { enabled: true } });
-      console.log("channels enabled — restart to apply");
-    } else if (sub === "off") {
-      updateRawConfig({ channels: { enabled: false } });
-      console.log("channels disabled — restart to apply");
+    if (sub === "on" || sub === "off") {
+      const enabled = sub === "on";
+      updateRawConfig({ channels: { enabled } });
+      const pid = readPid();
+      if (pid && isRunning()) {
+        process.kill(pid, "SIGHUP");
+        console.log(`channels ${enabled ? "enabled" : "disabled"}`);
+      } else {
+        console.log(`channels ${enabled ? "enabled" : "disabled"} — start nia to apply`);
+      }
     } else {
       console.log(`channels: ${getConfig().channels.enabled ? "on" : "off"}`);
     }

package/src/core/daemon.ts

@@ -1,17 +1,17 @@
 import { closeSync, existsSync, mkdirSync, openSync, readFileSync, unlinkSync, writeFileSync } from "fs";
 import { dirname } from "path";
 import { getPaths } from "../utils/paths";
-import { getConfig } from "../utils/config";
+import { getConfig, resetConfig } from "../utils/config";
 import { log } from "../utils/log";
 import { ActiveEngine } from "../db/models";
 import { runMigrations } from "../db/migrate";
 import { closeDb, getSql } from "../db/connection";
-import { registerAllChannels, startChannels, stopChannels } from "../channels";
+import { registerAllChannels, startChannels, stopChannels, getStarted } from "../channels";
 import type { Channel } from "../types";
 import { startScheduler, stopScheduler, recomputeAllNextRuns } from "./scheduler";
 import { startAlive, stopAlive } from "./alive";
 import { createNiaMcpServer } from "../mcp/server";
-import { setMcpServers } from "../mcp";
+import { setMcpFactory } from "../mcp";
 
 export function writePid(pid: number): void {
   const { pid: pidPath } = getPaths();
@@ -218,14 +218,9 @@ export async function runDaemon(): Promise<void> {
     log.info({ recovered }, "recovered stale running jobs");
   }
 
-  // Initialize MCP server (in-process, no HTTP needed)
-  try {
-    const mcpConfig = createNiaMcpServer();
-    setMcpServers({ nia: mcpConfig });
-    log.info("MCP server initialized");
-  } catch (err) {
-    log.error({ err }, "failed to initialize MCP server");
-  }
+  // Initialize MCP server factory (each query gets its own Protocol instance)
+  setMcpFactory(() => ({ nia: createNiaMcpServer() }));
+  log.info("MCP server factory initialized");
 
   // Register and start channels
   registerAllChannels();
@@ -265,9 +260,26 @@ export async function runDaemon(): Promise<void> {
     log.warn({ err }, "could not subscribe to nia_jobs, falling back to SIGHUP only");
   }
 
-  // SIGHUP as manual fallback
+  // SIGHUP: reload config, reconcile channels, recompute jobs
   process.on("SIGHUP", async () => {
-    log.info("received SIGHUP, recomputing job schedules");
+    log.info("received SIGHUP, reloading config");
+    resetConfig();
+    const fresh = getConfig();
+
+    const running = getStarted();
+    const wantChannels = fresh.channels.enabled;
+    const haveChannels = running.length > 0;
+
+    if (wantChannels && !haveChannels) {
+      log.info("SIGHUP: starting channels");
+      const result = await startChannels();
+      channels = result.started;
+    } else if (!wantChannels && haveChannels) {
+      log.info("SIGHUP: stopping channels");
+      await stopChannels(running);
+      channels = [];
+    }
+
     await recomputeAllNextRuns().catch(() => {});
   });
 

package/src/mcp/index.ts

@@ -1,10 +1,10 @@
-/** Shared MCP server config — set by daemon, read by chat engine creators. */
-let _mcpServers: Record<string, unknown> | null = null;
+/** Factory for per-query MCP servers — each query gets its own Protocol instance. */
+let _mcpFactory: (() => Record<string, unknown>) | null = null;
 
-export function setMcpServers(servers: Record<string, unknown>): void {
-  _mcpServers = servers;
+export function setMcpFactory(factory: () => Record<string, unknown>): void {
+  _mcpFactory = factory;
 }
 
 export function getMcpServers(): Record<string, unknown> | undefined {
-  return _mcpServers ?? undefined;
+  return _mcpFactory?.() ?? undefined;
 }