chore: simplify change array building

nrkno · Oct 5, 2023 · 7682c4e · 7682c4e
1 parent b485f9e
commit 7682c4e
Showing 1 changed file with 13 additions and 22 deletions.
diff --git a/src/Ember/Client/index.ts b/src/Ember/Client/index.ts
@@ -502,7 +502,7 @@ export class EmberClient extends EventEmitter<EmberClientEvents> {
 	}
 
 	private _applyRootToTree(node: Root): Array<Change> {
-		let changes: Array<Change> = []
+		const changes: Array<Change> = []
 
 		if ('id' in node) {
 			// node is an InvocationResult
@@ -538,13 +538,13 @@ export class EmberClient extends EventEmitter<EmberClientEvents> {
 					if (!tree) {
 						if (path.length) {
 							// Assuming this means that no get directory was done on the root of the tree.
-							changes = [...changes, { path: rootElement.path, node: rootElement }]
+							changes.push({ path: rootElement.path, node: rootElement })
 							continue
 						} else {
 							const number = Number(rootElement.path)
 							// Insert node into root
 							this.tree[number] = new NumberedTreeNodeImpl(number, rootElement.contents, rootElement.children)
-							changes = [...changes, { path: undefined, node: this.tree[number] }]
+							changes.push({ path: undefined, node: this.tree[number] })
 							continue
 						}
 					}
@@ -557,33 +557,24 @@ export class EmberClient extends EventEmitter<EmberClientEvents> {
 								number: Number(number),
 								parent: tree,
 							}
-							changes = [
-								...changes,
-								{
-									path: rootElement.path.split('.').slice(0, -1).join('.'),
-									node: tree,
-								},
-							]
+							changes.push({
+								path: rootElement.path.split('.').slice(0, -1).join('.'),
+								node: tree,
+							})
 							inserted = true
 							break
 						}
 						tree = tree.children[Number(number)]
 					}
 
 					if (inserted) continue
-					changes = [...changes, ...this._updateTree(rootElement, tree)]
+					changes.push(...this._updateTree(rootElement, tree))
 				} else {
 					if (this.tree[rootElement.number]) {
-						changes = [
-							...changes,
-							...this._updateTree(
-								rootElement as NumberedTreeNode<EmberElement>,
-								this.tree[(rootElement as NumberedTreeNode<EmberElement>).number]
-							),
-						]
+						changes.push(...this._updateTree(rootElement, this.tree[rootElement.number]))
 					} else {
 						this.tree[rootElement.number] = rootElement
-						changes = [...changes, { path: undefined, node: rootElement }]
+						changes.push({ path: undefined, node: rootElement })
 					}
 				}
 			}
@@ -593,7 +584,7 @@ export class EmberClient extends EventEmitter<EmberClientEvents> {
 	}
 
 	private _updateTree(update: TreeElement<EmberElement>, tree: NumberedTreeNode<EmberElement>): Array<Change> {
-		let changes: Array<Change> = []
+		const changes: Array<Change> = []
 
 		if (update.contents.type === tree.contents.type) {
 			changes.push({ path: getPath(tree), node: tree })
@@ -613,8 +604,8 @@ export class EmberClient extends EventEmitter<EmberClientEvents> {
 			// Update children
 			for (const child of Object.values<NumberedTreeNode<EmberElement>>(update.children)) {
 				const i = child.number
-				const oldChild = tree.children[i]
-				changes = [...changes, ...this._updateTree(child, oldChild)]
+				const oldChild = tree.children[i] // as NumberedTreeNode<EmberElement> | undefined // TODO
+				changes.push(...this._updateTree(child, oldChild))
 			}
 		} else if (update.children) {
 			changes.push({ path: getPath(tree), node: tree })