Conversation
There was a problem hiding this comment.
Code Review
This pull request introduces a significant and well-structured refactoring to create a centralized a2ui-agent package for managing A2UI schemas. The changes include a build-time hook to package schema files, a robust schema manager with fallback loading mechanisms, and a validator. The contact lookup sample has been updated to use this new package, which greatly improves maintainability and decouples the agent from hardcoded schemas. The overall approach is excellent. I've identified a few critical and high-severity issues related to exception handling and a NameError that could break the code at runtime, along with some medium-severity suggestions to improve the robustness of the build script and validation logic.
a2a_agents/python/a2ui_agent/src/a2ui/inference/schema/manager.py
Outdated
Show resolved
Hide resolved
a2a_agents/python/a2ui_agent/src/a2ui/inference/schema/manager.py
Outdated
Show resolved
Hide resolved
a2a_agents/python/a2ui_agent/src/a2ui/inference/schema/manager.py
Outdated
Show resolved
Hide resolved
6bc12ec to
6714d4f
Compare
557fe78 to
7f8af23
Compare
7f8af23 to
187797d
Compare
187797d to
91121f2
Compare
zeroasterisk
left a comment
There was a problem hiding this comment.
LGTM — clean CI improvements and schema packing hook. 👍
91121f2 to
ec97517
Compare
|
Rebased and ready for another look. |
Tested: - [x] The contact_lookup client successfully connected to the contact_lookup agent and rendered the response correctly.
ec97517 to
36fa487
Compare
Tested:
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.