SyntaxError: Invalid regular expression returned when parsing

[jillyj]

Grammar file: https://github.com/opencybersecurityalliance/kestrel-lang/blob/develop/src/kestrel/syntax/kestrel.lark Generated parser: kestrelParser.js.zip

When parsing this statement procs2 = GET process abc, the parser throws the exception like below which is not caught by parser.

Code

function handle_errors(e) { return true; }

  try {
    treeData = parser.parse(text, null, handle_errors).children[0];
  } catch (e) {
    console.debug("uncaught error:", e)
  }

Expected: This kind of error can be handled by the parser, so we can get the parsing tree and the error info like Unexpected character or Unexpected Token.

[jillyj]

@erezsh could you please take a look at this issue?

[erezsh]

Sorry, I had a few busy weeks.

I'll give it a look.

[erezsh]

This happens because Javascript's regex implementation doesn't support all the features that Python has.

This might take a bit longer to fix.

Meanwhile, a possible work around is to change

FUNCNAME: (MIN|MAX|SUM|AVG|COUNT|NUNIQUE)

to

funcname: (MIN|MAX|SUM|AVG|COUNT|NUNIQUE)

[jillyj]

Thank you for the response. Looking forward to your fix. :)

[jillyj]

This happens because Javascript's regex implementation doesn't support all the features that Python has.

This might take a bit longer to fix.

Meanwhile, a possible work around is to change

FUNCNAME: (MIN|MAX|SUM|AVG|COUNT|NUNIQUE)

to

funcname: (MIN|MAX|SUM|AVG|COUNT|NUNIQUE)

Do you mean to edit the generated parser.js? However, I could not find the string.

 FUNCNAME: (MIN|MAX|SUM|AVG|COUNT|NUNIQUE)

[erezsh]

No... edit the grammar!

[jillyj]

Got it. Thanks! Let me try.

[jillyj]

yeah, updating the grammar from FUNCNAME to funcname works. Looking forward to your fix. :)

[erezsh]

The "fix" is most likely going to be preventing users from doing what you were trying to do and throwing an error instead.

I don't know if there is a way to make it work in Javascript. At least, without implementing part of the regex mechanism myself.

[jillyj]

I got it. Thanks. Would you mind to release a version which contains other fixes first?

[erezsh]

Released https://github.com/lark-parser/Lark.js/releases/tag/0.1.3