Removed unused code, comments, and console logs

Author: Christopher Stamper

src/app/containers/ButtonsContainer.tsx

@@ -54,11 +54,9 @@ function ButtonsContainer(): JSX.Element {
   // const [{ tabs, currentTab, currentTabInApp }, dispatch] = useStoreContext();
   // const [state, dispatch] = useStoreContext();
   const dispatch = useDispatch();
-  const currentTab = useSelector((state: any) => state.main.currentTab);
-  const tabs = useSelector((state: any)=>state.main.tabs);
-  const currentTabInApp = useSelector((state: any)=> state.main.currentTabInApp);
-  const {connectionStatus, hasInitialized} = useSelector((state: any)=> state.main);
+  const {currentTab, tabs, currentTabInApp, connectionStatus} = useSelector((state: any)=> state.main);
   const { mode: { paused }} = tabs[currentTab];
+
   //adding a local state using useState for the reconnect button functionality
   const [reconnectDialogOpen, setReconnectDialogOpen] = useState(false);
   const [disconnectedDialogOpen, setDisconnectedDialogOpen] = useState(false);
@@ -71,7 +69,6 @@ function ButtonsContainer(): JSX.Element {
   const handleReconnectConfirm = () => {
     //reconnection logic here
     dispatch(startReconnect());
-    // reconnectDialogOpen ? setReconnectDialogOpen(false) : setDisconnectedDialogOpen(false);
     handleReconnectCancel();
   }
 

src/app/containers/MainContainer.tsx

@@ -14,8 +14,7 @@ import {
   noDev,
   setCurrentLocation,
   disconnected,
-  endReconnect,
-  pause,
+  endReconnect
 } from '../RTKslices';
 import { useDispatch, useSelector } from 'react-redux';
 //importing mainContainerslice 
@@ -59,18 +58,16 @@ function MainContainer(): JSX.Element {
 
   const handleDisconnect = (msg): void => {
     if (msg === 'portDisconnect') {
-      console.log('unexpected port disconnect: ', msg);
+      console.log('unexpected port disconnection');
       dispatch(disconnected());
     }
   }
 
   useEffect(() => {
-    console.log('LOL: ', port);
     if (port) return; // only open port once so if it exists, do not run useEffect again
-    console.log('Okie')
+
     // chrome.runtime allows our application to retrieve our service worker (our eventual bundles/background.bundle.js after running npm run build), details about the manifest, and allows us to listen and respond to events in our application lifecycle.
     const currentPort = chrome.runtime.connect(); // we connect to our service worker
-    console.log('currentPort', currentPort);
 
     // listen for a message containing snapshots from the /extension/build/background.js service worker
     currentPort.onMessage.addListener(
@@ -119,24 +116,22 @@ function MainContainer(): JSX.Element {
           }
           default:
         }
-        // return true; // we return true so that the connection stays open, otherwise the message channel will close
       },
     );
 
 
     if (chrome.runtime.onMessage.hasListener(handleDisconnect))
-      chrome.runtime.onMessage.removeListener(handleDisconnect);
+    chrome.runtime.onMessage.removeListener(handleDisconnect);
 
     // used to track when the above connection closes unexpectedly. Remember that it should persist throughout the application lifecycle
     chrome.runtime.onMessage.addListener(handleDisconnect);
 
-    // setTimeout(() => {
-    //   console.log('disconnecting')
-    //   currentPort.disconnect();
-    // }, 30000);
+    // assign port to state so it could be used by other components
+    if (currentPort)
+      dispatch(setPort(currentPort));
 
-    if (currentPort) dispatch(setPort(currentPort)); // assign port to state so it could be used by other components
-    if (!connectionStatus && reconnectRequested) dispatch(endReconnect());
+    if (!connectionStatus && reconnectRequested)
+      dispatch(endReconnect());
   });
 
   // Error Page launch IF(Content script not launched OR RDT not installed OR Target not React app)

src/extension/background.js

@@ -159,14 +159,10 @@ chrome.runtime.onConnect.addListener((port) => {
     Again, this port object is used for communication within your extension, not for communication with external ports or tabs in the Chrome browser. If you need to interact with specific tabs or external ports, you would use other APIs or methods, such as chrome.tabs or other Chrome Extension APIs.
   */
 
-  console.log('ATTEMPTING CONNECTION');
   portsArr.push(port); // push each Reactime communication channel object to the portsArr
-  console.log('portsArr: ', portsArr);
-
+ 
   // On Reactime launch: make sure RT's active tab is correct
   if (portsArr.length > 0) {
-    console.log('yo');
-    console.log('activeTab: ', activeTab)
     portsArr.forEach((bg) => {// go through each port object (each Reactime instance)
       bg.postMessage({  // send passed in action object as a message to the current port
         action: 'changeTab',
@@ -177,7 +173,6 @@ chrome.runtime.onConnect.addListener((port) => {
 
   // send tabs obj to the connected devtools as soon as connection to devtools is made
   if (Object.keys(tabsObj).length > 0) {
-    console.log('yuh');
     port.postMessage({
       action: 'initialConnectSnapshots',
       payload: tabsObj,
@@ -186,7 +181,7 @@ chrome.runtime.onConnect.addListener((port) => {
 
   // every time devtool is closed, remove the port from portsArr
   port.onDisconnect.addListener((e) => {
-    console.log('PORT DISCONNECTED BACKGROUND');
+    console.log('PORT DISCONNECTED');
     for (let i = 0; i < portsArr.length; i += 1) {
       if (portsArr[i] === e) {
         portsArr.splice(i, 1);
@@ -199,7 +194,6 @@ chrome.runtime.onConnect.addListener((port) => {
   // listen for message containing a snapshot from devtools and send it to contentScript -
   // (i.e. they're all related to the button actions on Reactime)
   port.onMessage.addListener((msg) => {
-    console.log('pls: ', msg);
     // msg is action denoting a time jump in devtools
     // ---------------------------------------------------------------
     // message incoming from devTools should look like this:

src/extension/contentScript.ts

@@ -19,11 +19,6 @@ window.addEventListener('message', (msg) => {
     // One-time request tells the background script that the tab has reloaded.
     chrome.runtime.sendMessage({ action: 'tabReload' });
     firstMessage = false;
-    // const keepAliveContentScript = setInterval(() => { // interval to keep connection to service worker connection alive
-      // chrome.runtime.sendMessage({
-      //   action: 'keepAlive' // messages sent to port to keep connection alive
-      // })
-    // }, 295000) // messages must happen within five minutes
   }
 
   // After tabs object has been created from firstMessage, backend (linkFiber.ts)
@@ -57,11 +52,9 @@ chrome.runtime.onMessage.addListener((request) => {
     }
     // After the jumpToSnap action has been sent back to background js,
     // it will send the same action to backend files (index.ts) for it execute the jump feature
-    if (action === 'portDisconnect') console.log('RECEIVED PORT DISCONNECT MESSAGE contentScript')
     // '*' == target window origin required for event to be dispatched, '*' = no preference
-    // window.postMessage(request, '*');
+    window.postMessage(request, '*');
   }
-  // return true; // attempt to fix port closing console error
 });