Skip to content

Fix Vertex AI tool response roles#697

Closed
Gujiassh wants to merge 1 commit intocrmne:mainfrom
Gujiassh:fix/vertexai-tool-response-roles
Closed

Fix Vertex AI tool response roles#697
Gujiassh wants to merge 1 commit intocrmne:mainfrom
Gujiassh:fix/vertexai-tool-response-roles

Conversation

@Gujiassh
Copy link
Copy Markdown

Summary

  • normalize Gemini-style tool response messages to role: 'user' when Vertex AI builds generateContent payloads
  • keep the fix scoped to the Vertex AI chat wrapper without changing Gemini's native payload shape
  • add a focused provider spec proving function-response payloads use the Vertex AI-compatible role while preserving the existing function response structure

Validation

  • bundle exec rspec spec/ruby_llm/providers/vertex_ai_spec.rb
  • bundle exec rubocop lib/ruby_llm/providers/vertexai/chat.rb spec/ruby_llm/providers/vertex_ai_spec.rb

Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-opencode)

Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>
@Gujiassh
Copy link
Copy Markdown
Author

Closing this stale radar PR to keep the queue clean: it has been open for weeks without maintainer response, and I am not going to keep it in the active follow-up set without renewed upstream interest.

@Gujiassh Gujiassh closed this Apr 11, 2026
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