Skip to content

Conversation

@bokelley
Copy link
Contributor

Summary

Alternative fix for #324. Instead of hard-coding v2 URLs that will eventually go stale, this removes the unnecessary $schema references entirely:

  • Removed default fields from $schema property definitions (JSON Schema validators don't auto-insert defaults, so these weren't functional)
  • Removed $schema fields from all examples (the field is optional in adagents.json files)

This avoids version maintenance issues without losing any functionality.

Context

The original issue was that https://adcontextprotocol.org/schemas/adagents.json returns 404 while /schemas/v2/adagents.json works. The build system transforms $id and $ref but not default values or example content.

Rather than hard-coding a version that will need updating when v3 releases, we simply remove these non-functional references.

Test plan

  • JSON syntax validated
  • All schema tests pass
  • No other schemas have this pattern

🤖 Generated with Claude Code

The `default` field for $schema wasn't providing functional value (JSON
Schema validators don't auto-insert defaults), and the example $schema
URLs were pointing to non-existent paths that would drift with versions.

Since $schema is optional in adagents.json files, removing these
references avoids version maintenance issues without losing any
functionality.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Add $schema back to examples using relative /schemas/ path
- Update build script to transform $schema fields (not just $id/$ref)
- This allows examples to be testable while still being version-agnostic in source

The build will transform /schemas/adagents.json → /schemas/2.5.0/adagents.json

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@bokelley bokelley merged commit 1e7a808 into main Dec 17, 2025
5 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