peter/devclaw-gitea
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: block sessions_send in addition to sessions_spawn (#19) (#20)

Browse Source 
Workers shouldn't spawn sub-agents or message other sessions directly.
All coordination should go through DevClaw tools (task_pickup, task_complete, etc.).
This commit is contained in:
Lauren ten Hoor
2026-02-10 00:54:36 +08:00
committed by GitHub
parent aae6c2ee78
commit 6ec01537a6
1 changed files with 6 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
lib/setup.ts
View File

@@ -279,7 +279,8 @@ async function resolveWorkspacePath(agentId: string): Promise<string> {
/** /**
* Write DevClaw model tier config and devClawAgentIds to openclaw.json plugins section. * Write DevClaw model tier config and devClawAgentIds to openclaw.json plugins section.
* Also adds tool restrictions (deny sessions_spawn) to DevClaw agents. * Also adds tool restrictions (deny sessions_spawn, sessions_send) to DevClaw agents.
* This prevents workers from spawning sub-agents or messaging other sessions directly.
* Configures subagent cleanup interval to keep development sessions alive. * Configures subagent cleanup interval to keep development sessions alive.
* Read-modify-write to preserve existing config. * Read-modify-write to preserve existing config.
*/ */
@@ -325,13 +326,15 @@ async function writePluginConfig(
config.plugins.entries.devclaw.config.devClawAgentIds = [...existing, agentId]; config.plugins.entries.devclaw.config.devClawAgentIds = [...existing, agentId];
} }
// Add tool restrictions (deny sessions_spawn) to the agent // Add tool restrictions to the agent
// Workers shouldn't spawn sub-agents or message other sessions directly
// All coordination should go through DevClaw tools (task_pickup, task_complete, etc.)
const agent = config.agents?.list?.find((a: { id: string }) => a.id === agentId); const agent = config.agents?.list?.find((a: { id: string }) => a.id === agentId);
if (agent) { if (agent) {
if (!agent.tools) { if (!agent.tools) {
agent.tools = {}; agent.tools = {};
} }
agent.tools.deny = ["sessions_spawn"]; agent.tools.deny = ["sessions_spawn", "sessions_send"];
// Clear any conflicting allow list // Clear any conflicting allow list
delete agent.tools.allow; delete agent.tools.allow;
} }