ignore functions with NoSerializeSymbol

Author: Varixo

packages/qwik/src/core/shared/shared-serialization.unit.ts

@@ -955,6 +955,20 @@ describe('shared-serialization', () => {
         (17 chars)"
       `);
     });
+    it('should ignore functions with NoSerializeSymbol', async () => {
+      const ignore = () => console.warn();
+      (ignore as any)[NoSerializeSymbol] = true;
+      const obj = { hi: true, ignore };
+      const state = await serialize(obj);
+      expect(dumpState(state)).toMatchInlineSnapshot(`
+        "
+        0 Object [
+          String "hi"
+          Constant true
+        ]
+        (17 chars)"
+      `);
+    });
     it('should ignore NoSerializeSymbol', async () => {
       const obj = { hi: true, [NoSerializeSymbol]: true };
       const state = await serialize(obj);

packages/qwik/src/core/shared/utils/serialize-utils.ts

@@ -102,8 +102,8 @@ export const shouldSerialize = (obj: unknown): boolean => {
 export const fastSkipSerialize = (obj: object | Function): boolean => {
   return (
     obj &&
-    ((typeof obj === 'object' && (NoSerializeSymbol in obj || noSerializeSet.has(obj))) ||
-      (typeof obj === 'function' && noSerializeSet.has(obj)))
+    (typeof obj === 'object' || typeof obj === 'function') &&
+    (NoSerializeSymbol in obj || noSerializeSet.has(obj))
   );
 };