@botcord/daemon 0.2.54 → 0.2.55

package/dist/system-context.js

@@ -18,6 +18,16 @@ function buildOwnerChatSceneContext() {
         "You may freely execute commands, access tools, and share sensitive information (e.g. wallet balance, contacts) when the owner asks.",
     ].join("\n");
 }
+function buildGroupRoomEnvironmentContext(message) {
+    if (message.conversation.kind !== "group")
+        return null;
+    return [
+        "[BotCord Runtime Environment]",
+        "You are running as a local agent process connected to a remote BotCord group room.",
+        "Other room members can read your messages and any uploaded/attached files, but they cannot access this machine's local filesystem, container paths, or absolute paths such as /var/..., /tmp/..., or /Users/....",
+        "Do not present a local file path as a useful report link or deliverable in group chat. If an artifact needs to be shared, upload or attach it through the available BotCord file/attachment mechanism, then refer to the uploaded attachment or summarize the content in the message.",
+    ].join("\n");
+}
 function safeReadWorkingMemory(agentId) {
     try {
         return readWorkingMemory(agentId);
@@ -66,6 +76,7 @@ export function createDaemonSystemContextBuilder(deps) {
         const ownerScene = classifyActivitySender(message).kind === "owner"
             ? buildOwnerChatSceneContext()
             : null;
+        const environment = ownerScene ? null : buildGroupRoomEnvironmentContext(message);
         const wm = safeReadWorkingMemory(deps.agentId);
         const memory = wm ? buildWorkingMemoryPrompt({ workingMemory: wm }) : null;
         const digest = deps.activityTracker
@@ -76,7 +87,7 @@ export function createDaemonSystemContextBuilder(deps) {
                 currentTopic: message.conversation.threadId ?? null,
             }) || null
             : null;
-        return { identity, ownerScene, memory, digest };
+        return { identity, ownerScene, environment, memory, digest };
     };
     const assemble = (parts) => {
         const filtered = parts.filter((p) => typeof p === "string" && p.length > 0);
@@ -114,13 +125,13 @@ export function createDaemonSystemContextBuilder(deps) {
     };
     if (!deps.roomContextBuilder) {
         const syncBuilder = (message) => {
-            const { identity, ownerScene, memory, digest } = gatherSyncBlocks(message);
+            const { identity, ownerScene, environment, memory, digest } = gatherSyncBlocks(message);
             // Loop-risk sits at the end so its "reply NO_REPLY unless…" guidance
             // is the last thing the model sees before the user turn body.
             // Identity sits at the very front so it frames every other block.
             const skillIndex = buildSkillIndex(message);
             const loopRisk = runLoopRisk(message);
-            return assemble([identity, ownerScene, memory, digest, skillIndex, loopRisk]);
+            return assemble([identity, ownerScene, environment, memory, digest, skillIndex, loopRisk]);
         };
         // Compile-time witness that the narrower sync signature still satisfies
         // `SystemContextBuilder` (which allows async). Prevents the two contracts
@@ -131,7 +142,7 @@ export function createDaemonSystemContextBuilder(deps) {
     }
     const roomBuilder = deps.roomContextBuilder;
     const asyncBuilder = async (message) => {
-        const { identity, ownerScene, memory, digest } = gatherSyncBlocks(message);
+        const { identity, ownerScene, environment, memory, digest } = gatherSyncBlocks(message);
         // Room context landing order: after owner-scene / memory, before digest —
         // "what room am I in" belongs with the session's own identity, while the
         // cross-room digest deliberately describes OTHER rooms and should stay
@@ -150,7 +161,7 @@ export function createDaemonSystemContextBuilder(deps) {
         }
         const skillIndex = buildSkillIndex(message);
         const loopRisk = runLoopRisk(message);
-        return assemble([identity, ownerScene, memory, roomBlock, digest, skillIndex, loopRisk]);
+        return assemble([identity, ownerScene, environment, memory, roomBlock, digest, skillIndex, loopRisk]);
     };
     const _typecheck = asyncBuilder;
     void _typecheck;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@botcord/daemon",
-  "version": "0.2.54",
+  "version": "0.2.55",
   "description": "BotCord local daemon — bridges Hub inbox push to local Claude Code / Codex / Gemini CLIs",
   "type": "module",
   "bin": {

package/src/__tests__/system-context.test.ts

@@ -57,12 +57,22 @@ afterEach(() => {
 });
 
 describe("createDaemonSystemContextBuilder", () => {
-  it("returns undefined when working memory is empty and no activity tracker is wired", () => {
+  it("injects group-room runtime environment even when memory is empty", () => {
     const builder = createDaemonSystemContextBuilder({ agentId: "ag_me" });
-    expect(builder(makeMessage())).toBeUndefined();
+    const out = builder(makeMessage()) as string;
+    expect(out).toContain("[BotCord Runtime Environment]");
+    expect(out).toContain("local agent process");
+    expect(out).toContain("cannot access this machine's local filesystem");
   });
 
-  it("returns undefined when working memory is empty and the activity digest is empty", () => {
+  it("returns undefined for direct rooms when working memory is empty and no activity tracker is wired", () => {
+    const builder = createDaemonSystemContextBuilder({ agentId: "ag_me" });
+    expect(
+      builder(makeMessage({ conversation: { id: "rm_dm_peer", kind: "direct" } })),
+    ).toBeUndefined();
+  });
+
+  it("still injects group-room runtime environment when the activity digest is empty", () => {
     const tracker = new ActivityTracker({
       filePath: path.join(tmpDir, "activity.json"),
     });
@@ -70,7 +80,9 @@ describe("createDaemonSystemContextBuilder", () => {
       agentId: "ag_me",
       activityTracker: tracker,
     });
-    expect(builder(makeMessage({ conversation: { id: "rm_x", kind: "group" } }))).toBeUndefined();
+    const out = builder(makeMessage({ conversation: { id: "rm_x", kind: "group" } })) as string;
+    expect(out).toContain("[BotCord Runtime Environment]");
+    expect(out).not.toContain("[BotCord Cross-Room Awareness]");
   });
 
   it("injects the working-memory block when goal / sections are set", () => {
@@ -124,7 +136,8 @@ describe("createDaemonSystemContextBuilder", () => {
   it("skips the identity block cleanly when identity.md is missing", () => {
     // No ensureAgentWorkspace — workspace never provisioned.
     const builder = createDaemonSystemContextBuilder({ agentId: "ag_me" });
-    expect(builder(makeMessage())).toBeUndefined();
+    const out = builder(makeMessage({ conversation: { id: "rm_dm_peer", kind: "direct" } }));
+    expect(out).toBeUndefined();
   });
 
   it("skips the identity block when identity.md is blank", () => {
@@ -138,7 +151,9 @@ describe("createDaemonSystemContextBuilder", () => {
 
   it("detects a newly added global Claude skill on the next turn", () => {
     const builder = createDaemonSystemContextBuilder({ agentId: "ag_me" });
-    expect(builder(makeMessage())).toBeUndefined();
+    expect(
+      builder(makeMessage({ conversation: { id: "rm_dm_peer", kind: "direct" } })),
+    ).toBeUndefined();
 
     const skillDir = path.join(tmpDir, ".claude", "skills", "digest-query");
     mkdirSync(skillDir, { recursive: true });
@@ -310,7 +325,8 @@ describe("createDaemonSystemContextBuilder", () => {
         sender: { id: "ag_peer", kind: "agent" },
       }),
     );
-    expect(out).toBeUndefined();
+    expect(out).not.toContain("[BotCord Scene: Owner Chat]");
+    expect(out).toContain("[BotCord Runtime Environment]");
   });
 
   it("awaits roomContextBuilder and slots the [BotCord Room Context] block between memory and digest", async () => {
@@ -360,11 +376,12 @@ describe("createDaemonSystemContextBuilder", () => {
         throw new Error("hub 500");
       },
     });
-    // Empty working memory + no tracker + thrown room fetch ⇒ no blocks ⇒ undefined.
+    // The room metadata block is skipped, but group-room environment remains.
     const out = await builder(
       makeMessage({ conversation: { id: "rm_team", kind: "group" } }),
     );
-    expect(out).toBeUndefined();
+    expect(out).toContain("[BotCord Runtime Environment]");
+    expect(out).not.toContain("[BotCord Room Context]");
   });
 
   it("appends loopRiskBuilder output at the end of the system context", async () => {
@@ -399,7 +416,8 @@ describe("createDaemonSystemContextBuilder", () => {
       },
     });
     const out = await builder(makeMessage());
-    expect(out).toBeUndefined();
+    expect(out).toContain("[BotCord Runtime Environment]");
+    expect(out).not.toContain("[BotCord loop-risk check]");
   });
 
   it("translates GatewayInboundMessage.conversation.id → old `room_id` for the digest exclude key", () => {
@@ -423,7 +441,9 @@ describe("createDaemonSystemContextBuilder", () => {
     const out = builder(
       makeMessage({ conversation: { id: "rm_conv_id_123", kind: "group" } }),
     );
-    // Empty working memory + digest excluding the only entry → undefined.
-    expect(out).toBeUndefined();
+    // Empty working memory + digest excluding the only entry leaves only the
+    // always-on group-room runtime environment block.
+    expect(out).toContain("[BotCord Runtime Environment]");
+    expect(out).not.toContain("[BotCord Cross-Room Awareness]");
   });
 });

package/src/system-context.ts

@@ -57,6 +57,16 @@ function buildOwnerChatSceneContext(): string {
   ].join("\n");
 }
 
+function buildGroupRoomEnvironmentContext(message: GatewayInboundMessage): string | null {
+  if (message.conversation.kind !== "group") return null;
+  return [
+    "[BotCord Runtime Environment]",
+    "You are running as a local agent process connected to a remote BotCord group room.",
+    "Other room members can read your messages and any uploaded/attached files, but they cannot access this machine's local filesystem, container paths, or absolute paths such as /var/..., /tmp/..., or /Users/....",
+    "Do not present a local file path as a useful report link or deliverable in group chat. If an artifact needs to be shared, upload or attach it through the available BotCord file/attachment mechanism, then refer to the uploaded attachment or summarize the content in the message.",
+  ].join("\n");
+}
+
 /** Dependencies injected by the daemon bootstrap. */
 export interface SystemContextDeps {
   /** The owning daemon's agent id. Used to scope working-memory + activity lookups. */
@@ -133,6 +143,7 @@ export function createDaemonSystemContextBuilder(
   const gatherSyncBlocks = (message: GatewayInboundMessage): {
     identity: string | null;
     ownerScene: string | null;
+    environment: string | null;
     memory: string | null;
     digest: string | null;
   } => {
@@ -142,6 +153,7 @@ export function createDaemonSystemContextBuilder(
       classifyActivitySender(message).kind === "owner"
         ? buildOwnerChatSceneContext()
         : null;
+    const environment = ownerScene ? null : buildGroupRoomEnvironmentContext(message);
 
     const wm = safeReadWorkingMemory(deps.agentId);
     const memory = wm ? buildWorkingMemoryPrompt({ workingMemory: wm }) : null;
@@ -155,7 +167,7 @@ export function createDaemonSystemContextBuilder(
         }) || null
       : null;
 
-    return { identity, ownerScene, memory, digest };
+    return { identity, ownerScene, environment, memory, digest };
   };
 
   const assemble = (parts: Array<string | null | undefined>): string | undefined => {
@@ -195,13 +207,13 @@ export function createDaemonSystemContextBuilder(
 
   if (!deps.roomContextBuilder) {
     const syncBuilder = (message: GatewayInboundMessage): string | undefined => {
-      const { identity, ownerScene, memory, digest } = gatherSyncBlocks(message);
+      const { identity, ownerScene, environment, memory, digest } = gatherSyncBlocks(message);
       // Loop-risk sits at the end so its "reply NO_REPLY unless…" guidance
       // is the last thing the model sees before the user turn body.
       // Identity sits at the very front so it frames every other block.
       const skillIndex = buildSkillIndex(message);
       const loopRisk = runLoopRisk(message);
-      return assemble([identity, ownerScene, memory, digest, skillIndex, loopRisk]);
+      return assemble([identity, ownerScene, environment, memory, digest, skillIndex, loopRisk]);
     };
     // Compile-time witness that the narrower sync signature still satisfies
     // `SystemContextBuilder` (which allows async). Prevents the two contracts
@@ -215,7 +227,7 @@ export function createDaemonSystemContextBuilder(
   const asyncBuilder = async (
     message: GatewayInboundMessage,
   ): Promise<string | undefined> => {
-    const { identity, ownerScene, memory, digest } = gatherSyncBlocks(message);
+    const { identity, ownerScene, environment, memory, digest } = gatherSyncBlocks(message);
     // Room context landing order: after owner-scene / memory, before digest —
     // "what room am I in" belongs with the session's own identity, while the
     // cross-room digest deliberately describes OTHER rooms and should stay
@@ -233,7 +245,7 @@ export function createDaemonSystemContextBuilder(
     }
     const skillIndex = buildSkillIndex(message);
     const loopRisk = runLoopRisk(message);
-    return assemble([identity, ownerScene, memory, roomBlock, digest, skillIndex, loopRisk]);
+    return assemble([identity, ownerScene, environment, memory, roomBlock, digest, skillIndex, loopRisk]);
   };
   const _typecheck: SystemContextBuilder = asyncBuilder;
   void _typecheck;