fix excludeNotDocumented behavior

Author: Cary Hu

src/lib/converter/plugins/CommentPlugin.ts

@@ -203,6 +203,26 @@ export class CommentPlugin extends ConverterComponent {
         this.removeExcludedTags(comment);
     }
 
+    private getHiddenItems(reflection: Reflection) {
+        const hiddenItems = new Set<Reflection>();
+        if (reflection.kindOf(ReflectionKind.FunctionOrMethod)) {
+            const decl = reflection as DeclarationReflection;
+            if (decl.signatures) {
+                for (const sig of decl.signatures) {
+                    if (!sig.comment) {
+                        hiddenItems.add(sig);
+                    }
+                }
+            }
+            if (!decl.signatures?.some(sig => sig.comment)) {
+                hiddenItems.add(decl);
+            }
+        } else {
+            hiddenItems.add(reflection);
+        }
+        return hiddenItems || [];
+    }
+
     /**
      * Triggered when the converter begins resolving a project.
      *
@@ -225,7 +245,7 @@ export class CommentPlugin extends ConverterComponent {
             }
 
             if (this.isHidden(ref)) {
-                hidden.add(ref);
+                this.getHiddenItems(ref).forEach(item => hidden.add(item));
             }
         }
         hidden.forEach((reflection) => project.removeReflection(reflection));

src/test/converter2/issues/gh1994.d.ts

@@ -0,0 +1,48 @@
+/**
+ *
+ * test 2
+ */
+declare class Collection {
+
+    /**
+     *
+     * @param {boolean} n Test boolean
+     */
+    public case1(n: boolean)
+    public case1(n: number)
+    private case2()
+}
+
+
+/**
+ * Comment for case4
+ */
+export declare type Case4 = {
+    case5: string;
+
+    /**
+     * comment for case6
+     *
+     * @param number1 test comment
+     */
+    case6: (number1: number) => boolean;
+}
+
+/**
+ * Comment for case7
+ */
+export declare type case7 = (number1: number) => boolean;
+
+/**
+ * Rounds or truncates a number to a specified precision.
+ *
+ * @param value Value to round or truncate.
+ * @param prec Number of decimal digits for the result.
+ * @param truncate Whether to truncate or round the original value.
+ */
+declare function case3(value: number, prec: number, truncate: boolean): number;
+
+export {
+    Collection,
+    case3
+}

src/test/issueTests.ts

@@ -608,4 +608,13 @@ export const issueTests: {
         logger.discardDebugMessages();
         logger.expectNoOtherMessages();
     },
+
+    gh1994(project, logger) {
+        const case1 = query(project, "Collection.case1");
+        // TODO: I don't know how to add test case for this case
+        // and I added a test simple file in 'issues' folder.
+        // But I don't know how to test it.
+        logger.discardDebugMessages();
+        logger.expectNoOtherMessages();
+    }
 };