Skip to content

[llm] Fix google mapper in llm#24

Merged
roxblnfk merged 1 commit intomainfrom
bugfix/google-mapper
Oct 19, 2025
Merged

[llm] Fix google mapper in llm#24
roxblnfk merged 1 commit intomainfrom
bugfix/google-mapper

Conversation

@msmakouz
Copy link
Contributor

What was changed

Fixed two bugs in the Google mapper:

  1. Removed passing an empty table in the args parameter of functionCall.
  2. Messages with the developer role are now sent to the API as user messages instead of system instructions. Instructions on how to use the generated artifacts have this role, and they were previously being included in the agent’s prompt along with other system instructions.

@msmakouz msmakouz requested review from roxblnfk and wolfy-j October 18, 2025 17:04
@msmakouz msmakouz self-assigned this Oct 18, 2025
@roxblnfk roxblnfk merged commit 1d43194 into main Oct 19, 2025
@roxblnfk roxblnfk deleted the bugfix/google-mapper branch October 19, 2025 11:36
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.

2 participants