From 19aaf5d329e5a7ac052fd2f0465125957d59b948 Mon Sep 17 00:00:00 2001 From: ririnto Date: Mon, 18 May 2026 00:54:01 +0900 Subject: [PATCH] fix(prompts): point planning guidance at plan subagent --- src/agents/dynamic-agent-core-sections.ts | 2 +- src/agents/sisyphus/gpt-5-4.ts | 2 +- src/features/team-mode/team-registry/validator.test.ts | 2 +- src/features/team-mode/types.test.ts | 4 ++-- src/features/team-mode/types.ts | 2 +- 5 files changed, 6 insertions(+), 6 deletions(-) diff --git a/src/agents/dynamic-agent-core-sections.ts b/src/agents/dynamic-agent-core-sections.ts index 69742ff16..1f91c7a78 100644 --- a/src/agents/dynamic-agent-core-sections.ts +++ b/src/agents/dynamic-agent-core-sections.ts @@ -196,7 +196,7 @@ export function buildNonClaudePlannerSection(model: string): string { Multi-step task? **ALWAYS consult Plan Agent first.** Do NOT start implementation without a plan. - Single-file fix or trivial change → proceed directly -- Anything else (2+ steps, unclear scope, architecture) → \`task(subagent_type="prometheus", ...)\` FIRST +- Anything else (2+ steps, unclear scope, architecture) → \`task(subagent_type="plan", ...)\` FIRST - Use \`task_id\` to resume the same Plan Agent - ask follow-up questions aggressively - If ANY part of the task is ambiguous, ask Plan Agent before guessing diff --git a/src/agents/sisyphus/gpt-5-4.ts b/src/agents/sisyphus/gpt-5-4.ts index 5f5e5bc2c..964087f13 100644 --- a/src/agents/sisyphus/gpt-5-4.ts +++ b/src/agents/sisyphus/gpt-5-4.ts @@ -288,7 +288,7 @@ Every implementation task follows this cycle. No exceptions. Follow \`\` protocol for tool usage and agent prompts. 2. PLAN - List files to modify, specific changes, dependencies, complexity estimate. - Multi-step (2+) → consult Plan Agent via \`task(subagent_type="prometheus", ...)\`. + Multi-step (2+) → consult Plan Agent via \`task(subagent_type="plan", ...)\`. Single-step → mental plan is sufficient. diff --git a/src/features/team-mode/team-registry/validator.test.ts b/src/features/team-mode/team-registry/validator.test.ts index ffc9ad263..12d64015c 100644 --- a/src/features/team-mode/team-registry/validator.test.ts +++ b/src/features/team-mode/team-registry/validator.test.ts @@ -13,7 +13,7 @@ import { } from "./validator" const PROMETHEUS_REJECTION_MESSAGE = - "Agent 'prometheus' is plan-mode-only; can only write to .omo/*.md (enforced by prometheusMdOnly hook). Cannot write to team mailbox. Use category: 'plan' instead." + "Agent 'prometheus' is plan-mode-only; can only write to .omo/*.md (enforced by prometheusMdOnly hook). Cannot write to team mailbox. Use delegate-task with subagent_type: 'plan' instead." function createCategoryMember(name: string): Member { return { diff --git a/src/features/team-mode/types.test.ts b/src/features/team-mode/types.test.ts index 1d944eec0..0a047a930 100644 --- a/src/features/team-mode/types.test.ts +++ b/src/features/team-mode/types.test.ts @@ -136,7 +136,7 @@ describe("team-mode types", () => { ], [ "prometheus", - "Agent 'prometheus' is plan-mode-only; can only write to .omo/*.md (enforced by prometheusMdOnly hook). Cannot write to team mailbox. Use category: 'plan' instead.", + "Agent 'prometheus' is plan-mode-only; can only write to .omo/*.md (enforced by prometheusMdOnly hook). Cannot write to team mailbox. Use delegate-task with subagent_type: 'plan' instead.", ], ] as const @@ -286,7 +286,7 @@ describe("team-mode types", () => { "Agent 'momus' is read-only (plan reviewer). Cannot write to mailbox as team member. Use delegate-task for plan review instead.", ) expect(AGENT_ELIGIBILITY_REGISTRY.prometheus.rejectionMessage).toBe( - "Agent 'prometheus' is plan-mode-only; can only write to .omo/*.md (enforced by prometheusMdOnly hook). Cannot write to team mailbox. Use category: 'plan' instead.", + "Agent 'prometheus' is plan-mode-only; can only write to .omo/*.md (enforced by prometheusMdOnly hook). Cannot write to team mailbox. Use delegate-task with subagent_type: 'plan' instead.", ) expect(CategoryMemberSchema).toBeDefined() expect(SubagentMemberSchema).toBeDefined() diff --git a/src/features/team-mode/types.ts b/src/features/team-mode/types.ts index 21f7a0d6a..a218cf3c0 100644 --- a/src/features/team-mode/types.ts +++ b/src/features/team-mode/types.ts @@ -229,7 +229,7 @@ export const AGENT_ELIGIBILITY_REGISTRY: Readonly