fix(ses): ses depends on env-options

endojs · Jun 1, 2023 · ca3ffd1 · ca3ffd1
1 parent d1d7c9b
commit ca3ffd1
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 119 deletions.
diff --git a/packages/env-options/NEWS.md b/packages/env-options/NEWS.md
diff --git a/packages/ses/package.json b/packages/ses/package.json
@@ -59,6 +59,9 @@
     "test": "tsd && ava",
     "test:platform-compatibility": "node test/package/test.cjs"
   },
+  "dependencies": {
+    "@endo/env-options": "^0.1.0"
+  },
   "devDependencies": {
     "@endo/compartment-mapper": "^0.8.4",
     "@endo/static-module-record": "^0.7.19",

diff --git a/packages/ses/src/environment-options.js b/packages/ses/src/environment-options.js
diff --git a/packages/ses/src/lockdown-shim.js b/packages/ses/src/lockdown-shim.js
@@ -14,6 +14,7 @@
 
 // @ts-check
 
+import { makeEnvironmentCaptor } from '@endo/env-options';
 import {
   FERAL_FUNCTION,
   FERAL_EVAL,
@@ -49,7 +50,6 @@ import { tameDomains } from './tame-domains.js';
 import { tameConsole } from './error/tame-console.js';
 import tameErrorConstructor from './error/tame-error-constructor.js';
 import { assert, makeAssert } from './error/assert.js';
-import { makeEnvironmentCaptor } from './environment-options.js';
 import { getAnonymousIntrinsics } from './get-anonymous-intrinsics.js';
 import { makeCompartmentConstructor } from './compartment-shim.js';
 import { tameHarden } from './tame-harden.js';
@@ -297,7 +297,6 @@ export const repairIntrinsics = (options = {}) => {
     optGetStackString = intrinsics['%InitialGetStackString%'];
   }
   const consoleRecord = tameConsole(
-    // @ts-expect-error tameConsole does its own input validation
     consoleTaming,
     errorTrapping,
     unhandledRejectionTrapping,