Consider JSDoc tags of individual export specifiers + their parent declaration

Author: webpro

fixtures/re-exports-public/index.ts

@@ -0,0 +1,9 @@
+import { something } from './module.js';
+something;
+
+/** @public */
+export {
+  /** @public */
+  somethingToIgnore,
+  somethingIgnoredAnyway,
+} from './module.js';

fixtures/re-exports-public/module.ts

@@ -0,0 +1,3 @@
+export const something = 1;
+export const somethingToIgnore = 1;
+export const somethingIgnoredAnyway = 1;

fixtures/re-exports-public/package.json

@@ -0,0 +1,3 @@
+{
+  "name": "@fixtures/re-exports-public"
+}

fixtures/re-exports/1-entry.ts

@@ -1,2 +1,8 @@
 import { something } from './2-re-export-star.js';
 something;
+
+export {
+  /** @public */
+  somethingToIgnore,
+  somethingNotToIgnore,
+} from './2-re-export-star.js';

fixtures/re-exports/3-re-export-named.ts

@@ -1 +1 @@
-export { something } from './4-my-module';
+export { something, somethingToIgnore, somethingNotToIgnore } from './4-my-module';

fixtures/re-exports/4-my-module.ts

@@ -1 +1,4 @@
 export const something = {};
+
+export const somethingToIgnore = {};
+export const somethingNotToIgnore = {};

src/typescript/ast-helpers.ts

@@ -133,8 +133,11 @@ export const isInModuleBlock = (node: ts.Node) => {
 
 export const getJSDocTags = (node: ts.Node) => {
   const tags = new Set<string>();
-  const declaration = ts.isExportSpecifier(node) || ts.isBindingElement(node) ? node.parent.parent : node;
-  for (const tagNode of ts.getJSDocTags(declaration)) {
+  let tagNodes = ts.getJSDocTags(node);
+  if (ts.isExportSpecifier(node) || ts.isBindingElement(node)) {
+    tagNodes = [...tagNodes, ...ts.getJSDocTags(node.parent.parent)];
+  }
+  for (const tagNode of tagNodes) {
     const match = tagNode.getText()?.match(/@\S+/);
     if (match) tags.add(match[0]);
   }

test/re-exports-public.test.ts

@@ -0,0 +1,22 @@
+import assert from 'node:assert/strict';
+import test from 'node:test';
+import { main } from '../src/index.js';
+import { resolve } from '../src/util/path.js';
+import baseArguments from './helpers/baseArguments.js';
+import baseCounters from './helpers/baseCounters.js';
+
+const cwd = resolve('fixtures/re-exports-public');
+
+test('Ignore re-exports from included entry files', async () => {
+  const { counters } = await main({
+    ...baseArguments,
+    cwd,
+    isIncludeEntryExports: true,
+  });
+
+  assert.deepEqual(counters, {
+    ...baseCounters,
+    processed: 2,
+    total: 2,
+  });
+});

test/re-exports.test.ts

@@ -0,0 +1,38 @@
+import assert from 'node:assert/strict';
+import test from 'node:test';
+import { main } from '../src/index.js';
+import { resolve } from '../src/util/path.js';
+import baseArguments from './helpers/baseArguments.js';
+import baseCounters from './helpers/baseCounters.js';
+
+const cwd = resolve('fixtures/re-exports');
+
+test('Ignore re-exports from entry files', async () => {
+  const { counters } = await main({
+    ...baseArguments,
+    cwd,
+  });
+
+  assert.deepEqual(counters, {
+    ...baseCounters,
+    processed: 4,
+    total: 4,
+  });
+});
+
+test('Ignore re-exports from entry files (include entry + ignore @public)', async () => {
+  const { issues, counters } = await main({
+    ...baseArguments,
+    cwd,
+    isIncludeEntryExports: true,
+  });
+
+  assert(issues.exports['1-entry.ts']['somethingNotToIgnore']);
+
+  assert.deepEqual(counters, {
+    ...baseCounters,
+    exports: 1,
+    processed: 4,
+    total: 4,
+  });
+});