Build successful! You can test your changes in the REPL here: https://babeljs.io/repl/build/60125

Open in StackBlitz

commit: 2a21c37

Thanks!
The issue comes from here,

We can just revert the change in

@liuxingbaoyu should I just remove this line instead of checking path?

Removing the !process.env.BABEL_8_BREAKING check should fix this regression.
This was added in #17043.
But please don't merge it yet; I'll try again later to see if there are any other issues.

@liuxingbaoyu as I see only removing check for !process.env.BABEL_8_BREAKING makes test fail, we also pass BABEL_8_BREAKING to run the test:

make build && BABEL_8_BREAKING=true yarn jest babel-traverse

So with this line or without, additional check for path is still neaded.

Pending @liuxingbaoyu's investigation result.

Well, forceSetScope is back. 🤦‍♂️
This is a bit complicated. I'll try to explain it.

I added forceSetScope or !process.env.BABEL_8_BREAKING && because noScope polluted the cache.
Later, we stopped using noScope in traverse.hasType, and the issue in tests no longer appeared.
The issue reappeared in #17563.

The fix we made in this PR conflicts with #17565, preventing it from fixing #17563.
So I merged them into this PR.