协作头脑风暴
brainstorming
在实现前澄清需求、探索设计选择,并形成清晰的开发方向。
ai-summary-comment【开发与工程】Posts or updates automated progress comments on GitHub PRs. Use after completing any PR agent phase (pre-flight, tests, gate, fix, report). Triggers on 'post comment to PR', 'update PR progress', 'comment on PR with results', 'post pre-flight comment'. Creates single aggregated review comment with collapsible sections per commit.
独立 GitHub 快照 · 自动分类 · 以原作者文档为准
This skill posts automated progress comments to GitHub Pull Requests during the PR review workflow. Comments are self-contained with collapsible Review Session details, providing rich context to maintainers and contributors.
⚠️ Self-Contained Rule: All content in PR comments must be self-contained. Never reference local files like CustomAgentLogsTmp/ - GitHub users cannot access your local filesystem.
✨ Key Features:
<!-- AI Summary --> comment and updates it-DryRun to preview changes in a local file before postingCustomAgentLogsTmp/PRState/Most scripts post to the same single comment identified by <!-- AI Summary -->. Each script updates its own section:
<!-- AI Summary -->
## 🤖 AI Summary
<!-- SECTION:PR-REVIEW -->
... PR review phases ...
<!-- /SECTION:PR-REVIEW -->
<!-- SECTION:TRY-FIX -->
... try-fix attempts ...
<!-- /SECTION:TRY-FIX -->
<!-- SECTION:WRITE-TESTS -->
... write-tests attempts ...
<!-- /SECTION:WRITE-TESTS -->
<!-- SECTION:VERIFY-TESTS -->
... test verification results ...
<!-- /SECTION:VERIFY-TESTS -->
Behavior:
The post-pr-finalize-comment.ps1 script posts a separate comment identified by <!-- PR-FINALIZE-COMMENT -->. This comment contains two sections:
If an existing finalize comment exists, it will be replaced with the updated Title and Description sections. This keeps finalization reviews distinct from automated analysis.
| Section | Script | Location |
|---|---|---|
PR-REVIEW | post-ai-summary-comment.ps1 | .github/skills/ai-summary-comment/scripts/ |
TRY-FIX | post-try-fix-comment.ps1 | .github/skills/ai-summary-comment/scripts/ |
WRITE-TESTS | post-write-tests-comment.ps1 | .github/skills/ai-summary-comment/scripts/ |
VERIFY-TESTS | post-verify-tests-comment.ps1 | .github/skills/ai-summary-comment/scripts/ |
| Comment | Script | Marker |
|---|---|---|
| PR Finalization | post-pr-finalize-comment.ps1 | <!-- PR-FINALIZE-COMMENT --> |
| Phase | Description | When to Post | What This Enables Next |
|---|---|---|---|
pre-flight | Context gathering complete | After documenting issue, files, and discussion | Tests Phase: Agent can now verify/create test files that reproduce the bug |
tests | Test analysis complete | After identifying test files and coverage | Gate Phase: Agent can run tests to verify they catch the bug |
gate | Test validation complete | After running tests and verifying bug reproduction | Fix Phase: Agent can explore alternative fixes (tests proven to catch bug) |
fix | Solution comparison complete | After comparing PR fix with alternatives | Report Phase: Agent can finalize recommendation based on fix comparison |
report | Final analysis complete | After generating comprehensive review | PR Decision: Maintainers can approve/merge or request changes based on full analysis |
# Auto-loads CustomAgentLogsTmp/PRState/pr-27246.md
pwsh .github/skills/ai-summary-comment/scripts/post-ai-summary-comment.ps1 -PRNumber 27246
# PRNumber auto-extracted from filename (pr-27246.md → 27246)
pwsh .github/skills/ai-summary-comment/scripts/post-ai-summary-comment.ps1 -StateFile CustomAgentLogsTmp/PRState/pr-27246.md
pwsh .github/skills/ai-summary-comment/scripts/post-ai-summary-comment.ps1 -PRNumber 12345 -Content "$(cat CustomAgentLogsTmp/PRState/pr-12345.md)"
| Parameter | Required | Description | Example |
|---|---|---|---|
PRNumber | No* | Pull request number | 12345 |
StateFile | No* | Path to state file (PRNumber auto-extracted from pr-XXXXX.md naming) | CustomAgentLogsTmp/PRState/pr-27246.md |
Content | No* | Full state file content (legacy, can be piped via stdin) | Content from state file |
DryRun | No | Preview changes in local file instead of posting to GitHub | -DryRun |
PreviewFile | No | Path to local preview file for DryRun mode (default: CustomAgentLogsTmp/PRState/{PRNumber}/ai-summary-comment-preview.md) | -PreviewFile ./preview.md |
SkipValidation | No | Skip validation checks (not recommended) | -SkipValidation |
*At least one of PRNumber, StateFile, or Content is required. The script will:
-PRNumber provided: Auto-load CustomAgentLogsTmp/PRState/pr-{PRNumber}.md-StateFile provided: Load the file and extract PRNumber from pr-XXXXX.md filename-Content provided: Use content directly (legacy, requires -PRNumber)Use -DryRun to preview the combined comment before posting to GitHub. Each script updates the same preview file, mirroring how the actual GitHub comment is updated.
# Step 1: Run verify-tests script (creates preview file)
pwsh .github/skills/ai-summary-comment/scripts/post-verify-tests-comment.ps1 -PRNumber 32891 -DryRun
# Step 2: Run try-fix script (updates same preview file)
pwsh .github/skills/ai-summary-comment/scripts/post-try-fix-comment.ps1 -IssueNumber 32891 -DryRun
# Step 3: Review the combined preview
open CustomAgentLogsTmp/PRState/32891/ai-summary-comment-preview.md
# Step 4: Post for real (remove -DryRun)
pwsh .github/skills/ai-summary-comment/scripts/post-verify-tests-comment.ps1 -PRNumber 32891
pwsh .github/skills/ai-summary-comment/scripts/post-try-fix-comment.ps1 -IssueNumber 32891
Key behavior: The preview file exactly matches what will be posted to GitHub. Multiple scripts accumulate their sections in the same file.
Sections appear in the unified comment in this order (based on which scripts run first):
Each section is wrapped with markers like <!-- SECTION:TRY-FIX --> and <!-- /SECTION:TRY-FIX -->.
To reset the preview file for a fresh start:
rm CustomAgentLogsTmp/PRState/{PRNumber}/ai-summary-comment-preview.md
Scripts require GitHub CLI authentication:
gh auth status # Verify authentication before running
Comments are formatted with:
When the same PR is reviewed multiple times (e.g., after new commits), the script updates the single aggregated review comment and adds a new expandable section for each commit-based review session.
## 🔍 Pre-Flight: Context Gathering Complete
✅ **Status**: Phase completed successfully
### Summary
- **Issue**: #33356 - CollectionView crash on iOS
- **Platforms Affected**: iOS, MacCatalyst
- **Files Changed**: 2 implementation files, 1 test file
- **Discussion**: 3 key reviewer comments identified
### Key Findings
- Crash occurs when scrolling rapidly with large datasets
- Existing PR adds null check in ItemsViewController
- Test coverage includes iOS device test
### Next Steps
→ **Phase 2: Tests** - Analyzing test files and coverage
---
*Posted by PR Agent @ 2026-01-17 14:23:45 UTC*
post-ai-summary-comment.ps1 - Posts or updates the aggregated PR agent review commentpost-try-fix-comment.ps1 - Posts or updates try-fix attempts commentThe post-try-fix-comment.ps1 script updates the <!-- SECTION:TRY-FIX --> section of the unified AI Summary comment. It aggregates all try-fix attempts into collapsible sections. Works for both issues and PRs (GitHub treats PR comments as issue comments).
✨ Auto-Loading from CustomAgentLogsTmp: The script automatically discovers and aggregates ALL attempt directories from CustomAgentLogsTmp/PRState/{IssueNumber}/try-fix/.
# All parameters auto-loaded from directory structure
pwsh .github/skills/ai-summary-comment/scripts/post-try-fix-comment.ps1 `
-TryFixDir CustomAgentLogsTmp/PRState/27246/try-fix/attempt-1
# Auto-discovers and posts latest attempt from CustomAgentLogsTmp/PRState/27246/try-fix/
pwsh .github/skills/ai-summary-comment/scripts/post-try-fix-comment.ps1 -IssueNumber 27246
pwsh .github/skills/ai-summary-comment/scripts/post-try-fix-comment.ps1 `
-IssueNumber 19806 `
-AttemptNumber 1 `
-Approach "LayoutExtensions Width Constraint" `
-RootCause "ComputeFrame only constrains width for Fill alignment" `
-FilesChanged "| File | Changes |`n|------|---------|`n| LayoutExtensions.cs | +17/-3 |" `
-Status "Compiles" `
-CodeSnippet "else if (!hasExplicitWidth) { ... }" `
-Analysis "Core project compiles successfully"
| Parameter | Required | Description |
|---|---|---|
TryFixDir | No* | Path to try-fix attempt directory (auto-loads all parameters) |
IssueNumber | No* | Issue or PR number to post comment on |
AttemptNumber | No* | Attempt number (1, 2, 3, etc.) - auto-detected from TryFixDir |
Approach | No* | Brief description of fix approach |
RootCause | No | Description of root cause identified |
FilesChanged | No* | Markdown table of files changed - auto-generated from diff |
Status | No* | "Compiles", "Pass", or "Fail" - loaded from result.txt |
CodeSnippet | No | Code snippet showing the fix - loaded from fix.diff |
Analysis | No | Analysis of why it worked/failed - loaded from analysis.md |
DryRun | No | Print comment instead of posting |
*When using -TryFixDir, all marked parameters are auto-loaded from files in the directory.
CustomAgentLogsTmp/PRState/{IssueNumber}/try-fix/
├── attempt-1/
│ ├── approach.md # Brief description of the approach (required)
│ ├── result.txt # "Pass", "Fail", or "Compiles" (required)
│ ├── fix.diff # Git diff of the fix (optional)
│ └── analysis.md # Detailed analysis (optional)
├── attempt-2/
│ └── ...
└── attempt-3/
└── ...
## 🔧 Try-Fix Attempts for Issue #XXXXX
<!-- TRY-FIX-COMMENT -->
<details>
<summary>📊 <strong>Expand Full Details</strong></summary>
**Issue:** [#XXXXX](link)
---
<details>
<summary><strong>🔧 Attempt #1: Approach Name</strong> ✅ Status</summary>
... attempt details ...
</details>
---
*This fix was developed independently.*
</details>
<!-- TRY-FIX-COMMENT --> markerThe post-verify-tests-comment.ps1 script updates the <!-- SECTION:VERIFY-TESTS --> section of the unified AI Summary comment. It documents test verification results (whether tests fail without fix and pass with fix).
✨ Auto-Loading from CustomAgentLogsTmp: The script automatically loads verification results from CustomAgentLogsTmp/PRState/{PRNumber}/verify-tests-fail/verification-report.md.
# Auto-loads from CustomAgentLogsTmp/PRState/{PRNumber}/verify-tests-fail/
pwsh .github/skills/ai-summary-comment/scripts/post-verify-tests-comment.ps1 -PRNumber 32891
pwsh .github/skills/ai-summary-comment/scripts/post-verify-tests-comment.ps1 `
-PRNumber 32891 `
-ReportFile CustomAgentLogsTmp/PRState/32891/verify-tests-fail/verification-report.md
| Parameter | Required | Description |
|---|---|---|
PRNumber | Yes | Pull request number |
ReportFile | No | Path to verification report (auto-discovered if not provided) |
Status | No | "Passed" or "Failed" - auto-detected from report |
Platform | No | Platform tested (ios, android, etc.) - auto-detected from report |
Mode | No | "FailureOnly" or "FullVerification" - auto-detected from report |
DryRun | No | Preview changes in local file instead of posting |
PreviewFile | No | Path to local preview file for DryRun mode |
CustomAgentLogsTmp/PRState/{PRNumber}/verify-tests-fail/
├── verification-report.md # Full verification report (required)
├── verification-log.txt # Detailed log (optional)
├── test-without-fix.log # Test output without fix (optional)
└── test-with-fix.log # Test output with fix (optional)
The post-write-tests-comment.ps1 script updates the <!-- SECTION:WRITE-TESTS --> section of the unified AI Summary comment. It documents test writing attempts for an issue.
✨ Auto-Loading from CustomAgentLogsTmp: The script can automatically load test details from the write-tests output directory structure.
# All parameters auto-loaded from directory structure
pwsh .github/skills/ai-summary-comment/scripts/post-write-tests-comment.ps1 `
-TestDir CustomAgentLogsTmp/PRState/27246/write-tests/attempt-1
# Auto-discovers and posts latest attempt from CustomAgentLogsTmp/PRState/27246/write-tests/
pwsh .github/skills/ai-summary-comment/scripts/post-write-tests-comment.ps1 -IssueNumber 27246
pwsh .github/skills/ai-summary-comment/scripts/post-write-tests-comment.ps1 `
-IssueNumber 33331 `
-AttemptNumber 1 `
-TestDescription "Verifies Picker.IsOpen property changes correctly" `
-HostAppFile "src/Controls/tests/TestCases.HostApp/Issues/Issue33331.cs" `
-TestFile "src/Controls/tests/TestCases.Shared.Tests/Tests/Issues/Issue33331.cs" `
-TestMethod "PickerIsOpenPropertyChanges" `
-Category "Picker" `
-VerificationStatus "Verified"
| Parameter | Required | Description |
|---|---|---|
TestDir | No* | Path to write-tests attempt directory (auto-loads all parameters) |
IssueNumber | No* | Issue or PR number to post comment on |
AttemptNumber | No* | Attempt number (1, 2, 3, etc.) - auto-detected from TestDir |
TestDescription | No* | Brief description of what the test verifies |
HostAppFile | No* | Path to the HostApp test page file |
TestFile | No* | Path to the NUnit test file |
TestMethod | No* | Name of the test method |
Category | No* | UITestCategories category used |
VerificationStatus | No* | "Verified", "Failed", or "Unverified" - loaded from result.txt |
Platforms | No | Platforms the test runs on (default: "All") |
Notes | No | Additional notes - loaded from notes.md |
DryRun | No | Print comment instead of posting |
*When using -TestDir, all marked parameters are auto-loaded from files in the directory.
CustomAgentLogsTmp/PRState/{IssueNumber}/write-tests/
├── attempt-1/
│ ├── description.md # Brief test description (required)
│ ├── test-info.json # {HostAppFile, TestFile, TestMethod, Category} (required)
│ ├── result.txt # "Verified", "Pass", "Failed", or "Unverified" (required)
│ └── notes.md # Additional notes (optional)
├── attempt-2/
│ └── ...
└── attempt-3/
└── ...
{
"HostAppFile": "src/Controls/tests/TestCases.HostApp/Issues/Issue27246.cs",
"TestFile": "src/Controls/tests/TestCases.Shared.Tests/Tests/Issues/Issue27246.cs",
"TestMethod": "ScrollToFirstItemWithHeader",
"Category": "CollectionView"
}
<!-- AI Summary -->gh api for create/update operations