Skip to content

feat(compliance): add validation result ids#5381

Open
sangilish wants to merge 1 commit into
adcontextprotocol:mainfrom
sangilish:sangilish/spec-validation-result-ids-5337
Open

feat(compliance): add validation result ids#5381
sangilish wants to merge 1 commit into
adcontextprotocol:mainfrom
sangilish:sangilish/spec-validation-result-ids-5337

Conversation

@sangilish
Copy link
Copy Markdown
Contributor

Summary

  • add an optional storyboard validation id field for stable per-assertion references
  • require runners to echo authored validation ids as validation_result.id
  • bump the runner output contract from 2.5.0 to 2.6.0 and add a minor changeset

This is the spec-only slice for #5337. Addie pass-through and adcp-client runner emission remain separate follow-ups.

Validation

  • npx prettier --check .changeset/validation-result-ids.md
  • rg -n 'version: "2\\.6\\.0"|validation_result\\.id|id: string \\(optional stable|echo this value' static/compliance/source/universal/storyboard-schema.yaml static/compliance/source/universal/runner-output-contract.yaml .changeset/validation-result-ids.md\n\nNote: I did not reformat the touched contract YAML files because runner-output-contract.yaml already fails full-file Prettier/YAML parsing on main; this PR keeps the diff to the contract change only.

Copy link
Copy Markdown
Contributor

@aao-release-bot aao-release-bot Bot left a comment

Choose a reason for hiding this comment

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

⚠️ Argus review could not complete

The automated review encountered an issue (possibly reached max turns, timed out, or failed to post the final gh pr review). A human reviewer should take this PR.

View workflow run

This is an automated message from the Argus AI review workflow.

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.

1 participant