Merge pull request #29 from open-wc/fix/default-export-handling

fix: improve handling of classes that are default exported

Author: thepassle

packages/analyzer/browser/index.js

@@ -7,16 +7,64 @@
       "path": "fixtures/-default/package/bar.js",
       "declarations": [
         {
-          "kind": "function",
-          "name": "bar"
+          "kind": "class",
+          "description": "",
+          "name": "default",
+          "cssParts": [
+            {
+              "name": "foo"
+            }
+          ],
+          "members": [
+            {
+              "kind": "field",
+              "name": "propertyDecorator",
+              "type": {
+                "text": "string"
+              },
+              "default": "hi"
+            },
+            {
+              "kind": "field",
+              "name": "mobileopen",
+              "privacy": "public",
+              "default": "false"
+            }
+          ],
+          "attributes": [
+            {
+              "name": "propertyDecorator",
+              "type": {
+                "text": "string"
+              },
+              "default": "hi",
+              "fieldName": "propertyDecorator"
+            },
+            {
+              "name": "mobileopen",
+              "fieldName": "mobileopen"
+            }
+          ],
+          "superclass": {
+            "name": "LitElement"
+          },
+          "customElement": true
         }
       ],
       "exports": [
         {
           "kind": "js",
-          "name": "bar",
+          "name": "default",
+          "declaration": {
+            "name": "default",
+            "module": "fixtures/-default/package/bar.js"
+          }
+        },
+        {
+          "kind": "custom-element-definition",
+          "name": "top-bar",
           "declaration": {
-            "name": "bar",
+            "name": "TopBar",
             "module": "fixtures/-default/package/bar.js"
           }
         }

packages/analyzer/custom-elements.json

@@ -1,6 +1,6 @@
 {
   "name": "@custom-elements-manifest/analyzer",
-  "version": "0.2.2",
+  "version": "0.2.3",
   "description": "",
   "license": "MIT",
   "type": "module",

packages/analyzer/package.json

@@ -13,7 +13,8 @@ export function classJsDocPlugin() {
     analyzePhase({ts, node, moduleDoc}){
       switch (node.kind) {
         case ts.SyntaxKind.ClassDeclaration:
-          const className = node?.name?.text;
+          const hasDefaultModifier = node?.modifiers?.some(mod => ts.SyntaxKind.DefaultKeyword === mod.kind);
+          const className = hasDefaultModifier ? 'default' : node?.name?.getText();
           const classDoc = moduleDoc?.declarations?.find(declaration => declaration.name === className);
 
           /**

packages/analyzer/src/features/analyse-phase/class-jsdoc.js

@@ -8,7 +8,8 @@ export function attrDecoratorPlugin() {
     analyzePhase({ts, node, moduleDoc}){
       switch(node.kind) {
         case ts.SyntaxKind.ClassDeclaration:
-          const className = node?.name?.text;
+          const hasDefaultModifier = node?.modifiers?.some(mod => ts.SyntaxKind.DefaultKeyword === mod.kind);
+          const className = hasDefaultModifier ? 'default' : node?.name?.getText();
           const classDoc = moduleDoc?.declarations?.find(declaration => declaration.name === className);
 
           /**

packages/analyzer/src/features/framework-plugins/decorators/attr.js

@@ -14,7 +14,8 @@ export function propertyDecoratorPlugin() {
     analyzePhase({ts, node, moduleDoc}){
       switch (node.kind) {
         case ts.SyntaxKind.ClassDeclaration:    
-          const className = node?.name?.getText();
+          const hasDefaultModifier = node?.modifiers?.some(mod => ts.SyntaxKind.DefaultKeyword === mod.kind);
+          const className = hasDefaultModifier ? 'default' : node?.name?.getText();
           const currClass = moduleDoc?.declarations?.find(declaration => declaration.name === className);
 
           /**

packages/analyzer/src/features/framework-plugins/lit/property-decorator.js

@@ -13,7 +13,8 @@ export function staticPropertiesPlugin() {
     analyzePhase({ts, node, moduleDoc}){
       switch (node.kind) {
         case ts.SyntaxKind.ClassDeclaration:    
-          const className = node?.name?.getText();
+          const hasDefaultModifier = node?.modifiers?.some(mod => ts.SyntaxKind.DefaultKeyword === mod.kind);
+          const className = hasDefaultModifier ? 'default' : node?.name?.getText();
           const currClass = moduleDoc?.declarations?.find(declaration => declaration.name === className);
 
           node?.members?.forEach(member => {
@@ -39,7 +40,8 @@ export function staticPropertiesPlugin() {
                   if(attributeName) {
                     attribute.name = attributeName;
                   }
-                  currClass?.attributes.push(attribute);
+
+                  currClass.attributes = [...(currClass?.attributes || []), attribute]
                 }
 
                 currClass?.members.push(classMember);

packages/analyzer/src/features/framework-plugins/lit/static-properties.js

@@ -16,14 +16,14 @@ export function stencilPlugin() {
            * Add tagName to classDoc, extracted from `@Component({tag: 'foo-bar'})` decorator
            * Add custom-element-definition to exports
            */ 
-          const className = node?.name?.getText();
-
           const componentDecorator = node?.decorators?.find(decorator('Component'))?.expression;
 
           const tagName = componentDecorator?.arguments?.[0]?.properties?.find(prop => {
             return prop?.name?.getText() === 'tag'
           })?.initializer?.text;
 
+          const hasDefaultModifier = node?.modifiers?.some(mod => ts.SyntaxKind.DefaultKeyword === mod.kind);
+          const className = hasDefaultModifier ? 'default' : node?.name?.getText();
           const currClass = moduleDoc?.declarations?.find(declaration => declaration.name === className);
           if(tagName) {
             currClass.tagName = tagName;