Nico Flaig

Results 147 issues of Nico Flaig

**Motivation** Noticed we copy-pasted command types/utils in different packages but we only really maintain the ones in the CLI package. - https://github.com/ChainSafe/lodestar/pull/4576 - https://github.com/ChainSafe/lodestar/pull/6438 This PR aims to consolidate types...

**Motivation** Closes https://github.com/ChainSafe/lodestar/issues/5505 **Description** Add eslint rule to prevent unused expressions

Need track of unused expressions to clean up the code and avoid potential bugs. For example this was missed due to missing lint rule. The assertion error has never been...

prio-low
scope-devex

### Describe the bug From Holesky validator client metrics, the `getAttesterDuties` response time has gotten a lot more spiky since Deneb ![image](https://github.com/ChainSafe/lodestar/assets/38436224/0573bede-c045-49e2-84dc-72d11d785528) There might also be a slight increase in...

scope-performance
meta-investigate
meta-bug

### Describe the bug On my Lodestar v1.14.0-rc.1 node I am also seeing a bunch of snappy errors ``` Jan-17 06:32:00.009[network] error: Error on ReqResp.registerProtocol protocolID=/eth2/beacon_chain/req/ping/1/ssz_snappy - Handler already registered...

scope-networking
meta-investigate
meta-bug

After reviewing the spec values I noticed we are still missing plenty of network values which should be part of the config ([configs/mainnet.yaml#L113-L150](https://github.com/ethereum/consensus-specs/blob/3727a757450feb6110e2d4e403604572a478343d/configs/mainnet.yaml#L113-L150)). We either don't support them or those...

prio-low
scope-ux

From reviewing the spec, I noticed that is states that `sync_aggregate` fork might be different and is based on `signature_slot` [specs/altair/light-client/p2p-interface.md?plain=1#L245](https://github.com/ethereum/consensus-specs/blob/3727a757450feb6110e2d4e403604572a478343d/specs/altair/light-client/p2p-interface.md?plain=1#L245) > A `ForkDigest`-context based on `compute_fork_version(compute_epoch_at_slot(optimistic_update.attested_header.beacon.slot))` is used to...

meta-discussion
scope-light-clients

### Describe the bug After recent libp2p upgrade (https://github.com/ChainSafe/lodestar/pull/6285) I am seeing the following error (with `--trace-warnings` enabled) ``` (node:206518) MaxListenersExceededWarning: Possible EventTarget memory leak detected. 11 abort listeners added...

meta-investigate
meta-bug

### Describe the bug There are quite frequent eth1 chain cache timeout errors on startup ``` Sep-02 20:36:05.274[eth1] error: Error updating eth1 chain cache Timeout request Error: Timeout request at...

help-wanted
meta-bug

### Describe the bug We have this function `handleAttesterDutiesReorg` which is mostly called when head (`onNewHead`) is updated - and dependent root for next epoch changed - or dependent root...

prio-low
good first issue
meta-bug