Skip to content

Allow scoped Dependabot updates through PR validation - #187

Open
yousefi-msft wants to merge 1 commit into
mainfrom
fix/failing-automated-prs
Open

yousefi-msft wants to merge 1 commit into
mainfrom
fix/failing-automated-prs

Conversation

@yousefi-msft

Copy link
Copy Markdown
Contributor

Summary

  • allow authenticated Dependabot branches to update only the protected manifests configured for their matching ecosystem
  • keep spoofed authors, mismatched branches, nested workflows, unconfigured files, and mixed-scope changes blocked by POL-021
  • document the scoped exception and add regression coverage

Why

Dependabot has repository permission none, so POL-021 classified its normal updates as public-contributor changes and rejected trusted paths. This caused PR validation failures on #171, #172, #174, #175, #176, #178, #179, #180, #182, #185, and #186.

Validation

  • python -m pytest .github/tests/test_validate_pr_pol021.py .github/tests/test_pr_review_workflow_security.py -q (34 passed)
  • python -m pytest .github/tests/test_security_automation.py -q (30 passed)
  • replayed all 11 affected PR author/head-ref/file-list combinations through run_validation (all passed)
  • python .github/scripts/generate_rule_docs.py --check

The full Windows test run still has pre-existing platform-specific failures in POL-008/POL-020 and the aka.ms probe newline test; no failures involve the changed modules.

Permit authenticated Dependabot branches to update only the protected workflow and dependency manifests configured for their matching ecosystems, while keeping mixed, spoofed, and unconfigured changes blocked by POL-021.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@github-actions

Copy link
Copy Markdown

✅ Automated Check Results — All checks passed

All structural, schema, content, documentation, and secret scan checks passed.

This PR is ready for human review. The maintainers have been automatically requested.

Reminder: 1 approval from a CODEOWNERS reviewer is required before this PR can be merged.

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

Labels

needs-human-review Awaiting human approval pr-validation-passed The pr-review workflow's validator passed. Other status checks report separately.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant