Skip to content

fix(server-ai): sanitize empty-role generic messages#440

Open
ponyfly6 wants to merge 1 commit into
mainfrom
fix/empty-role-generic-messages-clean
Open

fix(server-ai): sanitize empty-role generic messages#440
ponyfly6 wants to merge 1 commit into
mainfrom
fix/empty-role-generic-messages-clean

Conversation

@ponyfly6

Copy link
Copy Markdown
Contributor

Summary

  • add a shared sanitizer for empty-role generic LangGraph messages before LLM calls
  • apply the sanitizer in both subgraph and summarize model invocations
  • cover assistant/tool/malformed message recovery with a targeted unit test

Testing

  • yarn nx test server-ai --runInBand --testPathPattern=sanitize-messages-for-llm.test.ts

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant