material-ui icon indicating copy to clipboard operation
material-ui copied to clipboard

[code-infra] Add types for markdown loader

Open Janpot opened this issue 6 months ago • 2 comments

Added a few types to better under stand how the code flows wrt webpack cache. May as well commit them for future use. The code didn't alter semantically in any way, it's 100% types.

@dav-is I remember you mentioned before that this loader doesn't do correct dependency tracking. I'm seeing addDependency all over the place, did you have a specific case in mind?

Janpot avatar Oct 14 '25 09:10 Janpot

Netlify deploy preview

https://deploy-preview-47075--material-ui.netlify.app/

Bundle size report

Bundle Parsed size Gzip size
@mui/material 0B(0.00%) 0B(0.00%)
@mui/lab 0B(0.00%) 0B(0.00%)
@mui/system 0B(0.00%) 0B(0.00%)
@mui/utils 0B(0.00%) 0B(0.00%)

Details of bundle changes

Generated by :no_entry_sign: dangerJS against 29c3b366d8500cac1e41a8a61fb230e24698c149

mui-bot avatar Oct 14 '25 09:10 mui-bot

@dav-is I remember you mentioned before that this loader doesn't do correct dependency tracking.

It was Base UI's types loader that doesn't have proper dependency tracking. This one looks like it's adding dependencies properly

dav-is avatar Oct 14 '25 11:10 dav-is