feat(eslint): support the new serializer API

Author: wmertens

packages/eslint-plugin-qwik/src/validLexicalScope.ts

@@ -287,7 +287,15 @@ function _isTypeCapturable(
     return;
   }
   seen.add(type);
-  if (type.getProperty('__no_serialize__') || type.getProperty('__qwik_serializable__')) {
+  if (
+    type
+      .getProperties()
+      .some((p) =>
+        /(__no_serialize__|__qwik_serializable__|NoSerializeSymbol|SerializerSymbol)/i.test(
+          p.escapedName as string
+        )
+      )
+  ) {
     return;
   }
   const isUnknown = type.flags & ts.TypeFlags.Unknown;

packages/eslint-plugin-qwik/tests/valid-lexical-scope/valid-no-serialize-symbol.tsx

@@ -0,0 +1,37 @@
+import {
+  useTask$,
+  component$,
+  NoSerializeSymbol,
+  SerializerSymbol,
+  useSerializer$,
+} from '@qwik.dev/core';
+
+class NoSerSym {
+  [NoSerializeSymbol] = true;
+}
+
+class SerSym {
+  [SerializerSymbol]() {
+    return this.toString();
+  }
+}
+
+export interface Value {
+  value: number;
+  obj1: NoSerSym;
+  obj2: SerSym;
+}
+
+export const HelloWorld = component$(() => {
+  const state: Value = { value: 12, obj1: new NoSerSym(), obj2: new SerSym() };
+  const ser = useSerializer$({
+    deserialize: () => new SerSym(),
+    serialize: (obj) => obj.toString(),
+  });
+
+  useTask$(() => {
+    // eslint-disable-next-line no-console
+    console.log(state.value, ser.value);
+  });
+  return <div></div>;
+});

packages/qwik/public.d.ts

@@ -29,6 +29,8 @@ export {
   JSXOutput,
   noSerialize,
   NoSerialize,
+  NoSerializeSymbol,
+  SerializerSymbol,
   OnVisibleTaskOptions,
   PrefetchGraph,
   PrefetchServiceWorker,

packages/qwik/src/core/signal/signal.public.ts

@@ -51,7 +51,7 @@ export interface ComputedSignal<T> extends ReadonlySignal<T> {
  */
 export interface SerializerSignal<T> extends ComputedSignal<T> {
   /** Fake property to make the serialization linter happy */
-  __noSerialize__: true;
+  __no_serialize__: true;
 }
 
 /**