Netlify deploy preview

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

Bundle size report

No bundle size changes (Toolpad)
No bundle size changes

Generated by 🚫 dangerJS against ce3e799

I'm not sure what we gain from this refactor, the code still contains an ignore directive, which means the whole file still doesn't get transformed.

I believe the canonical solution to this problem would be to expose the setNode in a context instead of the ref. But I could be wrong, I have just as little mileage with React 19 as the next guy 🙂

I'm not sure what we gain from this refactor, the code still contains an ignore directive, which means the whole file still doesn't get transformed.

I believe the canonical solution to this problem would be to expose the setNode in a context instead of the ref. But I could be wrong, I have just as little mileage with React 19 as the next guy 🙂

Yea noticed CI testlint failing now when i turned the eslintrc check back to false again with "// eslint-disable-next-line react-compiler/react-compiler" with now giving errors ..

I would argue that im still new to open source development and MUI project, so thankful for the fast feedback! Will look into exposing the setNode into the context instead with that seeming to be a more thoughtful approach! 😄

yes, but even that, isn't this just a false positive in the eslint plugin? I assume it's only complaining because it can't infer that in fact rootNode is a mutable ref.

yes, but even that, isn't this just a false positive in the eslint plugin? I assume it's only complaining because it can't infer that in fact rootNode is a mutable ref.

Yeah, that makes sense - it does seem like a false positive, especially since setting eslint.rc to false resolves it. And yes, rootNode is indeed mutable. Should we consider adjusting the ESLint configuration if the inline comment isn’t working as intended, or how should we proceed forward?

Suggested push from feedback from @Janpot with:

• changed setRootNode to setNode (my previous pr commit)
• exposed the setNode inside the CodeBlockContext and passing arguments to setNode directly
• added useMemo to prevent rerendering inside the codeCopy provider with contextValue

This removed the eslint error and tested it with compiling and used codeCopy and seems to work 😃

So open for feedback/suggestions/ Reviews!

Hello! Is there an update or possibility to get a review @Janpot @aarongarciah :)