finishing touches

packages/nostr/NOTES.md

@@ -1,2 +0,0 @@
-- Don't use arrays for callbacks because there's no way to clear them,
-passing undefined or null to on* should clear the callback

packages/nostr/src/client/conn.ts

@@ -49,7 +49,9 @@ export class Conn {
         if (err instanceof ProtocolError) {
           this.#errorCallback?.(err)
         } else {
-          // TODO Not sure if this is the case?
+          // TODO Not sure if this is the best idea.
+          // Investigate what WebSocket does if the callback throws?
+          // Either way it seems like the best idea is to have `onError` called on all types of errors
           throw err
         }
       }

packages/nostr/src/client/index.ts

@@ -226,7 +226,7 @@ interface ConnState {
    */
   auth: boolean
   /**
-   * Should this connection be used for receiving messages?
+   * Should this connection be used for receiving events?
    */
   read: boolean
   /**