@fluidframework/datastore 2.41.0-338186 → 2.41.0

package/src/dataStoreRuntime.ts

@@ -201,22 +201,25 @@ export class FluidDataStoreRuntime
 		return this.dataStoreContext.idCompressor;
 	}
 
-	public get IFluidHandleContext() {
+	// TODO: the methods below should have more specific return typing, per the interfaces they are implementing.
+	// Doing so would be a breaking change.
+
+	public get IFluidHandleContext(): this {
 		return this;
 	}
 
-	public get rootRoutingContext() {
+	public get rootRoutingContext(): this {
 		return this;
 	}
-	public get channelsRoutingContext() {
+	public get channelsRoutingContext(): this {
 		return this;
 	}
-	public get objectsRoutingContext() {
+	public get objectsRoutingContext(): this {
 		return this;
 	}
 
 	private _disposed = false;
-	public get disposed() {
+	public get disposed(): boolean {
 		return this._disposed;
 	}
 
@@ -234,6 +237,8 @@ export class FluidDataStoreRuntime
 
 	public readonly id: string;
 
+	// TODO: use something other than `any` here (breaking change)
+	// eslint-disable-next-line @typescript-eslint/no-explicit-any
 	public readonly options: Record<string | number, any>;
 	public readonly deltaManagerInternal: IDeltaManager<
 		ISequencedDocumentMessage,
@@ -337,10 +342,10 @@ export class FluidDataStoreRuntime
 
 		// Must always receive the data store type inside of the attributes
 		if (tree?.trees !== undefined) {
-			Object.entries(tree.trees).forEach(([path, subtree]) => {
+			for (const [path, subtree] of Object.entries(tree.trees)) {
 				// Issue #4414
 				if (path === "_search") {
-					return;
+					continue;
 				}
 
 				let channelContext: RemoteChannelContext | RehydratedLocalChannelContext;
@@ -353,10 +358,10 @@ export class FluidDataStoreRuntime
 					// data store, if the data store is loaded after the container is attached, then we missed making
 					// the channel visible. So do it now. Otherwise, add it to local channel context queue, so
 					// that it can be make it visible later with the data store.
-					if (dataStoreContext.attachState !== AttachState.Detached) {
-						channelContext.makeVisible();
-					} else {
+					if (dataStoreContext.attachState === AttachState.Detached) {
 						this.localChannelContextQueue.set(path, channelContext);
+					} else {
+						channelContext.makeVisible();
 					}
 				} else {
 					channelContext = new RemoteChannelContext(
@@ -376,7 +381,7 @@ export class FluidDataStoreRuntime
 				}
 
 				this.contexts.set(path, channelContext);
-			});
+			}
 		}
 
 		this.entryPoint = new FluidObjectHandle<FluidObject>(
@@ -508,7 +513,7 @@ export class FluidDataStoreRuntime
 	 * IDs cannot start with "_" as it could result in collision of IDs with auto-assigned (by FF) short IDs.
 	 * @param id - channel ID.
 	 */
-	protected validateChannelId(id: string) {
+	protected validateChannelId(id: string): void {
 		if (id.includes("/")) {
 			throw new UsageError(`Id cannot contain slashes: ${id}`);
 		}
@@ -546,10 +551,7 @@ export class FluidDataStoreRuntime
 	public createChannel(idArg: string | undefined, type: string): IChannel {
 		let id: string;
 
-		if (idArg !== undefined) {
-			id = idArg;
-			this.validateChannelId(id);
-		} else {
+		if (idArg === undefined) {
 			/**
 			 * Return uuid if short-ids are explicitly disabled via feature flags.
 			 */
@@ -562,18 +564,21 @@ export class FluidDataStoreRuntime
 				// - uuids
 				// In first two cases we will encode result as strings in more compact form, with leading underscore,
 				// to ensure no overlap with user-provided DDS names (see validateChannelId())
-				if (this.visibilityState !== VisibilityState.GloballyVisible) {
-					// container is detached, only one client observes content, no way to hit collisions with other clients.
-					id = encodeCompactIdToString(2 * this.contexts.size, "_");
-				} else {
+				if (this.visibilityState === VisibilityState.GloballyVisible) {
 					// Due to back-compat, we could not depend yet on generateDocumentUniqueId() being there.
 					// We can remove the need to leverage uuid() as fall-back in couple releases.
 					const res =
 						this.dataStoreContext.containerRuntime.generateDocumentUniqueId?.() ?? uuid();
 					id = typeof res === "number" ? encodeCompactIdToString(2 * res + 1, "_") : res;
+				} else {
+					// container is detached, only one client observes content, no way to hit collisions with other clients.
+					id = encodeCompactIdToString(2 * this.contexts.size, "_");
 				}
 			}
 			assert(!id.includes("/"), 0x8fc /* slash */);
+		} else {
+			id = idArg;
+			this.validateChannelId(id);
 		}
 
 		this.verifyNotClosed();
@@ -592,7 +597,7 @@ export class FluidDataStoreRuntime
 		return channel;
 	}
 
-	private createChannelContext(channel: IChannel) {
+	private createChannelContext(channel: IChannel): void {
 		this.notBoundedChannelContextSet.add(channel.id);
 		const context = new LocalChannelContext(
 			channel,
@@ -610,7 +615,7 @@ export class FluidDataStoreRuntime
 		id: string,
 		tree: ISnapshotTree,
 		flatBlobs?: Map<string, ArrayBufferLike>,
-	) {
+	): RehydratedLocalChannelContext {
 		return new RehydratedLocalChannelContext(
 			id,
 			this.sharedObjectRegistry,
@@ -672,15 +677,15 @@ export class FluidDataStoreRuntime
 	 * visible, it will mark us globally visible. Otherwise, it will mark us globally visible when it becomes
 	 * globally visible.
 	 */
-	public makeVisibleAndAttachGraph() {
+	public makeVisibleAndAttachGraph(): void {
 		if (this.visibilityState !== VisibilityState.NotVisible) {
 			return;
 		}
 		this.visibilityState = VisibilityState.LocallyVisible;
 
-		this.pendingHandlesToMakeVisible.forEach((handle) => {
+		for (const handle of this.pendingHandlesToMakeVisible) {
 			handle.attachGraph();
-		});
+		}
 		this.pendingHandlesToMakeVisible.clear();
 		this.dataStoreContext.makeLocallyVisible();
 	}
@@ -688,7 +693,7 @@ export class FluidDataStoreRuntime
 	/**
 	 * This function is called when a handle to this data store is added to a visible DDS.
 	 */
-	public attachGraph() {
+	public attachGraph(): void {
 		this.makeVisibleAndAttachGraph();
 	}
 
@@ -701,7 +706,7 @@ export class FluidDataStoreRuntime
 		this.pendingHandlesToMakeVisible.add(toFluidHandleInternal(handle));
 	}
 
-	public setConnectionState(connected: boolean, clientId?: string) {
+	public setConnectionState(connected: boolean, clientId?: string): void {
 		this.verifyNotClosed();
 
 		for (const [, object] of this.contexts) {
@@ -745,7 +750,7 @@ export class FluidDataStoreRuntime
 	private createRemoteChannelContext(
 		attachMessage: IAttachMessage,
 		summarizerNodeParams: CreateChildSummarizerNodeParam,
-	) {
+	): RemoteChannelContext {
 		const flatBlobs = new Map<string, ArrayBufferLike>();
 		const snapshotTree = buildSnapshotTree(attachMessage.snapshot.entries, flatBlobs);
 
@@ -773,7 +778,7 @@ export class FluidDataStoreRuntime
 	 * store.
 	 * @param messageCollection - The collection of messages to process.
 	 */
-	private processChannelMessages(messageCollection: IRuntimeMessageCollection) {
+	private processChannelMessages(messageCollection: IRuntimeMessageCollection): void {
 		this.verifyNotClosed();
 
 		/*
@@ -783,9 +788,9 @@ export class FluidDataStoreRuntime
 		 */
 		let currentAddress: string | undefined;
 		let currentMessagesContent: IRuntimeMessagesContent[] = [];
-		const { messagesContent, local } = messageCollection;
+		const { messagesContent, local, envelope } = messageCollection;
 
-		const sendBunchedMessages = () => {
+		const sendBunchedMessages = (): void => {
 			// Current address will be undefined for the first message in the list.
 			if (currentAddress === undefined) {
 				return;
@@ -796,7 +801,7 @@ export class FluidDataStoreRuntime
 			assert(!!channelContext, 0xa6b /* Channel context not found */);
 
 			channelContext.processMessages({
-				envelope: messageCollection.envelope,
+				envelope,
 				messagesContent: currentMessagesContent,
 				local,
 			});
@@ -823,7 +828,7 @@ export class FluidDataStoreRuntime
 		sendBunchedMessages();
 	}
 
-	private processAttachMessages(messageCollection: IRuntimeMessageCollection) {
+	private processAttachMessages(messageCollection: IRuntimeMessageCollection): void {
 		const { envelope, messagesContent, local } = messageCollection;
 		for (const { contents } of messagesContent) {
 			const attachMessage = contents as IAttachMessage;
@@ -871,12 +876,14 @@ export class FluidDataStoreRuntime
 		const { envelope, messagesContent } = messageCollection;
 		try {
 			switch (envelope.type) {
-				case DataStoreMessageType.ChannelOp:
+				case DataStoreMessageType.ChannelOp: {
 					this.processChannelMessages(messageCollection);
 					break;
-				case DataStoreMessageType.Attach:
+				}
+				case DataStoreMessageType.Attach: {
 					this.processAttachMessages(messageCollection);
 					break;
+				}
 				default:
 			}
 		} catch (error) {
@@ -892,7 +899,7 @@ export class FluidDataStoreRuntime
 		}
 	}
 
-	public processSignal(message: IInboundSignalMessage, local: boolean) {
+	public processSignal(message: IInboundSignalMessage, local: boolean): void {
 		this.emit("signal", message, local);
 	}
 
@@ -929,7 +936,7 @@ export class FluidDataStoreRuntime
 	 * - Adds a node for this channel.
 	 * @param builder - The builder that contains the GC nodes for this channel's children.
 	 */
-	private updateGCNodes(builder: GCDataBuilder) {
+	private updateGCNodes(builder: GCDataBuilder): void {
 		// Add a back route to self in each child's GC nodes. If any child is referenced, then its parent should
 		// be considered referenced as well.
 		builder.addRouteToAllNodes(this.absolutePath);
@@ -993,7 +1000,7 @@ export class FluidDataStoreRuntime
 	 * update their used routes.
 	 * @param usedRoutes - The routes that are used in all contexts in this channel.
 	 */
-	public updateUsedRoutes(usedRoutes: string[]) {
+	public updateUsedRoutes(usedRoutes: string[]): void {
 		// Get a map of channel ids to routes used in it.
 		const usedContextRoutes = unpackChildNodesUsedRoutes(usedRoutes);
 
@@ -1132,7 +1139,13 @@ export class FluidDataStoreRuntime
 		}
 	}
 
-	public submitMessage(type: DataStoreMessageType, content: any, localOpMetadata: unknown) {
+	public submitMessage(
+		type: DataStoreMessageType,
+		// TODO: use something other than `any` here (breaking change)
+		// eslint-disable-next-line @typescript-eslint/explicit-module-boundary-types, @typescript-eslint/no-explicit-any
+		content: any,
+		localOpMetadata: unknown,
+	): void {
 		this.submit(type, content, localOpMetadata);
 	}
 
@@ -1142,9 +1155,9 @@ export class FluidDataStoreRuntime
 	 * @param content - Content of the signal. Should be a JSON serializable object or primitive.
 	 * @param targetClientId - When specified, the signal is only sent to the provided client id.
 	 */
-	public submitSignal(type: string, content: unknown, targetClientId?: string) {
+	public submitSignal(type: string, content: unknown, targetClientId?: string): void {
 		this.verifyNotClosed();
-		return this.dataStoreContext.submitSignal(type, content, targetClientId);
+		this.dataStoreContext.submitSignal(type, content, targetClientId);
 	}
 
 	/**
@@ -1201,14 +1214,14 @@ export class FluidDataStoreRuntime
 		context.makeVisible();
 	}
 
-	private submitChannelOp(address: string, contents: any, localOpMetadata: unknown) {
+	private submitChannelOp(address: string, contents: unknown, localOpMetadata: unknown): void {
 		const envelope: IEnvelope = { address, contents };
 		this.submit(DataStoreMessageType.ChannelOp, envelope, localOpMetadata);
 	}
 
 	private submit(
 		type: DataStoreMessageType,
-		content: any,
+		content: unknown,
 		localOpMetadata: unknown = undefined,
 	): void {
 		this.verifyNotClosed();
@@ -1224,10 +1237,12 @@ export class FluidDataStoreRuntime
 	 */
 	public reSubmit(
 		type: DataStoreMessageType,
+		// TODO: use something other than `any` here (breaking change)
+		// eslint-disable-next-line @typescript-eslint/explicit-module-boundary-types, @typescript-eslint/no-explicit-any
 		content: any,
 		localOpMetadata: unknown,
 		squash?: boolean,
-	) {
+	): void {
 		this.verifyNotClosed();
 
 		switch (type) {
@@ -1239,12 +1254,14 @@ export class FluidDataStoreRuntime
 				channelContext.reSubmit(envelope.contents, localOpMetadata, squash);
 				break;
 			}
-			case DataStoreMessageType.Attach:
+			case DataStoreMessageType.Attach: {
 				// For Attach messages, just submit them again.
 				this.submit(type, content, localOpMetadata);
 				break;
-			default:
+			}
+			default: {
 				unreachableCase(type);
+			}
 		}
 	}
 
@@ -1253,7 +1270,13 @@ export class FluidDataStoreRuntime
 	 * @param content - The content of the original message.
 	 * @param localOpMetadata - The local metadata associated with the original message.
 	 */
-	public rollback?(type: DataStoreMessageType, content: any, localOpMetadata: unknown) {
+	public rollback?(
+		type: DataStoreMessageType,
+		// TODO: use something other than `any` here (breaking change)
+		// eslint-disable-next-line @typescript-eslint/explicit-module-boundary-types, @typescript-eslint/no-explicit-any
+		content: any,
+		localOpMetadata: unknown,
+	): void {
 		this.verifyNotClosed();
 
 		switch (type) {
@@ -1265,15 +1288,20 @@ export class FluidDataStoreRuntime
 				channelContext.rollback(envelope.contents, localOpMetadata);
 				break;
 			}
-			default:
+			default: {
 				throw new LoggingError(`Can't rollback ${type} message`);
+			}
 		}
 	}
 
+	// TODO: use something other than `any` here
+	// eslint-disable-next-line @typescript-eslint/explicit-module-boundary-types, @typescript-eslint/no-explicit-any
 	public async applyStashedOp(content: any): Promise<unknown> {
+		// eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
 		const type = content?.type as DataStoreMessageType;
 		switch (type) {
 			case DataStoreMessageType.Attach: {
+				// eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
 				const attachMessage = content.content as IAttachMessage;
 
 				const flatBlobs = new Map<string, ArrayBufferLike>();
@@ -1295,14 +1323,16 @@ export class FluidDataStoreRuntime
 				return;
 			}
 			case DataStoreMessageType.ChannelOp: {
+				// eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
 				const envelope = content.content as IEnvelope;
 				const channelContext = this.contexts.get(envelope.address);
 				assert(!!channelContext, 0x184 /* "There should be a channel context for the op" */);
 				await channelContext.getChannel();
 				return channelContext.applyStashedOp(envelope.contents);
 			}
-			default:
+			default: {
 				unreachableCase(type);
+			}
 		}
 	}
 
@@ -1311,23 +1341,25 @@ export class FluidDataStoreRuntime
 		this.dataStoreContext.setChannelDirty(address);
 	}
 
-	private attachListener() {
+	private attachListener(): void {
 		this.setMaxListeners(Number.MAX_SAFE_INTEGER);
 
 		// back-compat, to be removed in the future.
 		// Added in "2.0.0-rc.2.0.0" timeframe.
+		// eslint-disable-next-line @typescript-eslint/no-unsafe-member-access, @typescript-eslint/no-unsafe-call, @typescript-eslint/no-explicit-any
 		(this.dataStoreContext as any).once?.("attaching", () => {
 			this.setAttachState(AttachState.Attaching);
 		});
 
 		// back-compat, to be removed in the future.
 		// Added in "2.0.0-rc.2.0.0" timeframe.
+		// eslint-disable-next-line @typescript-eslint/no-unsafe-member-access, @typescript-eslint/no-unsafe-call, @typescript-eslint/no-explicit-any
 		(this.dataStoreContext as any).once?.("attached", () => {
 			this.setAttachState(AttachState.Attached);
 		});
 	}
 
-	private verifyNotClosed() {
+	private verifyNotClosed(): void {
 		if (this._disposed) {
 			throw new LoggingError("Runtime is closed");
 		}
@@ -1342,7 +1374,7 @@ export class FluidDataStoreRuntime
 		eventName: string,
 		channelId: string,
 		channelType: string,
-	) {
+	): void {
 		if (this.clientDetails.type !== "summarizer" || this.localChangesTelemetryCount <= 0) {
 			return;
 		}
@@ -1365,7 +1397,7 @@ export class FluidDataStoreRuntime
 
 	public setAttachState(attachState: AttachState.Attaching | AttachState.Attached): void {
 		switch (attachState) {
-			case AttachState.Attaching:
+			case AttachState.Attaching: {
 				/**
 				 * back-compat 0.59.1000 - Ideally, attachGraph() should have already been called making the data store
 				 * locally visible. However, before visibility state was added, this may not have been the case and data
@@ -1386,16 +1418,17 @@ export class FluidDataStoreRuntime
 
 				// Mark the data store globally visible and make its child channels visible as well.
 				this.visibilityState = VisibilityState.GloballyVisible;
-				this.localChannelContextQueue.forEach((channel) => {
+				for (const [, channel] of this.localChannelContextQueue) {
 					channel.makeVisible();
-				});
+				}
 				this.localChannelContextQueue.clear();
 
 				// This promise resolution will be moved to attached event once we fix the scheduler.
 				this.deferredAttached.resolve();
 				this.emit("attaching");
 				break;
-			case AttachState.Attached:
+			}
+			case AttachState.Attached: {
 				assert(
 					this.visibilityState === VisibilityState.GloballyVisible,
 					0x2d2 /* "Data store should be globally visible when its attached." */,
@@ -1403,8 +1436,10 @@ export class FluidDataStoreRuntime
 				this._attachState = AttachState.Attached;
 				this.emit("attached");
 				break;
-			default:
+			}
+			default: {
 				unreachableCase(attachState, "unreached");
+			}
 		}
 	}
 }
@@ -1420,9 +1455,9 @@ export class FluidDataStoreRuntime
 export const mixinRequestHandler = (
 	requestHandler: (request: IRequest, runtime: FluidDataStoreRuntime) => Promise<IResponse>,
 	Base: typeof FluidDataStoreRuntime = FluidDataStoreRuntime,
-) =>
+): typeof FluidDataStoreRuntime =>
 	class RuntimeWithRequestHandler extends Base {
-		public async request(request: IRequest) {
+		public async request(request: IRequest): Promise<IResponse> {
 			const response = await super.request(request);
 			if (response.status === 404) {
 				return requestHandler(request, this);
@@ -1444,9 +1479,9 @@ export const mixinSummaryHandler = (
 		runtime: FluidDataStoreRuntime,
 	) => Promise<{ path: string[]; content: string } | undefined>,
 	Base: typeof FluidDataStoreRuntime = FluidDataStoreRuntime,
-) =>
+): typeof FluidDataStoreRuntime =>
 	class RuntimeWithSummarizerHandler extends Base {
-		private addBlob(summary: ISummaryTreeWithStats, path: string[], content: string) {
+		private addBlob(summary: ISummaryTreeWithStats, path: string[], content: string): void {
 			const firstName = path.shift();
 			if (firstName === undefined) {
 				throw new LoggingError("Path can't be empty");
@@ -1469,7 +1504,8 @@ export const mixinSummaryHandler = (
 			summary.summary.tree[firstName] = blob;
 		}
 
-		async summarize(...args: any[]) {
+		async summarize(...args: any[]): Promise<ISummaryTreeWithStats> {
+			// eslint-disable-next-line @typescript-eslint/no-unsafe-argument
 			const summary = await super.summarize(...args);
 
 			try {
@@ -1477,9 +1513,9 @@ export const mixinSummaryHandler = (
 				if (content !== undefined) {
 					this.addBlob(summary, content.path, content.content);
 				}
-			} catch (e) {
+			} catch (error) {
 				// Any error coming from app-provided handler should be marked as DataProcessingError
-				throw DataProcessingError.wrapIfUnrecognized(e, "mixinSummaryHandler");
+				throw DataProcessingError.wrapIfUnrecognized(error, "mixinSummaryHandler");
 			}
 
 			return summary;

package/src/fluidHandle.ts

@@ -77,6 +77,9 @@ export class FluidObjectHandle<
 	/**
 	 * {@inheritDoc @fluidframework/core-interfaces#IFluidHandle.get}
 	 */
+	// TODO: Return `Promise<T>` instead of `Promise<any>`.
+	// This was clearly the intended typing of this API, but fixing it would be a breaking change.
+	// eslint-disable-next-line @typescript-eslint/no-explicit-any
 	public async get(): Promise<any> {
 		// Note that this return works whether we received a T or a Promise<T> for this.value in the constructor.
 		return this.value;
@@ -91,18 +94,19 @@ export class FluidObjectHandle<
 		}
 
 		this.locallyVisible = true;
-		this.pendingHandlesToMakeVisible.forEach((handle) => {
+		for (const handle of this.pendingHandlesToMakeVisible) {
 			handle.attachGraph();
-		});
+		}
 		this.pendingHandlesToMakeVisible.clear();
 		this.routeContext.attachGraph();
 	}
 
+	// eslint-disable-next-line jsdoc/require-description
 	/**
 	 * @deprecated No replacement provided. Arbitrary handles may not serve as a bind source.
 	 * @privateRemarks This implementation will be moved to SharedObjectHandle once this is removed.
 	 */
-	public bind(handle: IFluidHandleInternal) {
+	public bind(handle: IFluidHandleInternal): void {
 		// If this handle is visible, attach the graph of the incoming handle as well.
 		if (this.visible) {
 			handle.attachGraph();

package/src/localChannelContext.ts

@@ -42,7 +42,9 @@ import { ISharedObjectRegistry } from "./dataStoreRuntime.js";
  */
 export abstract class LocalChannelContextBase implements IChannelContext {
 	private globallyVisible = false;
-	/** Tracks the messages for this channel that are sent while it's not loaded */
+	/**
+	 * Tracks the messages for this channel that are sent while it's not loaded
+	 */
 	protected pendingMessagesState: IPendingMessagesState = {
 		messageCollections: [],
 		pendingCount: 0,
@@ -57,7 +59,7 @@ export abstract class LocalChannelContextBase implements IChannelContext {
 		assert(!this.id.includes("/"), 0x30f /* Channel context ID cannot contain slashes */);
 	}
 
-	protected get isGloballyVisible() {
+	protected get isGloballyVisible(): boolean {
 		return this.globallyVisible;
 	}
 
@@ -72,7 +74,7 @@ export abstract class LocalChannelContextBase implements IChannelContext {
 		return this._channel !== undefined;
 	}
 
-	public setConnectionState(connected: boolean, clientId?: string) {
+	public setConnectionState(connected: boolean, clientId?: string): void {
 		// Connection events are ignored if the data store is not yet globallyVisible or loaded
 		if (this.globallyVisible && this.isLoaded) {
 			this.services.value.deltaConnection.setConnectionState(connected);
@@ -101,13 +103,13 @@ export abstract class LocalChannelContextBase implements IChannelContext {
 			);
 			const propsCopy = {
 				...messageCollection,
-				messagesContent: Array.from(messageCollection.messagesContent),
+				messagesContent: [...messageCollection.messagesContent],
 			};
 			this.pendingMessagesState.messageCollections.push(propsCopy);
 		}
 	}
 
-	public reSubmit(content: any, localOpMetadata: unknown, squash: boolean) {
+	public reSubmit(content: unknown, localOpMetadata: unknown, squash: boolean): void {
 		assert(this.isLoaded, 0x18a /* "Channel should be loaded to resubmit ops" */);
 		assert(
 			this.globallyVisible,
@@ -115,7 +117,7 @@ export abstract class LocalChannelContextBase implements IChannelContext {
 		);
 		this.services.value.deltaConnection.reSubmit(content, localOpMetadata, squash);
 	}
-	public rollback(content: any, localOpMetadata: unknown) {
+	public rollback(content: unknown, localOpMetadata: unknown): void {
 		assert(this.isLoaded, 0x2ee /* "Channel should be loaded to rollback ops" */);
 		assert(
 			this.globallyVisible,
@@ -198,7 +200,7 @@ export abstract class LocalChannelContextBase implements IChannelContext {
 		return channel.getGCData(fullGC);
 	}
 
-	public updateUsedRoutes(usedRoutes: string[]) {
+	public updateUsedRoutes(usedRoutes: string[]): void {
 		/**
 		 * Currently, DDSes are always considered referenced and are not garbage collected.
 		 * Once we have GC at DDS level, this channel context's used routes will be updated as per the passed
@@ -216,7 +218,7 @@ export class RehydratedLocalChannelContext extends LocalChannelContextBase {
 		dataStoreContext: IFluidDataStoreContext,
 		storageService: IDocumentStorageService,
 		logger: ITelemetryLoggerExt,
-		submitFn: (content: any, localOpMetadata: unknown) => void,
+		submitFn: (content: unknown, localOpMetadata: unknown) => void,
 		dirtyFn: (address: string) => void,
 		private readonly snapshotTree: ISnapshotTree,
 		extraBlob?: Map<string, ArrayBufferLike>,
@@ -267,9 +269,9 @@ export class RehydratedLocalChannelContext extends LocalChannelContextBase {
 						this.services.value.deltaConnection.processMessages(messageCollection);
 					}
 					return channel;
-				} catch (err) {
+				} catch (error) {
 					throw DataProcessingError.wrapIfUnrecognized(
-						err,
+						error,
 						"rehydratedLocalChannelContextFailedToLoadChannel",
 						undefined,
 					);
@@ -282,7 +284,7 @@ export class RehydratedLocalChannelContext extends LocalChannelContextBase {
 		};
 	}
 
-	public override applyStashedOp(content) {
+	public override applyStashedOp(content: unknown): unknown {
 		return this.services.value.deltaConnection.applyStashedOp(content);
 	}
 
@@ -293,12 +295,12 @@ export class RehydratedLocalChannelContext extends LocalChannelContextBase {
 		let sanitize = false;
 		const blobsContents = snapshotTree.blobsContents;
 		if (blobsContents !== undefined) {
-			Object.entries(blobsContents).forEach(([key, value]) => {
+			for (const [key, value] of Object.entries(blobsContents)) {
 				blobMap.set(key, value);
 				if (snapshotTree.blobs[key] !== undefined) {
 					sanitize = true;
 				}
-			});
+			}
 		}
 		for (const value of Object.values(snapshotTree.trees)) {
 			sanitize = sanitize || this.isSnapshotInOldFormatAndCollectBlobs(value, blobMap);
@@ -306,7 +308,7 @@ export class RehydratedLocalChannelContext extends LocalChannelContextBase {
 		return sanitize;
 	}
 
-	private sanitizeSnapshot(snapshotTree: ISnapshotTree) {
+	private sanitizeSnapshot(snapshotTree: ISnapshotTree): void {
 		const blobMapInitial = new Map(Object.entries(snapshotTree.blobs));
 		for (const [blobName, blobId] of blobMapInitial.entries()) {
 			const blobValue = blobMapInitial.get(blobId);
@@ -329,7 +331,7 @@ export class LocalChannelContext extends LocalChannelContextBase {
 		dataStoreContext: IFluidDataStoreContext,
 		storageService: IDocumentStorageService,
 		logger: ITelemetryLoggerExt,
-		submitFn: (content: any, localOpMetadata: unknown) => void,
+		submitFn: (content: unknown, localOpMetadata: unknown) => void,
 		dirtyFn: (address: string) => void,
 	) {
 		super(
@@ -355,7 +357,7 @@ export class LocalChannelContext extends LocalChannelContextBase {
 		};
 	}
 
-	public applyStashedOp() {
+	public applyStashedOp(): void {
 		throw new Error("no stashed ops on local channel");
 	}
 }