Skip to content

Allow Q workflow to modify .github/workflows/** via allowed-files#21067

Closed
Copilot wants to merge 2 commits intomainfrom
copilot/aw-fix-q-failed-issue
Closed

Allow Q workflow to modify .github/workflows/** via allowed-files#21067
Copilot wants to merge 2 commits intomainfrom
copilot/aw-fix-q-failed-issue

Conversation

Copy link
Contributor

Copilot AI commented Mar 15, 2026

Q was blocked from creating PRs that touch .github/workflows/ files (e.g. daily-workflow-updater.md) because the protected-files check denied the push.

Changes

  • q.md: Added allowed-files: [.github/workflows/**] to the create-pull-request safe-output config, giving Q explicit permission to modify workflow files in its PRs.
safe-outputs:
  create-pull-request:
    allowed-files:
      - .github/workflows/**

Copilot AI and others added 2 commits March 15, 2026 12:58
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI linked an issue Mar 15, 2026 that may be closed by this pull request
@pelikhan pelikhan marked this pull request as ready for review March 15, 2026 13:25
Copilot AI review requested due to automatic review settings March 15, 2026 13:25
@pelikhan pelikhan closed this Mar 15, 2026
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adjusts the Q agent workflow safe-output configuration to permit workflow-file updates by explicitly allowlisting .github/workflows/** for create-pull-request.

Changes:

  • Added allowed-files: [.github/workflows/**] to Q’s create-pull-request safe-output configuration.
  • Regenerated the compiled workflow (q.lock.yml) to reflect the updated safe-outputs handler config/metadata.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File Description
.github/workflows/q.md Adds an allowed-files allowlist under safe-outputs.create-pull-request to scope Q’s PR edits to workflow files.
.github/workflows/q.lock.yml Updates the generated workflow output to include allowed_files in GH_AW_SAFE_OUTPUTS_HANDLER_CONFIG and refresh metadata hash.

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

You can also share your feedback on Copilot code review. Take the survey.

@@ -35,6 +35,8 @@ safe-outputs:
reviewers: copilot
draft: false
if-no-changes: "ignore"
Comment on lines +38 to +39
allowed-files:
- .github/workflows/**
GITHUB_SERVER_URL: ${{ github.server_url }}
GITHUB_API_URL: ${{ github.api_url }}
GH_AW_SAFE_OUTPUTS_HANDLER_CONFIG: "{\"add_comment\":{\"max\":1},\"create_pull_request\":{\"draft\":false,\"expires\":48,\"if_no_changes\":\"ignore\",\"labels\":[\"automation\",\"workflow-optimization\"],\"max\":1,\"max_patch_size\":1024,\"protected_files\":[\"package.json\",\"bun.lockb\",\"bunfig.toml\",\"deno.json\",\"deno.jsonc\",\"deno.lock\",\"global.json\",\"NuGet.Config\",\"Directory.Packages.props\",\"mix.exs\",\"mix.lock\",\"go.mod\",\"go.sum\",\"stack.yaml\",\"stack.yaml.lock\",\"pom.xml\",\"build.gradle\",\"build.gradle.kts\",\"settings.gradle\",\"settings.gradle.kts\",\"gradle.properties\",\"package-lock.json\",\"yarn.lock\",\"pnpm-lock.yaml\",\"npm-shrinkwrap.json\",\"requirements.txt\",\"Pipfile\",\"Pipfile.lock\",\"pyproject.toml\",\"setup.py\",\"setup.cfg\",\"Gemfile\",\"Gemfile.lock\",\"uv.lock\",\"AGENTS.md\"],\"protected_path_prefixes\":[\".github/\",\".agents/\"],\"reviewers\":[\"copilot\"],\"title_prefix\":\"[q] \"},\"missing_data\":{},\"missing_tool\":{}}"
GH_AW_SAFE_OUTPUTS_HANDLER_CONFIG: "{\"add_comment\":{\"max\":1},\"create_pull_request\":{\"allowed_files\":[\".github/workflows/**\"],\"draft\":false,\"expires\":48,\"if_no_changes\":\"ignore\",\"labels\":[\"automation\",\"workflow-optimization\"],\"max\":1,\"max_patch_size\":1024,\"protected_files\":[\"package.json\",\"bun.lockb\",\"bunfig.toml\",\"deno.json\",\"deno.jsonc\",\"deno.lock\",\"global.json\",\"NuGet.Config\",\"Directory.Packages.props\",\"mix.exs\",\"mix.lock\",\"go.mod\",\"go.sum\",\"stack.yaml\",\"stack.yaml.lock\",\"pom.xml\",\"build.gradle\",\"build.gradle.kts\",\"settings.gradle\",\"settings.gradle.kts\",\"gradle.properties\",\"package-lock.json\",\"yarn.lock\",\"pnpm-lock.yaml\",\"npm-shrinkwrap.json\",\"requirements.txt\",\"Pipfile\",\"Pipfile.lock\",\"pyproject.toml\",\"setup.py\",\"setup.cfg\",\"Gemfile\",\"Gemfile.lock\",\"uv.lock\",\"AGENTS.md\"],\"protected_path_prefixes\":[\".github/\",\".agents/\"],\"reviewers\":[\"copilot\"],\"title_prefix\":\"[q] \"},\"missing_data\":{},\"missing_tool\":{}}"
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.

[aw] Q failed

3 participants