Skip to content

update names for the message list and object wrapper#1112

Merged
wrenj merged 2 commits intogoogle:mainfrom
wrenj:wrap
Apr 9, 2026
Merged

update names for the message list and object wrapper#1112
wrenj merged 2 commits intogoogle:mainfrom
wrenj:wrap

Conversation

@wrenj
Copy link
Copy Markdown
Collaborator

@wrenj wrenj commented Apr 9, 2026

should be clearer

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request refactors the message list schemas by changing the root type from an object containing a 'messages' property to a raw array. To maintain backward compatibility, new wrapper schemas have been introduced to preserve the original object structure. I have no further feedback, though please note that the schema changes are breaking for direct consumers, and the new files should be registered in the test suite's configuration as suggested by the reviewer.

@wrenj wrenj merged commit b557ad4 into google:main Apr 9, 2026
13 checks passed
@github-project-automation github-project-automation bot moved this from Todo to Done in A2UI Apr 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants