Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Feature]: Expose nested modules in compilation.chunkGraph.getChunkModules #8130

Open
abettadapur opened this issue Oct 15, 2024 · 1 comment · May be fixed by #8192
Open

[Feature]: Expose nested modules in compilation.chunkGraph.getChunkModules #8130

abettadapur opened this issue Oct 15, 2024 · 1 comment · May be fixed by #8192
Assignees
Labels
feat New feature or request

Comments

@abettadapur
Copy link

What problem does this feature solve?

When using moduleConcatenation:true, the call to compilation.chunkGraph.getChunkModules(chunk) only returns the top level concatenated modules, and there is no way to access the nested modules.

In Webpack, you can access the nested modules with

type ConcatenatedModule {
    ...
    modules: Module[]
}

I saw #6784 exposes this information to the stats plugin. Can we also expose this information to the JS API?

What does the proposed API of configuration look like?

Expose modules property on concatenated modules, which would be a list of the modules that are contained in the ConcatenatedModule

@abettadapur abettadapur added feat New feature or request pending triage The issue/PR is currently untouched. labels Oct 15, 2024
@LingyuCoder LingyuCoder removed the pending triage The issue/PR is currently untouched. label Oct 17, 2024
@SyMind
Copy link
Member

SyMind commented Oct 22, 2024

@abettadapur Would you like to give me a use case.

@SyMind SyMind linked a pull request Oct 22, 2024 that will close this issue
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants