Improve try-fix comment parsing and summary

[kubaflo]

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!

Root Cause

The post-try-fix-comment script had several formatting issues:

1. Duplicated headers - Old "### 🔧 Try-Fix Analysis" headers accumulated when updating comments
2. Stray formatting - Horizontal rules and <br> tags accumulated between attempts
3. No summary - Users had to expand each attempt to see overall status
4. Fragile parsing - Didn't handle both old (header-based) and new (collapsible) formats

Description of Change

Refactored comment parsing to:

1. Robust Content Extraction

• Extract inner attempts area, supporting both:
  • New format: <details><summary><b>🔧 Try-Fix Analysis...</summary>...[attempts]...</details>
  • Old format: ### 🔧 Try-Fix Analysis\n\n...[attempts]...
• Strip stray headers (### 🔧...), horizontal rules (---), and <br> tags

2. Accurate Attempt Counting

• Count existing attempts by status emoji: ✅ (passed), ❌ (failed)
• Recalculate after insertion/replacement to ensure consistency
• Use counts to build summary line: "✅ 2 passed, ❌ 1 failed"

3. Clean Attempt Insertion

• Operate on inner content only (avoids wrapper duplication)
• Replace existing attempt by number, or append new
• Add newline before Approach text for proper spacing

4. Single Collapsible Section

• Wrap all attempts in one <details> with summary showing status counts
• Example: 🔧 Try-Fix Analysis: ✅ 2 passed, ❌ 1 failed
• Eliminates duplicated wrappers and headers

Key Changes

File: .github/skills/ai-summary-comment/scripts/post-try-fix-comment.ps1

Change	Before	After
Header handling	Added ### 🔧 header each time	Single collapsible with summary
Content extraction	Used entire section as-is	Extract inner attempts, strip wrappers
Counting	Not implemented	Count ✅/❌ and show in summary
Attempt matching	Matched old "Attempt N:" format	Match new "Fix N" with emoji
Approach spacing	"$Approach\n\n"	"\n$Approach\n\n" (blank line before)

Issues Fixed

None (internal improvement to pr agent infrastructure)

Testing

• Tested with existing try-fix comments (both old and new formats)
• Verified attempt counting and summary generation
• Confirmed no header/wrapper duplication

[Copilot]

Pull request overview

This PR refactors the try-fix comment logic in post-try-fix-comment.ps1 to more robustly parse and preserve existing attempts. The changes aim to improve handling of both new and old comment formats, provide better counting of attempt statuses, and create a cleaner collapsible summary section.

Changes:

• Enhanced extraction logic to support both new collapsible format and old header format
• Added counting logic to track passed/failed attempts for display in summary
• Modified attempt insertion/replacement to work on extracted inner content rather than full section
• Created a new collapsible wrapper with status summary (e.g., "✅ 2 passed, ❌ 1 failed")

[rmarinho]

📋 PR Finalization Review

Title: ✅ Good

Current: Improve try-fix comment parsing and summary

Description: ❌ Needs Rewrite

Description needs updates. See details below.

✨ Suggested PR Description

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 from this PR and let us know in a comment if this change resolves your issue. Thank you!

Root Cause

The post-try-fix-comment script had several formatting issues:

1. Duplicated headers - Old "### 🔧 Try-Fix Analysis" headers accumulated when updating comments
2. Stray formatting - Horizontal rules and <br> tags accumulated between attempts
3. No summary - Users had to expand each attempt to see overall status
4. Fragile parsing - Didn't handle both old (header-based) and new (collapsible) formats

Description of Change

Refactored comment parsing to:

1. Robust Content Extraction

• Extract inner attempts area, supporting both:
  • New format: <details><summary><b>🔧 Try-Fix Analysis...</summary>...[attempts]...</details>
  • Old format: ### 🔧 Try-Fix Analysis\n\n...[attempts]...
• Strip stray headers (### 🔧...), horizontal rules (---), and <br> tags

2. Accurate Attempt Counting

• Count existing attempts by status emoji: ✅ (passed), ❌ (failed)
• Recalculate after insertion/replacement to ensure consistency
• Use counts to build summary line: "✅ 2 passed, ❌ 1 failed"

3. Clean Attempt Insertion

• Operate on inner content only (avoids wrapper duplication)
• Replace existing attempt by number, or append new
• Add newline before Approach text for proper spacing

4. Single Collapsible Section

• Wrap all attempts in one <details> with summary showing status counts
• Example: 🔧 Try-Fix Analysis: ✅ 2 passed, ❌ 1 failed
• Eliminates duplicated wrappers and headers

Key Changes

File: .github/skills/ai-summary-comment/scripts/post-try-fix-comment.ps1

Change	Before	After
Header handling	Added ### 🔧 header each time	Single collapsible with summary
Content extraction	Used entire section as-is	Extract inner attempts, strip wrappers
Counting	Not implemented	Count ✅/❌ and show in summary
Attempt matching	Matched old "Attempt N:" format	Match new "Fix N" with emoji
Approach spacing	"$Approach\n\n"	"\n$Approach\n\n" (blank line before)

Issues Fixed

None (internal improvement to pr agent infrastructure)

Testing

• Tested with existing try-fix comments (both old and new formats)
• Verified attempt counting and summary generation
• Confirmed no header/wrapper duplication