[Fix] do not crash when --disable-proto=throw

.github/workflows/node-noproto.yml

@@ -0,0 +1,25 @@
+name: 'Tests: node --disable-proto=throw'
+
+on: [pull_request, push]
+
+jobs:
+  throw:
+    uses: ljharb/actions/.github/workflows/node.yml@main
+    with:
+      range: '>= 12.17'
+      type: minors
+      command: "NODE_OPTIONS='--disable-proto=throw' npx tape 'tests/**/*.js'"
+
+  delete:
+    uses: ljharb/actions/.github/workflows/node.yml@main
+    with:
+      range: '>= 12.17'
+      type: minors
+      command: "NODE_OPTIONS='--disable-proto=delete' npx tape 'tests/**/*.js'"
+
+  node:
+    name: 'node --disable-proto'
+    needs: [throw, delete]
+    runs-on: ubuntu-latest
+    steps:
+      - run: true

.github/workflows/node-twenties.yml

@@ -13,32 +13,8 @@ jobs:
       type: minors
       command: npm run tests-only
 
-  no-proto:
-    name: 'node, --disable-proto=delete'
-    runs-on: ubuntu-latest
-    steps:
-      - name: Harden Runner
-        uses: step-security/harden-runner@v2
-        with:
-          disable-sudo: true
-          egress-policy: audit
-          allowed-endpoints: >
-            github.com:443
-            raw.githubusercontent.com:443
-            nodejs.org:443
-            iojs.org:443
-            registry.npmjs.org:443
-            actions-results-receiver-production.githubapp.com:443
-
-      - uses: actions/checkout@v4
-        with:
-          show-progress: false
-      - uses: ljharb/actions/node/install@main
-      - run: NODE_OPTIONS=--disable-proto=delete npm run tests-only
-      - uses: codecov/codecov-action@v3.1.5
-
   finisher:
-    needs: [tests, no-proto]
+    needs: tests
     name: 'node >= 20'
     runs-on: ubuntu-latest
     steps:

get.js

@@ -3,11 +3,18 @@
 var callBind = require('call-bind-apply-helpers');
 var gOPD = require('gopd');
 
-// eslint-disable-next-line no-extra-parens, no-proto
-var hasProtoAccessor = /** @type {{ __proto__?: typeof Array.prototype }} */ ([]).__proto__ === Array.prototype;
+var hasProtoAccessor;
+try {
+	// eslint-disable-next-line no-extra-parens, no-proto
+	hasProtoAccessor = /** @type {{ __proto__?: typeof Array.prototype }} */ ([]).__proto__ === Array.prototype;
+} catch (e) {
+	if (!e || typeof e !== 'object' || !('code' in e) || e.code !== 'ERR_PROTO_ACCESS') {
+		throw e;
+	}
+}
 
 // eslint-disable-next-line no-extra-parens
-var desc = hasProtoAccessor && gOPD && gOPD(Object.prototype, /** @type {keyof typeof Object.prototype} */ ('__proto__'));
+var desc = !!hasProtoAccessor && gOPD && gOPD(Object.prototype, /** @type {keyof typeof Object.prototype} */ ('__proto__'));
 
 var $Object = Object;
 var $getPrototypeOf = $Object.getPrototypeOf;

set.js

@@ -6,7 +6,13 @@ var $TypeError = require('es-errors/type');
 
 /** @type {{ __proto__?: object | null }} */
 var obj = {};
-obj.__proto__ = null; // eslint-disable-line no-proto
+try {
+	obj.__proto__ = null; // eslint-disable-line no-proto
+} catch (e) {
+	if (!e || typeof e !== 'object' || !('code' in e) || e.code !== 'ERR_PROTO_ACCESS') {
+		throw e;
+	}
+}
 
 var hasProtoMutator = !('toString' in obj);
 

test/set.js

@@ -35,7 +35,16 @@ test('setDunderProto', { skip: !setDunderProto }, function (t) {
 });
 
 test('no dunder proto', { skip: !!setDunderProto }, function (t) {
-	t.notOk('__proto__' in Object.prototype, 'no __proto__ in Object.prototype');
+	if ('__proto__' in Object.prototype) {
+		t['throws'](
+			// @ts-expect-error
+			function () { ({}).__proto__ = null; }, // eslint-disable-line no-proto
+			Error,
+			'throws when setting Object.prototype.__proto__'
+		);
+	} else {
+		t.notOk('__proto__' in Object.prototype, 'no __proto__ in Object.prototype');
+	}
 
 	t.end();
 });