own-keys 1.0.0 → 1.0.1

package/.eslintrc

@@ -4,6 +4,7 @@
 	"extends": "@ljharb",
 
 	"rules": {
+		"func-style": "off",
 		"id-length": "off",
 		"max-lines-per-function": "off",
 		"new-cap": ["error", {

package/CHANGELOG.md

@@ -5,6 +5,13 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/)
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [v1.0.1](https://github.com/ljharb/own-keys/compare/v1.0.0...v1.0.1) - 2024-12-29
+
+### Commits
+
+- [Tests] sort with a proper comparator [`4a65b56`](https://github.com/ljharb/own-keys/commit/4a65b569d10985032a0773806dcdec8866132baa)
+- [Fix] fix function name when `Reflect.ownKeys` is absent [`5cf17cb`](https://github.com/ljharb/own-keys/commit/5cf17cb1c9adfa836a1ddc467da4da20973db2ae)
+
 ## v1.0.0 - 2024-12-28
 
 ### Commits

package/index.js

@@ -11,11 +11,11 @@ var $gOPS = GetIntrinsic('%Object.getOwnPropertySymbols%', true);
 var keys = require('object-keys');
 
 /** @type {import('.')} */
-module.exports = $ownKeys || function OwnPropertyKeys(source) {
+module.exports = $ownKeys || function ownKeys(source) {
 	/** @type {(keyof typeof source)[]} */
-	var ownKeys = ($gOPN || keys)(source);
+	var sourceKeys = ($gOPN || keys)(source);
 	if ($gOPS) {
-		safePushApply(ownKeys, $gOPS(source));
+		safePushApply(sourceKeys, $gOPS(source));
 	}
-	return ownKeys;
+	return sourceKeys;
 };

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "own-keys",
-	"version": "1.0.0",
+	"version": "1.0.1",
 	"description": "Robustly get an object's own property keys (strings and symbols), including non-enumerables when possible",
 	"main": "index.js",
 	"exports": {

package/test/index.js

@@ -6,6 +6,17 @@ var hasPropertyDescriptors = require('has-property-descriptors')();
 
 var ownKeys = require('../');
 
+/** @type {(a: PropertyKey, b: PropertyKey) => number} */
+function comparator(a, b) {
+	if (typeof a === 'string' && typeof b === 'string') {
+		return a.localeCompare(b);
+	}
+	if (typeof a === 'number' && typeof b === 'number') {
+		return a - b;
+	}
+	return typeof a === 'symbol' ? 1 : -1;
+}
+
 test('ownKeys', function (t) {
 	t.equal(typeof ownKeys, 'function', 'is a function');
 	t.equal(
@@ -26,8 +37,8 @@ test('ownKeys', function (t) {
 		}
 
 		st.deepEqual(
-			ownKeys(obj).sort(),
-			(hasPropertyDescriptors ? ['a', 'b', 'c'] : ['a', 'b']).sort(),
+			ownKeys(obj).sort(comparator),
+			(hasPropertyDescriptors ? ['a', 'b', 'c'] : ['a', 'b']).sort(comparator),
 			'includes non-enumerable properties'
 		);
 
@@ -49,8 +60,8 @@ test('ownKeys', function (t) {
 		});
 
 		st.deepEqual(
-			ownKeys(obj).sort(),
-			['a', sym, nonEnumSym].sort(),
+			ownKeys(obj).sort(comparator),
+			['a', sym, nonEnumSym].sort(comparator),
 			'works with symbols, both enum and non-enum'
 		);