Skip to content

Commit 8255cde

Browse files
committed
fix(yaml): carry style: 'llm' through LLM planner config for observability
CodeRabbit caught: compileMissionYamlWithLlmPlan omitted `style` from the agentos plannerConfig. The agentos compiler ignores `style` when `plan` is set, so behavior was unchanged — but downstream code that reads plannerConfig.style for logging/telemetry would see undefined. Set it explicitly so the choice is observable everywhere it might be inspected.
1 parent cb90481 commit 8255cde

1 file changed

Lines changed: 5 additions & 0 deletions

File tree

src/orchestration/yaml-compiler.ts

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -657,6 +657,11 @@ async function compileMissionYamlWithLlmPlan(
657657
maxSteps: doc.planner.maxSteps ?? 8,
658658
maxIterationsPerNode: doc.planner.maxIterations,
659659
...(doc.planner.parallelTools !== undefined ? { parallelTools: doc.planner.parallelTools } : {}),
660+
// Carry through `style: 'llm'` so any downstream code that inspects
661+
// `plannerConfig.style` for logging/telemetry sees the actual choice;
662+
// the agentos compiler ignores `style` when `plan` is set, so this
663+
// is observability-only.
664+
style: 'llm' as const,
660665
plan,
661666
} as any);
662667

0 commit comments

Comments
 (0)