Fix workflow port prompts - #872
Merged
Quim Muntal (qmuntal) merged 2 commits intoAug 20, 2026
Merged
Conversation
Michelle Clayton (michelle-clayton-work)
requested a review
from a team
as a code owner
August 19, 2026 20:03
Copilot started reviewing on behalf of
Michelle Clayton (michelle-clayton-work)
August 19, 2026 20:06
View session
github-actions
Bot
added
area:github
Contributor
There was a problem hiding this comment.
Pull request overview
Updates the repository’s agentic-workflow prompt docs to clarify how to classify upstream .NET changes (API/feature vs fix/test) and how parity reviewers should reason about user-visible behavior that may be implemented via unexported Go code.
Changes:
- Expanded the Go API consistency review guidance to treat observable behavior/defaults/feature gating as in-scope even when changes are in private code.
- Revised nightly porting workflow prompts to require upstream-contract-based classification (and full upstream diff inspection) before implementing ports.
- Regenerated gh-aw lock files to reflect the prompt updates.
Reviewed changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| .github/workflows/go-api-consistency-review.md | Tightens parity-review rubric around observable behavior, defaults, and upstream feature gating. |
| .github/workflows/go-api-consistency-review.lock.yml | Updates gh-aw metadata hashes after prompt edit. |
| .github/workflows/dotnet-port-fixes-nightly.md | Clarifies fixes-workflow classification rules and selection/verification steps. |
| .github/workflows/dotnet-port-fixes-nightly.lock.yml | Updates gh-aw metadata hashes after prompt edit. |
| .github/workflows/dotnet-port-api-nightly.md | Clarifies API-workflow classification rules and selection/verification steps. |
| .github/workflows/dotnet-port-api-nightly.lock.yml | Updates gh-aw metadata hashes after prompt edit. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
github-actions
Bot
added
kind:ci
Quim Muntal (qmuntal)
deleted the
dev/mclayton/fix-dotnet-port-classification
branch
August 20, 2026 05:32
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes the wording to better clarify what should be classified as api and fixes
in response to: #863 (review)