Skip to content

[workflow-style] Normalize structural analysis report template heading hierarchy#36921

Merged
pelikhan merged 3 commits into
mainfrom
copilot/fix-9919-1036865607-d5c9acba-a315-48f5-9590-ad9ee7205ec2
Jun 4, 2026
Merged

[workflow-style] Normalize structural analysis report template heading hierarchy#36921
pelikhan merged 3 commits into
mainfrom
copilot/fix-9919-1036865607-d5c9acba-a315-48f5-9590-ad9ee7205ec2

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Jun 4, 2026

The structural analysis workflow’s discussion template used ## headings inside the <details> report block, conflicting with its own rule requiring ### or lower. This aligns the template hierarchy so generated discussions follow the documented reporting style.

  • Report template hierarchy normalization

    • Updated /home/runner/work/gh-aw/gh-aw/.github/workflows/github-mcp-structural-analysis.md inside <details><summary>Full Structural Analysis Report</summary>.
    • Demoted all section headings from #####.
    • Demoted nested visualization headings from #######.
  • Compiled workflow metadata refresh

    • Regenerated corresponding lock metadata in /home/runner/work/gh-aw/gh-aw/.github/workflows/github-mcp-structural-analysis.lock.yml after source template update.
<details>
<summary>Full Structural Analysis Report</summary>

### Executive Summary
...
### Visualizations

#### Response Size by Toolset
...
</details>

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title [WIP] Copilot Request [workflow-style] Normalize structural analysis report template heading hierarchy Jun 4, 2026
Copilot AI requested a review from pelikhan June 4, 2026 14:31
@pelikhan pelikhan marked this pull request as ready for review June 4, 2026 14:38
Copilot AI review requested due to automatic review settings June 4, 2026 14:38
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the structural analysis discussion report template to follow the documented heading hierarchy rules within the <details> “Full Structural Analysis Report” block, and refreshes the compiled workflow lock metadata accordingly.

Changes:

  • Demoted report section headings inside the <details> report template from ## to ###.
  • Demoted nested visualization headings from ### to ####.
  • Regenerated the corresponding .lock.yml metadata to reflect the template update.
Show a summary per file
File Description
.github/workflows/github-mcp-structural-analysis.md Normalizes heading levels inside the report template’s <details> block to match the “h3 or lower” reporting rule.
.github/workflows/github-mcp-structural-analysis.lock.yml Updates the compiled metadata hash to reflect the source template change.

Copilot's findings

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

  • Files reviewed: 2/2 changed files
  • Comments generated: 0

@pelikhan pelikhan merged commit 26af338 into main Jun 4, 2026
14 of 17 checks passed
@pelikhan pelikhan deleted the copilot/fix-9919-1036865607-d5c9acba-a315-48f5-9590-ad9ee7205ec2 branch June 4, 2026 14:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[workflow-style] Normalize report formatting for non-compliant workflows

3 participants