@fluidframework/container-runtime - npm Package Compare versions

Comparing version 0.32.3 to 0.32.4

dist/containerRuntime.d.ts

@@ -100,3 +100,3 @@ /*!
     get IFluidRouter(): this;
-    readonly runtimeVersion = "0.32.3";
+    readonly runtimeVersion = "0.32.4";
     /**
@@ -103,0 +103,0 @@ * Load the stores from a snapshot and returns the runtime.

dist/packageVersion.d.ts

@@ -8,3 +8,3 @@ /*!
 export declare const pkgName = "@fluidframework/container-runtime";
-export declare const pkgVersion = "0.32.3";
+export declare const pkgVersion = "0.32.4";
 //# sourceMappingURL=packageVersion.d.ts.map

dist/packageVersion.js

@@ -10,3 +10,3 @@ "use strict";
 exports.pkgName = "@fluidframework/container-runtime";
-exports.pkgVersion = "0.32.3";
+exports.pkgVersion = "0.32.4";
 //# sourceMappingURL=packageVersion.js.map

dist/summaryCollection.js

@@ -212,3 +212,13 @@ "use strict";
         const summary = this.pendingSummaries.get(seq);
-        common_utils_1.assert(!!summary); // We should never see an ack without an op
+        if (!summary) {
+            // Summary ack without an op should be rare. We could fetch the
+            // reference sequence number from the snapshot, but instead we
+            // will not emit this ack. It should be the case that the summary
+            // op that this ack is for is earlier than this file was loaded
+            // from. i.e. initialSequenceNumber > summarySequenceNumber.
+            // We really don't care about it for now, since it is older than
+            // the one we loaded from.
+            common_utils_1.assert(this.initialSequenceNumber > seq, "Missing summary op for ack, but summary op seq > initialSequenceNumber");
+            return;
+        }
         summary.ackNack(op);
@@ -215,0 +225,0 @@ this.pendingSummaries.delete(seq);

lib/containerRuntime.d.ts

@@ -100,3 +100,3 @@ /*!
     get IFluidRouter(): this;
-    readonly runtimeVersion = "0.32.3";
+    readonly runtimeVersion = "0.32.4";
     /**
@@ -103,0 +103,0 @@ * Load the stores from a snapshot and returns the runtime.

lib/packageVersion.d.ts

@@ -8,3 +8,3 @@ /*!
 export declare const pkgName = "@fluidframework/container-runtime";
-export declare const pkgVersion = "0.32.3";
+export declare const pkgVersion = "0.32.4";
 //# sourceMappingURL=packageVersion.d.ts.map

lib/packageVersion.js

@@ -8,3 +8,3 @@ /*!
 export const pkgName = "@fluidframework/container-runtime";
-export const pkgVersion = "0.32.3";
+export const pkgVersion = "0.32.4";
 //# sourceMappingURL=packageVersion.js.map

lib/summaryCollection.js

@@ -210,3 +210,13 @@ /*!
         const summary = this.pendingSummaries.get(seq);
-        assert(!!summary); // We should never see an ack without an op
+        if (!summary) {
+            // Summary ack without an op should be rare. We could fetch the
+            // reference sequence number from the snapshot, but instead we
+            // will not emit this ack. It should be the case that the summary
+            // op that this ack is for is earlier than this file was loaded
+            // from. i.e. initialSequenceNumber > summarySequenceNumber.
+            // We really don't care about it for now, since it is older than
+            // the one we loaded from.
+            assert(this.initialSequenceNumber > seq, "Missing summary op for ack, but summary op seq > initialSequenceNumber");
+            return;
+        }
         summary.ackNack(op);
@@ -213,0 +223,0 @@ this.pendingSummaries.delete(seq);

package.json

 {
   "name": "@fluidframework/container-runtime",
-  "version": "0.32.3",
+  "version": "0.32.4",
   "description": "Fluid container runtime",
@@ -58,18 +58,18 @@ "homepage": "https://fluidframework.com",
   "dependencies": {
-    "@fluidframework/agent-scheduler": "^0.32.3",
+    "@fluidframework/agent-scheduler": "^0.32.4",
     "@fluidframework/common-definitions": "^0.19.1",
     "@fluidframework/common-utils": "^0.26.0",
-    "@fluidframework/container-definitions": "^0.32.3",
-    "@fluidframework/container-runtime-definitions": "^0.32.3",
-    "@fluidframework/container-utils": "^0.32.3",
-    "@fluidframework/core-interfaces": "^0.32.3",
-    "@fluidframework/datastore": "^0.32.3",
-    "@fluidframework/driver-definitions": "^0.32.3",
-    "@fluidframework/driver-utils": "^0.32.3",
-    "@fluidframework/garbage-collector": "^0.32.3",
+    "@fluidframework/container-definitions": "^0.32.4",
+    "@fluidframework/container-runtime-definitions": "^0.32.4",
+    "@fluidframework/container-utils": "^0.32.4",
+    "@fluidframework/core-interfaces": "^0.32.4",
+    "@fluidframework/datastore": "^0.32.4",
+    "@fluidframework/driver-definitions": "^0.32.4",
+    "@fluidframework/driver-utils": "^0.32.4",
+    "@fluidframework/garbage-collector": "^0.32.4",
     "@fluidframework/protocol-base": "^0.1017.1",
     "@fluidframework/protocol-definitions": "^0.1017.1",
-    "@fluidframework/runtime-definitions": "^0.32.3",
-    "@fluidframework/runtime-utils": "^0.32.3",
-    "@fluidframework/telemetry-utils": "^0.32.3",
+    "@fluidframework/runtime-definitions": "^0.32.4",
+    "@fluidframework/runtime-utils": "^0.32.4",
+    "@fluidframework/telemetry-utils": "^0.32.4",
     "assert": "^2.0.0",
@@ -83,4 +83,4 @@ "debug": "^4.1.1",
     "@fluidframework/eslint-config-fluid": "^0.21.0",
-    "@fluidframework/mocha-test-setup": "^0.32.3",
-    "@fluidframework/test-runtime-utils": "^0.32.3",
+    "@fluidframework/mocha-test-setup": "^0.32.4",
+    "@fluidframework/test-runtime-utils": "^0.32.4",
     "@microsoft/api-extractor": "^7.7.2",
@@ -87,0 +87,0 @@ "@types/assert": "^1.5.1",

src/packageVersion.ts

@@ -9,2 +9,2 @@ /*!
 export const pkgName = "@fluidframework/container-runtime";
-export const pkgVersion = "0.32.3";
+export const pkgVersion = "0.32.4";

src/summaryCollection.ts

@@ -301,3 +301,14 @@ /*!
         const summary = this.pendingSummaries.get(seq)!;
-        assert(!!summary); // We should never see an ack without an op
+        if (!summary) {
+            // Summary ack without an op should be rare. We could fetch the
+            // reference sequence number from the snapshot, but instead we
+            // will not emit this ack. It should be the case that the summary
+            // op that this ack is for is earlier than this file was loaded
+            // from. i.e. initialSequenceNumber > summarySequenceNumber.
+            // We really don't care about it for now, since it is older than
+            // the one we loaded from.
+            assert(this.initialSequenceNumber > seq,
+                "Missing summary op for ack, but summary op seq > initialSequenceNumber");
+            return;
+        }
         summary.ackNack(op);
@@ -304,0 +315,0 @@ this.pendingSummaries.delete(seq);

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

Improved metrics

Total package byte prevSize

1842142

increased by0.18%

Lines of code

19407

increased by0.16%

Dependency changes

• Updated@fluidframework/agent-scheduler@^0.32.4

• Updated@fluidframework/container-definitions@^0.32.4

• Updated@fluidframework/container-runtime-definitions@^0.32.4

• Updated@fluidframework/container-utils@^0.32.4

• Updated@fluidframework/core-interfaces@^0.32.4

• Updated@fluidframework/datastore@^0.32.4

• Updated@fluidframework/driver-definitions@^0.32.4

• Updated@fluidframework/driver-utils@^0.32.4

• Updated@fluidframework/garbage-collector@^0.32.4

• Updated@fluidframework/runtime-definitions@^0.32.4

• Updated@fluidframework/runtime-utils@^0.32.4

• Updated@fluidframework/telemetry-utils@^0.32.4