applesauce-actions 6.1.0 → 6.2.0

package/dist/action-runner.d.ts

@@ -43,7 +43,7 @@ export declare class ActionRunner {
     publish(event: NostrEvent | NostrEvent[], relays?: string[]): Promise<void>;
     /** Run an action and publish events using the publish method */
     run<Args extends Array<any>>(builder: ActionBuilder<Args>, ...args: Args): Promise<void>;
-    /** Run an action without publishing the events */
+    /** Run an action without waiting for publishing */
     exec<Args extends Array<any>>(builder: ActionBuilder<Args>, ...args: Args): Observable<NostrEvent>;
 }
 /** @deprecated Use ActionRunner instead */

package/dist/action-runner.js

@@ -38,23 +38,23 @@ export class ActionRunner {
             await Promise.all(event.map((e) => this.publish(e, relays)));
             return;
         }
-        if (this.publishMethod) {
-            let result;
-            if ("publish" in this.publishMethod)
-                result = this.publishMethod.publish(event, relays);
-            else if (typeof this.publishMethod === "function")
-                result = this.publishMethod(event, relays);
-            else
-                throw new Error("Invalid publish method");
-            if (isObservable(result)) {
-                await lastValueFrom(result);
-            }
-            else if (result instanceof Promise) {
-                await result;
-            }
-            // Optionally save the event to the store
-            if (this.saveToStore)
-                this.events.add(event);
+        // Save the event to the store first so subscribers (and the UI) update immediately, before the
+        // potentially slow relay publish resolves. The event is already signed and valid at this point, so
+        // there is no reason to gate the local store update on the network round-trip.
+        if (this.saveToStore)
+            this.events.add(event);
+        let result;
+        if ("publish" in this.publishMethod)
+            result = this.publishMethod.publish(event, relays);
+        else if (typeof this.publishMethod === "function")
+            result = this.publishMethod(event, relays);
+        else
+            throw new Error("Invalid publish method");
+        if (isObservable(result)) {
+            await lastValueFrom(result);
+        }
+        else if (result instanceof Promise) {
+            await result;
         }
     }
     /** Run an action and publish events using the publish method */
@@ -63,7 +63,7 @@ export class ActionRunner {
         const context = await this.getContext();
         await builder(...args)(context);
     }
-    /** Run an action without publishing the events */
+    /** Run an action without waiting for publishing */
     exec(builder, ...args) {
         return from(this.getContext()).pipe(
         // Run the action

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "applesauce-actions",
-  "version": "6.1.0",
+  "version": "6.2.0",
   "description": "A package for performing common nostr actions",
   "type": "module",
   "main": "dist/index.js",
@@ -32,14 +32,14 @@
     }
   },
   "dependencies": {
-    "applesauce-common": "^6.1.0",
-    "applesauce-core": "^6.1.0",
+    "applesauce-common": "^6.2.0",
+    "applesauce-core": "^6.2.0",
     "rxjs": "^7.8.1"
   },
   "devDependencies": {
     "@hirez_io/observer-spy": "^2.2.0",
     "@types/debug": "^4.1.12",
-    "applesauce-signers": "^6.0.0",
+    "applesauce-signers": "^6.2.0",
     "nanoid": "^5.1.5",
     "rimraf": "^6.0.1",
     "typescript": "^5.8.3",