Publish

- @0x/asset-swapper@16.49.1
Updated CHANGELOGS & MD docs
2022-01-31 07:20:03 +00:00 · 2022-01-31 07:19:59 +00:00 · 2022-01-31 17:01:17 +10:00
4 changed files with 22 additions and 7 deletions
--- a/packages/asset-swapper/CHANGELOG.json
+++ b/packages/asset-swapper/CHANGELOG.json
@@ -1,4 +1,14 @@
 [
+    {
+        "version": "16.49.1",
+        "changes": [
+            {
+                "note": "Fix WorstCaseQuoteInfo encoding bug",
+                "pr": 402
+            }
+        ],
+        "timestamp": 1643613597
+    },
    {
        "version": "16.49.0",
        "changes": [
--- a/packages/asset-swapper/CHANGELOG.md
+++ b/packages/asset-swapper/CHANGELOG.md
@@ -5,6 +5,10 @@ Edit the package's CHANGELOG.json file only.

 CHANGELOG

+## v16.49.1 - _January 31, 2022_
+
+    * Fix WorstCaseQuoteInfo encoding bug (#402)
+
 ## v16.49.0 - _January 28, 2022_

    * Add more curve pools (#409)
--- a/packages/asset-swapper/package.json
+++ b/packages/asset-swapper/package.json
@@ -1,6 +1,6 @@
 {
    "name": "@0x/asset-swapper",
-    "version": "16.49.0",
+    "version": "16.49.1",
    "engines": {
        "node": ">=6.12"
    },
--- a/packages/asset-swapper/src/swap_quoter.ts
+++ b/packages/asset-swapper/src/swap_quoter.ts
@@ -599,24 +599,25 @@ function calculateTwoHopQuoteInfo(
            secondHopSource: _.pick(secondHopFill, 'source', 'fillData'),
        }),
    ).toNumber();
+    const isSell = operation === MarketOperation.Sell;
    return {
        bestCaseQuoteInfo: {
-            makerAmount: operation === MarketOperation.Sell ? secondHopFill.output : secondHopFill.input,
-            takerAmount: operation === MarketOperation.Sell ? firstHopFill.input : firstHopFill.output,
-            totalTakerAmount: operation === MarketOperation.Sell ? firstHopFill.input : firstHopFill.output,
+            makerAmount: isSell ? secondHopFill.output : secondHopFill.input,
+            takerAmount: isSell ? firstHopFill.input : firstHopFill.output,
+            totalTakerAmount: isSell ? firstHopFill.input : firstHopFill.output,
            feeTakerTokenAmount: constants.ZERO_AMOUNT,
            protocolFeeInWeiAmount: constants.ZERO_AMOUNT,
            gas,
        },
        // TODO jacob consolidate this with quote simulation worstCase
        worstCaseQuoteInfo: {
-            makerAmount: MarketOperation.Sell
+            makerAmount: isSell
                ? secondHopOrder.makerAmount.times(1 - slippage).integerValue()
                : secondHopOrder.makerAmount,
-            takerAmount: MarketOperation.Sell
+            takerAmount: isSell
                ? firstHopOrder.takerAmount
                : firstHopOrder.takerAmount.times(1 + slippage).integerValue(),
-            totalTakerAmount: MarketOperation.Sell
+            totalTakerAmount: isSell
                ? firstHopOrder.takerAmount
                : firstHopOrder.takerAmount.times(1 + slippage).integerValue(),
            feeTakerTokenAmount: constants.ZERO_AMOUNT,
Author	SHA1	Message	Date
Github Actions	f470d282ee	Publish - @0x/asset-swapper@16.49.1	2022-01-31 07:20:03 +00:00
Github Actions	b8a2526da5	Updated CHANGELOGS & MD docs	2022-01-31 07:19:59 +00:00
Jacob Evans	97575bbde9	fix: worstCaseQuoteInfo calculating buys incorrectly [TKR-296] (#402 ) * fix: WorstCaseQuoteInfo buy encoding * CHANGELOG	2022-01-31 17:01:17 +10:00