Skip to content

Fix protected conformance producer resolution drift on main - #329

Closed
BunsDev with Copilot wants to merge 5 commits into
mainfrom
copilot/fix-github-actions-job
Closed

BunsDev with Copilot wants to merge 5 commits into
mainfrom
copilot/fix-github-actions-job

Conversation

Copilot AI commented Sep 20, 2026 •

Copy link
Copy Markdown
Contributor

Protected client-v1 conformance runs were failing at phase1.stage.evidence-authority.checkout after main advanced past the producer revision reviewed by the SDK validator. The workflow now resolves omitted producer_revision inputs to the validator-reviewed Chat producer instead of implicitly validating the dispatch tip.

  • Producer resolution

    • Default producer_revision to the producer commit recorded in the protected SDK validator lock.
    • Preserve the existing ancestry gate for explicit overrides: requested producer revisions must still be exact lowercase OIDs already merged into the dispatch ref.
  • Validator trust checks

    • Load the validator lock from the protected validator revision when it is not already present locally.
    • Verify the protected validator revision remains reachable from OpenCoven/sdk main before accepting its reviewed producer commit.
  • Workflow contract coverage

    • Extend the workflow test to assert the validator-bound default resolution path and the protected-validator ancestry check.
    • Refresh the conformance guide’s workflow metadata row and document the new default producer-selection behavior.
producer_revision:
  description: >
    Exact merged Chat commit to validate. Defaults to the validator's
    reviewed producer commit.

Copilot AI and others added 4 commits September 20, 2026 08:29
Co-authored-by: BunsDev <68980965+BunsDev@users.noreply.github.com>
Co-authored-by: BunsDev <68980965+BunsDev@users.noreply.github.com>
Co-authored-by: BunsDev <68980965+BunsDev@users.noreply.github.com>
Co-authored-by: BunsDev <68980965+BunsDev@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix failing GitHub Actions job 'platform-conformance (darwin-arm64)' Fix protected conformance producer resolution drift on main Sep 20, 2026
Copilot AI requested a review from BunsDev September 20, 2026 08:37
@BunsDev

BunsDev commented Sep 21, 2026

Copy link
Copy Markdown
Member

Closing: producer resolution is already explicit. #314 added the producer_revision dispatch input with an ancestry gate, and protected runs are dispatched with the revision the SDK validator binds (for example run 35566636457 with producer ac1c4f4). The failures these branches chase were a MAX_PATH overflow in aws-lc-sys on Windows, diagnosed on #219 (issuecomment-5757703171) and fixed in #348. Defaulting the producer from the SDK lock would also change a governed workflow outside the repin process.

@BunsDev BunsDev closed this Sep 21, 2026
@BunsDev
BunsDev deleted the copilot/fix-github-actions-job branch September 21, 2026 21:18
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