Skip to content

docs(conformance): fix stale example paths after examples/ reorg#54

Merged
rvguha merged 1 commit into
ards-project:mainfrom
BenMcGit:fix/conformance-example-paths
Jun 29, 2026
Merged

docs(conformance): fix stale example paths after examples/ reorg#54
rvguha merged 1 commit into
ards-project:mainfrom
BenMcGit:fix/conformance-example-paths

Conversation

@BenMcGit

@BenMcGit BenMcGit commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

Hi! 👋

I'm Ben, an engineer on the Agent Discovery team at Zapier.

We’re excited to see how this space evolves and will be following this repo closely. Thanks for all the work you’ve put into it so far! If you’re ever interested in chatting or collaborating, feel free to reach out.

Summary

Two references to the mock catalog manifest were left pointing at old locations after the c60a002 reorg moved it into examples/basic/:

  • conformance/README.md — the documented manual-validation command (./bin/conformance-test manifest examples/ai-catalog.json) now 404s; updated to examples/basic/ai-catalog.json so copy/paste works.
  • conformance/examples/basic/registry-server.py — comment claimed the mock data was seeded from spec/examples/ai-catalog.json, a path that has not existed since the earlier reorg in 5c191fb. Repointed to ./ai-catalog.json (its real, current sibling).

No spec or behavioral changes — docs/comment only.

Test plan

  • ./bin/conformance-test manifest examples/basic/ai-catalog.json — passes (was previously a Local file does not exist failure with the README's stale path)
  • ./bin/run-conformance-demo — full end-to-end demo passes (ALL CONFORMANCE CHECKS PASSED)

The c60a002 reorg moved ai-catalog.json into examples/basic/ but two
references were not updated:

- conformance/README.md showed 'examples/ai-catalog.json' as the manual
  validation example; copying that command verbatim now 404s.
- conformance/examples/basic/registry-server.py had a comment pointing
  at spec/examples/ai-catalog.json (a path that has not existed since
  an earlier reorg in 5c191fb).

@rvguha rvguha 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.

Ben, thank you for the fix!

@rvguha rvguha merged commit 4ef09d2 into ards-project:main Jun 29, 2026
1 check 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