Kevin Scott

> If you submit a PR with a validating test, I will merge it in. [Added here](https://github.com/lark-parser/Lark.js/pull/40). Tests pass, but I don't see any tests around `InteractiveParser` (so I don't...

Test added.

I think these are some great comments. I would in particular like to second the request for ESM support, which I _think_ is as simple as [updating the export syntax...

I thought of a way to achieve this. One, modify the `copy` method like so: ``` function copy(obj) { if (typeof obj == "object") { if ('__copy__' in obj.constructor) {...

That's a fair question. I was trying to align with the original Python implementations, which seem to have a [`copy` instance method](https://github.com/lark-parser/lark/blob/master/lark/parsers/lalr_interactive_parser.py#L72) (which simply wraps the call to the native...

Roger that - so something like: ``` function copy(obj) { if (typeof obj == "object") { if ('__copy__' in obj) { return obj.__copy__(); } let empty_clone = Object.create(Object.getPrototypeOf(obj)); return Object.assign(empty_clone,...

Though this fixes the specific bug I encountered, I may have introduced another bug. Should `t` refer to the _key_ of the `choices` dictionary, or the value? It seems like...

Ah, very true. Good catch! https://docs.python.org/3/library/stdtypes.html#str.isupper > Return True if all cased characters [[4]](https://docs.python.org/3/library/stdtypes.html?highlight=isupper#id15) in the string are uppercase and there is at least one cased character, False otherwise. It...

It looks like any model I haven't previously cached is reporting the error:

I don't _think_ this is related to the typical browser cache, but just for good measure I cleared my cache, but same issue.