small incremental changes

Author: zacharydfreeman

src/backend/tree.ts

@@ -142,7 +142,7 @@ class Tree {
     const uniqueName = this.checkForDuplicates(name);
     // instantiate new sibilng tree with state, uniqueName, componentName and rtid
     const newSibling: Tree = new Tree(state, uniqueName, componentData, rtid);
-    // updating newSibling parent to be the parent of "this" which refers to sibling node
+    // updating newSibling's parent to be the parent of "this" which refers to sibling node
     newSibling.parent = this.parent;
     // adds newSibling to children array
     this.parent.children.push(newSibling);
@@ -160,27 +160,31 @@ class Tree {
      */
     // if we havent made a copy of the tree, increment copyInstances and clear cicularComponentTable set
     if (copyInstances === 0) {
+      // increment copyInstances
       copyInstances++;
+      // clear circularComponentTable
       circularComponentTable.clear();
     }
     // creates copy of present node
     let copy: Tree = new Tree(this.state, this.name, this.componentData, this.rtid);
-    // you want to get rid of the parentNode?? not sure why
+    // you want to get rid of the parentNode becuase right now copy and "this" have the same parent and you dont want that
     delete copy.parent;
     // add to circularComponentTable
     circularComponentTable.add(this);
-    //
+    // remove unserializable Trees
     copy = scrubUnserializableMembers(copy);
 
-    // creates copy of each child of the present node
+    // creates copy of each child of the present node and assigns it to children property of the new copy Tree
     copy.children = this.children.map((child: Tree): Tree | string => {
+      // if child isnt in circularComponent table, return recursive call of cleanTreeCopy() on child. We need to do this to fully build out the tree
       if (!circularComponentTable.has(child)) {
         return child.cleanTreeCopy();
       }
       return 'circular';
     });
-
+    // reset copyInstances back to zero becuase we are done making a copy of the tree
     copyInstances--;
+    // return the copy
     return copy;
   }
 }

src/backend/types/backendTypes.ts

@@ -4,27 +4,31 @@
 import Tree from '../tree';
 
 /**
- * The snapshot of the current tree
- * @param tree - {Tree} - The tree structure to send to front end
- * @param unfilteredTree - {null} - The current mode (i.e. jumping, time-traveling, or paused)
+ * @type Tree - The snapshot of the current tree
+ * @member tree - {Tree} - The tree structure to send to front end
+ * @member unfilteredTree - {null} - The current mode (i.e. jumping, time-traveling, or paused)
  */
 export interface Snapshot {
   tree: Tree;
   unfilteredTree: null;
 }
 
 /**
- * Where we
- * @param jumping - whether we are jumping steps by
- * @param paused - true/false for whether pausing to see the state
+ * @type Mode - object that describes where we are
+ * @member jumping - whether we are jumping steps by
+ * @member paused - true/false for whether pausing to see the state
  */
 export interface Mode {
   jumping: boolean;
   paused: boolean;
 }
 
 /**
- *
+ * This is what is shown in developer tools??
+ * @type SnapshotNode
+ * @member name -
+ * @member state -
+ * @member children -
  */
 export interface SnapshotNode {
   name: string;
@@ -34,7 +38,8 @@ export interface SnapshotNode {
   children: any[];
 }
 /**
- * @param data - an object with action & payload properties
+ * @type MsgData - obj with data object that will be sent to window?
+ * @member data - an object with action & payload properties
  */
 export interface MsgData {
   data: {
@@ -44,14 +49,14 @@ export interface MsgData {
 }
 
 /**
- *
- * @param index -
- * @param hooksIndex -
- * @param actualDuration -
- * @param actualStartTime -
- * @param selfBaseDuration -
- * @param treeBaseDuration -
- * @param props -
+ * @type ComponentData -
+ * @member index -
+ * @member hooksIndex -
+ * @member actualDuration -
+ * @member actualStartTime -
+ * @member selfBaseDuration -
+ * @member treeBaseDuration -
+ * @member props -
  */
 export interface ComponentData {
   index?: number;
@@ -64,8 +69,9 @@ export interface ComponentData {
 }
 
 /**
- * @param state -
- * @param component -
+ * @type HookStateItem
+ * @member state -
+ * @member component -
  */
 export interface HookStateItem {
   state: any;