fixup! make shim opt-in via process.env

Author: erights

packages/non-trapping-shim/package.json

@@ -20,6 +20,7 @@
   "module": "./index.js",
   "exports": {
     "./shim.js": "./shim.js",
+    "./prepare-enable-shim.js": "./prepare-enable-shim.js",
     "./package.json": "./package.json"
   },
   "scripts": {
@@ -35,6 +36,9 @@
     "test:c8": "c8 $C8_OPTIONS ava --config=ava-nesm.config.js",
     "test:xs": "exit 0"
   },
+  "dependencies": {
+    "@endo/env-options": "^1.1.8"
+  },
   "devDependencies": {
     "ava": "^6.1.3",
     "c8": "^7.14.0",

packages/non-trapping-shim/prepare-enable-shim.js

@@ -0,0 +1,7 @@
+/* global globalThis */
+
+// TODO consider adding env option setting APIs to @endo/env-options
+// TODO should set up globalThis.process.env if absent
+const env = (globalThis.process || {}).env || {};
+
+env.SES_NON_TRAPPING_SHIM = 'enabled';

packages/non-trapping-shim/shim.js

@@ -1,10 +1,22 @@
 /* global globalThis */
+import { getEnvironmentOption } from '@endo/env-options';
 import { ReflectPlus, ObjectPlus, ProxyPlus } from './src/non-trapping-pony.js';
 
-globalThis.Reflect = ReflectPlus;
+const nonTrappingShimOption = getEnvironmentOption(
+  'SES_NON_TRAPPING_SHIM',
+  'disabled',
+  ['enabled'],
+);
 
-globalThis.Object = ObjectPlus;
-// eslint-disable-next-line no-extend-native
-Object.prototype.constructor = ObjectPlus;
+if (nonTrappingShimOption === 'enabled') {
+  // TODO figure this out, either remove directive or change to
+  // at-ts-expect-error.
+  // @ts-ignore type of ReflectPlus vs Reflect, I think
+  globalThis.Reflect = ReflectPlus;
 
-globalThis.Proxy = ProxyPlus;
+  globalThis.Object = ObjectPlus;
+  // eslint-disable-next-line no-extend-native
+  Object.prototype.constructor = ObjectPlus;
+
+  globalThis.Proxy = ProxyPlus;
+}

packages/non-trapping-shim/test/non-trapping-shim.test.js

@@ -1,3 +1,4 @@
+import '../prepare-enable-shim.js';
 // Uses 'ava' rather than @endo/ses-ava to avoid worries about cyclic
 // dependencies. We will need similar tests is higher level packages, in order
 // to test compat with ses and ses-ava.

yarn.lock

@@ -403,7 +403,7 @@ __metadata:
   languageName: unknown
   linkType: soft
 
-"@endo/env-options@workspace:^, @endo/env-options@workspace:packages/env-options":
+"@endo/env-options@npm:^1.1.8, @endo/env-options@workspace:^, @endo/env-options@workspace:packages/env-options":
   version: 0.0.0-use.local
   resolution: "@endo/env-options@workspace:packages/env-options"
   dependencies:
@@ -706,6 +706,7 @@ __metadata:
   version: 0.0.0-use.local
   resolution: "@endo/non-trapping-shim@workspace:packages/non-trapping-shim"
   dependencies:
+    "@endo/env-options": "npm:^1.1.8"
     ava: "npm:^6.1.3"
     c8: "npm:^7.14.0"
     tsd: "npm:^0.31.2"