1771751 - Add ESLint rule that rejects static import for system ESM from non-system ESM

Status: RESOLVED FIXED

(Developer Infrastructure :: Lint and Formatting, task)

Description

[Tooru Fujisawa [:arai]]

At the top-level of the system ESM, other system ESMs can be imported with static import declaration.
This works as expected only inside system ESM, and doing the same in non-system ESM (such as module loaded by <script type="module" ...> in chrome-priv window) doesn't work as expected:

• the module (and dependents) is loaded into different global than then shared global, and it's not the per-process singleton
• it doesn't immediately throw error, because system ESM is subset of standard ESM

And this situation likely to happen by copy-paste the code that imports system ESM, from system ESM to non-system ESM.

we'd better having an ESLint rule that detects such situation and throw error.

Comment 1

[Tooru Fujisawa [:arai]]

Attachment: Bug 1771751 - Reject static import for system module from non-system module. r?Standard8!

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/b94c92657e73
Reject static import for system module from non-system module. r=Standard8

Comment 3

[Noemi Erli[:noemi_erli]]

https://hg.mozilla.org/mozilla-central/rev/b94c92657e73