Removed console logs and comments

Author: MarkTeets

src/backend/controllers/statePropExtractors.ts

@@ -95,9 +95,6 @@ export function getHooksNames(elementType: string): { hookName: string; varName:
   let AST: any;
   try {
     AST = JSXParser.parse(elementType).body;
-    console.log('AST');
-    console.log(AST);
-    let count = 0;
     // Begin search for hook names, only if ast has a body property.
     // Statements get all the names of the hooks. For example: useCount, useWildcard, ...
     const statements: { hookName: string; varName: string }[] = [];
@@ -106,67 +103,42 @@ export function getHooksNames(elementType: string): { hookName: string; varName:
      * Iterate through AST of every function(al component) declaration
      * Check within each function(al component) declaration if there are hook declarations & variable name declaration */
     AST.forEach((functionDec: any) => {
-      // console.log('functionDec');
-      // console.log(functionDec);
       let declarationBody: any;
       if (functionDec.expression?.body) declarationBody = functionDec.expression.body.body;
       // check if functionDec.expression.body exists, then set declarationBody to functionDec's body
       else declarationBody = functionDec.body?.body ?? [];
       // Traverse through the function's funcDecs and Expression Statements
-      console.log('declarationBody');
-      console.log( declarationBody);
       declarationBody.forEach((elem: any) => {
         // Hooks will always be contained in a variable declaration
-        console.log('elem type:', count++, elem.type);
-        console.log(elem);
-
         if (elem.type === 'VariableDeclaration') {
           // Obtain the declarations array from elem.
           const { declarations } = elem;
           // Obtain the reactHook:
           // Due to difference in babel transpilation in browser vs for jest test, expression is stored in differen location
-          console.log('Found variable declaration, elem declarations:', declarations)
-          // console.log('dec[0]', declarations[0]);
-          // console.log('init',declarations[0]?.init);
-          // console.log('callee',declarations[0]?.init?.callee);
-          // console.log('expressions', declarations[0]?.init?.callee?.expressions);
-          // console.log('Line 2!');
-          // console.log('args', declarations[0]?.init?.arguments);
-          // console.log('args[0]', declarations[0]?.init?.arguments?.arguments[0]);
-          // console.log('callee', declarations[0]?.init?.arguments?.arguments[0]?.callee);
-          // console.log('expression', declarations[0]?.init?.arguments?.arguments[0]?.callee?.expressions);
           const expression =
             declarations[0]?.init?.callee?.expressions || //work for browser
-            //Mark's notes: so this was where the app was breaking. ES6 functions (e.g. const handleClick = () => {}) inside functional components were hitting this line and crashing when it tried to access arguments[0] and arguments didn't exist.
             declarations[0]?.init?.arguments?.[0]?.callee?.expressions; //work for jest test; 
 
-          console.log('looked for expression, found:', expression);
-          //Mark's Note: for a functional definition that isn't a hook, it won't have the callee being searched for above. This line will cause this forEach execution to stop here in this case.
+          //For a functional definition that isn't a hook, it won't have the callee being searched for above. This line will cause this forEach execution to stop here in this case.
           if (expression === undefined) return; 
           let reactHook: string;
           reactHook = expression[1].property?.name;
           if (reactHook === 'useState') {
             // Obtain the variable being set:
-            //Mark's note: changed to point to second to last element of declarations because webpack adds an extra variable when converting files that use ES6, so the previous pointer wasn't working for this case
+            //This points to second to last element of declarations because webpack adds an extra variable when converting files that use ES6, so the previous pointer wasn't working for this case
             let varName: string =
               declarations[declarations.length - 2]?.id?.name || // work react application; 
               (Array.isArray(declarations[0]?.id?.elements)
                 ? declarations[0]?.id?.elements[0]?.name
                 : undefined); //work for nextJS application
             // Obtain the setState method:
-            //Mark's note: changed to point to last element of declarations because webpack adds an extra variable when converting files that use ES6, so the previous pointer wasn't working for this case
+            //This points to last element of declarations because webpack adds an extra variable when converting files that use ES6, so the previous pointer wasn't working for this case
             let hookName: string =
               declarations[declarations.length - 1]?.id?.name || // work react application; 
               (Array.isArray(declarations[0]?.id?.elements)
                 ? declarations[0]?.id?.elements[0]?.name
                 : undefined); //work for nextJS & Remix
             // Push reactHook & varName to statements array
-            /**
-             * Mark's notes, I'd like to alter the structure of the data
-             * to pass on the reactHook 'useState'. That way the user will 
-             * eventually be able to view the difference between variables
-             * stored via useState and useContext
-             */
             statements.push({ hookName, varName });
           }
         }