Skip to content

feat(cli): freeze agent-substrate @v1 schemas and contract tests#14794

Draft
stevencartavia wants to merge 2 commits into
steven/agent-script-streamfrom
steven/agent-substrate-freeze
Draft

feat(cli): freeze agent-substrate @v1 schemas and contract tests#14794
stevencartavia wants to merge 2 commits into
steven/agent-script-streamfrom
steven/agent-substrate-freeze

Conversation

@stevencartavia
Copy link
Copy Markdown
Collaborator

@stevencartavia stevencartavia commented May 16, 2026

Update (deny.toml):** the new jsonschema test dep transitively pulls in borrow-or-share, which is licensed MIT-0 (MIT No Attribution).

Stacked on #14791.

Commits a JSON Schema (Draft 2020-12) for every shipped foundry:*@v1 identifier under docs/agents/schemas/ and wires schema validation into the existing machine-mode tests so the production code paths are pinned against the committed contract.

Also adds:

binary-level agent_contract.rs tests for forge and cast covering --introspect, --machine --help, and --machine
CI guards that prevent dangling schema refs and pin exact result_schema_ref / event_schema_ref per adopted command
additive chain.broadcast_failed diagnostic for upcoming chain-write commands (script.broadcast_failed is untouched)
docs updates for --markdown-help deprecation, --json clarification, and chain-write retry-safety
No new commands are adopted; this PR locks down the contract already shipped by the previous PRs.

@p5hzehxa

This comment was marked as spam.

Copy link
Copy Markdown

@p5hzehxa p5hzehxa left a comment

Choose a reason for hiding this comment

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

Transitive dep of jsonschema (foundry-test-utils only).
Pulled in by the new schema validation harness from PR #14794.

Amp-Thread-ID: https://ampcode.com/threads/T-019e2eb6-1816-72f5-87a7-a0c6297e9dd8
Co-authored-by: Amp <amp@ampcode.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

2 participants