@exodus/activity-txs 5.0.2 → 6.0.0

package/CHANGELOG.md

@@ -3,6 +3,34 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+## [6.0.0](https://github.com/ExodusMovement/exodus-hydra/compare/@exodus/activity-txs@5.0.2...@exodus/activity-txs@6.0.0) (2026-01-26)
+
+### ⚠ BREAKING CHANGES
+
+- **orders:** use typescript (#14786)
+
+### Features
+
+- feat: update all consumers to use safe strings for error namespaces (#14429)
+
+### Bug Fixes
+
+- fix(activity-txs): fiat order duplicates wallet order (#14959)
+
+- fix: update NFT transaction sent status calculation (#14359)
+
+- refactor(orders)!: use typescript (#14786)
+
+## [5.1.0](https://github.com/ExodusMovement/exodus-hydra/compare/@exodus/activity-txs@5.0.2...@exodus/activity-txs@5.1.0) (2025-11-19)
+
+### Features
+
+- feat: update all consumers to use safe strings for error namespaces (#14429)
+
+### Bug Fixes
+
+- fix: update NFT transaction sent status calculation (#14359)
+
 ## [5.0.2](https://github.com/ExodusMovement/exodus-hydra/compare/@exodus/activity-txs@5.0.1...@exodus/activity-txs@5.0.2) (2025-11-05)
 
 ### Bug Fixes

package/README.md

@@ -4,7 +4,7 @@ This feature generates user-friendly activity from transactions, orders, fiat or
 
 ## How it works
 
-Each asset’s activity consists of a set of transactions transformed using `asset.api.getActivityTxs`, which returns an array of Tx [models](https://github.com/ExodusMovement/exodus-core/blob/cdff25bb962b2301613588c615c5ac09b05f260a/packages/models/src/tx/index.js#L10): `[Tx, Tx, Tx]`.
+Each asset’s activity consists of a set of transactions transformed using `asset.api.getActivityTxs`, which returns an array of Tx [models](https://github.com/ExodusOSS/hydra/blob/7743f4ffcbdb0605deb4943a90ef2e718776c3bf/libraries/models/src/tx/index.ts#L60): `[Tx, Tx, Tx]`.
 
 **Most assets don't have this API and simply return the original transactions.**
 

package/atoms/activity-txs.js

@@ -147,7 +147,10 @@ const createActivityTxsAtom = ({
 
                 return [assetName, activityTxs]
               } catch (e) {
-                errorTracking.track({ error: e, namespace: 'activityTxs', context: { assetName } })
+                errorTracking.track({
+                  error: e,
+                  context: { assetName },
+                })
                 logger.warn(`failed to get activity txs for asset: ${assetName}`, e)
                 return [assetName, []]
               }

package/index.js

@@ -1,8 +1,9 @@
 import { activityTxsAtomDefinition } from './atoms/index.js'
 import activityTxsPluginDefinition from './plugin/index.js'
+import { safeString } from '@exodus/safe-string'
 
 const activityTxs = () => ({
-  id: 'activityTxs',
+  id: safeString`activityTxs`,
   definitions: [
     {
       definition: activityTxsAtomDefinition,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@exodus/activity-txs",
-  "version": "5.0.2",
+  "version": "6.0.0",
   "description": "The activity-txs feature",
   "author": "Exodus Movement, Inc.",
   "repository": {
@@ -37,6 +37,7 @@
     "@exodus/core-selectors": "^4.0.0",
     "@exodus/models": "^12.0.1",
     "@exodus/multi-account-redux": "^2.0.1",
+    "@exodus/safe-string": "^1.3.0",
     "@exodus/typeforce": "^1.19.0",
     "lodash": "^4.17.21",
     "minimalistic-assert": "^1.0.1",
@@ -45,15 +46,15 @@
   "devDependencies": {
     "@exodus/assets": "^11.0.0",
     "@exodus/assets-base": "^10.0.0",
-    "@exodus/assets-feature": "^8.7.1",
-    "@exodus/available-assets": "^8.8.0",
+    "@exodus/assets-feature": "^8.8.0",
+    "@exodus/available-assets": "^8.10.1",
     "@exodus/bitcoin-plugin": "^1.4.2",
     "@exodus/connected-origins": "^4.3.0",
-    "@exodus/error-tracking": "^3.2.0",
-    "@exodus/fiat-ramp": "^15.5.0",
+    "@exodus/error-tracking": "^3.3.0",
+    "@exodus/fiat-ramp": "^15.10.0",
     "@exodus/logger": "^1.2.3",
-    "@exodus/nfts": "^9.6.1",
-    "@exodus/orders": "^5.5.2",
+    "@exodus/nfts": "^9.6.2",
+    "@exodus/orders": "^6.0.1",
     "@exodus/personal-notes": "^3.9.0",
     "@exodus/redux-dependency-injection": "^4.1.2",
     "@exodus/wallet-accounts": "^20.2.1",
@@ -64,5 +65,5 @@
     "access": "public",
     "provenance": false
   },
-  "gitHead": "e327423c6bddd97049003f3a2d31af9ba7aedc44"
+  "gitHead": "d818fc93bdc6d714f1fbbdcef90b5f1cc21680e2"
 }

package/redux/selectors/create-with-fiat-activity.js

@@ -2,7 +2,7 @@ import { memoize } from '@exodus/basic-utils'
 import { createSelector } from 'reselect'
 import assert from 'minimalistic-assert'
 import { TX_TYPES } from '../constants/tx-types.js'
-import { ORDER_STATUS } from '@exodus/models/lib/fiat-order/constants.js'
+import { ORDER_STATUS, ORDER_TYPES } from '@exodus/models/lib/fiat-order/constants.js'
 
 const selectorFactory =
   (visibleOrdersSelector, assetsSelector) =>
@@ -24,8 +24,29 @@ const selectorFactory =
             }
 
             let hasChanges = false
+            const pendingFiatOrdersWithoutTx = []
+            const fiatOrdersWithoutTx = []
+
+            if (displayFiatOrdersWithoutTx) {
+              for (const order of fiatOrders) {
+                if (order.txId) continue
+
+                const isRelatedBuyOrder =
+                  order.toWalletAccount === walletAccount && order.toAsset === assetName
+                const isRelatedSellOrder =
+                  order.fromWalletAccount === walletAccount && order.fromAsset === assetName
+
+                if (isRelatedBuyOrder || isRelatedSellOrder) {
+                  fiatOrdersWithoutTx.push(order)
+                  if (order.exodusStatus === ORDER_STATUS.in_progress) {
+                    pendingFiatOrdersWithoutTx.push(order)
+                  }
+                }
+              }
+            }
 
             const resultActivity = []
+
             for (const activityItem of activity) {
               const fiatOrdersByTx = fiatOrders.getAllByTxId(activityItem.txId)
 
@@ -41,34 +62,38 @@ const selectorFactory =
                   })
                 }
               } else {
-                resultActivity.push(activityItem)
-              }
-            }
+                if (activityItem.pending && pendingFiatOrdersWithoutTx.length > 0) {
+                  const matchingOrderIndex = pendingFiatOrdersWithoutTx.findIndex((order) => {
+                    const orderAmount =
+                      order.orderType === ORDER_TYPES.sell ? order.fromAmount : order.toAmount
 
-            if (displayFiatOrdersWithoutTx) {
-              for (const fiatOrder of fiatOrders) {
-                const isOrderWithoutTx = !fiatOrder.txId
-                const isRelatedBuyOrder =
-                  fiatOrder.toWalletAccount === walletAccount && fiatOrder.toAsset === assetName
-                const isRelatedSellOrder =
-                  fiatOrder.fromWalletAccount === walletAccount && fiatOrder.fromAsset === assetName
-
-                if (isOrderWithoutTx && (isRelatedBuyOrder || isRelatedSellOrder)) {
-                  hasChanges = true
-                  const activityItem = {
-                    id: `${assetName}.fiat-order-without-tx.${fiatOrder.orderId}`,
-                    assetName,
-                    walletAccount,
-                    fiatOrder,
-                    type: TX_TYPES.FIAT,
-                    pending: fiatOrder.exodusStatus === ORDER_STATUS.in_progress,
-                    date: fiatOrder.date || new Date(),
+                    return activityItem.tx?.coinAmount?.abs().equals(orderAmount)
+                  })
+
+                  if (matchingOrderIndex !== -1) {
+                    hasChanges = true
+                    pendingFiatOrdersWithoutTx.splice(matchingOrderIndex, 1)
+                    continue
                   }
-                  resultActivity.push(activityItem)
                 }
+
+                resultActivity.push(activityItem)
               }
             }
 
+            for (const fiatOrder of fiatOrdersWithoutTx) {
+              hasChanges = true
+              resultActivity.push({
+                id: `${assetName}.fiat-order-without-tx.${fiatOrder.orderId}`,
+                assetName,
+                walletAccount,
+                fiatOrder,
+                type: TX_TYPES.FIAT,
+                pending: fiatOrder.exodusStatus === ORDER_STATUS.in_progress,
+                date: fiatOrder.date || new Date(),
+              })
+            }
+
             if (!hasChanges) return activity
 
             return resultActivity

package/redux/utils/activity-formatters/format-nft.js

@@ -1,8 +1,28 @@
-import { getSharedProps } from './get-shared-props.js'
 import { TX_TYPES } from '../../constants/tx-types.js'
 import { isPending } from './format-tx-activity.js'
+import { getSharedProps } from './get-shared-props.js'
+
+const determineNftSent = (nft, tx) => {
+  if (tx) {
+    if (tx.received !== undefined) {
+      return !tx.received
+    }
+
+    if (tx.coinAmount) {
+      return !tx.coinAmount.isPositive
+    }
+  }
+
+  if (nft.sent !== undefined) {
+    return nft.sent
+  }
+
+  return false
+}
 
 export const formatUnindexedNftTx = ({ assetName, nftTx }) => {
+  const sent = nftTx.sent === undefined ? false : nftTx.sent
+
   return {
     ...getSharedProps({
       txId: nftTx.txId,
@@ -12,14 +32,16 @@ export const formatUnindexedNftTx = ({ assetName, nftTx }) => {
       id: `nft.${assetName}.${nftTx.txId}`,
       pending: false,
       failed: false,
-      sent: nftTx.sent || false,
-      received: !nftTx.sent,
+      sent,
+      received: !sent,
     }),
     nft: nftTx,
   }
 }
 
 export const formatNftTx = ({ tx, nft, assetName, asset, ...rest }) => {
+  const sent = determineNftSent(nft, tx)
+
   return {
     ...rest,
     ...getSharedProps({
@@ -30,8 +52,8 @@ export const formatNftTx = ({ tx, nft, assetName, asset, ...rest }) => {
       id: `nft.${assetName}.${tx.txId}`,
       pending: isPending(tx, asset),
       failed: !!tx.failed,
-      sent: nft.sent || false,
-      received: !nft.sent,
+      sent,
+      received: !sent,
     }),
     nft,
     tx,