add better handling for typescript compiler

@plotly/dash-generator-test-component-typescript/package.json

@@ -20,6 +20,7 @@
     "@babel/preset-env": "^7.19.4",
     "@babel/preset-react": "^7.18.6",
     "@types/jest": "^29.2.0",
+    "@types/ramda": "^0.28.14",
     "@types/react": "^17.0.39",
     "babel-loader": "^8.2.5",
     "jest": "^29.2.1",

@plotly/dash-generator-test-component-typescript/src/components/WrappedHTML.tsx

@@ -1,5 +1,5 @@
 import React from 'react';
-import {WrappedHTMLProps} from '../props';
+import { WrappedHTMLProps } from '../props';
 
 /**
  * Component docstring

@plotly/dash-generator-test-component-typescript/src/index.ts

@@ -17,5 +17,5 @@ export {
   FCComponent,
   EmptyComponent,
   MixedComponent,
-  RequiredChildrenComponent,
+  RequiredChildrenComponent
 };

dash/extract-meta.js

@@ -25,7 +25,7 @@ function help() {
     console.error('usage: ');
     console.error(
         'extract-meta ^fileIgnorePattern ^forbidden$|^props$|^patterns$' +
-            ' path/to/component(s) [path/to/more/component(s) ...] > metadata.json'
+        ' path/to/component(s) [path/to/more/component(s) ...] > metadata.json'
     );
 }
 
@@ -98,7 +98,7 @@ function isReservedPropName(propName) {
         if (reservedPattern.test(propName)) {
             process.stderr.write(
                 `\nERROR: "${propName}" matches reserved word ` +
-                    `pattern: ${reservedPattern.toString()}\n`
+                `pattern: ${reservedPattern.toString()}\n`
             );
             failedBuild = true;
         }
@@ -118,9 +118,15 @@ function checkDocstring(name, value) {
 function docstringWarning(doc) {
     checkDocstring(doc.displayName, doc.description);
 
-    Object.entries(doc.props || {}).forEach(([name, p]) =>
-        checkDocstring(`${doc.displayName}.${name}`, p.description)
-    );
+    const exportedDisplayName = doc.exportedName || doc.displayName
+    if (doc && doc.props) {
+        Object.entries(doc.props || {}).forEach(([name, p]) =>
+            checkDocstring(`${exportedDisplayName}.${name}`, p.description)
+        );
+    } else {
+        // Soft fail
+        console.warn(`${exportedDisplayName} does not have any props`)
+    }
 }
 
 function zipArrays(...arrays) {
@@ -160,9 +166,9 @@ function gatherComponents(sources, components = {}) {
         const extension = path.extname(filepath);
         if (['.jsx', '.js'].includes(extension)) {
             components[cleanPath(filepath)] = parseJSX(filepath);
-        } else if (filepath.endsWith('.tsx')) {
+        } else if (['.tsx', 'ts'].includes(extension)) {
             try {
-                const name = /(.*)\.tsx/.exec(path.basename(filepath))[1];
+                const name = /(.*)\.(ts|tsx)/.exec(path.basename(filepath))[1];
                 filepaths.push(filepath);
                 names.push(name);
             } catch (err) {
@@ -720,6 +726,24 @@ function gatherComponents(sources, components = {}) {
                 }
             }
 
+            const isArrowFunction = ts.isArrowFunction(declaration)
+            const isClass = ts.isClassDeclaration(declaration);
+            const isFunction = ts.isFunctionDeclaration(declaration)
+
+            if (!isArrowFunction && !isClass && !isFunction) {
+                // we do not care about these exports
+                return null
+            }
+
+            if (isArrowFunction || isFunction) {
+                const signature = checker.getSignaturesOfType(type, ts.SignatureKind.Call)[0];
+                const returnType = checker.typeToString(signature.getReturnType());
+                if (!['Element', 'any', 'null'].includes(returnType)) {
+                    // Not JSX so no need to classifiy as compnent
+                    return null;
+                }
+            }
+
             let defaultProps = getDefaultProps(typeSymbol, source);
             const propsType = getPropsForFunctionalComponent(type);
             const isContext = !!type.getProperty('isContext');
@@ -764,9 +788,12 @@ function gatherComponents(sources, components = {}) {
                 displayName: name,
                 description,
                 props,
-                isContext
+                isContext,
+                exportedName: rootExp.name
             };
             docstringWarning(doc);
+
+            // todo: Add support for mutiple components in single file
             components[cleanPath(filepath)] = doc;
         });
     });