PrettierLintBear regex is incorrect

[saksham189]

The output of the prettierLintBear to stderr stream on syntax error for the input function b(){ is:
[error] ../../../../var/folders/x6/p95tfncj57q1lzsbbm02599xmts4q9/T/tmphekpuu5k: SyntaxError: Unexpected token (2:1)\n[error] 1 | function b(){\n[error] > 2 | \n[error] | ^\n.

This does not match r'L(?P<line>\d+)C(?P<column>\d+): (?P<message>.*)' the regex that is included inside the bear for stderr

[gitmate-bot]

GitMate.io thinks possibly related issues are #1532 (Regex linter), #2290 (PrettierLintBear: Add more tests), #512 (Bad User Experience), #1395 (PycodestyleBear: Ignore regex (^.*https?://)), and #450 (java/CheckstyleBear: incorrect regex).

[ankitxjoshi]

Yes, I see. It's a problem for any type of syntax error.

[saksham189]

updated the issue. An error is still thrown as corrected output is empty but stderr will never throw any error. This happens for every syntax error

[saksham189]

Can you assign this to me @jayvdb

[newbazz]

@anctartica are you still on this?

[jayvdb]

cc @Nshul

[saksham189]

yes working will make a PR today

[jayvdb]

Maybe this is a version thing? Different versions produced different stderr?

[jayvdb]

#2421 is semi-related

[saksham189]

When the PrettierBear was written not many tests were added. I later added the tests #2341 so, it is possible that this could have slipped. The format for the syntax error message produced by prettier is here: https://github.com/prettier/prettier/blob/master/src/common/parser-create-error.js which does not match the previous regex.

[Nshul]

Apologies for my late response.
When PrettierBear was written not much tests were added. I am really sorry that the regex is incorrect, I took the idea of regex by looking at the official tutorial by coala and the other linter bears, it seemed fine on the tests i conducted.
i think i did miss the link @anctartica has sent from the official prettier repo.

[jayvdb]

Thanks for letting us know.