niahere 0.2.31 → 0.2.33

package/package.json

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

package/src/chat/identity.ts

@@ -1,14 +1,12 @@
-import { existsSync, readFileSync, readdirSync } from "fs";
-import { join, resolve } from "path";
-import { homedir } from "os";
-import yaml from "js-yaml";
-import { getNiaHome, getPaths } from "../utils/paths";
+import { existsSync, readFileSync } from "fs";
+import { join } from "path";
+import { getPaths } from "../utils/paths";
 import { getEnvironmentPrompt, getModePrompt, getChannelPrompt } from "../prompts";
-import { log } from "../utils/log";
+import { getSkillsSummary } from "../core/skills";
 import type { Mode } from "../types";
 
-// niahere project root (resolved from this file's location)
-const PROJECT_ROOT = resolve(import.meta.dir, "../..");
+// Re-export for backwards compat
+export { scanSkills as loadSkills, getSkillNames as loadSkillNames, type SkillInfo } from "../core/skills";
 
 function loadFile(dir: string, name: string): string {
   const filePath = join(dir, name);
@@ -22,71 +20,6 @@ export function loadIdentity(): string {
   return files.map((f) => loadFile(selfDir, f)).filter(Boolean).join("\n\n");
 }
 
-function scanSkills(): { name: string; description: string }[] {
-  const home = homedir();
-  const cwd = process.cwd();
-  const niaHome = getNiaHome();
-  const skillDirs = [
-    join(cwd, "skills"),
-    join(PROJECT_ROOT, "skills"),
-    join(niaHome, "skills"),
-    join(home, ".shared", "skills"),
-    join(home, ".claude", "skills"),
-    join(home, ".codex", "skills"),
-  ];
-
-  const skills: { name: string; description: string }[] = [];
-  const seen = new Set<string>();
-
-  for (const dir of skillDirs) {
-    if (!existsSync(dir)) continue;
-
-    for (const entry of readdirSync(dir, { withFileTypes: true })) {
-      if (!entry.isDirectory() || entry.name.startsWith(".")) continue;
-
-      const skillFile = join(dir, entry.name, "SKILL.md");
-      if (!existsSync(skillFile)) continue;
-
-      const content = readFileSync(skillFile, "utf8");
-      const fmMatch = content.match(/^---\n([\s\S]*?)\n---/);
-      if (!fmMatch) continue;
-
-      let meta: Record<string, unknown> = {};
-      try {
-        meta = (yaml.load(fmMatch[1]) as Record<string, unknown>) || {};
-      } catch (err) {
-        log.warn({ err, skill: entry.name, path: skillFile }, "failed to parse skill metadata, skipping");
-        continue;
-      }
-      const name = (typeof meta.name === "string" ? meta.name : "") || entry.name;
-
-      if (seen.has(name)) continue;
-      seen.add(name);
-
-      skills.push({
-        name,
-        description: typeof meta.description === "string" ? meta.description : "",
-      });
-    }
-  }
-
-  return skills;
-}
-
-export function loadSkillNames(): string[] {
-  return scanSkills().map((s) => s.name);
-}
-
-export function loadSkillsSummary(): string {
-  const skills = scanSkills();
-  if (skills.length === 0) return "";
-
-  const lines = skills.map((s) =>
-    s.description ? `- /${s.name}: ${s.description}` : `- /${s.name}`,
-  );
-  return `Available skills:\n${lines.join("\n")}`;
-}
-
 export function buildSystemPrompt(mode: Mode = "chat", channel: string = "terminal"): string {
   const parts: string[] = [];
 
@@ -101,7 +34,7 @@ export function buildSystemPrompt(mode: Mode = "chat", channel: string = "termin
   const channelPrompt = getChannelPrompt(channel);
   if (channelPrompt) parts.push(channelPrompt);
 
-  const skills = loadSkillsSummary();
+  const skills = getSkillsSummary();
   if (skills) parts.push(skills);
 
   return parts.join("\n\n");

package/src/cli/index.ts

@@ -287,12 +287,19 @@ switch (command) {
   }
 
   case "skills": {
-    const { loadSkillNames } = await import("../chat/identity");
-    const names = loadSkillNames();
-    if (names.length === 0) {
-      console.log("No skills found.");
+    const { scanSkills: loadSkills } = await import("../core/skills");
+    const filter = process.argv[3]; // e.g. "project", "nia", "shared", "claude"
+    let skills = loadSkills();
+    if (filter) {
+      skills = skills.filter((s) => s.source === filter);
+    }
+    if (skills.length === 0) {
+      console.log(filter ? `No skills found in "${filter}".` : "No skills found.");
     } else {
-      for (const name of names) console.log(`  ${name}`);
+      for (const s of skills) {
+        const tag = filter ? "" : `  [${s.source}]`;
+        console.log(`  ${s.name}${tag}`);
+      }
     }
     break;
   }

package/src/core/skills.ts

@@ -0,0 +1,67 @@
+import { existsSync, readFileSync, readdirSync } from "fs";
+import { join, resolve } from "path";
+import { homedir } from "os";
+import yaml from "js-yaml";
+import { getNiaHome } from "../utils/paths";
+import { log } from "../utils/log";
+
+// niahere project root (resolved from this file's location)
+const PROJECT_ROOT = resolve(import.meta.dir, "../..");
+
+export type SkillInfo = { name: string; description: string; source: string };
+
+const SKILL_DIRS: { dir: string; source: string }[] = [
+  { dir: join(process.cwd(), "skills"), source: "cwd" },
+  { dir: join(PROJECT_ROOT, "skills"), source: "project" },
+  { dir: join(getNiaHome(), "skills"), source: "nia" },
+  { dir: join(homedir(), ".shared", "skills"), source: "shared" },
+  { dir: join(homedir(), ".claude", "skills"), source: "claude" },
+  { dir: join(homedir(), ".codex", "skills"), source: "codex" },
+];
+
+export function scanSkills(): SkillInfo[] {
+  const skills: SkillInfo[] = [];
+  const seen = new Set<string>();
+
+  for (const { dir, source } of SKILL_DIRS) {
+    if (!existsSync(dir)) continue;
+
+    for (const entry of readdirSync(dir, { withFileTypes: true })) {
+      if (!entry.isDirectory() || entry.name.startsWith(".")) continue;
+
+      const skillFile = join(dir, entry.name, "SKILL.md");
+      if (!existsSync(skillFile)) continue;
+
+      const content = readFileSync(skillFile, "utf8");
+      const fmMatch = content.match(/^---\n([\s\S]*?)\n---/);
+      if (!fmMatch) continue;
+
+      let meta: Record<string, unknown> = {};
+      try {
+        meta = (yaml.load(fmMatch[1]) as Record<string, unknown>) || {};
+      } catch (err) {
+        log.warn({ err, skill: entry.name, path: skillFile }, "failed to parse skill metadata, skipping");
+        continue;
+      }
+      const name = (typeof meta.name === "string" ? meta.name : "") || entry.name;
+
+      if (seen.has(name)) continue;
+      seen.add(name);
+
+      skills.push({ name, description: typeof meta.description === "string" ? meta.description : "", source });
+    }
+  }
+
+  return skills;
+}
+
+export function getSkillNames(): string[] {
+  return scanSkills().map((s) => s.name);
+}
+
+export function getSkillsSummary(): string {
+  const skills = scanSkills();
+  if (skills.length === 0) return "";
+  const lines = skills.map((s) => s.description ? `- /${s.name}: ${s.description}` : `- /${s.name}`);
+  return `Available skills:\n${lines.join("\n")}`;
+}

package/src/mcp/server.ts

@@ -135,9 +135,17 @@ export function createNiaMcpServer() {
           content: [{ type: "text" as const, text: handlers.addRule(args.rule) }],
         }),
       ),
+      tool(
+        "read_memory",
+        "Read all saved memories. Use this to check what you already know before saving duplicates, or to recall context about the owner, past incidents, preferences, etc.",
+        {},
+        async () => ({
+          content: [{ type: "text" as const, text: handlers.readMemory() }],
+        }),
+      ),
       tool(
         "add_memory",
-        "Save a concise factual memory for future reference. Memories are read on demand, not loaded automatically. Use for preferences, corrections, or patterns worth keeping. RULES: Max 300 chars. One insight per entry. NO raw logs, NO conversation transcripts, NO status dumps, NO duplicate observations. Bad: pasting nia status output. Good: 'curator job can get stuck in running state — needs timeout recovery'.",
+        "Save a concise factual memory for future reference. Proactively save personal facts (travel, schedule), work context (decisions, deadlines), and corrections — don't wait to be asked. RULES: Max 300 chars. One insight per entry. NO raw logs, NO transcripts, NO status dumps.",
         {
           entry: z.string().max(300).describe("A single concise insight (max 300 chars, no raw logs or transcripts)"),
         },

package/src/mcp/tools.ts

@@ -273,6 +273,17 @@ export function disableWatchChannel(name: string): string {
   return `Watch channel "${name}" disabled. Takes effect on next message.`;
 }
 
+export function readMemory(): string {
+  const { selfDir } = getPaths();
+  const memoryPath = join(selfDir, "memory.md");
+  if (!existsSync(memoryPath)) return "No memories saved yet.";
+  const content = readFileSync(memoryPath, "utf8").trim();
+  // Extract just the entries, skip the header/instructions
+  const lines = content.split("\n").filter((l) => l.startsWith("- ") || l.startsWith("## "));
+  if (lines.length === 0) return "No memories saved yet.";
+  return lines.join("\n");
+}
+
 export function addMemory(entry: string): string {
   // Guard: reject raw logs, transcripts, and overly long entries
   const trimmed = entry.trim();

package/src/prompts/environment.md

@@ -34,7 +34,8 @@ You have MCP tools for managing jobs directly (preferred over CLI for speed):
 - **remove_watch_channel** — stop watching a Slack channel. Hot-reloads.
 - **enable_watch_channel** / **disable_watch_channel** — toggle a watch channel on/off without removing it. Hot-reloads.
 - **add_rule** — save a behavioral rule (loaded into every session, no restart needed). Use when told "from now on", "always", "never", or "remember to always..."
-- **add_memory** — save a factual memory (read on demand). Use when told "remember that...", or when you learn something surprising worth keeping
+- **read_memory** — recall all saved memories. Check before saving to avoid duplicates, or when you need context about the owner.
+- **add_memory** — save a factual memory. Proactively save personal facts, work context, corrections — don't wait to be asked.
 
 Active hours: {{activeStart}}–{{activeEnd}} ({{timezone}}). Jobs respect this; crons (always=true) don't.
 
@@ -90,9 +91,18 @@ Your persona files live in {{selfDir}}/:
 **Memory** (`memory.md`) = facts and context. Read on demand when relevant.
 - "2026-03-13: DB was down, Telegram send failed"
 - "Aman prefers terminal over Slack for debugging"
-- Use `add_memory` tool to save new memories.
+- Use `read_memory` to recall what you know. Use `add_memory` to save new memories.
 
 **Which to use?**
 - "From now on, do X" → rule
 - "Remember that X happened" / "I prefer X" → memory
+
+### When to save (proactive)
+Don't wait for the user to say "remember this." Proactively save when you learn:
+- Personal facts: travel plans, location, schedule, preferences
+- Work context: project decisions, team changes, deadlines
+- Corrections: user corrected you on something worth remembering
+- Patterns: recurring requests, preferred communication style
+
+Example: if the owner says "I'm going home on the 21st, early morning flight" — save it as a memory without being asked. These are facts future sessions need.
 - If unsure, ask.