Program visitor not being run for styled-components macro

[quantizor]

Relevant babel plugin code: styled-components/babel-plugin-styled-components@e3829d2#diff-1fdf421c05c1140f6d71444ea2b27638L13

Relevant macro code: https://github.com/styled-components/styled-components/blob/master/packages/styled-components/src/macro/index.js

Basically when I switched the root level JSXAttribute visitor into one that's a subtraversal of Program (necessary to create the component AST the other visitors then further modify) the part of the babel plugin inside the Program visitor stopped working for macro users.

Here's a repro sandbox: https://codesandbox.io/embed/magical-brook-ctyxs (styled-components@* and babel-plugin-styled-components@1.10.1, the text should be green.)

To be totally honest I generally am not great at writing Babel code so maybe I'm just doing something wrong? But it's odd because I have a bunch of tests in the repo to make sure the transforms happen properly and they do seem to function as expected, it's just this Program subtraversal that isn't working for the macro use case...

[kentcdodds]

Hi @probablyup!

I don't know what's up and I'm afraid I don't have the bandwidth to look into it right now :(

[quantizor]

One interesting thing is @jamesknelson was looking into this and switching from state.file.path.traverse() to using @babel/traverse inside the macro made it work. Maybe switching to that inside this library during applyMacros might work?

[kentcdodds]

I'm good with that solution is someone wants to make a pull request.

[bySabi]

@probablyup , I am trying to make a macro for this babel plugin: https://github.com/insin/babel-plugin-react-html-attrs/blob/master/lib/index.js
... and it is not clear to me if it can be done or not.

I think it's a problem similar to yours, what do you think?

Here it has also been mentioned in: #31

[github-actions]

🎉 This issue has been resolved in version 3.0.0 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀

[conartist6]

@probablyup Sorry to bother you. It looks like the fix implemented for this issue caused a regression for a different issue. I want to make sure we don't cause the issue described here again as we try to fix the regression, but unfortunately the code sandbox link your provided when you created this issue is now gone. Do you happen to have a copy of it, or can you help me understand what was happening?

[conartist6]

Actually I think I understand what you were talking about now. I see where you changed the plugin to traverse inside of the Program visitor. I'll dig into it some more tomorrow.