@fluidframework/container-loader 2.0.0-internal.3.0.5 → 2.0.0-internal.3.1.1

package/package.json

@@ -1,116 +1,117 @@
 {
-  "name": "@fluidframework/container-loader",
-  "version": "2.0.0-internal.3.0.5",
-  "description": "Fluid container loader",
-  "homepage": "https://fluidframework.com",
-  "repository": {
-    "type": "git",
-    "url": "https://github.com/microsoft/FluidFramework.git",
-    "directory": "packages/loader/container-loader"
-  },
-  "license": "MIT",
-  "author": "Microsoft and contributors",
-  "sideEffects": false,
-  "main": "dist/index.js",
-  "module": "lib/index.js",
-  "types": "dist/index.d.ts",
-  "scripts": {
-    "build": "npm run build:genver && concurrently npm:build:compile npm:lint && npm run build:docs",
-    "build:commonjs": "npm run tsc && npm run typetests:gen && npm run build:test",
-    "build:compile": "concurrently npm:build:commonjs npm:build:esnext",
-    "build:docs": "api-extractor run --local --typescript-compiler-folder ../../../node_modules/typescript && copyfiles -u 1 ./_api-extractor-temp/doc-models/* ../../../_api-extractor-temp/",
-    "build:esnext": "tsc --project ./tsconfig.esnext.json",
-    "build:full": "npm run build",
-    "build:full:compile": "npm run build:compile",
-    "build:genver": "gen-version",
-    "build:test": "tsc --project ./src/test/tsconfig.json",
-    "ci:build:docs": "api-extractor run --typescript-compiler-folder ../../../node_modules/typescript && copyfiles -u 1 ./_api-extractor-temp/* ../../../_api-extractor-temp/",
-    "clean": "rimraf dist lib *.tsbuildinfo *.build.log",
-    "eslint": "eslint --format stylish src",
-    "eslint:fix": "eslint --format stylish src --fix --fix-type problem,suggestion,layout",
-    "format": "npm run prettier:fix",
-    "lint": "npm run eslint",
-    "lint:fix": "npm run eslint:fix",
-    "prettier": "prettier --check . --ignore-path ../../../.prettierignore",
-    "prettier:fix": "prettier --write . --ignore-path ../../../.prettierignore",
-    "test": "npm run test:mocha",
-    "test:coverage": "nyc npm test -- --reporter xunit --reporter-option output=nyc/junit-report.xml",
-    "test:mocha": "mocha --ignore 'dist/test/types/*' --recursive dist/test -r node_modules/@fluidframework/mocha-test-setup --unhandled-rejections=strict",
-    "test:mocha:verbose": "cross-env FLUID_TEST_VERBOSE=1 npm run test:mocha",
-    "tsc": "tsc",
-    "tsc:watch": "tsc --watch",
-    "typetests:gen": "flub generate typetests --generate --dir .",
-    "typetests:prepare": "flub generate typetests --prepare --dir . --pin"
-  },
-  "nyc": {
-    "all": true,
-    "cache-dir": "nyc/.cache",
-    "exclude": [
-      "src/test/**/*.ts",
-      "dist/test/**/*.js"
-    ],
-    "exclude-after-remap": false,
-    "include": [
-      "src/**/*.ts",
-      "dist/**/*.js"
-    ],
-    "report-dir": "nyc/report",
-    "reporter": [
-      "cobertura",
-      "html",
-      "text"
-    ],
-    "temp-directory": "nyc/.nyc_output"
-  },
-  "dependencies": {
-    "@fluidframework/common-definitions": "^0.20.1",
-    "@fluidframework/common-utils": "^1.0.0",
-    "@fluidframework/container-definitions": ">=2.0.0-internal.3.0.5 <2.0.0-internal.4.0.0",
-    "@fluidframework/container-utils": ">=2.0.0-internal.3.0.5 <2.0.0-internal.4.0.0",
-    "@fluidframework/core-interfaces": ">=2.0.0-internal.3.0.5 <2.0.0-internal.4.0.0",
-    "@fluidframework/driver-definitions": ">=2.0.0-internal.3.0.5 <2.0.0-internal.4.0.0",
-    "@fluidframework/driver-utils": ">=2.0.0-internal.3.0.5 <2.0.0-internal.4.0.0",
-    "@fluidframework/protocol-base": "^0.1038.2000",
-    "@fluidframework/protocol-definitions": "^1.1.0",
-    "@fluidframework/telemetry-utils": ">=2.0.0-internal.3.0.5 <2.0.0-internal.4.0.0",
-    "abort-controller": "^3.0.0",
-    "double-ended-queue": "^2.1.0-0",
-    "events": "^3.1.0",
-    "lodash": "^4.17.21",
-    "url": "^0.11.0",
-    "uuid": "^8.3.1"
-  },
-  "devDependencies": {
-    "@fluid-tools/build-cli": "^0.8.0",
-    "@fluidframework/build-common": "^1.1.0",
-    "@fluidframework/build-tools": "^0.8.0",
-    "@fluidframework/container-loader-previous": "npm:@fluidframework/container-loader@2.0.0-internal.3.0.0",
-    "@fluidframework/eslint-config-fluid": "^2.0.0",
-    "@fluidframework/mocha-test-setup": ">=2.0.0-internal.3.0.5 <2.0.0-internal.4.0.0",
-    "@fluidframework/test-loader-utils": ">=2.0.0-internal.3.0.5 <2.0.0-internal.4.0.0",
-    "@microsoft/api-extractor": "^7.22.2",
-    "@rushstack/eslint-config": "^2.5.1",
-    "@types/double-ended-queue": "^2.1.0",
-    "@types/events": "^3.0.0",
-    "@types/lodash": "^4.14.118",
-    "@types/mocha": "^9.1.1",
-    "@types/node": "^14.18.36",
-    "@types/sinon": "^7.0.13",
-    "concurrently": "^6.2.0",
-    "copyfiles": "^2.4.1",
-    "cross-env": "^7.0.2",
-    "eslint": "~8.6.0",
-    "mocha": "^10.0.0",
-    "nyc": "^15.0.0",
-    "prettier": "~2.6.2",
-    "rimraf": "^2.6.2",
-    "sinon": "^7.4.2",
-    "typescript": "~4.5.5"
-  },
-  "typeValidation": {
-    "version": "2.0.0-internal.3.0.1",
-    "previousVersionStyle": "previousPatch",
-    "baselineRange": "2.0.0-internal.3.0.0",
-    "broken": {}
-  }
+	"name": "@fluidframework/container-loader",
+	"version": "2.0.0-internal.3.1.1",
+	"description": "Fluid container loader",
+	"homepage": "https://fluidframework.com",
+	"repository": {
+		"type": "git",
+		"url": "https://github.com/microsoft/FluidFramework.git",
+		"directory": "packages/loader/container-loader"
+	},
+	"license": "MIT",
+	"author": "Microsoft and contributors",
+	"sideEffects": false,
+	"main": "dist/index.js",
+	"module": "lib/index.js",
+	"types": "dist/index.d.ts",
+	"scripts": {
+		"build": "npm run build:genver && concurrently npm:build:compile npm:lint && npm run build:docs",
+		"build:commonjs": "npm run tsc && npm run typetests:gen && npm run build:test",
+		"build:compile": "concurrently npm:build:commonjs npm:build:esnext",
+		"build:docs": "api-extractor run --local --typescript-compiler-folder ../../../node_modules/typescript && copyfiles -u 1 ./_api-extractor-temp/doc-models/* ../../../_api-extractor-temp/",
+		"build:esnext": "tsc --project ./tsconfig.esnext.json",
+		"build:full": "npm run build",
+		"build:full:compile": "npm run build:compile",
+		"build:genver": "gen-version",
+		"build:test": "tsc --project ./src/test/tsconfig.json",
+		"ci:build:docs": "api-extractor run --typescript-compiler-folder ../../../node_modules/typescript && copyfiles -u 1 ./_api-extractor-temp/* ../../../_api-extractor-temp/",
+		"clean": "rimraf dist lib *.tsbuildinfo *.build.log",
+		"eslint": "eslint --format stylish src",
+		"eslint:fix": "eslint --format stylish src --fix --fix-type problem,suggestion,layout",
+		"format": "npm run prettier:fix",
+		"lint": "npm run prettier && npm run eslint",
+		"lint:fix": "npm run prettier:fix && npm run eslint:fix",
+		"prettier": "prettier --check . --ignore-path ../../../.prettierignore",
+		"prettier:fix": "prettier --write . --ignore-path ../../../.prettierignore",
+		"test": "npm run test:mocha",
+		"test:coverage": "nyc npm test -- --reporter xunit --reporter-option output=nyc/junit-report.xml",
+		"test:mocha": "mocha --ignore 'dist/test/types/*' --recursive dist/test -r node_modules/@fluidframework/mocha-test-setup --unhandled-rejections=strict",
+		"test:mocha:verbose": "cross-env FLUID_TEST_VERBOSE=1 npm run test:mocha",
+		"tsc": "tsc",
+		"tsc:watch": "tsc --watch",
+		"typetests:gen": "flub generate typetests --generate --dir .",
+		"typetests:prepare": "flub generate typetests --prepare --dir . --pin"
+	},
+	"nyc": {
+		"all": true,
+		"cache-dir": "nyc/.cache",
+		"exclude": [
+			"src/test/**/*.ts",
+			"dist/test/**/*.js"
+		],
+		"exclude-after-remap": false,
+		"include": [
+			"src/**/*.ts",
+			"dist/**/*.js"
+		],
+		"report-dir": "nyc/report",
+		"reporter": [
+			"cobertura",
+			"html",
+			"text"
+		],
+		"temp-directory": "nyc/.nyc_output"
+	},
+	"dependencies": {
+		"@fluidframework/common-definitions": "^0.20.1",
+		"@fluidframework/common-utils": "^1.0.0",
+		"@fluidframework/container-definitions": ">=2.0.0-internal.3.1.1 <2.0.0-internal.4.0.0",
+		"@fluidframework/container-utils": ">=2.0.0-internal.3.1.1 <2.0.0-internal.4.0.0",
+		"@fluidframework/core-interfaces": ">=2.0.0-internal.3.1.1 <2.0.0-internal.4.0.0",
+		"@fluidframework/driver-definitions": ">=2.0.0-internal.3.1.1 <2.0.0-internal.4.0.0",
+		"@fluidframework/driver-utils": ">=2.0.0-internal.3.1.1 <2.0.0-internal.4.0.0",
+		"@fluidframework/protocol-base": "^0.1038.2000",
+		"@fluidframework/protocol-definitions": "^1.1.0",
+		"@fluidframework/telemetry-utils": ">=2.0.0-internal.3.1.1 <2.0.0-internal.4.0.0",
+		"abort-controller": "^3.0.0",
+		"double-ended-queue": "^2.1.0-0",
+		"events": "^3.1.0",
+		"lodash": "^4.17.21",
+		"url": "^0.11.0",
+		"uuid": "^8.3.1"
+	},
+	"devDependencies": {
+		"@fluid-tools/build-cli": "^0.9.0",
+		"@fluidframework/build-common": "^1.1.0",
+		"@fluidframework/build-tools": "^0.9.0",
+		"@fluidframework/container-loader-previous": "npm:@fluidframework/container-loader@2.0.0-internal.3.0.1",
+		"@fluidframework/eslint-config-fluid": "^2.0.0",
+		"@fluidframework/mocha-test-setup": ">=2.0.0-internal.3.1.1 <2.0.0-internal.4.0.0",
+		"@fluidframework/test-loader-utils": ">=2.0.0-internal.3.1.1 <2.0.0-internal.4.0.0",
+		"@microsoft/api-extractor": "^7.22.2",
+		"@rushstack/eslint-config": "^2.5.1",
+		"@types/double-ended-queue": "^2.1.0",
+		"@types/events": "^3.0.0",
+		"@types/lodash": "^4.14.118",
+		"@types/mocha": "^9.1.1",
+		"@types/node": "^14.18.36",
+		"@types/sinon": "^7.0.13",
+		"concurrently": "^6.2.0",
+		"copyfiles": "^2.4.1",
+		"cross-env": "^7.0.2",
+		"eslint": "~8.6.0",
+		"mocha": "^10.0.0",
+		"nyc": "^15.0.0",
+		"prettier": "~2.6.2",
+		"rimraf": "^2.6.2",
+		"sinon": "^7.4.2",
+		"typescript": "~4.5.5"
+	},
+	"typeValidation": {
+		"version": "2.0.0-internal.3.1.1",
+		"previousVersionStyle": "~previousMinor",
+		"baselineRange": ">=2.0.0-internal.3.0.0 <2.0.0-internal.3.1.0",
+		"baselineVersion": "2.0.0-internal.3.0.1",
+		"broken": {}
+	}
 }

package/prettier.config.cjs

@@ -4,5 +4,5 @@
  */
 
 module.exports = {
-    ...require("@fluidframework/build-common/prettier.config.cjs"),
+	...require("@fluidframework/build-common/prettier.config.cjs"),
 };

package/src/audience.ts

@@ -11,55 +11,60 @@ import { IClient } from "@fluidframework/protocol-definitions";
  * Audience represents all clients connected to the op stream.
  */
 export class Audience extends EventEmitter implements IAudienceOwner {
-    private readonly members = new Map<string, IClient>();
+	private readonly members = new Map<string, IClient>();
 
-    public on(event: "addMember" | "removeMember", listener: (clientId: string, client: IClient) => void): this;
-    public on(event: string, listener: (...args: any[]) => void): this {
-        return super.on(event, listener);
-    }
+	public on(
+		event: "addMember" | "removeMember",
+		listener: (clientId: string, client: IClient) => void,
+	): this;
+	public on(event: string, listener: (...args: any[]) => void): this {
+		return super.on(event, listener);
+	}
 
-    /**
-     * Adds a new client to the audience
-     */
-    public addMember(clientId: string, details: IClient) {
-        // Given that signal delivery is unreliable process, we might observe same client being added twice
-        // In such case we should see exactly same payload (IClient), and should not raise event twice!
-        if (this.members.has(clientId)) {
-            const client = this.members.get(clientId);
-            assert(JSON.stringify(client) === JSON.stringify(details), 0x4b2 /* new client has different payload from existing one */);
-        }
-        else {
-            this.members.set(clientId, details);
-            this.emit("addMember", clientId, details);
-        }
-    }
+	/**
+	 * Adds a new client to the audience
+	 */
+	public addMember(clientId: string, details: IClient) {
+		// Given that signal delivery is unreliable process, we might observe same client being added twice
+		// In such case we should see exactly same payload (IClient), and should not raise event twice!
+		if (this.members.has(clientId)) {
+			const client = this.members.get(clientId);
+			assert(
+				JSON.stringify(client) === JSON.stringify(details),
+				0x4b2 /* new client has different payload from existing one */,
+			);
+		} else {
+			this.members.set(clientId, details);
+			this.emit("addMember", clientId, details);
+		}
+	}
 
-    /**
-     * Removes a client from the audience. Only emits an event if a client is actually removed
-     * @returns if a client was removed from the audience
-     */
-    public removeMember(clientId: string): boolean {
-        const removedClient = this.members.get(clientId);
-        if (removedClient !== undefined) {
-            this.members.delete(clientId);
-            this.emit("removeMember", clientId, removedClient);
-            return true;
-        } else {
-            return false;
-        }
-    }
+	/**
+	 * Removes a client from the audience. Only emits an event if a client is actually removed
+	 * @returns if a client was removed from the audience
+	 */
+	public removeMember(clientId: string): boolean {
+		const removedClient = this.members.get(clientId);
+		if (removedClient !== undefined) {
+			this.members.delete(clientId);
+			this.emit("removeMember", clientId, removedClient);
+			return true;
+		} else {
+			return false;
+		}
+	}
 
-    /**
-     * Retrieves all the members in the audience
-     */
-    public getMembers(): Map<string, IClient> {
-        return new Map(this.members);
-    }
+	/**
+	 * Retrieves all the members in the audience
+	 */
+	public getMembers(): Map<string, IClient> {
+		return new Map(this.members);
+	}
 
-    /**
-     * Retrieves a specific member of the audience
-     */
-    public getMember(clientId: string): IClient | undefined {
-        return this.members.get(clientId);
-    }
+	/**
+	 * Retrieves a specific member of the audience
+	 */
+	public getMember(clientId: string): IClient | undefined {
+		return this.members.get(clientId);
+	}
 }

package/src/catchUpMonitor.ts

@@ -19,41 +19,43 @@ export type ICatchUpMonitor = IDisposable;
  * that were known at the time the monitor was created.
  */
 export class CatchUpMonitor implements ICatchUpMonitor {
-    private readonly targetSeqNumber: number;
-    private caughtUp: boolean = false;
-
-    private readonly opHandler = (message: Pick<ISequencedDocumentMessage, "sequenceNumber">) => {
-        if (!this.caughtUp && message.sequenceNumber >= this.targetSeqNumber) {
-            this.caughtUp = true;
-            this.listener();
-        }
-    };
-
-    /**
-     * Create the CatchUpMonitor, setting the target sequence number to wait for based on DeltaManager's current state.
-     */
-    constructor(
-        private readonly deltaManager: IDeltaManager<any, any>,
-        private readonly listener: CaughtUpListener,
-    ) {
-        this.targetSeqNumber = this.deltaManager.lastKnownSeqNumber;
-
-        assert(this.targetSeqNumber >= this.deltaManager.lastSequenceNumber,
-            0x37c /* Cannot wait for seqNumber below last processed sequence number */);
-
-        this.deltaManager.on("op", this.opHandler);
-
-        // Simulate the last processed op to set caughtUp in case we already are
-        this.opHandler({ sequenceNumber: this.deltaManager.lastSequenceNumber });
-    }
-
-    public disposed: boolean = false;
-    public dispose() {
-        if (this.disposed) {
-            return;
-        }
-        this.disposed = true;
-
-        this.deltaManager.off("op", this.opHandler);
-    }
+	private readonly targetSeqNumber: number;
+	private caughtUp: boolean = false;
+
+	private readonly opHandler = (message: Pick<ISequencedDocumentMessage, "sequenceNumber">) => {
+		if (!this.caughtUp && message.sequenceNumber >= this.targetSeqNumber) {
+			this.caughtUp = true;
+			this.listener();
+		}
+	};
+
+	/**
+	 * Create the CatchUpMonitor, setting the target sequence number to wait for based on DeltaManager's current state.
+	 */
+	constructor(
+		private readonly deltaManager: IDeltaManager<any, any>,
+		private readonly listener: CaughtUpListener,
+	) {
+		this.targetSeqNumber = this.deltaManager.lastKnownSeqNumber;
+
+		assert(
+			this.targetSeqNumber >= this.deltaManager.lastSequenceNumber,
+			0x37c /* Cannot wait for seqNumber below last processed sequence number */,
+		);
+
+		this.deltaManager.on("op", this.opHandler);
+
+		// Simulate the last processed op to set caughtUp in case we already are
+		this.opHandler({ sequenceNumber: this.deltaManager.lastSequenceNumber });
+	}
+
+	public disposed: boolean = false;
+	public dispose() {
+		if (this.disposed) {
+			return;
+		}
+		this.disposed = true;
+
+		this.deltaManager.off("op", this.opHandler);
+	}
 }

package/src/collabWindowTracker.ts

@@ -30,82 +30,87 @@ const defaultNoopCountFrequency = 50;
 // 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;
-    private readonly timer: Timer | undefined;
+	private opsCountSinceNoop = 0;
+	private readonly timer: Timer | undefined;
 
-    constructor(
-        private readonly submit: (type: MessageType) => void,
-        NoopTimeFrequency: number = defaultNoopTimeFrequency,
-        private readonly NoopCountFrequency: number = defaultNoopCountFrequency,
-    ) {
-        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 */);
-                }
-            });
-        }
-    }
+	constructor(
+		private readonly submit: (type: MessageType) => void,
+		NoopTimeFrequency: number = defaultNoopTimeFrequency,
+		private readonly NoopCountFrequency: number = defaultNoopCountFrequency,
+	) {
+		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 */);
+				}
+			});
+		}
+	}
 
-    /**
-     * 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;
-        }
+	/**
+	 * 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;
+		}
 
-        // 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
-        // due to runtime ops from other clients.
-        if (!isRuntimeMessage(message)) {
-            return;
-        }
+		// 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
+		// due to runtime ops from other clients.
+		if (!isRuntimeMessage(message)) {
+			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;
-                }
-                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;
+				}
+				return;
+			});
+		}
 
-        if (this.timer !== undefined) {
-            if (this.opsCountSinceNoop === 1) {
-                this.timer.restart();
-            }
+		if (this.timer !== undefined) {
+			if (this.opsCountSinceNoop === 1) {
+				this.timer.restart();
+			}
 
-            assert(this.timer.hasTimer, 0x242 /* "has timer" */);
-        }
-    }
+			assert(this.timer.hasTimer, 0x242 /* "has timer" */);
+		}
+	}
 
-    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!" */);
-    }
+	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 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 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();
+	}
 }