fix(agent-config): pass useTaskSystem to sisyphus-junior when task_system is enabled
sisyphus-junior prompt always used todo-based discipline text regardless of experimental.task_system setting because the useTaskSystem flag was never forwarded from agent-config-handler to createSisyphusJuniorAgentWithOverrides.
This commit is contained in:
parent
498fda11a0
commit
7eb67521cb
@ -200,6 +200,46 @@ describe("createSisyphusJuniorAgentWithOverrides", () => {
|
|||||||
})
|
})
|
||||||
})
|
})
|
||||||
|
|
||||||
|
describe("useTaskSystem integration", () => {
|
||||||
|
test("useTaskSystem=true produces Task_Discipline prompt for Claude", () => {
|
||||||
|
//#given
|
||||||
|
const override = { model: "anthropic/claude-sonnet-4-5" }
|
||||||
|
|
||||||
|
//#when
|
||||||
|
const result = createSisyphusJuniorAgentWithOverrides(override, undefined, true)
|
||||||
|
|
||||||
|
//#then
|
||||||
|
expect(result.prompt).toContain("TaskCreate")
|
||||||
|
expect(result.prompt).toContain("TaskUpdate")
|
||||||
|
expect(result.prompt).not.toContain("todowrite")
|
||||||
|
})
|
||||||
|
|
||||||
|
test("useTaskSystem=true produces task_discipline_spec prompt for GPT", () => {
|
||||||
|
//#given
|
||||||
|
const override = { model: "openai/gpt-5.2" }
|
||||||
|
|
||||||
|
//#when
|
||||||
|
const result = createSisyphusJuniorAgentWithOverrides(override, undefined, true)
|
||||||
|
|
||||||
|
//#then
|
||||||
|
expect(result.prompt).toContain("<task_discipline_spec>")
|
||||||
|
expect(result.prompt).toContain("TaskCreate")
|
||||||
|
expect(result.prompt).not.toContain("<todo_discipline_spec>")
|
||||||
|
})
|
||||||
|
|
||||||
|
test("useTaskSystem=false (default) produces Todo_Discipline prompt", () => {
|
||||||
|
//#given
|
||||||
|
const override = {}
|
||||||
|
|
||||||
|
//#when
|
||||||
|
const result = createSisyphusJuniorAgentWithOverrides(override)
|
||||||
|
|
||||||
|
//#then
|
||||||
|
expect(result.prompt).toContain("todowrite")
|
||||||
|
expect(result.prompt).not.toContain("TaskCreate")
|
||||||
|
})
|
||||||
|
})
|
||||||
|
|
||||||
describe("prompt composition", () => {
|
describe("prompt composition", () => {
|
||||||
test("base prompt contains discipline constraints", () => {
|
test("base prompt contains discipline constraints", () => {
|
||||||
// given
|
// given
|
||||||
|
|||||||
@ -101,9 +101,11 @@ export async function applyAgentConfig(params: {
|
|||||||
sisyphus: builtinAgents.sisyphus,
|
sisyphus: builtinAgents.sisyphus,
|
||||||
};
|
};
|
||||||
|
|
||||||
|
const useTaskSystem = params.pluginConfig.experimental?.task_system ?? false;
|
||||||
agentConfig["sisyphus-junior"] = createSisyphusJuniorAgentWithOverrides(
|
agentConfig["sisyphus-junior"] = createSisyphusJuniorAgentWithOverrides(
|
||||||
params.pluginConfig.agents?.["sisyphus-junior"],
|
params.pluginConfig.agents?.["sisyphus-junior"],
|
||||||
undefined,
|
undefined,
|
||||||
|
useTaskSystem,
|
||||||
);
|
);
|
||||||
|
|
||||||
if (builderEnabled) {
|
if (builderEnabled) {
|
||||||
|
|||||||
Loading…
x
Reference in New Issue
Block a user