ci: restore main required check contexts#8691
Draft
scarmani wants to merge 6 commits into
Draft
Conversation
Run OpenAPI and SDK required wrappers on main pushes, and make the PR-bound merge-quorum workflow emit an explicit green mainline context after a protected PR merge. This keeps PR quorum semantics unchanged while restoring required-context health on default-branch heads. Co-authored-by: codex[bot] <codex[bot]@users.noreply.github.com>
Contributor
|
OpenAPI Spec Update The OpenAPI specification has changed. Please review the generated spec in the workflow artifacts. |
The SDK integration pytest suite invokes the TypeScript compile test, so install the TypeScript SDK npm dependencies in that job before pytest. This keeps the workflow-only required-context repair from carrying an unrelated red SDK integration job. Co-authored-by: codex[bot] <codex[bot]@users.noreply.github.com>
Contributor
|
OpenAPI Spec Update The OpenAPI specification has changed. Please review the generated spec in the workflow artifacts. |
…quired-contexts-20260629
Contributor
|
OpenAPI Spec Update The OpenAPI specification has changed. Please review the generated spec in the workflow artifacts. |
…quired-contexts-20260629
…quired-contexts-20260629
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
pushonmaintoOpenAPI Specso the requiredGenerate & Validatewrapper is produced on default-branch heads.pushonmaintoSDK Testsso the requiredTypeScript SDK Type Checkwrapper is produced on default-branch heads.pushonmainsuccess path forAragora Merge Quorumso already-merged main commits get a greenaragora-merge-quorumcontext, while the PR-bound quorum evaluator remains unchanged forpull_requestandworkflow_dispatch.SDK Integration Tests, because that pytest suite invokesnpx tsc --noEmitand otherwise fails without@types/node.Investigation
origin/mainhead checked:80a1776b4ccb9c37661cc290f88b3fa003225d13from refactor(protocols): collapse protocol split onto aragora.protocols (VAL-P4A-009) #8678. While this PR was open, refactor(observability): re-home telemetry + monitoring onto aragora.observability (VAL-P4A-010/011) #8690 merged andorigin/mainadvanced to2228fef65f28681e218afc028c405bac5f8bc918; this branch was updated by merging that current base.lint,typecheck,sdk-parity,Generate & Validate,TypeScript SDK Type Check, andaragora-merge-quorum; no branch-protection expectation drift found.pushtriggers.Validation
actionlint .github/workflows/openapi.yml .github/workflows/sdk-test.yml .github/workflows/aragora-merge-quorum.ymlcheck yaml,detect private key,gitleaks, portability guardgit diff --check origin/main..HEADnpm ci && npx tsc --noEmitinsdk/typescriptbash scripts/automation_pr_preflight.sh origin/main HEADgh pr checks 8691 --requiredis green at71c4707cf8827168696222fb30cf84a12afa388cRisk / settlement
This touches GitHub Actions required-check behavior and the merge-quorum workflow. Treat as Tier 4 / merge-authority self-modification; do not merge without exact-head human settlement.