Skip to content

Conversation

@kbkpbot
Copy link
Contributor

@kbkpbot kbkpbot commented Feb 1, 2026

Introduce a new unified error handling system with dedicated modules for context management, deduplication, display formatting, and centralized error reporting. This refactoring improves code organization, maintainability, and consistency across the compiler.

Changes:

  • Add ErrorHandler interface and DefaultErrorHandler implementation
  • Implement ErrorDeduplicator for preventing duplicate messages
  • Add ErrorContext for managing error details and suggestions
  • Implement DisplayManager for formatted error output
  • Refactor checker errors to use new error handler
  • Add message limit checking and should_abort synchronization
  • Remove unused warn_or_error helper function

Introduce a new unified error handling system with dedicated modules
for context management, deduplication, display formatting, and
centralized error reporting. This refactoring improves code
organization, maintainability, and consistency across the compiler.

Changes:
- Add ErrorHandler interface and DefaultErrorHandler implementation
- Implement ErrorDeduplicator for preventing duplicate messages
- Add ErrorContext for managing error details and suggestions
- Implement DisplayManager for formatted error output
- Refactor checker errors to use new error handler
- Add message limit checking and should_abort synchronization
- Remove unused warn_or_error helper function
- Remove unused standardized error message generators
@kbkpbot kbkpbot marked this pull request as draft February 1, 2026 05:53
@kbkpbot kbkpbot changed the title Refactor error handling with unified ErrorHandler compiler: Refactor error handling with unified ErrorHandler Feb 1, 2026
@kbkpbot
Copy link
Contributor Author

kbkpbot commented Feb 4, 2026

need a better way to show detail and suggestion.

@kbkpbot kbkpbot closed this Feb 4, 2026
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.

1 participant