Skip to content

ci: actually enforce Agent Plugins conformance in the pipeline - #8

Merged
jpantsjoha merged 1 commit into
mainfrom
ci/enforce-conformance
Aug 7, 2026
Merged

ci: actually enforce Agent Plugins conformance in the pipeline#8
jpantsjoha merged 1 commit into
mainfrom
ci/enforce-conformance

Conversation

@jpantsjoha

Copy link
Copy Markdown
Owner

The bug

The CI workflow invokes each validator script directly rather than running make gate. Adding spec to the Makefile target therefore never reached CIvalidate_agent_plugins.py has not run on a single push, including the pushes that shipped v0.2.0.

That makes the README line published in #7"Enforced in CI by make spec" — false at the time of writing. This PR makes it true.

The plugin is conformant; that was verified by hand and by a fresh public clone. What was missing is the automation that stops it regressing.

Changes

  • Adds the validate_agent_plugins.py step to .github/workflows/gate.yml.
  • Adds an advisory step validating against the upstream skills-ref reference validator and the published JSON schemas, rather than our in-repo transcription of them. A spec revision upstream now surfaces as a warning rather than drifting silently. It is network-dependent, so it uses continue-on-error — the deterministic in-repo gate remains the blocking one.
  • Refreshes research/coverage-matrix.md, which still carried a 2026-07-23 date and no conformance target.

Credit

The gate.yml change is cherry-picked from the parallel branch on #4, which caught this. #4 itself conflicts with main and its remaining content has been superseded — see the note there.

The CI workflow invokes each validator script directly rather than running
`make gate`, so adding `spec` to the Makefile target never reached CI —
validate_agent_plugins.py has not run on a single push. The README claim that
conformance is "enforced in CI by make spec" was therefore false when written.

Adds the missing step, and an advisory one that validates against the UPSTREAM
reference validator and the published JSON schemas rather than our transcription
of them, so a spec revision upstream surfaces as a warning instead of silently
drifting. That step is network-dependent, so it does not block; the
deterministic in-repo gate does.

Cherry-picked from the parallel branch on PR #4, which spotted this.

Also refreshes research/coverage-matrix.md, which still carried a 2026-07-23
date and no conformance target.
@jpantsjoha
jpantsjoha merged commit 4afe365 into main Aug 7, 2026
1 check passed
@jpantsjoha
jpantsjoha deleted the ci/enforce-conformance branch August 7, 2026 11:32
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