Skip to content

Improve try-fix comment parsing and summary #33794

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
PureWeen merged 2 commits into from
Jan 31, 2026
Merged

Improve try-fix comment parsing and summary #33794

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
f5c751c
Improve try-fix comment parsing and summary
kubaflo Jan 30, 2026
33cad42
Address Copilot review suggestions
kubaflo Jan 30, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
68 changes: 56 additions & 12 deletions .github/skills/ai-summary-comment/scripts/post-try-fix-comment.ps1
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -261,7 +261,6 @@ $statusEmoji = switch ($Status) {
}

# Build the new attempt section - compact format
# Note: blank line after </summary> is required for proper markdown rendering
$attemptSection = @"
<details>
<summary>$statusEmoji Fix $AttemptNumber</summary>
Expand All @@ -270,7 +269,7 @@ $attemptSection = @"

# Show brief approach description
if (-not [string]::IsNullOrWhiteSpace($Approach)) {
$attemptSection += "$Approach`n`n"
$attemptSection += "`n$Approach`n`n"
}

# Only show diff if available
Expand Down Expand Up @@ -326,7 +325,10 @@ try {
}

# Build the try-fix section content
$tryFixHeader = "### 🔧 Try-Fix Analysis`n`n"
# Count existing attempts to show in summary
$existingAttemptCount = 0
$passCount = 0
$failCount = 0

# Extract existing try-fix section to preserve previous attempts
$existingTryFixContent = ""
Expand All @@ -336,22 +338,64 @@ if ($existingBody -match "(?s)$startPattern(.*?)$endPattern") {
$existingTryFixContent = $Matches[1].Trim()
}

# Extract just the inner attempt details (strip outer wrapper and headers)
$innerAttempts = ""
if ($existingTryFixContent -match '(?s)<details>\s*<summary><b>🔧 Try-Fix Analysis.*?</summary>\s*(.*?)\s*</details>\s*$') {
# New format - extract content inside the outer details
$innerAttempts = $Matches[1].Trim()
} elseif ($existingTryFixContent -match '(?s)### 🔧.*?`n`n(.*)') {
# Old header format
$innerAttempts = $Matches[1].Trim()
} else {
# Just use as-is but strip any stray headers
$innerAttempts = $existingTryFixContent -replace "(?s)^### 🔧[^\n]*\n+", ""
}

# Strip any leading horizontal rules, <br> tags, or whitespace before the first <details>
$innerAttempts = $innerAttempts -replace "(?s)^\s*---\s*\n+", ""
$innerAttempts = $innerAttempts -replace "(?s)^(<br>\s*)+", ""
$innerAttempts = $innerAttempts.TrimStart()

# Count existing attempts (only count inner <details> that are Fix attempts)
$existingAttemptCount = ([regex]::Matches($innerAttempts, '<details>\s*<summary>[✅❌🔨⚪]')).Count
$passCount = ([regex]::Matches($innerAttempts, '<details>\s*<summary>✅')).Count
$failCount = ([regex]::Matches($innerAttempts, '<details>\s*<summary>❌')).Count

# Check if this attempt number already exists and replace it, or add new
# Match both old format (Attempt N:) and new format (Fix N)
$attemptPattern = "(?s)<details>\s*<summary>.*?(Attempt $AttemptNumber`:|Fix $AttemptNumber).*?</details>"
if ($existingTryFixContent -match $attemptPattern) {
$attemptPattern = "(?s)<details>\s*<summary>[✅❌🔨⚪]\s*Fix $AttemptNumber</summary>.*?</details>"
if ($innerAttempts -match $attemptPattern) {
Write-Host "Replacing existing Fix $AttemptNumber..." -ForegroundColor Yellow
$tryFixContent = $existingTryFixContent -replace $attemptPattern, $attemptSection
} elseif (-not [string]::IsNullOrWhiteSpace($existingTryFixContent)) {
$tryFixInnerContent = $innerAttempts -replace $attemptPattern, $attemptSection
} elseif (-not [string]::IsNullOrWhiteSpace($innerAttempts)) {
Write-Host "Adding new Fix $AttemptNumber..." -ForegroundColor Yellow
# Remove header if present to avoid duplication
$existingTryFixContent = $existingTryFixContent -replace "^### 🔧 (Try-Fix Analysis|Fix Attempts)\s*`n*", ""
$tryFixContent = $tryFixHeader + $existingTryFixContent.TrimEnd() + "`n`n" + $attemptSection
$tryFixInnerContent = $innerAttempts.TrimEnd() + "`n`n" + $attemptSection
} else {
Write-Host "Creating first fix..." -ForegroundColor Yellow
$tryFixContent = $tryFixHeader + $attemptSection
$tryFixInnerContent = $attemptSection
}

# Recalculate attempt statistics from the final content to ensure consistency
$totalAttemptCount = ([regex]::Matches($tryFixInnerContent, '<details>\s*<summary>[✅❌🔨⚪]')).Count
$passCount = ([regex]::Matches($tryFixInnerContent, '<details>\s*<summary>✅')).Count
$failCount = ([regex]::Matches($tryFixInnerContent, '<details>\s*<summary>❌')).Count

# Build summary line with counts
$summaryStatus = if ($passCount -gt 0) { "✅ $passCount passed" } else { "" }
if ($failCount -gt 0) {
if ($summaryStatus -ne "") { $summaryStatus += ", " }
$summaryStatus += "❌ $failCount failed"
}
if ($summaryStatus -eq "") { $summaryStatus = "$totalAttemptCount attempt(s)" }

# Wrap everything in a single collapsible section
$tryFixContent = @"
<details>
<summary><b>🔧 Try-Fix Analysis: $summaryStatus</b></summary><br>
$tryFixInnerContent

</details>
"@

# Build the section with markers
$tryFixSection = @"
$SECTION_START
Expand Down