fix: __NO_SIDE_EFFECTS__ annotation for async function

[antfu]

This PR contains:

• bugfix
• feature
• refactor
• documentation
• other

Are tests included?

• yes (bugfixes and features will not be merged without tests)
• no

Breaking Changes?

• yes (breaking changes will not be merged unless absolutely necessary)
• no

List any relevant issue numbers:

Description

Thanks to @evanw for his finding: evanw/esbuild#3149 (comment). I missed async functions and generators in the initial PR. I think __NO_SIDE_EFFECTS__ should take higher priority than super.hasEffectsOnInteractionAtPath.

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
rollup	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jun 11, 2023 9:52pm

[codecov]

Codecov Report

Merging #5031 (5274c69) into master (de2d6ff) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master    #5031   +/-   ##
=======================================
  Coverage   98.96%   98.96%           
=======================================
  Files         222      222           
  Lines        8210     8213    +3     
  Branches     2258     2259    +1     
=======================================
+ Hits         8125     8128    +3     
  Misses         30       30           
  Partials       55       55           

Impacted Files	Coverage Δ
src/ast/nodes/ArrowFunctionExpression.ts	95.45% <100.00%> (+0.21%)	⬆️
src/ast/nodes/shared/FunctionBase.ts	98.36% <100.00%> (+0.05%)	⬆️

[rollup-bot]

This PR has been released as part of rollup@3.25.1. You can test it via npm install rollup.