Bugfix: use Object.prototype for correct usage of hasOwnProperty (#1304)

* fix(#1303): use Object.prototype for correct usage of hasOwnProperty

* bugfix(#1303): use hasOwn function on Object + test coverage

---------

Co-authored-by: Jared Wray <me@jaredwray.com>

Author: nolliebigspin

packages/serialize/src/index.ts

@@ -28,8 +28,8 @@ export const defaultSerialize = (data: any): string => {
 		// eslint-disable-next-line guard-for-in
 		for (const k in data) {
 			const ignore = typeof data[k] === 'function' || (!array && data[k] === undefined);
-			// eslint-disable-next-line no-prototype-builtins
-			if (!data.hasOwnProperty(k) || ignore) {
+			// eslint-disable-next-line @typescript-eslint/no-unsafe-argument
+			if (!Object.hasOwn(data, k) || ignore) {
 				continue;
 			}
 

packages/serialize/test/test.ts

@@ -71,6 +71,14 @@ test.it('defaultSerialize detects base64 on string', t => {
 	t.expect(result.encoded.toString()).toBe('hello world');
 });
 
+test.it('defaultSerialize accepts objects created with null', t => {
+	const json = Object.create(null);
+	json.someKey = 'value';
+
+	const result = defaultSerialize<{someKey: string}>(json);
+	t.expect(result).toStrictEqual('{"someKey":"value"}');
+});
+
 test.it('removes the first colon from strings not prefixed by base64', t => {
 	const json = JSON.stringify({
 		simple: ':hello',