const -> let

Author: kiacolbert

src/browser/chrome/scripts/parser.js

@@ -48,45 +48,30 @@ function commitAllHostEffectsReplacement() {
       }
     }
 
-    // The following switch statement is only concerned about placement,
-    // updates, and deletions. To avoid needing to add a case for every
-    // possible bitmap value, we remove the secondary effects from the
-    // effect tag and switch on that value.
     let primaryEffectTag = effectTag & (Placement | Update | Deletion);
     switch (primaryEffectTag) {
       case Placement:
       {
-        // editbyme
         timeTravelTracker.push({
             primaryEffectTag: 'PLACEMENT',
             effect: _.cloneDeep(nextEffect),
         });
 
         commitPlacement(nextEffect);
-        // Clear the "placement" from effect tag so that we know that this is inserted, before
-        // any life-cycles like componentDidMount gets called.
-        // TODO: findDOMNode doesn't rely on this any more but isMounted
-        // does and isMounted is deprecated anyway so we should be able
-        // to kill this.
+
         nextEffect.effectTag &= ~Placement;
         break;
       }
       case PlacementAndUpdate:
       {
-        // Placement
         commitPlacement(nextEffect);
-        // Clear the "placement" from effect tag so that we know that this is inserted, before
-        // any life-cycles like componentDidMount gets called.
         nextEffect.effectTag &= ~Placement;
-
-        // Update
         let _current = nextEffect.alternate;
         commitWork(_current, nextEffect);
         break;
       }
       case Update:
       {
-        // editbyme
         timeTravelTracker.push({
           primaryEffectTag: 'UPDATE',
           effect: _.cloneDeep(nextEffect),
@@ -99,7 +84,6 @@ function commitAllHostEffectsReplacement() {
       }
       case Deletion:
       {
-        // editbyme
         timeTravelTracker.push({
           primaryEffectTag: 'DELETION',
           effect: _.cloneDeep(nextEffect),
@@ -119,13 +103,14 @@ function commitAllHostEffectsReplacement() {
 // regex method signatures
 const uRsig = new RegExp(/\b(useReducer)\b\(reducer, initialArg, init\)/);
 const cAHEsig = new RegExp(/\b(function)\b\s\b(commitAllHostEffects)\b\(\)/, 'g');
+
 // get replacer method bodies
-const injectableUseReducer = esprima.parseScript(useReducerReplacement.toString());
-const injectableUseReducerString = escodegen.generate(injectableUseReducer.body[0].body);
+let injectableUseReducer = esprima.parseScript(useReducerReplacement.toString());
+let injectableUseReducerString = escodegen.generate(injectableUseReducer.body[0].body);
+
+let injectableCommitAllHostEffects = esprima.parseScript(commitAllHostEffectsReplacement.toString());
+let injectableCommitAllHostEffectsString = escodegen.generate(injectableCommitAllHostEffects.body[0].body);
 
-const injectableCommitAllHostEffects = esprima.parseScript(commitAllHostEffectsReplacement.toString());
-// const injectableCommitAllHostEffects = injectableCommitAllHostEffects.body[0].body;
-const injectableCommitAllHostEffectsString = escodegen.generate(injectableCommitAllHostEffects.body[0].body);
 // traverse ast to find method and replace body with our node's body
 function traverseTree(replacementNode, functionName, ast) {
   console.log('traverse called');
@@ -169,7 +154,7 @@ const parseAndGenerate = (codeString) => {
     try {
       ast = esprima.parseModule(codeString);
     } catch (error) {
-      // exprima throws parsing error webpack devtool setting generates code
+      // esprima throws parsing error webpack devtool setting generates code
       console.log('unable to use esprima parser');
       codeString = stringParser(codeString, injectableUseReducerString, uRsig);
       codeString = stringParser(codeString, injectableCommitAllHostEffectsString, cAHEsig);