@fluidframework/merge-tree 2.2.0 → 2.3.0

package/src/partialLengths.ts

@@ -92,21 +92,15 @@ class PartialSequenceLengthsSet extends SortedSet<PartialSequenceLength, number>
 		const mindex = this.latestLeqIndex(minSeq);
 		let minLength = 0;
 		if (mindex >= 0) {
-			// TODO Non null asserting, why is this not null?
-			// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
-			minLength = this.keySortedItems[mindex]!.len;
+			minLength = this.keySortedItems[mindex].len;
 			const seqCount = this.size;
 			if (mindex <= seqCount - 1) {
 				// Still some entries remaining
 				const remainingCount = seqCount - mindex - 1;
 				// Copy down
 				for (let i = 0; i < remainingCount; i++) {
-					// TODO Non null asserting, why is this not null?
-					// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
-					this.keySortedItems[i] = this.keySortedItems[i + mindex + 1]!;
-					// TODO Non null asserting, why is this not null?
-					// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
-					this.keySortedItems[i]!.len -= minLength;
+					this.keySortedItems[i] = this.keySortedItems[i + mindex + 1];
+					this.keySortedItems[i].len -= minLength;
 				}
 				this.keySortedItems.length = remainingCount;
 			}
@@ -307,9 +301,7 @@ export class PartialSequenceLengths {
 		let hasInternalChild = false;
 		const childPartials: PartialSequenceLengths[] = [];
 		for (let i = 0; i < block.childCount; i++) {
-			// TODO Non null asserting, why is this not null?
-			// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
-			const child = block.children[i]!;
+			const child = block.children[i];
 			if (!child.isLeaf()) {
 				hasInternalChild = true;
 				if (recur) {
@@ -343,9 +335,7 @@ export class PartialSequenceLengths {
 			const childOverlapRemoves: LocalPartialSequenceLength[][] = [];
 			for (let i = 0; i < childPartialsLen; i++) {
 				const { segmentCount, minLength, partialLengths, unsequencedRecords } =
-					// TODO Non null asserting, why is this not null?
-					// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
-					childPartials[i]!;
+					childPartials[i];
 				combinedPartialLengths.segmentCount += segmentCount;
 				combinedPartialLengths.minLength += minLength;
 				childPartialLengths.push(partialLengths.items as PartialSequenceLength[]);
@@ -397,9 +387,7 @@ export class PartialSequenceLengths {
 		combinedPartialLengths.segmentCount = block.childCount;
 
 		for (let i = 0; i < block.childCount; i++) {
-			// TODO Non null asserting, why is this not null?
-			// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
-			const child = block.children[i]!;
+			const child = block.children[i];
 			if (child.isLeaf()) {
 				// Leaf segment
 				const segment = child;
@@ -687,17 +675,13 @@ export class PartialSequenceLengths {
 			segmentLen = -segmentLen;
 			// The client who performed the remove is always stored
 			// in the first position of removalInfo.
-			// TODO Non null asserting, why is this not null?
-			// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
-			clientId = removalInfo.removedClientIds[0]!;
+			clientId = removalInfo.removedClientIds[0];
 			const hasOverlap = removalInfo.removedClientIds.length > 1;
 			removeClientOverlap = hasOverlap ? removalInfo.removedClientIds : undefined;
 		} else if (moveInfo) {
 			// The client who performed the move is always stored
 			// in the first position of moveInfo.
-			// TODO Non null asserting, why is this not null?
-			// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
-			clientId = moveInfo.movedClientIds[0]!;
+			clientId = moveInfo.movedClientIds[0];
 
 			// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
 			seqOrLocalSeq = moveIsLocal ? moveInfo.localMovedSeq! : moveInfo.movedSeq;
@@ -733,9 +717,7 @@ export class PartialSequenceLengths {
 		if (moveInfo && removalInfo && removeHappenedFirst && !moveIsLocal) {
 			// The client who performed the remove is always stored
 			// in the first position of removalInfo.
-			// TODO Non null asserting, why is this not null?
-			// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
-			const moveClientId = moveInfo.movedClientIds[0]!;
+			const moveClientId = moveInfo.movedClientIds[0];
 			const hasOverlap = moveInfo.movedClientIds.length > 1;
 
 			PartialSequenceLengths.updatePartialsAfterInsertion(
@@ -758,9 +740,7 @@ export class PartialSequenceLengths {
 				: removalInfo.removedSeq;
 			// The client who performed the remove is always stored
 			// in the first position of removalInfo.
-			// TODO Non null asserting, why is this not null?
-			// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
-			const removeClientId = removalInfo.removedClientIds[0]!;
+			const removeClientId = removalInfo.removedClientIds[0];
 			const hasOverlap = removalInfo.removedClientIds.length > 1;
 
 			PartialSequenceLengths.updatePartialsAfterInsertion(
@@ -806,11 +786,7 @@ export class PartialSequenceLengths {
 				localIndexFirstGTE < unsequencedRecords.overlappingRemoves.length;
 				localIndexFirstGTE++
 			) {
-				if (
-					// TODO Non null asserting, why is this not null?
-					// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
-					unsequencedRecords.overlappingRemoves[localIndexFirstGTE]!.seq >= seqOrLocalSeq
-				) {
+				if (unsequencedRecords.overlappingRemoves[localIndexFirstGTE].seq >= seqOrLocalSeq) {
 					break;
 				}
 			}
@@ -938,9 +914,7 @@ export class PartialSequenceLengths {
 		let segCount = 0;
 		// Compute length for seq across children
 		for (let i = 0; i < node.childCount; i++) {
-			// TODO Non null asserting, why is this not null?
-			// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
-			const child = node.children[i]!;
+			const child = node.children[i];
 			if (child.isLeaf()) {
 				const segment = child;
 				const removalInfo = toRemovalInfo(segment);
@@ -1023,9 +997,7 @@ export class PartialSequenceLengths {
 		);
 		this.clientSeqNumbers[clientId] ??= new PartialSequenceLengthsSet();
 		PartialSequenceLengths.addSeq(
-			// TODO Non null asserting, why is this not null?
-			// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
-			this.clientSeqNumbers[clientId]!,
+			this.clientSeqNumbers[clientId],
 			seq,
 			seqSeglen + remoteObliteratedLen,
 			undefined,
@@ -1051,16 +1023,12 @@ export class PartialSequenceLengths {
 	public getPartialLength(refSeq: number, clientId: number, localSeq?: number): number {
 		let pLen = this.minLength;
 		const cliLatestIndex = this.cliLatest(clientId);
-		// TODO Non null asserting, why is this not null?
-		// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
-		const cliSeq = this.clientSeqNumbers[clientId]!;
+		const cliSeq = this.clientSeqNumbers[clientId];
 		pLen += this.partialLengths.latestLeq(refSeq)?.len ?? 0;
 
 		if (localSeq === undefined) {
 			if (cliLatestIndex >= 0) {
-				// TODO Non null asserting, why is this not null?
-				// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
-				const cliLatest = cliSeq.items[cliLatestIndex]!;
+				const cliLatest = cliSeq.items[cliLatestIndex];
 				if (cliLatest.seq > refSeq) {
 					// The client has local edits after refSeq, add in the length adjustments
 					pLen += cliLatest.len;
@@ -1146,15 +1114,13 @@ export class PartialSequenceLengths {
 			buf += `(${partial.seq},${partial.len}) `;
 		}
 
-		// TODO Non null asserting, why is this not null?
-		// eslint-disable-next-line @typescript-eslint/no-for-in-array, no-restricted-syntax, guard-for-in
+		// eslint-disable-next-line @typescript-eslint/no-for-in-array, no-restricted-syntax
 		for (const clientId in this.clientSeqNumbers) {
-			const seqNumber = this.clientSeqNumbers[clientId];
-			if (seqNumber !== undefined && seqNumber.size > 0) {
+			if (this.clientSeqNumbers[clientId].size > 0) {
 				buf += `Client `;
 				buf += glc ? `${glc(+clientId)}` : `${clientId}`;
 				buf += "[";
-				for (const partial of seqNumber.items) {
+				for (const partial of this.clientSeqNumbers[clientId].items) {
 					buf += `(${partial.seq},${partial.len})`;
 				}
 				buf += "]";
@@ -1180,9 +1146,7 @@ export class PartialSequenceLengths {
 
 	private addClientSeqNumber(clientId: number, seq: number, seglen: number): void {
 		this.clientSeqNumbers[clientId] ??= new PartialSequenceLengthsSet();
-		// TODO Non null asserting, why is this not null?
-		// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
-		const cli = this.clientSeqNumbers[clientId]!;
+		const cli = this.clientSeqNumbers[clientId];
 		cli.addOrUpdate({ seq, len: 0, seglen });
 	}
 
@@ -1491,16 +1455,9 @@ function mergeSortedListsBySeq<T extends PartialSequenceLength>(lists: T[][]): I
 			let currentMin: T | undefined;
 			let currentMinIndex: number | undefined;
 			for (let i = 0; i < len; i++) {
-				// TODO Non null asserting, why is this not null?
-				// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
-				const candidateIndex = this.nextSmallestIndex[i]!;
-				// TODO Non null asserting, why is this not null?
-				// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
-				const list = this.sublists[i]!;
-				if (candidateIndex < list.length) {
-					// TODO Non null asserting, why is this not null?
-					// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
-					const candidate = list[candidateIndex]!;
+				const candidateIndex = this.nextSmallestIndex[i];
+				if (candidateIndex < this.sublists[i].length) {
+					const candidate = this.sublists[i][candidateIndex];
 					if (!currentMin || candidate.seq < currentMin.seq) {
 						currentMin = candidate;
 						currentMinIndex = i;
@@ -1524,9 +1481,7 @@ function mergeSortedListsBySeq<T extends PartialSequenceLength>(lists: T[][]): I
 function insertIntoList<T>(list: T[], index: number, elem: T): void {
 	if (index < list.length) {
 		for (let k = list.length; k > index; k--) {
-			// TODO Non null asserting, why is this not null?
-			// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
-			list[k] = list[k - 1]!;
+			list[k] = list[k - 1];
 		}
 		list[index] = elem;
 	} else {

package/src/properties.ts

@@ -122,9 +122,7 @@ export function extendIfUndefined<T>(
 		// eslint-disable-next-line no-restricted-syntax
 		for (const key in extension) {
 			if (base[key] === undefined) {
-				// TODO Non null asserting, why is this not null?
-				// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
-				base[key] = extension[key]!;
+				base[key] = extension[key];
 			}
 		}
 	}

package/src/revertibles.ts

@@ -121,9 +121,7 @@ function appendLocalInsertToRevertibles(
 			trackingGroup: new UnorderedTrackingGroup(),
 		});
 	}
-	// TODO Non null asserting, why is this not null?
-	// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
-	const last = revertibles[revertibles.length - 1]!;
+	const last = revertibles[revertibles.length - 1];
 	for (const t of deltaArgs.deltaSegments) last.trackingGroup.link(t.segment);
 
 	return revertibles;
@@ -139,13 +137,9 @@ function appendLocalRemoveToRevertibles(
 			trackingGroup: new UnorderedTrackingGroup(),
 		});
 	}
-	// TODO Non null asserting, why is this not null?
-	// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
-	const last = revertibles[revertibles.length - 1]!;
+	const last = revertibles[revertibles.length - 1];
 
-	// TODO Non null asserting, why is this not null?
-	// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
-	const mergeTreeWithRevert = findMergeTreeWithRevert(deltaArgs.deltaSegments[0]!.segment);
+	const mergeTreeWithRevert = findMergeTreeWithRevert(deltaArgs.deltaSegments[0].segment);
 
 	for (const t of deltaArgs.deltaSegments) {
 		const props: RemoveSegmentRefProperties = {
@@ -259,9 +253,7 @@ function revertLocalInsert(
 	revertible: TypedRevertible<typeof MergeTreeDeltaType.INSERT>,
 ): void {
 	while (revertible.trackingGroup.size > 0) {
-		// TODO Non null asserting, why is this not null?
-		// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
-		const tracked = revertible.trackingGroup.tracked[0]!;
+		const tracked = revertible.trackingGroup.tracked[0];
 		assert(
 			tracked.trackingCollection.unlink(revertible.trackingGroup),
 			0x3f1 /* tracking group removed */,
@@ -280,9 +272,7 @@ function revertLocalRemove(
 	revertible: TypedRevertible<typeof MergeTreeDeltaType.REMOVE>,
 ): void {
 	while (revertible.trackingGroup.size > 0) {
-		// TODO Non null asserting, why is this not null?
-		// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
-		const tracked = revertible.trackingGroup.tracked[0]!;
+		const tracked = revertible.trackingGroup.tracked[0];
 
 		assert(
 			tracked.trackingCollection.unlink(revertible.trackingGroup),
@@ -386,9 +376,7 @@ function revertLocalAnnotate(
 	revertible: TypedRevertible<typeof MergeTreeDeltaType.ANNOTATE>,
 ): void {
 	while (revertible.trackingGroup.size > 0) {
-		// TODO Non null asserting, why is this not null?
-		// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
-		const tracked = revertible.trackingGroup.tracked[0]!;
+		const tracked = revertible.trackingGroup.tracked[0];
 		const unlinked = tracked.trackingCollection.unlink(revertible.trackingGroup);
 		assert(unlinked && tracked.isLeaf(), 0x3f7 /* annotates must track segments */);
 		if (toRemovalInfo(tracked) === undefined) {
@@ -423,9 +411,7 @@ export function revertMergeTreeDeltaRevertibles(
 		const r = revertibles.pop()!;
 		const operation = r.operation;
 		if (r.trackingGroup.size > 0) {
-			// TODO Non null asserting, why is this not null?
-			// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
-			mergeTreeWithRevert ??= findMergeTreeWithRevert(r.trackingGroup.tracked[0]!);
+			mergeTreeWithRevert ??= findMergeTreeWithRevert(r.trackingGroup.tracked[0]);
 			switch (operation) {
 				case MergeTreeDeltaType.INSERT: {
 					revertLocalInsert(driver, mergeTreeWithRevert, r);

package/src/segmentGroupCollection.ts

@@ -70,9 +70,7 @@ export class SegmentGroupCollection {
 			// duplicate the previousProps for this segment
 			const index = segmentGroup.segments.indexOf(sourceSegment);
 			if (index !== -1) {
-				// TODO Non null asserting, why is this not null?
-				// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
-				segmentGroup.previousProps.push(segmentGroup.previousProps[index]!);
+				segmentGroup.previousProps.push(segmentGroup.previousProps[index]);
 			}
 		}
 	}

package/src/segmentPropertiesManager.ts

@@ -50,7 +50,6 @@ export class PropertiesManager {
 		for (const [key, value] of Object.entries(props)) {
 			if (value !== undefined && this.pendingKeyUpdateCount?.[key] !== undefined) {
 				assert(
-					// TODO Non null asserting, why is this not null?
 					this.pendingKeyUpdateCount[key]! > 0,
 					0x05c /* "Trying to update more annotate props than do exist!" */,
 				);

package/src/snapshotLoader.ts

@@ -87,8 +87,7 @@ export class SnapshotLoader {
 			// TODO: The 'Snapshot.catchupOps' tree entry is purely for backwards compatibility.
 			//       (See https://github.com/microsoft/FluidFramework/issues/84)
 
-			// TODO Non null asserting, why is this not null?
-			return this.loadCatchupOps(services.readBlob(blobs[0]!), this.serializer);
+			return this.loadCatchupOps(services.readBlob(blobs[0]), this.serializer);
 		} else if (blobs.length !== headerChunk.headerMetadata!.orderedChunkMetadata.length) {
 			throw new Error("Unexpected blobs in snapshot");
 		}
@@ -217,8 +216,7 @@ export class SnapshotLoader {
 		) {
 			const chunk = await SnapshotV1.loadChunk(
 				services,
-				// TODO Non null asserting, why is this not null?
-				headerMetadata.orderedChunkMetadata[chunkIndex]!.id,
+				headerMetadata.orderedChunkMetadata[chunkIndex].id,
 				this.logger,
 				this.mergeTree.options,
 				this.serializer,

package/src/snapshotV1.ts

@@ -92,22 +92,14 @@ export class SnapshotV1 {
 		let segmentCount = 0;
 		let hasAttribution = false;
 		while (length < approxSequenceLength && startIndex + segmentCount < allSegments.length) {
-			// TODO Non null asserting, why is this not null?
-			// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
-			const pseg = allSegments[startIndex + segmentCount]!;
+			const pseg = allSegments[startIndex + segmentCount];
 			segments.push(pseg);
-			// TODO Non null asserting, why is this not null?
-			// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
-			length += allLengths[startIndex + segmentCount]!;
+			length += allLengths[startIndex + segmentCount];
 			if (attributionCollections[startIndex + segmentCount]) {
 				hasAttribution = true;
 				collections.push({
-					// TODO Non null asserting, why is this not null?
-					// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
-					attribution: attributionCollections[startIndex + segmentCount]!,
-					// TODO Non null asserting, why is this not null?
-					// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
-					cachedLength: allLengths[startIndex + segmentCount]!,
+					attribution: attributionCollections[startIndex + segmentCount],
+					cachedLength: allLengths[startIndex + segmentCount],
 				});
 			}
 			segmentCount++;
@@ -314,9 +306,7 @@ export class SnapshotV1 {
 					raw.removedClient =
 						segment.removedClientIds === undefined
 							? undefined
-							: // TODO Non null asserting, why is this not null?
-								// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
-								this.getLongClientId(segment.removedClientIds[0]!);
+							: this.getLongClientId(segment.removedClientIds[0]);
 
 					raw.removedClientIds = segment.removedClientIds?.map((id) =>
 						this.getLongClientId(id),

package/src/snapshotlegacy.ts

@@ -83,8 +83,7 @@ export class SnapshotLegacy {
 			sequenceLength < approxSequenceLength &&
 			startIndex + segCount < allSegments.length
 		) {
-			// TODO Non null asserting, why is this not null?
-			const pseg = allSegments[startIndex + segCount]!;
+			const pseg = allSegments[startIndex + segCount];
 			segs.push(pseg);
 			if (pseg.attribution) {
 				segsWithAttribution++;

package/src/sortedSegmentSet.ts

@@ -27,7 +27,6 @@ export type SortedSegmentSetItem =
  * the segments changes. This invariant allows us to ensure the segments stay
  * ordered and unique, and that new segments can be inserted into that order.
  *
- * @deprecated This functionality was not meant to be exported and will be removed in a future release
  * @internal
  */
 // eslint-disable-next-line import/no-deprecated
@@ -65,9 +64,7 @@ export class SortedSegmentSet<T extends SortedSegmentSetItem = ISegment> extends
 
 		while (start <= end) {
 			index = start + Math.floor((end - start) / 2);
-			// TODO Non null asserting, why is this not null?
-			// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
-			const indexKey = this.getKey(this.keySortedItems[index]!);
+			const indexKey = this.getKey(this.keySortedItems[index]);
 			if (indexKey > itemKey) {
 				if (start === index) {
 					return { exists: false, index };
@@ -87,9 +84,7 @@ export class SortedSegmentSet<T extends SortedSegmentSetItem = ISegment> extends
 				}
 				for (
 					let b = index - 1;
-					// TODO Non null asserting, why is this not null?
-					// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
-					b >= 0 && this.getKey(this.keySortedItems[b]!) === itemKey;
+					b >= 0 && this.getKey(this.keySortedItems[b]) === itemKey;
 					b--
 				) {
 					if (this.keySortedItems[b] === item) {
@@ -99,9 +94,7 @@ export class SortedSegmentSet<T extends SortedSegmentSetItem = ISegment> extends
 				for (
 					index + 1;
 					index < this.keySortedItems.length &&
-					// TODO Non null asserting, why is this not null?
-					// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
-					this.getKey(this.keySortedItems[index]!) === itemKey;
+					this.getKey(this.keySortedItems[index]) === itemKey;
 					index++
 				) {
 					if (this.keySortedItems[index] === item) {

package/src/sortedSet.ts

@@ -23,9 +23,7 @@ export abstract class SortedSet<T, U extends string | number> {
 	public addOrUpdate(newItem: T, update?: (existingItem: T, newItem: T) => void): void {
 		const position = this.findItemPosition(newItem);
 		if (position.exists) {
-			// Non null asserting here because we know the item exists
-			// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
-			update?.(this.keySortedItems[position.index]!, newItem);
+			update?.(this.keySortedItems[position.index], newItem);
 		} else {
 			this.keySortedItems.splice(position.index, 0, newItem);
 		}
@@ -56,9 +54,7 @@ export abstract class SortedSet<T, U extends string | number> {
 
 		while (start <= end) {
 			index = start + Math.floor((end - start) / 2);
-			// TODO Non null asserting, why is this not null?
-			// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
-			const indexKey = this.getKey(this.keySortedItems[index]!);
+			const indexKey = this.getKey(this.keySortedItems[index]);
 			if (indexKey > itemKey) {
 				if (start === index) {
 					return { exists: false, index };

package/src/zamboni.ts

@@ -57,8 +57,7 @@ export function zamboniSegments(
 				block.childCount = newChildCount;
 				block.children = childrenCopy;
 				for (let j = 0; j < newChildCount; j++) {
-					// Non null asserting here since its looping though childrenCopy.length so j at childrenCopy will always exist
-					block.assignChild(childrenCopy[j]!, j, false);
+					block.assignChild(childrenCopy[j], j, false);
 				}
 
 				if (underflow(block) && block.parent) {
@@ -107,8 +106,7 @@ export function packParent(parent: MergeBlock, mergeTree: MergeTree): void {
 			}
 			const packedBlock = mergeTree.makeBlock(nodeCount);
 			for (let packedNodeIndex = 0; packedNodeIndex < nodeCount; packedNodeIndex++) {
-				// TODO Non null asserting, why is this not null?
-				const nodeToPack = holdNodes[childrenPackedCount++]!;
+				const nodeToPack = holdNodes[childrenPackedCount++];
 				packedBlock.assignChild(nodeToPack, packedNodeIndex, false);
 			}
 			packedBlock.parent = parent;
@@ -117,8 +115,7 @@ export function packParent(parent: MergeBlock, mergeTree: MergeTree): void {
 		}
 		parent.children = packedBlocks;
 		for (let j = 0; j < childCount; j++) {
-			// TODO Non null asserting, why is this not null?
-			parent.assignChild(packedBlocks[j]!, j, false);
+			parent.assignChild(packedBlocks[j], j, false);
 		}
 		parent.childCount = childCount;
 	} else {
@@ -138,8 +135,7 @@ function scourNode(node: MergeBlock, holdNodes: IMergeNode[], mergeTree: MergeTr
 	// when possible.
 	let prevSegment: ISegment | undefined;
 	for (let k = 0; k < node.childCount; k++) {
-		// TODO Non null asserting, why is this not null?
-		const childNode = node.children[k]!;
+		const childNode = node.children[k];
 		if (!childNode.isLeaf() || !childNode.segmentGroups.empty) {
 			holdNodes.push(childNode);
 			prevSegment = undefined;

package/tsconfig.json

@@ -7,6 +7,7 @@
 		"outDir": "./lib",
 		"preserveConstEnums": true,
 		"noImplicitAny": true,
+		"noUncheckedIndexedAccess": false,
 		"exactOptionalPropertyTypes": false,
 	},
 }