Skip to content

fix: update spec tests for improved parse error display#32669

Open
bartlomieju wants to merge 3 commits intomainfrom
fix/improve-parse-error-display
Open

fix: update spec tests for improved parse error display#32669
bartlomieju wants to merge 3 commits intomainfrom
fix/improve-parse-error-display

Conversation

@bartlomieju
Copy link
Member

Summary

Before:

error: The module's source code could not be parsed: Expected ',', got 'following' at .../error_syntax.js:3:6

  (the following is a syntax error  ^^ ! )
       ~~~~~~~~~

After:

error: Expected ',', got 'following'
    at .../error_syntax.js:3:6
  |
3 | (the following is a syntax error  ^^ ! )
  |      ~~~~~~~~~

Test plan

  • specs::run::error_syntax passes
  • specs::run::error_syntax_empty_trailing_line passes
  • specs::run::swc_syntax_error passes

🤖 Generated with Claude Code

Updates expected output files to match the new structured parse error
format from deno_graph, which uses location on its own line with pipe
margins, line numbers, and carets.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
bartlomieju and others added 2 commits March 12, 2026 17:49
Location line now appears after the code snippet, and string literal
token descriptions are simplified.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
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