@fluidframework/merge-tree 2.1.0-276985 → 2.1.0

package/src/snapshotChunks.ts

@@ -89,7 +89,7 @@ export function serializeAsMinSupportedVersion(
 	options: PropertySet | undefined,
 	serializer: IFluidSerializer,
 	bind: IFluidHandle,
-) {
+): string {
 	let targetChuck: MergeTreeChunkLegacy;
 
 	if (chunk.version !== undefined) {
@@ -102,7 +102,7 @@ export function serializeAsMinSupportedVersion(
 	}
 
 	switch (chunk.version) {
-		case undefined:
+		case undefined: {
 			targetChuck = chunk as MergeTreeChunkLegacy;
 			targetChuck.headerMetadata = buildHeaderMetadataForLegacyChunk(
 				path,
@@ -110,8 +110,9 @@ export function serializeAsMinSupportedVersion(
 				options,
 			);
 			break;
+		}
 
-		case "1":
+		case "1": {
 			const chunkV1 = chunk as MergeTreeChunkV1;
 			const headerMetadata =
 				path === SnapshotLegacy.header ? chunkV1.headerMetadata : undefined;
@@ -128,9 +129,11 @@ export function serializeAsMinSupportedVersion(
 				headerMetadata,
 			};
 			break;
+		}
 
-		default:
+		default: {
 			throw new Error(`Unsupported chunk path: ${path} version: ${chunk.version}`);
+		}
 	}
 	return serializer.stringify(targetChuck, bind);
 }
@@ -142,7 +145,7 @@ export function serializeAsMaxSupportedVersion(
 	options: PropertySet | undefined,
 	serializer: IFluidSerializer,
 	bind: IFluidHandle,
-) {
+): string {
 	const targetChuck = toLatestVersion(path, chunk, logger, options);
 	return serializer.stringify(targetChuck, bind);
 }
@@ -166,11 +169,13 @@ export function toLatestVersion(
 				attribution: chunkLegacy.attribution,
 			};
 		}
-		case "1":
+		case "1": {
 			return chunk as MergeTreeChunkV1;
+		}
 
-		default:
+		default: {
 			throw new Error(`Unsupported chunk path: ${path} version: ${chunk.version}`);
+		}
 	}
 }
 

package/src/snapshotLoader.ts

@@ -58,8 +58,8 @@ export class SnapshotLoader {
 
 		const catchupOpsP = this.loadBodyAndCatchupOps(headerLoadedP, services);
 
-		catchupOpsP.catch((err) =>
-			this.logger.sendErrorEvent({ eventName: "CatchupOpsLoadFailure" }, err),
+		catchupOpsP.catch((error) =>
+			this.logger.sendErrorEvent({ eventName: "CatchupOpsLoadFailure" }, error),
 		);
 
 		await headerLoadedP;
@@ -80,22 +80,24 @@ export class SnapshotLoader {
 
 		const blobs = await blobsP;
 		if (blobs.length === headerChunk.headerMetadata!.orderedChunkMetadata.length + 1) {
-			headerChunk.headerMetadata!.orderedChunkMetadata.forEach((md) =>
-				blobs.splice(blobs.indexOf(md.id), 1),
-			);
+			for (const md of headerChunk.headerMetadata!.orderedChunkMetadata)
+				blobs.splice(blobs.indexOf(md.id), 1);
 			assert(blobs.length === 1, 0x060 /* There should be only one blob with catch up ops */);
 
 			// TODO: The 'Snapshot.catchupOps' tree entry is purely for backwards compatibility.
 			//       (See https://github.com/microsoft/FluidFramework/issues/84)
 
-			return this.loadCatchupOps(services.readBlob(blobs[0]), this.serializer);
+			// TODO Non null asserting, why is this not null?
+			return this.loadCatchupOps(services.readBlob(blobs[0]!), this.serializer);
 		} else if (blobs.length !== headerChunk.headerMetadata!.orderedChunkMetadata.length) {
 			throw new Error("Unexpected blobs in snapshot");
 		}
 		return [];
 	}
 
-	private readonly specToSegment = (spec: IJSONSegment | IJSONSegmentWithMergeInfo) => {
+	private readonly specToSegment = (
+		spec: IJSONSegment | IJSONSegmentWithMergeInfo,
+	): ISegment => {
 		let seg: ISegment;
 
 		if (hasMergeInfo(spec)) {
@@ -104,9 +106,9 @@ export class SnapshotLoader {
 			// `specToSegment()` initializes `seg` with the LocalClientId.  Overwrite this with
 			// the `spec` client (if specified).  Otherwise overwrite with `NonCollabClient`.
 			seg.clientId =
-				spec.client !== undefined
-					? this.client.getOrAddShortClientId(spec.client)
-					: NonCollabClient;
+				spec.client === undefined
+					? NonCollabClient
+					: this.client.getOrAddShortClientId(spec.client);
 
 			seg.seq = spec.seq ?? UniversalSequenceNumber;
 
@@ -159,7 +161,7 @@ export class SnapshotLoader {
 			this.mergeTree.options,
 			this.serializer,
 		);
-		const segs = chunk.segments.map(this.specToSegment);
+		const segs = chunk.segments.map((element) => this.specToSegment(element));
 		this.extractAttribution(segs, chunk);
 
 		this.mergeTree.reloadFromSegments(segs);
@@ -205,7 +207,7 @@ export class SnapshotLoader {
 			return;
 		}
 
-		let chunksWithAttribution = chunk1.attribution !== undefined ? 1 : 0;
+		let chunksWithAttribution = chunk1.attribution === undefined ? 0 : 1;
 		const segs: ISegment[] = [];
 		let lengthSofar = chunk1.length;
 		for (
@@ -215,16 +217,17 @@ export class SnapshotLoader {
 		) {
 			const chunk = await SnapshotV1.loadChunk(
 				services,
-				headerMetadata.orderedChunkMetadata[chunkIndex].id,
+				// TODO Non null asserting, why is this not null?
+				headerMetadata.orderedChunkMetadata[chunkIndex]!.id,
 				this.logger,
 				this.mergeTree.options,
 				this.serializer,
 			);
 			lengthSofar += chunk.length;
 			// Deserialize each chunk segment and append it to the end of the MergeTree.
-			const newSegs = chunk.segments.map(this.specToSegment);
+			const newSegs = chunk.segments.map((element) => this.specToSegment(element));
 			this.extractAttribution(newSegs, chunk);
-			chunksWithAttribution += chunk.attribution !== undefined ? 1 : 0;
+			chunksWithAttribution += chunk.attribution === undefined ? 0 : 1;
 			segs.push(...newSegs);
 		}
 
@@ -243,7 +246,7 @@ export class SnapshotLoader {
 
 		// Helper to insert segments at the end of the MergeTree.
 		const mergeTree = this.mergeTree;
-		const append = (segments: ISegment[], cli: number, seq: number) => {
+		const append = (segments: ISegment[], cli: number, seq: number): void => {
 			mergeTree.insertSegments(
 				mergeTree.root.cachedLength ?? 0,
 				segments,
@@ -256,7 +259,7 @@ export class SnapshotLoader {
 
 		// Helpers to batch-insert segments that are below the min seq
 		const batch: ISegment[] = [];
-		const flushBatch = () => {
+		const flushBatch = (): void => {
 			if (batch.length > 0) {
 				append(batch, NonCollabClient, UniversalSequenceNumber);
 			}

package/src/snapshotV1.ts

@@ -23,6 +23,7 @@ import { UnassignedSequenceNumber } from "./constants.js";
 import { MergeTree } from "./mergeTree.js";
 import { walkAllChildSegments } from "./mergeTreeNodeWalk.js";
 import { ISegment } from "./mergeTreeNodes.js";
+import type { IJSONSegment } from "./ops.js";
 import { PropertySet, matchProperties } from "./properties.js";
 import {
 	IJSONSegmentWithMergeInfo,
@@ -31,6 +32,7 @@ import {
 	MergeTreeHeaderMetadata,
 	serializeAsMaxSupportedVersion,
 	toLatestVersion,
+	type VersionedMergeTreeChunk,
 } from "./snapshotChunks.js";
 import { SnapshotLegacy } from "./snapshotlegacy.js";
 
@@ -90,14 +92,22 @@ export class SnapshotV1 {
 		let segmentCount = 0;
 		let hasAttribution = false;
 		while (length < approxSequenceLength && startIndex + segmentCount < allSegments.length) {
-			const pseg = allSegments[startIndex + segmentCount];
+			// TODO Non null asserting, why is this not null?
+			// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
+			const pseg = allSegments[startIndex + segmentCount]!;
 			segments.push(pseg);
-			length += allLengths[startIndex + segmentCount];
+			// TODO Non null asserting, why is this not null?
+			// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
+			length += allLengths[startIndex + segmentCount]!;
 			if (attributionCollections[startIndex + segmentCount]) {
 				hasAttribution = true;
 				collections.push({
-					attribution: attributionCollections[startIndex + segmentCount],
-					cachedLength: allLengths[startIndex + segmentCount],
+					// 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]!,
 				});
 			}
 			segmentCount++;
@@ -149,7 +159,7 @@ export class SnapshotV1 {
 		headerChunk.headerMetadata = this.header;
 		headerChunk.headerMetadata.orderedChunkMetadata = [{ id: SnapshotLegacy.header }];
 		const blobs: [key: string, content: string][] = [];
-		chunks.forEach((chunk, index) => {
+		for (const [index, chunk] of chunks.entries()) {
 			const id = `${SnapshotLegacy.body}_${index}`;
 			this.header.orderedChunkMetadata.push({ id });
 			blobs.push([
@@ -163,7 +173,7 @@ export class SnapshotV1 {
 					bind,
 				),
 			]);
-		});
+		}
 
 		const builder = new SummaryTreeBuilder();
 		builder.addBlob(
@@ -177,14 +187,14 @@ export class SnapshotV1 {
 				bind,
 			),
 		);
-		blobs.forEach((value) => {
+		for (const value of blobs) {
 			builder.addBlob(value[0], value[1]);
-		});
+		}
 
 		return builder.getSummaryTree();
 	}
 
-	extractSync() {
+	extractSync(): JsonSegmentSpecs[] {
 		const mergeTree = this.mergeTree;
 		const minSeq = this.header.minSequenceNumber;
 
@@ -196,7 +206,7 @@ export class SnapshotV1 {
 			json: JsonSegmentSpecs,
 			length: number,
 			attribution: IAttributionCollection<AttributionKey> | undefined,
-		) => {
+		): void => {
 			segmentsAfterCombine += 1;
 			this.segments.push(json);
 			this.segmentLengths.push(length);
@@ -206,18 +216,22 @@ export class SnapshotV1 {
 		};
 
 		// Helper to serialize the given `segment` and add it to the snapshot (if a segment is provided).
-		const pushSeg = (segment?: ISegment) => {
+		const pushSeg = (segment?: ISegment): void => {
 			if (segment) {
 				if (segment.properties !== undefined && Object.keys(segment.properties).length === 0) {
 					segment.properties = undefined;
 					segment.propertyManager = undefined;
 				}
-				pushSegRaw(segment.toJSONObject(), segment.cachedLength, segment.attribution);
+				pushSegRaw(
+					segment.toJSONObject() as JsonSegmentSpecs,
+					segment.cachedLength,
+					segment.attribution,
+				);
 			}
 		};
 
 		let prev: ISegment | undefined;
-		const extractSegment = (segment: ISegment) => {
+		const extractSegment = (segment: ISegment): boolean => {
 			// Elide segments that do not need to be included in the snapshot.  A segment may be elided if
 			// either condition is true:
 			//   a) The segment has not yet been ACKed.  We do not need to snapshot unACKed segments because
@@ -279,7 +293,7 @@ export class SnapshotV1 {
 					segment.propertyManager = undefined;
 				}
 				const raw: IJSONSegmentWithMergeInfo & { removedClient?: string } = {
-					json: segment.toJSONObject(),
+					json: segment.toJSONObject() as IJSONSegment,
 				};
 				// If the segment insertion is above the MSN, record the insertion merge info.
 				// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
@@ -298,9 +312,11 @@ export class SnapshotV1 {
 
 					// back compat for when we split overlap and removed client
 					raw.removedClient =
-						segment.removedClientIds !== undefined
-							? this.getLongClientId(segment.removedClientIds[0])
-							: undefined;
+						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]!);
 
 					raw.removedClientIds = segment.removedClientIds?.map((id) =>
 						this.getLongClientId(id),
@@ -373,7 +389,9 @@ export class SnapshotV1 {
 		options: PropertySet | undefined,
 		serializer?: IFluidSerializer,
 	): MergeTreeChunkV1 {
-		const chunkObj = serializer ? serializer.parse(chunk) : JSON.parse(chunk);
+		const chunkObj: VersionedMergeTreeChunk = serializer
+			? (serializer.parse(chunk) as VersionedMergeTreeChunk)
+			: (JSON.parse(chunk) as VersionedMergeTreeChunk);
 		return toLatestVersion(path, chunkObj, logger, options);
 	}
 }

package/src/snapshotlegacy.ts

@@ -83,7 +83,8 @@ export class SnapshotLegacy {
 			sequenceLength < approxSequenceLength &&
 			startIndex + segCount < allSegments.length
 		) {
-			const pseg = allSegments[startIndex + segCount];
+			// TODO Non null asserting, why is this not null?
+			const pseg = allSegments[startIndex + segCount]!;
 			segs.push(pseg);
 			if (pseg.attribution) {
 				segsWithAttribution++;
@@ -178,9 +179,10 @@ export class SnapshotLegacy {
 			// Messages used to have a "term" property which has since been removed.
 			// It is benign so it doesn't really need to be deleted here, but doing so permits snapshot tests
 			// to pass with an exact match (and matching the updated definition of ISequencedDocumentMessage).
-			catchUpMsgs.forEach((message) => {
+			for (const message of catchUpMsgs) {
+				// eslint-disable-next-line @typescript-eslint/no-explicit-any, @typescript-eslint/no-unsafe-member-access
 				delete (message as any).term;
-			});
+			}
 			builder.addBlob(
 				this.mergeTree.options?.catchUpBlobName ?? SnapshotLegacy.catchupOps,
 				serializer ? serializer.stringify(catchUpMsgs, bind) : JSON.stringify(catchUpMsgs),
@@ -190,7 +192,7 @@ export class SnapshotLegacy {
 		return builder.getSummaryTree();
 	}
 
-	extractSync() {
+	extractSync(): ISegment[] {
 		const collabWindow = this.mergeTree.collabWindow;
 		this.seq = collabWindow.minSeq;
 		this.header = {
@@ -212,7 +214,7 @@ export class SnapshotLegacy {
 			clientId: number,
 			start: number | undefined,
 			end: number | undefined,
-		) => {
+		): boolean => {
 			if (
 				segment.seq !== UnassignedSequenceNumber &&
 				segment.seq! <= this.seq! &&

package/src/sortedSegmentSet.ts

@@ -65,7 +65,9 @@ export class SortedSegmentSet<T extends SortedSegmentSetItem = ISegment> extends
 
 		while (start <= end) {
 			index = start + Math.floor((end - start) / 2);
-			const indexKey = this.getKey(this.keySortedItems[index]);
+			// 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]!);
 			if (indexKey > itemKey) {
 				if (start === index) {
 					return { exists: false, index };
@@ -85,7 +87,9 @@ export class SortedSegmentSet<T extends SortedSegmentSetItem = ISegment> extends
 				}
 				for (
 					let b = index - 1;
-					b >= 0 && this.getKey(this.keySortedItems[b]) === itemKey;
+					// 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--
 				) {
 					if (this.keySortedItems[b] === item) {
@@ -95,7 +99,9 @@ export class SortedSegmentSet<T extends SortedSegmentSetItem = ISegment> extends
 				for (
 					index + 1;
 					index < this.keySortedItems.length &&
-					this.getKey(this.keySortedItems[index]) === itemKey;
+					// 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;
 					index++
 				) {
 					if (this.keySortedItems[index] === item) {

package/src/sortedSet.ts

@@ -20,10 +20,12 @@ export abstract class SortedSet<T, U extends string | number> {
 		return this.keySortedItems;
 	}
 
-	public addOrUpdate(newItem: T, update?: (existingItem: T, newItem: T) => void) {
+	public addOrUpdate(newItem: T, update?: (existingItem: T, newItem: T) => void): void {
 		const position = this.findItemPosition(newItem);
 		if (position.exists) {
-			update?.(this.keySortedItems[position.index], newItem);
+			// 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);
 		} else {
 			this.keySortedItems.splice(position.index, 0, newItem);
 		}
@@ -54,7 +56,9 @@ export abstract class SortedSet<T, U extends string | number> {
 
 		while (start <= end) {
 			index = start + Math.floor((end - start) / 2);
-			const indexKey = this.getKey(this.keySortedItems[index]);
+			// 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]!);
 			if (indexKey > itemKey) {
 				if (start === index) {
 					return { exists: false, index };

package/src/textSegment.ts

@@ -40,7 +40,7 @@ export class TextSegment extends BaseSegment {
 		return segment.type === TextSegment.type;
 	}
 
-	public static make(text: string, props?: PropertySet) {
+	public static make(text: string, props?: PropertySet): TextSegment {
 		const seg = new TextSegment(text);
 		if (props) {
 			seg.addProperties(props);
@@ -48,7 +48,7 @@ export class TextSegment extends BaseSegment {
 		return seg;
 	}
 
-	public static fromJSONObject(spec: any) {
+	public static fromJSONObject(spec: string | IJSONSegment): TextSegment | undefined {
 		if (typeof spec === "string") {
 			return new TextSegment(spec);
 		} else if (spec && typeof spec === "object" && "text" in spec) {
@@ -69,8 +69,8 @@ export class TextSegment extends BaseSegment {
 		return this.properties ? { text: this.text, props: { ...this.properties } } : this.text;
 	}
 
-	public clone(start = 0, end?: number) {
-		const text = this.text.substring(start, end);
+	public clone(start = 0, end?: number): TextSegment {
+		const text = this.text.slice(start, end);
 		const b = TextSegment.make(text, this.properties);
 		this.cloneInto(b);
 		return b;
@@ -85,20 +85,20 @@ export class TextSegment extends BaseSegment {
 		);
 	}
 
-	public toString() {
+	public toString(): string {
 		return this.text;
 	}
 
-	public append(segment: ISegment) {
+	public append(segment: ISegment): void {
 		assert(TextSegment.is(segment), 0x447 /* can only append text segment */);
 		super.append(segment);
 		this.text += segment.text;
 	}
 
-	protected createSplitSegmentAt(pos: number) {
+	protected createSplitSegmentAt(pos: number): TextSegment | undefined {
 		if (pos > 0) {
-			const remainingText = this.text.substring(pos);
-			this.text = this.text.substring(0, pos);
+			const remainingText = this.text.slice(Math.max(0, pos));
+			this.text = this.text.slice(0, Math.max(0, pos));
 			this.cachedLength = this.text.length;
 			const leafSegment = new TextSegment(remainingText);
 			return leafSegment;

package/src/zamboni.ts

@@ -21,14 +21,14 @@ import {
 import { matchProperties } from "./properties.js";
 
 export const zamboniSegmentsMax = 2;
-function underflow(node: MergeBlock) {
+function underflow(node: MergeBlock): boolean {
 	return node.childCount < MaxNodesInBlock / 2;
 }
 
 export function zamboniSegments(
 	mergeTree: MergeTree,
 	zamboniSegmentsMaxCount = zamboniSegmentsMax,
-) {
+): void {
 	if (!mergeTree.collabWindow.collaborating) {
 		return;
 	}
@@ -57,7 +57,8 @@ export function zamboniSegments(
 				block.childCount = newChildCount;
 				block.children = childrenCopy;
 				for (let j = 0; j < newChildCount; j++) {
-					block.assignChild(childrenCopy[j], j, false);
+					// Non null asserting here since its looping though childrenCopy.length so j at childrenCopy will always exist
+					block.assignChild(childrenCopy[j]!, j, false);
 				}
 
 				if (underflow(block) && block.parent) {
@@ -72,7 +73,7 @@ export function zamboniSegments(
 }
 
 // Interior node with all node children
-export function packParent(parent: MergeBlock, mergeTree: MergeTree) {
+export function packParent(parent: MergeBlock, mergeTree: MergeTree): void {
 	const children = parent.children;
 	let childIndex: number;
 	let childBlock: MergeBlock;
@@ -96,7 +97,7 @@ export function packParent(parent: MergeBlock, mergeTree: MergeTree) {
 		}
 		const baseNodesInBlockCount = Math.floor(totalNodeCount / childCount);
 		let remainderCount = totalNodeCount % childCount;
-		const packedBlocks = new Array<MergeBlock>(MaxNodesInBlock);
+		const packedBlocks: IMergeNode[] = Array.from({ length: MaxNodesInBlock });
 		let childrenPackedCount = 0;
 		for (let nodeIndex = 0; nodeIndex < childCount; nodeIndex++) {
 			let nodeCount = baseNodesInBlockCount;
@@ -106,7 +107,8 @@ export function packParent(parent: MergeBlock, mergeTree: MergeTree) {
 			}
 			const packedBlock = mergeTree.makeBlock(nodeCount);
 			for (let packedNodeIndex = 0; packedNodeIndex < nodeCount; packedNodeIndex++) {
-				const nodeToPack = holdNodes[childrenPackedCount++];
+				// TODO Non null asserting, why is this not null?
+				const nodeToPack = holdNodes[childrenPackedCount++]!;
 				packedBlock.assignChild(nodeToPack, packedNodeIndex, false);
 			}
 			packedBlock.parent = parent;
@@ -115,7 +117,8 @@ export function packParent(parent: MergeBlock, mergeTree: MergeTree) {
 		}
 		parent.children = packedBlocks;
 		for (let j = 0; j < childCount; j++) {
-			parent.assignChild(packedBlocks[j], j, false);
+			// TODO Non null asserting, why is this not null?
+			parent.assignChild(packedBlocks[j]!, j, false);
 		}
 		parent.childCount = childCount;
 	} else {
@@ -130,12 +133,13 @@ export function packParent(parent: MergeBlock, mergeTree: MergeTree) {
 	}
 }
 
-function scourNode(node: MergeBlock, holdNodes: IMergeNode[], mergeTree: MergeTree) {
+function scourNode(node: MergeBlock, holdNodes: IMergeNode[], mergeTree: MergeTree): void {
 	// The previous segment is tracked while scouring for the purposes of merging adjacent segments
 	// when possible.
 	let prevSegment: ISegment | undefined;
 	for (let k = 0; k < node.childCount; k++) {
-		const childNode = node.children[k];
+		// TODO Non null asserting, why is this not null?
+		const childNode = node.children[k]!;
 		if (!childNode.isLeaf() || !childNode.segmentGroups.empty) {
 			holdNodes.push(childNode);
 			prevSegment = undefined;
@@ -191,7 +195,7 @@ function scourNode(node: MergeBlock, holdNodes: IMergeNode[], mergeTree: MergeTr
 					);
 
 					segment.parent = undefined;
-					segment.trackingCollection.trackingGroups.forEach((tg) => tg.unlink(segment));
+					for (const tg of segment.trackingCollection.trackingGroups) tg.unlink(segment);
 				} else {
 					holdNodes.push(segment);
 					prevSegment = segmentHasPositiveLength ? segment : undefined;

package/tsconfig.json

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