@agoric/orchestration 0.1.1-dev-e3b1b71.0 → 0.1.1-dev-8718c07.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@agoric/orchestration",
-  "version": "0.1.1-dev-e3b1b71.0+e3b1b71",
+  "version": "0.1.1-dev-8718c07.0+8718c07",
   "description": "Chain abstraction for Agoric's orchestration clients",
   "type": "module",
   "main": "index.js",
@@ -33,19 +33,19 @@
   },
   "homepage": "https://github.com/Agoric/agoric-sdk#readme",
   "dependencies": {
-    "@agoric/async-flow": "0.1.1-dev-e3b1b71.0+e3b1b71",
-    "@agoric/cosmic-proto": "0.4.1-dev-e3b1b71.0+e3b1b71",
-    "@agoric/ertp": "0.16.3-dev-e3b1b71.0+e3b1b71",
-    "@agoric/internal": "0.3.3-dev-e3b1b71.0+e3b1b71",
-    "@agoric/network": "0.1.1-dev-e3b1b71.0+e3b1b71",
-    "@agoric/notifier": "0.6.3-dev-e3b1b71.0+e3b1b71",
-    "@agoric/store": "0.9.3-dev-e3b1b71.0+e3b1b71",
-    "@agoric/time": "0.3.3-dev-e3b1b71.0+e3b1b71",
-    "@agoric/vat-data": "0.5.3-dev-e3b1b71.0+e3b1b71",
-    "@agoric/vats": "0.15.2-dev-e3b1b71.0+e3b1b71",
-    "@agoric/vow": "0.1.1-dev-e3b1b71.0+e3b1b71",
-    "@agoric/zoe": "0.26.3-dev-e3b1b71.0+e3b1b71",
-    "@agoric/zone": "0.2.3-dev-e3b1b71.0+e3b1b71",
+    "@agoric/async-flow": "0.1.1-dev-8718c07.0+8718c07",
+    "@agoric/cosmic-proto": "0.4.1-dev-8718c07.0+8718c07",
+    "@agoric/ertp": "0.16.3-dev-8718c07.0+8718c07",
+    "@agoric/internal": "0.3.3-dev-8718c07.0+8718c07",
+    "@agoric/network": "0.1.1-dev-8718c07.0+8718c07",
+    "@agoric/notifier": "0.6.3-dev-8718c07.0+8718c07",
+    "@agoric/store": "0.9.3-dev-8718c07.0+8718c07",
+    "@agoric/time": "0.3.3-dev-8718c07.0+8718c07",
+    "@agoric/vat-data": "0.5.3-dev-8718c07.0+8718c07",
+    "@agoric/vats": "0.15.2-dev-8718c07.0+8718c07",
+    "@agoric/vow": "0.1.1-dev-8718c07.0+8718c07",
+    "@agoric/zoe": "0.26.3-dev-8718c07.0+8718c07",
+    "@agoric/zone": "0.2.3-dev-8718c07.0+8718c07",
     "@endo/base64": "^1.0.5",
     "@endo/errors": "^1.2.2",
     "@endo/far": "^1.1.2",
@@ -54,7 +54,7 @@
     "@noble/hashes": "github:paulmillr/noble-hashes#ae060da"
   },
   "devDependencies": {
-    "@agoric/swingset-liveslots": "0.10.3-dev-e3b1b71.0+e3b1b71",
+    "@agoric/swingset-liveslots": "0.10.3-dev-8718c07.0+8718c07",
     "@chain-registry/client": "^1.47.4",
     "@cosmjs/amino": "^0.32.3",
     "@cosmjs/proto-signing": "^0.32.3",
@@ -94,5 +94,5 @@
   "typeCoverage": {
     "atLeast": 98.05
   },
-  "gitHead": "e3b1b71b15707cc442dbce757cdece8c70eeca97"
+  "gitHead": "8718c0748521ceeb0f7bab04bbe8fe5d1bc643b0"
 }

package/src/exos/cosmos-interchain-service.d.ts

@@ -4,35 +4,38 @@ export function prepareCosmosInterchainService(zone: Zone, vowTools: VowTools):
      * @param {IBCConnectionID} hostConnectionId the counterparty
      *   connection_id
      * @param {IBCConnectionID} controllerConnectionId self connection_id
+     * @param {ICAChannelAddressOpts} [opts] optional to configure the
+     *   channel address, such as version and ordering
      * @returns {Vow<IcaAccount>}
      */
-    makeAccount(chainId: string, hostConnectionId: IBCConnectionID, controllerConnectionId: IBCConnectionID): Vow<IcaAccount>;
+    makeAccount(chainId: string, hostConnectionId: IBCConnectionID, controllerConnectionId: IBCConnectionID, opts?: ICAChannelAddressOpts | undefined): Vow<IcaAccount>;
     /**
      * @param {IBCConnectionID} controllerConnectionId
+     * @param {string} [version]
      * @returns {Vow<ICQConnection> | ICQConnection}
      */
-    provideICQConnection(controllerConnectionId: IBCConnectionID): Vow<ICQConnection> | ICQConnection;
+    provideICQConnection(controllerConnectionId: IBCConnectionID, version?: string | undefined): Vow<ICQConnection> | ICQConnection;
 }>;
 export type OrchestrationPowers = {
     portAllocator: Remote<PortAllocator>;
+    /**
+     * reserve a state key for future use. can hold
+     * an additional power or a record of powers
+     */
+    reserved: undefined;
 };
-/**
- * PowerStore is used so additional powers can be added on upgrade. See
- * [#7337](https://github.com/Agoric/agoric-sdk/issues/7337) for tracking on Exo
- * state migrations.
- */
-export type PowerStore = MapStore<keyof OrchestrationPowers, OrchestrationPowers[keyof OrchestrationPowers]>;
-export type ICQConnectionStore = MapStore<IBCConnectionID, ICQConnectionKit>;
+export type ICQConnectionStore = MapStore<string, ICQConnectionKit>;
 export type ConnectionKit = ChainAccountKit | ICQConnectionKit;
 export type OrchestrationState = {
-    powers: PowerStore;
     icqConnections: ICQConnectionStore;
-};
+    sharedICQPort: Remote<Port> | undefined;
+} & OrchestrationPowers;
 export type MakeCosmosInterchainService = ReturnType<typeof prepareCosmosInterchainService>;
 export type CosmosInterchainService = ReturnType<MakeCosmosInterchainService>;
 import type { Zone } from '@agoric/base-zone';
 import type { VowTools } from '@agoric/vow';
 import type { IBCConnectionID } from '@agoric/vats';
+import type { ICAChannelAddressOpts } from '../utils/address.js';
 import type { IcaAccount } from '../types.js';
 import type { Vow } from '@agoric/vow';
 import type { ICQConnection } from '../types.js';
@@ -40,4 +43,5 @@ import type { PortAllocator } from '@agoric/network';
 import type { Remote } from '@agoric/internal';
 import type { ICQConnectionKit } from '../types.js';
 import type { ChainAccountKit } from '../types.js';
+import type { Port } from '@agoric/network';
 //# sourceMappingURL=cosmos-interchain-service.d.ts.map

package/src/exos/cosmos-interchain-service.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"cosmos-interchain-service.d.ts","sourceRoot":"","sources":["cosmos-interchain-service.js"],"names":[],"mappings":"AAkPO,qDAHI,IAAI,YACJ,QAAQ;IAnDX;;;;;;OAMG;yBALQ,MAAM,oBACN,eAAe,0BAEf,eAAe,GACb,IAAI,UAAU,CAAC;IAiB5B;;;OAGG;iDAFQ,eAAe,GACb,IAAI,aAAa,CAAC,GAAG,aAAa;GA4CtD;;mBAvOa,OAAO,aAAa,CAAC;;;;;;;yBAQtB,QAAQ,CACpB,MAAU,mBAAmB,EAC7B,mBAAuB,CAAC,MAAM,mBAAmB,CAAC,CAC/C;iCAGU,QAAQ,CAAC,eAAe,EAAE,gBAAgB,CAAC;4BAE3C,eAAe,GAAG,gBAAgB;iCAYlC;IAAE,MAAM,EAAE,UAAU,CAAC;IAAC,cAAc,EAAE,kBAAkB,CAAA;CAAE;0CA8M1D,UAAU,CAAC,OAAO,8BAA8B,CAAC;sCACjD,UAAU,CAAC,2BAA2B,CAAC;0BAvP9B,mBAAmB;8BAKV,aAAa;qCAFX,cAAc;gCAG+B,aAAa;yBAD5D,aAAa;mCACkC,aAAa;mCAJ1C,iBAAiB;4BAD1C,kBAAkB;sCAKoC,aAAa;qCAAb,aAAa"}
+{"version":3,"file":"cosmos-interchain-service.d.ts","sourceRoot":"","sources":["cosmos-interchain-service.js"],"names":[],"mappings":"AA6PO,qDAHI,IAAI,YACJ,QAAQ;IAnEX;;;;;;;;OAQG;yBAPQ,MAAM,oBACN,eAAe,0BAEf,eAAe,6CAGb,IAAI,UAAU,CAAC;IAkB5B;;;;OAIG;iDAHQ,eAAe,iCAEb,IAAI,aAAa,CAAC,GAAG,aAAa;GAwDtD;;mBAjPa,OAAO,aAAa,CAAC;;;;;cACrB,SAAS;;iCAIT,QAAQ,CAAC,MAAM,EAAE,gBAAgB,CAAC;4BAElC,eAAe,GAAG,gBAAgB;iCAGnC;IACZ,cAAkB,EAAE,kBAAkB,CAAC;IACvC,aAAiB,EAAE,OAAO,IAAI,CAAC,GAAG,SAAS,CAAC;CACzC,GAAG,mBAAmB;0CAuOZ,UAAU,CAAC,OAAO,8BAA8B,CAAC;sCACjD,UAAU,CAAC,2BAA2B,CAAC;0BAlQ9B,mBAAmB;8BAKV,aAAa;qCAFX,cAAc;2CAIR,qBAAqB;gCADkB,aAAa;yBAD5D,aAAa;mCACkC,aAAa;mCAJ1C,iBAAiB;4BAD1C,kBAAkB;sCAKoC,aAAa;qCAAb,aAAa;0BAJ1C,iBAAiB"}

package/src/exos/cosmos-interchain-service.js

@@ -1,12 +1,12 @@
 /** @file Orchestration service */
 
-import { Fail, b } from '@endo/errors';
 import { E } from '@endo/far';
-import { M } from '@endo/patterns';
+import { M, mustMatch } from '@endo/patterns';
 import { Shape as NetworkShape } from '@agoric/network';
 import { prepareChainAccountKit } from './chain-account-kit.js';
 import { prepareICQConnectionKit } from './icq-connection-kit.js';
 import {
+  DEFAULT_ICQ_VERSION,
   makeICAChannelAddress,
   makeICQChannelAddress,
 } from '../utils/address.js';
@@ -19,40 +19,39 @@ import {
  * @import {RemoteIbcAddress} from '@agoric/vats/tools/ibc-utils.js';
  * @import {Vow, VowTools} from '@agoric/vow';
  * @import {ICQConnection, IcaAccount, ICQConnectionKit, ChainAccountKit} from '../types.js';
+ * @import {ICAChannelAddressOpts} from '../utils/address.js';
  */
 
 const { Vow$ } = NetworkShape; // TODO #9611
 /**
  * @typedef {object} OrchestrationPowers
  * @property {Remote<PortAllocator>} portAllocator
+ * @property {undefined} reserved reserve a state key for future use. can hold
+ *   an additional power or a record of powers
  */
 
-/**
- * PowerStore is used so additional powers can be added on upgrade. See
- * [#7337](https://github.com/Agoric/agoric-sdk/issues/7337) for tracking on Exo
- * state migrations.
- *
- * @typedef {MapStore<
- *   keyof OrchestrationPowers,
- *   OrchestrationPowers[keyof OrchestrationPowers]
- * >} PowerStore
- */
-
-/** @typedef {MapStore<IBCConnectionID, ICQConnectionKit>} ICQConnectionStore */
+/** @typedef {MapStore<string, ICQConnectionKit>} ICQConnectionStore */
 
 /** @typedef {ChainAccountKit | ICQConnectionKit} ConnectionKit */
 
 /**
- * @template {keyof OrchestrationPowers} K
- * @param {PowerStore} powers
- * @param {K} name
+ * @typedef {{
+ *   icqConnections: ICQConnectionStore;
+ *   sharedICQPort: Remote<Port> | undefined;
+ * } & OrchestrationPowers} OrchestrationState
  */
-const getPower = (powers, name) => {
-  powers.has(name) || Fail`need powers.${b(name)} for this method`;
-  return /** @type {OrchestrationPowers[K]} */ (powers.get(name));
-};
 
-/** @typedef {{ powers: PowerStore; icqConnections: ICQConnectionStore }} OrchestrationState */
+/**
+ * Creates a key for the icqConnections mapStore based on connectionId and
+ * version
+ *
+ * @param {IBCConnectionID} controllerConnectionId
+ * @param {string} [version]
+ * @returns {string}
+ */
+const getICQConnectionKey = (controllerConnectionId, version) => {
+  return `${controllerConnectionId}:${version || DEFAULT_ICQ_VERSION}`;
+};
 
 /**
  * @param {Zone} zone
@@ -62,7 +61,7 @@ const getPower = (powers, name) => {
  */
 const prepareCosmosOrchestrationServiceKit = (
   zone,
-  { watch },
+  { watch, asVow },
   makeChainAccountKit,
   makeICQConnectionKit,
 ) =>
@@ -78,7 +77,7 @@ const prepareCosmosOrchestrationServiceKit = (
         onFulfilled: M.call(M.remotable('Port'))
           .optional({
             remoteConnAddr: M.string(),
-            controllerConnectionId: M.string(),
+            icqLookupKey: M.string(),
           })
           .returns(Vow$(NetworkShape.Connection)),
       }),
@@ -87,31 +86,30 @@ const prepareCosmosOrchestrationServiceKit = (
           .optional(
             M.splitRecord(
               { connectionKit: M.record(), returnFacet: M.string() },
-              { saveICQConnection: M.string() },
+              { icqLookupKey: M.string() },
             ),
           )
           .returns(M.remotable('ConnectionKit Holder facet')),
       }),
       public: M.interface('CosmosInterchainService', {
-        makeAccount: M.call(M.string(), M.string(), M.string()).returns(
-          Vow$(M.remotable('ChainAccountKit')),
-        ),
-        provideICQConnection: M.call(M.string()).returns(
-          Vow$(M.remotable('ICQConnection')),
-        ),
+        makeAccount: M.call(M.string(), M.string(), M.string())
+          .optional(M.record())
+          .returns(Vow$(M.remotable('ChainAccountKit'))),
+        provideICQConnection: M.call(M.string())
+          .optional(M.string())
+          .returns(Vow$(M.remotable('ICQConnection'))),
       }),
     },
-    /** @param {Partial<OrchestrationPowers>} [initialPowers] */
-    initialPowers => {
-      /** @type {PowerStore} */
-      const powers = zone.detached().mapStore('PowerStore');
-      if (initialPowers) {
-        for (const [name, power] of Object.entries(initialPowers)) {
-          powers.init(/** @type {keyof OrchestrationPowers} */ (name), power);
-        }
-      }
+    /** @param {Partial<OrchestrationPowers>} powers */
+    powers => {
+      mustMatch(powers?.portAllocator, M.remotable('PortAllocator'));
       const icqConnections = zone.detached().mapStore('ICQConnections');
-      return /** @type {OrchestrationState} */ ({ powers, icqConnections });
+      return /** @type {OrchestrationState} */ ({
+        icqConnections,
+        sharedICQPort: undefined,
+        reserved: undefined,
+        ...powers,
+      });
     },
     {
       requestICAChannelWatcher: {
@@ -140,19 +138,21 @@ const prepareCosmosOrchestrationServiceKit = (
          * @param {Port} port
          * @param {{
          *   remoteConnAddr: RemoteIbcAddress;
-         *   controllerConnectionId: IBCConnectionID;
+         *   icqLookupKey: string;
          * }} watchContext
          */
-        onFulfilled(port, { remoteConnAddr, controllerConnectionId }) {
+        onFulfilled(port, { remoteConnAddr, icqLookupKey }) {
+          if (!this.state.sharedICQPort) {
+            this.state.sharedICQPort = port;
+          }
           const connectionKit = makeICQConnectionKit(port);
-          /** @param {ICQConnectionKit} kit */
           return watch(
             E(port).connect(remoteConnAddr, connectionKit.connectionHandler),
             this.facets.channelOpenWatcher,
             {
               connectionKit,
               returnFacet: 'connection',
-              saveICQConnection: controllerConnectionId,
+              icqLookupKey,
             },
           );
         },
@@ -168,23 +168,18 @@ const prepareCosmosOrchestrationServiceKit = (
          * @param {{
          *   connectionKit: ConnectionKit;
          *   returnFacet: string;
-         *   saveICQConnection?: IBCConnectionID;
+         *   icqLookupKey?: string;
          * }} watchContext
          */
-        onFulfilled(
-          _connection,
-          { connectionKit, returnFacet, saveICQConnection },
-        ) {
-          if (saveICQConnection) {
+        onFulfilled(_connection, { connectionKit, returnFacet, icqLookupKey }) {
+          if (icqLookupKey) {
             this.state.icqConnections.init(
-              saveICQConnection,
+              icqLookupKey,
               /** @type {ICQConnectionKit} */ (connectionKit),
             );
           }
           return connectionKit[returnFacet];
         },
-        // TODO #9317 if we fail, should we revoke the port (if it was created in this flow)?
-        // onRejected() {}
       },
       public: {
         /**
@@ -192,14 +187,17 @@ const prepareCosmosOrchestrationServiceKit = (
          * @param {IBCConnectionID} hostConnectionId the counterparty
          *   connection_id
          * @param {IBCConnectionID} controllerConnectionId self connection_id
+         * @param {ICAChannelAddressOpts} [opts] optional to configure the
+         *   channel address, such as version and ordering
          * @returns {Vow<IcaAccount>}
          */
-        makeAccount(chainId, hostConnectionId, controllerConnectionId) {
+        makeAccount(chainId, hostConnectionId, controllerConnectionId, opts) {
           const remoteConnAddr = makeICAChannelAddress(
             hostConnectionId,
             controllerConnectionId,
+            opts,
           );
-          const portAllocator = getPower(this.state.powers, 'portAllocator');
+          const { portAllocator } = this.state;
           return watch(
             E(portAllocator).allocateICAControllerPort(),
             this.facets.requestICAChannelWatcher,
@@ -211,29 +209,42 @@ const prepareCosmosOrchestrationServiceKit = (
         },
         /**
          * @param {IBCConnectionID} controllerConnectionId
+         * @param {string} [version]
          * @returns {Vow<ICQConnection> | ICQConnection}
          */
-        provideICQConnection(controllerConnectionId) {
-          if (this.state.icqConnections.has(controllerConnectionId)) {
-            // TODO #9281 do not return synchronously. see https://github.com/Agoric/agoric-sdk/pull/9454#discussion_r1626898694
-            return this.state.icqConnections.get(controllerConnectionId)
-              .connection;
+        provideICQConnection(controllerConnectionId, version) {
+          const icqLookupKey = getICQConnectionKey(
+            controllerConnectionId,
+            version,
+          );
+          if (this.state.icqConnections.has(icqLookupKey)) {
+            return asVow(
+              () => this.state.icqConnections.get(icqLookupKey).connection,
+            );
           }
-          const remoteConnAddr = makeICQChannelAddress(controllerConnectionId);
-          const portAllocator = getPower(this.state.powers, 'portAllocator');
-          return watch(
-            // allocate a new Port for every Connection
-            // TODO #9317 optimize ICQ port allocation
-            E(portAllocator).allocateICQControllerPort(),
-            this.facets.requestICQChannelWatcher,
-            {
-              remoteConnAddr,
-              controllerConnectionId,
-            },
+          const remoteConnAddr = makeICQChannelAddress(
+            controllerConnectionId,
+            version,
           );
+          const { portAllocator, sharedICQPort } = this.state;
+          const portOrPortVow =
+            sharedICQPort || E(portAllocator).allocateICQControllerPort();
+
+          return watch(portOrPortVow, this.facets.requestICQChannelWatcher, {
+            remoteConnAddr,
+            icqLookupKey,
+          });
         },
       },
     },
+    {
+      stateShape: {
+        icqConnections: M.remotable('icqConnections mapStore'),
+        sharedICQPort: M.or(M.remotable('Port'), M.undefined()),
+        portAllocator: M.remotable('PortAllocator'),
+        reserved: M.any(),
+      },
+    },
   );
 
 /**

package/src/proposals/orchestration-proposal.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"orchestration-proposal.d.ts","sourceRoot":"","sources":["orchestration-proposal.js"],"names":[],"mappings":"AAyBO,mJAVI,eAAe,GAAG;IAC5B,OAAW,EAAE;QACb,aAAmB,EAAE,aAAa,CAAC;KAC9B,CAAC;IACN,OAAW,EAAE;QACb,gBAAsB,EAAE,QAAQ,CAAC,GAAG,CAAC,CAAC;KACjC,CAAC;CACH,WACO;IAAE,OAAO,EAAE;QAAE,gBAAgB,EAAE,YAAY,CAAA;KAAE,CAAA;CAAE,iBA6BzD;AAoEM,+FAFI,eAAe,iBAYzB;AAGM;;;;;;;;;;;;;;;;;;;;;;;;;;;;EAsBL;mCAjJ8B,iBAAiB"}
+{"version":3,"file":"orchestration-proposal.d.ts","sourceRoot":"","sources":["orchestration-proposal.js"],"names":[],"mappings":"AAwBO,mJAVI,eAAe,GAAG;IAC5B,OAAW,EAAE;QACb,aAAmB,EAAE,aAAa,CAAC;KAC9B,CAAC;IACN,OAAW,EAAE;QACb,gBAAsB,EAAE,QAAQ,CAAC,GAAG,CAAC,CAAC;KACjC,CAAC;CACH,WACO;IAAE,OAAO,EAAE;QAAE,gBAAgB,EAAE,YAAY,CAAA;KAAE,CAAA;CAAE,iBA6BzD;AAoEM,+FAFI,eAAe,iBAYzB;AAGM;;;;;;;;;;;;;;;;;;;;;;;;;;;;EAsBL;mCAhJ8B,iBAAiB"}

package/src/proposals/orchestration-proposal.js

@@ -9,7 +9,6 @@ const trace = makeTracer('CoreEvalOrchestration', true);
 
 /**
  * @import {PortAllocator} from '@agoric/network';
- * @import {CosmosInterchainService} from '../exos/cosmos-interchain-service.js'
  */
 
 /**

package/src/utils/address.d.ts

@@ -1,13 +1,27 @@
-export function makeICAChannelAddress(hostConnectionId: IBCConnectionID, controllerConnectionId: IBCConnectionID, { version, encoding, ordering, txType, }?: {
+export function makeICAChannelAddress(hostConnectionId: IBCConnectionID, controllerConnectionId: IBCConnectionID, { version, encoding, ordering, txType, }?: ICAChannelAddressOpts | undefined): RemoteIbcAddress;
+export const DEFAULT_ICQ_VERSION: "icq-1";
+export function makeICQChannelAddress(controllerConnectionId: IBCConnectionID, version?: string): RemoteIbcAddress;
+export function findAddressField(remoteAddressString: RemoteIbcAddress): ChainAddress["value"] | undefined;
+export type ICAChannelAddressOpts = {
+    /**
+     * message encoding format for the
+     * channel
+     */
     encoding?: string | undefined;
+    /**
+     * channel ordering.
+     * currently only `ordered` is supported for ics27-1
+     */
     ordering?: "ordered" | "unordered" | undefined;
+    /**
+     * default is `sdk_multi_msg`
+     */
     txType?: string | undefined;
+    /**
+     * default is `ics27-1`
+     */
     version?: string | undefined;
-} | undefined): RemoteIbcAddress;
-export function makeICQChannelAddress(controllerConnectionId: IBCConnectionID, { version }?: {
-    version?: string;
-} | undefined): RemoteIbcAddress;
-export function findAddressField(remoteAddressString: RemoteIbcAddress): ChainAddress["value"] | undefined;
+};
 import type { IBCConnectionID } from '@agoric/vats';
 import type { RemoteIbcAddress } from '@agoric/vats/tools/ibc-utils.js';
 import type { ChainAddress } from '../types.js';

package/src/utils/address.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"address.d.ts","sourceRoot":"","sources":["address.js"],"names":[],"mappings":"AAoBO,wDAXI,eAAe,0BACf,eAAe;;;;;gBAQb,gBAAgB,CAuB5B;AAQM,8DAJI,eAAe;cACH,MAAM;gBAChB,gBAAgB,CAQ5B;AAaM,sDAJI,gBAAgB,GAEd,YAAY,CAAC,OAAO,CAAC,GAAG,SAAS,CAW7C;qCA1EiC,cAAc;sCAEb,iCAAiC;kCADrC,aAAa"}
+{"version":3,"file":"address.d.ts","sourceRoot":"","sources":["address.js"],"names":[],"mappings":"AAwBO,wDALI,eAAe,0BACf,eAAe,iFAEb,gBAAgB,CAuB5B;AAGD,0CAA2C;AAOpC,8DAJI,eAAe,YACf,MAAM,GACJ,gBAAgB,CAQ5B;AAaM,sDAJI,gBAAgB,GAEd,YAAY,CAAC,OAAO,CAAC,GAAG,SAAS,CAW7C;;;;;;;;;;;;;;;;;;;;;qCAhFiC,cAAc;sCAEb,iCAAiC;kCADrC,aAAa"}

package/src/utils/address.js

@@ -6,16 +6,20 @@ import { Fail } from '@endo/errors';
  * @import {RemoteIbcAddress} from '@agoric/vats/tools/ibc-utils.js';
  */
 
+/**
+ * @typedef {object} ICAChannelAddressOpts
+ * @property {string} [encoding='proto3'] message encoding format for the
+ *   channel
+ * @property {'ordered' | 'unordered'} [ordering='ordered'] channel ordering.
+ *   currently only `ordered` is supported for ics27-1
+ * @property {string} [txType='sdk_multi_msg'] default is `sdk_multi_msg`
+ * @property {string} [version='ics27-1'] default is `ics27-1`
+ */
+
 /**
  * @param {IBCConnectionID} hostConnectionId Counterparty Connection ID
  * @param {IBCConnectionID} controllerConnectionId Self Connection ID
- * @param {object} [opts]
- * @param {string} [opts.encoding] - message encoding format for the channel.
- *   default is `proto3`
- * @param {'ordered' | 'unordered'} [opts.ordering] - channel ordering.
- *   currently only `ordered` is supported for ics27-1
- * @param {string} [opts.txType] - default is `sdk_multi_msg`
- * @param {string} [opts.version] - default is `ics27-1`
+ * @param {ICAChannelAddressOpts} [opts]
  * @returns {RemoteIbcAddress}
  */
 export const makeICAChannelAddress = (
@@ -42,14 +46,16 @@ export const makeICAChannelAddress = (
 };
 harden(makeICAChannelAddress);
 
+export const DEFAULT_ICQ_VERSION = 'icq-1';
+
 /**
  * @param {IBCConnectionID} controllerConnectionId
- * @param {{ version?: string }} [opts]
+ * @param {string} version defaults to icq-1
  * @returns {RemoteIbcAddress}
  */
 export const makeICQChannelAddress = (
   controllerConnectionId,
-  { version = 'icq-1' } = {},
+  version = DEFAULT_ICQ_VERSION,
 ) => {
   controllerConnectionId || Fail`controllerConnectionId is required`;
   return `/ibc-hop/${controllerConnectionId}/ibc-port/icqhost/unordered/${version}`;

package/src/vat-orchestration.d.ts

@@ -1,14 +1,14 @@
 export function buildRootObject(_vatPowers: any, _args: any, baggage: any): {
     /** @param {Partial<OrchestrationPowers>} [initialPowers] */
     makeCosmosInterchainService(initialPowers?: Partial<OrchestrationPowers> | undefined): import("@endo/exo").Guarded<{
-        makeAccount(chainId: string, hostConnectionId: import("@agoric/vats").IBCConnectionID, controllerConnectionId: import("@agoric/vats").IBCConnectionID): import("@agoric/vow").Vow<import("./cosmos-api.js").IcaAccount>;
-        provideICQConnection(controllerConnectionId: import("@agoric/vats").IBCConnectionID): import("@agoric/vow").Vow<import("./types.js").ICQConnection> | import("./types.js").ICQConnection;
+        makeAccount(chainId: string, hostConnectionId: import("@agoric/vats").IBCConnectionID, controllerConnectionId: import("@agoric/vats").IBCConnectionID, opts?: import("./utils/address.js").ICAChannelAddressOpts | undefined): import("@agoric/vow").Vow<import("./cosmos-api.js").IcaAccount>;
+        provideICQConnection(controllerConnectionId: import("@agoric/vats").IBCConnectionID, version?: string | undefined): import("@agoric/vow").Vow<import("./types.js").ICQConnection> | import("./types.js").ICQConnection;
     }>;
 } & import("@endo/pass-style").RemotableObject<`Alleged: ${string}`> & import("@endo/eventual-send").RemotableBrand<{}, {
     /** @param {Partial<OrchestrationPowers>} [initialPowers] */
     makeCosmosInterchainService(initialPowers?: Partial<OrchestrationPowers> | undefined): import("@endo/exo").Guarded<{
-        makeAccount(chainId: string, hostConnectionId: import("@agoric/vats").IBCConnectionID, controllerConnectionId: import("@agoric/vats").IBCConnectionID): import("@agoric/vow").Vow<import("./cosmos-api.js").IcaAccount>;
-        provideICQConnection(controllerConnectionId: import("@agoric/vats").IBCConnectionID): import("@agoric/vow").Vow<import("./types.js").ICQConnection> | import("./types.js").ICQConnection;
+        makeAccount(chainId: string, hostConnectionId: import("@agoric/vats").IBCConnectionID, controllerConnectionId: import("@agoric/vats").IBCConnectionID, opts?: import("./utils/address.js").ICAChannelAddressOpts | undefined): import("@agoric/vow").Vow<import("./cosmos-api.js").IcaAccount>;
+        provideICQConnection(controllerConnectionId: import("@agoric/vats").IBCConnectionID, version?: string | undefined): import("@agoric/vow").Vow<import("./types.js").ICQConnection> | import("./types.js").ICQConnection;
     }>;
 }>;
 export type OrchestrationVat = ReturnType<typeof buildRootObject>;