@fluidframework/id-compressor 2.1.0-276985 → 2.1.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@fluidframework/id-compressor",
-  "version": "2.1.0-276985",
+  "version": "2.1.0",
   "description": "ID compressor",
   "homepage": "https://fluidframework.com",
   "repository": {
@@ -79,24 +79,24 @@
     "temp-directory": "nyc/.nyc_output"
   },
   "dependencies": {
-    "@fluid-internal/client-utils": "2.1.0-276985",
-    "@fluidframework/core-interfaces": "2.1.0-276985",
-    "@fluidframework/core-utils": "2.1.0-276985",
-    "@fluidframework/telemetry-utils": "2.1.0-276985",
+    "@fluid-internal/client-utils": "~2.1.0",
+    "@fluidframework/core-interfaces": "~2.1.0",
+    "@fluidframework/core-utils": "~2.1.0",
+    "@fluidframework/telemetry-utils": "~2.1.0",
     "@tylerbu/sorted-btree-es6": "^1.8.0",
     "uuid": "^9.0.0"
   },
   "devDependencies": {
     "@arethetypeswrong/cli": "^0.15.2",
-    "@biomejs/biome": "^1.7.3",
-    "@fluid-internal/mocha-test-setup": "2.1.0-276985",
-    "@fluid-private/stochastic-test-utils": "2.1.0-276985",
+    "@biomejs/biome": "~1.8.3",
+    "@fluid-internal/mocha-test-setup": "~2.1.0",
+    "@fluid-private/stochastic-test-utils": "~2.1.0",
     "@fluid-tools/benchmark": "^0.48.0",
-    "@fluid-tools/build-cli": "^0.39.0",
+    "@fluid-tools/build-cli": "^0.41.0",
     "@fluidframework/build-common": "^2.0.3",
-    "@fluidframework/build-tools": "^0.39.0",
+    "@fluidframework/build-tools": "^0.41.0",
     "@fluidframework/eslint-config-fluid": "^5.3.0",
-    "@fluidframework/id-compressor-previous": "npm:@fluidframework/id-compressor@2.0.0-rc.5.0.0",
+    "@fluidframework/id-compressor-previous": "npm:@fluidframework/id-compressor@2.0.0",
     "@microsoft/api-extractor": "^7.45.1",
     "@types/mocha": "^9.1.1",
     "@types/node": "^18.19.0",
@@ -122,18 +122,19 @@
     "api-extractor:commonjs": "flub generate entrypoints --outDir ./dist",
     "api-extractor:esnext": "flub generate entrypoints --outDir ./lib --node10TypeCompat",
     "build": "fluid-build . --task build",
+    "build:api-reports": "concurrently \"npm:build:api-reports:*\"",
+    "build:api-reports:current": "api-extractor run --local --config api-extractor/api-extractor.current.json",
+    "build:api-reports:legacy": "api-extractor run --local --config api-extractor/api-extractor.legacy.json",
     "build:commonjs": "fluid-build . --task commonjs",
     "build:compile": "fluid-build . --task compile",
-    "build:docs": "concurrently \"npm:build:docs:*\"",
-    "build:docs:current": "api-extractor run --local",
-    "build:docs:legacy": "api-extractor run --local --config api-extractor/api-extractor.legacy.json",
+    "build:docs": "api-extractor run --local",
     "build:esnext": "tsc --project ./tsconfig.json",
     "build:genver": "gen-version",
     "build:test": "npm run build:test:esm && npm run build:test:cjs",
     "build:test:cjs": "fluid-tsc commonjs --project ./src/test/tsconfig.cjs.json",
     "build:test:esm": "tsc --project ./src/test/tsconfig.json",
     "check:are-the-types-wrong": "attw --pack . --exclude-entrypoints ./internal/test-utils",
-    "check:biome": "biome check . --formatter-enabled=true",
+    "check:biome": "biome check .",
     "check:exports": "concurrently \"npm:check:exports:*\"",
     "check:exports:bundle-release-tags": "api-extractor run --config api-extractor/api-extractor-lint-bundle.json",
     "check:exports:cjs:legacy": "api-extractor run --config api-extractor/api-extractor-lint-legacy.cjs.json",
@@ -142,14 +143,15 @@
     "check:exports:esm:public": "api-extractor run --config api-extractor/api-extractor-lint-public.esm.json",
     "check:format": "npm run check:biome",
     "check:prettier": "prettier --check . --cache --ignore-path ../../../.prettierignore",
-    "ci:build:docs": "concurrently \"npm:ci:build:docs:*\"",
-    "ci:build:docs:current": "api-extractor run",
-    "ci:build:docs:legacy": "api-extractor run --config api-extractor/api-extractor.legacy.json",
+    "ci:build:api-reports": "concurrently \"npm:ci:build:api-reports:*\"",
+    "ci:build:api-reports:current": "api-extractor run --config api-extractor/api-extractor.current.json",
+    "ci:build:api-reports:legacy": "api-extractor run --config api-extractor/api-extractor.legacy.json",
+    "ci:build:docs": "api-extractor run",
     "clean": "rimraf --glob dist lib \"*.d.ts\" \"**/*.tsbuildinfo\" \"**/*.build.log\" _api-extractor-temp nyc",
     "eslint": "eslint --format stylish src",
     "eslint:fix": "eslint --format stylish src --fix --fix-type problem,suggestion,layout",
     "format": "npm run format:biome",
-    "format:biome": "biome check . --formatter-enabled=true --apply",
+    "format:biome": "biome check . --write",
     "format:prettier": "prettier --write . --cache --ignore-path ../../../.prettierignore",
     "lint": "fluid-build . --task lint",
     "lint:fix": "fluid-build . --task eslint:fix --task format",

package/src/finalSpace.ts

@@ -63,11 +63,12 @@ export class FinalSpace {
 	}
 
 	public equals(other: FinalSpace): boolean {
-		for (let i = 0; i < this.clusterList.length; i++) {
-			if (!clustersEqual(this.clusterList[i], other.clusterList[i])) {
+		for (const [index, value] of Object.entries(this.clusterList)) {
+			if (!clustersEqual(value, other.clusterList[index])) {
 				return false;
 			}
 		}
+
 		return this.clusterList.length === other.clusterList.length;
 	}
 }

package/src/idCompressor.ts

@@ -697,7 +697,12 @@ export class IdCompressor implements IIdCompressor, IIdCompressorCore {
 		let baseFinalId = 0;
 		for (let i = 0; i < clusterCount; i++) {
 			const sessionIndex = readNumber(index);
-			const session = sessions[sessionIndex + sessionOffset][1];
+			const sessionArray = sessions[sessionIndex + sessionOffset];
+			assert(
+				sessionArray !== undefined,
+				0x9d8 /* sessionArray is undefined in IdCompressor.deserialize2_0 */,
+			);
+			const session = sessionArray[1];
 			const capacity = readNumber(index);
 			const count = readNumber(index);
 			const cluster = session.addNewCluster(baseFinalId as FinalCompressedId, capacity, count);

package/src/packageVersion.ts

@@ -6,4 +6,4 @@
  */
 
 export const pkgName = "@fluidframework/id-compressor";
-export const pkgVersion = "2.1.0-276985";
+export const pkgVersion = "2.1.0";

package/src/persistanceUtilities.ts

@@ -4,6 +4,8 @@
  */
 
 /* eslint-disable no-bitwise */
+import { assert } from "@fluidframework/core-utils/internal";
+
 import { NumericUuid } from "./identifiers.js";
 
 const halfNumeric = BigInt("0xFFFFFFFFFFFFFFFF");
@@ -41,13 +43,16 @@ export interface Index {
 
 export function readNumber(index: Index): number {
 	const value = index.bufferFloat[index.index];
+	assert(value !== undefined, 0x9d9 /* value is undefined in readNumber */);
 	index.index += 1;
 	return value;
 }
 
 export function readNumericUuid(index: Index): NumericUuid {
 	const lowerHalf = index.bufferUint[index.index];
+	assert(lowerHalf !== undefined, 0x9da /* lowerHalf is undefined in readNumericUuid */);
 	const upperHalf = index.bufferUint[index.index + 1];
+	assert(upperHalf !== undefined, 0x9db /* upperHalf is undefined in readNumericUuid */);
 	const value = (upperHalf << sixtyFour) | lowerHalf;
 	index.index += 2;
 	return value as NumericUuid;

package/src/sessionSpaceNormalizer.ts

@@ -75,6 +75,10 @@ export class SessionSpaceNormalizer {
 		if (ranges.length === 0) {
 			return ranges;
 		}
+		assert(
+			ranges[0] !== undefined,
+			0x9dd /* ranges[0] is undefined in SessionSpaceNormalizer.getRangesBetween */,
+		);
 
 		// now we touch up the first and last ranges to ensure that if they contain the
 		// queried IDs they are trimmed to start/end with the queried IDs
@@ -93,11 +97,16 @@ export class SessionSpaceNormalizer {
 		}
 
 		const lastRangeIndex = ranges.length - 1;
-		const [limitGenCount, limitCount] = ranges[lastRangeIndex];
-		if (this.rangeContains(ranges[lastRangeIndex], lastGenCount)) {
+		const lastRange = ranges[lastRangeIndex];
+		assert(
+			lastRange !== undefined,
+			0x9de /* lastRange is undefined in SessionSpaceNormalizer.getRangesBetween */,
+		);
+		const [limitGenCount, limitCount] = lastRange;
+		if (this.rangeContains(lastRange, lastGenCount)) {
 			ranges[lastRangeIndex] = [limitGenCount, lastGenCount - limitGenCount + 1];
 			assert(
-				this.rangeContains(ranges[lastRangeIndex], lastGenCount),
+				this.rangeContains(lastRange, lastGenCount),
 				0x954 /* Expected the touched up range to contain the queried ID */,
 			);
 		} else {

package/src/sessions.ts

@@ -280,9 +280,11 @@ export class Session {
 		let right = arr.length - 1;
 		while (left <= right) {
 			const mid = Math.floor((left + right) / 2);
-			const c = comparator(search, arr[mid]);
+			const value = arr[mid];
+			assert(value !== undefined, 0x9dc /* value is undefined in Session.binarySearch */);
+			const c = comparator(search, value);
 			if (c === 0) {
-				return arr[mid]; // Found the target, return its index.
+				return value; // Found the target, return its index.
 			} else if (c > 0) {
 				left = mid + 1; // Continue search on right half.
 			} else {
@@ -293,11 +295,12 @@ export class Session {
 	}
 
 	public equals(other: Session): boolean {
-		for (let i = 0; i < this.clusterChain.length; i++) {
-			if (!clustersEqual(this.clusterChain[i], other.clusterChain[i])) {
+		for (const [index, value] of Object.entries(this.clusterChain)) {
+			if (!clustersEqual(value, other.clusterChain[index])) {
 				return false;
 			}
 		}
+
 		return this.sessionUuid === other.sessionUuid;
 	}
 }

package/src/types/identifiers.ts

@@ -24,7 +24,7 @@ export type OpSpaceCompressedId = number & {
 };
 
 /**
- * A version 4, variant 1 uuid (https://datatracker.ietf.org/doc/html/rfc4122).
+ * A {@link https://datatracker.ietf.org/doc/html/rfc4122 | version 4, variant 1 uuid}.
  * A 128-bit Universally Unique IDentifier. Represented here
  * with a string of the form xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx,
  * where x is a lowercase hex digit.

package/tsconfig.json

@@ -4,7 +4,6 @@
 		"rootDir": "./src",
 		"outDir": "./lib",
 		"exactOptionalPropertyTypes": false,
-		"noUncheckedIndexedAccess": false,
 	},
 	"include": ["src/**/*"],
 	"exclude": ["src/test/**/*"],