Why no generic value event?

[wijonja]

Hi uhop,

First of all thanks for publishing this. It has certainly saved me some time.

I was wondering why there's no generic value event type? I can't imagine many people needing separate events for null, true, false, string and number out of the gate. It has made my downstream code more slightly complicated than it needs to be.

Any chance of a simple value event?

And while you're at it, is there any documentation of the complete event/token name list? I had to dig into the source to find it.

Cheers mate

[uhop]

As per README all information is in Wiki.

I was wondering why there's no generic value event type?

I didn't have any need for it.

Any chance of a simple value event?

Could you describe how you see it and a super-simple fragment of code that uses it?

And while you're at it, is there any documentation of the complete event/token name list?

In the Wiki where Parser is documented: https://github.com/uhop/stream-json/wiki/Parser#stream-of-tokens

[wijonja]

Could you describe how you see it and a super-simple fragment of code that uses it?

I would suggest

emitter.on('value', (value: string | number | boolean | null ) => { ... });

Splitting tokens on type is closer to a Java or C pattern. Can you help me out? How do you typically consume this?

[uhop]

How do you typically consume this?

I don't. That's why it is not implemented. But I am eagerly await a PR from you.

[wijonja]

You don't consume your own library?

I'm open to raising a PR. It should only be a few lines in Emitter. But I'll be honest, I've switched to @streamparser/json. If I switch back I'll look into it.

[uhop]

You don't consume your own library?

I do, but I usually I don't deal with individual tokens and don't use events.

I'm open to raising a PR. It should only be a few lines in Emitter. But I'll be honest, I've switched to @streamparser/json. If I switch back I'll look into it.

OK. Got it. Good luck.