@agoric/cosmos 0.35.0-u19.2 → 0.35.0-u20.0

package/CHANGELOG.md

@@ -3,23 +3,7 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
-## [0.35.0-u19.2](https://github.com/Agoric/agoric-sdk/compare/@agoric/cosmos@0.35.0-u19.1...@agoric/cosmos@0.35.0-u19.2) (2025-03-13)
-
-**Note:** Version bump only for package @agoric/cosmos
-
-
-
-
-
-## [0.35.0-u19.1](https://github.com/Agoric/agoric-sdk/compare/@agoric/cosmos@0.35.0-u19.0...@agoric/cosmos@0.35.0-u19.1) (2025-03-03)
-
-**Note:** Version bump only for package @agoric/cosmos
-
-
-
-
-
-## [0.35.0-u19.0](https://github.com/Agoric/agoric-sdk/compare/@agoric/cosmos@0.34.1...@agoric/cosmos@0.35.0-u19.0) (2025-02-24)
+## [0.35.0-u20.0](https://github.com/Agoric/agoric-sdk/compare/@agoric/cosmos@0.34.1...@agoric/cosmos@0.35.0-u20.0) (2025-04-16)
 
 
 ### ⚠ BREAKING CHANGES
@@ -64,7 +48,8 @@ See [Conventional Commits](https://conventionalcommits.org) for commit guideline
 * **cosmos:** impose defaults when sending VM actions ([a710d68](https://github.com/Agoric/agoric-sdk/commit/a710d68512cf9983bdf5230e2e99f267521c7210))
 * **cosmos:** KVEntry implements json Marshaler and Unmarshaller ([1bba859](https://github.com/Agoric/agoric-sdk/commit/1bba8592eee0e24e480c407095f0f63ccca4a242))
 * **cosmos:** make vm comms use `context.Context` ([fa1754c](https://github.com/Agoric/agoric-sdk/commit/fa1754c791ba6c848e6b94f3a75051c6b4cc69f6))
-* **cosmos:** Next Upgrade is agoric-upgrade-19 ([e224889](https://github.com/Agoric/agoric-sdk/commit/e2248899164cf58ecdd086801453a8290fe222df))
+* **cosmos:** new `IBCPacket` helpers to tame JSON ([f5a10b6](https://github.com/Agoric/agoric-sdk/commit/f5a10b6457501c9be9ae396bd1c7a8d43ea42dd6))
+* **cosmos:** Next upgrade is agoric-upgrade-20 ([2631f84](https://github.com/Agoric/agoric-sdk/commit/2631f847b827a284c4ad11632c7f0ee914f78950))
 * **cosmos:** prevent VM port handlers from panicking ([afd6017](https://github.com/Agoric/agoric-sdk/commit/afd60170d453865cfa871a01e8d8a74ffef9221a))
 * **cosmos:** propagate and handle shutdown message ([ac12e6d](https://github.com/Agoric/agoric-sdk/commit/ac12e6d57b3a50e079744f7a6faced7f4bebb957))
 * **cosmos:** separate swing-store export data from genesis file ([f476bd5](https://github.com/Agoric/agoric-sdk/commit/f476bd50fb3ca5df0c4f43b20e059e05a2659b4c))
@@ -74,8 +59,10 @@ See [Conventional Commits](https://conventionalcommits.org) for commit guideline
 * **cosmos:** support core proposals set by upgrade handler ([605eb4b](https://github.com/Agoric/agoric-sdk/commit/605eb4b8f33d7646c3a9084d43ecd51029e12b80))
 * **cosmos:** support snapshot export ([4386f8e](https://github.com/Agoric/agoric-sdk/commit/4386f8e67136f184b94febb5d65990a26e101cf3))
 * **cosmos:** un-wire x/crisis ([#8582](https://github.com/Agoric/agoric-sdk/issues/8582)) ([7153535](https://github.com/Agoric/agoric-sdk/commit/7153535c5c10fed309dc60f12f981c81841fdb93))
+* **cosmos:** upgrade IBC vat for next release ([c994490](https://github.com/Agoric/agoric-sdk/commit/c99449081560480e7e2dd6fc069b12dbcc630370))
 * **cosmos:** upgrade provisioning vat ([#8901](https://github.com/Agoric/agoric-sdk/issues/8901)) ([174e37d](https://github.com/Agoric/agoric-sdk/commit/174e37d7499b372c33ecaf6e05f82f43ebfff902))
 * **cosmos:** upgrade skips proposals with no variant ([4ce1372](https://github.com/Agoric/agoric-sdk/commit/4ce13721201d9a62a40352d909f97a44d8e5b25d))
+* **cosmos:** upgrade vat-bank ([8727b79](https://github.com/Agoric/agoric-sdk/commit/8727b79cf7b9e520630b793ea963a9f375d9707a))
 * **cosmos:** use `x/vbank` ConsensusVersion to upgrade monitoring ([0e367d3](https://github.com/Agoric/agoric-sdk/commit/0e367d3e9870622acc8a38afe57c2b6cbe629341))
 * **cosmos:** use operational artifact level for swingset state export/restore ([161ddd3](https://github.com/Agoric/agoric-sdk/commit/161ddd34ca6c16da0ad3ef716fa5da3d2ba86b68))
 * **cosmos:** wire in vlocalchain to the Cosmos app ([3ea527d](https://github.com/Agoric/agoric-sdk/commit/3ea527d9844dcf2b5f2c60d1bfb1760e064ec0f7))
@@ -138,6 +125,7 @@ See [Conventional Commits](https://conventionalcommits.org) for commit guideline
 * **builders:** use proper `oracleBrand` subkey case ([52f02b7](https://github.com/Agoric/agoric-sdk/commit/52f02b75b6706ee455a32ff83617dd5afb7342a7))
 * **cosmic-swingset:** add missing bits for maxVatsOnline ([8c0c177](https://github.com/Agoric/agoric-sdk/commit/8c0c17752f7439db6f7aee9f88be1dedce2a1bf1))
 * **cosmic-swingset:** plumbing for maxVatsOnline ([45a759a](https://github.com/Agoric/agoric-sdk/commit/45a759a71c8abc724618a12dfd8ae72552b9783e))
+* **cosmos-vstorage:** split 'size_delta' counter into 'size_increase' and 'size_decrease' ([#11063](https://github.com/Agoric/agoric-sdk/issues/11063)) ([3825031](https://github.com/Agoric/agoric-sdk/commit/38250315cedf68dc801b75bee867818a3846fae7)), closes [#11062](https://github.com/Agoric/agoric-sdk/issues/11062) [#10938](https://github.com/Agoric/agoric-sdk/issues/10938) [/github.com/Agoric/agoric-sdk/pull/10997#discussion_r1956583919](https://github.com/Agoric//github.com/Agoric/agoric-sdk/pull/10997/issues/discussion_r1956583919)
 * **cosmos:** add action context to core evals ([7cfae88](https://github.com/Agoric/agoric-sdk/commit/7cfae88a1d2c31d91030a8742972d9f03d331059))
 * **cosmos:** Add support for iavl options ([c6770ec](https://github.com/Agoric/agoric-sdk/commit/c6770ece8dd7f3f56c40b3cb80be245a99e01c9d))
 * **cosmos:** correctly detect presence of Agoric VM ([c7e266e](https://github.com/Agoric/agoric-sdk/commit/c7e266e035cc454bf74e88e96959f862ece146f8))

package/app/app.go

@@ -894,7 +894,7 @@ func NewAgoricApp(
 	for _, name := range upgradeNamesOfThisVersion {
 		app.UpgradeKeeper.SetUpgradeHandler(
 			name,
-			upgrade19Handler(app, name),
+			upgrade20Handler(app, name),
 		)
 	}
 

package/app/upgrade.go

@@ -3,9 +3,6 @@ package gaia
 import (
 	"encoding/json"
 	"fmt"
-	"reflect"
-	"strings"
-	"text/template"
 
 	"github.com/Agoric/agoric-sdk/golang/cosmos/vm"
 	swingsetkeeper "github.com/Agoric/agoric-sdk/golang/cosmos/x/swingset/keeper"
@@ -15,11 +12,7 @@ import (
 )
 
 var upgradeNamesOfThisVersion = []string{
-	"agoric-upgrade-19-mainnet",
-	"agoric-upgrade-19-devnet",
-	"agoric-upgrade-19-emerynet",
-	"agoric-upgrade-19-a3p",
-	"agoric-upgrade-19-basic",
+	"agoric-upgrade-20",
 }
 
 // isUpgradeNameOfThisVersion returns whether the provided plan name is a
@@ -53,11 +46,7 @@ func isPrimaryUpgradeName(name string) bool {
 		return false
 	}
 	switch name {
-	case validUpgradeName("agoric-upgrade-19-mainnet"),
-		validUpgradeName("agoric-upgrade-19-devnet"),
-		validUpgradeName("agoric-upgrade-19-emerynet"),
-		validUpgradeName("agoric-upgrade-19-a3p"),
-		validUpgradeName("agoric-upgrade-19-basic"):
+	case validUpgradeName("agoric-upgrade-20"):
 		return true
 	default:
 		panic(fmt.Errorf("unexpected upgrade name %s", validUpgradeName(name)))
@@ -75,60 +64,45 @@ func isFirstTimeUpgradeOfThisVersion(app *GaiaApp, ctx sdk.Context) bool {
 	return true
 }
 
-func buildProposalStepWithArgs(moduleName string, entrypoint string, extra any) (vm.CoreProposalStep, error) {
-	t := template.Must(template.New("").Parse(`{
-  "module": "{{.moduleName}}",
-  "entrypoint": "{{.entrypoint}}",
-  "args": {{.args}}
-}`))
-
-	var args []byte
-	var err error
-	if extra == nil {
-		// The specified entrypoint will be called with no extra arguments after powers.
-		args = []byte(`[]`)
-	} else if reflect.TypeOf(extra).Kind() == reflect.Map && reflect.TypeOf(extra).Key().Kind() == reflect.String {
-		// The specified entrypoint will be called with this options argument after powers.
-		args, err = json.Marshal([]any{extra})
-	} else if reflect.TypeOf(extra).Kind() == reflect.Slice {
-		// The specified entrypoint will be called with each of these arguments after powers.
-		args, err = json.Marshal(extra)
-	} else {
-		return nil, fmt.Errorf("proposal extra must be nil, array, or string map, not %v", extra)
-	}
+// buildProposalStepWithArgs returns a CoreProposal representing invocation of
+// the specified module-specific entry point with arbitrary Jsonable arguments
+// provided after core-eval powers.
+func buildProposalStepWithArgs(moduleName string, entrypoint string, args ...vm.Jsonable) (vm.CoreProposalStep, error) {
+	argsBz, err := json.Marshal(args)
 	if err != nil {
 		return nil, err
 	}
 
-	var result strings.Builder
-	err = t.Execute(&result, map[string]any{
-		"moduleName": moduleName,
-		"entrypoint": entrypoint,
-		"args":       string(args),
-	})
+	mea := struct {
+		Module     string          `json:"module"`
+		Entrypoint string          `json:"entrypoint"`
+		Args       json.RawMessage `json:"args"`
+	}{
+		Module:     moduleName,
+		Entrypoint: entrypoint,
+		Args:       argsBz,
+	}
+
+	jsonBz, err := json.Marshal(mea)
 	if err != nil {
 		return nil, err
 	}
-	jsonStr := result.String()
-	jsonBz := []byte(jsonStr)
-	if !json.Valid(jsonBz) {
-		return nil, fmt.Errorf("invalid JSON: %s", jsonStr)
-	}
+
 	proposal := vm.ArbitraryCoreProposal{Json: jsonBz}
 	return vm.CoreProposalStepForModules(proposal), nil
 }
 
 func getVariantFromUpgradeName(upgradeName string) string {
 	switch upgradeName {
-	case "agoric-upgrade-19-a3p":
+	case "UNRELEASED_A3P_INTEGRATION":
 		return "A3P_INTEGRATION"
-	case "agoric-upgrade-19-mainnet":
+	case "UNRELEASED_main":
 		return "MAINNET"
-	case "agoric-upgrade-19-devnet":
+	case "UNRELEASED_devnet":
 		return "DEVNET"
-	case "agoric-upgrade-19-emerynet":
+	case "UNRELEASED_emerynet":
 		return "EMERYNET"
-	case "agoric-upgrade-19-basic":
+	case "UNRELEASED_BASIC":
 		// Noupgrade for this version.
 		return ""
 	default:
@@ -136,14 +110,6 @@ func getVariantFromUpgradeName(upgradeName string) string {
 	}
 }
 
-func upgradeMintHolderCoreProposal(targetUpgrade string) (vm.CoreProposalStep, error) {
-	return buildProposalStepFromScript(targetUpgrade, "@agoric/builders/scripts/vats/upgrade-mintHolder.js")
-}
-
-func restartFeeDistributorCoreProposal(targetUpgrade string) (vm.CoreProposalStep, error) {
-	return buildProposalStepFromScript(targetUpgrade, "@agoric/builders/scripts/inter-protocol/replace-feeDistributor-combo.js")
-}
-
 func buildProposalStepFromScript(targetUpgrade string, builderScript string) (vm.CoreProposalStep, error) {
 	variant := getVariantFromUpgradeName(targetUpgrade)
 
@@ -154,14 +120,17 @@ func buildProposalStepFromScript(targetUpgrade string, builderScript string) (vm
 	return buildProposalStepWithArgs(
 		builderScript,
 		"defaultProposalBuilder",
-		map[string]any{
-			"variant": variant,
+		// Map iteration is randomised; use an anonymous struct instead.
+		struct {
+			Variant string `json:"variant"`
+		}{
+			Variant: variant,
 		},
 	)
 }
 
-// upgrade19Handler performs standard upgrade actions plus custom actions for upgrade-19.
-func upgrade19Handler(app *GaiaApp, targetUpgrade string) func(sdk.Context, upgradetypes.Plan, module.VersionMap) (module.VersionMap, error) {
+// upgrade20Handler performs standard upgrade actions plus custom actions for upgrade-20.
+func upgrade20Handler(app *GaiaApp, targetUpgrade string) func(sdk.Context, upgradetypes.Plan, module.VersionMap) (module.VersionMap, error) {
 	return func(ctx sdk.Context, plan upgradetypes.Plan, fromVm module.VersionMap) (module.VersionMap, error) {
 		app.CheckControllerInited(false)
 
@@ -180,6 +149,11 @@ func upgrade19Handler(app *GaiaApp, targetUpgrade string) func(sdk.Context, upgr
 			// Each CoreProposalStep runs sequentially, and can be constructed from
 			// one or more modules executing in parallel within the step.
 			CoreProposalSteps = append(CoreProposalSteps,
+				// Orchestration vats: Fix memory leak in vow tools
+				// vat-ibc (included in orchestration): Accommodate string sequence numbers.
+				vm.CoreProposalStepForModules(
+					"@agoric/builders/scripts/vats/upgrade-orchestration.js",
+				),
 				// Register a new ZCF to be used for all future contract instances and upgrades
 				vm.CoreProposalStepForModules(
 					"@agoric/builders/scripts/vats/upgrade-zcf.js",
@@ -189,40 +163,11 @@ func upgrade19Handler(app *GaiaApp, targetUpgrade string) func(sdk.Context, upgr
 				vm.CoreProposalStepForModules(
 					"@agoric/builders/scripts/smart-wallet/build-wallet-factory2-upgrade.js",
 				),
-			)
-
-			upgradeMintHolderStep, err := upgradeMintHolderCoreProposal(targetUpgrade)
-			if err != nil {
-				return nil, err
-			} else if upgradeMintHolderStep != nil {
-				CoreProposalSteps = append(CoreProposalSteps, upgradeMintHolderStep)
-			}
-			restartFeeDistributorStep, err := restartFeeDistributorCoreProposal(targetUpgrade)
-			if err != nil {
-				return nil, err
-			} else if restartFeeDistributorStep != nil {
-				CoreProposalSteps = append(CoreProposalSteps, restartFeeDistributorStep)
-			}
-
-			CoreProposalSteps = append(CoreProposalSteps,
-				vm.CoreProposalStepForModules(
-					"@agoric/builders/scripts/vats/upgrade-paRegistry.js",
-				),
-				vm.CoreProposalStepForModules(
-					"@agoric/builders/scripts/vats/upgrade-provisionPool.js",
-				),
+				// vat-bank is slowly leaking, possibly because of the liveslots resolved promise leak
+				// (https://github.com/Agoric/agoric-sdk/issues/11118). Restart to pick up the fix.
 				vm.CoreProposalStepForModules(
 					"@agoric/builders/scripts/vats/upgrade-bank.js",
 				),
-				vm.CoreProposalStepForModules(
-					"@agoric/builders/scripts/vats/upgrade-agoricNames.js",
-				),
-				vm.CoreProposalStepForModules(
-					"@agoric/builders/scripts/vats/upgrade-asset-reserve.js",
-				),
-				vm.CoreProposalStepForModules(
-					"@agoric/builders/scripts/vats/upgrade-psm.js",
-				),
 			)
 		}
 

package/app/upgrade_test.go

@@ -0,0 +1,63 @@
+package gaia
+
+import (
+	"testing"
+
+	"github.com/Agoric/agoric-sdk/golang/cosmos/vm"
+	"github.com/stretchr/testify/assert"
+)
+
+func TestBuildProposalSteps(t *testing.T) {
+	testCases := []struct {
+		name         string
+		makeStep     func() (vm.CoreProposalStep, error)
+		expectedJson string
+	}{
+		{
+			"buildProposalStepFromScript",
+			func() (vm.CoreProposalStep, error) {
+				return buildProposalStepFromScript(
+					"UNRELEASED_A3P_INTEGRATION",
+					"@agoric/builders/scripts/vats/upgrade-orchestration.js",
+				)
+			},
+			`{"module":"@agoric/builders/scripts/vats/upgrade-orchestration.js",` +
+				`"entrypoint":"defaultProposalBuilder",` +
+				`"args":[{"variant":"A3P_INTEGRATION"}]}`,
+		},
+		{
+			"buildProposalStepWithArgs",
+			func() (vm.CoreProposalStep, error) {
+				return buildProposalStepWithArgs(
+					"@agoric/builders/scripts/vats/upgrade-vats.js",
+					"upgradeVatsProposalBuilder",
+					// Map iteration is randomised; use an anonymous struct instead.
+					struct {
+						Ibc string `json:"ibc"`
+					}{
+						Ibc: "@agoric/vats/src/vat-ibc.js",
+					},
+				)
+			},
+			`{"module":"@agoric/builders/scripts/vats/upgrade-vats.js",` +
+				`"entrypoint":"upgradeVatsProposalBuilder",` +
+				`"args":[{"ibc":"@agoric/vats/src/vat-ibc.js"}]}`,
+		},
+	}
+
+	for _, tc := range testCases {
+		t.Run(tc.name, func(t *testing.T) {
+			ps, err := tc.makeStep()
+
+			assert.NoError(t, err)
+			expected := vm.CoreProposalStepForModules(*vm.NewArbitraryCoreProposal(
+				tc.expectedJson,
+			))
+
+			assert.EqualValues(t, len(expected), len(ps), "Expected length %d to be %d", len(expected), len(ps))
+			for i, prop := range ps {
+				assert.EqualValues(t, expected[i], prop, "Expected %d to be %v, got %v", i, expected[i], prop)
+			}
+		})
+	}
+}

package/git-revision.txt

@@ -1 +1 @@
-f0ae74b84c
+8e4207fa19

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@agoric/cosmos",
-  "version": "0.35.0-u19.2",
+  "version": "0.35.0-u20.0",
   "description": "Connect JS to the Cosmos blockchain SDK",
   "parsers": {
     "js": "mjs"
@@ -38,5 +38,5 @@
   "typeCoverage": {
     "atLeast": 0
   },
-  "gitHead": "f0ae74b84cb6de3724bfdcd18b4bea7e8199dee1"
+  "gitHead": "8e4207fa19dabf76c1f91f8779b5b5b93570ecea"
 }

package/types/address_hooks.go

@@ -8,8 +8,6 @@ import (
 	"github.com/cosmos/cosmos-sdk/types/bech32"
 
 	transfertypes "github.com/cosmos/ibc-go/v6/modules/apps/transfer/types"
-	clienttypes "github.com/cosmos/ibc-go/v6/modules/core/02-client/types"
-	channeltypes "github.com/cosmos/ibc-go/v6/modules/core/04-channel/types"
 	ibcexported "github.com/cosmos/ibc-go/v6/modules/core/exported"
 )
 
@@ -176,7 +174,7 @@ func extractBaseTransferData(transferData transfertypes.FungibleTokenPacketData,
 // address is exactly the original address.
 // If newPacket is not nil, it is populated with a new transfer packet whose
 // corresponding Sender or Receiver is replaced with the extracted base address.
-func ExtractBaseAddressFromPacket(cdc codec.Codec, packet ibcexported.PacketI, role AddressRole, newPacket *channeltypes.Packet) (string, error) {
+func ExtractBaseAddressFromPacket(cdc codec.Codec, packet ibcexported.PacketI, role AddressRole, newPacket *IBCPacket) (string, error) {
 	var newDataP *[]byte
 	if newPacket != nil {
 		// Capture the data for the new packet.
@@ -192,13 +190,8 @@ func ExtractBaseAddressFromPacket(cdc codec.Codec, packet ibcexported.PacketI, r
 	}
 
 	// Create a new packet with the new transfer packet data.
-	*newPacket = channeltypes.NewPacket(
-		*newDataP, packet.GetSequence(),
-		packet.GetSourcePort(), packet.GetSourceChannel(),
-		packet.GetDestPort(), packet.GetDestChannel(),
-		clienttypes.MustParseHeight(packet.GetTimeoutHeight().String()),
-		packet.GetTimeoutTimestamp(),
-	)
+	*newPacket = CopyToIBCPacket(packet)
+	newPacket.Data = *newDataP
 
 	return target, nil
 }

package/types/address_hooks_test.go

@@ -12,7 +12,7 @@ import (
 	clienttypes "github.com/cosmos/ibc-go/v6/modules/core/02-client/types"
 	channeltypes "github.com/cosmos/ibc-go/v6/modules/core/04-channel/types"
 
-	"github.com/Agoric/agoric-sdk/golang/cosmos/types"
+	agtypes "github.com/Agoric/agoric-sdk/golang/cosmos/types"
 )
 
 func TestSplitHookedAddress(t *testing.T) {
@@ -49,7 +49,7 @@ func TestSplitHookedAddress(t *testing.T) {
 	for _, tc := range cases {
 		tc := tc
 		t.Run(tc.name, func(t *testing.T) {
-			baseAddr, hookData, err := types.SplitHookedAddress(tc.hook)
+			baseAddr, hookData, err := agtypes.SplitHookedAddress(tc.hook)
 			if len(tc.err) > 0 {
 				require.Error(t, err)
 				require.Equal(t, tc.err, err.Error())
@@ -90,9 +90,9 @@ func TestExtractBaseAddress(t *testing.T) {
 		for _, s := range suffixes {
 			s := s
 			t.Run(b.name+" "+s.hookStr, func(t *testing.T) {
-				addrHook, err := types.JoinHookedAddress(b.addr, []byte(s.hookStr))
+				addrHook, err := agtypes.JoinHookedAddress(b.addr, []byte(s.hookStr))
 				require.NoError(t, err)
-				addr, err := types.ExtractBaseAddress(addrHook)
+				addr, err := agtypes.ExtractBaseAddress(addrHook)
 				if s.isErr {
 					require.Error(t, err)
 				} else {
@@ -101,7 +101,7 @@ func TestExtractBaseAddress(t *testing.T) {
 						require.NotEqual(t, b.addr, addr)
 					} else {
 						require.Equal(t, b.addr, addr)
-						addr, hookData, err := types.SplitHookedAddress(addrHook)
+						addr, hookData, err := agtypes.SplitHookedAddress(addrHook)
 						require.NoError(t, err)
 						require.Equal(t, b.addr, addr)
 						require.Equal(t, s.hookStr, string(hookData))
@@ -121,48 +121,48 @@ func TestExtractBaseAddressFromPacket(t *testing.T) {
 
 	cosmosAddr := "cosmos1qqxuevtt"
 	cosmosHookStr := "?foo=bar&baz=bot#fragment"
-	cosmosHook, err := types.JoinHookedAddress(cosmosAddr, []byte(cosmosHookStr))
+	cosmosHook, err := agtypes.JoinHookedAddress(cosmosAddr, []byte(cosmosHookStr))
 	require.NoError(t, err)
-	addr, hookData, err := types.SplitHookedAddress(cosmosHook)
+	addr, hookData, err := agtypes.SplitHookedAddress(cosmosHook)
 	require.NoError(t, err)
 	require.Equal(t, cosmosAddr, addr)
 	require.Equal(t, cosmosHookStr, string(hookData))
 
 	agoricAddr := "agoric1qqp0e5ys"
 	agoricHookStr := "?bingo=again"
-	agoricHook, err := types.JoinHookedAddress(agoricAddr, []byte(agoricHookStr))
+	agoricHook, err := agtypes.JoinHookedAddress(agoricAddr, []byte(agoricHookStr))
 	require.NoError(t, err)
-	addr, hookData, err = types.SplitHookedAddress(agoricHook)
+	addr, hookData, err = agtypes.SplitHookedAddress(agoricHook)
 	require.NoError(t, err)
 	require.Equal(t, agoricAddr, addr)
 	require.Equal(t, agoricHookStr, string(hookData))
 
 	cases := []struct {
 		name  string
-		addrs map[types.AddressRole]struct{ addr, baseAddr string }
+		addrs map[agtypes.AddressRole]struct{ addr, baseAddr string }
 	}{
 		{"sender has params",
-			map[types.AddressRole]struct{ addr, baseAddr string }{
-				types.RoleSender:   {cosmosHook, "cosmos1qqxuevtt"},
-				types.RoleReceiver: {"agoric1qqp0e5ys", "agoric1qqp0e5ys"},
+			map[agtypes.AddressRole]struct{ addr, baseAddr string }{
+				agtypes.RoleSender:   {cosmosHook, "cosmos1qqxuevtt"},
+				agtypes.RoleReceiver: {"agoric1qqp0e5ys", "agoric1qqp0e5ys"},
 			},
 		},
 		{"receiver has params",
-			map[types.AddressRole]struct{ addr, baseAddr string }{
-				types.RoleSender:   {"cosmos1qqxuevtt", "cosmos1qqxuevtt"},
-				types.RoleReceiver: {agoricHook, "agoric1qqp0e5ys"},
+			map[agtypes.AddressRole]struct{ addr, baseAddr string }{
+				agtypes.RoleSender:   {"cosmos1qqxuevtt", "cosmos1qqxuevtt"},
+				agtypes.RoleReceiver: {agoricHook, "agoric1qqp0e5ys"},
 			},
 		},
 		{"both are base",
-			map[types.AddressRole]struct{ addr, baseAddr string }{
-				types.RoleSender:   {"cosmos1qqxuevtt", "cosmos1qqxuevtt"},
-				types.RoleReceiver: {"agoric1qqp0e5ys", "agoric1qqp0e5ys"},
+			map[agtypes.AddressRole]struct{ addr, baseAddr string }{
+				agtypes.RoleSender:   {"cosmos1qqxuevtt", "cosmos1qqxuevtt"},
+				agtypes.RoleReceiver: {"agoric1qqp0e5ys", "agoric1qqp0e5ys"},
 			},
 		},
 		{"both have params",
-			map[types.AddressRole]struct{ addr, baseAddr string }{
-				types.RoleSender:   {agoricHook, "agoric1qqp0e5ys"},
-				types.RoleReceiver: {cosmosHook, "cosmos1qqxuevtt"},
+			map[agtypes.AddressRole]struct{ addr, baseAddr string }{
+				agtypes.RoleSender:   {agoricHook, "agoric1qqp0e5ys"},
+				agtypes.RoleReceiver: {cosmosHook, "cosmos1qqxuevtt"},
 			},
 		},
 	}
@@ -170,29 +170,29 @@ func TestExtractBaseAddressFromPacket(t *testing.T) {
 	for _, tc := range cases {
 		tc := tc
 		t.Run(tc.name, func(t *testing.T) {
-			ftPacketData := transfertypes.NewFungibleTokenPacketData("denom", "100", tc.addrs[types.RoleSender].addr, tc.addrs[types.RoleReceiver].addr, "my-favourite-memo")
+			ftPacketData := transfertypes.NewFungibleTokenPacketData("denom", "100", tc.addrs[agtypes.RoleSender].addr, tc.addrs[agtypes.RoleReceiver].addr, "my-favourite-memo")
 			packetBz := ftPacketData.GetBytes()
-			packet := channeltypes.NewPacket(packetBz, 1234, "my-port", "my-channel", "their-port", "their-channel", clienttypes.NewHeight(133, 445), 10999)
+			packet := agtypes.MakeIBCPacket(packetBz, 1234, "my-port", "my-channel", "their-port", "their-channel", clienttypes.NewHeight(133, 445), 10999)
 
 			for role, addrs := range tc.addrs {
 				addrs := addrs
 				role := role
 
 				t.Run(string(role), func(t *testing.T) {
-					baseAddr, err := types.ExtractBaseAddress(addrs.addr)
+					baseAddr, err := agtypes.ExtractBaseAddress(addrs.addr)
 					require.NoError(t, err)
 					require.Equal(t, addrs.baseAddr, baseAddr)
 
-					packetBaseAddr0, err := types.ExtractBaseAddressFromData(cdc, packet.GetData(), role, nil)
+					packetBaseAddr0, err := agtypes.ExtractBaseAddressFromData(cdc, packet.GetData(), role, nil)
 					require.NoError(t, err)
 					require.Equal(t, addrs.baseAddr, packetBaseAddr0)
 
-					packetBaseAddr1, err := types.ExtractBaseAddressFromPacket(cdc, packet, role, nil)
+					packetBaseAddr1, err := agtypes.ExtractBaseAddressFromPacket(cdc, packet, role, nil)
 					require.NoError(t, err)
 					require.Equal(t, addrs.baseAddr, packetBaseAddr1)
 
-					var newPacket channeltypes.Packet
-					packetBaseAddr2, err := types.ExtractBaseAddressFromPacket(cdc, packet, role, &newPacket)
+					var newPacket agtypes.IBCPacket
+					packetBaseAddr2, err := agtypes.ExtractBaseAddressFromPacket(cdc, packet, role, &newPacket)
 					require.NoError(t, err)
 					require.Equal(t, addrs.baseAddr, packetBaseAddr2)
 
@@ -203,10 +203,10 @@ func TestExtractBaseAddressFromPacket(t *testing.T) {
 					// Check that the only difference between the packet data is the baseAddr.
 					packetData := basePacketData
 					switch role {
-					case types.RoleSender:
+					case agtypes.RoleSender:
 						require.Equal(t, addrs.baseAddr, basePacketData.Sender)
 						packetData.Sender = addrs.addr
-					case types.RoleReceiver:
+					case agtypes.RoleReceiver:
 						require.Equal(t, addrs.baseAddr, basePacketData.Receiver)
 						packetData.Receiver = addrs.addr
 					default:

package/types/codec.go

@@ -0,0 +1,23 @@
+package types
+
+import (
+	"github.com/cosmos/cosmos-sdk/codec"
+	cdctypes "github.com/cosmos/cosmos-sdk/codec/types"
+)
+
+var (
+	// packageCdc is the codec used for proto3 JSON serialization of proto.Message
+	// data structures that wouldn't otherwise survive round-tripping via a
+	// regular "encoding/json" Marshal->JSON.parse.
+	//
+	// The naïve json.Marshal output for an int64 (64-bit precision) is a JSON
+	// number, which is subject to rounding errors when parsed by JavaScript
+	// (whose numbers are IEEE 754 binary64 values with only 53-bit precision).
+	// The codec.ProtoCodec uses a custom JSON marshaller that converts each int64
+	// to and from a string with no loss of precision.
+	//
+	// The current package's IBCPacket was one such affected data structure, which
+	// now implements Marshaler and Unmarshaler interfaces for "encoding/json" to
+	// take advantage of the packageCdc.
+	packageCdc = codec.NewProtoCodec(cdctypes.NewInterfaceRegistry())
+)

package/types/ibc_packet.go

@@ -0,0 +1,64 @@
+package types
+
+import (
+	"encoding/json"
+
+	clienttypes "github.com/cosmos/ibc-go/v6/modules/core/02-client/types"
+	channeltypes "github.com/cosmos/ibc-go/v6/modules/core/04-channel/types"
+	"github.com/cosmos/ibc-go/v6/modules/core/exported"
+)
+
+var _ json.Marshaler = IBCPacket{}
+var _ json.Unmarshaler = (*IBCPacket)(nil)
+var _ exported.PacketI = IBCPacket{}
+
+type IBCPacket struct {
+	channeltypes.Packet
+}
+
+func (p IBCPacket) MarshalJSON() ([]byte, error) {
+	return packageCdc.MarshalJSON(&p.Packet)
+}
+
+func (p *IBCPacket) UnmarshalJSON(bz []byte) error {
+	return packageCdc.UnmarshalJSON(bz, &p.Packet)
+}
+
+func MakeIBCPacket(
+	data []byte,
+	sequence uint64,
+	sourcePort string,
+	sourceChannel string,
+	destPort string,
+	destChannel string,
+	timeoutHeight clienttypes.Height,
+	timeoutTimestamp uint64,
+) IBCPacket {
+	cp := channeltypes.NewPacket(
+		data, sequence,
+		sourcePort, sourceChannel,
+		destPort, destChannel,
+		timeoutHeight, timeoutTimestamp,
+	)
+	return IBCPacket{Packet: cp}
+}
+
+func CopyToChannelPacket(packet exported.PacketI) channeltypes.Packet {
+	timeoutHeight := clienttypes.MustParseHeight(packet.GetTimeoutHeight().String())
+	return channeltypes.NewPacket(
+		packet.GetData(), packet.GetSequence(),
+		packet.GetSourcePort(), packet.GetSourceChannel(),
+		packet.GetDestPort(), packet.GetDestChannel(),
+		timeoutHeight, packet.GetTimeoutTimestamp(),
+	)
+}
+
+func CopyToIBCPacket(packet exported.PacketI) IBCPacket {
+	timeoutHeight := clienttypes.MustParseHeight(packet.GetTimeoutHeight().String())
+	return MakeIBCPacket(
+		packet.GetData(), packet.GetSequence(),
+		packet.GetSourcePort(), packet.GetSourceChannel(),
+		packet.GetDestPort(), packet.GetDestChannel(),
+		timeoutHeight, packet.GetTimeoutTimestamp(),
+	)
+}