@dxos/echo-pipeline 0.4.4-main.fcf0b00 → 0.4.4-next.75bfacc

package/dist/types/src/automerge/mesh-network-adapter.d.ts

@@ -0,0 +1,18 @@
+import { NetworkAdapter, type Message, type PeerId } from '@dxos/automerge/automerge-repo';
+import { AutomergeReplicator } from '@dxos/teleport-extension-automerge-replicator';
+/**
+ * Used to replicate with other peers over the network.
+ */
+export declare class MeshNetworkAdapter extends NetworkAdapter {
+    private readonly _extensions;
+    private _connected;
+    /**
+     * Emits `ready` event. That signals to `Repo` that it can start using the adapter.
+     */
+    ready(): void;
+    connect(peerId: PeerId): void;
+    send(message: Message): void;
+    disconnect(): void;
+    createExtension(): AutomergeReplicator;
+}
+//# sourceMappingURL=mesh-network-adapter.d.ts.map

package/dist/types/src/automerge/mesh-network-adapter.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"mesh-network-adapter.d.ts","sourceRoot":"","sources":["../../../../src/automerge/mesh-network-adapter.ts"],"names":[],"mappings":"AAKA,OAAO,EAAE,cAAc,EAAE,KAAK,OAAO,EAAE,KAAK,MAAM,EAAQ,MAAM,gCAAgC,CAAC;AAIjG,OAAO,EAAE,mBAAmB,EAAE,MAAM,+CAA+C,CAAC;AAEpF;;GAEG;AACH,qBAAa,kBAAmB,SAAQ,cAAc;IACpD,OAAO,CAAC,QAAQ,CAAC,WAAW,CAA+C;IAC3E,OAAO,CAAC,UAAU,CAAiB;IAEnC;;OAEG;IACH,KAAK;IAQI,OAAO,CAAC,MAAM,EAAE,MAAM,GAAG,IAAI;IAK7B,IAAI,CAAC,OAAO,EAAE,OAAO,GAAG,IAAI;IAO5B,UAAU,IAAI,IAAI;IAI3B,eAAe,IAAI,mBAAmB;CA6DvC"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dxos/echo-pipeline",
-  "version": "0.4.4-main.fcf0b00",
+  "version": "0.4.4-next.75bfacc",
   "description": "ECHO database.",
   "homepage": "https://dxos.org",
   "bugs": "https://github.com/dxos/dxos/issues",
@@ -36,41 +36,41 @@
   ],
   "dependencies": {
     "crc-32": "^1.2.2",
-    "@dxos/async": "0.4.4-main.fcf0b00",
-    "@dxos/codec-protobuf": "0.4.4-main.fcf0b00",
-    "@dxos/context": "0.4.4-main.fcf0b00",
-    "@dxos/credentials": "0.4.4-main.fcf0b00",
-    "@dxos/crypto": "0.4.4-main.fcf0b00",
-    "@dxos/debug": "0.4.4-main.fcf0b00",
-    "@dxos/document-model": "0.4.4-main.fcf0b00",
-    "@dxos/automerge": "0.4.4-main.fcf0b00",
-    "@dxos/echo-db": "0.4.4-main.fcf0b00",
-    "@dxos/feed-store": "0.4.4-main.fcf0b00",
-    "@dxos/hypercore": "0.4.4-main.fcf0b00",
-    "@dxos/keyring": "0.4.4-main.fcf0b00",
-    "@dxos/invariant": "0.4.4-main.fcf0b00",
-    "@dxos/keys": "0.4.4-main.fcf0b00",
-    "@dxos/messaging": "0.4.4-main.fcf0b00",
-    "@dxos/log": "0.4.4-main.fcf0b00",
-    "@dxos/model-factory": "0.4.4-main.fcf0b00",
-    "@dxos/network-manager": "0.4.4-main.fcf0b00",
-    "@dxos/protocols": "0.4.4-main.fcf0b00",
-    "@dxos/node-std": "0.4.4-main.fcf0b00",
-    "@dxos/random-access-storage": "0.4.4-main.fcf0b00",
-    "@dxos/rpc": "0.4.4-main.fcf0b00",
-    "@dxos/teleport": "0.4.4-main.fcf0b00",
-    "@dxos/teleport-extension-automerge-replicator": "0.4.4-main.fcf0b00",
-    "@dxos/teleport-extension-gossip": "0.4.4-main.fcf0b00",
-    "@dxos/teleport-extension-object-sync": "0.4.4-main.fcf0b00",
-    "@dxos/text-model": "0.4.4-main.fcf0b00",
-    "@dxos/teleport-extension-replicator": "0.4.4-main.fcf0b00",
-    "@dxos/timeframe": "0.4.4-main.fcf0b00",
-    "@dxos/tracing": "0.4.4-main.fcf0b00",
-    "@dxos/typings": "0.4.4-main.fcf0b00",
-    "@dxos/util": "0.4.4-main.fcf0b00"
+    "@dxos/async": "0.4.4-next.75bfacc",
+    "@dxos/automerge": "0.4.4-next.75bfacc",
+    "@dxos/codec-protobuf": "0.4.4-next.75bfacc",
+    "@dxos/context": "0.4.4-next.75bfacc",
+    "@dxos/credentials": "0.4.4-next.75bfacc",
+    "@dxos/crypto": "0.4.4-next.75bfacc",
+    "@dxos/debug": "0.4.4-next.75bfacc",
+    "@dxos/document-model": "0.4.4-next.75bfacc",
+    "@dxos/echo-db": "0.4.4-next.75bfacc",
+    "@dxos/feed-store": "0.4.4-next.75bfacc",
+    "@dxos/hypercore": "0.4.4-next.75bfacc",
+    "@dxos/invariant": "0.4.4-next.75bfacc",
+    "@dxos/keyring": "0.4.4-next.75bfacc",
+    "@dxos/keys": "0.4.4-next.75bfacc",
+    "@dxos/log": "0.4.4-next.75bfacc",
+    "@dxos/messaging": "0.4.4-next.75bfacc",
+    "@dxos/network-manager": "0.4.4-next.75bfacc",
+    "@dxos/node-std": "0.4.4-next.75bfacc",
+    "@dxos/model-factory": "0.4.4-next.75bfacc",
+    "@dxos/protocols": "0.4.4-next.75bfacc",
+    "@dxos/rpc": "0.4.4-next.75bfacc",
+    "@dxos/random-access-storage": "0.4.4-next.75bfacc",
+    "@dxos/teleport-extension-automerge-replicator": "0.4.4-next.75bfacc",
+    "@dxos/teleport": "0.4.4-next.75bfacc",
+    "@dxos/teleport-extension-object-sync": "0.4.4-next.75bfacc",
+    "@dxos/teleport-extension-gossip": "0.4.4-next.75bfacc",
+    "@dxos/text-model": "0.4.4-next.75bfacc",
+    "@dxos/timeframe": "0.4.4-next.75bfacc",
+    "@dxos/teleport-extension-replicator": "0.4.4-next.75bfacc",
+    "@dxos/tracing": "0.4.4-next.75bfacc",
+    "@dxos/typings": "0.4.4-next.75bfacc",
+    "@dxos/util": "0.4.4-next.75bfacc"
   },
   "devDependencies": {
-    "fast-check": "~3.3.0",
+    "fast-check": "^3.15.1",
     "hypercore-protocol": "^8.0.7",
     "source-map-support": "^0.5.12",
     "wait-for-expect": "^3.0.2"

package/src/automerge/automerge-host.test.ts

@@ -7,7 +7,14 @@ import expect from 'expect';
 import waitForExpect from 'wait-for-expect';
 
 import { Trigger, asyncTimeout, sleep } from '@dxos/async';
-import { type Message, NetworkAdapter, type PeerId, Repo, type HandleState } from '@dxos/automerge/automerge-repo';
+import {
+  type Message,
+  NetworkAdapter,
+  type PeerId,
+  Repo,
+  type HandleState,
+  type DocumentId,
+} from '@dxos/automerge/automerge-repo';
 import { invariant } from '@dxos/invariant';
 import { log } from '@dxos/log';
 import { StorageType, createStorage } from '@dxos/random-access-storage';
@@ -15,7 +22,9 @@ import { TestBuilder as TeleportBuilder, TestPeer as TeleportPeer } from '@dxos/
 import { afterTest, describe, test } from '@dxos/test';
 import { arrayToBuffer, bufferToArray } from '@dxos/util';
 
-import { AutomergeHost, AutomergeStorageAdapter, MeshNetworkAdapter } from './automerge-host';
+import { AutomergeHost } from './automerge-host';
+import { AutomergeStorageAdapter } from './automerge-storage-adapter';
+import { MeshNetworkAdapter } from './mesh-network-adapter';
 
 describe('AutomergeHost', () => {
   test('can create documents', () => {
@@ -78,6 +87,97 @@ describe('AutomergeHost', () => {
     expect((await asyncTimeout(docOnClient.doc(), 1000)).text).toEqual(text);
   });
 
+  test('share policy gets enabled afterwards', async () => {
+    const [hostAdapter, clientAdapter] = TestAdapter.createPair();
+    let sharePolicy = false;
+
+    const host = new Repo({
+      network: [hostAdapter],
+      peerId: 'host' as PeerId,
+      sharePolicy: async () => sharePolicy,
+    });
+    const client = new Repo({
+      network: [clientAdapter],
+      peerId: 'client' as PeerId,
+      sharePolicy: async () => sharePolicy,
+    });
+    hostAdapter.ready();
+    clientAdapter.ready();
+    await hostAdapter.onConnect.wait();
+    await clientAdapter.onConnect.wait();
+    hostAdapter.peerCandidate(clientAdapter.peerId!);
+    clientAdapter.peerCandidate(hostAdapter.peerId!);
+
+    const handle = host.create();
+    const text = 'Hello world';
+    handle.change((doc: any) => {
+      doc.text = text;
+    });
+
+    {
+      const docOnClient = client.find(handle.url);
+      await asyncTimeout(docOnClient.whenReady(['unavailable']), 1000);
+    }
+
+    sharePolicy = true;
+
+    {
+      const docOnClient = client.find(handle.url);
+      // TODO(mykola): We expect the document to be available here, but it's not.
+      await asyncTimeout(docOnClient.whenReady(['unavailable']), 1000);
+    }
+  });
+
+  test('two documents and share policy switching', async () => {
+    const [hostAdapter, clientAdapter] = TestAdapter.createPair();
+    const allowedDocs: DocumentId[] = [];
+
+    const host: Repo = new Repo({
+      network: [hostAdapter],
+      peerId: 'host' as PeerId,
+      sharePolicy: async (_, docId) => (docId ? allowedDocs.includes(docId) && !!host.handles[docId] : false),
+    });
+
+    const client: Repo = new Repo({
+      network: [clientAdapter],
+      peerId: 'client' as PeerId,
+      sharePolicy: async (_, docId) => (docId ? allowedDocs.includes(docId) && !!client.handles[docId] : false),
+    });
+
+    const firstHandle = host.create();
+    firstHandle.change((doc: any) => (doc.text = 'Hello world'));
+    await host.find(firstHandle.url).whenReady();
+    allowedDocs.push(firstHandle.documentId);
+
+    {
+      // Initiate connection.
+      hostAdapter.ready();
+      clientAdapter.ready();
+      await hostAdapter.onConnect.wait();
+      await clientAdapter.onConnect.wait();
+      hostAdapter.peerCandidate(clientAdapter.peerId!);
+      clientAdapter.peerCandidate(hostAdapter.peerId!);
+    }
+
+    {
+      const firstDocOnClient = client.find(firstHandle.url);
+      await asyncTimeout(firstDocOnClient.whenReady(), 1000);
+      expect(firstDocOnClient.docSync().text).toEqual('Hello world');
+    }
+
+    const secondHandle = host.create();
+    secondHandle.change((doc: any) => (doc.text = 'Hello world'));
+    await host.find(secondHandle.url).whenReady();
+    // await sleep(100);
+    allowedDocs.push(secondHandle.documentId);
+
+    {
+      const secondDocOnClient = client.find(secondHandle.url);
+      await asyncTimeout(secondDocOnClient.whenReady(), 1000);
+      expect(secondDocOnClient.docSync().text).toEqual('Hello world');
+    }
+  });
+
   test('recovering from a lost connection', async () => {
     let connectionState: 'on' | 'off' = 'on';
 

package/src/automerge/automerge-host.ts

@@ -2,29 +2,23 @@
 // Copyright 2023 DXOS.org
 //
 
-import { Trigger } from '@dxos/async';
 import { next as automerge } from '@dxos/automerge/automerge';
-import {
-  Repo,
-  NetworkAdapter,
-  StorageAdapter,
-  type Message,
-  type PeerId,
-  type Chunk,
-  type StorageKey,
-  cbor,
-} from '@dxos/automerge/automerge-repo';
+import { Repo, type StorageAdapter, type PeerId, type DocumentId } from '@dxos/automerge/automerge-repo';
 import { IndexedDBStorageAdapter } from '@dxos/automerge/automerge-repo-storage-indexeddb';
-import { Stream } from '@dxos/codec-protobuf';
-import { invariant } from '@dxos/invariant';
+import { type Stream } from '@dxos/codec-protobuf';
 import { PublicKey } from '@dxos/keys';
 import { log } from '@dxos/log';
 import { type HostInfo, type SyncRepoRequest, type SyncRepoResponse } from '@dxos/protocols/proto/dxos/echo/service';
-import { type PeerInfo } from '@dxos/protocols/proto/dxos/mesh/teleport/automerge';
 import { StorageType, type Directory } from '@dxos/random-access-storage';
-import { AutomergeReplicator } from '@dxos/teleport-extension-automerge-replicator';
+import { type AutomergeReplicator } from '@dxos/teleport-extension-automerge-replicator';
 import { trace } from '@dxos/tracing';
-import { ComplexMap, ComplexSet, arrayToBuffer, bufferToArray, defaultMap, mapValues } from '@dxos/util';
+import { ComplexMap, ComplexSet, defaultMap, mapValues } from '@dxos/util';
+
+import { AutomergeStorageAdapter } from './automerge-storage-adapter';
+import { LocalHostNetworkAdapter } from './local-host-network-adapter';
+import { MeshNetworkAdapter } from './mesh-network-adapter';
+
+export type { DocumentId };
 
 @trace.resource()
 export class AutomergeHost {
@@ -38,6 +32,8 @@ export class AutomergeHost {
    */
   private readonly _authorizedDevices = new ComplexMap<PublicKey, ComplexSet<PublicKey>>(PublicKey.hash);
 
+  public _requestedDocs = new Set<string>();
+
   constructor(storageDirectory: Directory) {
     this._meshNetwork = new MeshNetworkAdapter();
     this._clientNetwork = new LocalHostNetworkAdapter();
@@ -47,8 +43,9 @@ export class AutomergeHost {
       storageDirectory.type === StorageType.IDB
         ? new IndexedDBStorageAdapter(storageDirectory.path, 'data')
         : new AutomergeStorageAdapter(storageDirectory);
+    const localPeerId = `host-${PublicKey.random().toHex()}` as PeerId;
     this._repo = new Repo({
-      peerId: `host-${PublicKey.random().toHex()}` as PeerId,
+      peerId: localPeerId,
       network: [this._clientNetwork, this._meshNetwork],
       storage: this._storage,
 
@@ -65,8 +62,9 @@ export class AutomergeHost {
 
         const doc = this._repo.handles[documentId]?.docSync();
         if (!doc) {
-          log('doc not found for share policy check', { peerId, documentId });
-          return false;
+          const isRequested = this._requestedDocs.has(`automerge:${documentId}`);
+          log('doc share policy check', { peerId, documentId, isRequested });
+          return isRequested;
         }
 
         try {
@@ -87,13 +85,20 @@ export class AutomergeHost {
           const deviceKey = PublicKey.from(deviceKeyHex);
 
           const isAuthorized = authorizedDevices?.has(deviceKey) ?? false;
-          log('share policy check', { peerId, documentId, deviceKey, spaceKey, isAuthorized });
+          log('share policy check', {
+            localPeer: localPeerId,
+            remotePeer: peerId,
+            documentId,
+            deviceKey,
+            spaceKey,
+            isAuthorized,
+          });
           return isAuthorized;
         } catch (err) {
           log.catch(err);
           return false;
         }
-      }, // Share everything.
+      },
     });
     this._clientNetwork.ready();
     this._meshNetwork.ready();
@@ -147,288 +152,7 @@ export class AutomergeHost {
   }
 
   authorizeDevice(spaceKey: PublicKey, deviceKey: PublicKey) {
+    log('authorizeDevice', { spaceKey, deviceKey });
     defaultMap(this._authorizedDevices, spaceKey, () => new ComplexSet(PublicKey.hash)).add(deviceKey);
   }
 }
-
-type ClientSyncState = {
-  connected: boolean;
-  send: (message: Message) => void;
-  disconnect: () => void;
-};
-
-/**
- * Used to replicate with apps running on the same device.
- */
-class LocalHostNetworkAdapter extends NetworkAdapter {
-  private readonly _peers: Map<PeerId, ClientSyncState> = new Map();
-
-  /**
-   * Emits `ready` event. That signals to `Repo` that it can start using the adapter.
-   */
-  ready() {
-    // NOTE: Emitting `ready` event in NetworkAdapter`s constructor causes a race condition
-    //       because `Repo` waits for `ready` event (which it never receives) before it starts using the adapter.
-    this.emit('ready', {
-      network: this,
-    });
-  }
-
-  private _connected = new Trigger();
-
-  override connect(peerId: PeerId): void {
-    this.peerId = peerId;
-    this._connected.wake();
-    // No-op. Client always connects first
-  }
-
-  override send(message: Message): void {
-    const peer = this._peers.get(message.targetId);
-    invariant(peer, 'Peer not found.');
-    peer.send(message);
-  }
-
-  async close() {
-    this._peers.forEach((peer) => peer.disconnect());
-    this.emit('close');
-  }
-
-  override disconnect(): void {
-    // TODO(mykola): `disconnect` is not used anywhere in `Repo` from `@automerge/automerge-repo`. Should we remove it?
-    // No-op
-  }
-
-  syncRepo({ id, syncMessage }: SyncRepoRequest): Stream<SyncRepoResponse> {
-    const peerId = this._getPeerId(id);
-
-    return new Stream(({ next, close }) => {
-      invariant(!this._peers.has(peerId), 'Peer already connected.');
-      this._peers.set(peerId, {
-        connected: true,
-        send: (message) => {
-          next({
-            syncMessage: cbor.encode(message),
-          });
-        },
-        disconnect: () => {
-          this._peers.delete(peerId);
-          close();
-          this.emit('peer-disconnected', {
-            peerId,
-          });
-        },
-      });
-
-      this._connected
-        .wait({ timeout: 1_000 })
-        .then(() => {
-          this.emit('peer-candidate', {
-            peerMetadata: {},
-            peerId,
-          });
-        })
-        .catch((err) => log.catch(err));
-    });
-  }
-
-  async sendSyncMessage({ id, syncMessage }: SyncRepoRequest): Promise<void> {
-    await this._connected.wait({ timeout: 1_000 });
-    const message = cbor.decode(syncMessage!) as Message;
-    this.emit('message', message);
-  }
-
-  async getHostInfo(): Promise<HostInfo> {
-    await this._connected.wait({ timeout: 1_000 });
-    invariant(this.peerId, 'Peer id not set.');
-    return {
-      peerId: this.peerId,
-    };
-  }
-
-  private _getPeerId(id: string): PeerId {
-    return id as PeerId;
-  }
-}
-
-/**
- * Used to replicate with other peers over the network.
- */
-export class MeshNetworkAdapter extends NetworkAdapter {
-  private readonly _extensions: Map<string, AutomergeReplicator> = new Map();
-  private _connected = new Trigger();
-
-  /**
-   * Emits `ready` event. That signals to `Repo` that it can start using the adapter.
-   */
-  ready() {
-    // NOTE: Emitting `ready` event in NetworkAdapter`s constructor causes a race condition
-    //       because `Repo` waits for `ready` event (which it never receives) before it starts using the adapter.
-    this.emit('ready', {
-      network: this,
-    });
-  }
-
-  override connect(peerId: PeerId): void {
-    this.peerId = peerId;
-    this._connected.wake();
-  }
-
-  override send(message: Message): void {
-    const receiverId = message.targetId;
-    const extension = this._extensions.get(receiverId);
-    invariant(extension, 'Extension not found.');
-    extension.sendSyncMessage({ payload: cbor.encode(message) }).catch((err) => log.catch(err));
-  }
-
-  override disconnect(): void {
-    // No-op
-  }
-
-  createExtension(): AutomergeReplicator {
-    invariant(this.peerId, 'Peer id not set.');
-
-    let peerInfo: PeerInfo;
-    const extension = new AutomergeReplicator(
-      {
-        peerId: this.peerId,
-      },
-      {
-        onStartReplication: async (info, remotePeerId /** Teleport ID */) => {
-          await this._connected.wait();
-
-          // Note: We store only one extension per peer.
-          //       There can be a case where two connected peers have more than one teleport connection between them
-          //       and each of them uses different teleport connections to send messages.
-          //       It works because we receive messages from all teleport connections and Automerge Repo dedup them.
-          // TODO(mykola): Use only one teleport connection per peer.
-          if (!this._extensions.has(info.id)) {
-            peerInfo = info;
-            // TODO(mykola): Fix race condition?
-            this._extensions.set(info.id, extension);
-          } else {
-            // TODO(mykola): retry hack.
-            this.emit('peer-disconnected', { peerId: info.id as PeerId });
-          }
-
-          this.emit('peer-candidate', {
-            // TODO(mykola): Hack, stop abusing `peerMetadata` field.
-            peerMetadata: {
-              dxos_deviceKey: remotePeerId.toHex(),
-            } as any,
-            peerId: info.id as PeerId,
-          });
-        },
-        onSyncMessage: async ({ payload }) => {
-          const message = cbor.decode(payload) as Message;
-          // Note: automerge Repo dedup messages.
-          this.emit('message', message);
-        },
-        onClose: async () => {
-          if (!peerInfo) {
-            return;
-          }
-          this.emit('peer-disconnected', {
-            peerId: peerInfo.id as PeerId,
-          });
-          this._extensions.delete(peerInfo.id);
-        },
-      },
-    );
-    return extension;
-  }
-}
-
-export class AutomergeStorageAdapter extends StorageAdapter {
-  // TODO(mykola): Hack for restricting automerge Repo to access storage if Host is `closed`.
-  //               Automerge Repo do not have any lifetime management.
-  private _state: 'opened' | 'closed' = 'opened';
-
-  constructor(private readonly _directory: Directory) {
-    super();
-  }
-
-  override async load(key: StorageKey): Promise<Uint8Array | undefined> {
-    if (this._state !== 'opened') {
-      return undefined;
-    }
-    const filename = this._getFilename(key);
-    const file = this._directory.getOrCreateFile(filename);
-    const { size } = await file.stat();
-    if (!size || size === 0) {
-      return undefined;
-    }
-    const buffer = await file.read(0, size);
-    return bufferToArray(buffer);
-  }
-
-  override async save(key: StorageKey, data: Uint8Array): Promise<void> {
-    if (this._state !== 'opened') {
-      return undefined;
-    }
-    const filename = this._getFilename(key);
-    const file = this._directory.getOrCreateFile(filename);
-    await file.write(0, arrayToBuffer(data));
-    await file.truncate?.(data.length);
-
-    await file.flush?.();
-  }
-
-  override async remove(key: StorageKey): Promise<void> {
-    if (this._state !== 'opened') {
-      return undefined;
-    }
-    // TODO(dmaretskyi): Better deletion.
-    const filename = this._getFilename(key);
-    const file = this._directory.getOrCreateFile(filename);
-    await file.destroy();
-  }
-
-  override async loadRange(keyPrefix: StorageKey): Promise<Chunk[]> {
-    if (this._state !== 'opened') {
-      return [];
-    }
-    const filename = this._getFilename(keyPrefix);
-    const entries = await this._directory.list();
-    return Promise.all(
-      entries
-        .filter((entry) => entry.startsWith(filename))
-        .map(async (entry): Promise<Chunk> => {
-          const file = this._directory.getOrCreateFile(entry);
-          const { size } = await file.stat();
-          const buffer = await file.read(0, size);
-          return {
-            key: this._getKeyFromFilename(entry),
-            data: bufferToArray(buffer),
-          };
-        }),
-    );
-  }
-
-  override async removeRange(keyPrefix: StorageKey): Promise<void> {
-    if (this._state !== 'opened') {
-      return undefined;
-    }
-    const filename = this._getFilename(keyPrefix);
-    const entries = await this._directory.list();
-    await Promise.all(
-      entries
-        .filter((entry) => entry.startsWith(filename))
-        .map(async (entry): Promise<void> => {
-          const file = this._directory.getOrCreateFile(entry);
-          await file.destroy();
-        }),
-    );
-  }
-
-  async close(): Promise<void> {
-    this._state = 'closed';
-  }
-
-  private _getFilename(key: StorageKey): string {
-    return key.map((k) => k.replaceAll('%', '%25').replaceAll('-', '%2D')).join('-');
-  }
-
-  private _getKeyFromFilename(filename: string): StorageKey {
-    return filename.split('-').map((k) => k.replaceAll('%2D', '-').replaceAll('%25', '%'));
-  }
-}