@fluidframework/container-loader 2.0.0-internal.5.1.1 → 2.0.0-internal.5.3.0

package/src/{collabWindowTracker.ts → noopHeuristic.ts}

@@ -3,13 +3,18 @@
  * Licensed under the MIT License.
  */
 
-import { assert, Timer } from "@fluidframework/common-utils";
-import { ISequencedDocumentMessage, MessageType } from "@fluidframework/protocol-definitions";
-import { isRuntimeMessage, MessageType2 } from "@fluidframework/driver-utils";
+import { assert, Timer, TypedEventEmitter } from "@fluidframework/common-utils";
+import { ISequencedDocumentMessage } from "@fluidframework/protocol-definitions";
+import { isRuntimeMessage } from "@fluidframework/driver-utils";
+import { IEvent } from "@fluidframework/common-definitions";
 
 const defaultNoopTimeFrequency = 2000;
 const defaultNoopCountFrequency = 50;
 
+export interface INoopSenderEvents extends IEvent {
+	(event: "wantsNoop", listener: () => void);
+}
+
 // Here are key considerations when deciding conditions for when to send non-immediate noops:
 // 1. Sending them too often results in increase in file size and bandwidth, as well as catch up performance
 // 2. Sending too infrequently ensures that collab window is large, and as result Sequence DDS would have
@@ -29,21 +34,21 @@ const defaultNoopCountFrequency = 50;
 //    server timeout of 2000ms should be reconsidered to be increased.
 // 2. If there are more than 50 ops received without sending any ops, send noop to keep collab window small.
 //    Note that system ops (including noops themselves) are excluded, so it's 1 noop per 50 real ops.
-export class CollabWindowTracker {
-	private opsCountSinceNoop = 0;
+export class NoopHeuristic extends TypedEventEmitter<INoopSenderEvents> {
+	private opsProcessedSinceOpSent = 0;
 	private readonly timer: Timer | undefined;
 
 	constructor(
-		private readonly submit: (type: MessageType) => void,
 		NoopTimeFrequency: number = defaultNoopTimeFrequency,
 		private readonly NoopCountFrequency: number = defaultNoopCountFrequency,
 	) {
+		super();
 		if (NoopTimeFrequency !== Infinity) {
 			this.timer = new Timer(NoopTimeFrequency, () => {
-				// Can get here due to this.stopSequenceNumberUpdate() not resetting timer.
-				// Also timer callback can fire even after timer cancellation if it was queued before cancellation.
-				if (this.opsCountSinceNoop !== 0) {
-					this.submitNoop(false /* immediate */);
+				// We allow the timer to expire even if an op is sent or we disconnect.
+				// This condition is to guard against trying to send a noop anyway in that case.
+				if (this.opsProcessedSinceOpSent !== 0) {
+					this.emit("wantsNoop");
 				}
 			});
 		}
@@ -52,17 +57,7 @@ export class CollabWindowTracker {
 	/**
 	 * Schedules as ack to the server to update the reference sequence number
 	 */
-	public scheduleSequenceNumberUpdate(
-		message: ISequencedDocumentMessage,
-		immediateNoOp: boolean,
-	): void {
-		// While processing a message, an immediate no-op can be requested.
-		// i.e. to expedite approve or commit phase of quorum.
-		if (immediateNoOp) {
-			this.submitNoop(true /* immediate */);
-			return;
-		}
-
+	public notifyMessageProcessed(message: ISequencedDocumentMessage): void {
 		// We don't acknowledge no-ops to avoid acknowledgement cycles (i.e. ack the MSN
 		// update, which updates the MSN, then ack the update, etc...).
 		// Intent here is for runtime (and DDSes) not to keep too much tracking state / memory
@@ -71,22 +66,27 @@ export class CollabWindowTracker {
 			return;
 		}
 
-		this.opsCountSinceNoop++;
-		if (this.opsCountSinceNoop === this.NoopCountFrequency) {
-			// Ensure we only send noop after a batch of many ops is processed
-			// eslint-disable-next-line @typescript-eslint/no-floating-promises
-			Promise.resolve().then(() => {
-				if (this.opsCountSinceNoop >= this.NoopCountFrequency) {
-					this.submitNoop(false /* immediate */);
-					// reset count now that all ops are processed
-					this.opsCountSinceNoop = 0;
+		this.opsProcessedSinceOpSent++;
+		if (this.opsProcessedSinceOpSent === this.NoopCountFrequency) {
+			// Wait to send a noop if we are still synchronously processing ops.  This guards against two things:
+			// 1. If we're processing many ops, we may pass the frequency threshold many times.  We only need to send one noop at the very end in this case.
+			// 2. We may send another (non-noop) op in response to processing those ops, e.g. an Accept op.
+			queueMicrotask(() => {
+				if (this.opsProcessedSinceOpSent >= this.NoopCountFrequency) {
+					this.emit("wantsNoop");
+					assert(
+						this.opsProcessedSinceOpSent === 0,
+						0x243 /* "Expected a noop to be synchronously sent" */,
+					);
 				}
 				return;
 			});
 		}
 
 		if (this.timer !== undefined) {
-			if (this.opsCountSinceNoop === 1) {
+			// Start the timer if we newly have ops that want a noop.
+			// If the timer was already running (e.g. we surpassed the op count and sent a noop) this will reset it to its full duration.
+			if (this.opsProcessedSinceOpSent === 1) {
 				this.timer.restart();
 			}
 
@@ -94,23 +94,13 @@ export class CollabWindowTracker {
 		}
 	}
 
-	private submitNoop(immediate: boolean) {
-		// Anything other than null is immediate noop
-		// ADO:1385: Remove cast and use MessageType once definition changes propagate
-		this.submit(immediate ? (MessageType2.Accept as unknown as MessageType) : MessageType.NoOp);
-		assert(
-			this.opsCountSinceNoop === 0,
-			0x243 /* "stopSequenceNumberUpdate should be called as result of sending any op!" */,
-		);
+	public notifyDisconnect(): void {
+		// No need to noop for any ops processed prior to disconnect - we are already removed from MSN calculation.
+		this.opsProcessedSinceOpSent = 0;
 	}
 
-	public stopSequenceNumberUpdate(): void {
-		this.opsCountSinceNoop = 0;
-		// Ideally, we cancel timer here. But that will result in too often set/reset cycle if this client
-		// keeps sending ops. In most cases it's actually better to let it expire (at most - 4 times per second)
-		// for nothing, then have a ton of set/reset cycles.
-		// Note that Timer.restart() is smart and will not change timer expiration if we keep extending timer
-		// expiration - it will restart the timer instead when it fires with adjusted expiration.
-		// this.timer.clear();
+	public notifyMessageSent(): void {
+		// Sending any message is as good as a noop for updating MSN.
+		this.opsProcessedSinceOpSent = 0;
 	}
 }

package/src/packageVersion.ts

@@ -6,4 +6,4 @@
  */
 
 export const pkgName = "@fluidframework/container-loader";
-export const pkgVersion = "2.0.0-internal.5.1.1";
+export const pkgVersion = "2.0.0-internal.5.3.0";

package/src/protocol.ts

@@ -11,14 +11,9 @@ import {
 } from "@fluidframework/protocol-base";
 import {
 	IDocumentAttributes,
-	IProcessMessageResult,
-	ISequencedClient,
-	ISequencedDocumentMessage,
 	ISignalClient,
 	ISignalMessage,
-	MessageType,
 } from "@fluidframework/protocol-definitions";
-import { canBeCoalescedByService } from "@fluidframework/driver-utils";
 
 // "term" was an experimental feature that is being removed.  The only safe value to use is 1.
 export const OnlyValidTermValue = 1 as const;
@@ -30,17 +25,6 @@ export enum SignalType {
 	Clear = "clear", // used only by client for synthetic signals
 }
 
-/**
- * ADO: #4277: ConnectionStateHandler can mutate Quorum members, but shouldn't
- * This interface might go away after the above ADO item is done
- */
-export interface ILocalSequencedClient extends ISequencedClient {
-	/**
-	 * True if the client should have left the quorum, false otherwise
-	 */
-	shouldHaveLeft?: boolean;
-}
-
 /**
  * Function to be used for creating a protocol handler.
  */
@@ -82,29 +66,6 @@ export class ProtocolHandler extends ProtocolOpHandler implements IProtocolHandl
 		}
 	}
 
-	public processMessage(
-		message: ISequencedDocumentMessage,
-		local: boolean,
-	): IProcessMessageResult {
-		const client: ILocalSequencedClient | undefined = this.quorum.getMember(message.clientId);
-
-		// Check and report if we're getting messages from a clientId that we previously
-		// flagged as shouldHaveLeft, or from a client that's not in the quorum but should be
-		if (message.clientId != null) {
-			if (client === undefined && message.type !== MessageType.ClientJoin) {
-				// pre-0.58 error message: messageClientIdMissingFromQuorum
-				throw new Error("Remote message's clientId is missing from the quorum");
-			}
-
-			if (client?.shouldHaveLeft === true && !canBeCoalescedByService(message)) {
-				// pre-0.58 error message: messageClientIdShouldHaveLeft
-				throw new Error("Remote message's clientId already should have left");
-			}
-		}
-
-		return super.processMessage(message, local);
-	}
-
 	public processSignal(message: ISignalMessage) {
 		const innerContent = message.content as { content: any; type: string };
 		switch (innerContent.type) {
@@ -138,3 +99,21 @@ export class ProtocolHandler extends ProtocolOpHandler implements IProtocolHandl
 		}
 	}
 }
+
+/**
+ * Function to check whether the protocol handler should process the Signal.
+ * The protocol handler should strictly handle only ClientJoin, ClientLeave
+ * and Clear signal types.
+ */
+export function protocolHandlerShouldProcessSignal(message: ISignalMessage) {
+	// Signal originates from server
+	if (message.clientId === null) {
+		const innerContent = message.content as { content: unknown; type: string };
+		return (
+			innerContent.type === SignalType.Clear ||
+			innerContent.type === SignalType.ClientJoin ||
+			innerContent.type === SignalType.ClientLeave
+		);
+	}
+	return false;
+}

package/src/protocolTreeDocumentStorageService.ts

@@ -3,7 +3,7 @@
  * Licensed under the MIT License.
  */
 
-import { IDisposable } from "@fluidframework/common-definitions";
+import { IDisposable } from "@fluidframework/core-interfaces";
 import { IDocumentStorageService, ISummaryContext } from "@fluidframework/driver-definitions";
 import { ISummaryTree } from "@fluidframework/protocol-definitions";
 

package/src/quorum.ts

@@ -2,38 +2,9 @@
  * Copyright (c) Microsoft Corporation and contributors. All rights reserved.
  * Licensed under the MIT License.
  */
-import { assert, EventForwarder, doIfNotDisposed } from "@fluidframework/common-utils";
+import { assert } from "@fluidframework/common-utils";
 import { IFluidCodeDetails } from "@fluidframework/core-interfaces";
-import {
-	ICommittedProposal,
-	IQuorum,
-	IQuorumEvents,
-	ISequencedClient,
-} from "@fluidframework/protocol-definitions";
-
-/**
- * Proxies Quorum events.
- */
-export class QuorumProxy extends EventForwarder<IQuorumEvents> implements IQuorum {
-	public readonly propose: (key: string, value: any) => Promise<void>;
-	public readonly has: (key: string) => boolean;
-	public readonly get: (key: string) => any;
-	public readonly getMembers: () => Map<string, ISequencedClient>;
-	public readonly getMember: (clientId: string) => ISequencedClient | undefined;
-
-	constructor(quorum: IQuorum) {
-		super(quorum);
-
-		// This is heavily used object, increase limit at which Node prints warnings.
-		super.setMaxListeners(50);
-
-		this.propose = doIfNotDisposed(this, quorum.propose.bind(quorum));
-		this.has = doIfNotDisposed(this, quorum.has.bind(quorum));
-		this.get = doIfNotDisposed(this, quorum.get.bind(quorum));
-		this.getMembers = doIfNotDisposed(this, quorum.getMembers.bind(quorum));
-		this.getMember = doIfNotDisposed(this, quorum.getMember.bind(quorum));
-	}
-}
+import { ICommittedProposal } from "@fluidframework/protocol-definitions";
 
 export function getCodeDetailsFromQuorumValues(
 	quorumValues: [string, ICommittedProposal][],

package/src/retriableDocumentStorageService.ts

@@ -18,7 +18,8 @@ import {
 	ISummaryTree,
 	IVersion,
 } from "@fluidframework/protocol-definitions";
-import { IDisposable } from "@fluidframework/common-definitions";
+import { IDisposable } from "@fluidframework/core-interfaces";
+
 import { ITelemetryLoggerExt } from "@fluidframework/telemetry-utils";
 import { runWithRetry } from "@fluidframework/driver-utils";
 

package/dist/collabWindowTracker.d.ts

@@ -1,19 +0,0 @@
-/*!
- * Copyright (c) Microsoft Corporation and contributors. All rights reserved.
- * Licensed under the MIT License.
- */
-import { ISequencedDocumentMessage, MessageType } from "@fluidframework/protocol-definitions";
-export declare class CollabWindowTracker {
-    private readonly submit;
-    private readonly NoopCountFrequency;
-    private opsCountSinceNoop;
-    private readonly timer;
-    constructor(submit: (type: MessageType) => void, NoopTimeFrequency?: number, NoopCountFrequency?: number);
-    /**
-     * Schedules as ack to the server to update the reference sequence number
-     */
-    scheduleSequenceNumberUpdate(message: ISequencedDocumentMessage, immediateNoOp: boolean): void;
-    private submitNoop;
-    stopSequenceNumberUpdate(): void;
-}
-//# sourceMappingURL=collabWindowTracker.d.ts.map

package/dist/collabWindowTracker.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"collabWindowTracker.d.ts","sourceRoot":"","sources":["../src/collabWindowTracker.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAGH,OAAO,EAAE,yBAAyB,EAAE,WAAW,EAAE,MAAM,sCAAsC,CAAC;AAyB9F,qBAAa,mBAAmB;IAK9B,OAAO,CAAC,QAAQ,CAAC,MAAM;IAEvB,OAAO,CAAC,QAAQ,CAAC,kBAAkB;IANpC,OAAO,CAAC,iBAAiB,CAAK;IAC9B,OAAO,CAAC,QAAQ,CAAC,KAAK,CAAoB;gBAGxB,MAAM,EAAE,CAAC,IAAI,EAAE,WAAW,KAAK,IAAI,EACpD,iBAAiB,GAAE,MAAiC,EACnC,kBAAkB,GAAE,MAAkC;IAaxE;;OAEG;IACI,4BAA4B,CAClC,OAAO,EAAE,yBAAyB,EAClC,aAAa,EAAE,OAAO,GACpB,IAAI;IAuCP,OAAO,CAAC,UAAU;IAUX,wBAAwB,IAAI,IAAI;CASvC"}

package/dist/collabWindowTracker.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"collabWindowTracker.js","sourceRoot":"","sources":["../src/collabWindowTracker.ts"],"names":[],"mappings":";AAAA;;;GAGG;;;AAEH,+DAA6D;AAC7D,+EAA8F;AAC9F,+DAA8E;AAE9E,MAAM,wBAAwB,GAAG,IAAI,CAAC;AACtC,MAAM,yBAAyB,GAAG,EAAE,CAAC;AAErC,6FAA6F;AAC7F,4GAA4G;AAC5G,yGAAyG;AACzG,2CAA2C;AAC3C,oHAAoH;AACpH,2FAA2F;AAC3F,kHAAkH;AAClH,+CAA+C;AAC/C,gHAAgH;AAChH,yFAAyF;AACzF,qHAAqH;AACrH,oDAAoD;AACpD,EAAE;AACF,kDAAkD;AAClD,oGAAoG;AACpG,iHAAiH;AACjH,sEAAsE;AACtE,4GAA4G;AAC5G,qGAAqG;AACrG,MAAa,mBAAmB;IAI/B,YACkB,MAAmC,EACpD,oBAA4B,wBAAwB,EACnC,qBAA6B,yBAAyB;QAFtD,WAAM,GAAN,MAAM,CAA6B;QAEnC,uBAAkB,GAAlB,kBAAkB,CAAoC;QANhE,sBAAiB,GAAG,CAAC,CAAC;QAQ7B,IAAI,iBAAiB,KAAK,QAAQ,EAAE;YACnC,IAAI,CAAC,KAAK,GAAG,IAAI,oBAAK,CAAC,iBAAiB,EAAE,GAAG,EAAE;gBAC9C,2EAA2E;gBAC3E,mGAAmG;gBACnG,IAAI,IAAI,CAAC,iBAAiB,KAAK,CAAC,EAAE;oBACjC,IAAI,CAAC,UAAU,CAAC,KAAK,CAAC,eAAe,CAAC,CAAC;iBACvC;YACF,CAAC,CAAC,CAAC;SACH;IACF,CAAC;IAED;;OAEG;IACI,4BAA4B,CAClC,OAAkC,EAClC,aAAsB;QAEtB,mEAAmE;QACnE,sDAAsD;QACtD,IAAI,aAAa,EAAE;YAClB,IAAI,CAAC,UAAU,CAAC,IAAI,CAAC,eAAe,CAAC,CAAC;YACtC,OAAO;SACP;QAED,gFAAgF;QAChF,+DAA+D;QAC/D,sFAAsF;QACtF,yCAAyC;QACzC,IAAI,CAAC,IAAA,+BAAgB,EAAC,OAAO,CAAC,EAAE;YAC/B,OAAO;SACP;QAED,IAAI,CAAC,iBAAiB,EAAE,CAAC;QACzB,IAAI,IAAI,CAAC,iBAAiB,KAAK,IAAI,CAAC,kBAAkB,EAAE;YACvD,kEAAkE;YAClE,mEAAmE;YACnE,OAAO,CAAC,OAAO,EAAE,CAAC,IAAI,CAAC,GAAG,EAAE;gBAC3B,IAAI,IAAI,CAAC,iBAAiB,IAAI,IAAI,CAAC,kBAAkB,EAAE;oBACtD,IAAI,CAAC,UAAU,CAAC,KAAK,CAAC,eAAe,CAAC,CAAC;oBACvC,6CAA6C;oBAC7C,IAAI,CAAC,iBAAiB,GAAG,CAAC,CAAC;iBAC3B;gBACD,OAAO;YACR,CAAC,CAAC,CAAC;SACH;QAED,IAAI,IAAI,CAAC,KAAK,KAAK,SAAS,EAAE;YAC7B,IAAI,IAAI,CAAC,iBAAiB,KAAK,CAAC,EAAE;gBACjC,IAAI,CAAC,KAAK,CAAC,OAAO,EAAE,CAAC;aACrB;YAED,IAAA,qBAAM,EAAC,IAAI,CAAC,KAAK,CAAC,QAAQ,EAAE,KAAK,CAAC,iBAAiB,CAAC,CAAC;SACrD;IACF,CAAC;IAEO,UAAU,CAAC,SAAkB;QACpC,6CAA6C;QAC7C,8EAA8E;QAC9E,IAAI,CAAC,MAAM,CAAC,SAAS,CAAC,CAAC,CAAE,2BAAY,CAAC,MAAiC,CAAC,CAAC,CAAC,kCAAW,CAAC,IAAI,CAAC,CAAC;QAC5F,IAAA,qBAAM,EACL,IAAI,CAAC,iBAAiB,KAAK,CAAC,EAC5B,KAAK,CAAC,8EAA8E,CACpF,CAAC;IACH,CAAC;IAEM,wBAAwB;QAC9B,IAAI,CAAC,iBAAiB,GAAG,CAAC,CAAC;QAC3B,kGAAkG;QAClG,wGAAwG;QACxG,oDAAoD;QACpD,qGAAqG;QACrG,yFAAyF;QACzF,sBAAsB;IACvB,CAAC;CACD;AApFD,kDAoFC","sourcesContent":["/*!\n * Copyright (c) Microsoft Corporation and contributors. All rights reserved.\n * Licensed under the MIT License.\n */\n\nimport { assert, Timer } from \"@fluidframework/common-utils\";\nimport { ISequencedDocumentMessage, MessageType } from \"@fluidframework/protocol-definitions\";\nimport { isRuntimeMessage, MessageType2 } from \"@fluidframework/driver-utils\";\n\nconst defaultNoopTimeFrequency = 2000;\nconst defaultNoopCountFrequency = 50;\n\n// Here are key considerations when deciding conditions for when to send non-immediate noops:\n// 1. Sending them too often results in increase in file size and bandwidth, as well as catch up performance\n// 2. Sending too infrequently ensures that collab window is large, and as result Sequence DDS would have\n//    large catchUp blobs - see Issue #6364\n// 3. Similarly, processes that rely on \"core\" snapshot (and can't parse trailing ops, including above), like search\n//    parser in SPO, will result in non-accurate results due to presence of catch up blobs.\n// 4. Ordering service used 250ms timeout to coalesce non-immediate noops. It was changed to 2000 ms to allow more\n//    aggressive noop sending from client side.\n// 5. Number of ops sent by all clients is proportional to number of \"write\" clients (every client sends noops),\n//    but number of sequenced noops is a function of time (one op per 2 seconds at most).\n//    We should consider impact to both outbound traffic (might be huge, depends on number of clients) and file size.\n// Please also see Issue #5629 for more discussions.\n//\n// With that, the current algorithm is as follows:\n// 1. Sent noop 2000 ms of receiving an op if no ops were sent by this client within this timeframe.\n//    This will ensure that MSN moves forward with reasonable speed. If that results in too many sequenced noops,\n//    server timeout of 2000ms should be reconsidered to be increased.\n// 2. If there are more than 50 ops received without sending any ops, send noop to keep collab window small.\n//    Note that system ops (including noops themselves) are excluded, so it's 1 noop per 50 real ops.\nexport class CollabWindowTracker {\n\tprivate opsCountSinceNoop = 0;\n\tprivate readonly timer: Timer | undefined;\n\n\tconstructor(\n\t\tprivate readonly submit: (type: MessageType) => void,\n\t\tNoopTimeFrequency: number = defaultNoopTimeFrequency,\n\t\tprivate readonly NoopCountFrequency: number = defaultNoopCountFrequency,\n\t) {\n\t\tif (NoopTimeFrequency !== Infinity) {\n\t\t\tthis.timer = new Timer(NoopTimeFrequency, () => {\n\t\t\t\t// Can get here due to this.stopSequenceNumberUpdate() not resetting timer.\n\t\t\t\t// Also timer callback can fire even after timer cancellation if it was queued before cancellation.\n\t\t\t\tif (this.opsCountSinceNoop !== 0) {\n\t\t\t\t\tthis.submitNoop(false /* immediate */);\n\t\t\t\t}\n\t\t\t});\n\t\t}\n\t}\n\n\t/**\n\t * Schedules as ack to the server to update the reference sequence number\n\t */\n\tpublic scheduleSequenceNumberUpdate(\n\t\tmessage: ISequencedDocumentMessage,\n\t\timmediateNoOp: boolean,\n\t): void {\n\t\t// While processing a message, an immediate no-op can be requested.\n\t\t// i.e. to expedite approve or commit phase of quorum.\n\t\tif (immediateNoOp) {\n\t\t\tthis.submitNoop(true /* immediate */);\n\t\t\treturn;\n\t\t}\n\n\t\t// We don't acknowledge no-ops to avoid acknowledgement cycles (i.e. ack the MSN\n\t\t// update, which updates the MSN, then ack the update, etc...).\n\t\t// Intent here is for runtime (and DDSes) not to keep too much tracking state / memory\n\t\t// due to runtime ops from other clients.\n\t\tif (!isRuntimeMessage(message)) {\n\t\t\treturn;\n\t\t}\n\n\t\tthis.opsCountSinceNoop++;\n\t\tif (this.opsCountSinceNoop === this.NoopCountFrequency) {\n\t\t\t// Ensure we only send noop after a batch of many ops is processed\n\t\t\t// eslint-disable-next-line @typescript-eslint/no-floating-promises\n\t\t\tPromise.resolve().then(() => {\n\t\t\t\tif (this.opsCountSinceNoop >= this.NoopCountFrequency) {\n\t\t\t\t\tthis.submitNoop(false /* immediate */);\n\t\t\t\t\t// reset count now that all ops are processed\n\t\t\t\t\tthis.opsCountSinceNoop = 0;\n\t\t\t\t}\n\t\t\t\treturn;\n\t\t\t});\n\t\t}\n\n\t\tif (this.timer !== undefined) {\n\t\t\tif (this.opsCountSinceNoop === 1) {\n\t\t\t\tthis.timer.restart();\n\t\t\t}\n\n\t\t\tassert(this.timer.hasTimer, 0x242 /* \"has timer\" */);\n\t\t}\n\t}\n\n\tprivate submitNoop(immediate: boolean) {\n\t\t// Anything other than null is immediate noop\n\t\t// ADO:1385: Remove cast and use MessageType once definition changes propagate\n\t\tthis.submit(immediate ? (MessageType2.Accept as unknown as MessageType) : MessageType.NoOp);\n\t\tassert(\n\t\t\tthis.opsCountSinceNoop === 0,\n\t\t\t0x243 /* \"stopSequenceNumberUpdate should be called as result of sending any op!\" */,\n\t\t);\n\t}\n\n\tpublic stopSequenceNumberUpdate(): void {\n\t\tthis.opsCountSinceNoop = 0;\n\t\t// Ideally, we cancel timer here. But that will result in too often set/reset cycle if this client\n\t\t// keeps sending ops. In most cases it's actually better to let it expire (at most - 4 times per second)\n\t\t// for nothing, then have a ton of set/reset cycles.\n\t\t// Note that Timer.restart() is smart and will not change timer expiration if we keep extending timer\n\t\t// expiration - it will restart the timer instead when it fires with adjusted expiration.\n\t\t// this.timer.clear();\n\t}\n}\n"]}

package/dist/deltaManagerProxy.d.ts

@@ -1,42 +0,0 @@
-/*!
- * Copyright (c) Microsoft Corporation and contributors. All rights reserved.
- * Licensed under the MIT License.
- */
-import { IDeltaManager, IDeltaQueue, IDeltaQueueEvents } from "@fluidframework/container-definitions";
-import { EventForwarder } from "@fluidframework/common-utils";
-import { IDocumentMessage, ISequencedDocumentMessage, ISignalMessage } from "@fluidframework/protocol-definitions";
-import { DeltaManagerProxyBase } from "@fluidframework/container-utils";
-/**
- * Proxy to the real IDeltaQueue - used to restrict access
- */
-export declare class DeltaQueueProxy<T> extends EventForwarder<IDeltaQueueEvents<T>> implements IDeltaQueue<T> {
-    private readonly queue;
-    get paused(): boolean;
-    get length(): number;
-    get idle(): boolean;
-    constructor(queue: IDeltaQueue<T>);
-    peek(): T | undefined;
-    toArray(): T[];
-    systemPause(): Promise<void>;
-    pause(): Promise<void>;
-    systemResume(): Promise<void>;
-    resume(): Promise<void>;
-    waitTillProcessingDone(): Promise<{
-        count: number;
-        duration: number;
-    }>;
-}
-/**
- * Proxy to the real IDeltaManager - used to restrict access
- */
-export declare class DeltaManagerProxy extends DeltaManagerProxyBase implements IDeltaManager<ISequencedDocumentMessage, IDocumentMessage> {
-    get inbound(): IDeltaQueue<ISequencedDocumentMessage>;
-    private readonly _inbound;
-    get outbound(): IDeltaQueue<IDocumentMessage[]>;
-    private readonly _outbound;
-    get inboundSignal(): IDeltaQueue<ISignalMessage>;
-    private readonly _inboundSignal;
-    constructor(deltaManager: IDeltaManager<ISequencedDocumentMessage, IDocumentMessage>);
-    dispose(): void;
-}
-//# sourceMappingURL=deltaManagerProxy.d.ts.map

package/dist/deltaManagerProxy.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"deltaManagerProxy.d.ts","sourceRoot":"","sources":["../src/deltaManagerProxy.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAEH,OAAO,EACN,aAAa,EACb,WAAW,EACX,iBAAiB,EACjB,MAAM,uCAAuC,CAAC;AAC/C,OAAO,EAAE,cAAc,EAAE,MAAM,8BAA8B,CAAC;AAC9D,OAAO,EACN,gBAAgB,EAChB,yBAAyB,EACzB,cAAc,EACd,MAAM,sCAAsC,CAAC;AAC9C,OAAO,EAAE,qBAAqB,EAAE,MAAM,iCAAiC,CAAC;AAExE;;GAEG;AACH,qBAAa,eAAe,CAAC,CAAC,CAC7B,SAAQ,cAAc,CAAC,iBAAiB,CAAC,CAAC,CAAC,CAC3C,YAAW,WAAW,CAAC,CAAC,CAAC;IAcb,OAAO,CAAC,QAAQ,CAAC,KAAK;IAZlC,IAAW,MAAM,IAAI,OAAO,CAE3B;IAED,IAAW,MAAM,IAAI,MAAM,CAE1B;IAED,IAAW,IAAI,IAAI,OAAO,CAEzB;gBAE4B,KAAK,EAAE,WAAW,CAAC,CAAC,CAAC;IAI3C,IAAI,IAAI,CAAC,GAAG,SAAS;IAIrB,OAAO,IAAI,CAAC,EAAE;IAKR,WAAW,IAAI,OAAO,CAAC,IAAI,CAAC;IAI5B,KAAK,IAAI,OAAO,CAAC,IAAI,CAAC;IAKtB,YAAY,IAAI,OAAO,CAAC,IAAI,CAAC;IAI7B,MAAM,IAAI,OAAO,CAAC,IAAI,CAAC;IAIvB,sBAAsB;;;;CAGnC;AAED;;GAEG;AACH,qBAAa,iBACZ,SAAQ,qBACR,YAAW,aAAa,CAAC,yBAAyB,EAAE,gBAAgB,CAAC;IAErE,IAAW,OAAO,IAAI,WAAW,CAAC,yBAAyB,CAAC,CAE3D;IACD,OAAO,CAAC,QAAQ,CAAC,QAAQ,CAAyC;IAElE,IAAW,QAAQ,IAAI,WAAW,CAAC,gBAAgB,EAAE,CAAC,CAErD;IACD,OAAO,CAAC,QAAQ,CAAC,SAAS,CAAkC;IAE5D,IAAW,aAAa,IAAI,WAAW,CAAC,cAAc,CAAC,CAEtD;IACD,OAAO,CAAC,QAAQ,CAAC,cAAc,CAA8B;gBAEjD,YAAY,EAAE,aAAa,CAAC,yBAAyB,EAAE,gBAAgB,CAAC;IAQ7E,OAAO,IAAI,IAAI;CAMtB"}

package/dist/deltaManagerProxy.js

@@ -1,79 +0,0 @@
-"use strict";
-/*!
- * Copyright (c) Microsoft Corporation and contributors. All rights reserved.
- * Licensed under the MIT License.
- */
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.DeltaManagerProxy = exports.DeltaQueueProxy = void 0;
-const common_utils_1 = require("@fluidframework/common-utils");
-const container_utils_1 = require("@fluidframework/container-utils");
-/**
- * Proxy to the real IDeltaQueue - used to restrict access
- */
-class DeltaQueueProxy extends common_utils_1.EventForwarder {
-    constructor(queue) {
-        super(queue);
-        this.queue = queue;
-    }
-    get paused() {
-        return this.queue.paused;
-    }
-    get length() {
-        return this.queue.length;
-    }
-    get idle() {
-        return this.queue.idle;
-    }
-    peek() {
-        return this.queue.peek();
-    }
-    toArray() {
-        return this.queue.toArray();
-    }
-    // back-compat: usage removed in 0.33, remove in future versions
-    async systemPause() {
-        return this.pause();
-    }
-    async pause() {
-        return this.queue.pause();
-    }
-    // back-compat: usage removed in 0.33, remove in future versions
-    async systemResume() {
-        return this.resume();
-    }
-    async resume() {
-        this.queue.resume();
-    }
-    async waitTillProcessingDone() {
-        return this.queue.waitTillProcessingDone();
-    }
-}
-exports.DeltaQueueProxy = DeltaQueueProxy;
-/**
- * Proxy to the real IDeltaManager - used to restrict access
- */
-class DeltaManagerProxy extends container_utils_1.DeltaManagerProxyBase {
-    constructor(deltaManager) {
-        super(deltaManager);
-        this._inbound = new DeltaQueueProxy(deltaManager.inbound);
-        this._outbound = new DeltaQueueProxy(deltaManager.outbound);
-        this._inboundSignal = new DeltaQueueProxy(deltaManager.inboundSignal);
-    }
-    get inbound() {
-        return this._inbound;
-    }
-    get outbound() {
-        return this._outbound;
-    }
-    get inboundSignal() {
-        return this._inboundSignal;
-    }
-    dispose() {
-        this._inbound.dispose();
-        this._outbound.dispose();
-        this._inboundSignal.dispose();
-        super.dispose();
-    }
-}
-exports.DeltaManagerProxy = DeltaManagerProxy;
-//# sourceMappingURL=deltaManagerProxy.js.map

package/dist/deltaManagerProxy.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"deltaManagerProxy.js","sourceRoot":"","sources":["../src/deltaManagerProxy.ts"],"names":[],"mappings":";AAAA;;;GAGG;;;AAOH,+DAA8D;AAM9D,qEAAwE;AAExE;;GAEG;AACH,MAAa,eACZ,SAAQ,6BAAoC;IAe5C,YAA6B,KAAqB;QACjD,KAAK,CAAC,KAAK,CAAC,CAAC;QADe,UAAK,GAAL,KAAK,CAAgB;IAElD,CAAC;IAdD,IAAW,MAAM;QAChB,OAAO,IAAI,CAAC,KAAK,CAAC,MAAM,CAAC;IAC1B,CAAC;IAED,IAAW,MAAM;QAChB,OAAO,IAAI,CAAC,KAAK,CAAC,MAAM,CAAC;IAC1B,CAAC;IAED,IAAW,IAAI;QACd,OAAO,IAAI,CAAC,KAAK,CAAC,IAAI,CAAC;IACxB,CAAC;IAMM,IAAI;QACV,OAAO,IAAI,CAAC,KAAK,CAAC,IAAI,EAAE,CAAC;IAC1B,CAAC;IAEM,OAAO;QACb,OAAO,IAAI,CAAC,KAAK,CAAC,OAAO,EAAE,CAAC;IAC7B,CAAC;IAED,gEAAgE;IACzD,KAAK,CAAC,WAAW;QACvB,OAAO,IAAI,CAAC,KAAK,EAAE,CAAC;IACrB,CAAC;IAEM,KAAK,CAAC,KAAK;QACjB,OAAO,IAAI,CAAC,KAAK,CAAC,KAAK,EAAE,CAAC;IAC3B,CAAC;IAED,gEAAgE;IACzD,KAAK,CAAC,YAAY;QACxB,OAAO,IAAI,CAAC,MAAM,EAAE,CAAC;IACtB,CAAC;IAEM,KAAK,CAAC,MAAM;QAClB,IAAI,CAAC,KAAK,CAAC,MAAM,EAAE,CAAC;IACrB,CAAC;IAEM,KAAK,CAAC,sBAAsB;QAClC,OAAO,IAAI,CAAC,KAAK,CAAC,sBAAsB,EAAE,CAAC;IAC5C,CAAC;CACD;AAjDD,0CAiDC;AAED;;GAEG;AACH,MAAa,iBACZ,SAAQ,uCAAqB;IAkB7B,YAAY,YAAwE;QACnF,KAAK,CAAC,YAAY,CAAC,CAAC;QAEpB,IAAI,CAAC,QAAQ,GAAG,IAAI,eAAe,CAAC,YAAY,CAAC,OAAO,CAAC,CAAC;QAC1D,IAAI,CAAC,SAAS,GAAG,IAAI,eAAe,CAAC,YAAY,CAAC,QAAQ,CAAC,CAAC;QAC5D,IAAI,CAAC,cAAc,GAAG,IAAI,eAAe,CAAC,YAAY,CAAC,aAAa,CAAC,CAAC;IACvE,CAAC;IArBD,IAAW,OAAO;QACjB,OAAO,IAAI,CAAC,QAAQ,CAAC;IACtB,CAAC;IAGD,IAAW,QAAQ;QAClB,OAAO,IAAI,CAAC,SAAS,CAAC;IACvB,CAAC;IAGD,IAAW,aAAa;QACvB,OAAO,IAAI,CAAC,cAAc,CAAC;IAC5B,CAAC;IAWM,OAAO;QACb,IAAI,CAAC,QAAQ,CAAC,OAAO,EAAE,CAAC;QACxB,IAAI,CAAC,SAAS,CAAC,OAAO,EAAE,CAAC;QACzB,IAAI,CAAC,cAAc,CAAC,OAAO,EAAE,CAAC;QAC9B,KAAK,CAAC,OAAO,EAAE,CAAC;IACjB,CAAC;CACD;AAjCD,8CAiCC","sourcesContent":["/*!\n * Copyright (c) Microsoft Corporation and contributors. All rights reserved.\n * Licensed under the MIT License.\n */\n\nimport {\n\tIDeltaManager,\n\tIDeltaQueue,\n\tIDeltaQueueEvents,\n} from \"@fluidframework/container-definitions\";\nimport { EventForwarder } from \"@fluidframework/common-utils\";\nimport {\n\tIDocumentMessage,\n\tISequencedDocumentMessage,\n\tISignalMessage,\n} from \"@fluidframework/protocol-definitions\";\nimport { DeltaManagerProxyBase } from \"@fluidframework/container-utils\";\n\n/**\n * Proxy to the real IDeltaQueue - used to restrict access\n */\nexport class DeltaQueueProxy<T>\n\textends EventForwarder<IDeltaQueueEvents<T>>\n\timplements IDeltaQueue<T>\n{\n\tpublic get paused(): boolean {\n\t\treturn this.queue.paused;\n\t}\n\n\tpublic get length(): number {\n\t\treturn this.queue.length;\n\t}\n\n\tpublic get idle(): boolean {\n\t\treturn this.queue.idle;\n\t}\n\n\tconstructor(private readonly queue: IDeltaQueue<T>) {\n\t\tsuper(queue);\n\t}\n\n\tpublic peek(): T | undefined {\n\t\treturn this.queue.peek();\n\t}\n\n\tpublic toArray(): T[] {\n\t\treturn this.queue.toArray();\n\t}\n\n\t// back-compat: usage removed in 0.33, remove in future versions\n\tpublic async systemPause(): Promise<void> {\n\t\treturn this.pause();\n\t}\n\n\tpublic async pause(): Promise<void> {\n\t\treturn this.queue.pause();\n\t}\n\n\t// back-compat: usage removed in 0.33, remove in future versions\n\tpublic async systemResume(): Promise<void> {\n\t\treturn this.resume();\n\t}\n\n\tpublic async resume(): Promise<void> {\n\t\tthis.queue.resume();\n\t}\n\n\tpublic async waitTillProcessingDone() {\n\t\treturn this.queue.waitTillProcessingDone();\n\t}\n}\n\n/**\n * Proxy to the real IDeltaManager - used to restrict access\n */\nexport class DeltaManagerProxy\n\textends DeltaManagerProxyBase\n\timplements IDeltaManager<ISequencedDocumentMessage, IDocumentMessage>\n{\n\tpublic get inbound(): IDeltaQueue<ISequencedDocumentMessage> {\n\t\treturn this._inbound;\n\t}\n\tprivate readonly _inbound: IDeltaQueue<ISequencedDocumentMessage>;\n\n\tpublic get outbound(): IDeltaQueue<IDocumentMessage[]> {\n\t\treturn this._outbound;\n\t}\n\tprivate readonly _outbound: IDeltaQueue<IDocumentMessage[]>;\n\n\tpublic get inboundSignal(): IDeltaQueue<ISignalMessage> {\n\t\treturn this._inboundSignal;\n\t}\n\tprivate readonly _inboundSignal: IDeltaQueue<ISignalMessage>;\n\n\tconstructor(deltaManager: IDeltaManager<ISequencedDocumentMessage, IDocumentMessage>) {\n\t\tsuper(deltaManager);\n\n\t\tthis._inbound = new DeltaQueueProxy(deltaManager.inbound);\n\t\tthis._outbound = new DeltaQueueProxy(deltaManager.outbound);\n\t\tthis._inboundSignal = new DeltaQueueProxy(deltaManager.inboundSignal);\n\t}\n\n\tpublic dispose(): void {\n\t\tthis._inbound.dispose();\n\t\tthis._outbound.dispose();\n\t\tthis._inboundSignal.dispose();\n\t\tsuper.dispose();\n\t}\n}\n"]}

package/lib/collabWindowTracker.d.ts

@@ -1,19 +0,0 @@
-/*!
- * Copyright (c) Microsoft Corporation and contributors. All rights reserved.
- * Licensed under the MIT License.
- */
-import { ISequencedDocumentMessage, MessageType } from "@fluidframework/protocol-definitions";
-export declare class CollabWindowTracker {
-    private readonly submit;
-    private readonly NoopCountFrequency;
-    private opsCountSinceNoop;
-    private readonly timer;
-    constructor(submit: (type: MessageType) => void, NoopTimeFrequency?: number, NoopCountFrequency?: number);
-    /**
-     * Schedules as ack to the server to update the reference sequence number
-     */
-    scheduleSequenceNumberUpdate(message: ISequencedDocumentMessage, immediateNoOp: boolean): void;
-    private submitNoop;
-    stopSequenceNumberUpdate(): void;
-}
-//# sourceMappingURL=collabWindowTracker.d.ts.map

package/lib/collabWindowTracker.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"collabWindowTracker.d.ts","sourceRoot":"","sources":["../src/collabWindowTracker.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAGH,OAAO,EAAE,yBAAyB,EAAE,WAAW,EAAE,MAAM,sCAAsC,CAAC;AAyB9F,qBAAa,mBAAmB;IAK9B,OAAO,CAAC,QAAQ,CAAC,MAAM;IAEvB,OAAO,CAAC,QAAQ,CAAC,kBAAkB;IANpC,OAAO,CAAC,iBAAiB,CAAK;IAC9B,OAAO,CAAC,QAAQ,CAAC,KAAK,CAAoB;gBAGxB,MAAM,EAAE,CAAC,IAAI,EAAE,WAAW,KAAK,IAAI,EACpD,iBAAiB,GAAE,MAAiC,EACnC,kBAAkB,GAAE,MAAkC;IAaxE;;OAEG;IACI,4BAA4B,CAClC,OAAO,EAAE,yBAAyB,EAClC,aAAa,EAAE,OAAO,GACpB,IAAI;IAuCP,OAAO,CAAC,UAAU;IAUX,wBAAwB,IAAI,IAAI;CASvC"}

package/lib/collabWindowTracker.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"collabWindowTracker.js","sourceRoot":"","sources":["../src/collabWindowTracker.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAEH,OAAO,EAAE,MAAM,EAAE,KAAK,EAAE,MAAM,8BAA8B,CAAC;AAC7D,OAAO,EAA6B,WAAW,EAAE,MAAM,sCAAsC,CAAC;AAC9F,OAAO,EAAE,gBAAgB,EAAE,YAAY,EAAE,MAAM,8BAA8B,CAAC;AAE9E,MAAM,wBAAwB,GAAG,IAAI,CAAC;AACtC,MAAM,yBAAyB,GAAG,EAAE,CAAC;AAErC,6FAA6F;AAC7F,4GAA4G;AAC5G,yGAAyG;AACzG,2CAA2C;AAC3C,oHAAoH;AACpH,2FAA2F;AAC3F,kHAAkH;AAClH,+CAA+C;AAC/C,gHAAgH;AAChH,yFAAyF;AACzF,qHAAqH;AACrH,oDAAoD;AACpD,EAAE;AACF,kDAAkD;AAClD,oGAAoG;AACpG,iHAAiH;AACjH,sEAAsE;AACtE,4GAA4G;AAC5G,qGAAqG;AACrG,MAAM,OAAO,mBAAmB;IAI/B,YACkB,MAAmC,EACpD,oBAA4B,wBAAwB,EACnC,qBAA6B,yBAAyB;QAFtD,WAAM,GAAN,MAAM,CAA6B;QAEnC,uBAAkB,GAAlB,kBAAkB,CAAoC;QANhE,sBAAiB,GAAG,CAAC,CAAC;QAQ7B,IAAI,iBAAiB,KAAK,QAAQ,EAAE;YACnC,IAAI,CAAC,KAAK,GAAG,IAAI,KAAK,CAAC,iBAAiB,EAAE,GAAG,EAAE;gBAC9C,2EAA2E;gBAC3E,mGAAmG;gBACnG,IAAI,IAAI,CAAC,iBAAiB,KAAK,CAAC,EAAE;oBACjC,IAAI,CAAC,UAAU,CAAC,KAAK,CAAC,eAAe,CAAC,CAAC;iBACvC;YACF,CAAC,CAAC,CAAC;SACH;IACF,CAAC;IAED;;OAEG;IACI,4BAA4B,CAClC,OAAkC,EAClC,aAAsB;QAEtB,mEAAmE;QACnE,sDAAsD;QACtD,IAAI,aAAa,EAAE;YAClB,IAAI,CAAC,UAAU,CAAC,IAAI,CAAC,eAAe,CAAC,CAAC;YACtC,OAAO;SACP;QAED,gFAAgF;QAChF,+DAA+D;QAC/D,sFAAsF;QACtF,yCAAyC;QACzC,IAAI,CAAC,gBAAgB,CAAC,OAAO,CAAC,EAAE;YAC/B,OAAO;SACP;QAED,IAAI,CAAC,iBAAiB,EAAE,CAAC;QACzB,IAAI,IAAI,CAAC,iBAAiB,KAAK,IAAI,CAAC,kBAAkB,EAAE;YACvD,kEAAkE;YAClE,mEAAmE;YACnE,OAAO,CAAC,OAAO,EAAE,CAAC,IAAI,CAAC,GAAG,EAAE;gBAC3B,IAAI,IAAI,CAAC,iBAAiB,IAAI,IAAI,CAAC,kBAAkB,EAAE;oBACtD,IAAI,CAAC,UAAU,CAAC,KAAK,CAAC,eAAe,CAAC,CAAC;oBACvC,6CAA6C;oBAC7C,IAAI,CAAC,iBAAiB,GAAG,CAAC,CAAC;iBAC3B;gBACD,OAAO;YACR,CAAC,CAAC,CAAC;SACH;QAED,IAAI,IAAI,CAAC,KAAK,KAAK,SAAS,EAAE;YAC7B,IAAI,IAAI,CAAC,iBAAiB,KAAK,CAAC,EAAE;gBACjC,IAAI,CAAC,KAAK,CAAC,OAAO,EAAE,CAAC;aACrB;YAED,MAAM,CAAC,IAAI,CAAC,KAAK,CAAC,QAAQ,EAAE,KAAK,CAAC,iBAAiB,CAAC,CAAC;SACrD;IACF,CAAC;IAEO,UAAU,CAAC,SAAkB;QACpC,6CAA6C;QAC7C,8EAA8E;QAC9E,IAAI,CAAC,MAAM,CAAC,SAAS,CAAC,CAAC,CAAE,YAAY,CAAC,MAAiC,CAAC,CAAC,CAAC,WAAW,CAAC,IAAI,CAAC,CAAC;QAC5F,MAAM,CACL,IAAI,CAAC,iBAAiB,KAAK,CAAC,EAC5B,KAAK,CAAC,8EAA8E,CACpF,CAAC;IACH,CAAC;IAEM,wBAAwB;QAC9B,IAAI,CAAC,iBAAiB,GAAG,CAAC,CAAC;QAC3B,kGAAkG;QAClG,wGAAwG;QACxG,oDAAoD;QACpD,qGAAqG;QACrG,yFAAyF;QACzF,sBAAsB;IACvB,CAAC;CACD","sourcesContent":["/*!\n * Copyright (c) Microsoft Corporation and contributors. All rights reserved.\n * Licensed under the MIT License.\n */\n\nimport { assert, Timer } from \"@fluidframework/common-utils\";\nimport { ISequencedDocumentMessage, MessageType } from \"@fluidframework/protocol-definitions\";\nimport { isRuntimeMessage, MessageType2 } from \"@fluidframework/driver-utils\";\n\nconst defaultNoopTimeFrequency = 2000;\nconst defaultNoopCountFrequency = 50;\n\n// Here are key considerations when deciding conditions for when to send non-immediate noops:\n// 1. Sending them too often results in increase in file size and bandwidth, as well as catch up performance\n// 2. Sending too infrequently ensures that collab window is large, and as result Sequence DDS would have\n//    large catchUp blobs - see Issue #6364\n// 3. Similarly, processes that rely on \"core\" snapshot (and can't parse trailing ops, including above), like search\n//    parser in SPO, will result in non-accurate results due to presence of catch up blobs.\n// 4. Ordering service used 250ms timeout to coalesce non-immediate noops. It was changed to 2000 ms to allow more\n//    aggressive noop sending from client side.\n// 5. Number of ops sent by all clients is proportional to number of \"write\" clients (every client sends noops),\n//    but number of sequenced noops is a function of time (one op per 2 seconds at most).\n//    We should consider impact to both outbound traffic (might be huge, depends on number of clients) and file size.\n// Please also see Issue #5629 for more discussions.\n//\n// With that, the current algorithm is as follows:\n// 1. Sent noop 2000 ms of receiving an op if no ops were sent by this client within this timeframe.\n//    This will ensure that MSN moves forward with reasonable speed. If that results in too many sequenced noops,\n//    server timeout of 2000ms should be reconsidered to be increased.\n// 2. If there are more than 50 ops received without sending any ops, send noop to keep collab window small.\n//    Note that system ops (including noops themselves) are excluded, so it's 1 noop per 50 real ops.\nexport class CollabWindowTracker {\n\tprivate opsCountSinceNoop = 0;\n\tprivate readonly timer: Timer | undefined;\n\n\tconstructor(\n\t\tprivate readonly submit: (type: MessageType) => void,\n\t\tNoopTimeFrequency: number = defaultNoopTimeFrequency,\n\t\tprivate readonly NoopCountFrequency: number = defaultNoopCountFrequency,\n\t) {\n\t\tif (NoopTimeFrequency !== Infinity) {\n\t\t\tthis.timer = new Timer(NoopTimeFrequency, () => {\n\t\t\t\t// Can get here due to this.stopSequenceNumberUpdate() not resetting timer.\n\t\t\t\t// Also timer callback can fire even after timer cancellation if it was queued before cancellation.\n\t\t\t\tif (this.opsCountSinceNoop !== 0) {\n\t\t\t\t\tthis.submitNoop(false /* immediate */);\n\t\t\t\t}\n\t\t\t});\n\t\t}\n\t}\n\n\t/**\n\t * Schedules as ack to the server to update the reference sequence number\n\t */\n\tpublic scheduleSequenceNumberUpdate(\n\t\tmessage: ISequencedDocumentMessage,\n\t\timmediateNoOp: boolean,\n\t): void {\n\t\t// While processing a message, an immediate no-op can be requested.\n\t\t// i.e. to expedite approve or commit phase of quorum.\n\t\tif (immediateNoOp) {\n\t\t\tthis.submitNoop(true /* immediate */);\n\t\t\treturn;\n\t\t}\n\n\t\t// We don't acknowledge no-ops to avoid acknowledgement cycles (i.e. ack the MSN\n\t\t// update, which updates the MSN, then ack the update, etc...).\n\t\t// Intent here is for runtime (and DDSes) not to keep too much tracking state / memory\n\t\t// due to runtime ops from other clients.\n\t\tif (!isRuntimeMessage(message)) {\n\t\t\treturn;\n\t\t}\n\n\t\tthis.opsCountSinceNoop++;\n\t\tif (this.opsCountSinceNoop === this.NoopCountFrequency) {\n\t\t\t// Ensure we only send noop after a batch of many ops is processed\n\t\t\t// eslint-disable-next-line @typescript-eslint/no-floating-promises\n\t\t\tPromise.resolve().then(() => {\n\t\t\t\tif (this.opsCountSinceNoop >= this.NoopCountFrequency) {\n\t\t\t\t\tthis.submitNoop(false /* immediate */);\n\t\t\t\t\t// reset count now that all ops are processed\n\t\t\t\t\tthis.opsCountSinceNoop = 0;\n\t\t\t\t}\n\t\t\t\treturn;\n\t\t\t});\n\t\t}\n\n\t\tif (this.timer !== undefined) {\n\t\t\tif (this.opsCountSinceNoop === 1) {\n\t\t\t\tthis.timer.restart();\n\t\t\t}\n\n\t\t\tassert(this.timer.hasTimer, 0x242 /* \"has timer\" */);\n\t\t}\n\t}\n\n\tprivate submitNoop(immediate: boolean) {\n\t\t// Anything other than null is immediate noop\n\t\t// ADO:1385: Remove cast and use MessageType once definition changes propagate\n\t\tthis.submit(immediate ? (MessageType2.Accept as unknown as MessageType) : MessageType.NoOp);\n\t\tassert(\n\t\t\tthis.opsCountSinceNoop === 0,\n\t\t\t0x243 /* \"stopSequenceNumberUpdate should be called as result of sending any op!\" */,\n\t\t);\n\t}\n\n\tpublic stopSequenceNumberUpdate(): void {\n\t\tthis.opsCountSinceNoop = 0;\n\t\t// Ideally, we cancel timer here. But that will result in too often set/reset cycle if this client\n\t\t// keeps sending ops. In most cases it's actually better to let it expire (at most - 4 times per second)\n\t\t// for nothing, then have a ton of set/reset cycles.\n\t\t// Note that Timer.restart() is smart and will not change timer expiration if we keep extending timer\n\t\t// expiration - it will restart the timer instead when it fires with adjusted expiration.\n\t\t// this.timer.clear();\n\t}\n}\n"]}

package/lib/deltaManagerProxy.d.ts

@@ -1,42 +0,0 @@
-/*!
- * Copyright (c) Microsoft Corporation and contributors. All rights reserved.
- * Licensed under the MIT License.
- */
-import { IDeltaManager, IDeltaQueue, IDeltaQueueEvents } from "@fluidframework/container-definitions";
-import { EventForwarder } from "@fluidframework/common-utils";
-import { IDocumentMessage, ISequencedDocumentMessage, ISignalMessage } from "@fluidframework/protocol-definitions";
-import { DeltaManagerProxyBase } from "@fluidframework/container-utils";
-/**
- * Proxy to the real IDeltaQueue - used to restrict access
- */
-export declare class DeltaQueueProxy<T> extends EventForwarder<IDeltaQueueEvents<T>> implements IDeltaQueue<T> {
-    private readonly queue;
-    get paused(): boolean;
-    get length(): number;
-    get idle(): boolean;
-    constructor(queue: IDeltaQueue<T>);
-    peek(): T | undefined;
-    toArray(): T[];
-    systemPause(): Promise<void>;
-    pause(): Promise<void>;
-    systemResume(): Promise<void>;
-    resume(): Promise<void>;
-    waitTillProcessingDone(): Promise<{
-        count: number;
-        duration: number;
-    }>;
-}
-/**
- * Proxy to the real IDeltaManager - used to restrict access
- */
-export declare class DeltaManagerProxy extends DeltaManagerProxyBase implements IDeltaManager<ISequencedDocumentMessage, IDocumentMessage> {
-    get inbound(): IDeltaQueue<ISequencedDocumentMessage>;
-    private readonly _inbound;
-    get outbound(): IDeltaQueue<IDocumentMessage[]>;
-    private readonly _outbound;
-    get inboundSignal(): IDeltaQueue<ISignalMessage>;
-    private readonly _inboundSignal;
-    constructor(deltaManager: IDeltaManager<ISequencedDocumentMessage, IDocumentMessage>);
-    dispose(): void;
-}
-//# sourceMappingURL=deltaManagerProxy.d.ts.map

package/lib/deltaManagerProxy.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"deltaManagerProxy.d.ts","sourceRoot":"","sources":["../src/deltaManagerProxy.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAEH,OAAO,EACN,aAAa,EACb,WAAW,EACX,iBAAiB,EACjB,MAAM,uCAAuC,CAAC;AAC/C,OAAO,EAAE,cAAc,EAAE,MAAM,8BAA8B,CAAC;AAC9D,OAAO,EACN,gBAAgB,EAChB,yBAAyB,EACzB,cAAc,EACd,MAAM,sCAAsC,CAAC;AAC9C,OAAO,EAAE,qBAAqB,EAAE,MAAM,iCAAiC,CAAC;AAExE;;GAEG;AACH,qBAAa,eAAe,CAAC,CAAC,CAC7B,SAAQ,cAAc,CAAC,iBAAiB,CAAC,CAAC,CAAC,CAC3C,YAAW,WAAW,CAAC,CAAC,CAAC;IAcb,OAAO,CAAC,QAAQ,CAAC,KAAK;IAZlC,IAAW,MAAM,IAAI,OAAO,CAE3B;IAED,IAAW,MAAM,IAAI,MAAM,CAE1B;IAED,IAAW,IAAI,IAAI,OAAO,CAEzB;gBAE4B,KAAK,EAAE,WAAW,CAAC,CAAC,CAAC;IAI3C,IAAI,IAAI,CAAC,GAAG,SAAS;IAIrB,OAAO,IAAI,CAAC,EAAE;IAKR,WAAW,IAAI,OAAO,CAAC,IAAI,CAAC;IAI5B,KAAK,IAAI,OAAO,CAAC,IAAI,CAAC;IAKtB,YAAY,IAAI,OAAO,CAAC,IAAI,CAAC;IAI7B,MAAM,IAAI,OAAO,CAAC,IAAI,CAAC;IAIvB,sBAAsB;;;;CAGnC;AAED;;GAEG;AACH,qBAAa,iBACZ,SAAQ,qBACR,YAAW,aAAa,CAAC,yBAAyB,EAAE,gBAAgB,CAAC;IAErE,IAAW,OAAO,IAAI,WAAW,CAAC,yBAAyB,CAAC,CAE3D;IACD,OAAO,CAAC,QAAQ,CAAC,QAAQ,CAAyC;IAElE,IAAW,QAAQ,IAAI,WAAW,CAAC,gBAAgB,EAAE,CAAC,CAErD;IACD,OAAO,CAAC,QAAQ,CAAC,SAAS,CAAkC;IAE5D,IAAW,aAAa,IAAI,WAAW,CAAC,cAAc,CAAC,CAEtD;IACD,OAAO,CAAC,QAAQ,CAAC,cAAc,CAA8B;gBAEjD,YAAY,EAAE,aAAa,CAAC,yBAAyB,EAAE,gBAAgB,CAAC;IAQ7E,OAAO,IAAI,IAAI;CAMtB"}

package/lib/deltaManagerProxy.js

@@ -1,74 +0,0 @@
-/*!
- * Copyright (c) Microsoft Corporation and contributors. All rights reserved.
- * Licensed under the MIT License.
- */
-import { EventForwarder } from "@fluidframework/common-utils";
-import { DeltaManagerProxyBase } from "@fluidframework/container-utils";
-/**
- * Proxy to the real IDeltaQueue - used to restrict access
- */
-export class DeltaQueueProxy extends EventForwarder {
-    constructor(queue) {
-        super(queue);
-        this.queue = queue;
-    }
-    get paused() {
-        return this.queue.paused;
-    }
-    get length() {
-        return this.queue.length;
-    }
-    get idle() {
-        return this.queue.idle;
-    }
-    peek() {
-        return this.queue.peek();
-    }
-    toArray() {
-        return this.queue.toArray();
-    }
-    // back-compat: usage removed in 0.33, remove in future versions
-    async systemPause() {
-        return this.pause();
-    }
-    async pause() {
-        return this.queue.pause();
-    }
-    // back-compat: usage removed in 0.33, remove in future versions
-    async systemResume() {
-        return this.resume();
-    }
-    async resume() {
-        this.queue.resume();
-    }
-    async waitTillProcessingDone() {
-        return this.queue.waitTillProcessingDone();
-    }
-}
-/**
- * Proxy to the real IDeltaManager - used to restrict access
- */
-export class DeltaManagerProxy extends DeltaManagerProxyBase {
-    constructor(deltaManager) {
-        super(deltaManager);
-        this._inbound = new DeltaQueueProxy(deltaManager.inbound);
-        this._outbound = new DeltaQueueProxy(deltaManager.outbound);
-        this._inboundSignal = new DeltaQueueProxy(deltaManager.inboundSignal);
-    }
-    get inbound() {
-        return this._inbound;
-    }
-    get outbound() {
-        return this._outbound;
-    }
-    get inboundSignal() {
-        return this._inboundSignal;
-    }
-    dispose() {
-        this._inbound.dispose();
-        this._outbound.dispose();
-        this._inboundSignal.dispose();
-        super.dispose();
-    }
-}
-//# sourceMappingURL=deltaManagerProxy.js.map