1945458 - Add support for TypeScript definitions for pre-processed import AppConstants.sys.mjs

Status: RESOLVED FIXED

(Developer Infrastructure :: Lint and Formatting, task, P3)

Description

[Mark Banner (:standard8)]

TypeScript currently errors about AppConstants.sys.mjs imports:

js/xpconnect/loader/XPCOMUtils.sys.mjs:7:30 - error TS2792: Cannot find module 'resource://gre/modules/AppConstants.sys.mjs'. Did you mean to set the 'moduleResolution' option to 'nodenext', or to add aliases to the 'paths' option?

7 import { AppConstants } from "resource://gre/modules/AppConstants.sys.mjs";
                               ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

If we tell it about where the file is, then it can't handle it, as it is a pre-processed file.

We need to figure out a way of at least simulating the types defined in the file, and preferably automating the update if that is possible.

Comment 1

[Tomislav Jovanovic :zombie]

Attachment: Bug 1945458 - Add substitution type declaration for pre-processeed AppConstants.sys.mjs r?Standard8

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/7f74859ff2a1
Add substitution type declaration for pre-processeed AppConstants.sys.mjs r=Standard8

Comment 3

[Sandor Molnar[:smolnar]]

https://hg.mozilla.org/mozilla-central/rev/7f74859ff2a1