feat(testing): bundle adcp-spec schemas as adcp.schemas package, eliminate inlined copies

[bokelley]

Context

PR #406 inlined a copy of adcp-agents.json for tests and dropped the ^https:// pattern (drift caught manually in code review). If the schemas were bundled into the SDK (e.g. src/adcp/types/schemas/) and exported, tests would import the authoritative shape and drift would be caught by spec-resync rather than human review.

Acceptance Criteria

• Schemas bundled as adcp.schemas (or src/adcp/types/schemas/) package
• Tests import from the bundled package, not inlined copies
• Spec-resync workflow reads from the same source
• No more inlined JSON in tests/

Refs

• PR feat(server): /.well-known/adcp-agents.json discovery endpoint (#381) #406 (inlined adcp-agents.json with drift)

[bokelley]

Triage

Classification: Feature request
Bucket(s): validation, client (labels not in repo — gap noted)
Status: drafting-pr
Milestone: (omitted — no target version in issue)

What the experts said:

• code-reviewer: approved after 2 passes — path-traversal guard (verified via files("adcp.schemas") / "../../..." test), json.JSONDecodeError surface fixed, generator annotation corrected; all blockers resolved
• dx-expert: approved — load_schema(name) + ADCP_AGENTS constant is the right API shape; dynamic error message listing; cross-reference to adcp.validation.schema_loader in module docstring added

My take: The core ask (no more inlined JSON in tests) is addressed cleanly. The schema is stored in src/adcp/schemas/ rather than schemas/cache/ to survive sync_schemas.py wipes — a follow-up can wire the post-sync copy when upstream publishes the schema in the protocol bundle.

PR #442 (draft): feat(schemas): bundle adcp-agents.json as adcp.schemas package, eliminate inlined test copy

---

Triaged by Claude Code. Session: https://claude.ai/code/session_01TfYXsZysMjHdZBpJqkVW7P

---

Generated by Claude Code