[mui-codemod] Fix findComponentJSX util

[sai6855]

How to test the fix

1. go to https://github.com/mui/material-ui/blob/master/packages/mui-codemod/src/deprecations/accordion-props/accordion-props.js in master branch
2. add console.log(elementPath.node.openingElement.name.name); below this line

   material-ui/packages/mui-codemod/src/deprecations/accordion-props/accordion-props.js

   Line 14 in 97f225c

   findComponentJSX(j, { root, componentName: 'Accordion' }, (elementPath) => {

   and run node packages/mui-codemod/codemod deprecations/accordion-props packages/mui-codemod/src/deprecations/accordion-props/test-cases/actual.js
3. Notice it logs Accordion 2 times, everything is good until now
4. Now change componentName: "Accordion" to componentName: "sank" (or any random value) on this line and run node packages/mui-codemod/codemod deprecations/accordion-props packages/mui-codemod/src/deprecations/accordion-props/test-cases/actual.js
5. Notice it logs Accordion again 2 times, as per my understanding findComponentJSX function should filter out components which matches with componentName but i don't think that's the case now. This PR fixes it

Note: I'm really new to codemods and Regex, apologies in advance if my understanding or logic is wrong

• I have followed (at least) the PR section of the contributing guide.

[mui-bot]

Netlify deploy preview

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

Bundle size report

No bundle size changes (Toolpad)
No bundle size changes

Generated by 🚫 dangerJS against 8b714f8

[DiegoAndai]

Thanks for catching this @sai6855

[DiegoAndai]

We can make it simpler:

Suggested change

	.filter(
	(path) =>
	path.node.source.value.match(new RegExp(`^@mui/material/(${componentName})`)) ||
	path.node.source.value === '@mui/material',
	.filter((path) =>
	path.node.source.value.match(new RegExp(`^@mui/material(/${componentName})?$`)),

It's the same as your fix, as $ indicates to match the end of the line, so it's either

• string start (^) -> @mui/material -> string end ($)
• string start (^) -> @mui/material/componentName -> string end ($)

[sai6855]

Thanks for clear explaination. fixed in this commit 8b714f8

[DiegoAndai]

Nice! Did you test if it's fixed on your side after the refactor?

[sai6855]

Yes, ran codemod script. Now util is filtering out data correctly