Skip to content

Conversation

@wbreza
Copy link
Contributor

@wbreza wbreza commented Nov 8, 2025

Fixes #6056

Adds support to the azure.yaml schema where host: azure.ai.agent

  • Enables docker options
  • Enables custom configuration

Note

Also reorganizes conditionals improve readability.

Copy link
Member

@vhvb1989 vhvb1989 left a comment

Choose a reason for hiding this comment

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

It doesn't feel OK to me.
The schema would be there even w/o installing the extension
What if another extension wants to do the same?

@wbreza
Copy link
Contributor Author

wbreza commented Nov 11, 2025

It doesn't feel OK to me. The schema would be there even w/o installing the extension What if another extension wants to do the same?

I'm open to other ideas but JSON schema is not dynamic. Since we have auto install - if a user followed the schema, chose a host at runtime the extension would get installed and all would work. Perhaps this is the way for now for 1st party extensions till we figure out a better way to support this in the future.

@rajeshkamal5050 rajeshkamal5050 merged commit e22e229 into Azure:main Nov 11, 2025
20 checks passed
@wbreza wbreza deleted the ext-agent-schema-int branch November 12, 2025 00:18
@vhvb1989
Copy link
Member

It doesn't feel OK to me. The schema would be there even w/o installing the extension What if another extension wants to do the same?

I'm open to other ideas but JSON schema is not dynamic. Since we have auto install - if a user followed the schema, chose a host at runtime the extension would get installed and all would work. Perhaps this is the way for now for 1st party extensions till we figure out a better way to support this in the future.

My recommendation is to avoid adding anything to azure.yaml that is only understood by some extension.
Keep azure.yaml's schema clean and easy - it's overcomplicated enough already with azd's conditionals 🤓
Avoid to user azd as a midle-man between some extension and the user.
When it comes to extensions, it is hard for me to see the benefit(s) of using azure.yaml for something that is not only opt-in (extending azd functionality).

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.

[ai agents extension] Schema Validation Error for remoteBuild under extension service target

5 participants