[Refactor] use es-define-property

inspect-js · Feb 13, 2024 · f93a8c8 · f93a8c8
1 parent 35e9b46
commit f93a8c8
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 15 deletions.
diff --git a/index.js b/index.js
@@ -1,25 +1,14 @@
 'use strict';
 
-var GetIntrinsic = require('get-intrinsic');
-
-var $defineProperty = GetIntrinsic('%Object.defineProperty%', true);
+var $defineProperty = require('es-define-property');
 
 var hasPropertyDescriptors = function hasPropertyDescriptors() {
-	if ($defineProperty) {
-		try {
-			$defineProperty({}, 'a', { value: 1 });
-			return true;
-		} catch (e) {
-			// IE 8 has a broken defineProperty
-			return false;
-		}
-	}
-	return false;
+	return !!$defineProperty;
 };
 
 hasPropertyDescriptors.hasArrayLengthDefineBug = function hasArrayLengthDefineBug() {
 	// node v0.6 has a bug where array lengths can be Set but not Defined
-	if (!hasPropertyDescriptors()) {
+	if (!$defineProperty) {
 		return null;
 	}
 	try {

diff --git a/package.json b/package.json
@@ -56,7 +56,7 @@
 		"tape": "^5.7.4"
 	},
 	"dependencies": {
-		"get-intrinsic": "^1.2.4"
+		"es-define-property": "^1.0.0"
 	},
 	"testling": {
 		"files": "test/index.js"