[robhogan]

Summary

Inline requires only inlines requires assigned to an identifier, ie const foo = require('foo') or const foo = require('foo').foo. Despite what the plugin's doc comment says, const {foo, bar} = require('foo') has never been inlined - it only appeared to work because Babel presets used to lower destructuring into the identifier form first.

react/react-native#58965 stopped @react-native/babel-preset lowering destructuring for Hermes profiles, so since then destructured CommonJS requires are evaluated eagerly at module init. ESM imports aren't affected.

This adds direct support for top-level object patterns with plain, renamed or string keys. Each reference becomes require('foo').bar, exactly what the lowered form produced. With memoizeCalls, a module's properties share one memoised module, so each scope requires it at most once:

var _foo;
(_foo || (_foo = require("foo"))).a();
_foo.b.x();

Defaults, rest elements, nested patterns and computed keys are left alone, as are reassigned bindings and properties assigned on the module elsewhere - the same constraints as the existing identifier forms.

Changelog: [Performance] Inline destructured CommonJS requires (const {foo} = require('foo')) under inlineRequires

Test plan

New cases cover renamed and string keys, references across scopes and branches, unused requires and each unsupported pattern, with and without memoizeCalls.

End to end, I swapped this build into React Native's own Jest transform, which runs inline requires, on react/react-native#59303 (which preserves destructuring in that config). The 17 suites that relied on inline requires to load modules after their jest.mock calls pass in their original form, before react/react-native#59307 and react/react-native#59352 fixed them: 292 tests and 70 snapshots, against 110 failures without this change. RN's full suite passes too (6,289 tests).