@agoric/swingset-liveslots - npm Package Compare versions

Comparing version 0.10.3-dev-d4f2864.0 to 0.10.3-dev-d504b42.0

package.json

 {
   "name": "@agoric/swingset-liveslots",
-  "version": "0.10.3-dev-d4f2864.0+d4f2864",
+  "version": "0.10.3-dev-d504b42.0+d504b42",
   "description": "SwingSet ocap support layer",
@@ -17,27 +17,28 @@ "type": "module",
     "lint:types": "tsc",
-    "lint:eslint": "eslint ."
+    "lint:eslint": "eslint .",
+    "prepack": "tsc --build tsconfig.build.json",
+    "postpack": "git clean -f ':!src/types-index.d.ts' '*.d.ts*' '*.tsbuildinfo'"
   },
   "dependencies": {
-    "@agoric/internal": "0.3.3-dev-d4f2864.0+d4f2864",
-    "@agoric/store": "0.9.3-dev-d4f2864.0+d4f2864",
-    "@endo/env-options": "^1.1.7",
-    "@endo/errors": "^1.2.7",
-    "@endo/eventual-send": "^1.2.7",
-    "@endo/exo": "^1.5.6",
-    "@endo/far": "^1.1.8",
-    "@endo/init": "^1.1.6",
-    "@endo/marshal": "^1.6.1",
-    "@endo/nat": "^5.0.12",
-    "@endo/pass-style": "^1.4.6",
-    "@endo/patterns": "^1.4.6",
-    "@endo/promise-kit": "^1.1.7"
+    "@agoric/internal": "0.3.3-dev-d504b42.0+d504b42",
+    "@agoric/store": "0.9.3-dev-d504b42.0+d504b42",
+    "@endo/env-options": "^1.1.8",
+    "@endo/errors": "^1.2.9",
+    "@endo/eventual-send": "^1.3.0",
+    "@endo/exo": "^1.5.8",
+    "@endo/far": "^1.1.10",
+    "@endo/init": "^1.1.8",
+    "@endo/marshal": "^1.6.3",
+    "@endo/nat": "^5.0.14",
+    "@endo/pass-style": "^1.4.8",
+    "@endo/patterns": "^1.4.8",
+    "@endo/promise-kit": "^1.1.9"
   },
   "devDependencies": {
-    "@agoric/kmarshal": "0.1.1-dev-d4f2864.0+d4f2864",
+    "@agoric/kmarshal": "0.1.1-dev-d504b42.0+d504b42",
     "ava": "^5.3.0"
   },
   "files": [
-    "src/**/*.js",
-    "src/**/*.d.ts",
-    "test/**/*.js",
+    "src",
+    "test",
     "tools",
@@ -63,4 +64,3 @@ "exported.js"
     ],
-    "timeout": "20m",
-    "workerThreads": false
+    "timeout": "20m"
   },
@@ -71,5 +71,5 @@ "publishConfig": {
   "typeCoverage": {
-    "atLeast": 75.24
+    "atLeast": 75.2
   },
-  "gitHead": "d4f2864a838a177fb49e3c93dd8cc3fbb25527cb"
+  "gitHead": "d504b42c486c01fb644a150da4824199c361adb7"
 }

src/collectionManager.js

@@ -149,3 +149,2 @@ import { assert, q, Fail } from '@endo/errors';
       kindID: 0,
-      // eslint-disable-next-line no-use-before-define
       reanimator: reanimateMapStore,
@@ -157,3 +156,2 @@ durable: false,
       kindID: 0,
-      // eslint-disable-next-line no-use-before-define
       reanimator: reanimateWeakMapStore,
@@ -165,3 +163,2 @@ durable: false,
       kindID: 0,
-      // eslint-disable-next-line no-use-before-define
       reanimator: reanimateSetStore,
@@ -173,3 +170,2 @@ durable: false,
       kindID: 0,
-      // eslint-disable-next-line no-use-before-define
       reanimator: reanimateWeakSetStore,
@@ -181,3 +177,2 @@ durable: false,
       kindID: 0,
-      // eslint-disable-next-line no-use-before-define
       reanimator: reanimateMapStore,
@@ -189,3 +184,2 @@ durable: true,
       kindID: 0,
-      // eslint-disable-next-line no-use-before-define
       reanimator: reanimateWeakMapStore,
@@ -197,3 +191,2 @@ durable: true,
       kindID: 0,
-      // eslint-disable-next-line no-use-before-define
       reanimator: reanimateSetStore,
@@ -205,3 +198,2 @@ durable: true,
       kindID: 0,
-      // eslint-disable-next-line no-use-before-define
       reanimator: reanimateWeakSetStore,
@@ -229,3 +221,2 @@ durable: true,
         storeKindInfo[kind].reanimator,
-        // eslint-disable-next-line no-use-before-define
         deleteCollection,
@@ -325,3 +316,2 @@ storeKindInfo[kind].durable,
     const encodeRemotable = remotable => {
-      // eslint-disable-next-line no-use-before-define
       const ordinal = getOrdinal(remotable);
@@ -328,0 +318,0 @@ ordinal !== undefined || Fail`no ordinal for ${remotable}`;

src/liveslots.js

@@ -158,6 +158,4 @@ import { annotateError, assert, Fail, makeError, X } from '@endo/errors';
       if (virtual || durable) {
-        // eslint-disable-next-line no-use-before-define
         vrm.setExportStatus(vref, 'reachable');
       } else {
-        // eslint-disable-next-line no-use-before-define
         const remotable = requiredValForSlot(vref);
@@ -187,3 +185,2 @@ exportedRemotables.add(remotable);
       // we're in the COLLECTED state, or FINALIZED after a re-introduction
-      // eslint-disable-next-line no-use-before-define
       addToPossiblyDeadSet(baseRef);
@@ -214,7 +211,5 @@ slotToVal.delete(baseRef);
         if (disavowedPresences.has(o)) {
-          // eslint-disable-next-line no-use-before-define
           exitVatWithFailure(disavowalError);
           throw disavowalError;
         }
-        // eslint-disable-next-line no-use-before-define
         return queueMessage(slot, prop, args, returnedP);
@@ -228,3 +223,2 @@ },
         if (disavowedPresences.has(o)) {
-          // eslint-disable-next-line no-use-before-define
           exitVatWithFailure(disavowalError);
@@ -301,3 +295,2 @@ throw disavowalError;
         }
-        // eslint-disable-next-line no-use-before-define
         return queueMessage(vpid, prop, args, returnedP);
@@ -355,3 +348,2 @@ },
   function allocateExportID() {
-    // eslint-disable-next-line no-use-before-define
     return vrm.allocateNextID('exportID');
@@ -361,3 +353,2 @@ }
   function allocateCollectionID() {
-    // eslint-disable-next-line no-use-before-define
     return vrm.allocateNextID('collectionID');
@@ -367,3 +358,2 @@ }
   function allocatePromiseID() {
-    // eslint-disable-next-line no-use-before-define
     const promiseID = vrm.allocateNextID('promiseID');
@@ -396,6 +386,4 @@ return makeVatSlot('promise', true, promiseID);
       // kernel isn't surprised by a spurious resolution.
-      // eslint-disable-next-line no-use-before-define
       const p = requiredValForSlot(vpid);
       // if (!knownResolutions.has(p)) { // TODO really?
-      // eslint-disable-next-line no-use-before-define
       followForKernel(vpid, p);
@@ -412,3 +400,2 @@ return true;
 
-  // eslint-disable-next-line no-use-before-define
   const m = makeMarshal(convertValToSlot, convertSlotToVal, {
@@ -426,3 +413,2 @@ marshalName: `liveSlots:${forVatID}`,
   const unmeteredUnserialize = meterControl.unmetered(m.unserialize);
-  // eslint-disable-next-line no-use-before-define
   const unmeteredConvertSlotToVal = meterControl.unmetered(convertSlotToVal);
@@ -472,3 +458,2 @@
     requiredValForSlot,
-    // eslint-disable-next-line no-use-before-define
     registerValue,
@@ -486,6 +471,4 @@ m.serialize,
     allocateCollectionID,
-    // eslint-disable-next-line no-use-before-define
     convertValToSlot,
     unmeteredConvertSlotToVal,
-    // eslint-disable-next-line no-use-before-define
     registerValue,
@@ -502,3 +485,2 @@ m.serialize,
     collectionManager,
-    // eslint-disable-next-line no-use-before-define
     convertValToSlot,
@@ -533,3 +515,2 @@ convertSlotToVal: unmeteredConvertSlotToVal,
         if (disavowedPresences.has(val)) {
-          // eslint-disable-next-line no-use-before-define
           exitVatWithFailure(disavowalError);
@@ -704,3 +685,2 @@ throw disavowalError; // cannot reference a disavowed object
             const [priorRejected, priorRes] = priorResolution;
-            // eslint-disable-next-line no-use-before-define
             collect(slot, priorRejected, priorRes);
@@ -991,3 +971,2 @@ }
       // exported promise
-      // eslint-disable-next-line no-use-before-define
       followForKernel(resultVPID, p);
@@ -1269,3 +1248,2 @@ }
     valToSlot,
-    // eslint-disable-next-line no-use-before-define
     afterDispatchActions,
@@ -1362,3 +1340,3 @@ });
    * @param {import('./types.js').VatDeliveryObject} delivery
-   * @returns {void | Promise<void>}
+   * @returns {undefined | ReturnType<startVat>}
    */
@@ -1365,0 +1343,0 @@ function dispatchToUserspace(delivery) {

src/virtualObjectManager.js

 /* global globalThis */
-/* eslint-disable no-use-before-define, jsdoc/require-returns-type */
+/* eslint-disable jsdoc/require-returns-type */
 
@@ -4,0 +4,0 @@ import { environmentOptionsListHas } from '@endo/env-options';

src/virtualReferences.js

@@ -1,2 +0,2 @@
-/* eslint-disable no-use-before-define, jsdoc/require-returns-type */
+/* eslint-disable jsdoc/require-returns-type */
 
@@ -3,0 +3,0 @@ import { assert, Fail } from '@endo/errors';

src/watchedPromises.js

@@ -238,2 +238,15 @@ // @ts-check
 
+        promiseRegistrations.init(vpid, p);
+
+        // pseudoThen registers a settlement callback that will remove
+        // this promise from promiseRegistrations and
+        // watchedPromiseTable. To avoid triggering
+        // https://github.com/Agoric/agoric-sdk/issues/10757 and
+        // preventing slotToVal cleanup, the `pseudoThen()` should
+        // precede `maybeExportPromise()`. This isn't foolproof, but
+        // does mitigate in advance of a proper fix. See #10756 for
+        // details of this particular mitigation, and #10757 for the
+        // deeper bug.
+        pseudoThen(p, vpid);
+
         // Ensure that this vat's promises are rejected at termination.
@@ -243,5 +256,2 @@ if (maybeExportPromise(vpid)) {
         }
-
-        promiseRegistrations.init(vpid, p);
-        pseudoThen(p, vpid);
       }
@@ -248,0 +258,0 @@ });

tools/fakeVirtualSupport.js

@@ -237,3 +237,2 @@ /* global globalThis */
     }
-    // eslint-disable-next-line no-use-before-define
     registerEntry(baseRef, val, facet !== undefined);
@@ -240,0 +239,0 @@ if (!result) {

New alerts

Manifest confusion

Supply chain risk

This package has inconsistent metadata. This could be malicious or caused by an error when publishing the package.

Found 1 instance in 1 package

Fixed alerts

Major refactor

Supply chain risk

Package has recently undergone a major refactor. It may be unstable or indicate significant internal changes. Use caution when updating to versions that include significant changes.

Found 1 instance in 1 package

Manifest confusion

Supply chain risk

This package has inconsistent metadata. This could be malicious or caused by an error when publishing the package.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

850421

increased by16.74%

Number of package files

141

increased by83.12%

Lines of code

20045

increased by8.06%

Number of medium supply chain risk alerts

1

decreased by-50%

Dependency changes

• Updated@agoric/internal@0.3.3-dev-d504b42.0+d504b42

• Updated@agoric/store@0.9.3-dev-d504b42.0+d504b42

• Updated@endo/env-options@^1.1.8

• Updated@endo/errors@^1.2.9

• Updated@endo/eventual-send@^1.3.0

• Updated@endo/exo@^1.5.8

• Updated@endo/far@^1.1.10

• Updated@endo/init@^1.1.8

• Updated@endo/marshal@^1.6.3

• Updated@endo/nat@^5.0.14

• Updated@endo/pass-style@^1.4.8

• Updated@endo/patterns@^1.4.8

• Updated@endo/promise-kit@^1.1.9