@codeforbreakfast/commy-mcp 0.24.0 → 0.24.1

package/package.json

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

package/server.js

@@ -57343,13 +57343,16 @@ var zulipAdapter = (config3) => exports_Effect.gen(function* () {
     const byId = new Map;
     const byName = new Map;
     const byIdentityId = new Map;
+    const active2 = new Set;
     for (const u of members) {
       const ident = yield* toIdentity(u);
       byId.set(ZulipUserRef(u.user_id), ident);
       byName.set(u.full_name, ident);
       byIdentityId.set(ident.id, ZulipUserRef(u.user_id));
+      if (u.is_active)
+        active2.add(ident.id);
     }
-    return { byId, byName, byIdentityId };
+    return { byId, byName, byIdentityId, active: active2 };
   })));
   const recipientDirectory = () => buildDirectoryLookup().pipe(exports_Effect.map((lookup) => ({
     byId: new Map([...lookup.byId.entries()].map(([id3, ident]) => [id3, { kind: ident.kind }]))
@@ -57358,8 +57361,18 @@ var zulipAdapter = (config3) => exports_Effect.gen(function* () {
     byName: directory2.byName,
     byUserId: (userId) => directory2.byId.get(ZulipUserRef(userId))
   });
+  const notifiableMentionDirectory = (directory2) => {
+    const isActive2 = (identity7) => identity7 !== undefined && directory2.active.has(identity7.id);
+    return {
+      byName: new Map([...directory2.byName].filter(([, identity7]) => isActive2(identity7))),
+      byUserId: (userId) => {
+        const identity7 = directory2.byId.get(ZulipUserRef(userId));
+        return isActive2(identity7) ? identity7 : undefined;
+      }
+    };
+  };
   const validateOutboundMentions = (operation, body) => !mentionTokens(body).some(MentionToken.$is("UserToken")) ? exports_Effect.void : buildDirectoryLookup().pipe(exports_Effect.flatMap((directory2) => {
-    const dead = unresolvedMentions(body, mentionDirectory(directory2));
+    const dead = unresolvedMentions(body, notifiableMentionDirectory(directory2));
     return dead.length === 0 ? exports_Effect.void : exports_Effect.fail(new UnresolvedMention({ operation, tokens: dead, substrate: "zulip" }));
   }));
   const mapHistoricalReactions = (raw3, directory2) => exports_Effect.gen(function* () {
@@ -58359,7 +58372,7 @@ var catchUpChannels = (deps) => exports_Effect.gen(function* () {
   const since = yield* clampedSince(now2, deps.windowSeconds);
   const newTopicsSince = yield* clampedSince(now2, deps.newTopicsWindowSeconds ?? deps.windowSeconds);
   const batches = yield* exports_Effect.forEach(deps.intents, (intent) => fetchForIntent(intent, deps.history, since, newTopicsSince).pipe(catchUpAt("history")), { concurrency: 2 });
-  const messages = exports_Array.sort(exports_Array.flatten(batches), exports_Order.mapInput(exports_Order.number, (m) => m.ts));
+  const messages = exports_Array.sort(exports_Array.filter(exports_Array.flatten(batches), (m) => m.sender.id !== deps.botIdentityId), exports_Order.mapInput(exports_Order.number, (m) => m.ts));
   yield* exports_Effect.forEach(messages, (message) => exports_Effect.tryPromise({
     try: () => deps.notifier(formatMessage({ kind: "message-posted", message }, deps.botIdentityId)),
     catch: (cause3) => new CatchUpError({ stage: "dispatch", cause: cause3 })
@@ -58547,8 +58560,7 @@ var createSingleIdentityCache = (deps) => {
       const current2 = ensureBound.current();
       return current2 === undefined ? new Set : new Set([current2.identity.id]);
     },
-    releaseAllBound: () => exports_Effect.void,
-    sweepIdle: () => exports_Effect.void
+    releaseAllBound: () => exports_Effect.void
   };
 };
 var unboundStub = Object.assign(() => exports_Effect.fail(new UnboundEphemeralSession({
@@ -58563,24 +58575,23 @@ var createEphemeralIdentityCache = (deps) => exports_Effect.map(exports_Synchron
   });
   const afterAcquireFor = (project3, sessionId) => (acquired) => deps.onAcquire !== undefined ? deps.onAcquire(acquired, project3, sessionId) : exports_Effect.void;
   const acquireForTransition = (priorEnsure) => (n) => priorEnsure.current() !== undefined ? deps.release().pipe(exports_Effect.zipRight(deps.acquire(n))) : deps.acquire(n);
-  const mintSlot = (sessionId, project3, nowMs, prior) => {
+  const mintSlot = (sessionId, project3, prior) => {
     const name = deriveBotName(sessionId, project3);
     const acquire = prior !== undefined ? acquireForTransition(prior.ensureBound) : deps.acquire;
     return createEnsureBound({
       acquire,
       name,
       afterAcquire: afterAcquireFor(project3, sessionId)
-    }).pipe(exports_Effect.map((ensureBound) => [ensureBound, { sessionId, ensureBound, lastUsedMs: nowMs }]));
+    }).pipe(exports_Effect.map((ensureBound) => [ensureBound, { sessionId, ensureBound }]));
   };
   const releaseBoundSlot = exports_SynchronizedRef.updateEffect(slotRef, (slot) => exports_Effect.as(slot !== undefined && slot.ensureBound.current() !== undefined ? deps.release() : exports_Effect.void, undefined));
   return {
-    ensureBoundFor: (sessionId, project3) => sessionId === undefined ? exports_Effect.succeed(unboundStub) : exports_Clock.currentTimeMillis.pipe(exports_Effect.flatMap((nowMs) => exports_SynchronizedRef.modifyEffect(slotRef, (slot) => {
+    ensureBoundFor: (sessionId, project3) => sessionId === undefined ? exports_Effect.succeed(unboundStub) : exports_SynchronizedRef.modifyEffect(slotRef, (slot) => {
       if (slot !== undefined && slot.sessionId === sessionId) {
-        const bumped = { ...slot, lastUsedMs: nowMs };
-        return exports_Effect.succeed([slot.ensureBound, bumped]);
+        return exports_Effect.succeed([slot.ensureBound, slot]);
       }
-      return mintSlot(sessionId, project3, nowMs, slot);
-    }))),
+      return mintSlot(sessionId, project3, slot);
+    }),
     boundIdentityIds: () => {
       const slot = exports_Effect.runSync(exports_SynchronizedRef.get(slotRef));
       if (slot === undefined)
@@ -58588,8 +58599,7 @@ var createEphemeralIdentityCache = (deps) => exports_Effect.map(exports_Synchron
       const current2 = slot.ensureBound.current();
       return current2 === undefined ? new Set : new Set([current2.identity.id]);
     },
-    releaseAllBound: () => releaseBoundSlot,
-    sweepIdle: (nowMs) => exports_SynchronizedRef.updateEffect(slotRef, (slot) => slot !== undefined && slot.ensureBound.current() !== undefined && nowMs - slot.lastUsedMs > deps.idleReleaseMs ? exports_Effect.as(deps.release(), undefined) : exports_Effect.succeed(slot))
+    releaseAllBound: () => releaseBoundSlot
   };
 });
 // node_modules/.bun/@modelcontextprotocol+sdk@1.29.0/node_modules/@modelcontextprotocol/sdk/dist/esm/server/zod-compat.js
@@ -60013,7 +60023,7 @@ class Server extends Protocol {
 }
 
 // packages/mcp/mcp-server.ts
-var PLUGIN_VERSION = "0.24.0";
+var PLUGIN_VERSION = "0.24.1";
 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.
@@ -61020,12 +61030,6 @@ var registerTools = (server, deps) => {
 class BootError extends exports_Data.TaggedError("BootError") {
 }
 var RELEASE_TIMEOUT_MS = 5000;
-var EPHEMERAL_IDLE_RELEASE_MS = 60 * 60 * 1000;
-var EPHEMERAL_IDLE_SWEEP_INTERVAL_MS = 5 * 60 * 1000;
-var forkIdleSweep = (cache, intervalMs) => {
-  const sweep = exports_Clock.currentTimeMillis.pipe(exports_Effect.flatMap((nowMs) => cache.sweepIdle(nowMs)));
-  return exports_Effect.asVoid(exports_Effect.forkScoped(exports_Effect.repeat(sweep, exports_Schedule.spaced(exports_Duration.millis(intervalMs)))));
-};
 var createType2DefaultsOnAcquire = (narrowSet, inbox) => {
   const registerIntent = (intent) => exports_Effect.sync(() => narrowSet.add(intent)).pipe(exports_Effect.zipRight(inbox.subscribe(intentToTarget(intent))));
   return (project3) => (project3 !== undefined ? exports_Effect.all([decodeChannelName(project3), decodeThreadName("general")]).pipe(exports_Effect.flatMap(([channelName, threadName3]) => registerIntent({ kind: "thread", channelName, threadName: threadName3 }))) : exports_Effect.void).pipe(exports_Effect.catchAll((err) => exports_Effect.logError(`commy plugin: Type-2 default narrow registration failed: ${exports_Cause.pretty(exports_Cause.fail(err))}`)));
@@ -61047,7 +61051,6 @@ var buildIdentityCache = (adapter4, botName, ephemeralOnAcquire) => {
   return createEphemeralIdentityCache({
     acquire: adapter4.identity.acquire,
     release: adapter4.identity.release,
-    idleReleaseMs: EPHEMERAL_IDLE_RELEASE_MS,
     ...ephemeralOnAcquire !== undefined ? { onAcquire: ephemeralOnAcquire } : {}
   });
 };
@@ -61154,7 +61157,6 @@ var makeProgram = (params = {}) => exports_Effect.scoped(exports_Effect.gen(func
   }
   const releasePersistent = parsed.botName !== undefined;
   yield* exports_Effect.addFinalizer(() => identityCache.boundIdentityIds().size > 0 ? raceReleaseAgainstTimeout(adapter4.identity.release({ persistent: releasePersistent }), RELEASE_TIMEOUT_MS) : exports_Effect.void);
-  const runsIdleSweep = parsed.botName === undefined;
   const bootSessionId = yield* exports_Deferred.poll(sessionIdDeferred).pipe(exports_Effect.flatMap(exports_Option.match({
     onNone: () => exports_Effect.succeedNone,
     onSome: (awaitId) => exports_Effect.asSome(awaitId)
@@ -61233,9 +61235,6 @@ var makeProgram = (params = {}) => exports_Effect.scoped(exports_Effect.gen(func
     onRealmSettings: (settings) => exports_Effect.sync(() => toolsCache.setEditingAvailable(settings.editingAvailable)).pipe(exports_Effect.zipRight(exports_Effect.promise(() => mcp.sendToolListChanged()).pipe(exports_Effect.catchAllCause(logSettingsFailure(`tools/list_changed after editing=${settings.editingAvailable}`)))))
   });
   yield* exports_Effect.addFinalizer(() => pump.cancel);
-  if (runsIdleSweep) {
-    yield* forkIdleSweep(identityCache, EPHEMERAL_IDLE_SWEEP_INTERVAL_MS);
-  }
   yield* exports_Effect.forkScoped(rebuildNarrowSet);
   yield* exports_Effect.raceFirst(pump.done, params.shutdownSignal ?? exports_Effect.never);
 }));
@@ -61269,7 +61268,6 @@ if (__require.main == __require.module) {
 export {
   makeProgram,
   main,
-  forkIdleSweep,
   clientDisconnect,
   PlatformLive,
   MainLive,