[Fix] ensure throwing on a string in node < 4

Author: ljharb

implementation.js

@@ -10,8 +10,13 @@ var ToObject = require('es-object-atoms/ToObject');
 var ToString = require('es-abstract/2024/ToString');
 
 var $TypeError = require('es-errors/type');
+var isString = require('is-string');
 
 module.exports = function reverse() {
+	if (isString(this)) {
+		Set(String(this), 0, '', true);
+		throw new $TypeError('Cannot assign to read only property of ' + this);
+	}
 	var O = ToObject(this); // step 1
 	var len = LengthOfArrayLike(O); // step 2
 

package.json

@@ -55,7 +55,8 @@
 		"define-properties": "^1.2.1",
 		"es-abstract": "^1.23.9",
 		"es-errors": "^1.3.0",
-		"es-object-atoms": "^1.1.1"
+		"es-object-atoms": "^1.1.1",
+		"is-string": "^1.1.1"
 	},
 	"devDependencies": {
 		"@es-shims/api": "^2.5.1",

polyfill.js

@@ -3,5 +3,13 @@
 var implementation = require('./implementation');
 
 module.exports = function getPolyfill() {
-	return Array.prototype.reverse || implementation;
+	if (!Array.prototype.reverse) {
+		return implementation;
+	}
+	try {
+		Array.prototype.reverse.call('abc');
+		return implementation;
+	} catch (e) {
+		return Array.prototype.reverse;
+	}
 };