fix(orchestrator): load both system and orchestrator prompt files - #39
fix(orchestrator): load both system and orchestrator prompt files#39Oberonskii wants to merge 2 commits into
Conversation
OrchestratorNode.__call__ was only reading self._spec.prompts.system, silently dropping the orchestrator prompt file (self._spec.prompts.orchestrator). This meant routing instructions configured in orchestrator.md were never sent to the LLM, causing the orchestrator to lose its routing context. Now both prompt files are loaded and combined. When neither is configured, description is used as a last-resort fallback, preserving backward compatibility with existing configs that rely on description only. Fixes extra-org#38
| @@ -0,0 +1,3059 @@ | |||
| version = 1 | |||
There was a problem hiding this comment.
We are working with the build tool that call poertry please make an alignment
AmitAvital1
left a comment
There was a problem hiding this comment.
We are working with the build tool that call poertry please make an alignment
|
Please star the project so i can assign to you the issue |
|
Update: Habe die |
|
Removed the |
|
please write in english. |
Description
OrchestratorNode.__call__was only readingself._spec.prompts.system, silently dropping the orchestrator prompt file (self._spec.prompts.orchestrator). This meant routing instructions configured inorchestrator.mdwere never sent to the LLM, causing the orchestrator to lose its routing context and fall back to the brief YAMLdescriptionfield.What was changed
In
src/agent_engine/engine/langgraph/nodes.py, theOrchestratorNode.__call__method now loads bothsystemandorchestratorprompt files and combines them into the final prompt. When neither file is configured,descriptionis used as a last-resort fallback, preserving backward compatibility.How it was tested
load_filepattern already used byAgentNode._build_promptOrchestratorPromptSet()), behavior falls through todescriptionunchangedRelated issue