fix: avoid duplicate fields in litplugin if class jsdoc is used

Author: thepassle

packages/analyzer/custom-elements.json

@@ -7,37 +7,48 @@
       "path": "fixtures/-default/package/bar.js",
       "declarations": [
         {
-          "kind": "function",
-          "name": "bar"
+          "kind": "class",
+          "description": "",
+          "name": "CustomElementSchemaViewerElement",
+          "members": [
+            {
+              "type": {
+                "text": "boolean"
+              },
+              "description": "this is desc",
+              "name": "src",
+              "kind": "field",
+              "privacy": "public"
+            }
+          ],
+          "attributes": [
+            {
+              "name": "src",
+              "fieldName": "src"
+            }
+          ],
+          "superclass": {
+            "name": "LitElement"
+          },
+          "tagName": "customelement-schema-viewer",
+          "customElement": true
         }
       ],
       "exports": [
         {
           "kind": "js",
-          "name": "bar",
+          "name": "CustomElementSchemaViewerElement",
           "declaration": {
-            "name": "bar",
+            "name": "CustomElementSchemaViewerElement",
             "module": "fixtures/-default/package/bar.js"
           }
-        }
-      ]
-    },
-    {
-      "kind": "javascript-module",
-      "path": "fixtures/-default/package/foo.js",
-      "declarations": [
+        },
         {
-          "kind": "function",
-          "name": "foo"
-        }
-      ],
-      "exports": [
-        {
-          "kind": "js",
-          "name": "foo",
+          "kind": "custom-element-definition",
+          "name": "customelement-schema-viewer",
           "declaration": {
-            "name": "foo",
-            "module": "fixtures/-default/package/foo.js"
+            "name": "CustomElementSchemaViewerElement",
+            "module": "fixtures/-default/package/bar.js"
           }
         }
       ]

packages/analyzer/fixtures/-default/package/bar.js

@@ -1,5 +1,13 @@
-import type foo from 'ts';
-import { named } from 'thirdparty';
-export function bar() {
+/**
+ * @property {boolean} src this is desc
+ */
+@customElement("customelement-schema-viewer")
+export class CustomElementSchemaViewerElement extends LitElement {
 
+
+  static get properties() {
+    return {
+      src: { type: Boolean }
+    }
+  }
 }

packages/analyzer/fixtures/-default/package/foo.js

@@ -26,6 +26,10 @@
             {
               "kind": "field",
               "name": "prop1",
+              "type": {
+                "text": "string"
+              },
+              "description": "this is the description",
               "privacy": "public",
               "default": "'foo'"
             },

packages/analyzer/fixtures/plugin-lit/fixture/custom-elements.json

@@ -24,8 +24,12 @@
               "name": "decoratedPropertyNoAttr"
             },
             {
-              "kind": "field",
+              "type": {
+                "text": "string"
+              },
+              "description": "this is the description",
               "name": "prop1",
+              "kind": "field",
               "privacy": "public",
               "default": "'foo'"
             },

packages/analyzer/fixtures/plugin-lit/output.json

@@ -1,5 +1,8 @@
 import { LitElement, property, customElement } from 'lit-element';
 
+/**
+ * @prop {string} prop1 this is the description
+ */
 @customElement('my-element')
 class MyElement extends LitElement {
   static get properties() {

packages/analyzer/fixtures/plugin-lit/package/my-element.js

@@ -44,7 +44,14 @@ export function staticPropertiesPlugin() {
                   currClass.attributes = [...(currClass?.attributes || []), attribute]
                 }
 
-                currClass?.members.push(classMember);
+                
+                const existingField = currClass?.members?.find(field => field.name === classMember.name);
+
+                if(!existingField) {
+                  currClass.members.push(classMember);
+                } else {
+                  currClass.members = currClass?.members?.map(field => field.name === classMember.name ? ({...field, ...classMember}) : field);
+                }
               });
               return;
             }