feat(vats): publish vbank asset list with agoricNames (WIP)

Agoric · Dec 22, 2022 · b8ad33f · b8ad33f
1 parent edfc628
commit b8ad33f
Show file tree

Hide file tree

Showing 3 changed files with 50 additions and 2 deletions.
diff --git a/packages/vats/src/core/basic-behaviors.js b/packages/vats/src/core/basic-behaviors.js
@@ -354,7 +354,13 @@ harden(mintInitialSupply);
  * }} powers
  */
 export const addBankAssets = async ({
-  consume: { initialSupply, bridgeManager, loadCriticalVat, zoe },
+  consume: {
+    agoricNamesAdmin,
+    initialSupply,
+    bridgeManager,
+    loadCriticalVat,
+    zoe,
+  },
   produce: { bankManager, bldIssuerKit },
   installation: {
     consume: { mintHolder },
@@ -384,8 +390,13 @@ export const addBankAssets = async ({
   const bldKit = { mint: bldMint, issuer: bldIssuer, brand: bldBrand };
   bldIssuerKit.resolve(bldKit);
 
+  const assetAdmin = E(agoricNamesAdmin).lookupAdmin('vbankAsset');
+  const nameUpdater = Far('AssetHub', {
+    update: (name, val) => E(assetAdmin).update(name, val),
+  });
   const bankMgr = await E(E(loadCriticalVat)('bank')).makeBankManager(
     bridgeManager,
+    nameUpdater,
   );
   bankManager.resolve(bankMgr);
 

diff --git a/packages/vats/src/core/manifest.js b/packages/vats/src/core/manifest.js
@@ -184,6 +184,7 @@ const SHARED_CHAIN_BOOTSTRAP_MANIFEST = harden({
   },
   [addBankAssets.name]: {
     consume: {
+      agoricNamesAdmin: true,
       initialSupply: true,
       bridgeManager: true,
       // TODO: re-org loadCriticalVat to be subject to permits

diff --git a/packages/vats/src/vat-bank.js b/packages/vats/src/vat-bank.js
@@ -94,6 +94,13 @@ const makePurseController = (
  * @property {string} proposedName
  */
 
+/**
+ * @typedef { AssetDescriptor & {
+ *   issuer: Issuer<'nat'>, // settled identity
+ *   displayInfo: DisplayInfo,
+ * }} AssetInfo
+ */
+
 /**
  * @typedef {object} Bank
  * @property {() => Subscription<AssetDescriptor>} getAssetSubscription Returns
@@ -108,8 +115,13 @@ export function buildRootObject() {
      * @param {ERef<import('./types.js').BridgeManager | undefined>} [bankBridgeManagerP] a bridge
      * manager for the "remote" bank (such as on cosmos-sdk).  If not supplied
      * (such as on sim-chain), we just use local purses.
+     * @param {ERef<{ update: import('./types.js').NameAdmin['update'] }>} [nameAdmin] update facet of
+     *   a NameHub; see addAsset for detail.
      */
-    async makeBankManager(bankBridgeManagerP = undefined) {
+    async makeBankManager(
+      bankBridgeManagerP = undefined,
+      nameAdmin = undefined,
+    ) {
       const bankBridgeManager = await bankBridgeManagerP;
       /** @type {WeakStore<Brand, AssetRecord>} */
       const brandToAssetRecord = makeWeakStore('brand');
@@ -319,6 +331,10 @@ export function buildRootObject() {
 
         /**
          * Add an asset to the bank, and publish it to the subscriptions.
+         * If nameAdmin is defined, update with denom to AssetInfo entry.
+         *
+         * Note that AssetInfo has the settled identity of the issuer,
+         * not just a promise for it.
          *
          * @param {string} denom lower-level denomination string
          * @param {string} issuerName
@@ -361,6 +377,26 @@ export function buildRootObject() {
               proposedName,
             }),
           );
+
+          if (nameAdmin) {
+            // publish settled issuer identity
+            void Promise.all([kit.issuer, E(kit.brand).getDisplayInfo()]).then(
+              ([issuer, displayInfo]) =>
+                E(nameAdmin).update(
+                  denom,
+                  /** @type { AssetInfo } */ (
+                    harden({
+                      brand,
+                      issuer,
+                      issuerName,
+                      denom,
+                      proposedName,
+                      displayInfo,
+                    })
+                  ),
+                ),
+            );
+          }
         },
         getBankForAddress,
       });