feat: prefer error code for req-res protocol over exception

[fryorcraken]

This is a bug report/feature or change request/support request

Problem

Protocols currently use different pattern to handle errors. Light push uses error code while filter uses exception.

It is best for the user if a common pattern is used across the API. Also, there is an opportunity to re-use error code across the codebase and have similar pattern on the user side to handle them:

https://github.com/waku-org/js-waku/blob/b7dc3d7576e9444a5acbb036812c05cfccb25815/packages/interfaces/src/protocols.ts#L67

Proposed Solutions

Generally avoid usage of exception in all protocols (store, light push, filter, metadata) in favour of error code. Encourage developers to monitor this error code to recover from issue, e,g,:

• try again
• show user loss of internet access
• reporting of internal error
• etc

PRs

• [x] metadata: https://github.com/waku-org/js-waku/pull/1904
• [x] peer-exchange: https://github.com/waku-org/js-waku/pull/1907
• [x] filter: https://github.com/waku-org/js-waku/pull/1971
• [x] lightpush: https://github.com/waku-org/js-waku/pull/1971
• [ ] store

Notes

[danisharora099]

Only Store remains from this issue. Moving back to TODO as this is low priority. Might address it with #1685

[weboko]

I believe it can be closed