@unicitylabs/sphere-sdk 0.17.5 → 0.17.6

package/dist/impl/nodejs/index.d.cts

@@ -225,6 +225,7 @@ declare class FileStorageProvider implements StorageProvider {
     private isTxtMode;
     private network?;
     private data;
+    private loading;
     private status;
     private _identity;
     constructor(config: FileStorageProviderConfig | string);
@@ -232,6 +233,14 @@ declare class FileStorageProvider implements StorageProvider {
     getIdentity(): FullIdentity | null;
     connect(): Promise<void>;
     disconnect(): Promise<void>;
+    /**
+     * `data` holds the file's contents only after connect() has read them. Used before
+     * that, this provider answered reads from an empty map and — worse — save() rewrote
+     * the WHOLE file from it, so one set() on a never-connected provider erased the wallet
+     * and every payment journal (#811). Every data method now loads the file first; an
+     * explicit connect() stays the normal path and this is the safety net under it.
+     */
+    private ensureLoaded;
     isConnected(): boolean;
     getStatus(): ProviderStatus;
     get(key: string): Promise<string | null>;

package/dist/impl/nodejs/index.d.ts

@@ -225,6 +225,7 @@ declare class FileStorageProvider implements StorageProvider {
     private isTxtMode;
     private network?;
     private data;
+    private loading;
     private status;
     private _identity;
     constructor(config: FileStorageProviderConfig | string);
@@ -232,6 +233,14 @@ declare class FileStorageProvider implements StorageProvider {
     getIdentity(): FullIdentity | null;
     connect(): Promise<void>;
     disconnect(): Promise<void>;
+    /**
+     * `data` holds the file's contents only after connect() has read them. Used before
+     * that, this provider answered reads from an empty map and — worse — save() rewrote
+     * the WHOLE file from it, so one set() on a never-connected provider erased the wallet
+     * and every payment journal (#811). Every data method now loads the file first; an
+     * explicit connect() stays the normal path and this is the safety net under it.
+     */
+    private ensureLoaded;
     isConnected(): boolean;
     getStatus(): ProviderStatus;
     get(key: string): Promise<string | null>;

package/dist/impl/nodejs/index.js

@@ -23844,6 +23844,7 @@ var FileStorageProvider = class {
   isTxtMode;
   network;
   data = {};
+  loading = null;
   status = "disconnected";
   _identity = null;
   constructor(config) {
@@ -23908,9 +23909,25 @@ var FileStorageProvider = class {
     this.status = "connected";
   }
   async disconnect() {
-    await this.save();
+    if (this.status === "connected") {
+      await this.save();
+    }
     this.status = "disconnected";
   }
+  /**
+   * `data` holds the file's contents only after connect() has read them. Used before
+   * that, this provider answered reads from an empty map and — worse — save() rewrote
+   * the WHOLE file from it, so one set() on a never-connected provider erased the wallet
+   * and every payment journal (#811). Every data method now loads the file first; an
+   * explicit connect() stays the normal path and this is the safety net under it.
+   */
+  async ensureLoaded() {
+    if (this.status === "connected") return;
+    this.loading ??= this.connect().finally(() => {
+      this.loading = null;
+    });
+    await this.loading;
+  }
   isConnected() {
     return this.status === "connected";
   }
@@ -23918,24 +23935,29 @@ var FileStorageProvider = class {
     return this.status;
   }
   async get(key) {
+    await this.ensureLoaded();
     const fullKey = this.getFullKey(key);
     return this.data[fullKey] ?? null;
   }
   async set(key, value) {
+    await this.ensureLoaded();
     const fullKey = this.getFullKey(key);
     this.data[fullKey] = value;
     await this.save();
   }
   async remove(key) {
+    await this.ensureLoaded();
     const fullKey = this.getFullKey(key);
     delete this.data[fullKey];
     await this.save();
   }
   async has(key) {
+    await this.ensureLoaded();
     const fullKey = this.getFullKey(key);
     return fullKey in this.data;
   }
   async keys(prefix) {
+    await this.ensureLoaded();
     const allKeys = Object.keys(this.data);
     if (prefix) {
       return allKeys.filter((k) => k.startsWith(prefix));
@@ -23943,6 +23965,7 @@ var FileStorageProvider = class {
     return allKeys;
   }
   async clear(prefix) {
+    await this.ensureLoaded();
     if (prefix) {
       const keysToDelete = Object.keys(this.data).filter((k) => k.startsWith(prefix));
       for (const key of keysToDelete) {