Skip to content

fix(test): drop stale get_products_refine allowlist on proposal-mode adapter#1599

Merged
bokelley merged 1 commit intomainfrom
bokelley/fix-proposal-mode-stale-allowlist
May 8, 2026
Merged

fix(test): drop stale get_products_refine allowlist on proposal-mode adapter#1599
bokelley merged 1 commit intomainfrom
bokelley/fix-proposal-mode-stale-allowlist

Conversation

@bokelley
Copy link
Copy Markdown
Contributor

@bokelley bokelley commented May 8, 2026

Summary

Main is currently red on the proposal-mode adapter gate. The AdCP 3.0.7 schema bump (#1595) landed adcp#4088, which adds context_outputs / context_inputs chaining for proposal_id in proposal_finalize.yaml. Downstream phases now resolve the seller's freshly-minted proposal_id instead of the literal balanced_reach_q2 placeholder, so get_products_refine is no longer expected to fail.

The staleness guard from #1594 fired correctly — that's the design. This PR drops the now-unneeded allowlist entry.

Why this matters now

Main has been red since the 3.0.7 bump merged at 13:13 UTC. All open PRs (#1597, #1598) are blocked on this same failure.

Test plan

  • Test & Build job passes on the proposal-mode adapter step.
  • Main CI returns to green.

AdCP 3.0.7 (#1595) landed adcp#4088, fixing proposal_id chaining in
proposal_finalize.yaml. get_products_refine now passes — and the
cascade-skip that hid create_media_buy is gone, exposing a real
adapter bug: create_media_buy's response doesn't satisfy the 3.0.7
create-media-buy-response.json schema.

Allowlist retargeted to mask create_media_buy until the adapter is
fixed (tracked at #1600). Unblocks main CI.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
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