@venturewild/workspace 0.6.0 → 0.6.2

package/server/src/transcript.mjs

@@ -1,121 +1,121 @@
-// TranscriptRecorder — the conversation *content* channel (separate from the
-// events/health feed in session-reporter.mjs).
-//
-// WHY a separate channel: per the locked design, the feed carries WHAT happened
-// (redacted, no words); the actual conversation is captured here and synced via
-// bmo-sync — modeled on the team's transcript-backup (Claude `Stop` hook →
-// jsonl→markdown → claude_backups/<user>/<date>/). bmo-sync REPLACES that Dropbox
-// transport: each user's transcript is appended to an account-scoped store, so we
-// avoid the "… conflicted copy …" litter a shared mutable file accrues (CLAUDE.md
-// principle #1 — and the exact failure observed in the team's Dropbox folder).
-//
-// HOW: taps the ActivityBus (which carries the raw turn text the wild-workspace
-// chat already streams), buffers turns to markdown, and APPENDS incrementally
-// (append-only, never a rewrite — no shared mutable doc) to
-// ~/.wild-workspace/transcripts/<workspaceId>/<date>.md — OUTSIDE the synced
-// repo. The local write is the user's own data (harmless); forwarding to us is
-// consent-gated (the caller supplies `forwardImpl` only when consent is on).
-// Debounced ~10 min like hook_trigger.py.
-
-import fs from 'node:fs';
-import path from 'node:path';
-
-const FLUSH_MS = 10 * 60 * 1000;
-
-export class TranscriptRecorder {
-  constructor({
-    dir,
-    agentName = 'Agent',
-    flushMs = FLUSH_MS,
-    nowImpl = () => Date.now(),
-    appendImpl,
-    forwardImpl = null,
-  } = {}) {
-    this.dir = dir;
-    this.agentName = agentName || 'Agent';
-    this.flushMs = flushMs;
-    this.nowImpl = nowImpl;
-    this.appendImpl =
-      appendImpl ||
-      ((file, data) => {
-        fs.mkdirSync(path.dirname(file), { recursive: true });
-        fs.appendFileSync(file, data);
-      });
-    this.forwardImpl = forwardImpl;
-    this.turn = null; //   in-flight turn { user, assistant:[], tools:[], ts }
-    this.lines = []; //    un-flushed markdown lines
-    this.timer = null;
-  }
-
-  ingest(ev) {
-    if (!ev || typeof ev !== 'object') return;
-    switch (ev.type) {
-      case 'chat-user':
-        this.turn = { user: ev.text || '', assistant: [], tools: [], ts: ev.ts };
-        break;
-      case 'chat-stream': {
-        if (!this.turn) this.turn = { user: '', assistant: [], tools: [], ts: ev.ts };
-        const c = ev.chunk || {};
-        if (c.type === 'text' && typeof c.text === 'string') this.turn.assistant.push(c.text);
-        else if (c.type === 'tool-use') this.turn.tools.push(c.name || c.tool || 'tool');
-        break;
-      }
-      case 'chat-end':
-        this._finalizeTurn();
-        this._scheduleFlush();
-        break;
-      default:
-        break;
-    }
-  }
-
-  _finalizeTurn() {
-    if (!this.turn) return;
-    const t = this.turn;
-    this.turn = null;
-    const ts = new Date(t.ts || this.nowImpl()).toISOString();
-    if (t.user) this.lines.push(`### You — ${ts}`, '', t.user, '');
-    if (t.tools.length) this.lines.push(`> tools: ${[...new Set(t.tools)].join(', ')}`, '');
-    const assistant = t.assistant.join('').trim();
-    if (assistant) this.lines.push(`### ${this.agentName}`, '', assistant, '');
-  }
-
-  _scheduleFlush() {
-    if (this.timer) return;
-    this.timer = setTimeout(() => {
-      this.timer = null;
-      this.flush();
-    }, this.flushMs);
-    if (this.timer.unref) this.timer.unref();
-  }
-
-  /** Append the un-flushed markdown to today's file and forward it. Never throws. */
-  flush() {
-    if (!this.lines.length) return;
-    const body = this.lines.join('\n') + '\n';
-    this.lines = [];
-    const date = new Date(this.nowImpl()).toISOString().slice(0, 10);
-    const file = path.join(this.dir, `${date}.md`);
-    try {
-      this.appendImpl(file, body);
-    } catch {
-      /* read-only fs — local backup degrades, never breaks the chat */
-    }
-    if (this.forwardImpl) {
-      try {
-        this.forwardImpl({ file, date, markdown: body });
-      } catch {
-        /* forwarding is best-effort */
-      }
-    }
-  }
-
-  stop() {
-    if (this.timer) {
-      clearTimeout(this.timer);
-      this.timer = null;
-    }
-    this._finalizeTurn();
-    this.flush();
-  }
-}
+// TranscriptRecorder — the conversation *content* channel (separate from the
+// events/health feed in session-reporter.mjs).
+//
+// WHY a separate channel: per the locked design, the feed carries WHAT happened
+// (redacted, no words); the actual conversation is captured here and synced via
+// bmo-sync — modeled on the team's transcript-backup (Claude `Stop` hook →
+// jsonl→markdown → claude_backups/<user>/<date>/). bmo-sync REPLACES that Dropbox
+// transport: each user's transcript is appended to an account-scoped store, so we
+// avoid the "… conflicted copy …" litter a shared mutable file accrues (CLAUDE.md
+// principle #1 — and the exact failure observed in the team's Dropbox folder).
+//
+// HOW: taps the ActivityBus (which carries the raw turn text the wild-workspace
+// chat already streams), buffers turns to markdown, and APPENDS incrementally
+// (append-only, never a rewrite — no shared mutable doc) to
+// ~/.wild-workspace/transcripts/<workspaceId>/<date>.md — OUTSIDE the synced
+// repo. The local write is the user's own data (harmless); forwarding to us is
+// consent-gated (the caller supplies `forwardImpl` only when consent is on).
+// Debounced ~10 min like hook_trigger.py.
+
+import fs from 'node:fs';
+import path from 'node:path';
+
+const FLUSH_MS = 10 * 60 * 1000;
+
+export class TranscriptRecorder {
+  constructor({
+    dir,
+    agentName = 'Agent',
+    flushMs = FLUSH_MS,
+    nowImpl = () => Date.now(),
+    appendImpl,
+    forwardImpl = null,
+  } = {}) {
+    this.dir = dir;
+    this.agentName = agentName || 'Agent';
+    this.flushMs = flushMs;
+    this.nowImpl = nowImpl;
+    this.appendImpl =
+      appendImpl ||
+      ((file, data) => {
+        fs.mkdirSync(path.dirname(file), { recursive: true });
+        fs.appendFileSync(file, data);
+      });
+    this.forwardImpl = forwardImpl;
+    this.turn = null; //   in-flight turn { user, assistant:[], tools:[], ts }
+    this.lines = []; //    un-flushed markdown lines
+    this.timer = null;
+  }
+
+  ingest(ev) {
+    if (!ev || typeof ev !== 'object') return;
+    switch (ev.type) {
+      case 'chat-user':
+        this.turn = { user: ev.text || '', assistant: [], tools: [], ts: ev.ts };
+        break;
+      case 'chat-stream': {
+        if (!this.turn) this.turn = { user: '', assistant: [], tools: [], ts: ev.ts };
+        const c = ev.chunk || {};
+        if (c.type === 'text' && typeof c.text === 'string') this.turn.assistant.push(c.text);
+        else if (c.type === 'tool-use') this.turn.tools.push(c.name || c.tool || 'tool');
+        break;
+      }
+      case 'chat-end':
+        this._finalizeTurn();
+        this._scheduleFlush();
+        break;
+      default:
+        break;
+    }
+  }
+
+  _finalizeTurn() {
+    if (!this.turn) return;
+    const t = this.turn;
+    this.turn = null;
+    const ts = new Date(t.ts || this.nowImpl()).toISOString();
+    if (t.user) this.lines.push(`### You — ${ts}`, '', t.user, '');
+    if (t.tools.length) this.lines.push(`> tools: ${[...new Set(t.tools)].join(', ')}`, '');
+    const assistant = t.assistant.join('').trim();
+    if (assistant) this.lines.push(`### ${this.agentName}`, '', assistant, '');
+  }
+
+  _scheduleFlush() {
+    if (this.timer) return;
+    this.timer = setTimeout(() => {
+      this.timer = null;
+      this.flush();
+    }, this.flushMs);
+    if (this.timer.unref) this.timer.unref();
+  }
+
+  /** Append the un-flushed markdown to today's file and forward it. Never throws. */
+  flush() {
+    if (!this.lines.length) return;
+    const body = this.lines.join('\n') + '\n';
+    this.lines = [];
+    const date = new Date(this.nowImpl()).toISOString().slice(0, 10);
+    const file = path.join(this.dir, `${date}.md`);
+    try {
+      this.appendImpl(file, body);
+    } catch {
+      /* read-only fs — local backup degrades, never breaks the chat */
+    }
+    if (this.forwardImpl) {
+      try {
+        this.forwardImpl({ file, date, markdown: body });
+      } catch {
+        /* forwarding is best-effort */
+      }
+    }
+  }
+
+  stop() {
+    if (this.timer) {
+      clearTimeout(this.timer);
+      this.timer = null;
+    }
+    this._finalizeTurn();
+    this.flush();
+  }
+}

package/server/src/turn-mcp.mjs

@@ -1,46 +1,46 @@
-// Turn MCP config — what the agent's MCP toolbox is on a user chat turn.
-//
-// The turn exposes TWO in-repo MCP servers, isolated by --strict-mcp-config:
-//   - bazaar — reach onto the marketplace shelf (§3.7)
-//   - canvas — build the user a custom block on their canvas (§3.3)
-// Both run with the same node and share our state dir via WILD_WORKSPACE_GLOBAL_DIR,
-// so the spawned children are a single source of truth with the main server. We
-// write ONE merged config (a single --mcp-config the main server passes), rather
-// than relying on multiple --mcp-config flags, so the merge is explicit and tested.
-
-import fs from 'node:fs';
-import path from 'node:path';
-
-import { MCP_SERVER_PATH as BAZAAR_MCP_SERVER_PATH, BAZAAR_SYSTEM_PROMPT } from './bazaar/index.mjs';
-import { CANVAS_MCP_SERVER_PATH, CANVAS_SYSTEM_PROMPT } from './canvas/index.mjs';
-
-// The combined disposition the agent gets appended to its system prompt.
-export const TURN_SYSTEM_PROMPT = `${BAZAAR_SYSTEM_PROMPT}\n\n${CANVAS_SYSTEM_PROMPT}`;
-
-/**
- * Build the merged mcpServers map. Pure (no I/O) so it's unit-testable.
- * `globalDir` is pinned into each child's env so it finds the shared state dir.
- */
-export function turnMcpServers({ globalDir, nodePath = process.execPath } = {}) {
-  const env = globalDir ? { WILD_WORKSPACE_GLOBAL_DIR: globalDir } : {};
-  return {
-    bazaar: { command: nodePath, args: [BAZAAR_MCP_SERVER_PATH], env },
-    canvas: { command: nodePath, args: [CANVAS_MCP_SERVER_PATH], env },
-  };
-}
-
-/**
- * Write the merged --mcp-config the turn loads. Returns the config path, or null
- * if it can't be written (read-only fs → the turn just runs without MCP tools).
- */
-export function writeTurnMcpConfig({ baseDir, globalDir, nodePath = process.execPath } = {}) {
-  const cfg = { mcpServers: turnMcpServers({ globalDir, nodePath }) };
-  const file = path.join(baseDir, 'turn-mcp-config.json');
-  try {
-    fs.mkdirSync(baseDir, { recursive: true });
-    fs.writeFileSync(file, JSON.stringify(cfg, null, 2));
-    return file;
-  } catch {
-    return null;
-  }
-}
+// Turn MCP config — what the agent's MCP toolbox is on a user chat turn.
+//
+// The turn exposes TWO in-repo MCP servers, isolated by --strict-mcp-config:
+//   - bazaar — reach onto the marketplace shelf (§3.7)
+//   - canvas — build the user a custom block on their canvas (§3.3)
+// Both run with the same node and share our state dir via WILD_WORKSPACE_GLOBAL_DIR,
+// so the spawned children are a single source of truth with the main server. We
+// write ONE merged config (a single --mcp-config the main server passes), rather
+// than relying on multiple --mcp-config flags, so the merge is explicit and tested.
+
+import fs from 'node:fs';
+import path from 'node:path';
+
+import { MCP_SERVER_PATH as BAZAAR_MCP_SERVER_PATH, BAZAAR_SYSTEM_PROMPT } from './bazaar/index.mjs';
+import { CANVAS_MCP_SERVER_PATH, CANVAS_SYSTEM_PROMPT } from './canvas/index.mjs';
+
+// The combined disposition the agent gets appended to its system prompt.
+export const TURN_SYSTEM_PROMPT = `${BAZAAR_SYSTEM_PROMPT}\n\n${CANVAS_SYSTEM_PROMPT}`;
+
+/**
+ * Build the merged mcpServers map. Pure (no I/O) so it's unit-testable.
+ * `globalDir` is pinned into each child's env so it finds the shared state dir.
+ */
+export function turnMcpServers({ globalDir, nodePath = process.execPath } = {}) {
+  const env = globalDir ? { WILD_WORKSPACE_GLOBAL_DIR: globalDir } : {};
+  return {
+    bazaar: { command: nodePath, args: [BAZAAR_MCP_SERVER_PATH], env },
+    canvas: { command: nodePath, args: [CANVAS_MCP_SERVER_PATH], env },
+  };
+}
+
+/**
+ * Write the merged --mcp-config the turn loads. Returns the config path, or null
+ * if it can't be written (read-only fs → the turn just runs without MCP tools).
+ */
+export function writeTurnMcpConfig({ baseDir, globalDir, nodePath = process.execPath } = {}) {
+  const cfg = { mcpServers: turnMcpServers({ globalDir, nodePath }) };
+  const file = path.join(baseDir, 'turn-mcp-config.json');
+  try {
+    fs.mkdirSync(baseDir, { recursive: true });
+    fs.writeFileSync(file, JSON.stringify(cfg, null, 2));
+    return file;
+  } catch {
+    return null;
+  }
+}