From 8591ff16e2c863150637e4e2718a311516b33cc1 Mon Sep 17 00:00:00 2001
From: Juliusz Wajgelt <juliusz.wajgelt@swmansion.com>
Date: Wed, 19 Feb 2025 08:16:57 +0100
Subject: [PATCH] code review changes

---
 packages/vscode-extension/lib/instrumentation.js       |  4 ++--
 packages/vscode-extension/lib/wrapper.js               | 10 +++++-----
 .../plugins/render-outlines/render-outlines-plugin.ts  |  4 ++--
 3 files changed, 9 insertions(+), 9 deletions(-)

diff --git a/packages/vscode-extension/lib/instrumentation.js b/packages/vscode-extension/lib/instrumentation.js
index 304af6203..d45d1ba61 100644
--- a/packages/vscode-extension/lib/instrumentation.js
+++ b/packages/vscode-extension/lib/instrumentation.js
@@ -186,7 +186,7 @@ function onCommitFiberRoot(_rendererID, root) {
   onRender(renderedFibers);
 }
 
-export const setInstrumentationOptions = (partialOptions) => {
+export const updateInstrumentationOptions = (partialOptions) => {
   const componentBlocklist = partialOptions.componentBlocklist
     ? new Set(partialOptions.componentBlocklist)
     : options.componentBlocklist;
@@ -194,7 +194,7 @@ export const setInstrumentationOptions = (partialOptions) => {
   if (!inited) {
     inited = true;
     instrument({
-      onCommitFiberRoot: (rendererID, root) => onCommitFiberRoot(rendererID, root),
+      onCommitFiberRoot,
     });
   }
 };
diff --git a/packages/vscode-extension/lib/wrapper.js b/packages/vscode-extension/lib/wrapper.js
index d5cf7ff51..1bbfb0e77 100644
--- a/packages/vscode-extension/lib/wrapper.js
+++ b/packages/vscode-extension/lib/wrapper.js
@@ -39,8 +39,8 @@ const InternalImports = {
   get reduxDevtoolsExtensionCompose() {
     return require("./plugins/redux-devtools").compose;
   },
-  get setInstrumentationOptions() {
-    return require("./instrumentation").setInstrumentationOptions;
+  get updateInstrumentationOptions() {
+    return require("./instrumentation").updateInstrumentationOptions;
   },
 };
 
@@ -416,7 +416,7 @@ export function AppWrapper({ children, initialProps, fabric }) {
     if (!devtoolsAgent) {
       return;
     }
-    InternalImports.setInstrumentationOptions({
+    InternalImports.updateInstrumentationOptions({
       reportRenders: (blueprintOutlines) => {
         devtoolsAgent._bridge.send("RNIDE_rendersReported", { blueprintOutlines });
       },
@@ -427,9 +427,9 @@ export function AppWrapper({ children, initialProps, fabric }) {
   // TODO: get initial options when the app starts
   useAgentListener(
     devtoolsAgent,
-    "RNIDE_setInstrumentationOptions",
+    "RNIDE_updateInstrumentationOptions",
     (payload) => {
-      InternalImports.setInstrumentationOptions(payload);
+      InternalImports.updateInstrumentationOptions(payload);
     },
     []
   );
diff --git a/packages/vscode-extension/src/plugins/render-outlines/render-outlines-plugin.ts b/packages/vscode-extension/src/plugins/render-outlines/render-outlines-plugin.ts
index 41f70534e..799182818 100644
--- a/packages/vscode-extension/src/plugins/render-outlines/render-outlines-plugin.ts
+++ b/packages/vscode-extension/src/plugins/render-outlines/render-outlines-plugin.ts
@@ -19,7 +19,7 @@ export class RenderOutlinesPlugin implements ToolPlugin, RenderOutlinesInterface
       this.eventEmitter.emit("rendersReported", payload);
     }
     if (event === "RNIDE_queryInstrumentationOptions") {
-      this.devtools.send("RNIDE_setInstrumentationOptions", { isEnabled: this.isEnabled });
+      this.devtools.send("RNIDE_updateInstrumentationOptions", { isEnabled: this.isEnabled });
     }
   };
 
@@ -45,7 +45,7 @@ export class RenderOutlinesPlugin implements ToolPlugin, RenderOutlinesInterface
 
   setEnabled(isEnabled: boolean) {
     this.isEnabled = isEnabled;
-    this.devtools.send("RNIDE_setInstrumentationOptions", { isEnabled });
+    this.devtools.send("RNIDE_updateInstrumentationOptions", { isEnabled });
   }
 
   addEventListener<K extends keyof RenderOutlinesEventMap>(