@agoric/fast-usdc 0.1.1-dev-b1e63c2.0 → 0.1.1-dev-92714b3.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@agoric/fast-usdc",
-  "version": "0.1.1-dev-b1e63c2.0+b1e63c2",
+  "version": "0.1.1-dev-92714b3.0+92714b3",
   "description": "CLI and library for Fast USDC product",
   "type": "module",
   "files": [
@@ -23,9 +23,9 @@
     "lint:eslint": "eslint ."
   },
   "devDependencies": {
-    "@agoric/swingset-liveslots": "0.10.3-dev-b1e63c2.0+b1e63c2",
-    "@agoric/vats": "0.15.2-dev-b1e63c2.0+b1e63c2",
-    "@agoric/zone": "0.2.3-dev-b1e63c2.0+b1e63c2",
+    "@agoric/swingset-liveslots": "0.10.3-dev-92714b3.0+92714b3",
+    "@agoric/vats": "0.15.2-dev-92714b3.0+92714b3",
+    "@agoric/zone": "0.2.3-dev-92714b3.0+92714b3",
     "@fast-check/ava": "^2.0.1",
     "ava": "^5.3.0",
     "c8": "^10.1.2",
@@ -33,16 +33,16 @@
     "ts-blank-space": "^0.4.4"
   },
   "dependencies": {
-    "@agoric/client-utils": "0.1.1-dev-b1e63c2.0+b1e63c2",
-    "@agoric/cosmic-proto": "0.4.1-dev-b1e63c2.0+b1e63c2",
-    "@agoric/ertp": "0.16.3-dev-b1e63c2.0+b1e63c2",
-    "@agoric/internal": "0.3.3-dev-b1e63c2.0+b1e63c2",
-    "@agoric/notifier": "0.6.3-dev-b1e63c2.0+b1e63c2",
-    "@agoric/orchestration": "0.1.1-dev-b1e63c2.0+b1e63c2",
-    "@agoric/store": "0.9.3-dev-b1e63c2.0+b1e63c2",
-    "@agoric/vat-data": "0.5.3-dev-b1e63c2.0+b1e63c2",
-    "@agoric/vow": "0.1.1-dev-b1e63c2.0+b1e63c2",
-    "@agoric/zoe": "0.26.3-dev-b1e63c2.0+b1e63c2",
+    "@agoric/client-utils": "0.1.1-dev-92714b3.0+92714b3",
+    "@agoric/cosmic-proto": "0.4.1-dev-92714b3.0+92714b3",
+    "@agoric/ertp": "0.16.3-dev-92714b3.0+92714b3",
+    "@agoric/internal": "0.3.3-dev-92714b3.0+92714b3",
+    "@agoric/notifier": "0.6.3-dev-92714b3.0+92714b3",
+    "@agoric/orchestration": "0.1.1-dev-92714b3.0+92714b3",
+    "@agoric/store": "0.9.3-dev-92714b3.0+92714b3",
+    "@agoric/vat-data": "0.5.3-dev-92714b3.0+92714b3",
+    "@agoric/vow": "0.1.1-dev-92714b3.0+92714b3",
+    "@agoric/zoe": "0.26.3-dev-92714b3.0+92714b3",
     "@cosmjs/proto-signing": "^0.32.4",
     "@cosmjs/stargate": "^0.32.4",
     "@endo/base64": "^1.0.9",
@@ -82,5 +82,5 @@
   "publishConfig": {
     "access": "public"
   },
-  "gitHead": "b1e63c210080600f38ce7cf29523d77cb0c6434e"
+  "gitHead": "92714b306af99c26bd765f94dce5efea889b66ed"
 }

package/src/exos/advancer.js

@@ -338,7 +338,6 @@ export const prepareAdvancerKit = (
               e,
             );
           }
-          tmpReturnSeat.exit();
         },
         /**
          * @param {Error} error

package/src/exos/settler.js

@@ -13,6 +13,7 @@ import {
   EvmHashShape,
   makeNatAmountShape,
 } from '../type-guards.js';
+import { asMultiset } from '../utils/store.js';
 
 /**
  * @import {FungibleTokenPacketData} from '@agoric/cosmic-proto/ibc/applications/transfer/v2/packet.js';
@@ -149,6 +150,7 @@ export const prepareSettler = (
         ).returns(M.boolean()),
       }),
       self: M.interface('SettlerSelfI', {
+        addMintedEarly: M.call(M.string(), M.nat()).returns(),
         disburse: M.call(EvmHashShape, M.nat()).returns(M.promise()),
         forward: M.call(EvmHashShape, M.nat(), M.string()).returns(),
       }),
@@ -174,8 +176,8 @@ export const prepareSettler = (
         intermediateRecipient: config.intermediateRecipient,
         /** @type {HostInterface<TargetRegistration>|undefined} */
         registration: undefined,
-        /** @type {SetStore<ReturnType<typeof makeMintedEarlyKey>>} */
-        mintedEarly: zone.detached().setStore('mintedEarly'),
+        /** @type {MapStore<ReturnType<typeof makeMintedEarlyKey>, number>} */
+        mintedEarly: zone.detached().mapStore('mintedEarly'),
       };
     },
     {
@@ -221,7 +223,7 @@ export const prepareSettler = (
 
             case PendingTxStatus.Advancing:
               log('⚠️ tap: minted while advancing', nfa, amount);
-              this.state.mintedEarly.add(makeMintedEarlyKey(nfa, amount));
+              self.addMintedEarly(nfa, amount);
               return;
 
             case PendingTxStatus.Observed:
@@ -234,7 +236,7 @@ export const prepareSettler = (
               log('⚠️ tap: minted before observed', nfa, amount);
               // XXX consider capturing in vstorage
               // we would need a new key, as this does not have a txHash
-              this.state.mintedEarly.add(makeMintedEarlyKey(nfa, amount));
+              self.addMintedEarly(nfa, amount);
           }
         },
       },
@@ -256,7 +258,7 @@ export const prepareSettler = (
           const { value: fullValue } = fullAmount;
           const key = makeMintedEarlyKey(forwardingAddress, fullValue);
           if (mintedEarly.has(key)) {
-            mintedEarly.delete(key);
+            asMultiset(mintedEarly).remove(key);
             statusManager.advanceOutcomeForMintedEarly(txHash, success);
             if (success) {
               void this.facets.self.disburse(txHash, fullValue);
@@ -290,7 +292,7 @@ export const prepareSettler = (
               forwardingAddress,
               amount,
             );
-            mintedEarly.delete(key);
+            asMultiset(mintedEarly).remove(key);
             statusManager.advanceOutcomeForUnknownMint(evidence);
             void this.facets.self.forward(txHash, amount, destination.value);
             return true;
@@ -299,6 +301,16 @@ export const prepareSettler = (
         },
       },
       self: {
+        /**
+         * Helper function to track a minted-early transaction by incrementing or initializing its counter
+         * @param {NobleAddress} address
+         * @param {NatValue} amount
+         */
+        addMintedEarly(address, amount) {
+          const key = makeMintedEarlyKey(address, amount);
+          const { mintedEarly } = this.state;
+          asMultiset(mintedEarly).add(key);
+        },
         /**
          * @param {EvmHash} txHash
          * @param {NatValue} fullValue

package/src/fast-usdc.contract.js

@@ -175,7 +175,12 @@ export const contract = async (zcf, privateArgs, zone, tools) => {
       trace('connectToNoble', agoricChainId, nobleChainId, agoricToNoble);
       chainHub.updateConnection(agoricChainId, nobleChainId, agoricToNoble);
       // v1 has `NobleAccount` which we don't expect to ever settle.
-      const nobleAccountV = zone.makeOnce('NobleICA', () => makeNobleAccount());
+      // Including the connection_id in the zone key lets us use
+      // this before and after null-upgrade in multichain-testing.
+      const nobleAccountV = zone.makeOnce(
+        `NobleICA-${agoricToNoble.counterparty.connection_id}`,
+        () => makeNobleAccount(),
+      );
 
       return vowTools.when(nobleAccountV, nobleAccount => {
         trace('nobleAccount', nobleAccount);

package/src/utils/store.js

@@ -0,0 +1,119 @@
+import { Fail } from '@endo/errors';
+
+/**
+ * @import {Key} from '@endo/patterns';
+ */
+
+// TODO provide something like this in a more common place, perhaps as a BagStore
+/**
+ * Creates a bag (multi-set) API that wraps a MapStore where values are counts.
+ *
+ * @template {Key} K
+ * @param {MapStore<K, number>} mapStore
+ */
+export const asMultiset = mapStore =>
+  harden({
+    /**
+     * Add an item to the bag, incrementing its count.
+     *
+     * @param {K} item The item to add
+     * @param {number} [count] How many to add (defaults to 1)
+     */
+    add: (item, count = 1) => {
+      if (count <= 0) {
+        throw Fail`Cannot add a non-positive count ${count} to bag`;
+      }
+
+      if (mapStore.has(item)) {
+        const currentCount = mapStore.get(item);
+        mapStore.set(item, currentCount + count);
+      } else {
+        mapStore.init(item, count);
+      }
+    },
+
+    /**
+     * Remove an item from the bag, decrementing its count. If count reaches
+     * zero, the item is removed completely.
+     *
+     * @param {K} item The item to remove
+     * @param {number} [count] How many to remove (defaults to 1)
+     * @returns {boolean} Whether the removal was successful
+     * @throws {Error} If trying to remove more items than exist
+     */
+    remove: (item, count = 1) => {
+      if (count <= 0) {
+        throw Fail`Cannot remove a non-positive count ${count} from bag`;
+      }
+
+      if (!mapStore.has(item)) {
+        return false;
+      }
+
+      const currentCount = mapStore.get(item);
+      if (currentCount < count) {
+        throw Fail`Cannot remove ${count} of ${item} from bag; only ${currentCount} exist`;
+      }
+
+      if (currentCount === count) {
+        mapStore.delete(item);
+      } else {
+        mapStore.set(item, currentCount - count);
+      }
+      return true;
+    },
+
+    /**
+     * Get the count of an item in the bag.
+     *
+     * @param {K} item The item to check
+     * @returns {number} The count (0 if not present)
+     */
+    count: item => {
+      return mapStore.has(item) ? mapStore.get(item) : 0;
+    },
+
+    /**
+     * Check if the bag contains at least one of the item.
+     *
+     * @param {K} item The item to check
+     * @returns {boolean} Whether the item is in the bag
+     */
+    has: item => {
+      return mapStore.has(item);
+    },
+
+    /**
+     * Get all unique items in the bag.
+     *
+     * @returns {Iterable<K>} Iterable of unique items
+     */
+    keys: () => {
+      return mapStore.keys();
+    },
+
+    /**
+     * Get all entries (item, count) in the bag.
+     *
+     * @returns {Iterable<[K, number]>} Iterable of [item, count] pairs
+     */
+    entries: () => {
+      return mapStore.entries();
+    },
+
+    /**
+     * Get the total number of unique items in the bag.
+     *
+     * @returns {number} Number of unique items
+     */
+    size: () => {
+      return mapStore.getSize();
+    },
+
+    /**
+     * Remove all items from the bag.
+     */
+    clear: () => {
+      mapStore.clear();
+    },
+  });