Skip to content

Improve aspire-upgrade agentic workflow#1156

Merged
aaronpowell merged 2 commits intomainfrom
upgrade-workflow
Mar 13, 2026
Merged

Improve aspire-upgrade agentic workflow#1156
aaronpowell merged 2 commits intomainfrom
upgrade-workflow

Conversation

@aaronpowell
Copy link
Member

Summary

Improves the aspire-upgrade agentic workflow with better security, reliability, and prompt clarity.

Changes

  • Add description field for workflow discoverability
  • Add concurrency: aspire-upgrade to prevent conflicting parallel PRs
  • Remove unnecessary permissions (issues: read, pull-requests: read)
  • Narrow network allowlist to only defaults (removed python, node, go, java ecosystems)
  • Narrow GitHub toolsets from [default] to [repos, context] (only what's needed)
  • Add explicit Step 2 instructing agent to find and update all ~50 AppHost .csproj files (previously easy to miss)
  • Add validation step (dotnet restore) before PR creation to catch version resolution errors
  • Restructure prompt into clear numbered steps for better agent execution
  • Remove redundant engine: copilot (it's the default)

- Add description field for discoverability
- Add concurrency control to prevent conflicting PRs
- Remove unnecessary permissions (issues: read, pull-requests: read)
- Narrow network allowlist to only defaults (removed python/node/go/java)
- Narrow GitHub toolsets to [repos, context] instead of [default]
- Add explicit instructions to find and update all ~50 AppHost .csproj files
- Add dotnet restore validation step before PR creation
- Restructure prompt into clear numbered steps
- Remove redundant engine: copilot (it's the default)

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings March 12, 2026 23:46
@github-actions
Copy link
Contributor

github-actions bot commented Mar 12, 2026

🚀 Dogfood this PR with:

⚠️ WARNING: Do not do this without first carefully reviewing the code of this PR to satisfy yourself it is safe.

curl -fsSL https://raw.githubusercontent.com/CommunityToolkit/Aspire/main/eng/scripts/dogfood-pr.sh | bash -s -- 1156

Or

  • Run remotely in PowerShell:
iex "& { $(irm https://raw.githubusercontent.com/CommunityToolkit/Aspire/main/eng/scripts/dogfood-pr.ps1) } 1156"

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

Improves the repo’s aspire-upgrade GitHub Agentic Workflow (gh-aw) setup by tightening configuration (permissions/toolsets/network), improving prompt clarity for version upgrades, and adding supporting agent/workflow metadata files.

Changes:

  • Add gh-aw/Copilot support scaffolding (MCP config, Copilot setup workflow, actions lockfile, dispatcher agent definition).
  • Update aspire-upgrade agentic workflow prompt/metadata (description, concurrency, narrowed toolsets/network) and regenerate the compiled .lock.yml.
  • Mark generated gh-aw lock workflows as linguist-generated (and configure merge behavior).

Reviewed changes

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

Show a summary per file
File Description
.vscode/mcp.json Adds local MCP server configuration for gh aw mcp-server.
.github/workflows/copilot-setup-steps.yml Adds a setup workflow to install the gh-aw CLI extension for Copilot Agent runs.
.github/workflows/aspire-upgrade.md Refactors and clarifies the agentic prompt (explicit steps + restore validation).
.github/workflows/aspire-upgrade.lock.yml Regenerated compiled workflow output from the .md source.
.github/aw/actions-lock.json Adds action SHA lock entries for gh-aw related actions.
.github/agents/agentic-workflows.agent.md Adds a dispatcher agent doc for working with gh-aw workflows and prompts.
.gitattributes Marks .lock.yml workflows as generated and configures merge behavior.

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

# this only affects the repo's language statistics
*.h linguist-language=C

.github/workflows/*.lock.yml linguist-generated=true merge=ours No newline at end of file
@@ -0,0 +1,12 @@
{
"servers": {
"github-agentic-workflows": {
@aaronpowell aaronpowell merged commit af75f26 into main Mar 13, 2026
116 of 124 checks passed
@aaronpowell aaronpowell deleted the upgrade-workflow branch March 13, 2026 00:46
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.

2 participants