Skip to content

Conversation

@keyboardDrummer
Copy link
Contributor

@keyboardDrummer keyboardDrummer commented Jan 23, 2026

Changes

  • Remove file2dRange (r: Strata.File2dRange) from MetaData, since Strata's ASTs should contain 1d positions
  • Use DiagnosticModel instead of Format in much of the code to represent diagnostics. DiagnosticModel keeps the error location and error message separated, so the location can be converted to 2d later on.

Future work

  • There are many locations where an error is created but location information is not available. These use deprecated methods like DiagnosticModel.fromFormat to create a DiagnosticModel without a location. These locations should be updated so they create diagnostics with locations.
  • Allow DiagnosticModel to contain error codes, so the front-ends can translate the diagnostics as they desire
  • Allow DiagnosticModel to contain multiple locations and messages

Testing

  • Refactoring, uses existing tests

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@keyboardDrummer keyboardDrummer requested a review from a team as a code owner January 23, 2026 16:29
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