feat!: lighten retry logic for LightPush

[weboko]

Problem

LightPush doesn't use autoRetry in current version.

Solution

Add it and simplify

Notes

• Resolves - chore: LightPush doesn't use autoRetry #2181

[github-actions]

size-limit report 📦

Path	Size	Loading time (3g)	Running time (snapdragon)	Total time
Waku node	85.51 KB (-0.01% 🔽)	1.8 s (-0.01% 🔽)	2 s (-18.44% 🔽)	3.7 s
Waku Simple Light Node	137.1 KB (-0.33% 🔽)	2.8 s (-0.33% 🔽)	2.8 s (+16.85% 🔺)	5.5 s
ECIES encryption	22.88 KB (0%)	458 ms (0%)	760 ms (+3.29% 🔺)	1.3 s
Symmetric encryption	22.37 KB (0%)	448 ms (0%)	506 ms (+32.35% 🔺)	953 ms
DNS discovery	72.9 KB (0%)	1.5 s (0%)	2.1 s (-3.91% 🔽)	3.6 s
Peer Exchange discovery	74.1 KB (0%)	1.5 s (0%)	1.4 s (+1.91% 🔺)	2.9 s
Local Peer Cache Discovery	68.03 KB (0%)	1.4 s (0%)	1.9 s (+22.96% 🔺)	3.3 s
Privacy preserving protocols	76.02 KB (+0.04% 🔺)	1.6 s (+0.04% 🔺)	1.5 s (-11.29% 🔽)	3.1 s
Waku Filter	79.56 KB (-0.31% 🔽)	1.6 s (-0.31% 🔽)	2.3 s (+43.26% 🔺)	3.8 s
Waku LightPush	75.03 KB (-3.54% 🔽)	1.6 s (-3.54% 🔽)	1.4 s (-19.05% 🔽)	2.9 s
History retrieval protocols	77.03 KB (-0.04% 🔽)	1.6 s (-0.04% 🔽)	2 s (-9.66% 🔽)	3.6 s
Deterministic Message Hashing	7.39 KB (0%)	148 ms (0%)	297 ms (+130.29% 🔺)	445 ms

[weboko]

removed BaseProtocolSDK for now as it is not used
let's follow up in #2186

[adklempner]

this might be a breaking change since ProtocolUseOptions had a variable not in ISenderOptions. If someone has code where that optional variable was set, will it break upon updating?

[weboko]

true and no, ISenderOptions actually returns previously existed properties and now they are respected in this implementation

so the only breaking change is the naming

[adklempner]

new interface doesn't have forceUseAllPeers?: boolean;

[adklempner]

js-waku/packages/interfaces/src/protocols.ts

Line 41 in b2efce5

forceUseAllPeers?: boolean;

[weboko]

good spot, afaik it wasn't used outside some of our tests