Co-authored-by: tomasciccola <tomasciccola@users.noreply.github.com>

…-discovery

try to use those declarations and failed

Co-authored-by: tomasciccola <tomasciccola@users.noreply.github.com>

I dunno if this is the best solution (or if `secretStream` and `server`
should share the same `close` handler), but it works...

It seems that hypercore replication from on instance to another is failing.
There's probably a bug I haven't catch yet...

It was on the `handleConnection` which meant adding unnecessary handlers
for the same event (for each peer connected)

* remove `this.#socketConnections`
* attach listeners to `error` and `close` before `await once`
* remove type annotation solved by type guard
* start browser and advertiser 'together'
* `await` closing of server

The test added can fail (and others too probably), because sometimes
`this.#browser.on('serviceUp')` will return a `service.addresses` with
a IPv6 which yields the following error:
```bash
node:events:491
      throw er; // Unhandled 'error' event
      ^

Error: connect EINVAL fe80::fa9a:552c:fcb8:f2fb:41373 - Local (:::0)
    at internalConnect (node:net:1041:16)
    at defaultTriggerAsyncIdScope (node:internal/async_hooks:464:18)
    at node:net:1134:9
    at processTicksAndRejections (node:internal/process/task_queues:78:11)
Emitted 'error' event on Socket instance at:
    at emitErrorNT (node:internal/streams/destroy:157:8)
    at emitErrorCloseNT (node:internal/streams/destroy:122:3)
    at processTicksAndRejections (node:internal/process/task_queues:83:21) {
  errno: -22,
  code: 'EINVAL',
  syscall: 'connect',
  address: 'fe80::fa9a:552c:fcb8:f2fb',
  port: 41373
}
```

* main: (25 commits)
  add initial implementation of MemberApi (#232)
  feat: $blobs.getUrl and $blobs.create methods (#184)
  chore: update manager e2e tests (#237)
  feat: add capabilities (#231)
  feat: coreOwnership integration [3/3] (#230)
  feat: CoreOwnership class w getOwner & getCoreKey [2/3] (#229)
  feat: handle `coreOwnership` records in `IndexWriter` [1/3] (#214)
  fix: adjust storage options for MapeoManager and MapeoProject (#235)
  implement addProject method for MapeoManager class (#215)
  implement listProjects method for MapeoManager class (#208)
  feat: expose blobStore.writerDriveId (#219)
  implement wrapper client containing createProject and getProject methods (#199)
  add project settings functionality to MapeoProject (#187)
  feat: Add encode/decode for project keys [3/3] (#203)
  feat: update protobuf for RPC [2/3] (#202)
  chore: move protobuf messages into src/generated [1/3] (#201)
  feat: Add internal `dataType.createWithDocId()` (#192)
  chore: explicitly set "mode" opt for encryptionKeys column creation (#186)
  chore: fix linting and type checking (#183)
  chore: consolidate encryption key columns in projectKeys table (#181)
  ...

* main:
  chore: condense setup code in MemberAPI tests (#267)
  chore: replace queries opt in MemberAPI with dataType (#266)
  feat: add getMany method to member api (#263)
  feat: add getById method to member api (#262)
  chore: Update @digidem/types and remove patch (#269)
  feat: share all core keys via extension messages (#264)
  feat: send core "haves" bitfield on first connect (#254)
  fix: invite.encryptionKeys should be required (#260)
  feat: Add set & get deviceInfo & datatype (#250)
  expose member namespace in MapeoProject (#253)
  blobStore.createReadStream should not wait (#243)
  update MapeoManager to return and handle project public IDs (#247)
  return content hash from blobApi.create (#242)