Skip to content

Conversation

@jacobsimionato
Copy link
Collaborator

This PR updates the v0.9 specification to include iconUrl and agentDisplayName in the standard catalog theme definition.

Changes

  • standard_catalog.json: Added iconUrl and agentDisplayName to the theme object.
  • a2ui_protocol.md: Added a Theme section under Standard Component Catalog to document these new properties and explain their usage for identifying sub-agents and ensuring security against impersonation in orchestrator scenarios.

Changes:
- Removed outputFormat from DateTimeInput as it was underspecified.
- Added min and max properties to DateTimeInput for ISO 8601 range constraints.
- Added openUrl function to the standard catalog for handling links.
- Updated schema definitions to support 'void' return type for functions.
Copy link

@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 introduces identity fields (iconUrl and agentDisplayName) to the theme specification, which is a valuable addition for attribution and security in multi-agent scenarios. The changes are well-documented in a2ui_protocol.md. The PR also includes the addition of an openUrl function and updates to the DateTimeInput component. While these changes seem correct and beneficial, they are not mentioned in the PR description, which could be updated for clarity. The overall changes improve the specification. I've found one minor formatting issue in the documentation.

| :----------------- | :----- | :-------------------------------------------------------------------------------------------------------- |
| **primaryColor** | String | The primary UI color as a hexadecimal code (e.g., '#00BFFF'). |
| **iconUrl** | URI | A URL for an image (e.g., logo or avatar) that identifies the agent or tool associated with the surface. |
| **agentDisplayName**| String | Text to be displayed next to the surface to identify the agent or tool that created it (e.g. "Weather Bot").|

Choose a reason for hiding this comment

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

medium

There's a formatting issue in the markdown table. A space is missing between **agentDisplayName** and the following |, which will likely cause rendering issues for this row by merging the 'Property' and 'Type' columns.

Suggested change
| **agentDisplayName**| String | Text to be displayed next to the surface to identify the agent or tool that created it (e.g. "Weather Bot").|
| **agentDisplayName** | String | Text to be displayed next to the surface to identify the agent or tool that created it (e.g. "Weather Bot").|

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