Skip to content

docs: add typedoc-plugin-missing-exports#109

Open
moshams272 wants to merge 1 commit into
webpack:mainfrom
moshams272:add-missing-exports
Open

docs: add typedoc-plugin-missing-exports#109
moshams272 wants to merge 1 commit into
webpack:mainfrom
moshams272:add-missing-exports

Conversation

@moshams272
Copy link
Copy Markdown

What does this PR do?

Integrates typedoc-plugin-missing-exports to recover internally used types/interfaces that were not explicitly exported, resolving missing references in the docs.

I tried three different settings to find the best result:

Experiment 1: Default Plugin Behavior (~135 types)

Running the plugin with default settings recovered about 135 missing types. However, they were isolated in an <internal> module that typedoc didn't go throw them (Shallow Pass).

Experiment 2: Using excludeExternals: true (~60 types) -> [Current PR State]

I added this option to ignore basic JavaScript/Node types. The number dropped to about 60 Webpack types. They are grouped in an <internal> folder. I chose this approach because it keeps the docs clean and simple.

Experiment 3: Using excludeExternals: true & placeInternalsInOwningModule: true (~630 types)

I tried this option to put the types inside their original modules instead of the <internal> folder. But this made the plugin
go into chain, as after get them out it started seaching to the internally used types/interfaces that were not explicitly exported and put them inside their original modules and do it again to them...

While highly detailed, I felt the 630+ inline types would create too much noise for the average reader, so I reverted to Experiment 2.

Fixes #92

- Added plugin to recover internally referenced but unexported types.

- Set excludeExternals: true to filter out Node/language built-ins.

- Successfully recovered ~60 Webpack-specific internal types.
@vercel
Copy link
Copy Markdown

vercel Bot commented May 26, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
webpack-doc-kit Ready Ready Preview, Comment May 26, 2026 11:43am

Request Review

Copy link
Copy Markdown
Member

@avivkeller avivkeller left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we should find a new home for these types, rather than <internal>. Similarly to how #91 added a system for moving types to the subsystems that they are actually used in, can we do that here>

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Important Types Excluded From Generation

2 participants