fix: make it possible to set decimalPlaces when calling startPSM (#6348)

* fix: make it possible to set decimalPlaces when calling startPSM * fix: makeRatio wants bigInt * chore: type cleanup and provisioning permit * chore: some tests don't have defaults for anchor decimalPlaces * chore: rename stable token in startPSM to `minted` * fix(agoric-cli): look up correct brand for PSM trade * chore: ignore missing type for pair Co-authored-by: Dan Connolly <connolly@agoric.com> Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
Agoric · Sep 29, 2022 · cb58be3 · cb58be3
1 parent 7728f43
commit cb58be3
Show file tree

Hide file tree

Showing 4 changed files with 26 additions and 10 deletions.
diff --git a/packages/agoric-cli/src/lib/psm.js b/packages/agoric-cli/src/lib/psm.js
@@ -96,6 +96,8 @@ export const makePSMSpendAction = (instance, brands, opts) => {
     brands,
     opts,
     opts.feePct ? opts.feePct / 100 : undefined,
+    // @ts-expect-error please update types. Not sure where pair goees.
+    opts.pair[1],
   );
 
   console.warn('psm spend give', proposal.give);

diff --git a/packages/inter-protocol/src/proposals/startPSM.js b/packages/inter-protocol/src/proposals/startPSM.js
@@ -44,7 +44,7 @@ export const startPSM = async (
       consume: { contractGovernor, psm: psmInstall },
     },
     brand: {
-      consume: { [Stable.symbol]: stableP },
+      consume: { [Stable.symbol]: mintedP },
     },
   },
   {
@@ -60,9 +60,11 @@ export const startPSM = async (
     'string',
     X`anchorOptions.denom must be a string, not ${denom}`,
   );
-  const [stable, [anchorBrand, anchorIssuer], feeMintAccess] =
+  /** @type {[Brand<'nat'>, [Brand<'nat'>, Issuer<'nat'>], FeeMintAccess]} */
+  // @ts-expect-error cast
+  const [minted, [anchorBrand, anchorIssuer], feeMintAccess] =
     await Promise.all([
-      stableP,
+      mintedP,
       reserveThenGetNamePaths(agoricNamesAdmin, [
         ['brand', keyword],
         ['issuer', keyword],
@@ -79,23 +81,34 @@ export const startPSM = async (
       E(E(zoe).getInvitationIssuer()).getAmountOf(poserInvitationP),
     ]);
 
-  const mintLimit = AmountMath.make(stable, MINT_LIMIT);
+  const [anchorInfo, mintedInfo] = await Promise.all(
+    [anchorBrand, minted].map(b => E(b).getDisplayInfo()),
+  );
+
+  const mintLimit = AmountMath.make(minted, MINT_LIMIT);
+  const anchorDecimalPlaces = anchorInfo.decimalPlaces || 1n;
+  const mintedDecimalPlaces = mintedInfo.decimalPlaces || 1n;
   const terms = await deeplyFulfilledObject(
     harden({
       anchorBrand,
-      anchorPerMinted: makeRatio(100n, anchorBrand, 100n, stable),
+      anchorPerMinted: makeRatio(
+        10n ** BigInt(anchorDecimalPlaces),
+        anchorBrand,
+        10n ** BigInt(mintedDecimalPlaces),
+        minted,
+      ),
       governedParams: {
         [CONTRACT_ELECTORATE]: {
           type: ParamTypes.INVITATION,
           value: electorateInvitationAmount,
         },
         WantMintedFee: {
           type: ParamTypes.RATIO,
-          value: makeRatio(WantMintedFeeBP, stable, BASIS_POINTS),
+          value: makeRatio(WantMintedFeeBP, minted, BASIS_POINTS),
         },
         GiveMintedFee: {
           type: ParamTypes.RATIO,
-          value: makeRatio(GiveMintedFeeBP, stable, BASIS_POINTS),
+          value: makeRatio(GiveMintedFeeBP, minted, BASIS_POINTS),
         },
         MintLimit: { type: ParamTypes.AMOUNT, value: mintLimit },
       },
@@ -167,7 +180,7 @@ export const startPSM = async (
       psm,
       governorFacets.creatorFacet,
       anchorBrand,
-      stable,
+      minted,
     ),
     // @ts-expect-error TODO type for provisionPoolStartResult
     E(E.get(provisionPoolStartResult).creatorFacet).initPSM(

diff --git a/packages/inter-protocol/test/psm/gov-add-psm-permit.json b/packages/inter-protocol/test/psm/gov-add-psm-permit.json
@@ -8,6 +8,7 @@
     "feeMintAccess": "zoe",
     "economicCommitteeCreatorFacet": "economicCommittee",
     "psmCharterCreatorFacet": "psmCharter",
+    "provisionPoolStartResult": true,
     "psmFacets": true,
     "chainTimerService": "timer"
   },

diff --git a/packages/inter-protocol/test/psm/gov-add-psm.js b/packages/inter-protocol/test/psm/gov-add-psm.js
@@ -20,8 +20,8 @@ const DAI = {
 
 const config = {
   options: { anchorOptions: DAI },
-  WantStableFeeBP: 1n,
-  GiveStableFeeBP: 3n,
+  WantMintedFeeBP: 1n,
+  GiveMintedFeeBP: 3n,
   MINT_LIMIT: 0n,
 };