search

transitive-bullshit / agentic

AI agent stdlib that works with any LLM and TypeScript AI SDK.
https://agentic.so
MIT License
16.22k stars 2.12k forks source link

feat: support multiple system messages #551

Closed toxic-johann closed 1 year ago

toxic-johann commented 1 year ago

I support multiple system messages in this PR. It will allow users to pass multiple system messages instead of contacting them to string before passing them to the API.

transitive-bullshit commented 1 year ago

Thanks @toxic-johann 🙏

I think the real fix here is to allow the user to pass a messages array to sendMessage instead of a static system message (or multiple static system messages).

Going to close this for now as #416 is imho the correct long-term fix for this

toxic-johann commented 1 year ago

OK. I will check if I can help with that PR.

transitive-bullshit commented 1 year ago

Thanks @toxic-johann and sorry about that 🙏

I've been traveling lately and haven't had much time to review PRs, but I'll definitely follow up on this one since it's so important.

toxic-johann commented 1 year ago

No worry. Enjoy your vocation.