fix: endow with original unstructured assert

packages/SwingSet/src/controller/controller.js

@@ -211,7 +211,10 @@ export async function makeSwingsetController(
     filePrefix: 'kernel/...',
     endowments: {
       console: makeConsole(`${debugPrefix}SwingSet:kernel`),
-      assert,
+      // This is the underlying unstructured assert that @endo/errors
+      // uses to build the structured assert that it exports. So we
+      // cannot use the assert imported from @endo/errors here.
+      assert: globalThis.assert,
       require: kernelRequire,
       URL: globalThis.Base64, // Unavailable only on XSnap
       Base64: globalThis.Base64, // Available only on XSnap

packages/SwingSet/src/kernel/kernel.js

@@ -1644,7 +1644,14 @@ export default function buildKernel(
       assert(bundle);
       const NS = await importBundle(bundle, {
         filePrefix: `dev-${name}/...`,
-        endowments: harden({ ...vatEndowments, console: devConsole, assert }),
+        endowments: harden({
+          ...vatEndowments,
+          console: devConsole,
+          // This is the underlying unstructured assert that @endo/errors
+          // uses to build the structured assert that it exports. So we
+          // cannot use the assert imported from @endo/errors here.
+          assert: globalThis.assert
+        }),
       });
 
       if (deviceEndowments[name] || unendowed) {

packages/SwingSet/src/kernel/vat-loader/manager-local.js

@@ -74,7 +74,10 @@ export function makeLocalVatManagerFactory({
     const workerEndowments = harden({
       ...vatEndowments,
       console: makeVatConsole(makeLogMaker('vat')),
-      assert,
+      // This is the underlying unstructured assert that @endo/errors
+      // uses to build the structured assert that it exports. So we
+      // cannot use the assert imported from @endo/errors here.
+      assert: globalThis.assert,
       TextEncoder,
       TextDecoder,
       Base64: globalThis.Base64, // Available only on XSnap

packages/SwingSet/src/supervisors/subprocess-node/supervisor-subprocess-node.js

@@ -139,7 +139,10 @@ function handleSetBundle(margs) {
   // Enable or disable the console accordingly.
   const workerEndowments = {
     console: makeVatConsole(makeLogMaker(`SwingSet:vat:${vatID}`)),
-    assert,
+    // This is the underlying unstructured assert that @endo/errors
+    // uses to build the structured assert that it exports. So we
+    // cannot use the assert imported from @endo/errors here.
+    assert: globalThis.assert,
   };
 
   async function buildVatNamespace(lsEndowments, inescapableGlobalProperties) {

packages/SwingSet/test/zcf-ish-upgrade/pseudo-zcf.js

@@ -1,11 +1,10 @@
 // @ts-nocheck
-/* global VatData */
+/* global globalThis, VatData */
 /* eslint-disable no-unused-vars */
 
 import { Far } from '@endo/far';
 import { importBundle } from '@endo/import-bundle';
 import { defineDurableKind } from '@agoric/vat-data';
-import { assert } from '@agoric/assert';
 import {
   provideHandle,
   provideBaggageSubset as provideBaggageSubTree,
@@ -24,7 +23,14 @@ export const buildRootObject = async (vatPowers, vatParameters, baggage) => {
   const { D } = vatPowers;
   const { contractBundleCap } = vatParameters;
   const contractBundle = D(contractBundleCap).getBundle();
-  const endowments = { console, assert, VatData };
+  const endowments = {
+    console,
+    // This is the underlying unstructured assert that @endo/errors
+    // uses to build the structured assert that it exports. So we
+    // cannot use the assert imported from @endo/errors here.
+    assert: globalThis.assert,
+    VatData,
+  };
   const contractNS = await importBundle(contractBundle, { endowments });
   const { setupInstallation, setup: _ } = contractNS;
   if (!setupInstallation) {

packages/cosmic-swingset/scripts/clean-core-eval.js

@@ -12,7 +12,10 @@ export const compartmentEvaluate = code => {
   const globals = harden({
     ...modules,
     ...farExports,
-    assert,
+    // This is the underlying unstructured assert that @endo/errors
+    // uses to build the structured assert that it exports. So we
+    // cannot use the assert imported from @endo/errors here.
+    assert: globalThis.assert,
     console: {
       // Ensure we don't pollute stdout.
       debug: console.warn,

packages/spawner/src/vat-spawned.js

@@ -6,7 +6,10 @@ import { Far } from '@endo/marshal';
 const endowments = {
   VatData,
   console,
-  assert,
+  // This is the underlying unstructured assert that @endo/errors
+  // uses to build the structured assert that it exports. So we
+  // cannot use the assert imported from @endo/errors here.
+  assert: globalThis.assert,
   Base64: globalThis.Base64, // Present only on XSnap
   URL: globalThis.URL, // Absent only on XSnap
 };

packages/swingset-xsnap-supervisor/lib/supervisor-subprocess-xsnap.js

@@ -255,7 +255,10 @@ function makeWorker(port) {
 
     const workerEndowments = {
       console: makeVatConsole(makeLogMaker('vat')),
-      assert,
+      // This is the underlying unstructured assert that @endo/errors
+      // uses to build the structured assert that it exports. So we
+      // cannot use the assert imported from @endo/errors here.
+      assert: globalThis.assert,
       // bootstrap provides HandledPromise
       HandledPromise: globalThis.HandledPromise,
       TextEncoder,

packages/vats/src/core/chain-behaviors.js

@@ -39,7 +39,10 @@ export const bridgeCoreEval = async allPowers => {
   const endowments = {
     VatData: globalThis.VatData,
     console,
-    assert,
+    // This is the underlying unstructured assert that @endo/errors
+    // uses to build the structured assert that it exports. So we
+    // cannot use the assert imported from @endo/errors here.
+    assert: globalThis.assert,
     Base64: globalThis.Base64, // Present only on XSnap
     URL: globalThis.URL, // Absent only on XSnap
   };

packages/vats/src/repl.js

@@ -194,7 +194,10 @@ export function getReplHandler(replObjects, send) {
     ...farExports,
     ...vowExports,
     E,
-    assert,
+    // This is the underlying unstructured assert that @endo/errors
+    // uses to build the structured assert that it exports. So we
+    // cannot use the assert imported from @endo/errors here.
+    assert: globalThis.assert,
     console: replConsole,
     commands,
     history,

packages/xsnap/src/avaAssertXS.js

@@ -228,7 +228,7 @@ function makeTester(htest, out) {
     fail(message) {
       assert(false, message);
     },
-    assert,
+    assert, // Not the SES assert
     truthy,
     /**
      * @param {unknown} value

packages/zoe/src/contractFacet/evalContractCode.js

@@ -4,7 +4,6 @@
 /* global globalThis */
 
 import { importBundle } from '@endo/import-bundle';
-import { assert } from '@agoric/assert';
 import { handlePWarning } from '../handleWarning.js';
 
 const evalContractBundle = (bundle, additionalEndowments = {}) => {
@@ -16,7 +15,10 @@ const evalContractBundle = (bundle, additionalEndowments = {}) => {
 
   const defaultEndowments = {
     console: louderConsole,
-    assert,
+    // This is the underlying unstructured assert that @endo/errors
+    // uses to build the structured assert that it exports. So we
+    // cannot use the assert imported from @endo/errors here.
+    assert: globalThis.assert,
     VatData: globalThis.VatData,
   };