warn & pass through non objects in event-to-object

Author: Archmonger

src/js/packages/event-to-object/src/index.ts

@@ -7,10 +7,18 @@ export default function convert(
   classObject: { [key: string]: any },
   maxDepth: number = 10,
 ): object {
-  const visited = new WeakSet<any>();
-  visited.add(classObject);
+  // Immediately return `classObject` if given an unexpected (non-object) input
+  if (!classObject || typeof classObject !== "object") {
+    console.warn(
+      "eventToObject: Expected an object input, received:",
+      classObject,
+    );
+    return classObject;
+  }
 
   // Begin conversion
+  const visited = new WeakSet<any>();
+  visited.add(classObject);
   const convertedObj: { [key: string]: any } = {};
   for (const key in classObject) {
     // Skip keys that cannot be converted

src/js/packages/event-to-object/tests/event-to-object.test.ts

@@ -670,3 +670,10 @@ test("handles recursive HTML node structures", () => {
     expect(converted.children[0].parentNode).toBeUndefined();
   }
 });
+
+test("pass-through on unexpected non-object inputs", () => {
+  expect(convert(null as any)).toEqual(null);
+  expect(convert(undefined as any)).toEqual(undefined);
+  expect(convert(42 as any)).toEqual(42);
+  expect(convert("test" as any)).toEqual("test");
+});