bug: avoid trying to get properties of HTMLElement types

package.json

@@ -15,7 +15,8 @@
     "compile": "tsc",
     "prepare": "tsc",
     "test": "jest",
-    "lint:js": "eslint . --ext .js,.ts"
+    "lint:js": "eslint . --ext .js,.ts",
+    "lint:js:fix": "eslint . --ext .js,.ts --fix"
   },
   "dependencies": {
     "debug": "^4.3.4",

src/parser.ts

@@ -219,10 +219,18 @@ export default class Parser {
       };
     }
 
+    // check if one element has string ending with Element
+    let isElement = false;
+    [symbol?.getName(), typeAsString].forEach((v) => {
+      if (v && v.endsWith('Element')) {
+        isElement = true;
+      }
+    });
+
     if (
       symbol &&
-      this.isInterfaceOrObjectWithOptionalProperties(symbol, type) &&
-      !/Element$/.test(symbol.getName())
+      !isElement &&
+      this.isInterfaceOrObjectWithOptionalProperties(symbol, type)
     ) {
       const props = type.getApparentProperties();
       return {

tests/__fixtures__/button-args-only.ts

@@ -35,4 +35,4 @@ interface ButtonArgs {
  * The button component
  * @since 1.0.0
  */
-export default class Button extends Component<ButtonArgs> { }
+export default class Button extends Component<ButtonArgs> {}

tests/__fixtures__/button-with-extended-args.ts

@@ -14,4 +14,4 @@ interface Signature {
   Args: ButtonArgs;
 }
 
-export default class ButtonWithExtendedArgs extends Component<Signature> { }
+export default class ButtonWithExtendedArgs extends Component<Signature> {}

tests/__fixtures__/dropdown.d.ts

@@ -1,5 +1,6 @@
 import Component from '@glimmer/component';
 import { type ButtonSignature } from './button';
+// eslint-disable-next-line
 import type { WithBoundArgs, ModifierLike } from '@glint/template';
 interface DropdownArgs {
   /**
@@ -8,6 +9,8 @@ interface DropdownArgs {
    * @defaultValue true
    */
   closeOnItemSelect?: boolean;
+
+  someRandomElement?: HTMLElement;
 }
 
 interface Transition {

tests/__fixtures__/multiple-components.ts

@@ -16,5 +16,5 @@ interface Signature {
 // eslint-disable-next-line
 class NotExportedButton extends Component<Signature> { }
 
-export class ButtonNotDefault extends Component<Signature> { }
-export default class ButtonDefault extends Component<Signature> { }
+export class ButtonNotDefault extends Component<Signature> {}
+export default class ButtonDefault extends Component<Signature> {}

tests/__snapshots__/index.test.ts.snap

@@ -440,6 +440,17 @@ option allows to choose where it appears from.",
           "type": "boolean",
         },
       },
+      {
+        "defaultValue": undefined,
+        "description": "",
+        "identifier": "someRandomElement",
+        "isInternal": false,
+        "isRequired": false,
+        "tags": {},
+        "type": {
+          "type": "HTMLElement",
+        },
+      },
     ],
     "Blocks": [
       {

tests/experiment.ts

@@ -4,6 +4,7 @@ import path from 'path';
 // import fs from 'fs';
 
 function inspect(obj: unknown): void {
+  // eslint-disable-next-line
   console.log(util.inspect(obj, false, 15, true));
 }