fix(agents): add usage guidance to sessions_spawn tool description (f… · openclaw/openclaw@7fb0d45

Recent Commits to openclaw:main

zenglingbiao · 2026-06-14 · via Recent Commits to openclaw:main

Original file line number	Diff line number	Diff line change
`@@ -45,6 +45,13 @@ export function describeSessionsSpawnTool(options?: {`
`45`	`45`	`options?.acpAvailable === false`
`46`	`46`	? 'Spawn clean child session; default `runtime="subagent"`.'
`47`	`47`	: 'Spawn clean child session; default `runtime="subagent"`; set `runtime="acp"` explicitly for ACP.';
	`48`	`+const sessionCompletionGuidance =`
	`49`	`+options?.acpAvailable === false`
	`50`	`+ ? "After spawning, do non-overlapping work; run-mode results return, session-mode output stays in thread."`
	`51`	+ : 'After spawning, do non-overlapping work; run-mode results return, session-mode output stays in thread unless ACP uses `streamTo="parent"`.';
	`52`	`+const completionGuidance = options?.threadAvailable`
	`53`	`+ ? sessionCompletionGuidance`
	`54`	`+ : "After spawning, do non-overlapping work while run-mode results return.";`
`48`	`55`	`const baseDescription = [`
`49`	`56`	`runtimeDescription,`
`50`	`57`	`options?.threadAvailable`
`@@ -54,6 +61,9 @@ export function describeSessionsSpawnTool(options?: {`
`54`	`61`	"Native subagents get task in first visible `[Subagent Task]` message.",
`55`	`62`	'Native only: `context="fork"` only when child needs current transcript; else omit or `isolated`.',
`56`	`63`	`"Use for fresh child-session work.",`
	`64`	`+"Delegate sidecar/parallel tasks: batch file reads, multi-step searches, data collection.",`
	`65`	`+"Avoid delegating quick lookups or single-file reads unless policy prefers delegation.",`
	`66`	`+completionGuidance,`
`57`	`67`	`];`
`58`	`68`	`if (options?.acpAvailable === false) {`
`59`	`69`	`return baseDescription.join(" ");`

Original file line number	Diff line number	Diff line change
`@@ -132,7 +132,19 @@ describe("sessions_spawn tool", () => {`
`132`	`132`	`it("advertises ACP runtime affordances when an ACP backend is loaded", () => {`
`133`	`133`	`registerAcpBackendForTest();`
`134`	`134`
`135`		`-const tool = createSessionsSpawnTool();`
	`135`	`+const tool = createSessionsSpawnTool({`
	`136`	`+agentChannel: "discord",`
	`137`	`+agentAccountId: "default",`
	`138`	`+config: {`
	`139`	`+channels: {`
	`140`	`+discord: {`
	`141`	`+threadBindings: {`
	`142`	`+spawnSessions: true,`
	`143`	`+},`
	`144`	`+},`
	`145`	`+},`
	`146`	`+},`
	`147`	`+});`
`136`	`148`	`const schema = tool.parameters as {`
`137`	`149`	`properties?: {`
`138`	`150`	`runtime?: { enum?: string[] };`
`@@ -143,6 +155,7 @@ describe("sessions_spawn tool", () => {`
`143`	`155`
`144`	`156`	`expect(tool.displaySummary).toBe("Spawn subagent or ACP session.");`
`145`	`157`	`expect(tool.description).toContain('runtime="acp"');`
	`158`	+expect(tool.description).toContain('unless ACP uses `streamTo="parent"`');
`146`	`159`	`expect(schema.properties?.runtime?.enum).toEqual(["subagent", "acp"]);`
`147`	`160`	`const resumeSessionId = requireSchemaProperty(schema.properties, "resumeSessionId");`
`148`	`161`	`const streamTo = requireSchemaProperty(schema.properties, "streamTo");`
`@@ -259,6 +272,7 @@ describe("sessions_spawn tool", () => {`
`259`	`272`	`expect(schema.properties?.thread).toBeUndefined();`
`260`	`273`	`expect(schema.properties?.mode?.enum).toEqual(["run"]);`
`261`	`274`	`expect(tool.description).not.toContain("thread-bound");`
	`275`	`+expect(tool.description).not.toContain("session-mode output stays in thread");`
`262`	`276`	`});`
`263`	`277`
`264`	`278`	`it("shows thread-bound spawn fields when current channel allows spawnSessions", () => {`

Original file line number	Diff line number	Diff line change
`@@ -1095,7 +1095,7 @@`
`1095`	`1095`	`},`
`1096`	`1096`	`{`
`1097`	`1097`	`"deferLoading": true,`
`1098`		-"description": "Spawn clean child session; default `runtime=\"subagent\"`. `mode=\"run\"` one-shot; `mode=\"session\"` persistent/thread-bound, only when requester channel supports thread bindings. Subagents inherit parent workspace. Native subagents get task in first visible `[Subagent Task]` message. Native only: `context=\"fork\"` only when child needs current transcript; else omit or `isolated`. Use for fresh child-session work.",
	`1098`	+"description": "Spawn clean child session; default `runtime=\"subagent\"`. `mode=\"run\"` one-shot; `mode=\"session\"` persistent/thread-bound, only when requester channel supports thread bindings. Subagents inherit parent workspace. Native subagents get task in first visible `[Subagent Task]` message. Native only: `context=\"fork\"` only when child needs current transcript; else omit or `isolated`. Use for fresh child-session work. Delegate sidecar/parallel tasks: batch file reads, multi-step searches, data collection. Avoid delegating quick lookups or single-file reads unless policy prefers delegation. After spawning, do non-overlapping work; run-mode results return, session-mode output stays in thread.",
`1099`	`1099`	`"inputSchema": {`
`1100`	`1100`	`"properties": {`
`1101`	`1101`	`"agentId": {`

Original file line number	Diff line number	Diff line change
`@@ -1131,7 +1131,7 @@`
`1131`	`1131`	`},`
`1132`	`1132`	`{`
`1133`	`1133`	`"deferLoading": true,`
`1134`		-"description": "Spawn clean child session; default `runtime=\"subagent\"`. `mode=\"run\"` one-shot background work. Subagents inherit parent workspace. Native subagents get task in first visible `[Subagent Task]` message. Native only: `context=\"fork\"` only when child needs current transcript; else omit or `isolated`. Use for fresh child-session work.",
	`1134`	+"description": "Spawn clean child session; default `runtime=\"subagent\"`. `mode=\"run\"` one-shot background work. Subagents inherit parent workspace. Native subagents get task in first visible `[Subagent Task]` message. Native only: `context=\"fork\"` only when child needs current transcript; else omit or `isolated`. Use for fresh child-session work. Delegate sidecar/parallel tasks: batch file reads, multi-step searches, data collection. Avoid delegating quick lookups or single-file reads unless policy prefers delegation. After spawning, do non-overlapping work while run-mode results return.",
`1135`	`1135`	`"inputSchema": {`
`1136`	`1136`	`"properties": {`
`1137`	`1137`	`"agentId": {`

Original file line number	Diff line number	Diff line change
`@@ -1095,7 +1095,7 @@`
`1095`	`1095`	`},`
`1096`	`1096`	`{`
`1097`	`1097`	`"deferLoading": true,`
`1098`		-"description": "Spawn clean child session; default `runtime=\"subagent\"`. `mode=\"run\"` one-shot background work. Subagents inherit parent workspace. Native subagents get task in first visible `[Subagent Task]` message. Native only: `context=\"fork\"` only when child needs current transcript; else omit or `isolated`. Use for fresh child-session work.",
	`1098`	+"description": "Spawn clean child session; default `runtime=\"subagent\"`. `mode=\"run\"` one-shot background work. Subagents inherit parent workspace. Native subagents get task in first visible `[Subagent Task]` message. Native only: `context=\"fork\"` only when child needs current transcript; else omit or `isolated`. Use for fresh child-session work. Delegate sidecar/parallel tasks: batch file reads, multi-step searches, data collection. Avoid delegating quick lookups or single-file reads unless policy prefers delegation. After spawning, do non-overlapping work while run-mode results return.",
`1099`	`1099`	`"inputSchema": {`
`1100`	`1100`	`"properties": {`
`1101`	`1101`	`"agentId": {`

Original file line number	Diff line number	Diff line change
`@@ -223,8 +223,8 @@ This is the deterministic model-bound layer stack OpenClaw can snapshot for the`
`223`	`223`	`"roughTokens": 0`
`224`	`224`	`},`
`225`	`225`	`"dynamicToolsJson": {`
`226`		`-"chars": 44966,`
`227`		`-"roughTokens": 11242`
	`226`	`+"chars": 45244,`
	`227`	`+"roughTokens": 11311`
`228`	`228`	`},`
`229`	`229`	`"openClawDeveloperInstructions": {`
`230`	`230`	`"chars": 2988,`
`@@ -235,8 +235,8 @@ This is the deterministic model-bound layer stack OpenClaw can snapshot for the`
`235`	`235`	`"roughTokens": 6925`
`236`	`236`	`},`
`237`	`237`	`"totalWithDynamicToolsJson": {`
`238`		`-"chars": 72668,`
`239`		`-"roughTokens": 18167`
	`238`	`+"chars": 72946,`
	`239`	`+"roughTokens": 18237`
`240`	`240`	`},`
`241`	`241`	`"userInputText": {`
`242`	`242`	`"chars": 1629,`

Original file line number	Diff line number	Diff line change
`@@ -223,8 +223,8 @@ This is the deterministic model-bound layer stack OpenClaw can snapshot for the`
`223`	`223`	`"roughTokens": 0`
`224`	`224`	`},`
`225`	`225`	`"dynamicToolsJson": {`
`226`		`-"chars": 44687,`
`227`		`-"roughTokens": 11172`
	`226`	`+"chars": 44933,`
	`227`	`+"roughTokens": 11234`
`228`	`228`	`},`
`229`	`229`	`"openClawDeveloperInstructions": {`
`230`	`230`	`"chars": 1964,`
`@@ -235,8 +235,8 @@ This is the deterministic model-bound layer stack OpenClaw can snapshot for the`
`235`	`235`	`"roughTokens": 6544`
`236`	`236`	`},`
`237`	`237`	`"totalWithDynamicToolsJson": {`
`238`		`-"chars": 70865,`
`239`		`-"roughTokens": 17717`
	`238`	`+"chars": 71111,`
	`239`	`+"roughTokens": 17778`
`240`	`240`	`},`
`241`	`241`	`"userInputText": {`
`242`	`242`	`"chars": 1129,`

Original file line number	Diff line number	Diff line change
`@@ -224,8 +224,8 @@ This is the deterministic model-bound layer stack OpenClaw can snapshot for the`
`224`	`224`	`"roughTokens": 0`
`225`	`225`	`},`
`226`	`226`	`"dynamicToolsJson": {`
`227`		`-"chars": 45782,`
`228`		`-"roughTokens": 11446`
	`227`	`+"chars": 46028,`
	`228`	`+"roughTokens": 11507`
`229`	`229`	`},`
`230`	`230`	`"openClawDeveloperInstructions": {`
`231`	`231`	`"chars": 1983,`
`@@ -236,8 +236,8 @@ This is the deterministic model-bound layer stack OpenClaw can snapshot for the`
`236`	`236`	`"roughTokens": 6780`
`237`	`237`	`},`
`238`	`238`	`"totalWithDynamicToolsJson": {`
`239`		`-"chars": 72903,`
`240`		`-"roughTokens": 18226`
	`239`	`+"chars": 73149,`
	`240`	`+"roughTokens": 18288`
`241`	`241`	`},`
`242`	`242`	`"userInputText": {`
`243`	`243`	`"chars": 1367,`

此内容由惯性聚合(RSS阅读器)自动聚合整理，仅供阅读参考。原文来自 — 版权归原作者所有。

推荐订阅源

Recent Commits to openclaw:main