Issue with @react-aria/aria-modal-polyfill@3.4.1 distributed files

[JasonKaz]

🐛 Bug Report

The most recent update (3.4.1) to @react-aria/aria-modal-polyfill broke our local build process.

In version 3.4.0 this line was distributed as:
let addNode = Array.from(mutation.addedNodes).find(node => node.querySelector == null ? void 0 : node.querySelector('[aria-modal="true"], [data-ismodal="true"]'));

In 3.4.1 it is:
let addNode = Array.from(mutation.addedNodes).find((node)=>node.querySelector?.('[aria-modal="true"], [data-ismodal="true"]')

Our build broke because we are not transpiling the ?. operator for external modules. We could make it support that, but clearly something changed with the build process in this repo to change the output like that.

🤔 Expected Behavior

😯 Current Behavior

💁 Possible Solution

🔦 Context

💻 Code Sample

🌍 Your Environment

Software	Version(s)
react-spectrum
Browser
Operating System

🧢 Your Company/Team

Adobe/Admin Console team

🕷 Tracking Issue (optional)

[snowystinger]

We updated our list of supported browsers. So my guess is that's no longer being transpiled since the major browsers can handle this and have for a while now.

I'm surprised it broke your build process though, what part did it break? test environment? node environment?

[JasonKaz]

Local development environment, using Storybook. Our system was able to build and publish the module though. So there must be some mis-match between the build webpack vs the Storybook webpack.

[snowystinger]

might be related? storybookjs/storybook#14292

[exlame]

Chaining operator should now be removed after the last release.
#2860

From the discussion here :
#2857

They gave us until June to upgrade to Webpack 5 to support the new syntax