@endo/eventual-send 0.17.0 → 0.17.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@endo/eventual-send",
-  "version": "0.17.0",
+  "version": "0.17.2",
   "description": "Extend a Promise class to implement the eventual-send API",
   "type": "module",
   "main": "src/no-shim.js",
@@ -30,8 +30,8 @@
   },
   "homepage": "https://github.com/endojs/endo#readme",
   "devDependencies": {
-    "@endo/lockdown": "^0.1.26",
-    "@endo/ses-ava": "^0.2.38",
+    "@endo/lockdown": "^0.1.28",
+    "@endo/ses-ava": "^0.2.40",
     "ava": "^5.2.0",
     "c8": "^7.7.3",
     "tsd": "^0.24.1"
@@ -49,14 +49,9 @@
   ],
   "eslintConfig": {
     "extends": [
-      "@endo"
+      "plugin:@endo/internal"
     ]
   },
-  "prettier": {
-    "arrowParens": "avoid",
-    "trailingComma": "all",
-    "singleQuote": true
-  },
   "publishConfig": {
     "access": "public"
   },
@@ -66,5 +61,5 @@
     ],
     "timeout": "2m"
   },
-  "gitHead": "0053227a37ee76ee8a3dce6059dd55140ed0ce6d"
+  "gitHead": "38c2c59d6ae8c53f84cd333e6c7828e2d37604e2"
 }

package/src/handled-promise.js

@@ -508,7 +508,7 @@ export const makeHandledPromise = () => {
           return;
         }
         try {
-          resolve(trackedDoDispatch(handlerName, handler, o));
+          resolve(harden(trackedDoDispatch(handlerName, handler, o)));
         } catch (reason) {
           reject(harden(reason));
         }

package/src/index.test-d.ts

@@ -1,11 +1,11 @@
-/* eslint-disable @endo/no-polymorphic-call, import/no-extraneous-dependencies, no-restricted-globals, prettier/prettier */
+/* eslint-disable @endo/no-polymorphic-call, import/no-extraneous-dependencies, no-restricted-globals */
 import { expectType } from 'tsd';
 import { E } from '../test/get-hp.js';
 import { DataOnly, ERef } from './index.js';
 
 type FarRef<
   Primary,
-  Local = DataOnly<Primary>
+  Local = DataOnly<Primary>,
 > = import('@endo/eventual-send').FarRef<Primary, Local>;
 
 // Check the legacy ERef type

package/src/track-turns.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"track-turns.d.ts","sourceRoot":"","sources":["track-turns.js"],"names":[],"mappings":"AA+FO,mEAeN;;;;;uCAnCuB,GAAG,EAAE,KAAK,GAAG"}
+{"version":3,"file":"track-turns.d.ts","sourceRoot":"","sources":["track-turns.js"],"names":[],"mappings":"AAwGO,mEAeN;;;;;uCAnCuB,GAAG,EAAE,KAAK,GAAG"}

package/src/track-turns.js

@@ -1,6 +1,8 @@
 /* global globalThis */
 // @ts-nocheck
 
+const { freeze } = Object;
+
 // NOTE: We can't import these because they're not in scope before lockdown.
 // import { assert, details as X, Fail } from '@agoric/assert';
 
@@ -22,12 +24,19 @@ const env = (globalThis.process || {}).env || {};
 // Turn on if you seem to be losing error logging at the top of the event loop
 const VERBOSE = (env.DEBUG || '').split(':').includes('track-turns');
 
-// Track-turns is disabled by default and can be enabled by an environment
-// option. We intend to change the default after verifying that having
-// the feature enabled in production does not cause memory to leak.
-const ENABLED = env.TRACK_TURNS === 'enabled';
+const validOptionValues = freeze([undefined, 'enabled', 'disabled']);
+
+// Track-turns is enabled by default and can be disabled by an environment
+// option.
+const envOptionValue = env.TRACK_TURNS;
+if (!validOptionValues.includes(envOptionValue)) {
+  throw new TypeError(
+    `unrecognized TRACK_TURNS ${JSON.stringify(envOptionValue)}`,
+  );
+}
+const ENABLED = (envOptionValue || 'disabled') === 'enabled';
 
-// We hoist these functions out of trackTurns() to discourage the
+// We hoist the following functions out of trackTurns() to discourage the
 // closures from holding onto 'args' or 'func' longer than necessary,
 // which we've seen cause HandledPromise arguments to be retained for
 // a surprisingly long time.
@@ -66,7 +75,7 @@ const wrapFunction =
       // Must capture this now, not when the catch triggers.
       const detailsNote = X`Rejection from: ${hiddenPriorError}:${hiddenCurrentTurn}.${hiddenCurrentEvent}`;
       Promise.resolve(result).catch(addRejectionNote(detailsNote));
-      return harden(result);
+      return result;
     } finally {
       hiddenPriorError = undefined;
     }