@agoric/kmarshal 0.1.1-upgrade-16-dev-0df76a7.0 → 0.1.1-upgrade-17-dev-3b97a9f.0

package/CHANGELOG.md

@@ -1,11 +0,0 @@
-# Change Log
-
-All notable changes to this project will be documented in this file.
-See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
-
-### 0.1.1-u16.0 (2024-07-02)
-
-
-### Features
-
-* **SwingSet:** Add a tool for classifying unsettled promises ([feb1f93](https://github.com/Agoric/agoric-sdk/commit/feb1f93c81e68c680e07bdac8c64917ad69af602)), closes [#8336](https://github.com/Agoric/agoric-sdk/issues/8336)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@agoric/kmarshal",
-  "version": "0.1.1-upgrade-16-dev-0df76a7.0+0df76a7",
+  "version": "0.1.1-upgrade-17-dev-3b97a9f.0+3b97a9f",
   "description": "Token-only marshaller for kernel and tests",
   "type": "module",
   "main": "./src/kmarshal.js",
@@ -21,9 +21,9 @@
     "lint:eslint": "eslint ."
   },
   "dependencies": {
-    "@agoric/assert": "0.6.1-upgrade-16-dev-0df76a7.0+0df76a7",
-    "@endo/far": "^1.1.2",
-    "@endo/marshal": "^1.5.0"
+    "@endo/errors": "^1.2.5",
+    "@endo/far": "^1.1.5",
+    "@endo/marshal": "^1.5.3"
   },
   "devDependencies": {
     "ava": "^5.3.0"
@@ -40,5 +40,5 @@
     ],
     "timeout": "2m"
   },
-  "gitHead": "0df76a71058eda04cdc75a54fb4312d286f323a3"
+  "gitHead": "3b97a9f0bc7d5dcd24d20dd81ca2af5d59d6e43d"
 }

package/src/kmarshal.js

@@ -1,9 +1,8 @@
+import { assert, Fail } from '@endo/errors';
 import { Far, passStyleOf } from '@endo/far';
 import { makeMarshal } from '@endo/marshal';
-import { assert, Fail } from '@agoric/assert';
 
 /**
- * @import { ERef } from '@endo/far';
  * @import {ConvertSlotToVal} from '@endo/marshal';
  */
 
@@ -19,7 +18,7 @@ import { assert, Fail } from '@agoric/assert';
 
 const { toStringTag } = Symbol;
 
-const makeStringStandinPromise = kref => {
+const makeStandinPromise = kref => {
   const p = Promise.resolve(`${kref} stand in`);
   // eslint-disable-next-line @typescript-eslint/no-floating-promises
   Object.defineProperty(p, toStringTag, {
@@ -28,8 +27,9 @@ const makeStringStandinPromise = kref => {
   });
   return harden(p);
 };
+harden(makeStandinPromise);
 
-const getStringStandinPromiseTag = p => {
+const getStandinPromiseTag = p => {
   const desc = Object.getOwnPropertyDescriptor(p, toStringTag);
   assert(desc !== undefined, 'promise lacks own @@toStringTag property');
   const kref = desc.value;
@@ -37,69 +37,6 @@ const getStringStandinPromiseTag = p => {
   return kref;
 };
 
-const makeAccessorStandinPromise = kref => {
-  // TODO This Bizarro world hack is only for the version of Endo that
-  // agoric-sdk currently depends on, and is already inconsistent with
-  // the "current" endo. Once agoric-sdk depends only on the next endo,
-  // we should delete this code, and rename `makeStringStandinPromise`
-  // above to `makeStandinPromise`.
-  //
-  // Bizarro World hack for attaching a string property to a Promise, courtesy
-  // of MarkM.  Even though the @@toStringTag property nominally *is* a
-  // string, some unfortunate stuff in our hardened JS safety checks blows up
-  // if it actually is.  Eventually that will be fixed and we'll be able to
-  // use the @@toStringTag property directly, but for now we need to use this
-  // weird construct employing a sneaky getter function.  Note that this is
-  // only necessary in the first place because smallcaps encodes promise
-  // references differently from remotable object references, and the current
-  // version of the smallcaps decoder annoyingly insists that if the encoded
-  // body string says a slot is a promise, then convertSlotToVal had better by
-  // damn return an actual Promise, even if, as in the intended use case here,
-  // we neither want nor need a promise nor are capable of making any use of
-  // the fact that it is one.
-  const p = Promise.resolve(`${kref} stand in`);
-  // Bizarro World makes eslint hallucinate
-  // eslint-disable-next-line @typescript-eslint/no-floating-promises
-  Object.defineProperty(p, toStringTag, {
-    get: reveal => (reveal ? kref : NaN),
-    enumerable: false,
-  });
-  return harden(p);
-};
-
-const getAccessorStandinPromiseTag = p => {
-  // Other half of Bizarro World hack for handling promises.  Note the
-  // peculiar way the @@toStringTag getter is used.
-  const desc = Object.getOwnPropertyDescriptor(p, toStringTag);
-  assert(desc !== undefined, 'promise lacks toStringTag getter');
-
-  const getter = desc.get;
-  assert.typeof(getter, 'function', 'toStringTag getter is not a function');
-  // @ts-expect-error violates the norm that getters have zero parameters
-  const kref = getter(true);
-  assert.typeof(kref, 'string');
-  return kref;
-};
-
-const [makeStandinPromise, getStandinPromiseTag] = (() => {
-  // Use whatever works
-  try {
-    const p = makeStringStandinPromise('string mascot');
-    assert(passStyleOf(p) === 'promise');
-    return [makeStringStandinPromise, getStringStandinPromiseTag];
-  } catch (_1) {
-    try {
-      const p = makeAccessorStandinPromise('accessor mascot');
-      assert(passStyleOf(p) === 'promise');
-      return [makeAccessorStandinPromise, getAccessorStandinPromiseTag];
-    } catch (_2) {
-      throw Error('One of the promise tagging schemes should have worked');
-    }
-  }
-})();
-export { makeStandinPromise };
-harden(makeStandinPromise);
-
 /**
  * @type {ConvertSlotToVal<string>}
  */