feat: integrate Hyperliquid builder fee wallet address overrides

app/scripts/controllers/perps/infrastructure.ts

@@ -179,6 +179,14 @@
     setMeasurement: (name: string, value: number, unit: string) => {
       globalThis.sentry?.setMeasurement?.(name, value, unit);
     },
+    addBreadcrumb: (_breadcrumb: {
+      category: string;
+      message: string;
+      level: 'fatal' | 'error' | 'warning' | 'log' | 'info' | 'debug';
+      data?: Record<string, unknown>;
+    }) => {
+      // TODO: Integrate with Sentry breadcrumbs when ready
+    },
   };
 }
 

app/scripts/messenger-client-init/perps-controller-init.test.ts

@@ -154,6 +154,8 @@ describe('PerpsControllerInit', () => {
   beforeEach(() => {
     jest.clearAllMocks();
     delete process.env.MM_PERPS_BLOCKED_REGIONS;
+    delete process.env.MM_PERPS_HL_BUILDER_ADDRESS_MAINNET;
+    delete process.env.MM_PERPS_HL_BUILDER_ADDRESS_TESTNET;
   });
 
   describe('controller instantiation', () => {
@@ -177,6 +179,9 @@ describe('PerpsControllerInit', () => {
           fallbackHip3Enabled: true,
           fallbackHip3AllowlistMarkets: [],
           fallbackBlockedRegions: [],
+          providerCredentials: {
+            hyperliquid: undefined,
+          },
         },
         deferEligibilityCheck: true,
       });
@@ -344,6 +349,55 @@ describe('PerpsControllerInit', () => {
     });
   });
 
+  describe('getHyperLiquidBuilderAddresses', () => {
+    it('returns undefined when no env vars are set', () => {
+      PerpsControllerInit(getInitRequestMock());
+
+      const constructorCall = PerpsControllerMock.mock.calls[0][0];
+      expect(
+        constructorCall.clientConfig?.providerCredentials?.hyperliquid,
+      ).toBeUndefined();
+    });
+
+    it('passes mainnet builder address from env var', () => {
+      process.env.MM_PERPS_HL_BUILDER_ADDRESS_MAINNET = '0xabc123';
+      PerpsControllerInit(getInitRequestMock());
+
+      const constructorCall = PerpsControllerMock.mock.calls[0][0];
+      expect(
+        constructorCall.clientConfig?.providerCredentials?.hyperliquid,
+      ).toEqual({
+        builderAddressMainnet: '0xabc123',
+      });
+    });
+
+    it('passes testnet builder address from env var', () => {
+      process.env.MM_PERPS_HL_BUILDER_ADDRESS_TESTNET = '0xdef456';
+      PerpsControllerInit(getInitRequestMock());
+
+      const constructorCall = PerpsControllerMock.mock.calls[0][0];
+      expect(
+        constructorCall.clientConfig?.providerCredentials?.hyperliquid,
+      ).toEqual({
+        builderAddressTestnet: '0xdef456',
+      });
+    });
+
+    it('passes both builder addresses when both env vars are set', () => {
+      process.env.MM_PERPS_HL_BUILDER_ADDRESS_MAINNET = '0xabc123';
+      process.env.MM_PERPS_HL_BUILDER_ADDRESS_TESTNET = '0xdef456';
+      PerpsControllerInit(getInitRequestMock());
+
+      const constructorCall = PerpsControllerMock.mock.calls[0][0];
+      expect(
+        constructorCall.clientConfig?.providerCredentials?.hyperliquid,
+      ).toEqual({
+        builderAddressMainnet: '0xabc123',
+        builderAddressTestnet: '0xdef456',
+      });
+    });
+  });
+
   describe('api method delegation', () => {
     const apiToController: [string, string][] = [
       ['perpsInit', 'init'],

app/scripts/messenger-client-init/perps-controller-init.ts

@@ -24,6 +24,26 @@ function getFallbackBlockedRegions(): string[] {
     .filter(Boolean);
 }
 
+/**
+ * Read HyperLiquid builder fee wallet addresses from env vars.
+ * Returns undefined when neither env var is set (package defaults apply).
+ */
+function getHyperLiquidBuilderAddresses():
+  | { builderAddressMainnet?: string; builderAddressTestnet?: string }
+  | undefined {
+  const mainnet =
+    process.env.MM_PERPS_HL_BUILDER_ADDRESS_MAINNET?.trim() || undefined;
+  const testnet =
+    process.env.MM_PERPS_HL_BUILDER_ADDRESS_TESTNET?.trim() || undefined;
+  if (!mainnet && !testnet) {
+    return undefined;
+  }
+  return {
+    ...(mainnet ? { builderAddressMainnet: mainnet } : {}),
+    ...(testnet ? { builderAddressTestnet: testnet } : {}),
+  };
+}
+
 export const PerpsControllerInit: ControllerInitFunction<
   PerpsController,
   PerpsControllerMessenger
@@ -50,6 +70,9 @@ export const PerpsControllerInit: ControllerInitFunction<
       fallbackHip3Enabled: true,
       fallbackHip3AllowlistMarkets: [],
       fallbackBlockedRegions,
+      providerCredentials: {
+        hyperliquid: getHyperLiquidBuilderAddresses(),
+      },
     },
     deferEligibilityCheck: !completedOnboarding || !useExternalServices,
   });

builds.yml

@@ -378,6 +378,9 @@ env:
 
   # Perps geoblock fallback (comma-separated region codes when remote flag unavailable)
   - MM_PERPS_BLOCKED_REGIONS: null
+  # Hyperliquid builder fee wallet address overrides (defaults to package-defined addresses when unset)
+  - MM_PERPS_HL_BUILDER_ADDRESS_MAINNET: null
+  - MM_PERPS_HL_BUILDER_ADDRESS_TESTNET: null
   # Build-time gate: omit PerpsController and perps RPC when false (mitigation for critical failures)
   - PERPS_ENABLED: false
   # Build-time gate: when false AssetsController is instantiated but state remains empty.

development/webpack/test/webpack.config.test.ts

@@ -403,119 +403,6 @@ ${Object.entries(env)
     );
   });
 
-  it('keeps the MYX provider ignore only while the warning still exists', async () => {
-    const config: Configuration = getWebpackConfig();
-    const myxWarningPattern = /MYXProvider\.mjs/u;
-    const ignoreWarnings = config.ignoreWarnings ?? [];
-    const getPerpsControllerWarnings = async (): Promise<string[]> => {
-      const tempDirectory = fs.mkdtempSync(
-        join(tmpdir(), 'webpack-perps-controller-warning-'),
-      );
-      const repositoryRoot = resolve(__dirname, '../../..');
-      const entryFilePath = join(tempDirectory, 'entry.js');
-
-      fs.writeFileSync(entryFilePath, "import '@metamask/perps-controller';\n");
-
-      try {
-        const compiler = webpack({
-          mode: 'development',
-          context: repositoryRoot,
-          entry: entryFilePath,
-          output: {
-            path: join(tempDirectory, 'dist'),
-            filename: 'bundle.js',
-            clean: true,
-          },
-          resolve: {
-            modules: [join(repositoryRoot, 'node_modules'), 'node_modules'],
-          },
-        });
-
-        return await new Promise((resolveWarnings, reject) => {
-          compiler.run((error, stats) => {
-            const closeCompiler = (callback: () => void) => {
-              compiler.close((closeError) => {
-                if (closeError) {
-                  reject(closeError);
-                  return;
-                }
-
-                callback();
-              });
-            };
-
-            if (error) {
-              closeCompiler(() => reject(error));
-              return;
-            }
-
-            if (!stats) {
-              closeCompiler(() =>
-                reject(new Error('Webpack finished without returning stats.')),
-              );
-              return;
-            }
-
-            const errors =
-              stats.toJson({ all: false, errors: true }).errors ?? [];
-            if (errors.length > 0) {
-              closeCompiler(() =>
-                reject(
-                  new Error(
-                    `Webpack build failed:\n${errors
-                      .map((item) =>
-                        typeof item === 'string' ? item : item.message,
-                      )
-                      .join('\n\n')}`,
-                  ),
-                ),
-              );
-              return;
-            }
-
-            const warnings =
-              stats.toJson({ all: false, warnings: true }).warnings ?? [];
-            closeCompiler(() =>
-              resolveWarnings(
-                warnings.map((item) =>
-                  typeof item === 'string' ? item : item.message,
-                ),
-              ),
-            );
-          });
-        });
-      } finally {
-        fs.rmSync(tempDirectory, { recursive: true, force: true });
-      }
-    };
-
-    assert.ok(
-      ignoreWarnings.some(
-        (warning) =>
-          warning instanceof RegExp && warning.test('MYXProvider.mjs'),
-      ),
-      'Expected webpack config to ignore MYXProvider.mjs while that warning still exists.',
-    );
-
-    // The published perps-controller package currently omits MYXProvider.mjs.
-    // This assertion should fail once that warning goes away, which is the cue
-    // to remove the ignoreWarnings entry from webpack.config.ts.
-    const warnings = await getPerpsControllerWarnings();
-    assert.ok(
-      warnings.some((warning) => myxWarningPattern.test(warning)),
-      [
-        'Good news: webpack no longer warns about missing MYXProvider.mjs.',
-        'That means the temporary workaround is probably no longer needed.',
-        'Cleanup steps:',
-        '1. Remove this test: "keeps the MYX provider ignore only while the warning still exists".',
-        '2. Remove /MYXProvider\\.mjs/u from ignoreWarnings in development/webpack/webpack.config.ts.',
-        '3. Re-run the webpack unit tests.',
-        'Warnings seen in this run:',
-        ...warnings,
-      ].join('\n'),
-    );
-  });
-
   it('should write the `dry-run` message then call exit(0)', () => {
     const exit = mock.method(process, 'exit', noop, { times: 1 });
     const error = mock.method(console, 'error', noop, { times: 1 });