fix: Add default props dependency in dependency map (appsmithorg#9418)

* Add cloneDeep
* Add comments

app/client/src/workers/DataTreeEvaluator.ts

@@ -92,9 +92,21 @@ export default class DataTreeEvaluator {
  this.widgetConfigMap = widgetConfigMap;
  }
 
+ /**
+ * This method takes unEvalTree as input and does following
+ * 1. parseJSActions and updates JS
+ * 2. Creates dependencyMap, sorted dependencyMap
+ * 3. Generates inverseDependencyTree
+ * 4. Finally, evaluates the unEvalTree and returns that with JSUpdates
+ *
+ * @param {DataTree} unEvalTree
+ * @return {*}
+ * @memberof DataTreeEvaluator
+ */
  createFirstTree(unEvalTree: DataTree) {
  const totalStart = performance.now();
- let localUnEvalTree = JSON.parse(JSON.stringify(unEvalTree));
+ // cloneDeep will make sure not to omit key which has value as undefined.
+ let localUnEvalTree = _.cloneDeep(unEvalTree);
  let jsUpdates: Record<string, JSUpdate> = {};
  //parse js collection to get functions
  //save current state of js collection action and variables to be added to uneval tree
@@ -129,7 +141,7 @@ export default class DataTreeEvaluator {
  this.evalTree = getValidatedTree(evaluatedTree);
  const validateEnd = performance.now();
 
- this.oldUnEvalTree = JSON.parse(JSON.stringify(localUnEvalTree));
+ this.oldUnEvalTree = _.cloneDeep(localUnEvalTree);
  const totalEnd = performance.now();
  const timeTakenForFirstTree = {
  total: (totalEnd - totalStart).toFixed(2),