@agoric/marshal - npm Package Compare versions

Comparing version 0.5.1-dev-f0f3a75.0 to 0.5.1-dev-f43e88e.0

package.json

 {
   "name": "@agoric/marshal",
-  "version": "0.5.1-dev-f0f3a75.0+f0f3a75",
+  "version": "0.5.1-dev-f43e88e.0+f43e88e",
   "description": "marshal",
@@ -15,4 +15,2 @@ "type": "module",
     "test:xs": "exit 0",
-    "pretty-fix": "prettier --write '**/*.js'",
-    "pretty-check": "prettier --check '**/*.js'",
     "lint-fix": "yarn lint:eslint --fix && yarn lint:types",
@@ -38,8 +36,8 @@ "lint-check": "yarn lint",
   "dependencies": {
-    "@agoric/eventual-send": "0.14.1-dev-f0f3a75.0+f0f3a75",
+    "@agoric/eventual-send": "0.14.1-dev-f43e88e.0+f43e88e",
     "@agoric/nat": "^4.1.0",
-    "@agoric/promise-kit": "0.2.30-dev-f0f3a75.0+f0f3a75"
+    "@agoric/promise-kit": "0.2.30-dev-f43e88e.0+f43e88e"
   },
   "devDependencies": {
-    "@agoric/lockdown": "0.1.2-dev-f0f3a75.0+f0f3a75",
+    "@agoric/lockdown": "0.1.2-dev-f43e88e.0+f43e88e",
     "@endo/ses-ava": "^0.2.13",
@@ -59,6 +57,2 @@ "ava": "^3.12.1",
   },
-  "prettier": {
-    "trailingComma": "all",
-    "singleQuote": true
-  },
   "publishConfig": {
@@ -73,3 +67,3 @@ "access": "public"
   },
-  "gitHead": "f0f3a7548d591fe9fe27e247c09097c647bc076d"
+  "gitHead": "f43e88e14dd52f7cf89c453a1abecc1bc74c6b3b"
 }

src/deeplyFulfilled.js

 // @ts-check
 
-// eslint-disable-next-line spaced-comment
 /// <reference types="ses"/>
@@ -5,0 +4,0 @@

src/dot-membrane.js

 /* eslint-disable no-use-before-define */
 // @ts-check
 
-// eslint-disable-next-line spaced-comment
 /// <reference types="ses"/>
@@ -6,0 +5,0 @@

src/helpers/copyArray.js

 // @ts-check
 
-// eslint-disable-next-line spaced-comment
 /// <reference types="ses"/>
@@ -5,0 +4,0 @@

src/helpers/copyRecord.js

 // @ts-check
 
-// eslint-disable-next-line spaced-comment
 /// <reference types="ses"/>
@@ -5,0 +4,0 @@

src/helpers/error.js

 // @ts-check
 
-// eslint-disable-next-line spaced-comment
 /// <reference types="ses"/>
@@ -56,3 +55,3 @@
     const proto = getPrototypeOf(candidate);
-    const { name } = candidate;
+    const { name } = proto;
     const EC = getErrorConstructor(name);
@@ -67,2 +66,3 @@ if (!EC || EC.prototype !== proto) {
     const {
+      // Must allow `cause`, `errors`
       message: mDesc,
@@ -69,0 +69,0 @@ // Allow but ignore only extraneous own `stack` property.

src/helpers/internal-types.js

 // @ts-check
 
-// eslint-disable-next-line spaced-comment
 /// <reference path="../extra-types.d.ts" />
@@ -5,0 +4,0 @@

src/helpers/passStyle-helpers.js

 // @ts-check
 
-// eslint-disable-next-line spaced-comment
 /// <reference types="ses"/>
@@ -5,0 +4,0 @@

src/helpers/remotable.js

 // @ts-check
 
-// eslint-disable-next-line spaced-comment
 /// <reference types="ses"/>
@@ -5,0 +4,0 @@

src/helpers/tagged.js

 // @ts-check
 
-// eslint-disable-next-line spaced-comment
 /// <reference types="ses"/>
@@ -5,0 +4,0 @@

src/make-far.js

 // @ts-check
 
-// eslint-disable-next-line spaced-comment
 /// <reference types="ses"/>
@@ -5,0 +4,0 @@

src/makeTagged.js

 // @ts-check
 
-// eslint-disable-next-line spaced-comment
 /// <reference types="ses"/>
@@ -5,0 +4,0 @@

src/marshal-justin.js

 // @ts-check
 
-// eslint-disable-next-line spaced-comment
 /// <reference types="ses"/>
@@ -5,0 +4,0 @@

src/marshal.js

 // @ts-check
 
-// eslint-disable-next-line spaced-comment
 /// <reference types="ses"/>
@@ -81,2 +80,3 @@
       if (slotMap.has(val)) {
+        // TODO assert that it's the same iface as before
         slotIndex = slotMap.get(val);
@@ -93,2 +93,3 @@ assert.typeof(slotIndex, 'number');
 
+      // TODO explore removing this special case
       if (iface === undefined) {
@@ -128,2 +129,4 @@ return harden({
         marshalSaveError(err);
+        // Must encode `cause`, `errors`.
+        // nested non-passable errors must be ok from here.
         return harden({
@@ -285,2 +288,4 @@ [QCLASS]: 'error',
       }
+      // TODO SECURITY HAZARD: must enfoce that remotable vs promise
+      // is according to the encoded string.
       const slot = slots[Number(Nat(index))];
@@ -386,2 +391,3 @@ const val = convertSlotToVal(slot, iface);
           case 'error': {
+            // Must decode `cause` and `errors` properties
             const { name, message, errorId } = rawTree;
@@ -388,0 +394,0 @@ assert.typeof(

src/passStyleOf.js

 // @ts-check
 
-// eslint-disable-next-line spaced-comment
 /// <reference types="ses"/>
@@ -5,0 +4,0 @@

src/types.js

 // @ts-nocheck TODO Fix the recursive types to it checks. Will this
 // require a .d.ts file? I don't know.
 
-// eslint-disable-next-line spaced-comment
 /// <reference path="extra-types.d.ts" />
@@ -6,0 +5,0 @@

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

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

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

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

Worsened metrics

Total package byte prevSize

134674

decreased by-0.3%

Lines of code

2880

decreased by-0.24%

Dependency changes

• Updated@agoric/eventual-send@0.14.1-dev-f43e88e.0+f43e88e

• Updated@agoric/promise-kit@0.2.30-dev-f43e88e.0+f43e88e