Skip to content

allow subagent model selection#461

Merged
strawgate merged 5 commits intomainfrom
add-copilot-subagent
Feb 28, 2026
Merged

allow subagent model selection#461
strawgate merged 5 commits intomainfrom
add-copilot-subagent

Conversation

@strawgate
Copy link
Collaborator

@strawgate strawgate commented Feb 28, 2026

Summary

  • Add a Configure Copilot CLI settings step in .github/workflows/gh-aw-fragments/runtime-setup.md so each run creates ~/.copilot and ensures $HOME/.copilot/config.json contains "chat.customAgentInSubagent.enabled": true (merge with jq if the file already exists, create it otherwise).
  • Update PR-context preparation in .github/workflows/gh-aw-fragments/pr-context.md to precompute three changed-file orderings (A→Z, Z→A, and largest diff first) and document those artifacts in /tmp/pr-context/README.md.
  • Update .github/workflows/gh-aw-pr-review.md to consume the new precomputed ordering files instead of recomputing orderings in sub-agent instructions.
  • Update PR Review Addresser workflow sources for safer reruns and bot handling:
    • add workflow-level concurrency (pr-review-addresser-$\{\{ github.event.pull_request.number }} with cancel-in-progress: true) in gh-agent-workflows/pr-review-addresser/example.yml, and
    • add allowed-bot-users: "coderabbitai[bot],github-actions[bot],copilot" in gh-agent-workflows/pr-review-addresser/dogfood-with.yml.
  • Update .github/workflows/trigger-pr-review-addresser.yml to mirror those source changes by adding the same concurrency group and passing allowed-bot-users to gh-aw-pr-review-addresser.lock.yml.
  • Recompile workflow lock files so source and generated workflows stay in sync.

Validation

make compile
make lint

Generated by Update PR Body for issue #461

@coderabbitai
Copy link

coderabbitai bot commented Feb 28, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 44917fc and 0e74495.

📒 Files selected for processing (7)
  • .github/workflows/gh-aw-fragments/pr-context.md
  • .github/workflows/gh-aw-mention-in-pr-by-id.lock.yml
  • .github/workflows/gh-aw-mention-in-pr-no-sandbox.lock.yml
  • .github/workflows/gh-aw-mention-in-pr.lock.yml
  • .github/workflows/gh-aw-pr-review-addresser.lock.yml
  • .github/workflows/gh-aw-pr-review.lock.yml
  • .github/workflows/gh-aw-pr-review.md
🚧 Files skipped from review as they are similar to previous changes (2)
  • .github/workflows/gh-aw-mention-in-pr.lock.yml
  • .github/workflows/gh-aw-pr-review-addresser.lock.yml

📝 Walkthrough

Walkthrough

A new GitHub Actions step "Configure Copilot CLI settings" was added across many workflows. The step ensures ~/.copilot exists and creates or updates ~/.copilot/config.json, setting chat.customAgentInSubagent.enabled to true (merging into an existing JSON via jq if present, otherwise writing a new file). No exported or public API declarations were changed.

Possibly related PRs

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch add-copilot-subagent

Comment @coderabbitai help to get the list of available commands and usage tips.

coderabbitai[bot]

This comment was marked as resolved.

github-actions[bot]

This comment was marked as resolved.

strawgate and others added 2 commits February 27, 2026 20:25
The run block content was not indented under `run: |`, making the YAML
invalid. The mv command was also split across two lines. Added shell:
bash and set -euo pipefail for consistency with other steps.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)
.github/workflows/trigger-pr-review-addresser.yml (1)

1-2: ⚠️ Potential issue | 🟡 Minor

Regenerate this auto-generated workflow before merge.

Line 1 and Line 2 mark this as generated, and CI already reported a modified file after make compile. Please re-run compile and commit regenerated artifacts to keep generated workflows deterministic.

🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In @.github/workflows/trigger-pr-review-addresser.yml around lines 1 - 2, The
workflow file .github/workflows/trigger-pr-review-addresser.yml is out of date
because it is generated; regenerate the artifact by running the generation step
and committing the result: open the source template
gh-agent-workflows/pr-review-addresser/example.yml (or update it if needed), run
the generator (scripts/dogfood.sh or simply run make compile as the repo
expects), verify the regenerated
.github/workflows/trigger-pr-review-addresser.yml matches CI expectations, and
commit the regenerated file so the auto-generated header and contents are
deterministic.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Outside diff comments:
In @.github/workflows/trigger-pr-review-addresser.yml:
- Around line 1-2: The workflow file
.github/workflows/trigger-pr-review-addresser.yml is out of date because it is
generated; regenerate the artifact by running the generation step and committing
the result: open the source template
gh-agent-workflows/pr-review-addresser/example.yml (or update it if needed), run
the generator (scripts/dogfood.sh or simply run make compile as the repo
expects), verify the regenerated
.github/workflows/trigger-pr-review-addresser.yml matches CI expectations, and
commit the regenerated file so the auto-generated header and contents are
deterministic.

ℹ️ Review info

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between c1b96e9 and 008489f.

📒 Files selected for processing (60)
  • .github/workflows/agent-deep-dive.lock.yml
  • .github/workflows/agent-efficiency.lock.yml
  • .github/workflows/downstream-users.lock.yml
  • .github/workflows/gh-aw-agent-suggestions.lock.yml
  • .github/workflows/gh-aw-autonomy-atomicity-analyzer.lock.yml
  • .github/workflows/gh-aw-branch-actions-detective.lock.yml
  • .github/workflows/gh-aw-breaking-change-detect.lock.yml
  • .github/workflows/gh-aw-breaking-change-detector.lock.yml
  • .github/workflows/gh-aw-bug-exterminator.lock.yml
  • .github/workflows/gh-aw-bug-hunter.lock.yml
  • .github/workflows/gh-aw-code-duplication-detector.lock.yml
  • .github/workflows/gh-aw-code-duplication-fixer.lock.yml
  • .github/workflows/gh-aw-code-simplifier.lock.yml
  • .github/workflows/gh-aw-deep-research.lock.yml
  • .github/workflows/gh-aw-dependency-review.lock.yml
  • .github/workflows/gh-aw-docs-drift.lock.yml
  • .github/workflows/gh-aw-docs-patrol.lock.yml
  • .github/workflows/gh-aw-estc-actions-resource-not-accessible-detector.lock.yml
  • .github/workflows/gh-aw-estc-docs-patrol-external.lock.yml
  • .github/workflows/gh-aw-estc-docs-pr-review.lock.yml
  • .github/workflows/gh-aw-estc-downstream-health.lock.yml
  • .github/workflows/gh-aw-estc-newbie-contributor-patrol-external.lock.yml
  • .github/workflows/gh-aw-estc-pr-buildkite-detective.lock.yml
  • .github/workflows/gh-aw-flaky-test-investigator.lock.yml
  • .github/workflows/gh-aw-fragments/runtime-setup.md
  • .github/workflows/gh-aw-framework-best-practices.lock.yml
  • .github/workflows/gh-aw-information-architecture.lock.yml
  • .github/workflows/gh-aw-issue-fixer.lock.yml
  • .github/workflows/gh-aw-issue-triage.lock.yml
  • .github/workflows/gh-aw-mention-in-issue-no-sandbox.lock.yml
  • .github/workflows/gh-aw-mention-in-issue.lock.yml
  • .github/workflows/gh-aw-mention-in-pr-by-id.lock.yml
  • .github/workflows/gh-aw-mention-in-pr-no-sandbox.lock.yml
  • .github/workflows/gh-aw-mention-in-pr.lock.yml
  • .github/workflows/gh-aw-newbie-contributor-fixer.lock.yml
  • .github/workflows/gh-aw-newbie-contributor-patrol.lock.yml
  • .github/workflows/gh-aw-performance-profiler.lock.yml
  • .github/workflows/gh-aw-plan.lock.yml
  • .github/workflows/gh-aw-pr-actions-detective.lock.yml
  • .github/workflows/gh-aw-pr-actions-fixer.lock.yml
  • .github/workflows/gh-aw-pr-ci-detective.lock.yml
  • .github/workflows/gh-aw-pr-review-addresser.lock.yml
  • .github/workflows/gh-aw-pr-review.lock.yml
  • .github/workflows/gh-aw-product-manager-impersonator.lock.yml
  • .github/workflows/gh-aw-project-summary.lock.yml
  • .github/workflows/gh-aw-refactor-opportunist.lock.yml
  • .github/workflows/gh-aw-release-update.lock.yml
  • .github/workflows/gh-aw-scheduled-audit.lock.yml
  • .github/workflows/gh-aw-scheduled-fix.lock.yml
  • .github/workflows/gh-aw-small-problem-fixer.lock.yml
  • .github/workflows/gh-aw-stale-issues.lock.yml
  • .github/workflows/gh-aw-test-improvement.lock.yml
  • .github/workflows/gh-aw-test-improver.lock.yml
  • .github/workflows/gh-aw-text-auditor.lock.yml
  • .github/workflows/gh-aw-text-beautifier.lock.yml
  • .github/workflows/gh-aw-update-pr-body.lock.yml
  • .github/workflows/gh-aw-ux-design-patrol.lock.yml
  • .github/workflows/trigger-pr-review-addresser.yml
  • .github/workflows/upgrade-check.lock.yml
  • .github/workflows/workflow-patrol.lock.yml
🚧 Files skipped from review as they are similar to previous changes (1)
  • .github/workflows/gh-aw-fragments/runtime-setup.md

strawgate and others added 2 commits February 27, 2026 20:36
Add concurrency group to example.yml and create dogfood-with.yml so
that make compile preserves the overrides instead of losing them.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Generate file_order_az.txt, file_order_za.txt, and file_order_largest.txt
in the pr-context step so the parent agent can point each sub-agent at a
file instead of computing orderings at runtime.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@strawgate strawgate merged commit d50f098 into main Feb 28, 2026
18 checks passed
@strawgate strawgate deleted the add-copilot-subagent branch February 28, 2026 03:09
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.

1 participant