fix: adopt VTRANSFER_IBC_EVENT as an action-type (#9671)

closes: #9670

## Description
Ensure that the `VTRANSFER_IBC_EVENT` messages are plumbed through the `cosmic-swingset` layer.

### Security Considerations
Prevents chain halt when the `x/vtransfer` facilities are used.

### Testing Considerations
This bug was overlooked because end-to-end testing of this facility was not completed until a late stage. Earlier unit testing used mocks of the bridge device that did not have the same problematic behaviour. More automated end-to-end IBC testing in a close-to-production setting is needed.

### Upgrade Considerations
Fixing this `cosmic-swingset` behaviour does not require upgrade of any other components.

packages/boot/test/bootstrapTests/vtransfer.test.ts

@@ -6,7 +6,8 @@ import type { TestFn } from 'ava';
 import type { ScopedBridgeManager } from '@agoric/vats';
 import type { TransferMiddleware } from '@agoric/vats/src/transfer.js';
 import type { TransferVat } from '@agoric/vats/src/vat-transfer.js';
-import { BridgeId, VTRANSFER_IBC_EVENT } from '@agoric/internal';
+import { BridgeId } from '@agoric/internal';
+import { VTRANSFER_IBC_EVENT } from '@agoric/internal/src/action-types.js';
 import { makeSwingsetTestKit } from '../../tools/supports.ts';
 
 const makeDefaultTestContext = async t => {

packages/cosmic-swingset/src/launch-chain.js

@@ -581,6 +581,11 @@ export async function launch({
         break;
       }
 
+      case ActionType.VTRANSFER_IBC_EVENT: {
+        p = doBridgeInbound(BRIDGE_ID.VTRANSFER, action, inboundNum);
+        break;
+      }
+
       case ActionType.PLEASE_PROVISION: {
         p = doBridgeInbound(BRIDGE_ID.PROVISION, action, inboundNum);
         break;

packages/internal/src/action-types.js

@@ -15,3 +15,4 @@ export const VBANK_BALANCE_UPDATE = 'VBANK_BALANCE_UPDATE';
 export const WALLET_ACTION = 'WALLET_ACTION';
 export const WALLET_SPEND_ACTION = 'WALLET_SPEND_ACTION';
 export const INSTALL_BUNDLE = 'INSTALL_BUNDLE';
+export const VTRANSFER_IBC_EVENT = 'VTRANSFER_IBC_EVENT';

packages/internal/src/config.js

@@ -30,8 +30,6 @@ export const BridgeId = /** @type {const} */ ({
 harden(BridgeId);
 /** @typedef {(typeof BridgeId)[keyof typeof BridgeId]} BridgeIdValue */
 
-export const VTRANSFER_IBC_EVENT = 'VTRANSFER_IBC_EVENT';
-
 export const CosmosInitKeyToBridgeId = {
   vbankPort: BridgeId.BANK,
   vibcPort: BridgeId.DIBC,

packages/orchestration/test/supports.ts

@@ -1,5 +1,5 @@
 import { makeIssuerKit } from '@agoric/ertp';
-import { VTRANSFER_IBC_EVENT } from '@agoric/internal';
+import { VTRANSFER_IBC_EVENT } from '@agoric/internal/src/action-types.js';
 import { makeFakeStorageKit } from '@agoric/internal/src/storage-test-utils.js';
 import { prepareLocalChainTools } from '@agoric/vats/src/localchain.js';
 import { prepareBridgeTargetModule } from '@agoric/vats/src/bridge-target.js';

packages/vats/src/proposals/transfer-proposal.js

@@ -1,6 +1,7 @@
 // @ts-check
 import { E } from '@endo/far';
-import { BridgeId as BRIDGE_ID, VTRANSFER_IBC_EVENT } from '@agoric/internal';
+import { BridgeId as BRIDGE_ID } from '@agoric/internal';
+import { VTRANSFER_IBC_EVENT } from '@agoric/internal/src/action-types.js';
 import { makeScopedBridge } from '../bridge.js';
 
 /**

packages/vats/src/transfer.js

@@ -2,7 +2,7 @@
 import { Fail, b } from '@endo/errors';
 import { E } from '@endo/far';
 import { M } from '@endo/patterns';
-import { VTRANSFER_IBC_EVENT } from '@agoric/internal';
+import { VTRANSFER_IBC_EVENT } from '@agoric/internal/src/action-types.js';
 import { coerceToByteSource, byteSourceToBase64 } from '@agoric/network';
 import { TargetAppI, AppTransformerI } from './bridge-target.js';
 

packages/vats/test/localchain.test.js

@@ -1,12 +1,13 @@
 // @ts-check
 import { test as anyTest } from '@agoric/swingset-vat/tools/prepare-test-env-ava.js';
 
-import { NonNullish, VTRANSFER_IBC_EVENT } from '@agoric/internal';
+import { NonNullish } from '@agoric/internal';
 import { AmountMath, AssetKind, makeIssuerKit } from '@agoric/ertp';
 import { reincarnate } from '@agoric/swingset-liveslots/tools/setup-vat-data.js';
 import { withAmountUtils } from '@agoric/zoe/tools/test-utils.js';
 import { makeDurableZone } from '@agoric/zone/durable.js';
 import { getInterfaceOf } from '@endo/marshal';
+import { VTRANSFER_IBC_EVENT } from '@agoric/internal/src/action-types.js';
 import { prepareVowTools, heapVowE as E } from '@agoric/vow/vat.js';
 import { prepareLocalChainTools } from '../src/localchain.js';
 import { prepareBridgeTargetModule } from '../src/bridge-target.js';