@jbarket/loomai 0.5.0 → 0.6.0

package/dist/config.d.ts

@@ -25,6 +25,20 @@ export declare function assertContextBootable(contextDir: string, defaultPath?:
 export declare function resolveRepoRoot(): string;
 export declare function resolveSqliteDbPath(contextDir: string): string;
 export declare function resolveKnowledgeDbPath(contextDir: string): string;
+/**
+ * Resolve the acting identity's name for actor attribution (knowledge
+ * pages.author / page_revisions.actor / verifications.verifier). This is
+ * deliberately NOT a tool argument — an LLM-supplied free-text identity is
+ * spoofable (any caller could claim to be any identity), so it is derived
+ * server-side from the connection instead.
+ *
+ * LOOM_IDENTITY overrides explicitly (useful for the multi-identity
+ * knowledge service, where each bearer token maps to a known identity).
+ * Otherwise derived from the context dir's real path basename — this
+ * resolves symlink aliases (e.g. `~/.config/loom/default` -> the real
+ * `.../loom/art` dir) to the actual identity name rather than the alias.
+ */
+export declare function resolveIdentityName(contextDir: string): string;
 /**
  * Local-only observation logs (e.g. `telemetry/recall.jsonl`). Lives inside
  * the context dir so it travels with the agent and never leaves the machine.

package/dist/config.js

@@ -14,10 +14,10 @@
  *   2. --context-dir CLI argument
  *   3. ~/.config/loom/default (fallback)
  */
-import { resolve, dirname } from 'node:path';
+import { resolve, dirname, basename } from 'node:path';
 import { homedir } from 'node:os';
 import { fileURLToPath } from 'node:url';
-import { existsSync, readFileSync, writeFileSync, statSync } from 'node:fs';
+import { existsSync, readFileSync, writeFileSync, statSync, realpathSync } from 'node:fs';
 /**
  * The single canonical helper for the default context path. Every call
  * site that needs `~/.config/loom/default` must go through this —
@@ -70,6 +70,29 @@ export function resolveSqliteDbPath(contextDir) {
 export function resolveKnowledgeDbPath(contextDir) {
     return process.env.LOOM_KNOWLEDGE_DB_PATH ?? resolve(contextDir, 'knowledge.db');
 }
+/**
+ * Resolve the acting identity's name for actor attribution (knowledge
+ * pages.author / page_revisions.actor / verifications.verifier). This is
+ * deliberately NOT a tool argument — an LLM-supplied free-text identity is
+ * spoofable (any caller could claim to be any identity), so it is derived
+ * server-side from the connection instead.
+ *
+ * LOOM_IDENTITY overrides explicitly (useful for the multi-identity
+ * knowledge service, where each bearer token maps to a known identity).
+ * Otherwise derived from the context dir's real path basename — this
+ * resolves symlink aliases (e.g. `~/.config/loom/default` -> the real
+ * `.../loom/art` dir) to the actual identity name rather than the alias.
+ */
+export function resolveIdentityName(contextDir) {
+    if (process.env.LOOM_IDENTITY)
+        return process.env.LOOM_IDENTITY;
+    try {
+        return basename(realpathSync(contextDir));
+    }
+    catch {
+        return basename(contextDir);
+    }
+}
 /**
  * Local-only observation logs (e.g. `telemetry/recall.jsonl`). Lives inside
  * the context dir so it travels with the agent and never leaves the machine.

package/dist/index.js

@@ -2,10 +2,16 @@
 /**
  * Loom — CLI + stdio MCP entry point.
  *
- * When argv[2] is a known CLI subcommand or --help/--version, routes to
- * src/cli/index.ts. Otherwise (or if argv is empty / only flags), falls
- * through to the MCP stdio server so existing .mcp.json configs keep
- * working.
+ * When the first non-global token is a known CLI subcommand or
+ * --help/--version, routes to src/cli/index.ts. Otherwise (or if argv is
+ * empty / only flags), falls through to the MCP stdio server so existing
+ * .mcp.json configs keep working.
+ *
+ * "First non-global token", not argv[2]: --context-dir and friends are
+ * documented as global flags, so `loom --context-dir DIR knowledge write`
+ * has to reach the CLI. Routing on argv[2] alone sent it to the stdio
+ * server instead, which read an empty stdin and exited 0 — a write verb
+ * reporting success having never entered the write path (t-665).
  *
  * Configure via environment variables:
  *   LOOM_CONTEXT_DIR         — path to identity/memory directory (required)
@@ -16,13 +22,15 @@
  */
 import { realpathSync } from 'node:fs';
 import { fileURLToPath } from 'node:url';
-import { StdioServerTransport } from '@modelcontextprotocol/sdk/server/stdio.js';
+import { StdioServerTransport } from '@modelcontextprotocol/server/stdio';
 import { createLoomServer } from './server.js';
 import { resolveContextDir } from './config.js';
 import { SUBCOMMANDS } from './cli/subcommands.js';
+import { firstNonGlobalIndex } from './cli/args.js';
 const CLI_KEYWORDS = new Set(SUBCOMMANDS);
 function isCliInvocation(argv) {
-    const first = argv[2];
+    const args = argv.slice(2);
+    const first = args[firstNonGlobalIndex(args)];
     if (first === undefined)
         return false;
     if (first === '--help' || first === '-h')
@@ -35,7 +43,14 @@ export { isCliInvocation };
 async function main() {
     if (isCliInvocation(process.argv)) {
         const { runCli } = await import('./cli/index.js');
-        process.exit(await runCli(process.argv.slice(2)));
+        const code = await runCli(process.argv.slice(2));
+        // Drain stdout before exiting — without this, large outputs (e.g.
+        // `loom memory list --json` on a big store) truncate at 64 KiB when
+        // piped because process.exit() tears down the stream before the OS
+        // buffer drains (t-198). process.stdout.write('', cb) fires after all
+        // prior writes have been handed to the OS.
+        process.stdout.write('', () => process.exit(code));
+        return;
     }
     const contextDir = resolveContextDir();
     const { server } = createLoomServer({ contextDir });

package/dist/injection/harnesses.js

@@ -2,9 +2,15 @@
  * Harness preset table for `loom inject`. Each entry names a target
  * harness, its canonical default path, and the MCP tool prefix to emit
  * in the injected instruction block.
+ *
+ * toolPrefix is NOT redeclared here — it's read from
+ * `src/install/harnesses.ts`'s INSTALL_TARGETS, which is the single
+ * source of truth for the prefix per harness (t-397: the two tables used
+ * to carry independent, disagreeing hardcoded values for codex).
  */
 import { homedir } from 'node:os';
 import { join } from 'node:path';
+import { INSTALL_TARGETS } from '../install/harnesses.js';
 export const HARNESS_KEYS = [
     'claude-code',
     'codex',
@@ -15,19 +21,19 @@ export const HARNESSES = {
         key: 'claude-code',
         display: 'Claude Code',
         defaultPath: join(homedir(), '.claude', 'CLAUDE.md'),
-        toolPrefix: 'mcp__loom__',
+        toolPrefix: INSTALL_TARGETS['claude-code'].toolPrefix,
     },
     'codex': {
         key: 'codex',
         display: 'Codex',
         defaultPath: join(homedir(), '.codex', 'AGENTS.md'),
-        toolPrefix: 'mcp__loom__',
+        toolPrefix: INSTALL_TARGETS['codex'].toolPrefix,
     },
     'gemini-cli': {
         key: 'gemini-cli',
         display: 'Gemini CLI',
         defaultPath: join(homedir(), '.gemini', 'GEMINI.md'),
-        toolPrefix: 'mcp__loom__',
+        toolPrefix: INSTALL_TARGETS['gemini-cli'].toolPrefix,
     },
 };
 export function isHarnessKey(s) {

package/dist/install/harnesses.js

@@ -27,7 +27,9 @@ export const INSTALL_TARGETS = {
         mcpConfigHint: '~/.codex/config.toml',
         invoke: 'use the loom-setup skill',
         restart: 'restart the Codex session',
-        toolPrefix: 'mcp_loom_',
+        // mcp__loom__, not mcp_loom_: a live Codex session's own tool catalog
+        // names it mcp__loom__identity (t-397, verified 2026-09-05 by mark).
+        toolPrefix: 'mcp__loom__',
     },
     'gemini-cli': {
         key: 'gemini-cli',
@@ -36,6 +38,10 @@ export const INSTALL_TARGETS = {
         mcpConfigHint: '~/.gemini/settings.json',
         invoke: 'use the loom-setup skill',
         restart: 'exit and restart Gemini CLI',
+        // mcp_loom_, single underscores: gemini-cli joins server + '_' + tool
+        // and prepends 'mcp_' (MCP_QUALIFIED_NAME_SEPARATOR / generateValidName in
+        // packages/core/src/tools/mcp-tool.ts @ b25669d), so the model sees
+        // mcp_loom_identity (t-397, caught by mark in review of #122).
         toolPrefix: 'mcp_loom_',
     },
     'opencode': {

package/dist/server.d.ts

@@ -1,6 +1,13 @@
-import { McpServer } from '@modelcontextprotocol/sdk/server/mcp.js';
+import { McpServer } from '@modelcontextprotocol/server';
 export interface LoomServerConfig {
     contextDir: string;
+    /**
+     * Optional path to the knowledge-gaps directory (e.g. `~/Art/.knowledge-gaps`).
+     * When set, zero-result knowledge_recall calls append a miss entry to
+     * `${gapsDir}/recall-miss.jsonl`, feeding the expansion engine's second
+     * signal channel. Set via LOOM_GAPS_DIR env var in production.
+     */
+    gapsDir?: string;
 }
 export interface LoomServerInstance {
     server: McpServer;