Improve some more code

src/lib/converter/context.ts

@@ -122,15 +122,11 @@ export class Context {
         if (!nodeType) {
             if (node.symbol) {
                 nodeType = this.checker.getDeclaredTypeOfSymbol(node.symbol);
-            } else if (node.parent && node.parent.symbol) {
+            } else if (node.parent?.symbol) {
                 nodeType = this.checker.getDeclaredTypeOfSymbol(
                     node.parent.symbol
                 );
-            } else if (
-                node.parent &&
-                node.parent.parent &&
-                node.parent.parent.symbol
-            ) {
+            } else if (node.parent?.parent?.symbol) {
                 nodeType = this.checker.getDeclaredTypeOfSymbol(
                     node.parent.parent.symbol
                 );

src/lib/converter/plugins/SourceLinkPlugin.ts

@@ -220,7 +220,7 @@ export class SourceLinkPlugin extends ConverterComponent {
     /**
      * List of paths known to be not under git control.
      */
-    private ignoredPaths: string[] = [];
+    private ignoredPaths = new Set<string>();
 
     @BindOption("gitRevision")
     readonly gitRevision!: string;
@@ -252,15 +252,16 @@ export class SourceLinkPlugin extends ConverterComponent {
     private getRepository(fileName: string): Repository | undefined {
         // Check for known non-repositories
         const dirName = Path.dirname(fileName);
-        for (let i = 0, c = this.ignoredPaths.length; i < c; i++) {
-            if (this.ignoredPaths[i] === dirName) {
+        const segments = dirName.split("/");
+        for (let i = segments.length; i > 0; i--) {
+            if (this.ignoredPaths.has(segments.slice(0, i).join("/"))) {
                 return;
             }
         }
 
         // Check for known repositories
         for (const path of Object.keys(this.repositories)) {
-            if (fileName.substring(0, path.length).toLowerCase() === path) {
+            if (fileName.toLowerCase().startsWith(path)) {
                 return this.repositories[path];
             }
         }
@@ -277,10 +278,7 @@ export class SourceLinkPlugin extends ConverterComponent {
         }
 
         // No repository found, add path to ignored paths
-        const segments = dirName.split("/");
-        for (let i = segments.length; i > 0; i--) {
-            this.ignoredPaths.push(segments.slice(0, i).join("/"));
-        }
+        this.ignoredPaths.add(dirName);
     }
 
     /**
@@ -298,11 +296,10 @@ export class SourceLinkPlugin extends ConverterComponent {
             }
         });
 
-        for (const key in project.reflections) {
-            const reflection = project.reflections[key];
+        for (const reflection of Object.values(project.reflections)) {
             if (reflection.sources) {
                 reflection.sources.forEach((source: SourceReference) => {
-                    if (source.file && source.file.url) {
+                    if (source.file?.url) {
                         source.url =
                             source.file.url +
                             "#" +

src/lib/models/reflections/utils.ts

@@ -19,9 +19,10 @@ export function splitUnquotedString(
             return [input];
         } else {
             const remainder = input.substring(closingQuoteIndex + 1);
-            const result = [input.substring(0, closingQuoteIndex + 1)];
-            result.push(...splitUnquotedString(remainder, delimiter));
-            return result;
+            return [
+                input.substring(0, closingQuoteIndex + 1),
+                ...splitUnquotedString(remainder, delimiter),
+            ];
         }
     } else {
         return input.split(delimiter);

src/lib/output/themes/default/DefaultTheme.tsx

@@ -167,8 +167,7 @@ export class DefaultTheme extends Theme {
             event.project.groups.forEach(DefaultTheme.applyGroupClasses);
         }
 
-        for (const id in event.project.reflections) {
-            const reflection = event.project.reflections[id];
+        for (const reflection of Object.values(event.project.reflections)) {
             if (reflection instanceof DeclarationReflection) {
                 DefaultTheme.applyReflectionClasses(reflection);
             }

src/lib/output/themes/default/partials/type.tsx

@@ -54,7 +54,7 @@ function getUniquePath(reflection: Reflection): Reflection[] {
 function getNamespacedPath(reflection: Reflection): Reflection[] {
     const path = [reflection];
     let parent = reflection.parent;
-    while (parent && parent.kindOf(ReflectionKind.Namespace)) {
+    while (parent?.kindOf(ReflectionKind.Namespace)) {
         path.unshift(parent);
         parent = parent.parent;
     }

src/lib/output/themes/lib.tsx

@@ -26,12 +26,11 @@ export function stringify(data: unknown) {
 export function wbr(str: string): (string | JSX.Element)[] {
     // TODO surely there is a better way to do this, but I'm tired.
     const ret: (string | JSX.Element)[] = [];
-    const re = /[\s\S]*?(?:([^_-][_-])(?=[^_-])|([^A-Z])(?=[A-Z][^A-Z]))/g;
+    const re = /[\s\S]*?(?:[^_-][_-](?=[^_-])|[^A-Z](?=[A-Z][^A-Z]))/g;
     let match: RegExpExecArray | null;
     let i = 0;
     while ((match = re.exec(str))) {
-        ret.push(match[0]);
-        ret.push(<wbr />);
+        ret.push(match[0], <wbr />);
         i += match[0].length;
     }
     ret.push(str.slice(i));
@@ -70,9 +69,8 @@ export function renderFlags(flags: ReflectionFlags, comment: Comment | undefined
 }
 
 export function classNames(names: Record<string, boolean | null | undefined>, extraCss?: string) {
-    const css = Object.entries(names)
-        .filter(([, include]) => include)
-        .map(([key]) => key)
+    const css = Object.keys(names)
+        .filter((key) => names[key])
         .concat(extraCss || "")
         .join(" ")
         .trim()
@@ -83,10 +81,11 @@ export function classNames(names: Record<string, boolean | null | undefined>, ex
 export function hasTypeParameters(
     reflection: Reflection
 ): reflection is Reflection & { typeParameters: TypeParameterReflection[] } {
-    if (reflection instanceof DeclarationReflection || reflection instanceof SignatureReflection) {
-        return reflection.typeParameters != null && reflection.typeParameters.length > 0;
-    }
-    return false;
+    return (
+        (reflection instanceof DeclarationReflection || reflection instanceof SignatureReflection) &&
+        reflection.typeParameters != null &&
+        reflection.typeParameters.length > 0
+    );
 }
 
 export function renderTypeParametersSignature(