niahere 0.4.0 → 0.4.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "niahere",
-  "version": "0.4.0",
+  "version": "0.4.2",
   "description": "A personal AI assistant daemon — chat, scheduled jobs, persona system, extensible via skills.",
   "type": "module",
   "scripts": {
@@ -44,9 +44,9 @@
   "license": "MIT",
   "private": false,
   "dependencies": {
-    "@anthropic-ai/claude-agent-sdk": "^0.2.126",
-    "@anthropic-ai/sdk": "^0.88.0",
-    "@modelcontextprotocol/sdk": "^1.27.1",
+    "@anthropic-ai/claude-agent-sdk": "^0.3.190",
+    "@anthropic-ai/sdk": "^0.105.0",
+    "@modelcontextprotocol/sdk": "^1.29.0",
     "@slack/bolt": "^4.6.0",
     "cron-parser": "^5.5.0",
     "grammy": "^1.41.1",

package/src/agent/backends/claude-normalize.ts

@@ -1,5 +1,5 @@
 import type { AgentEvent, Normalizer } from "../types";
-import { truncate, formatToolUse } from "../../utils/format-activity";
+import { truncate } from "../../utils/format-activity";
 import { isRetryableApiError, isProviderDownError } from "../../utils/retry";
 
 /**
@@ -30,22 +30,14 @@ export class SdkNormalizer implements Normalizer {
     }
 
     if (msg.type === "tool_use_summary") {
-      return [
-        {
-          type: "tool",
-          name: msg.tool_name || "tool",
-          summary: formatToolUse(msg.tool_name || "tool", msg.tool_input),
-        },
-      ];
+      // The SDK provides a ready-made human-readable summary (e.g. "Read foo.ts").
+      // (Older code read tool_name/tool_input, which this event does not carry.)
+      return msg.summary ? [{ type: "tool", name: "tool", summary: truncate(msg.summary, 70) }] : [];
     }
 
     if (msg.type === "tool_progress") {
-      if (msg.tool_name === "Bash" && msg.content) {
-        return [{ type: "tool", name: "Bash", summary: `$ ${truncate(msg.content, 60)}` }];
-      }
-      if (msg.content) {
-        return [{ type: "tool", name: msg.tool_name || "tool", summary: truncate(msg.content, 70) }];
-      }
+      // Carries only tool_use_id/tool_name/elapsed_time — no displayable content,
+      // and fires repeatedly. tool_use_summary already covers tool activity.
       return [];
     }
 

package/src/agent/backends/claude.ts

@@ -151,8 +151,14 @@ class ClaudeSession implements AgentSession {
             retry = true;
             break;
           }
-          yield ev;
-          if (ev.type === "result" || ev.type === "error") {
+          // A retryable error that survived all our retries means the provider is
+          // effectively down for us — flag it so the consumer can fail over.
+          const out =
+            ev.type === "error" && ev.retryable && this.retryCount >= MAX_SEND_RETRIES
+              ? { ...ev, providerDown: true }
+              : ev;
+          yield out;
+          if (out.type === "result" || out.type === "error") {
             this.retryCount = 0;
             return;
           }

package/src/agent/backends/codex.ts

@@ -1,9 +1,37 @@
+import { existsSync, readdirSync } from "fs";
+import { homedir } from "os";
+import { join, dirname } from "path";
 import type { AgentBackend, AgentSession, AgentSessionContext, AgentEvent } from "../types";
 import type { Attachment } from "../../types/attachment";
 import type { McpSourceContext } from "../../mcp";
 import { CodexNormalizer } from "./codex-normalize";
 import { mintRun, revokeRun } from "../mcp-endpoint";
 
+/**
+ * Resolve the codex binary's absolute path. The daemon runs under launchd with a
+ * minimal PATH (`/usr/bin:/bin:...`) that excludes nvm/homebrew bins, so a bare
+ * `codex` spawn would fail. Search the likely install locations (env override,
+ * the runtime's own bin, homebrew, every nvm node bin, bun) and fall back to
+ * PATH only as a last resort. Cached after first resolution.
+ */
+let cachedCodexBin: string | null = null;
+export function resolveCodexBin(): string {
+  if (cachedCodexBin) return cachedCodexBin;
+  const candidates: string[] = [];
+  if (process.env.CODEX_PATH) candidates.push(process.env.CODEX_PATH);
+  candidates.push(join(dirname(process.execPath), "codex")); // sibling of bun/node
+  candidates.push("/opt/homebrew/bin/codex", "/usr/local/bin/codex");
+  try {
+    const nvm = join(homedir(), ".nvm", "versions", "node");
+    for (const v of readdirSync(nvm)) candidates.push(join(nvm, v, "bin", "codex"));
+  } catch {
+    /* no nvm */
+  }
+  candidates.push(join(homedir(), ".bun", "bin", "codex"));
+  cachedCodexBin = candidates.find((p) => existsSync(p)) ?? "codex";
+  return cachedCodexBin;
+}
+
 /** Minimal spawned-process surface, injectable so the session is unit-testable. */
 export interface CliProc {
   stdout: ReadableStream<Uint8Array>;
@@ -33,7 +61,12 @@ function scrubbedEnv(extra: Record<string, string>): Record<string, string> {
 }
 
 function defaultSpawn(args: string[], opts: { cwd: string; env: Record<string, string> }): CliProc {
-  const proc = Bun.spawn(["codex", ...args], { cwd: opts.cwd, env: opts.env, stdout: "pipe", stderr: "pipe" });
+  const proc = Bun.spawn([resolveCodexBin(), ...args], {
+    cwd: opts.cwd,
+    env: opts.env,
+    stdout: "pipe",
+    stderr: "pipe",
+  });
   return {
     stdout: proc.stdout as ReadableStream<Uint8Array>,
     stderr: proc.stderr as ReadableStream<Uint8Array>,