chore: cleanup chain info (#10108)

refs: #9807 ## Description - Removes `counterparty.prefix.key_prefix: 'FIXME'` from every `agoricNames.chainConnection` entry in vstorage - Removes `agoricNames.chain.agoriclocal` from vstorage ### Security Considerations n/a ### Scaling Considerations Incidentally reduces storage used on chain by writing less data to vstorage ### Documentation Considerations n/a ### Testing Considerations Existing tests were modified to accommodate these deletions. ### Upgrade Considerations We've yet to run `init-orchestration.js`, so this does not have upgrade considerations. It should be part of upgrade-17, though.
Agoric · Sep 18, 2024 · f6ca68a · f6ca68a
2 parents f291362 + 7a1c91c
commit f6ca68a
Show file tree

Hide file tree

Showing 19 changed files with 128 additions and 549 deletions.
diff --git a/packages/boot/test/bootstrapTests/snapshots/orchestration.test.ts.md b/packages/boot/test/bootstrapTests/snapshots/orchestration.test.ts.md
diff --git a/packages/boot/test/bootstrapTests/snapshots/orchestration.test.ts.snap b/packages/boot/test/bootstrapTests/snapshots/orchestration.test.ts.snap
diff --git a/packages/builders/scripts/testing/append-chain-info.js b/packages/builders/scripts/testing/append-chain-info.js
@@ -13,9 +13,6 @@ const chainInfo = {
         counterparty: {
           client_id: '07-tendermint-2',
           connection_id: 'connection-1',
-          prefix: {
-            key_prefix: '',
-          },
         },
         state: 3 /* IBCConnectionState.STATE_OPEN */,
         transferChannel: {

diff --git a/packages/builders/scripts/testing/tweak-chain-info.js b/packages/builders/scripts/testing/tweak-chain-info.js
@@ -16,9 +16,6 @@ const chainInfo = {
         counterparty: {
           client_id: '07-tendermint-2',
           connection_id: 'connection-1',
-          prefix: {
-            key_prefix: '',
-          },
         },
         state: 3 /* IBCConnectionState.STATE_OPEN */,
         transferChannel: {

diff --git a/packages/orchestration/src/chain-info.js b/packages/orchestration/src/chain-info.js
@@ -20,66 +20,15 @@ import { CosmosAssetInfoShape, CosmosChainInfoShape } from './typeGuards.js';
  */
 
 const knownChains = /** @satisfies {Record<string, ChainInfo>} */ (
-  harden({
-    ...fetchedChainInfo,
-    // FIXME does not have useful connections
-    // UNTIL https://github.com/Agoric/agoric-sdk/issues/9492
-    agoriclocal: {
-      chainId: 'agoriclocal',
-      connections: {
-        'cosmoshub-4': {
-          id: 'connection-1',
-          client_id: '07-tendermint-3',
-          counterparty: {
-            client_id: '07-tendermint-2',
-            connection_id: 'connection-1',
-            prefix: {
-              key_prefix: '',
-            },
-          },
-          state: 3 /* IBCConnectionState.STATE_OPEN */,
-          transferChannel: {
-            portId: 'transfer',
-            channelId: 'channel-1',
-            counterPartyChannelId: 'channel-1',
-            counterPartyPortId: 'transfer',
-            ordering: 1 /* Order.ORDER_UNORDERED */,
-            state: 3 /* IBCConnectionState.STATE_OPEN */,
-            version: 'ics20-1',
-          },
-        },
-        osmosislocal: {
-          id: 'connection-0',
-          client_id: '07-tendermint-2',
-          counterparty: {
-            client_id: '07-tendermint-2',
-            connection_id: 'connection-1',
-            prefix: {
-              key_prefix: '',
-            },
-          },
-          state: 3 /* IBCConnectionState.STATE_OPEN */,
-          transferChannel: {
-            portId: 'transfer',
-            channelId: 'channel-0',
-            counterPartyChannelId: 'channel-1',
-            counterPartyPortId: 'transfer',
-            ordering: 1 /* Order.ORDER_UNORDERED */,
-            state: 3 /* IBCConnectionState.STATE_OPEN */,
-            version: 'ics20-1',
-          },
-        },
-      },
-    },
-  })
+  harden(fetchedChainInfo)
 );
 
 /**
  * @typedef {typeof knownChains} KnownChains
  * @internal
  */
 
-// TODO(#9572): include this in registerChain
+// TODO(#9966, #9967): include this in registerChain
 /**
  * Register chain assets into agoricNames
  *

diff --git a/packages/orchestration/src/cosmos-api.ts b/packages/orchestration/src/cosmos-api.ts
@@ -46,9 +46,6 @@ export interface IBCConnectionInfo {
   counterparty: {
     client_id: string;
     connection_id: IBCConnectionID;
-    prefix: {
-      key_prefix: string;
-    };
   };
   transferChannel: {
     portId: string;

diff --git a/packages/orchestration/src/exos/chain-hub.js b/packages/orchestration/src/exos/chain-hub.js
@@ -92,9 +92,6 @@ const reverseConnInfo = connInfo => {
     counterparty: {
       client_id: connInfo.client_id,
       connection_id: connInfo.id,
-      prefix: {
-        key_prefix: 'FIXME',
-      },
     },
     state: connInfo.state,
     transferChannel: {