fix: remove redundant filter (#663)

Swaps BloomFilter for the ScalableCuckooFilter from `@libp2p/utils`

packages/utils/package.json

@@ -58,12 +58,10 @@
     "@ipld/dag-cbor": "^9.2.0",
     "@ipld/dag-json": "^10.2.0",
     "@ipld/dag-pb": "^4.1.0",
-    "@libp2p/crypto": "^5.0.0",
     "@libp2p/interface": "^2.0.0",
     "@libp2p/logger": "^5.0.0",
     "@libp2p/utils": "^6.0.0",
     "@multiformats/dns": "^1.0.1",
-    "@types/murmurhash3js-revisited": "^3.0.3",
     "any-signal": "^4.1.1",
     "blockstore-core": "^5.0.0",
     "cborg": "^4.0.9",
@@ -76,13 +74,12 @@
     "it-merge": "^3.0.3",
     "mortice": "^3.0.4",
     "multiformats": "^13.1.0",
-    "murmurhash3js-revisited": "^3.0.0",
     "p-defer": "^4.0.1",
     "progress-events": "^1.0.0",
-    "uint8arraylist": "^2.4.8",
     "uint8arrays": "^5.0.2"
   },
   "devDependencies": {
+    "@libp2p/crypto": "^5.0.0",
     "@libp2p/peer-id": "^5.0.0",
     "@types/sinon": "^17.0.3",
     "aegir": "^44.0.1",

packages/utils/src/abstract-session.ts

@@ -1,11 +1,12 @@
 import { DEFAULT_SESSION_MIN_PROVIDERS, DEFAULT_SESSION_MAX_PROVIDERS, InsufficientProvidersError } from '@helia/interface'
 import { TypedEventEmitter, setMaxListeners } from '@libp2p/interface'
+import { createScalableCuckooFilter } from '@libp2p/utils/filters'
 import { Queue } from '@libp2p/utils/queue'
 import { base64 } from 'multiformats/bases/base64'
 import pDefer from 'p-defer'
-import { BloomFilter } from './bloom-filter.js'
 import type { BlockBroker, BlockRetrievalOptions, CreateSessionOptions } from '@helia/interface'
 import type { AbortOptions, ComponentLogger, Logger } from '@libp2p/interface'
+import type { Filter } from '@libp2p/utils/filters'
 import type { CID } from 'multiformats/cid'
 import type { DeferredPromise } from 'p-defer'
 import type { ProgressEvent } from 'progress-events'
@@ -31,7 +32,7 @@ export abstract class AbstractSession<Provider, RetrieveBlockProgressEvents exte
   private readonly minProviders: number
   private readonly maxProviders: number
   public readonly providers: Provider[]
-  private readonly evictionFilter: BloomFilter
+  private readonly evictionFilter: Filter
 
   constructor (components: AbstractSessionComponents, init: AbstractCreateSessionOptions) {
     super()
@@ -44,7 +45,7 @@ export abstract class AbstractSession<Provider, RetrieveBlockProgressEvents exte
     this.minProviders = init.minProviders ?? DEFAULT_SESSION_MIN_PROVIDERS
     this.maxProviders = init.maxProviders ?? DEFAULT_SESSION_MAX_PROVIDERS
     this.providers = []
-    this.evictionFilter = BloomFilter.create(this.maxProviders)
+    this.evictionFilter = createScalableCuckooFilter(this.maxProviders)
   }
 
   async retrieve (cid: CID, options: BlockRetrievalOptions<RetrieveBlockProgressEvents> = {}): Promise<Uint8Array> {

packages/utils/src/index.ts

@@ -44,8 +44,6 @@ import type { MultihashHasher } from 'multiformats/hashes/interface'
 
 export { AbstractSession } from './abstract-session.js'
 export type { AbstractCreateSessionOptions, BlockstoreSessionEvents, AbstractSessionComponents } from './abstract-session.js'
-export { BloomFilter } from './bloom-filter.js'
-export type { BloomFilterOptions } from './bloom-filter.js'
 
 export type { BlockStorage, BlockStorageInit }