… API

Current Behavior

• we allow export * from for internal modules
• export * from is dissallowed for external packages only for dedups/tree shake-ability reasons

// @filename src/index.ts
// OK  ✅
export * from './Foo'
// Error 🚨
export * from 'lib-a'

// @filename src/Foo/index.ts
// OK ✅
export * from './Foo'
// Error 🚨
export * from 'lib-a'

New Behavior

• to improve tree-shaking capabilities even further we should dissallow export * for all public API's
  • this already happens for react-components although only for "external" dependencies - THIS PR adds check for internal ones ✅

// @filename src/index.ts
// Error 🚨 - export * is bad idea for tree shake-ability on public API surface
export * from './Foo'
// Error 🚨
export * from 'lib-a'

// @filename src/Foo/index.ts
// OK ✅ - this is fine, no need to be explicit in every module
export * from './Foo'
// Error 🚨
export * from 'lib-a'

To recap new functionality:

• export * for external packages is dissallowed in every module
• public apis of packages (index.ts,index.tsx,index.js) are dissalowed to use export *

Related Issue(s)

Follows #22046

Remarks

• for easy gradual migration the lint is set to warn, after we fix all v9 packages gradually it will be propagated to error