@fluidframework/odsp-driver 2.3.0-288113 → 2.3.1

package/src/odspDelayLoadedDeltaStream.ts

@@ -472,8 +472,7 @@ export class OdspDelayLoadedDeltaStream {
 	}
 
 	private emitMetaDataUpdateEvent(metadata: Record<string, string>): void {
-		// TODO Why are we non null asserting here?
-		const label = JSON.parse(metadata.sensitivityLabelsInfo!) as {
+		const label = JSON.parse(metadata.sensitivityLabelsInfo) as {
 			labels: unknown;
 			timestamp: number;
 		};
@@ -482,8 +481,7 @@ export class OdspDelayLoadedDeltaStream {
 		if (time > this.labelUpdateTimestamp) {
 			this.labelUpdateTimestamp = time;
 			this.metadataUpdateHandler({
-				// TODO Why are we non null asserting here?
-				sensitivityLabelsInfo: metadata.sensitivityLabelsInfo!,
+				sensitivityLabelsInfo: metadata.sensitivityLabelsInfo,
 			});
 		}
 	}

package/src/odspDeltaStorageService.ts

@@ -106,8 +106,7 @@ export class OdspDeltaStorageService {
 					clearTimeout(timer);
 					const deltaStorageResponse = response.content;
 					const messages =
-						// Non null asserting here because of the length check
-						deltaStorageResponse.value.length > 0 && "op" in deltaStorageResponse.value[0]!
+						deltaStorageResponse.value.length > 0 && "op" in deltaStorageResponse.value[0]
 							? (deltaStorageResponse.value as ISequencedDeltaOpMessage[]).map(
 									(operation) => operation.op,
 								)
@@ -188,8 +187,7 @@ export class OdspDeltaStorageWithCache implements IDocumentDeltaStorageService {
 					(op) => op.sequenceNumber >= from && op.sequenceNumber < to,
 				);
 				validateMessages("cached", messages, from, this.logger);
-				// Non null asserting here because of the length check
-				if (messages.length > 0 && messages[0]!.sequenceNumber === from) {
+				if (messages.length > 0 && messages[0].sequenceNumber === from) {
 					this.snapshotOps = this.snapshotOps.filter((op) => op.sequenceNumber >= to);
 					opsFromSnapshot += messages.length;
 					return { messages, partialResult: true };

package/src/odspDocumentDeltaConnection.ts

@@ -36,7 +36,6 @@ import { SocketIOClientStatic } from "./socketModule.js";
 const protocolVersions = ["^0.4.0", "^0.3.0", "^0.2.0", "^0.1.0"];
 const feature_get_ops = "api_get_ops";
 const feature_flush_ops = "api_flush_ops";
-const feature_submit_signals_v2 = "submit_signals_v2";
 
 export interface FlushResult {
 	lastPersistedSequenceNumber?: number;
@@ -296,9 +295,7 @@ export class OdspDocumentDeltaConnection extends DocumentDeltaConnection {
 			relayUserAgent: [client.details.environment, ` driverVersion:${pkgVersion}`].join(";"),
 		};
 
-		connectMessage.supportedFeatures = {
-			[feature_submit_signals_v2]: true,
-		};
+		connectMessage.supportedFeatures = {};
 
 		// Reference to this client supporting get_ops flow.
 		if (mc.config.getBoolean("Fluid.Driver.Odsp.GetOpsEnabled") !== false) {
@@ -591,10 +588,8 @@ export class OdspDocumentDeltaConnection extends DocumentDeltaConnection {
 				if (messages !== undefined && messages.length > 0) {
 					this.logger.sendPerformanceEvent({
 						...common,
-						// Non null asserting here because of the length check above
-						first: messages[0]!.sequenceNumber,
-						// Non null asserting here because of the length check above
-						last: messages[messages.length - 1]!.sequenceNumber,
+						first: messages[0].sequenceNumber,
+						last: messages[messages.length - 1].sequenceNumber,
 						length: messages.length,
 					});
 					this.emit("op", this.documentId, messages);

package/src/odspDocumentStorageServiceBase.ts

@@ -198,8 +198,7 @@ export abstract class OdspDocumentStorageServiceBase implements IDocumentStorage
 				// eslint-disable-next-line unicorn/no-null
 				return null;
 			}
-			// Non null asserting here because of the length check above
-			id = versions[0]!.id;
+			id = versions[0].id;
 		}
 
 		const snapshotTree = await this.readTree(id, scenarioName);
@@ -258,8 +257,7 @@ export abstract class OdspDocumentStorageServiceBase implements IDocumentStorage
 	protected combineProtocolAndAppSnapshotTree(snapshotTree: ISnapshotTree): ISnapshotTree {
 		// When we upload the container snapshot, we upload appTree in ".app" and protocol tree in ".protocol"
 		// So when we request the snapshot we get ".app" as tree and not as commit node as in the case just above.
-		// TODO Why are we non null asserting here?
-		const hierarchicalAppTree = snapshotTree.trees[".app"]!;
+		const hierarchicalAppTree = snapshotTree.trees[".app"];
 		const hierarchicalProtocolTree = snapshotTree.trees[".protocol"];
 		const summarySnapshotTree: ISnapshotTree = {
 			blobs: {

package/src/odspDriverUrlResolver.ts

@@ -262,8 +262,7 @@ export function decodeOdspUrl(url: string): {
 	}
 
 	return {
-		// TODO Why are we non null asserting here?
-		siteUrl: siteUrl!,
+		siteUrl,
 		driveId: decodeURIComponent(driveId),
 		itemId: decodeURIComponent(itemId),
 		path: decodeURIComponent(path),

package/src/odspSnapshotParser.ts

@@ -28,8 +28,7 @@ function buildHierarchy(flatTree: IOdspSnapshotCommit): ISnapshotTree {
 		const entryPathBase = entry.path.slice(lastIndex + 1);
 
 		// ODSP snapshots are created breadth-first so we can assume we see tree nodes prior to their contents
-		// TODO Why are we non null asserting here?
-		const node = lookup[entryPathDir]!;
+		const node = lookup[entryPathDir];
 
 		// Add in either the blob or tree
 		if (entry.type === "tree") {
@@ -70,19 +69,16 @@ export function convertOdspSnapshotToSnapshotTreeAndBlobs(
 		}
 	}
 
-	// TODO Why are we non null asserting here?
-	const sequenceNumber = odspSnapshot?.trees[0]!.sequenceNumber;
+	const sequenceNumber = odspSnapshot?.trees[0].sequenceNumber;
 
 	const val: ISnapshot = {
 		blobContents: blobsWithBufferContent,
 		ops: odspSnapshot.ops?.map((op) => op.op) ?? [],
 		sequenceNumber,
-		// TODO Why are we non null asserting here?
-		snapshotTree: buildHierarchy(odspSnapshot.trees[0]!),
+		snapshotTree: buildHierarchy(odspSnapshot.trees[0]),
 		latestSequenceNumber:
 			odspSnapshot.ops && odspSnapshot.ops.length > 0
-				? // Non null asserting here because of the length check above
-					odspSnapshot.ops[odspSnapshot.ops.length - 1]!.sequenceNumber
+				? odspSnapshot.ops[odspSnapshot.ops.length - 1].sequenceNumber
 				: sequenceNumber,
 		snapshotFormatV: 1,
 	};

package/src/odspSummaryUploadManager.ts

@@ -180,8 +180,7 @@ export class OdspSummaryUploadManager {
 		const keys = Object.keys(tree.tree);
 		for (const key of keys) {
 			assert(!key.includes("/"), 0x9cd /* id should not include slashes */);
-			// Non null asserting for now, this should be changed to Object.entries
-			const summaryObject = tree.tree[key]!;
+			const summaryObject = tree.tree[key];
 
 			let id: string | undefined;
 			let value: OdspSummaryTreeValue | undefined;

package/src/odspUrlHelper.ts

@@ -110,10 +110,8 @@ export async function getOdspUrlParts(url: URL): Promise<IOdspUrlParts | undefin
 			}
 		}
 
-		// TODO Why are we non null asserting here?
-		const driveId = joinSessionMatch[3] ?? joinSessionMatch[5]!;
-		// TODO Why are we non null asserting here?
-		const itemId = joinSessionMatch[4]!;
+		const driveId = joinSessionMatch[3] || joinSessionMatch[5];
+		const itemId = joinSessionMatch[4];
 
 		return { siteUrl: `${url.origin}${url.pathname}`, driveId, itemId };
 	} else {
@@ -122,10 +120,8 @@ export async function getOdspUrlParts(url: URL): Promise<IOdspUrlParts | undefin
 		if (joinSessionMatch === null) {
 			return undefined;
 		}
-		// TODO Why are we non null asserting here?
-		const driveId = joinSessionMatch[2]!;
-		// TODO Why are we non null asserting here?
-		const itemId = joinSessionMatch[3]!;
+		const driveId = joinSessionMatch[2];
+		const itemId = joinSessionMatch[3];
 
 		return { siteUrl: `${url.origin}${url.pathname}`, driveId, itemId };
 	}

package/src/packageVersion.ts

@@ -6,4 +6,4 @@
  */
 
 export const pkgName = "@fluidframework/odsp-driver";
-export const pkgVersion = "2.3.0-288113";
+export const pkgVersion = "2.3.1";

package/src/zipItDataRepresentationUtils.ts

@@ -301,46 +301,39 @@ export class NodeCore {
 	}
 
 	public get(index: number): NodeTypes {
-		// TODO Why are we non null asserting here?
-		return this.children[index]!;
+		return this.children[index];
 	}
 
 	public getString(index: number): string {
-		// TODO Why are we non null asserting here?
-		const node = this.children[index]!;
+		const node = this.children[index];
 		return getStringInstance(node, "getString should return string");
 	}
 
 	public getMaybeString(index: number): string | undefined {
-		// TODO Why are we non null asserting here?
-		const node = this.children[index]!;
+		const node = this.children[index];
 		return getMaybeStringInstance(node);
 	}
 
 	public getBlob(index: number): BlobCore {
-		// TODO Why are we non null asserting here?
-		const node = this.children[index]!;
+		const node = this.children[index];
 		assertBlobCoreInstance(node, "getBlob should return a blob");
 		return node;
 	}
 
 	public getNode(index: number): NodeCore {
-		// TODO Why are we non null asserting here?
-		const node = this.children[index]!;
+		const node = this.children[index];
 		assertNodeCoreInstance(node, "getNode should return a node");
 		return node;
 	}
 
 	public getNumber(index: number): number {
-		// TODO Why are we non null asserting here?
-		const node = this.children[index]!;
+		const node = this.children[index];
 		assertNumberInstance(node, "getNumber should return a number");
 		return node;
 	}
 
 	public getBool(index: number): boolean {
-		// TODO Why are we non null asserting here?
-		const node = this.children[index]!;
+		const node = this.children[index];
 		assertBoolInstance(node, "getBool should return a boolean");
 		return node;
 	}
@@ -556,8 +549,7 @@ export class NodeCore {
 
 		for (const el of stringsToResolve) {
 			for (let it = el.startPos; it < el.endPos; it++) {
-				// Non null asserting here because we are iterating over startPos
-				stringBuffer[length] = input[it]!;
+				stringBuffer[length] = input[it];
 				length++;
 			}
 			stringBuffer[length] = 0;
@@ -569,8 +561,7 @@ export class NodeCore {
 		if (result.length === stringsToResolve.length + 1) {
 			// All is good, we expect all the cases to get here
 			for (let i = 0; i < stringsToResolve.length; i++) {
-				// Non null asserting here because we are iterating over stringsToResolve
-				stringsToResolve[i]!.content = result[i];
+				stringsToResolve[i].content = result[i];
 			}
 		} else {
 			// String content has \0 chars!

package/tsconfig.json

@@ -6,5 +6,6 @@
 		"rootDir": "./src",
 		"outDir": "./lib",
 		"exactOptionalPropertyTypes": false,
+		"noUncheckedIndexedAccess": false,
 	},
 }