more fencing adjustment

MetaMask · Sep 18, 2023 · 63cbe75 · 63cbe75
1 parent ebfa26a
commit 63cbe75
Showing 1 changed file with 9 additions and 4 deletions.
diff --git a/app/scripts/metamask-controller.js b/app/scripts/metamask-controller.js
@@ -7,7 +7,7 @@ import { createEngineStream } from 'json-rpc-middleware-stream';
 import { providerAsMiddleware } from '@metamask/eth-json-rpc-middleware';
 import {
   debounce,
-  ///: BEGIN:ONLY_INCLUDE_IN(build-main)
+  ///: BEGIN:ONLY_INCLUDE_IN(snaps)
   throttle,
   ///: END:ONLY_INCLUDE_IN
 } from 'lodash';
@@ -228,6 +228,7 @@ import { securityProviderCheck } from './lib/security-provider-helpers';
 import { IndexedDBPPOMStorage } from './lib/ppom/indexed-db-backend';
 ///: END:ONLY_INCLUDE_IN
 import { updateCurrentLocale } from './translate';
+import { HandlerType } from '@metamask/snaps-utils';
 
 export const METAMASK_CONTROLLER_EVENTS = {
   // Fired after state changes that impact the extension badge (unapproved msg count)
@@ -1879,7 +1880,7 @@ export default class MetamaskController extends EventEmitter {
   }
   ///: END:ONLY_INCLUDE_IN
 
-  ///: BEGIN:ONLY_INCLUDE_IN(build-main)
+  ///: BEGIN:ONLY_INCLUDE_IN(snaps)
   /**
    * Tracks snaps export usage. Note: This function is throttled to 1 call per 60 seconds.
    *
@@ -1912,8 +1913,12 @@ export default class MetamaskController extends EventEmitter {
    */
   handleSnapRequest(args) {
     // we're not tracking at this point in flask because we eagerly fetch insights in v2
-    ///: BEGIN:ONLY_INCLUDE_IN(build-main)
-    this._trackSnapExportUsage(args.snapId, args.handler);
+    ///: BEGIN:ONLY_INCLUDE_IN(build-flask)
+    if (args.handler !== HandlerType.OnTransaction) {
+      ///: END:ONLY_INCLUDE_IN
+      this._trackSnapExportUsage(args.snapId, args.handler);
+      ///: BEGIN:ONLY_INCLUDE_IN(build-flask)
+    }
     ///: END:ONLY_INCLUDE_IN
 
     return this.controllerMessenger.call('SnapController:handleRequest', args);