fix(swingset): merge bundle/vat-admin devices, add waitForBundleCap()

Simplify the user experience by removing `devices.bundle` and merging its
functionality into vat-admin. Use mid-capitalized "bundleCap" in the
API (e.g. `getNamedBundleCap` instead of `getNamedBundlecap`).

* bootstrap() can go back to doing `vatAdminService =
E(vats.vatAdmin).createVatAdminService(devices.vatAdmin)` ,
instead of `createVatAdminService(devices.vatAdmin, devices.bundle)`
* you can get bundlecaps from `E(vatAdminService).getBundleCap` or
`.getNamedBundleCap`, you no longer need `D(devices.bundle)` for those
* `E(vatAdminService).waitForBundleCap(bundleID)` gives you a Promise that
fires when (and if) the bundle has been installed, so you can e.g. begin a
`zoe.install()` before the bundle is installed (closes #4521)

This also introduces a better way to export types from swingset, defines a
VatAdminRootDeviceNode type for use by clients who think they need that, and
exports a few ancillary types to support it.

closes #4566

packages/SwingSet/src/index.js

@@ -12,3 +12,6 @@ export { buildTimer } from './devices/timer.js';
 export { buildBridge } from './devices/bridge.js';
 export { default as buildCommand } from './devices/command.js';
 export { buildPlugin } from './devices/plugin.js';
+
+// eslint-disable-next-line import/export
+export * from './types-exported.js';

packages/SwingSet/src/initializeSwingset.js

@@ -53,7 +53,6 @@ export async function buildKernelBundles(options = {}) {
     kernel: src('./kernel/kernel.js'),
     adminDevice: src('./kernel/vatAdmin/vatAdmin-src'),
     adminVat: src('./kernel/vatAdmin/vatAdminWrapper'),
-    bundleDevice: src('./devices/bundle'),
     comms: src('./vats/comms'),
     vattp: src('./vats/vat-tp'),
     timer: src('./vats/vat-timerWrapper'),
@@ -327,9 +326,6 @@ export async function initializeSwingset(
   config.devices.vatAdmin = {
     bundle: kernelBundles.adminDevice,
   };
-  config.devices.bundle = {
-    bundle: kernelBundles.bundleDevice,
-  };
 
   // comms vat is added automatically, but TODO: bootstraps must still
   // connect it to vat-tp. TODO: test-message-patterns builds two comms and
@@ -365,7 +361,7 @@ export async function initializeSwingset(
   // The 'bundleName' option points into
   // config.bundles.BUNDLENAME.[bundle|bundleSpec|sourceSpec] , which can
   // also include arbitrary named bundles that will be made available to
-  // D(devices.bundle).getNamedBundlecap(bundleName) ,and temporarily as
+  // E(vatAdminService).getNamedBundleCap(bundleName) ,and temporarily as
   // E(vatAdminService).createVatByName(bundleName)
 
   // The 'kconfig' we pass through to initializeKernel has

packages/SwingSet/src/kernel/initializeKernel.js

@@ -93,7 +93,8 @@ export function initializeKernel(config, hostStorage, verbose = false) {
       }
       if (name === 'vatAdmin') {
         // Create a kref for the vatAdmin root, so the kernel can tell it
-        // about creation/termination of dynamic vats.
+        // about creation/termination of dynamic vats, and the installation
+        // of bundles
         const kref = exportRootObject(kernelKeeper, vatID);
         // Pin, to prevent it being GC'd when only the kvStore points to it
         kernelKeeper.pinObject(kref);

packages/SwingSet/src/kernel/kernel.js

@@ -1009,6 +1009,9 @@ export default function buildKernel(
 
     // the admin device is endowed directly by the kernel
     deviceEndowments.vatAdmin = {
+      hasBundle: kernelKeeper.hasBundle,
+      getBundle: kernelKeeper.getBundle,
+      getNamedBundleID: kernelKeeper.getNamedBundleID,
       pushCreateVatBundleEvent(bundle, dynamicOptions) {
         const source = { bundle };
         const vatID = kernelKeeper.allocateUnusedVatID();
@@ -1028,10 +1031,6 @@ export default function buildKernel(
         // later when it is created and a root object is available
         return vatID;
       },
-      getBundleIDByName(bundleName) {
-        // this throws if the name is unknown
-        return kernelKeeper.getNamedBundleID(bundleName);
-      },
       terminate: (vatID, reason) => terminateVat(vatID, true, reason),
       meterCreate: (remaining, threshold) =>
         kernelKeeper.allocateMeter(remaining, threshold),
@@ -1042,13 +1041,6 @@ export default function buildKernel(
       meterGet: meterID => kernelKeeper.getMeter(meterID),
     };
 
-    // same for bundle device
-    deviceEndowments.bundle = {
-      hasBundle: kernelKeeper.hasBundle,
-      getBundle: kernelKeeper.getBundle,
-      getNamedBundleID: kernelKeeper.getNamedBundleID,
-    };
-
     // instantiate all devices
     for (const [name, deviceID] of kernelKeeper.getDevices()) {
       logStartup(`starting device ${name} as ${deviceID}`);
@@ -1217,6 +1209,15 @@ export default function buildKernel(
     // bundleID is b1-HASH
     if (!kernelKeeper.hasBundle(bundleID)) {
       kernelKeeper.addBundle(bundleID, bundle);
+      const args = harden({ body: JSON.stringify([bundleID]), slots: [] });
+      if (vatAdminRootKref) {
+        // TODO: consider 'panic' instead of 'logFailure'
+        queueToKref(vatAdminRootKref, 'bundleInstalled', args, 'logFailure');
+      } else {
+        // this should only happen during unit tests that are too lazy to
+        // build a complete kernel: test/bundles/test-bundles-kernel.js
+        console.log(`installBundle cannot notify, missing vatAdminRootKref`);
+      }
       kernelKeeper.commitCrank();
     }
   }