Change Request: Allow processors to report syntax errors

ESLint version

v8.19.0

What problem do you want to solve?

Originally discussed in https://github.com/eslint/eslint/discussions/16015, the @graphql-eslint/graphql processor is encountering a syntax error in the file that is being processed (the “physical” file, if code blocks are “virtual” files). That syntax error results in an exception thrown within the processor that currently crashes the entire lint run. It would be better if processors could report syntax errors like other parsers without crashing the whole process.

What do you think is the correct solution?

We should mimic parse()'s behavior by wrapping the preprocess() call in try/catch and transforming the exception thrown by the processor into a lint error.

Participation

• I am willing to submit a pull request for this change.

Additional comments

In the discussion, @JounQin asked to take this, but I’m also willing to tackle it as a fallback if necessary.

[B2o5T]

Thanks @JounQin 💪🏻

[mdjermanovic]

Since @nzakas already approved the PR, it seems we have consensus to implement this change, so I’m marking it as accepted.