@metamask/transaction-controller 68.0.0 → 68.1.0

package/CHANGELOG.md

@@ -7,6 +7,18 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [68.1.0]
+
+### Changed
+
+- Add RPC method, chain ID, and endpoint type context to transaction provider errors, including raw transaction submission failures ([#9144](https://github.com/MetaMask/core/pull/9144))
+
+## [68.0.1]
+
+### Fixed
+
+- Set `isExternalSign` to `true` when `isGasFeeSponsored` is confirmed by simulation, so gas-sponsored transactions from accounts that cannot locally sign (e.g. Money Account keyring) skip `KeyringController:signTransaction` ([#9148](https://github.com/MetaMask/core/pull/9148))
+
 ## [68.0.0]
 
 ### Changed
@@ -2500,7 +2512,9 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
     All changes listed after this point were applied to this package following the monorepo conversion.
 
-[Unreleased]: https://github.com/MetaMask/core/compare/@metamask/transaction-controller@68.0.0...HEAD
+[Unreleased]: https://github.com/MetaMask/core/compare/@metamask/transaction-controller@68.1.0...HEAD
+[68.1.0]: https://github.com/MetaMask/core/compare/@metamask/transaction-controller@68.0.1...@metamask/transaction-controller@68.1.0
+[68.0.1]: https://github.com/MetaMask/core/compare/@metamask/transaction-controller@68.0.0...@metamask/transaction-controller@68.0.1
 [68.0.0]: https://github.com/MetaMask/core/compare/@metamask/transaction-controller@67.1.0...@metamask/transaction-controller@68.0.0
 [67.1.0]: https://github.com/MetaMask/core/compare/@metamask/transaction-controller@67.0.0...@metamask/transaction-controller@67.1.0
 [67.0.0]: https://github.com/MetaMask/core/compare/@metamask/transaction-controller@66.0.1...@metamask/transaction-controller@67.0.0

package/dist/TransactionController.cjs

@@ -425,6 +425,7 @@ class TransactionController extends base_controller_1.BaseController {
             isGasFeeTokenIgnoredIfBalance,
             isGasFeeIncluded,
             isGasFeeSponsored,
+            ...(isGasFeeSponsored ? { isExternalSign: true } : {}),
             // To avoid the property to be set as undefined.
             ...(excludeNativeTokenForFee === undefined
                 ? {}
@@ -1724,27 +1725,20 @@ async function _TransactionController_approveTransaction(transactionId, traceCon
         clearNonceLock?.();
     }
 }, _TransactionController_publishTransaction = async function _TransactionController_publishTransaction(transactionMeta, { skipSubmitHistory } = {}) {
-    try {
-        const { networkClientId, rawTx } = transactionMeta;
-        if (!rawTx) {
-            throw new Error('Missing raw transaction');
-        }
-        const transactionHash = (await (0, provider_1.rpcRequest)({
-            messenger: this.messenger,
-            networkClientId,
-            method: 'eth_sendRawTransaction',
-            params: [rawTx],
-        }));
-        if (skipSubmitHistory !== true) {
-            __classPrivateFieldGet(this, _TransactionController_instances, "m", _TransactionController_updateSubmitHistory).call(this, transactionMeta, transactionHash);
-        }
-        return transactionHash;
+    const { networkClientId, rawTx } = transactionMeta;
+    if (!rawTx) {
+        throw new Error('Missing raw transaction');
     }
-    catch (error) {
-        const errorObject = error;
-        const errorMessage = errorObject?.data?.message ?? errorObject?.message ?? String(error);
-        throw new Error(`RPC submit: ${errorMessage}`);
+    const transactionHash = (await (0, provider_1.rpcRequest)({
+        messenger: this.messenger,
+        networkClientId,
+        method: 'eth_sendRawTransaction',
+        params: [rawTx],
+    }));
+    if (skipSubmitHistory !== true) {
+        __classPrivateFieldGet(this, _TransactionController_instances, "m", _TransactionController_updateSubmitHistory).call(this, transactionMeta, transactionHash);
     }
+    return transactionHash;
 }, _TransactionController_rejectTransaction = function _TransactionController_rejectTransaction(transactionId, actionId, error) {
     const transactionMeta = __classPrivateFieldGet(this, _TransactionController_instances, "m", _TransactionController_getTransaction).call(this, transactionId);
     if (!transactionMeta) {
@@ -2197,7 +2191,11 @@ async function _TransactionController_approveTransaction(transactionId, traceCon
         skipResimulateCheck: Boolean(blockTime),
     }, (txMeta) => {
         txMeta.gasFeeTokens = gasFeeTokens;
-        txMeta.isGasFeeSponsored = isGasFeeSponsored;
+        txMeta.isGasFeeSponsored =
+            txMeta.isGasFeeSponsored ?? isGasFeeSponsored;
+        if (txMeta.isGasFeeSponsored) {
+            txMeta.isExternalSign = true;
+        }
         txMeta.gasUsed = gasUsed;
         if (!__classPrivateFieldGet(this, _TransactionController_instances, "m", _TransactionController_isBalanceChangesSkipped).call(this, txMeta)) {
             txMeta.simulationData = simulationData;