Skip to content

Conversation

Copilot
Copy link
Contributor

@Copilot Copilot AI commented Oct 15, 2025

Overview

The compile command now displays a clear list of workflows that failed compilation after the error/warning summary, making it easier to identify which workflows need attention without scrolling through detailed error messages.

Changes

Before

✗ failed to parse workflow file /path/bad-1.md: error message...
✗ failed to parse workflow file /path/bad-2.md: error message...
✓ good-1.md (83.5 KB)
✗ Compiled 3 workflow(s): 2 error(s), 0 warning(s)
✗ compilation failed

After

✗ failed to parse workflow file /path/bad-1.md: error message...
✗ failed to parse workflow file /path/bad-2.md: error message...
✓ good-1.md (83.5 KB)
✗ Compiled 3 workflow(s): 2 error(s), 0 warning(s)
✗ Failed workflows:
  - bad-1.md
  - bad-2.md
✗ compilation failed

Implementation Details

  • Extended CompilationStats struct with a FailedWorkflows field to track workflow names that fail compilation
  • Updated all compilation error tracking points (7 locations) to record failed workflow filenames
  • Enhanced printCompilationSummary to display the failed workflows list when errors are present
  • Only shows the list when there are actual failures (successful compilations remain unchanged)
  • Works consistently across all compilation modes: specific files, all files, and watch mode

Testing

  • Added TestPrintCompilationSummaryWithFailedWorkflows unit test
  • All existing tests pass without modification
  • Manually verified with multiple test scenarios including mixed success/failure cases

Benefits

  • Quick identification: Users can immediately see which workflows failed at a glance
  • Improved developer experience: No need to scroll through detailed error messages to find workflow names
  • Consistent behavior: Works the same way in all compilation modes
  • Minimal impact: Surgical changes that don't affect successful compilations or existing functionality
Original prompt

In the compile tool show the list of worflows that fail compilation after the error warning summary


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI and others added 2 commits October 15, 2025 00:17
- Add FailedWorkflows field to CompilationStats struct to track failed workflow names
- Update all compilation loops to track failed workflow names
- Enhance printCompilationSummary to show failed workflow list after error summary
- Add test for printCompilationSummary with failed workflows
- Manually verified with test workflows showing correct output format

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
@Copilot Copilot AI changed the title [WIP] Add compilation error warning summary in compile tool Show list of failed workflows in compile error summary Oct 15, 2025
@Copilot Copilot AI requested a review from pelikhan October 15, 2025 00:20
Copilot finished work on behalf of pelikhan October 15, 2025 00:20
@pelikhan pelikhan marked this pull request as ready for review October 15, 2025 00:43
Copy link
Contributor

Agentic Changeset Generator triggered by this pull request

@pelikhan pelikhan merged commit e879534 into main Oct 15, 2025
10 checks passed
@pelikhan pelikhan deleted the copilot/add-error-warning-summary branch October 15, 2025 02:54
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.

2 participants