[Nodejs SDK] should Client expose event handler for events through on/once
- Dominant language
- Rust
- Stars
- 4.9k
- Forks
- 432
- Avg merge
- 2d 10h
- Merged PRs (30d)
- 173
Description
To simplify and standardize how events are registered, should we add on/once methods to the handler event?
Current State
- only CommandResponseStream extending EventEmitter
or
expect:
```ts
const client = new Client()
const cb = ()=>console.log
client.on("connecting",cb)
client.on("connected",cb)
client.on("error",cb)
```
### Reference
- https://nodejs.org/en/learn/asynchronous-work/the-nodejs-event-emitter
Contributor guide
Research direction
Locate the Node.js SDK's Client entry point and the existing CommandResponseStream EventEmitter implementation. Compare how current events are exposed with the Node.js EventEmitter reference, then verify that Client supports on/once registration for connecting, connected, and error events.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- nodejs, typescript
- Domain
- api
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 55/100