Skip to content

docs: add provider guide standard and drift check#635

Open
caiopizzol wants to merge 2 commits into
amp-labs:mainfrom
caiopizzol:codex/provider-guide-standard-drift-ci
Open

docs: add provider guide standard and drift check#635
caiopizzol wants to merge 2 commits into
amp-labs:mainfrom
caiopizzol:codex/provider-guide-standard-drift-ci

Conversation

@caiopizzol

Copy link
Copy Markdown
Contributor

Adds a written standard for provider guides and a CI check that keeps a changed guide consistent with the connector catalog. Guides drift over time (action claims the catalog doesn't support, sample links that 404), and there's been no single place that says how to write one.

  • PROVIDER_GUIDE.md: the canonical standard. How a guide filename maps to a catalog key, the exact action-link format the drift check looks for, and the deep-connector and sample rules. Links to CONTRIBUTING.md for shared style instead of repeating it.
  • AGENTS.md / CLAUDE.md: short pointers so agents read the standard and run the drift check before finishing.
  • .github/workflows/provider-guide-drift.yml: runs the drift check in per-PR mode on changed guides, failing only on real errors in those guides (overclaimed actions, a slug that resolves to no catalog entry, dead sample links). It does not gate on the existing undocumented-provider backlog, so unrelated PRs aren't blocked.
  • README: points contributors to the standard.

Adds --changed-from/--changed-to so the workflow passes two SHAs instead of hand-building repeated --changed flags in bash. The CLI runs the git diff and fails closed if it cannot, so a shallow checkout no longer silently passes the gate. The existing --changed interface is unchanged.
@caiopizzol caiopizzol requested a review from laurenzlong June 20, 2026 13:46
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