polish: remove check for collecting forbidden directives from execution

[yaacovCR]

follow on from #3974

[github-actions]

Hi @yaacovCR, I'm @github-actions bot happy to help you with this PR 👋

Supported commands

Please post this commands in separate comments and only one per comment:

• @github-actions run-benchmark - Run benchmark comparing base and merge commits for this PR
• @github-actions publish-pr-on-npm - Build package from this PR and publish it on NPM

[JoviDeCroock]

I personally find the boolean approach a lot cleaner, js engines will also have an easier time optimising the top level function with a flag compared to the adhoc created func.

It's also a lot harder to reason about as we add a layer of indirection and the forbidden directives become a hook. I.e. we create an exact replica of shouldIncludeNode that just has an extra step to push to an outer scope array.

[yaacovCR]

@JoviDeCroock

I suppose this is a style issue, I prefer the changes in this PR as I find it simpler to reason about, but not strongly enough that I would push this.

In terms of performance, I added a benchmark to demonstrate that there is no significant/reproducible performance penalty for the dynamic function, which also confirms my suspicion that there is no significant/reproducible performance hit for the current approach, so it's all in terms of readability, and I am fine to stick with the status quo, although see #4361.