[Agents Extension] Registry update#8002
Conversation
Signed-off-by: trangevi <trangevi@microsoft.com>
There was a problem hiding this comment.
Pull request overview
Note
Copilot was unable to run its full agentic suite in this review.
Updates the AZD extensions registry to publish a new Azure AI Agents extension release and adjust the registry metadata.
Changes:
- Adds
azure-ai-agentsextension version0.1.29-previewwith updated artifacts and checksums. - Removes the top-level
schemaVersionfield from the registry JSON.
jongio
left a comment
jongio
left a comment
There was a problem hiding this comment.
Verified the 0.1.29-preview registry entry:
- Release artifacts exist at the expected URLs
- Checksum verified (spot-checked windows/amd64 - matches)
- JSON validates cleanly
- Structure and capabilities match the previous 0.1.27/0.1.28 entries
- The
schemaVersionremoval is fine since theRegistrystruct doesn't parse it
LGTM.
wbreza
left a comment
wbreza
left a comment
There was a problem hiding this comment.
Review Summary
TL;DR: The new �.1.29-preview version entry looks correct and well-formed. One question about an unrelated change bundled in.
⚠️ schemaVersion removal — intentional?
This PR removes the "schemaVersion": "1.0" field from the top of
egistry.json. This field was deliberately added in PR #7837 (merged 2 days ago) along with validation code in
egistry_version.go (CheckRegistrySchemaVersion()) to support future registry schema evolution.
The backward-compat logic treats a missing schemaVersion as "1.0", so nothing breaks — but this silently undoes the foundation for schema versioning.
Was this removal intentional? If so, should it be discussed separately from the registry update? If not, it may just need to be reverted (keep the schemaVersion line).
💬 Minor: PR description typo
The description reads Fixes #8001 8001 — the issue number appears duplicated.
✅ New version entry
The �.1.29-preview block looks good — all 6 platform artifacts, SHA256 checksums, consistent URL patterns and archive formats.
📋 Prioritization NoteThanks for the contribution! The linked issue isn't in the current milestone yet. |
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
|
/check-enforcer override |
* registry update Signed-off-by: trangevi <trangevi@microsoft.com> * Update cli/azd/extensions/registry.json Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> --------- Signed-off-by: trangevi <trangevi@microsoft.com> Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Co-authored-by: therealjohn <1501196+therealjohn@users.noreply.github.com>
Fixes #8001