Skip to content

Document PR ruleset patterns for enforcing agentic workflows - #49975

Merged
pelikhan merged 2 commits into
mainfrom
copilot/update-governance-docs-pr-rulesets
Aug 3, 2026
Merged

Document PR ruleset patterns for enforcing agentic workflows#49975
pelikhan merged 2 commits into
mainfrom
copilot/update-governance-docs-pr-rulesets

Conversation

Copilot AI commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

This updates governance and authoring guidance to make PR ruleset adoption predictable when injecting agentic workflow checks across repositories. It also aligns PR reviewer guidance with ruleset constraints to reduce drift and runtime failures.

  • Governance: ruleset rollout for agentic checks

    • Added a focused section to guides/governance.md on using org/repo rulesets to require agentic workflow checks.
    • Captures the operational invariants for required checks: stable workflow/job names, ruleset-relevant PR triggers, and constrained review events for bot reviewers.
  • Repository overlay instructions: ruleset-safe authoring defaults

    • Extended .github/aw/instructions.md with explicit guidance for workflows intended for ruleset enforcement:
      • keep required-check names stable
      • use inlined-imports: true when imports are present
  • PR reviewer guidance: ruleset compatibility

    • Updated .github/aw/pr-reviewer.md to call out ruleset-compatible trigger coverage (opened, synchronize, reopened, optionally ready_for_review).
    • Added a dedicated compatibility section tying reviewer behavior to required-check and import constraints.
---
on:
  pull_request:
    types: [opened, synchronize, reopened, ready_for_review]
inlined-imports: true
safe-outputs:
  submit-pull-request-review:
    allowed-events: [COMMENT, REQUEST_CHANGES]
---

Copilot AI and others added 2 commits August 3, 2026 12:12
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title Update governance docs for PR ruleset-based workflow injection Document PR ruleset patterns for enforcing agentic workflows Aug 3, 2026
Copilot AI requested a review from pelikhan August 3, 2026 12:18
@pelikhan
pelikhan marked this pull request as ready for review August 3, 2026 12:19
Copilot AI review requested due to automatic review settings August 3, 2026 12:19
@pelikhan
pelikhan merged commit 3112c2f into main Aug 3, 2026
@pelikhan
pelikhan deleted the copilot/update-governance-docs-pr-rulesets branch August 3, 2026 12:19

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Documents ruleset-safe patterns for enforcing agentic PR workflows.

Changes:

  • Adds governance guidance for required workflows and checks.
  • Updates reviewer and repository authoring guidance.
  • Registers token observability guidance with the workflow skill.
Show a summary per file
File Description
docs/src/content/docs/guides/governance.md Adds ruleset rollout recommendations.
.github/aw/pr-reviewer.md Adds ruleset compatibility guidance.
.github/aw/instructions.md Adds ruleset-safe authoring defaults.
.github/skills/agentic-workflows/SKILL.md Registers observability guidance.

Review details

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

  • Files reviewed: 4/4 changed files
  • Comments generated: 4
  • Review effort level: Balanced

Comment on lines +182 to +183
3. Trigger reviewer workflows on PR lifecycle events that matter to rulesets
(`opened`, `synchronize`, `reopened`, and optionally `ready_for_review`).
- Standard frontmatter defaults
- Frontmatter ordering/style conventions
- Security or policy constraints specific to this repository
- For workflows that will be enforced by repository or organization pull request rulesets, keep workflow/job names stable for required checks and use `inlined-imports: true` when imports are present
Comment thread .github/aw/pr-reviewer.md
Comment on lines +66 to +67
- Keep workflow and job names stable so required-check rulesets keep matching after updates.
- If imports are used, set `inlined-imports: true` to avoid runtime import failures in ruleset execution contexts.
Comment on lines +178 to +181
1. Keep the workflow `name:` and reviewer job names stable so required checks do not
drift between updates.
2. If the workflow uses imports, set `inlined-imports: true` so required-check runs
do not fail with runtime import resolution errors in ruleset contexts.
@github-actions

github-actions Bot commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

🎉 This pull request is included in a new release.

Release: v0.84.4

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