Stop enabling O.p.constructor to be overridden by assignment.

packages/ses/src/enablements.js

@@ -55,10 +55,24 @@
  *     leads to another record explaining which properties {@code
  *     Function.prototype} need to be repaired.
  */
-
 export const moderateEnablements = {
   '%ObjectPrototype%': {
-    constructor: true, // set by "acorn" v7.
+    // Acorn 7 does override `constructor` by assignment, but
+    // this is fixed as of acorn 8. Including the commented out
+    // line below in this list confuses the Node console.
+    // See https://github.com/Agoric/agoric-sdk/issues/2324
+    //
+    // So please update all
+    // acorn dependencies to at least 8 instead. We are unable to do
+    // so at this time due to a dependency via rollup. Instead we
+    // do a post-install patch of acorn.
+    // See https://github.com/Agoric/SES-shim/pull/588
+    // If you are similarly stuck, do likewise. Or uncomment out
+    // the following line and let us know why. The only known
+    // cost is the ugly display from the Node console.
+    //
+    // constructor: true, // set by acorn 7
+
     hasOwnProperty: true, // set by "vega-util".
     toString: true,
     valueOf: true,

packages/ses/test/test-enable-property-overrides.js

@@ -91,12 +91,7 @@ test('enablePropertyOverrides - on', t => {
 
   harden(intrinsics);
 
-  testOverriding(t, 'Object', {}, [
-    'constructor',
-    'hasOwnProperty',
-    'toString',
-    'valueOf',
-  ]);
+  testOverriding(t, 'Object', {}, ['hasOwnProperty', 'toString', 'valueOf']);
   // We allow 'length' *not* because it is in enablements; it is not;
   // but because each array instance has its own.
   testOverriding(t, 'Array', [], ['toString', 'length']);