@fluidframework/matrix - npm Package Compare versions

Comparing version 2.3.0-288113 to 2.3.0

CHANGELOG.md

 # @fluidframework/matrix
 
+## 2.3.0
+
+Dependency updates only.
+
 ## 2.2.0
@@ -4,0 +8,0 @@

dist/handlecache.js

@@ -46,7 +46,3 @@ "use strict";
         //       ~40% speedup when the position is in the cache (node v12 x64).
-        const handle = this.handles[index];
-        if (handle !== undefined) {
-            return handle;
-        }
-        return this.cacheMiss(position);
+        return index < this.handles.length ? this.handles[index] : this.cacheMiss(position);
     }
@@ -60,6 +56,3 @@ /**
         if (index < this.handles.length) {
-            (0, internal_1.assert)(
-            // Non null asserting, above we checked that the index is less than the length.
-            // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
-            !(0, handletable_js_1.isHandleValid)(this.handles[index]), 0x018 /* "Trying to insert handle into position with already valid handle!" */);
+            (0, internal_1.assert)(!(0, handletable_js_1.isHandleValid)(this.handles[index]), 0x018 /* "Trying to insert handle into position with already valid handle!" */);
             this.handles[index] = handle;
@@ -96,4 +89,2 @@ }
             this.start = _position;
-            // TODO why are we non null asserting here?
-            // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
             return this.handles[0];
@@ -107,4 +98,2 @@ }
             ];
-            // TODO why are we non null asserting here?
-            // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
             return this.handles[this.handles.length - 1];
@@ -111,0 +100,0 @@ }

dist/matrix.js

@@ -328,4 +328,2 @@ "use strict";
             for (let col = 0; col < this.colCount; col++) {
-                // TODO Non null asserting, why is this not null?
-                // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
                 const colHandle = this.colHandles.getHandle(col);
@@ -352,4 +350,2 @@ const value = this.cells.getCell(rowHandle, colHandle);
             for (let row = 0; row < this.rowCount; row++) {
-                // TODO Non null asserting, why is this not null?
-                // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
                 const rowHandle = this.rowHandles.getHandle(row);
@@ -356,0 +352,0 @@ const value = this.cells.getCell(rowHandle, colHandle);

dist/packageVersion.d.ts

@@ -8,3 +8,3 @@ /*!
 export declare const pkgName = "@fluidframework/matrix";
-export declare const pkgVersion = "2.3.0-288113";
+export declare const pkgVersion = "2.3.0";
 //# sourceMappingURL=packageVersion.d.ts.map

dist/packageVersion.js

@@ -11,3 +11,3 @@ "use strict";
 exports.pkgName = "@fluidframework/matrix";
-exports.pkgVersion = "2.3.0-288113";
+exports.pkgVersion = "2.3.0";
 //# sourceMappingURL=packageVersion.js.map

dist/permutationvector.js

@@ -166,4 +166,2 @@ "use strict";
         (0, internal_1.assert)(0 <= pos && pos < this.getLength(), 0x027 /* "Trying to get handle of out-of-bounds position!" */);
-        // TODO Non null asserting, why is this not null?
-        // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
         return this.handleCache.getHandle(pos);
@@ -170,0 +168,0 @@ }

dist/sparsearray2d.js

@@ -30,6 +30,3 @@ "use strict";
 // interleaved between the original bits. (e.g., 1111... -> 01010101...).
-const interlaceBitsX16 = (x16) => 
-// TODO Non null asserting, why is this not null?
-// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
-(x8ToInterlacedX16[byte2(x16)] << 16) | x8ToInterlacedX16[byte3(x16)];
+const interlaceBitsX16 = (x16) => (x8ToInterlacedX16[byte2(x16)] << 16) | x8ToInterlacedX16[byte3(x16)];
 const r0ToMorton16 = (row) => (interlaceBitsX16(row) << 1) >>> 0;
@@ -36,0 +33,0 @@ const c0ToMorton16 = (col) => interlaceBitsX16(col) >>> 0;

dist/undoprovider.js

@@ -72,4 +72,2 @@ "use strict";
                         while (removedTrackingGroup.size > 0) {
-                            // TODO Non null asserting, why is this not null?
-                            // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
                             const tracked = removedTrackingGroup.tracked[0];
@@ -97,4 +95,2 @@ removedTrackingGroup.unlink(tracked);
                     while (removedTrackingGroup.size > 0) {
-                        // TODO Non null asserting, why is this not null?
-                        // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
                         removedTrackingGroup.unlink(removedTrackingGroup.tracked[0]);
@@ -101,0 +97,0 @@ }

lib/handlecache.js

@@ -43,7 +43,3 @@ /*!
         //       ~40% speedup when the position is in the cache (node v12 x64).
-        const handle = this.handles[index];
-        if (handle !== undefined) {
-            return handle;
-        }
-        return this.cacheMiss(position);
+        return index < this.handles.length ? this.handles[index] : this.cacheMiss(position);
     }
@@ -57,6 +53,3 @@ /**
         if (index < this.handles.length) {
-            assert(
-            // Non null asserting, above we checked that the index is less than the length.
-            // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
-            !isHandleValid(this.handles[index]), 0x018 /* "Trying to insert handle into position with already valid handle!" */);
+            assert(!isHandleValid(this.handles[index]), 0x018 /* "Trying to insert handle into position with already valid handle!" */);
             this.handles[index] = handle;
@@ -93,4 +86,2 @@ }
             this.start = _position;
-            // TODO why are we non null asserting here?
-            // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
             return this.handles[0];
@@ -104,4 +95,2 @@ }
             ];
-            // TODO why are we non null asserting here?
-            // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
             return this.handles[this.handles.length - 1];
@@ -108,0 +97,0 @@ }

lib/matrix.js

@@ -322,4 +322,2 @@ /*!
             for (let col = 0; col < this.colCount; col++) {
-                // TODO Non null asserting, why is this not null?
-                // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
                 const colHandle = this.colHandles.getHandle(col);
@@ -346,4 +344,2 @@ const value = this.cells.getCell(rowHandle, colHandle);
             for (let row = 0; row < this.rowCount; row++) {
-                // TODO Non null asserting, why is this not null?
-                // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
                 const rowHandle = this.rowHandles.getHandle(row);
@@ -350,0 +346,0 @@ const value = this.cells.getCell(rowHandle, colHandle);

lib/packageVersion.d.ts

@@ -8,3 +8,3 @@ /*!
 export declare const pkgName = "@fluidframework/matrix";
-export declare const pkgVersion = "2.3.0-288113";
+export declare const pkgVersion = "2.3.0";
 //# sourceMappingURL=packageVersion.d.ts.map

lib/packageVersion.js

@@ -8,3 +8,3 @@ /*!
 export const pkgName = "@fluidframework/matrix";
-export const pkgVersion = "2.3.0-288113";
+export const pkgVersion = "2.3.0";
 //# sourceMappingURL=packageVersion.js.map

lib/permutationvector.js

@@ -163,4 +163,2 @@ /*!
         assert(0 <= pos && pos < this.getLength(), 0x027 /* "Trying to get handle of out-of-bounds position!" */);
-        // TODO Non null asserting, why is this not null?
-        // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
         return this.handleCache.getHandle(pos);
@@ -167,0 +165,0 @@ }

lib/sparsearray2d.js

@@ -27,6 +27,3 @@ /*!
 // interleaved between the original bits. (e.g., 1111... -> 01010101...).
-const interlaceBitsX16 = (x16) => 
-// TODO Non null asserting, why is this not null?
-// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
-(x8ToInterlacedX16[byte2(x16)] << 16) | x8ToInterlacedX16[byte3(x16)];
+const interlaceBitsX16 = (x16) => (x8ToInterlacedX16[byte2(x16)] << 16) | x8ToInterlacedX16[byte3(x16)];
 const r0ToMorton16 = (row) => (interlaceBitsX16(row) << 1) >>> 0;
@@ -33,0 +30,0 @@ const c0ToMorton16 = (col) => interlaceBitsX16(col) >>> 0;

lib/tsdoc-metadata.json

@@ -8,5 +8,5 @@ // This file is read by tools that parse documentation comments conforming to the TSDoc standard.
       "packageName": "@microsoft/api-extractor",
-      "packageVersion": "7.45.1"
+      "packageVersion": "7.47.8"
     }
   ]
 }

lib/undoprovider.js

@@ -69,4 +69,2 @@ /*!
                         while (removedTrackingGroup.size > 0) {
-                            // TODO Non null asserting, why is this not null?
-                            // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
                             const tracked = removedTrackingGroup.tracked[0];
@@ -94,4 +92,2 @@ removedTrackingGroup.unlink(tracked);
                     while (removedTrackingGroup.size > 0) {
-                        // TODO Non null asserting, why is this not null?
-                        // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
                         removedTrackingGroup.unlink(removedTrackingGroup.tracked[0]);
@@ -98,0 +94,0 @@ }

package.json

 {
   "name": "@fluidframework/matrix",
-  "version": "2.3.0-288113",
+  "version": "2.3.0",
   "description": "Distributed matrix",
@@ -66,13 +66,13 @@ "homepage": "https://fluidframework.com",
   "dependencies": {
-    "@fluid-internal/client-utils": "2.3.0-288113",
-    "@fluidframework/core-interfaces": "2.3.0-288113",
-    "@fluidframework/core-utils": "2.3.0-288113",
-    "@fluidframework/datastore-definitions": "2.3.0-288113",
-    "@fluidframework/driver-definitions": "2.3.0-288113",
-    "@fluidframework/driver-utils": "2.3.0-288113",
-    "@fluidframework/merge-tree": "2.3.0-288113",
-    "@fluidframework/runtime-definitions": "2.3.0-288113",
-    "@fluidframework/runtime-utils": "2.3.0-288113",
-    "@fluidframework/shared-object-base": "2.3.0-288113",
-    "@fluidframework/telemetry-utils": "2.3.0-288113",
+    "@fluid-internal/client-utils": "~2.3.0",
+    "@fluidframework/core-interfaces": "~2.3.0",
+    "@fluidframework/core-utils": "~2.3.0",
+    "@fluidframework/datastore-definitions": "~2.3.0",
+    "@fluidframework/driver-definitions": "~2.3.0",
+    "@fluidframework/driver-utils": "~2.3.0",
+    "@fluidframework/merge-tree": "~2.3.0",
+    "@fluidframework/runtime-definitions": "~2.3.0",
+    "@fluidframework/runtime-utils": "~2.3.0",
+    "@fluidframework/shared-object-base": "~2.3.0",
+    "@fluidframework/telemetry-utils": "~2.3.0",
     "@tiny-calc/nano": "0.0.0-alpha.5",
@@ -85,14 +85,14 @@ "double-ended-queue": "^2.1.0-0",
     "@biomejs/biome": "~1.8.3",
-    "@fluid-internal/mocha-test-setup": "2.3.0-288113",
-    "@fluid-private/stochastic-test-utils": "2.3.0-288113",
-    "@fluid-private/test-dds-utils": "2.3.0-288113",
+    "@fluid-internal/mocha-test-setup": "~2.3.0",
+    "@fluid-private/stochastic-test-utils": "~2.3.0",
+    "@fluid-private/test-dds-utils": "~2.3.0",
     "@fluid-tools/benchmark": "^0.50.0",
-    "@fluid-tools/build-cli": "^0.43.0",
+    "@fluid-tools/build-cli": "^0.46.0",
     "@fluidframework/build-common": "^2.0.3",
-    "@fluidframework/build-tools": "^0.43.0",
-    "@fluidframework/container-definitions": "2.3.0-288113",
-    "@fluidframework/eslint-config-fluid": "^5.3.0",
+    "@fluidframework/build-tools": "^0.46.0",
+    "@fluidframework/container-definitions": "~2.3.0",
+    "@fluidframework/eslint-config-fluid": "^5.4.0",
     "@fluidframework/matrix-previous": "npm:@fluidframework/matrix@2.2.0",
-    "@fluidframework/test-runtime-utils": "2.3.0-288113",
-    "@microsoft/api-extractor": "^7.45.1",
+    "@fluidframework/test-runtime-utils": "~2.3.0",
+    "@microsoft/api-extractor": "7.47.8",
     "@tiny-calc/micro": "0.0.0-alpha.5",
@@ -120,3 +120,4 @@ "@types/double-ended-queue": "^2.1.0",
   "typeValidation": {
-    "broken": {}
+    "broken": {},
+    "entrypoint": "internal"
   },
@@ -123,0 +124,0 @@ "scripts": {

src/handlecache.ts

@@ -53,7 +53,3 @@ /*!
 
-		const handle = this.handles[index];
-		if (handle !== undefined) {
-			return handle;
-		}
-		return this.cacheMiss(position);
+		return index < this.handles.length ? this.handles[index] : this.cacheMiss(position);
 	}
@@ -70,5 +66,3 @@
 			assert(
-				// Non null asserting, above we checked that the index is less than the length.
-				// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
-				!isHandleValid(this.handles[index]!),
+				!isHandleValid(this.handles[index]),
 				0x018 /* "Trying to insert handle into position with already valid handle!" */,
@@ -115,5 +109,3 @@ );
 			this.start = _position;
-			// TODO why are we non null asserting here?
-			// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
-			return this.handles[0]!;
+			return this.handles[0];
 		} else {
@@ -126,5 +118,3 @@ ensureRange(_position, this.vector.getLength());
 			];
-			// TODO why are we non null asserting here?
-			// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
-			return this.handles[this.handles.length - 1]!;
+			return this.handles[this.handles.length - 1];
 		}
@@ -131,0 +121,0 @@ }

src/matrix.ts

@@ -624,5 +624,3 @@ /*!
 			for (let col = 0; col < this.colCount; col++) {
-				// TODO Non null asserting, why is this not null?
-				// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
-				const colHandle = this.colHandles.getHandle(col)!;
+				const colHandle = this.colHandles.getHandle(col);
 				const value = this.cells.getCell(rowHandle, colHandle);
@@ -651,5 +649,3 @@ if (this.isAttached() && value !== undefined && value !== null) {
 			for (let row = 0; row < this.rowCount; row++) {
-				// TODO Non null asserting, why is this not null?
-				// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
-				const rowHandle = this.rowHandles.getHandle(row)!;
+				const rowHandle = this.rowHandles.getHandle(row);
 				const value = this.cells.getCell(rowHandle, colHandle);
@@ -656,0 +652,0 @@ if (this.isAttached() && value !== undefined && value !== null) {

src/packageVersion.ts

@@ -9,2 +9,2 @@ /*!
 export const pkgName = "@fluidframework/matrix";
-export const pkgVersion = "2.3.0-288113";
+export const pkgVersion = "2.3.0";

src/permutationvector.ts

@@ -173,5 +173,3 @@ /*!
 
-		// TODO Non null asserting, why is this not null?
-		// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
-		return this.handleCache.getHandle(pos)!;
+		return this.handleCache.getHandle(pos);
 	}
@@ -178,0 +176,0 @@

src/sparsearray2d.ts

@@ -35,5 +35,3 @@ /*!
 const interlaceBitsX16 = (x16: number): number =>
-	// TODO Non null asserting, why is this not null?
-	// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
-	(x8ToInterlacedX16[byte2(x16)]! << 16) | x8ToInterlacedX16[byte3(x16)]!;
+	(x8ToInterlacedX16[byte2(x16)] << 16) | x8ToInterlacedX16[byte3(x16)];
 
@@ -40,0 +38,0 @@ const r0ToMorton16 = (row: number): number => (interlaceBitsX16(row) << 1) >>> 0;

src/undoprovider.ts

@@ -113,5 +113,3 @@ /*!
 						while (removedTrackingGroup.size > 0) {
-							// TODO Non null asserting, why is this not null?
-							// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
-							const tracked = removedTrackingGroup.tracked[0]!;
+							const tracked = removedTrackingGroup.tracked[0];
 							removedTrackingGroup.unlink(tracked);
@@ -137,5 +135,3 @@ // if there are groups tracked, this in a revert of a remove.
 					while (removedTrackingGroup.size > 0) {
-						// TODO Non null asserting, why is this not null?
-						// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
-						removedTrackingGroup.unlink(removedTrackingGroup.tracked[0]!);
+						removedTrackingGroup.unlink(removedTrackingGroup.tracked[0]);
 					}
@@ -142,0 +138,0 @@ }

tsconfig.json

@@ -9,4 +9,5 @@ {
 		"noImplicitAny": true,
+		"noUncheckedIndexedAccess": false,
 		"exactOptionalPropertyTypes": false,
 	},
 }

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

No v1

Quality

Package is not semver >=1. This means it is not stable and does not support ^ ranges.

Found 1 instance in 1 package

Improved metrics

Number of low quality alerts

0

decreased by-100%

Worsened metrics

Total package byte prevSize

663826

decreased by-1.08%

Lines of code

6961

decreased by-0.98%

Dependency changes

• + Added@fluid-internal/client-utils@2.3.1(transitive)

• + Added@fluidframework/container-definitions@2.3.1(transitive)

• + Added@fluidframework/container-runtime@2.3.1(transitive)

• + Added@fluidframework/container-runtime-definitions@2.3.1(transitive)

• + Added@fluidframework/core-interfaces@2.3.1(transitive)

• + Added@fluidframework/core-utils@2.3.1(transitive)

• + Added@fluidframework/datastore@2.3.1(transitive)

• + Added@fluidframework/datastore-definitions@2.3.1(transitive)

• + Added@fluidframework/driver-definitions@2.3.1(transitive)

• + Added@fluidframework/driver-utils@2.3.1(transitive)

• + Added@fluidframework/id-compressor@2.3.1(transitive)

• + Added@fluidframework/merge-tree@2.3.1(transitive)

• + Added@fluidframework/runtime-definitions@2.3.1(transitive)

• + Added@fluidframework/runtime-utils@2.3.1(transitive)

• + Added@fluidframework/shared-object-base@2.3.1(transitive)

• + Added@fluidframework/telemetry-utils@2.3.1(transitive)

• - Removed@fluid-internal/client-utils@2.3.0-288113(transitive)

• - Removed@fluidframework/container-definitions@2.3.0-288113(transitive)

• - Removed@fluidframework/container-runtime@2.3.0-288113(transitive)

• - Removed@fluidframework/container-runtime-definitions@2.3.0-288113(transitive)

• - Removed@fluidframework/core-interfaces@2.3.0-288113(transitive)

• - Removed@fluidframework/core-utils@2.3.0-288113(transitive)

• - Removed@fluidframework/datastore@2.3.0-288113(transitive)

• - Removed@fluidframework/datastore-definitions@2.3.0-288113(transitive)

• - Removed@fluidframework/driver-definitions@2.3.0-288113(transitive)

• - Removed@fluidframework/driver-utils@2.3.0-288113(transitive)

• - Removed@fluidframework/id-compressor@2.3.0-288113(transitive)

• - Removed@fluidframework/merge-tree@2.3.0-288113(transitive)

• - Removed@fluidframework/runtime-definitions@2.3.0-288113(transitive)

• - Removed@fluidframework/runtime-utils@2.3.0-288113(transitive)

• - Removed@fluidframework/shared-object-base@2.3.0-288113(transitive)

• - Removed@fluidframework/telemetry-utils@2.3.0-288113(transitive)

• Updated@fluid-internal/client-utils@~2.3.0

• Updated@fluidframework/core-interfaces@~2.3.0

• Updated@fluidframework/core-utils@~2.3.0

• Updated@fluidframework/datastore-definitions@~2.3.0

• Updated@fluidframework/driver-definitions@~2.3.0

• Updated@fluidframework/driver-utils@~2.3.0

• Updated@fluidframework/merge-tree@~2.3.0

• Updated@fluidframework/runtime-definitions@~2.3.0

• Updated@fluidframework/runtime-utils@~2.3.0

• Updated@fluidframework/shared-object-base@~2.3.0

• Updated@fluidframework/telemetry-utils@~2.3.0