Fix phase numbering in pr-agent post-gate docs#34116
Conversation
Three references in the Fix (Phase 3) section incorrectly said 'Phase 4' instead of 'Phase 3'. Phase 4 is the Report phase. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
This is a documentation-only PR that fixes phase numbering errors in the pr-agent post-gate documentation. The file incorrectly referred to "Phase 4" in three places that should have said "Phase 3", causing confusion about which phase handles fix exploration versus final reporting.
Changes:
- Corrected three instances where "Phase 4" was incorrectly used to refer to the Fix phase, changing them to "Phase 3"
- Phase 3 is the Fix phase (explore alternatives with try-fix skill)
- Phase 4 is the Report phase (deliver final results)
| If try-fix cannot run due to environment issues, **STOP and ask the user**. Do NOT mark attempts as "BLOCKED" and continue. | ||
|
|
||
| ### 🚨 CRITICAL: Stop on Environment Blockers (Applies to Phase 4) | ||
| ### 🚨 CRITICAL: Stop on Environment Blockers (Applies to Phase 3) |
There was a problem hiding this comment.
While this PR correctly fixes phase numbering issues in this file, there are similar inconsistencies in related files that should be addressed:
-
.github/agents/pr/SHARED-RULES.mdline 130 says "Phase 4 uses these 5 AI models for try-fix exploration" but try-fix exploration happens in Phase 3 (Fix), not Phase 4 (Report). -
.github/agents/pr.mdline 52 says "Multi-Model Configuration (5 models for Phase 4)" but should reference Phase 3, as confirmed by line 60 which correctly states "Phase 3 uses a 5-model exploration workflow."
These additional inconsistencies follow the same pattern as the ones fixed in this PR and should be corrected for consistency.
<!-- Please let the below note in for people that find this PR --> > [!NOTE] > Are you waiting for the changes in this PR to be merged? > It would be very helpful if you could [test the resulting artifacts](https://github.com/dotnet/maui/wiki/Testing-PR-Builds) from this PR and let us know in a comment if this change resolves your issue. Thank you! ## Description Three references in the Fix (Phase 3) section of `.github/agents/pr/post-gate.md` incorrectly said "Phase 4" instead of "Phase 3". Phase 4 is the Report phase. ### Changes - Line 27: "Applies to Phase 4" → "Applies to Phase 3" - Line 49: "The purpose of Phase 4" → "The purpose of Phase 3" - Line 59: "Phase 4 uses a multi-model approach" → "Phase 3 uses a multi-model approach" Documentation-only change, no functional impact. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Note
Are you waiting for the changes in this PR to be merged?
It would be very helpful if you could test the resulting artifacts from this PR and let us know in a comment if this change resolves your issue. Thank you!
Description
Three references in the Fix (Phase 3) section of
.github/agents/pr/post-gate.mdincorrectly said "Phase 4" instead of "Phase 3". Phase 4 is the Report phase.Changes
Documentation-only change, no functional impact.