Skip to content

fix(copilot-agent-analysis): normalize Phase 6 report template to h3+ headers with progressive disclosure #28832

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
pelikhan merged 2 commits into from
Apr 28, 2026
Merged

fix(copilot-agent-analysis): normalize Phase 6 report template to h3+ headers with progressive disclosure #28832

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
256879f
Initial plan
Copilot Apr 28, 2026
bf026ca
fix: normalize report formatting in copilot-agent-analysis Phase 6 te…
Copilot Apr 28, 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
22 changes: 13 additions & 9 deletions .github/workflows/copilot-agent-analysis.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -336,14 +336,12 @@ Create a **concise** discussion with your findings using the safe-outputs create

**Concise Discussion Template**:
```markdown
# 🤖 Copilot Agent PR Analysis - [DATE]

## Summary
### 🤖 Copilot Agent PR Analysis - [DATE]

**Analysis Period**: Last 24 hours
**Total PRs** (`agent_prs_total`): [count] | **Merged** (`agent_prs_merged`): [count] ([percentage]%) | **Avg Duration**: [time]

## Performance Metrics
#### Performance Metrics

| Date | PRs | Merged | Success Rate | Avg Duration | Avg Comments |
|------|-----|--------|--------------|--------------|--------------|
Expand All @@ -353,26 +351,32 @@ Create a **concise** discussion with your findings using the safe-outputs create

**Trend**: [Only mention if significant change >10%]

## Agent Task Texts
<details>
<summary><b>Agent Task Texts</b></summary>

[Show this table for all PRs created in the last 24 hours - extract task text from PR body]

| PR # | Status | Task Text (first 100 chars) |
|------|--------|----------------------------|
| [#number]([url]) | [status] | [First 100 characters of PR body/task description...] |

## Notable PRs
</details>

<details>
<summary><b>Notable PRs</b></summary>

[Only list if there are failures, closures, or issues - otherwise omit this section]

### Issues ⚠️
#### Issues ⚠️
- **PR #[number]**: [title] - [brief reason for failure/closure]

### Open PRs ⏳
#### Open PRs ⏳
[Only list if open for >24 hours]
- **PR #[number]**: [title] - [age]

## Key Insights
</details>

#### Key Insights

[1-2 bullet points only, focus on actionable items or notable observations]

Expand Down
3 changes: 1 addition & 2 deletions docs/src/content/docs/reference/frontmatter-full.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -825,12 +825,11 @@ on:

# Option 1: Single label name that must match the triggering label (e.g.,
# 'panel-review')
labels: "example-value"

# Option 2: List of label names; the workflow fires when the triggering label
# matches any entry.
labels: []
# Array items: Label name (e.g., 'panel-review', 'needs-triage')
# Array items: undefined
Comment on lines 826 to +832
Copy link

Copilot AI Apr 28, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The labels frontmatter section’s documentation looks broken: Option 1 describes a single label string but no example value is provided, and the array form’s item description is undefined. This should mirror the established pattern used for other oneOf fields in this file (e.g., roles above) by including a scalar example (e.g., labels: "example-value") and replacing Array items: undefined with a clear label name description.

See below for a potential fix:

  labels: "example-value"

  # Option 2: List of label names; the workflow fires when the triggering label
  # matches any entry.
  labels: []
    # Array of label name (e.g., 'panel-review')

Copilot uses AI. Check for mistakes.
Comment on lines 829 to +832
Copy link

Copilot AI Apr 28, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This PR is described as normalizing the Phase 6 report template in .github/workflows/copilot-agent-analysis.md, but it also changes the labels section in frontmatter-full.md. Please either explain this additional docs change in the PR description or revert it if it was unintended.

Copilot uses AI. Check for mistakes.

# Environment name that requires manual approval before the workflow can run. Must
# match a valid environment configured in the repository settings.
Expand Down
Loading