Stop broadcasting events with no listener (#1198)

* Remove unused CLIENT_FOUND message

* Remove unused PANEL_CLOSED and PANEL_OPEN events

* Add changeset

.changeset/pretty-carrots-rest.md

@@ -0,0 +1,9 @@
+---
+"apollo-client-devtools": patch
+---
+
+Stop broadcasting messages that aren't listened to by the extension.
+
+- `client-found`
+- `panel-open`
+- `panel-closed`

src/extension/constants.ts

@@ -1,12 +1,9 @@
-export const CLIENT_FOUND = "client-found";
 export const REQUEST_TAB_ID = "request-tab-id";
 export const DEVTOOLS_INITIALIZED = "devtools-initialized";
 export const CREATE_DEVTOOLS_PANEL = "create-devtools-panel";
 export const ACTION_HOOK_FIRED = "action-hook-fired";
 export const REQUEST_DATA = "request-data";
 export const UPDATE = "update";
-export const PANEL_OPEN = "panel-open";
-export const PANEL_CLOSED = "panel-closed";
 export const EXPLORER_REQUEST = "explorer-request";
 export const EXPLORER_RESPONSE = "explorer-response";
 export const RELOADING_TAB = "reloading-tab";

src/extension/devtools/devtools.ts

@@ -5,8 +5,6 @@ import {
   CREATE_DEVTOOLS_PANEL,
   REQUEST_DATA,
   UPDATE,
-  PANEL_OPEN,
-  PANEL_CLOSED,
   EXPLORER_REQUEST,
   RELOADING_TAB,
   RELOAD_TAB_COMPLETE,
@@ -78,8 +76,6 @@ devtools.listen<string>(CREATE_DEVTOOLS_PANEL, async ({ payload }) => {
   let removeExplorerListener: () => void;
 
   panel.onShown.addListener((window) => {
-    sendMessageToClient(PANEL_OPEN);
-
     const {
       __DEVTOOLS_APPLICATION__: {
         initialize,
@@ -148,7 +144,6 @@ devtools.listen<string>(CREATE_DEVTOOLS_PANEL, async ({ payload }) => {
     removeReloadListener();
     removeExplorerListener();
     devtools.removeConnection("explorer");
-    sendMessageToClient(PANEL_CLOSED);
   });
 });
 

src/extension/tab/hook.ts

@@ -23,7 +23,6 @@ import {
 } from "./helpers";
 import { ExplorerResponse, QueryResult } from "../../types";
 import {
-  CLIENT_FOUND,
   DEVTOOLS_INITIALIZED,
   CREATE_DEVTOOLS_PANEL,
   ACTION_HOOK_FIRED,
@@ -280,7 +279,6 @@ function initializeHook() {
     });
 
     clearInterval(interval);
-    sendMessageToTab(CLIENT_FOUND);
     // incase initial update was missed because the client wasn't ready, send the create devtools event.
     // devtools checks to see if it's already created, so this won't create duplicate tabs
     sendHookDataToDevTools(CREATE_DEVTOOLS_PANEL);

src/extension/tab/tabRelay.ts

@@ -1,6 +1,5 @@
 import Relay from "../../Relay";
 import {
-  CLIENT_FOUND,
   REQUEST_TAB_ID,
   CREATE_DEVTOOLS_PANEL,
   ACTION_HOOK_FIRED,
@@ -41,7 +40,6 @@ export default new Promise(async ($export) => {
   });
 
   const devtools = `background:devtools-${id}`;
-  tab.forward(CLIENT_FOUND, devtools);
   tab.forward(CREATE_DEVTOOLS_PANEL, devtools);
   tab.forward(ACTION_HOOK_FIRED, devtools);
   tab.forward(UPDATE, devtools);