apache / apache/iggy

[Nodejs SDK] should Client expose event handler for events through on/once

Open
#2,396 2 comments 0 reactions 0 assignees View on GitHub
good first issue javascript
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

Open the contributing 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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.