[skill-update] fix(ard-registry-builder): align registry-api.md with upstream OpenAPI spec#23
Draft
github-actions[bot] wants to merge 1 commit into
Conversation
- Remove incorrect 'max 100' for POST /search pageSize; the OpenAPI spec only defines maximum: 100 for GET /agents, not for POST /search. - Add application/ai-registry (short form) as a valid registry referral type alongside application/ai-registry+json, per the OpenAPI enum in spec/schemas/ard.openapi.yaml. Source: https://github.com/ards-project/ard-spec/blob/afd447d88ed165427687d8af37e8d42398552b56/spec/schemas/ard.openapi.yaml#L189-L194 Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Changes
Two targeted corrections to
skills/ard-registry-builder/references/registry-api.mdbased on reconciliation with the upstream ARD spec atards-project/ard-spec @ afd447d.1. Remove incorrect "max 100" from
POST /searchpageSizeSource:
ard.openapi.yaml—SearchRequest.pageSizeThe OpenAPI spec defines
pageSizeforPOST /searchwith onlydefault: 10— nomaximum. Themaximum: 100constraint exists only onGET /agents(theListResponsepath). The previous skill text incorrectly propagated the agents-endpoint limit to the search endpoint.2. Add
application/ai-registryas a valid registry referral typeSource:
ard.openapi.yaml—RegistryReferral.typeenumThe OpenAPI schema defines
RegistryReferral.typeas an enum containing bothapplication/ai-registryandapplication/ai-registry+json. The skill previously only listed the+jsonform, which would cause agents to reject or ignore the shorter form encountered in the wild.Files changed
skills/ard-registry-builder/references/registry-api.mdValidation
validate-metadata.pypasses on existing SKILL.md name and description (no change to SKILL.md).assets/ai-catalog.schema.json) is unchanged and still matches upstream SHA37c4cb743b29741847e6f99f8bc8ccaaa2d6e422.Remaining risks
None identified. The spec is at v0.9 Draft status; these two corrections align the skill precisely with the machine-readable OpenAPI definition rather than prose interpretation.