Add default hidden context converter #59
Conversation
jiwon-oai
force-pushed
the
add-default-hidden-context-converter
branch
from
1a6d498 to
bf29372
Compare
| f"<HiddenContext>\n{item.content}\n</HiddenContext>" | ||
| ) |
There was a problem hiding this comment.
Hmm maybe this should just drop the auto-tags? Saw that we were doing that with <Task> in the default converter so added it here..
There was a problem hiding this comment.
Yeah... mostly I wanted to get away from being too opinionated about prompt engineering, but since these are all overridable (and should probably be overwritten in even a medium complexity integration), I think maybe it's fine to have some reasonable opinions here.
| f"<HiddenContext>\n{item.content}\n</HiddenContext>" | ||
| ) |
There was a problem hiding this comment.
Yeah... mostly I wanted to get away from being too opinionated about prompt engineering, but since these are all overridable (and should probably be overwritten in even a medium complexity integration), I think maybe it's fine to have some reasonable opinions here.
Have
simple_to_agent_inputhandle HiddenContextItems with str content so that simple use cases (like the cat lounge demo) don't have to implement ThreadItemConverter just to pass throughitem.contentw/o any meaningful transformation.