Skip to content

fix(compliance): add media_buy.governance_aware capability + gate governance_denied (#5665) [Option A]#5676

Merged
bokelley merged 1 commit into
adcontextprotocol:mainfrom
danyliukmykola:fix/5665-governance-aware-capability-gate
Jun 26, 2026
Merged

fix(compliance): add media_buy.governance_aware capability + gate governance_denied (#5665) [Option A]#5676
bokelley merged 1 commit into
adcontextprotocol:mainfrom
danyliukmykola:fix/5665-governance-aware-capability-gate

Conversation

@danyliukmykola

@danyliukmykola danyliukmykola commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

Closes #5665 via Option A (the opt-out path).

Adds a media_buy.governance_aware boolean to get-adcp-capabilities-response.json (mirrors the supports_proposals shape + opt-in semantics) and gates governance_denied + governance_denied_recovery on it. Sellers that don't implement outbound governance consultation grade not_applicable instead of false-failing on a GOVERNANCE_DENIED they have no mechanism to produce — the not_applicable path the issue says is missing.

Self-contained: schema field + two scenario gates, using the already-supported equals: matcher. No runner change required. governance_aware is a generic, vendor-neutral field per the schema conventions.

Alternative implementation (Option B — seller-local stub) is in a separate PR for comparison; we'd lean Option A since it's self-contained and actually creates the opt-out the issue asks for.

…ernance_denied (adcontextprotocol#5665)

Option A for adcontextprotocol#5665: add an opt-in governance_aware capability and gate the
governance_denied / governance_denied_recovery storyboards on it, so sellers
without outbound governance consultation grade not_applicable instead of
false-failing on a GOVERNANCE_DENIED they cannot produce.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@danyliukmykola danyliukmykola marked this pull request as ready for review June 25, 2026 22:51

@aao-release-bot aao-release-bot Bot 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.

⚠️ 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.

governance_denied/_recovery have no requires_capability opt-out — mandatory via sync_governance with no not_applicable path

2 participants