Skip to content

ci: restrict Dependabot auto-merge to minor and patch updates - #324

Closed
dinakarjs wants to merge 1 commit into
agentrust-io:mainfrom
dinakarjs:ci/dependabot-update-type-gate
Closed

dinakarjs wants to merge 1 commit into
agentrust-io:mainfrom
dinakarjs:ci/dependabot-update-type-gate

Conversation

@dinakarjs

Copy link
Copy Markdown
Contributor

The current workflow queues auto-merge for every Dependabot PR, including majors, and does not rerun when Dependabot updates a PR. This adds synchronize, verifies Dependabot metadata with a SHA-pinned action, and queues only nonempty groups whose every update is classified as semver minor or patch. Major, unknown and malformed metadata stays manual; the merge command matches the event head SHA.

Ports the safeguards prepared in weight-custody-manifest#196 in response to @rajnisht7's review on weight-custody-manifest#192. Corrects the root and examples pip comments to reflect their existing major-version ignore policy. No dependency policy, approval requirement or branch protection is changed.

Validation: 43 root tests passed, including 15 shell/metadata controls using real Bash and jq with a stubbed gh command; targeted Ruff and formatting checks passed; all 47 integration manifests, compatibility validation, generated index/catalog checks and git diff --check passed. The new tests run in the existing validation workflow with its hash-locked dependencies.

The bot-specific path is not exercised by this human-authored PR. After integration, validate a genuine Dependabot run and confirm current-head approval and required checks still block merging.

@github-actions

github-actions Bot commented Oct 15, 2026 •

Copy link
Copy Markdown
Contributor

🟡 Contributor Check: MEDIUM

Check Result
Profile MEDIUM
Credential LOW
Overall MEDIUM

Automated check by AgenTrust Contributor Check.

@github-actions github-actions Bot added the needs-review:MEDIUM Contributor check flagged MEDIUM risk label Oct 15, 2026

@imran-siddique imran-siddique left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Verified at fb1aec9: the workflow is byte-identical to the copies now on the cmcp, ca2a, agent-manifest, trace-spec, agentrust-telemetry and weight-custody-manifest branches, and the dependabot/fetch-metadata pin resolves to the v3.1.0 tag.

@imran-siddique

Copy link
Copy Markdown
Member

Closing in favour of #325, which removed the auto-merge workflow here: each integration's tests run on their own path filter, so no single required check covers a Dependabot bump. The grouping from #317 stays. Your gate is the version now on the auto-merge branches in the other six repos.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-review:MEDIUM Contributor check flagged MEDIUM risk

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants