@metamask-previews/perps-controller 1.2.0-preview-152e02d → 1.2.0-preview-e0f8f1907

package/CHANGELOG.md

@@ -7,6 +7,12 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+### Changed
+
+- Exclude `@myx-trade/sdk` from build output by default, reducing bundled size by ~57% ([#8234](https://github.com/MetaMask/core/pull/8234))
+  - MYX provider files are stripped from `dist/` unless `MM_PERPS_MYX_PROVIDER_ENABLED=true` is set at build time
+  - Static import of `MYXProvider` replaced with dynamic `import()` to break the eager dependency chain
+
 ## [1.2.0]
 
 ### Added

package/dist/PerpsController.cjs

@@ -24,7 +24,6 @@ const perpsConfig_1 = require("./constants/perpsConfig.cjs");
 const perpsErrorCodes_1 = require("./perpsErrorCodes.cjs");
 const AggregatedPerpsProvider_1 = require("./providers/AggregatedPerpsProvider.cjs");
 const HyperLiquidProvider_1 = require("./providers/HyperLiquidProvider.cjs");
-const MYXProvider_1 = require("./providers/MYXProvider.cjs");
 const AccountService_1 = require("./services/AccountService.cjs");
 const DataLakeService_1 = require("./services/DataLakeService.cjs");
 const DepositService_1 = require("./services/DepositService.cjs");
@@ -2836,17 +2835,27 @@ async function _PerpsController_performInitialization() {
                 messenger: this.messenger,
             });
             this.providers.set('hyperliquid', hyperLiquidProvider);
-            // Register MYX provider if enabled via feature flag
+            // Register MYX provider if enabled via feature flag.
+            // Uses dynamic import so @myx-trade/sdk is excluded from the bundle
+            // unless MM_PERPS_MYX_PROVIDER_ENABLED=true is set at build time.
+            // Wrapped in try/catch so a missing module (stripped at build time)
+            // only skips MYX registration instead of aborting initialization.
             const isMYXEnabled = __classPrivateFieldGet(this, _PerpsController_instances, "m", _PerpsController_isMYXProviderEnabled).call(this);
             if (isMYXEnabled) {
-                const myxProvider = new MYXProvider_1.MYXProvider({
-                    isTestnet: perpsConfig_1.PROVIDER_CONFIG.MYX_TESTNET_ONLY || this.state.isTestnet,
-                    platformDependencies: __classPrivateFieldGet(this, _PerpsController_options, "f").infrastructure,
-                });
-                this.providers.set('myx', myxProvider);
-                __classPrivateFieldGet(this, _PerpsController_instances, "m", _PerpsController_debugLog).call(this, 'PerpsController: MYX provider registered', {
-                    isTestnet: perpsConfig_1.PROVIDER_CONFIG.MYX_TESTNET_ONLY || this.state.isTestnet,
-                });
+                try {
+                    const { MYXProvider } = await import("./providers/MYXProvider.cjs");
+                    const myxProvider = new MYXProvider({
+                        isTestnet: perpsConfig_1.PROVIDER_CONFIG.MYX_TESTNET_ONLY || this.state.isTestnet,
+                        platformDependencies: __classPrivateFieldGet(this, _PerpsController_options, "f").infrastructure,
+                    });
+                    this.providers.set('myx', myxProvider);
+                    __classPrivateFieldGet(this, _PerpsController_instances, "m", _PerpsController_debugLog).call(this, 'PerpsController: MYX provider registered', {
+                        isTestnet: perpsConfig_1.PROVIDER_CONFIG.MYX_TESTNET_ONLY || this.state.isTestnet,
+                    });
+                }
+                catch {
+                    __classPrivateFieldGet(this, _PerpsController_instances, "m", _PerpsController_debugLog).call(this, 'PerpsController: MYX provider module not available (stripped from build), skipping registration');
+                }
             }
             // Set up active provider based on activeProvider value in state
             // 'aggregated' is treated as just another provider that wraps others