Skip to content

[WIP] Add compilation error report to compile command#7074

Merged
pelikhan merged 4 commits intomainfrom
copilot/add-compile-command-errors
Dec 20, 2025
Merged

[WIP] Add compilation error report to compile command#7074
pelikhan merged 4 commits intomainfrom
copilot/add-compile-command-errors

Conversation

Copy link
Contributor

Copilot AI commented Dec 20, 2025

  • Understand the codebase and existing compilation error tracking
  • Modify CompilationStats to track failed workflows with error counts
  • Update printCompilationSummary to display detailed error summary at the end
  • Update all places that add to FailedWorkflows to track error counts
  • Add/update tests for the new functionality
  • Manually test with multiple error scenarios
  • Run full test suite to ensure no regressions

Summary

The compile command now provides a list of workflows with compilation errors at the end of execution, showing:

  • File path: Full path to each failed workflow
  • Error count: Number of errors per workflow (e.g., "1 error" or "3 errors")

Example output:

✗ Compiled 3 workflow(s): 2 error(s), 0 warning(s)
✗ Failed workflows:
  - /tmp/test-compile-errors/error1.md (1 error)
  - /tmp/test-compile-errors/error2.md (1 error)

This makes it easier to identify and fix compilation issues across multiple workflows.

Original prompt

The compile command should provide a list of workflow with compilation error at the end of the execution (file path + number of errors)


💡 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 3 commits December 20, 2025 18:13
- Added WorkflowFailure struct to track file paths and error counts
- Enhanced CompilationStats to include FailureDetails field
- Updated printCompilationSummary to display detailed error information
- Modified all error tracking in compile_orchestrator to use trackWorkflowFailure helper
- Summary now shows "Failed workflows:" with file paths and error counts

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
- Added test cases for WorkflowFailure tracking with detailed error information
- Added TestTrackWorkflowFailure to verify helper function behavior
- Tests verify proper formatting of error counts (singular/plural)
- All tests pass successfully

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
All tests pass successfully:
- Unit tests verify error tracking and display
- Integration tests confirm proper behavior
- Linting and formatting pass
- Manual testing shows correct output format

Feature now displays failed workflows with:
- Full file path for easy identification
- Error count per workflow (with proper singular/plural)
- Clean, readable format at end of compilation

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
@pelikhan pelikhan marked this pull request as ready for review December 20, 2025 18:27
@pelikhan pelikhan merged commit 4ac7e05 into main Dec 20, 2025
4 checks passed
@pelikhan pelikhan deleted the copilot/add-compile-command-errors branch December 20, 2025 18:27
Copilot AI requested a review from pelikhan December 20, 2025 18:27
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