Update examples for deploying A2UI in Agent Engine#1208
Conversation
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request updates the Gemini Enterprise agent samples to use a2ui-agent-sdk v0.1.2, disables streaming, and refactors TaskUpdater logic. Feedback highlights critical issues including unresolved merge conflict markers in several files and the omission of contact_data.json from the deployment package, which will cause runtime errors. Additionally, it is recommended to restore dynamic UI extension activation and revert the use of print statements back to the logging module for better consistency and observability.
| <<<<<<< HEAD | ||
| "a2ui-agent-sdk>=0.1.1", | ||
| ======= | ||
| "a2ui-agent-sdk", | ||
| >>>>>>> main |
There was a problem hiding this comment.
Unresolved merge conflict markers found in the dependency list. This will break package installation and environment setup. Please resolve the conflict for the a2ui-agent-sdk version.
| <<<<<<< HEAD | ||
| from a2ui.a2a import try_activate_a2ui_extension | ||
| ======= | ||
| from a2ui.a2a.extension import try_activate_a2ui_extension | ||
| >>>>>>> main |
There was a problem hiding this comment.
Unresolved merge conflict markers detected in the imports. Please resolve the conflict and remove the markers to ensure the correct module is imported.
| new_task, | ||
| ) | ||
| from a2a.utils.errors import ServerError | ||
| from a2ui.a2a.extension import try_activate_a2ui_extension |
There was a problem hiding this comment.
The package path was incorrect. #1234 fixed it.
4 participants
Description
Replace this paragraph with a description of what this PR is changing or adding, and why. Consider including before/after screenshots.
List which issues are fixed by this PR. For larger changes, raising an issue first helps reduce redundant work.
Pre-launch Checklist
If you need help, consider asking for advice on the discussion board.