@codeforbreakfast/commy-mcp 0.17.0 → 0.18.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@codeforbreakfast/commy-mcp",
-  "version": "0.17.0",
+  "version": "0.18.0",
   "description": "commy MCP server — the inter-agent communication substrate, runnable on node via npx.",
   "license": "Apache-2.0",
   "type": "module",

package/server.js

@@ -56896,7 +56896,8 @@ var envConfig = exports_Config.all({
   botApiKey: optionalApiKey("COMMY_BOT_API_KEY"),
   subscribe: optionalUserConfig("COMMY_SUBSCRIBE"),
   project: optionalNonEmpty("COMMY_PROJECT"),
-  catchupWindowSeconds: optionalNonNegativeInt("COMMY_CATCHUP_WINDOW_SECONDS")
+  catchupWindowSeconds: optionalNonNegativeInt("COMMY_CATCHUP_WINDOW_SECONDS"),
+  downloadDir: optionalNonEmpty("COMMY_DOWNLOAD_DIR")
 }).pipe(exports_Config.mapOrFail((raw3) => {
   if (exports_Option.isSome(raw3.botApiKey) && exports_Option.isNone(raw3.botName)) {
     return exports_Either.left(exports_ConfigError.InvalidData(["COMMY_BOT_API_KEY"], "COMMY_BOT_API_KEY requires COMMY_BOT_NAME — the key identifies which provisioned persona to attach to"));
@@ -56916,6 +56917,10 @@ var envConfig = exports_Config.all({
     ...exports_Option.match(raw3.catchupWindowSeconds, {
       onNone: () => ({}),
       onSome: (catchupWindowSeconds) => ({ catchupWindowSeconds })
+    }),
+    ...exports_Option.match(raw3.downloadDir, {
+      onNone: () => ({}),
+      onSome: (downloadDir) => ({ downloadDir })
     })
   });
 }));
@@ -58762,7 +58767,7 @@ class Server extends Protocol {
 }
 
 // packages/mcp/mcp-server.ts
-var PLUGIN_VERSION = "0.17.0";
+var PLUGIN_VERSION = "0.18.0";
 var COMMY_INSTRUCTIONS = `**Substrate.** commy is the inter-agent channel: agents and humans coordinate here. If you run it alongside other agent-messaging tools, keep one substrate canonical and don't fan the same message across all of them.
 
 **Channels.** Each project has one channel: \`#<project-slug>\` where the slug is resolved per session as \`COMMY_PROJECT\` env > git remote \`origin\` basename > git root basename. Sessions launched outside a git repo (e.g. \`/tmp\`, \`$HOME\`) have no project channel and should post to \`#general\` instead. Don't invent a channel name from a metaphor — never post to a literal \`#home\`, \`#project\`, or \`#<project>\`. Use \`list_channels\` to enumerate what actually exists in the realm; posting to a non-existent channel throws \`UnknownChannel\` rather than silently routing to Notification Bot.
@@ -58773,7 +58778,7 @@ var COMMY_INSTRUCTIONS = `**Substrate.** commy is the inter-agent channel: agent
 
 **Links.** Every ref the substrate hands you carries a ready-to-click \`permalink\` — on \`post\` results, \`read_channel\`/\`read_thread\` messages (message \`permalink\` plus \`channel.permalink\` and \`thread.permalink\`), \`list_channels\` (channel \`permalink\`), and inbound \`<channel source="commy">\` frames (\`permalink\` / \`channel_permalink\` / \`thread_permalink\` meta, \`target_permalink\` on reaction frames). **Whenever you show a human a message, channel, or topic reference, render it as that clickable permalink — never a bare name or numeric id.** A human can click a permalink straight to the message; a bare \`#channel > topic\` or message number makes them hunt. When you hold only a message id (e.g. one cited elsewhere, with no permalink to hand), \`message_link(message_id, channel_name?, thread?)\` returns its \`{permalink}\`.
 
-**Tools.** \`post\` (channel; optionally thread or reply), \`react\`/\`unreact\` (emoji on a message), \`subscribe\`/\`unsubscribe\` (live target), \`read_channel\`/\`read_thread\` (history within a range), \`list_channels\` (enumerate channels in the realm), \`message_link\` (canonical permalink for a message id), \`resolve\` (identity by name), \`current_identity\` (passive — never acquires), \`download_file\` (fetch a \`/user_uploads/...\` attachment to a temp file — use Read on the returned path to view images), \`upload_file\` (upload a local file by absolute path; returns a \`reference\` string to embed in a \`post\` body).
+**Tools.** \`post\` (channel; optionally thread or reply), \`react\`/\`unreact\` (emoji on a message), \`subscribe\`/\`unsubscribe\` (live target), \`read_channel\`/\`read_thread\` (history within a range), \`list_channels\` (enumerate channels in the realm), \`message_link\` (canonical permalink for a message id), \`resolve\` (identity by name), \`current_identity\` (passive — never acquires), \`download_file\` (fetch a \`/user_uploads/...\` attachment to a temp file — rooted under the operator-set \`COMMY_DOWNLOAD_DIR\` when configured so it lands somewhere you can Read, else \`$TMPDIR\`; use Read on the returned path to view images), \`upload_file\` (upload a local file by absolute path; returns a \`reference\` string to embed in a \`post\` body).
 
 **session_id.** Pass your conversation's session id as the optional argument on \`post\`, \`edit_message\`, \`react\`, \`unreact\`, and \`current_identity\`. **Must be a UUID** (e.g. \`crypto.randomUUID()\`); anything else is rejected as malformed and the call routes to the "missing session_id" error rather than silently minting a \`cc-<garbage>\` identity. In Claude Code the plugin's PreToolUse hook injects the harness session id automatically. The server uses it to derive the ephemeral \`cc-<8>\` bot identity for this conversation and to detect transitions between conversations.`;
 var buildMcpServer = () => new Server({ name: "commy", version: PLUGIN_VERSION }, {
@@ -59602,7 +59607,7 @@ var buildToolDefs = (deps, cache) => {
     const download = deps.downloadFile;
     optionalTools.push({
       name: "download_file",
-      description: "Download a Zulip user upload by its /user_uploads/... path (visible in message bodies). Writes the file to a temp path and returns {file_path, content_type, size}. Use the Read tool on the returned file_path to view images.",
+      description: "Download a Zulip user upload by its /user_uploads/... path (visible in message bodies). Writes the file into a fresh temp directory and returns {file_path, content_type, size}. The temp directory is created under the operator-set COMMY_DOWNLOAD_DIR when configured (so the file lands in a directory you can Read), otherwise under $TMPDIR. Use the Read tool on the returned file_path to view images.",
       inputSchema: {
         type: "object",
         properties: {
@@ -59745,6 +59750,15 @@ var makeProgram = (params = {}) => exports_Effect.scoped(exports_Effect.gen(func
   }));
   const feedSessionId = (sessionId) => exports_Deferred.succeed(sessionIdDeferred, sessionId).pipe(exports_Effect.asVoid);
   const fs = yield* exports_FileSystem.FileSystem;
+  if (parsed.downloadDir !== undefined) {
+    const base = parsed.downloadDir;
+    const baseStat = yield* fs.stat(base).pipe(exports_Effect.option);
+    if (exports_Option.isNone(baseStat) || baseStat.value.type !== "Directory") {
+      return yield* envConfigError([
+        `COMMY_DOWNLOAD_DIR is set to ${base} but is not an existing directory`
+      ]);
+    }
+  }
   const loggerLayer = params.loggerLayer ?? stderrLoggerLayer;
   const commandExecutor = yield* exports_CommandExecutor.CommandExecutor;
   const readGitContext2 = params.readGitContext ?? ((cwd) => readGitContext(cwd).pipe(exports_Effect.provideService(exports_CommandExecutor.CommandExecutor, commandExecutor)));
@@ -59813,7 +59827,7 @@ var makeProgram = (params = {}) => exports_Effect.scoped(exports_Effect.gen(func
     feedSessionId,
     downloadFile: (urlPath) => exports_Effect.gen(function* () {
       const result = yield* adapter4.downloadFile(urlPath);
-      const dir = yield* fs.makeTempDirectory({ prefix: "commy-dl-" });
+      const dir = yield* fs.makeTempDirectory(parsed.downloadDir === undefined ? { prefix: "commy-dl-" } : { directory: parsed.downloadDir, prefix: "commy-dl-" });
       const filePath = join10(dir, basename2(urlPath));
       yield* fs.writeFile(filePath, result.data);
       return { filePath, contentType: result.contentType, size: result.data.byteLength };