Closed Bug 1790323 Opened 2 years ago Closed 2 years ago

[meta] Convert /remote JSM modules to ESM

Categories

(Remote Protocol :: Agent, task)

Product:
Remote Protocol
Component:
Agent
Type:
task

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: whimboo, Unassigned)

References

Details

(Keywords: meta, Whiteboard: [webdriver:m5])

For bug 1308512 we have to convert our JSM modules to ESM modules. This can easily be done by using the ./mach esmify script. Most of the work is done automatically, and in some cases it might require some manual fixes.

I would propose that we divide the work into the following batches:

  • components/, shared/ and server/
  • marionette/
  • webdriver-bidi/
  • cdp/

Each of these batches is most likely worth a single point.

Note that we still have a couple of module references from remote/shared to remote/marionette that we should get fixed first to avoid partial conversions of modules. This includes sync.js, cert.js, and accessibility.js.

If we all agree I'll go ahead in file individual bugs.

As Julian pointed out we should also fix the linting rules first for our remote/ folder. There is bug 1774367 for that work.

Depends on: 1774367

We will track for M5 and separate bugs should be filed. We may not need to move the modules to shared but lets check again.

Whiteboard: [webdriver:triage] → [webdriver:m5]
Flags: needinfo?(hskupin)
Blocks: webdriver:m5
Flags: needinfo?(hskupin)
No longer depends on: 1774367

This is all done.

Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.