@agoric/wallet-backend 0.14.4-dev-1933b27.0 → 0.14.4-dev-08f8549.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@agoric/wallet-backend",
-  "version": "0.14.4-dev-1933b27.0+1933b27",
+  "version": "0.14.4-dev-08f8549.0+08f8549",
   "description": "Wallet backend",
   "type": "module",
   "scripts": {
@@ -14,24 +14,24 @@
     "lint:eslint": "eslint ."
   },
   "devDependencies": {
-    "@agoric/vats": "0.15.2-dev-1933b27.0+1933b27",
+    "@agoric/vats": "0.15.2-dev-08f8549.0+08f8549",
     "@endo/bundle-source": "^3.2.3",
     "@endo/far": "^1.1.2",
     "@endo/init": "^1.1.2",
     "ava": "^5.3.0"
   },
   "dependencies": {
-    "@agoric/assert": "0.6.1-dev-1933b27.0+1933b27",
-    "@agoric/cache": "0.3.3-dev-1933b27.0+1933b27",
-    "@agoric/ertp": "0.16.3-dev-1933b27.0+1933b27",
-    "@agoric/internal": "0.3.3-dev-1933b27.0+1933b27",
-    "@agoric/notifier": "0.6.3-dev-1933b27.0+1933b27",
-    "@agoric/smart-wallet": "0.5.4-dev-1933b27.0+1933b27",
-    "@agoric/store": "0.9.3-dev-1933b27.0+1933b27",
-    "@agoric/time": "0.3.3-dev-1933b27.0+1933b27",
-    "@agoric/vat-data": "0.5.3-dev-1933b27.0+1933b27",
-    "@agoric/zoe": "0.26.3-dev-1933b27.0+1933b27",
-    "@agoric/zone": "0.2.3-dev-1933b27.0+1933b27",
+    "@agoric/assert": "0.6.1-dev-08f8549.0+08f8549",
+    "@agoric/cache": "0.3.3-dev-08f8549.0+08f8549",
+    "@agoric/ertp": "0.16.3-dev-08f8549.0+08f8549",
+    "@agoric/internal": "0.3.3-dev-08f8549.0+08f8549",
+    "@agoric/notifier": "0.6.3-dev-08f8549.0+08f8549",
+    "@agoric/smart-wallet": "0.5.4-dev-08f8549.0+08f8549",
+    "@agoric/store": "0.9.3-dev-08f8549.0+08f8549",
+    "@agoric/time": "0.3.3-dev-08f8549.0+08f8549",
+    "@agoric/vat-data": "0.5.3-dev-08f8549.0+08f8549",
+    "@agoric/zoe": "0.26.3-dev-08f8549.0+08f8549",
+    "@agoric/zone": "0.2.3-dev-08f8549.0+08f8549",
     "@endo/eventual-send": "^1.2.2",
     "@endo/marshal": "^1.5.0",
     "@endo/nat": "^5.0.7",
@@ -65,5 +65,5 @@
   "typeCoverage": {
     "atLeast": 0
   },
-  "gitHead": "1933b27bbb06d0292188ecf53379032f9a35b9aa"
+  "gitHead": "08f854940bab55f61cee0edc788811abe0ff5c8b"
 }

package/src/lib-wallet.js

@@ -1,5 +1,4 @@
 // @ts-check
-/* eslint @typescript-eslint/no-floating-promises: "warn" */
 
 /**
  * This file defines the wallet internals without dependency on the ag-solo on
@@ -579,7 +578,7 @@ export function makeWalletRoot({
       status: 'complete',
     });
     idToOffer.set(id, completedOffer);
-    updateInboxState(id, completedOffer);
+    void updateInboxState(id, completedOffer);
   }
 
   /**
@@ -588,8 +587,8 @@ export function makeWalletRoot({
    * @param {string} id
    * @param {ERef<UserSeat>} seat
    */
-  async function subscribeToUpdates(id, seat) {
-    E(E(seat).getExitSubscriber())
+  function subscribeToUpdates(id, seat) {
+    return E(E(seat).getExitSubscriber())
       .subscribeAfter()
       .then(update => updateOrResubscribe(id, seat, update));
   }
@@ -676,7 +675,7 @@ export function makeWalletRoot({
     );
     // By the time Zoe settles the seat promise, the escrow should be complete.
     // Reclaim if it is somehow not.
-    seat.finally(tryReclaimingWithdrawnPayments);
+    void seat.finally(tryReclaimingWithdrawnPayments);
 
     // Even if the seat doesn't settle, we can still pipeline our request for
     // payouts.
@@ -698,7 +697,7 @@ export function makeWalletRoot({
     // Regardless of the status of the offer, we try to clean up any of our
     // unclaimed payments.  Defensively, we want to do this as soon as possible
     // even if the seat doesn't settle.
-    depositedP.finally(tryReclaimingWithdrawnPayments);
+    void depositedP.finally(tryReclaimingWithdrawnPayments);
 
     // Return a promise that will resolve after successful deposit, as well as
     // the promise for the seat.
@@ -791,7 +790,7 @@ export function makeWalletRoot({
     // possible display in the wallet.
     petname = instanceMapping.suggestPetname(petname, instanceHandle);
     // We don't wait for the update before returning.
-    updateAllState();
+    void updateAllState();
     return `instance ${q(petname)} successfully added to wallet`;
   };
 
@@ -829,7 +828,7 @@ export function makeWalletRoot({
     await updatePursesState(petnameForPurse, purse);
 
     // Just notice the balance updates for the purse.
-    observeNotifier(E(purse).getCurrentAmountNotifier(), {
+    void observeNotifier(E(purse).getCurrentAmountNotifier(), {
       updateState(_balance) {
         updatePursesState(purseMapping.valToPetname.get(purse), purse).catch(
           e => console.error('cannot updateState', e),
@@ -1029,7 +1028,7 @@ export function makeWalletRoot({
               petname,
             });
             updateDapp(dappRecord);
-            updateAllState();
+            void updateAllState();
             return dappRecord.actions;
           },
           enable() {
@@ -1151,7 +1150,7 @@ export function makeWalletRoot({
       status: 'decline',
     });
     idToOffer.set(id, declinedOffer);
-    updateInboxState(id, declinedOffer);
+    void updateInboxState(id, declinedOffer);
 
     // Try to reclaim the invitation.
     const compiledOfferP = idToCompiledOfferP.get(id);
@@ -1178,7 +1177,7 @@ export function makeWalletRoot({
           status: 'cancel',
         });
         idToOffer.set(id, cancelledOffer);
-        updateInboxState(id, cancelledOffer);
+        return updateInboxState(id, cancelledOffer);
       })
       .catch(e => console.error(`Cannot cancel offer ${id}:`, e));
 
@@ -1204,7 +1203,7 @@ export function makeWalletRoot({
         error: `${e}`,
       });
       idToOffer.set(id, rejectOffer);
-      updateInboxState(id, rejectOffer);
+      void updateInboxState(id, rejectOffer);
     };
 
     await null;
@@ -1214,7 +1213,7 @@ export function makeWalletRoot({
         status: 'pending',
       });
       idToOffer.set(id, pendingOffer);
-      updateInboxState(id, pendingOffer);
+      void updateInboxState(id, pendingOffer);
       const compiledOffer = await idToCompiledOfferP.get(id);
 
       const { depositedP, seat } = await executeOffer(compiledOffer);
@@ -1226,11 +1225,11 @@ export function makeWalletRoot({
       idToSeat.set(id, seat);
       // The offer might have been postponed, or it might have been immediately
       // consummated. Only subscribe if it was postponed.
-      E(seat)
+      void E(seat)
         .hasExited()
         .then(exited => {
           if (!exited) {
-            subscribeToUpdates(id, seat);
+            return subscribeToUpdates(id, seat);
           }
         });
 
@@ -1253,7 +1252,7 @@ export function makeWalletRoot({
               status: 'accept',
             });
             idToOffer.set(id, acceptedOffer);
-            updateInboxState(id, acceptedOffer);
+            void updateInboxState(id, acceptedOffer);
           }
         })
         .catch(rejected);
@@ -1895,7 +1894,7 @@ export function makeWalletRoot({
   // don't really trust.
   // The param is{import('@agoric/vats/src/vat-bank.js').Bank} but that here triggers https://github.com/Agoric/agoric-sdk/issues/4620
   const importBankAssets = async bank => {
-    observeIteration(
+    void observeIteration(
       subscribeEach(E(bank).getAssetSubscription()),
       harden({
         async updateState({ proposedName, issuerName, issuer, brand }) {

package/src/wallet.js

@@ -1,5 +1,4 @@
 // @ts-check
-/* eslint @typescript-eslint/no-floating-promises: "warn" */
 
 /**
  * This file defines the vat launched by the spawner in the ../deploy.js script.
@@ -173,7 +172,7 @@ export function buildRootObject(vatPowers) {
         await approve();
         const pursesNotifier = walletAdmin.getAttenuatedPursesNotifier();
         const { notifier, updater } = makeNotifierKit();
-        observeIteration(makeApprovedNotifier(pursesNotifier), updater);
+        void observeIteration(makeApprovedNotifier(pursesNotifier), updater);
         return notifier;
       },
       async getCacheCoordinator() {
@@ -202,7 +201,7 @@ export function buildRootObject(vatPowers) {
           }));
         };
 
-        observeIteration(makeApprovedNotifier(offerNotifier), {
+        void observeIteration(makeApprovedNotifier(offerNotifier), {
           updateState(offers) {
             updater.updateState(filteredOffers(offers));
           },
@@ -441,7 +440,7 @@ export function buildRootObject(vatPowers) {
             },
           );
           if (notYetEnabled) {
-            E(otherSide).dappApproved(dappOrigin);
+            await E(otherSide).dappApproved(dappOrigin);
           }
         };
 

package/test/lib-wallet.test.js

@@ -1,5 +1,4 @@
 // @ts-check
-/* eslint @typescript-eslint/no-floating-promises: "warn" */
 import { test as anyTest } from '@agoric/zoe/tools/prepare-test-env-ava.js';
 
 import bundleSource from '@endo/bundle-source';
@@ -1639,7 +1638,7 @@ test('stamps from dateNow', async t => {
   await E(wallet).addPayment(pmt1);
   const { value: val1a, updateCount: count1a } =
     await E(paymentNotifier).getUpdateSince(count0);
-  E(wallet).addPayment(pmt4);
+  void E(wallet).addPayment(pmt4);
   const { value: val1, updateCount: count1 } =
     await E(paymentNotifier).getUpdateSince(count1a);