Skip to content

Align TRACE ownership and prepare separate conformance publishing - #462

Merged
imran-siddique merged 1 commit into
mainfrom
maintainer-coverage-20261002
Oct 2, 2026
Merged

imran-siddique merged 1 commit into
mainfrom
maintainer-coverage-20261002

Conversation

@imran-siddique

Copy link
Copy Markdown
Member

Align with the canonical maintainer coverage rollout. The gate reads validated policy only from the reviewed base commit, with a one-commit bootstrap for initial deployment. It preserves current-head human review and independent branch protection.

Louie and Rajnish have verified write access and regain explicit ownership of the imported conformance suite and its workflows. Active contribution/navigation links point to conformance. A separate conformance publisher reuses installed-artifact build checks, ignores root library tags and validates conformance release tags against the package version. Manual execution defaults to build-only. TestPyPI and PyPI publication each require an explicit enable variable and separate environment/trusted-publisher configuration. The old publisher and documentation deployment remain active until verified replacements exist.

Validation: nine Node approval-boundary tests passed; generated policy, owners and gate matched the canonical record. Reducing security approvals to one made three tests fail. Publisher configuration and real publication have not been performed. Repository CI and independent maintainer review remain required.

@imran-siddique
imran-siddique requested review from a team, lywinged and rajnisht7 as code owners October 2, 2026 21:24
@imran-siddique
imran-siddique enabled auto-merge (squash) October 2, 2026 21:28

@lywinged lywinged left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Checked the gate against the previous one, the policy and CODEOWNERS against the roster in agentrust-io/.github#52, and that root v* tags cannot trigger the conformance publisher. Approving.

@imran-siddique
imran-siddique merged commit 1b8ef83 into main Oct 2, 2026
12 checks passed
@imran-siddique
imran-siddique deleted the maintainer-coverage-20261002 branch October 2, 2026 21:35
@lywinged

lywinged commented Oct 3, 2026

Copy link
Copy Markdown
Collaborator

@imran-siddique a note on the gate from #462, seen on #454.

The script reads .github/maintainers.json at pr.base.sha. A pull request opened before #462 merged keeps the base it was opened on, which has no policy file and is not c5cfa00, so its next push makes the gate throw HttpError: Not Found. On #454 the push of 5e8e635 failed this way with base.sha still at 4d60e4a. Merging main into the branch moved base.sha to 99b7693 and the gate passed. It fails closed, so nothing gets through, but the log does not tell the author what to do.

The open pull requests whose base has no policy file are #363, #376, #404, #438, #456, #457, #458, #459 and #460. Each one clears once its branch is updated from main. If you would rather handle it in the gate, it could read the policy at the tip of the base branch, or use the bootstrap list for any base that has no policy file and is an ancestor of c5cfa00. Either keeps the policy coming from reviewed base-branch data.

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.

3 participants