From fbfc91bcba0cda66da44fef09c9688e04c8b5ffb Mon Sep 17 00:00:00 2001 From: Dudu-0223 Date: Mon, 2 Nov 2026 15:12:56 +0800 Subject: [PATCH] test(cli): follow agent names in profile e2e expectations --- apps/cli/tests/agent-team-headless.e2e.ts | 8 ++++---- apps/cli/tests/web-agent-presets.e2e.ts | 4 ++-- .../subagent-claude-code/tests/loader-composition.e2e.ts | 8 ++++---- .../subagent-codex/tests/loader-composition.e2e.ts | 6 +++--- 4 files changed, 13 insertions(+), 13 deletions(-) diff --git a/apps/cli/tests/agent-team-headless.e2e.ts b/apps/cli/tests/agent-team-headless.e2e.ts index e38a33777d..026a54903c 100644 --- a/apps/cli/tests/agent-team-headless.e2e.ts +++ b/apps/cli/tests/agent-team-headless.e2e.ts @@ -102,11 +102,11 @@ describe('dsh run with Agent Teams enabled', () => { expect(eventTypes).not.toContain('team/member') const catalog = root!.filter(record => record.type === 'subagent/catalog').map(record => record.data) expect(catalog).toEqual(expect.arrayContaining([ - expect.objectContaining({ version: 0, childId: workflowChild![0]?.id }), - expect.objectContaining({ version: 0, label: 'Own deterministic implementation.' }), - expect.objectContaining({ version: 0, label: 'Own deterministic research.' }), + expect.objectContaining({ version: 3, childId: workflowChild![0]?.id, name: 'agent-1' }), + expect.objectContaining({ version: 3, label: 'Own deterministic implementation.', name: 'agent-2' }), + expect.objectContaining({ version: 3, label: 'Own deterministic research.', name: 'agent-3' }), ])) - for (const entry of catalog) expect(entry).not.toHaveProperty('name') + for (const entry of catalog) expect(entry).toHaveProperty('name') expect(eventTypes).not.toContain('subagent-name/bound') expect(eventTypes).not.toContain('team/message/queued') expect(eventTypes).not.toContain('team/message/delivered') diff --git a/apps/cli/tests/web-agent-presets.e2e.ts b/apps/cli/tests/web-agent-presets.e2e.ts index 792187acae..7b21f59fe2 100644 --- a/apps/cli/tests/web-agent-presets.e2e.ts +++ b/apps/cli/tests/web-agent-presets.e2e.ts @@ -345,7 +345,7 @@ describe('the shipped Web composition', () => { 'model', 'provider', 'reasoning_effort', ])) for (const agent of [disabled.agent, enabled.agent]) { - expect(toolParameterNames(ctx, agent, 'subagent_fork')).toEqual(['cwd', 'description', 'prompt']) + expect(toolParameterNames(ctx, agent, 'subagent_fork')).toEqual(['cwd', 'description', 'name', 'prompt']) } } finally { await policy.dispose() @@ -802,7 +802,7 @@ describe.each(['web', 'desktop'] as const)('%s native subagent bundles', (surfac }) try { for (const product of installed) { - expect(toolParameterNames(productCtx, handle.agent, toolName(product))).toEqual(['cwd', 'description', 'prompt']) + expect(toolParameterNames(productCtx, handle.agent, toolName(product))).toEqual(['cwd', 'description', 'name', 'prompt']) } } finally { await handle.dispose() } expect(spawn).not.toHaveBeenCalled() diff --git a/packages/subagent/subagent-claude-code/tests/loader-composition.e2e.ts b/packages/subagent/subagent-claude-code/tests/loader-composition.e2e.ts index 26543fb219..6c3234b2f8 100644 --- a/packages/subagent/subagent-claude-code/tests/loader-composition.e2e.ts +++ b/packages/subagent/subagent-claude-code/tests/loader-composition.e2e.ts @@ -79,22 +79,22 @@ describe('product-provider public Loader composition', () => { tools: [ { name: 'subagent_codex', - parameterNames: ['cwd', 'description', 'prompt'], + parameterNames: ['cwd', 'description', 'name', 'prompt'], required: ['description', 'prompt'], }, { name: 'subagent_claude_code', - parameterNames: ['cwd', 'description', 'prompt'], + parameterNames: ['cwd', 'description', 'name', 'prompt'], required: ['description', 'prompt'], }, { name: 'subagent_claude_primary', - parameterNames: ['cwd', 'description', 'prompt'], + parameterNames: ['cwd', 'description', 'name', 'prompt'], required: ['description', 'prompt'], }, { name: 'subagent_claude_secondary', - parameterNames: ['cwd', 'description', 'prompt'], + parameterNames: ['cwd', 'description', 'name', 'prompt'], required: ['description', 'prompt'], }, ], diff --git a/packages/subagent/subagent-codex/tests/loader-composition.e2e.ts b/packages/subagent/subagent-codex/tests/loader-composition.e2e.ts index a9972381ae..e77ad6cd73 100644 --- a/packages/subagent/subagent-codex/tests/loader-composition.e2e.ts +++ b/packages/subagent/subagent-codex/tests/loader-composition.e2e.ts @@ -69,17 +69,17 @@ describe('Codex provider public Loader composition', () => { tools: [ { name: 'subagent_codex', - parameterNames: ['cwd', 'description', 'prompt'], + parameterNames: ['cwd', 'description', 'name', 'prompt'], required: ['description', 'prompt'], }, { name: 'subagent_codex_primary', - parameterNames: ['cwd', 'description', 'prompt'], + parameterNames: ['cwd', 'description', 'name', 'prompt'], required: ['description', 'prompt'], }, { name: 'subagent_codex_secondary', - parameterNames: ['cwd', 'description', 'prompt'], + parameterNames: ['cwd', 'description', 'name', 'prompt'], required: ['description', 'prompt'], }, ],