System prompts updated #1188
System prompts updated #1188
Conversation
Docs Preview
|
| ) | ||
| else: | ||
| # If the part is not dynamic, we replace it with the new system prompt. | ||
| msg.parts[i] = _messages.SystemPromptPart(self.system_prompts[0]) |
There was a problem hiding this comment.
We should probably only replace the first system prompt we encounter, not all
There was a problem hiding this comment.
probably this file should be turned into a test
|
I think we should rework this a bit to expose a hook for modifying message history, and the hook should have access to the agent state (things like its system prompts, etc.). That way we can make a change to the behavior but also make it easier for users to change it themselves if necessary. In particular, it's reasonable to NOT want the history updated, OR to want just the first system prompt updated. But it may also be desirable to want to change it in a more interesting way. I think exposing a hook for this is the easiest way to put it in users' hands if they want to do something more sophisticated. (Asking them to override the AgentGraph is ... not a good idea.) |
No description provided.