Skip to content

fix(spec): make displayName optional in CatalogEntry CDDL (align with ADR-0016)#56

Merged
tadasant merged 1 commit into
Agent-Card:mainfrom
Wolfe-Jam:fix/cddl-displayname-optional
Jun 30, 2026
Merged

fix(spec): make displayName optional in CatalogEntry CDDL (align with ADR-0016)#56
tadasant merged 1 commit into
Agent-Card:mainfrom
Wolfe-Jam:fix/cddl-displayname-optional

Conversation

@Wolfe-Jam

@Wolfe-Jam Wolfe-Jam commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

The normative CDDL schema (§CDDL Schema, CatalogEntry) still lists displayName: text as required, but ADR-0016 (#39) made displayName optional — and the prose (§Catalog Entry, §"Resolving an Artifact's Display Name") plus the Level-1 (Minimal) conformance definition already treat it as optional. The CDDL is the lone holdout, so it currently contradicts them.

One-line fix:

- displayName: text,
+ ? displayName: text,

Without it, a CDDL-only validator rejects otherwise-conformant entries that omit displayName — e.g. when the referenced artifact self-names (an A2A Agent Card / MCP Server Card), which ADR-0016 explicitly recommends. Publisher's displayName is left required, unchanged.

Assisted by Claude (Opus 4.8) · Approved by James Wolfe (@Wolfe-Jam)

… ADR-0016)

ADR-0016 (Agent-Card#39) made displayName OPTIONAL on Catalog Entry; the prose, the
Level-1 conformance definition, and the ADR all agree, but the normative
CDDL still listed displayName: text as required. This aligns the CDDL.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@Wolfe-Jam Wolfe-Jam requested a review from a team as a code owner June 30, 2026 05:00

@tadasant tadasant left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Good catch, thank you!

@tadasant tadasant merged commit 41e4442 into Agent-Card:main Jun 30, 2026
4 checks passed
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.

2 participants