Upgrade to TypeScript v5.0 and set `module{,Resolution}` option to `Node16` #3645

MajorLift · 2023-12-08T21:16:55Z

Explanation

TypeScript v5.0

As part of the Wallet Framework Team's OKRs (Q2 2024 O3KR4, Q3 2024 O2KR4), we are upgrading TypeScript to v5.0+ for all packages in the core monorepo.

These upgrades will give us access to new features, aid us in writing more type-safe and modern code, and also allow us to reach parity with Extension and other MetaMask projects.

`Node16`

In order to maximize the benefits of this upgrade, we are also enabling the Node16 setting for the module and moduleResolution tsconfig options.

Motivation

Interop: CJS modules referencing ESM modules

The core monorepo is a collection of CJS packages, which use CJS module resolution rules internally, and are treated as CJS modules by Node.js. This is true despite that fact that these packages are authored in TypeScript using ESM syntax (import/export statements) and are set up to output dual builds for both CJS and ESM.

With Node16 or NodeNext enabled, CJS modules are unable to reference ESM modules via static/synchronous import statements, as TypeScript assumes them to be compiled down to CJS-only require statements.

There are three solutions for this issue, of which we are utilizing the first two:

Update the ESM-only dependency so that it outputs a CJS build and type declaration as well.
Replace the static import statements with dynamic import syntax (which, based on CJS emit rules, are not transformed to require statements).
Migrate our module to ESM (by setting "type": "module" in package.json and renaming all of our scripts to *.cjs)

See https://www.typescriptlang.org/docs/handbook/modules/reference.html#interoperability-rules

With dependencies that we control or use extensively, we pursue the first option, as we're doing with superstruct and @metamask/utils (and all of the many core dependencies that are downstream of either or both).

With dependencies that see more limited usage, we are opting for either the second option (e.g. multiformats) or, if available, using a CJS-compatible alternative (e.g. replacing lodash-es with lodash).

The third solution of migrating to ESM is the most fundamental, long-term measure, but we are refraining from it at this stage until we can make a concerted effort to migrate our codebase as a whole. This is because any individual ESM migration can cause a cascading effect through the dependency tree where other packages are now required to migrate as well.

Reasons for moving away from `node`/`node10`

The following outlines the motivation for switching to Node16, backed by relevant entries from the official TypeScript documentation.

Starting with TypeScript v5.0, the previously used moduleResolution setting node is renamed to node10, and strongly discouraged from usage.
- "It reflects the CommonJS module resolution algorithm as it existed in Node.js versions earlier than v12. It should no longer be used."
- "Because node16 and nodenext are the only module options that reflect the complexities of Node.js’s dual module system, they are the only correct module options for all apps and libraries that are intended to run in Node.js v12 or later, whether they use ES modules or not."
The node10 setting is unable to guarantee correct module resolution for ESM-only dependencies.
- "Because Node.js v12 introduced different module resolution rules for ES modules, though, it’s a very bad model of modern versions of Node.js. It should not be used for new projects."
- "node16 and nodenext describe the full range of behavior for Node.js’s dual-format module system, and emit files in either CommonJS or ESM format. This is different from every other module option, which are runtime-agnostic and force all output files into a single format, leaving it to the user to ensure the output is valid for their runtime."
The node10 setting does not support the package.json "exports" field, which is used in our libraries to expose dual builds and type declarations.
- https://www.typescriptlang.org/docs/handbook/modules/reference.html#packagejson-exports
The Node16 and NodeNext settings maximize downstream compatibility.
- "When compiling a library, you don’t know where the output code will run, but you’d like it to run in as many places as possible. Using "module": "nodenext" (along with the implied "moduleResolution": "nodenext") is the best bet for maximizing the compatibility of the output JavaScript’s module specifiers, since it will force you to comply with Node.js’s stricter rules for import module resolution."
- ""moduleResolution": "nodenext" is only checking that the output works in Node.js, but in most cases, module code that works in Node.js will work in other runtimes and in bundlers"

`Node16` vs. `NodeNext`

The two settings are currently identical, and Node16 is intended to work with all current node versions v16 or higher. If additional capabilities are added to NodeNext or Node18/Node20 that we want to apply to our codebases, we will be able to introduce them after checking for disruptive regressions or breaking changes.

Relationship with other tsconfig options

--module nodenext or node16 implies and enforces the moduleResolution with the same name (and vice versa).
--module node16 implies (up to) --target es2022.
- --module nodenext implies (up to) --target esnext.

https://www.typescriptlang.org/docs/handbook/modules/reference.html#implied-and-enforced-options

Description

Replace superstruct dependency with @metamask/superstruct ^3.0.0.
- ^3.1.0
Replace all superstruct import statements with @metamask/superstruct
Bump @metamask/utils to ^8.5.0.
- ^9.0.0
  - remove yarn resolution to @metamask/superstruct@npm:3.1.0
Bump typescript to ~5.0.4
- Set module and moduleResolution tsconfig options to Node16
- ~~Bump TypeScript module output options target and lib to ES2022 #4507~~

Further context on why the superstruct and utils changes are necessary:

Release order roadmap

Due to interdependencies between the packages involved in this PR, we will need to update and release them in a specific order:

References

Contributes to Upgrade to TypeScript v5.0 #3651
Blocked by:
- superstruct
- utils
Blocked by downstream consumers of superstruct, utils:
Blocked by snaps monorepo releases
- snaps-sdk, snaps-utils
  - keyring-api
    - chore(deps): replace superstruct imports with @metamask/superstruct keyring-api#328
- snaps-controllers
  - eth-snap-keyring
    - chore(deps): replace superstruct imports with @metamask/superstruct eth-snap-keyring#311

Changelog

`@metamask/accounts-controller`

### Changed

- Bump `@metamask/keyring-api` from `^8.0.0` to `^8.0.1` ([#3645](https://github.com/MetaMask/core/pull/3645))
- Bump `@metamask/snaps-sdk` from `^4.2.0` to `^6.1.0` ([#3645](https://github.com/MetaMask/core/pull/3645))
- Bump `@metamask/snaps-utils` from `^7.4.0` to `^7.8.0` ([#3645](https://github.com/MetaMask/core/pull/3645))
- Bump peer dependency `@metamask/snaps-controllers` from `^8.1.1` to `^9.3.0` ([#3645](https://github.com/MetaMask/core/pull/3645))

`@metamask/assets-controllers` (major)

### Changed

- **BREAKING:** `getIpfsCIDv1AndPath`, `getFormattedIpfsUrl` are now async functions ([#3645](https://github.com/MetaMask/core/pull/3645))
- Add `immer` `^9.0.6` as a new dependenc. ([#3645](https://github.com/MetaMask/core/pull/3645))
- Bump `@metamask/abi-utils` from `^2.0.2` to `^2.0.3` ([#3645](https://github.com/MetaMask/core/pull/3645))
- Bump `multiformats` from `^9.5.2` to `^13.1.0` ([#3645](https://github.com/MetaMask/core/pull/3645))

`@metamask/chain-controller`

### Changed

- Bump `@metamask/chain-api` from `^0.0.1` to `^0.1.0` ([#3645](https://github.com/MetaMask/core/pull/3645))
- Bump `@metamask/keyring-api` from `^8.0.0` to `^8.0.1` ([#3645](https://github.com/MetaMask/core/pull/3645))
- Bump `@metamask/snaps-controllers` from `^8.1.1` to `^9.3.0` ([#3645](https://github.com/MetaMask/core/pull/3645))
- Bump `@metamask/snaps-sdk` from `^4.2.0` to `^6.1.0` ([#3645](https://github.com/MetaMask/core/pull/3645))
- Bump `@metamask/snaps-utils` from `^7.4.0` to `^7.8.0` ([#3645](https://github.com/MetaMask/core/pull/3645))

`@metamask/keyring-controller`

### Changed

- Bump `@metamask/eth-simple-keyring` from `^6.0.1` to `^6.0.2` ([#3645](https://github.com/MetaMask/core/pull/3645))
- Bump `@metamask/keyring-api` from `^8.0.0` to `^8.0.1` ([#3645](https://github.com/MetaMask/core/pull/3645))
- Set yarn resolution for `@metamask/snaps-sdk/@metamask/providers` to `17.1.1` ([#3645](https://github.com/MetaMask/core/pull/3645))
  - Remove once `@metamask/snaps-sdk` bumps its `@metamask/providers` version to `>=17.1.1`.

`@metamask/network-controller` (minor)

### Added

- Newly exports the following types: `AutoManagedNetworkClient`, `InfuraNetworkClientConfiguration`, `CustomNetworkClientConfiguration` ([#3645](https://github.com/MetaMask/core/pull/3645))

`@metamask/profile-sync-controller`

### Changed

- Bump dependency and peer dependency `@metamask/snaps-controllers` from `^8.1.1` to `^9.3.0` ([#3645](https://github.com/MetaMask/core/pull/3645))
- Bump `@metamask/snaps-sdk` from `^4.2.0` to `^6.1.0` ([#3645](https://github.com/MetaMask/core/pull/3645))
- Bump `@metamask/snaps-utils` from `^7.4.0` to `^7.8.0` ([#3645](https://github.com/MetaMask/core/pull/3645))

`@metamask/transaction-controller`

### Changed

- Bump `@metamask/keyring-api` from `^8.0.0` to `^8.0.1` ([#3645](https://github.com/MetaMask/core/pull/3645))

`@metamask/user-operation-controller`

### Fixed

- Replace `superstruct` with ESM-compatible `@metamask/superstruct` `^3.1.0` ([#3645](https://github.com/MetaMask/core/pull/3645))
  - This fixes the issue of this package being unusable by any TypeScript project that uses `Node16` or `NodeNext` as its `moduleResolution` option.

Checklist

I've updated the test suite for new or updated code as appropriate
I've updated documentation (JSDoc, Markdown, etc.) for new or updated code as appropriate
I've highlighted breaking changes using the "BREAKING" category above as appropriate

mcmire · 2024-03-14T23:17:52Z

I worked on this a bit and by bumping some packages and also using a local copy of utils which makes use of @metamask/superstruct instead of superstruct, I managed to get yarn build:types working. I created a separate branch and made a commit there with the necessary changes, and you can see the package bumps here: df14ce8. You'll want to use your own local copy of utils with the use-metamask-superstruct branch checked out (make sure to run yarn build first).

I haven't tried running yarn test yet, so I don't know if that works. I will try that next.

socket-security · 2024-06-25T19:43:57Z

👍 Dependency issues cleared. Learn more about Socket for GitHub ↗︎

This PR previously contained dependency changes with security issues that have been resolved, removed, or ignored.

Ignoring: npm/micromatch@4.0.7, npm/node-gyp@10.2.0, npm/validate-npm-package-name@5.0.1

View full report↗︎

Next steps

Take a deeper look at the dependency

Take a moment to review the security alert above. Review the linked package source code to understand the potential risk. Ensure the package is not malicious before proceeding. If you're unsure how to proceed, reach out to your security team or ask the Socket team for help at support [AT] socket [DOT] dev.

Remove the package

If you happen to install a dependency that Socket reports as Known Malware you should immediately remove it and select a different dependency. For other alert types, you may may wish to investigate alternative packages or consider if there are other ways to mitigate the specific risk posed by the dependency.

Mark a package as acceptable risk

To ignore an alert, reply with a comment starting with @SocketSecurity ignore followed by a space separated list of ecosystem/package-name@version specifiers. e.g. @SocketSecurity ignore npm/foo@1.0.0 or ignore all packages with @SocketSecurity ignore-all

…erstruct` in all core nested dependencies

…7.8.0`, `@metamask/snaps-controllers` to `^9.3.0`

…mask/providers` from `17.0.0` to `17.1.1`

MajorLift

Notes for reviewers

MajorLift · 2024-07-19T18:18:31Z

packages/assets-controllers/src/assetsUtil.ts

  const cid = index !== -1 ? url.substring(0, index) : url;
  const path = index !== -1 ? url.substring(index) : undefined;

+  const { CID } = await import('multiformats');


multiformats is an ESM-only package, and we are replacing its import statement with dynamic import syntax.

This causes the following changes:

getIpfsCIDv1AndPath, getFormattedIpfsUrl are now async functions (this is a breaking change for assets-controllers).

The --experimental-vm-modules flag needs to be prepended to our build scripts that run jest, with accompanying updates to our yarn constraints.

How did this work previously if it's ESM-only?

So far as I can tell, our previous moduleResolution option node/node10 will happily pass ESM-only packages into require calls without raising any warnings.

So it's likely the warnings we're now seeing with node16 should have been there all along. The package migrated to ESM 4 years ago (multiformats/js-multiformats@f925195#diff-7ae45ad102eab3b6d7e7896acd08c427a9b25b346470d7bc6507b6481575d519).

At runtime, either the module was interoperable by coincidence (not quite sure if this is possible or likely -- maybe if there's no ESM-specific syntax?), or it was breaking and we didn't notice.

MajorLift · 2024-07-19T18:18:53Z

constraints.pro


 % All non-root packages must have the same "test" script.
-gen_enforced_field(WorkspaceCwd, 'scripts.test', 'jest --reporters=jest-silent-reporter') :-
+gen_enforced_field(WorkspaceCwd, 'scripts.test', 'NODE_OPTIONS=--experimental-vm-modules jest --reporters=jest-silent-reporter') :-


The --experimental-vm-modules flag is necessary if there are dynamic imports in the compiled code used by our tests.

MajorLift · 2024-07-19T18:19:27Z

packages/network-controller/src/index.ts

+export type { AutoManagedNetworkClient } from './create-auto-managed-network-client';
 export * from './NetworkController';
 export * from './constants';
 export type { BlockTracker, Provider } from './types';
-export type { NetworkClientConfiguration } from './types';
+export type {
+  NetworkClientConfiguration,
+  InfuraNetworkClientConfiguration,
+  CustomNetworkClientConfiguration,
+} from './types';


These exports accommodate the new import statements in Token{Detection,Rate}Controller.test.ts.

Previously, these types were accessed through subpath imports, which are now not supported unless specified in the library's package.json exports field.

MajorLift · 2024-07-19T18:20:02Z

.depcheckrc.yml

  - '@metamask/create-release-branch'
  - 'depcheck'
  - 'eslint-interactive'
+  - 'rimraf'


Only listed as a devDep in the root package.json, where it's used in a build script but not in any modules.

MajorLift · 2024-07-19T18:37:38Z

packages/assets-controllers/src/TokenRatesController.test.ts

 import { useFakeTimers } from 'sinon';

 import { advanceTime } from '../../../tests/helpers';
+import { createMockInternalAccount } from '../../accounts-controller/src/tests/mocks';


With the Node{16,Next} option, subpath imports are no longer supported unless explicitly defined in the source package manifest's "exports" field.

Here, an import from @metamask/accounts-controller/src/tests/mocks needed to be replaced with a relative path import.

Ah I'm not sure how this slipped through the cracks! Makes sense.

MajorLift · 2024-07-19T18:38:55Z

package.json

    "pre-push": "yarn lint"
  },
  "resolutions": {
+    "@metamask/snaps-sdk@6.1.0/@metamask/providers": "17.1.1",


Fixes build error due to package version mismatch: MetaMask/core/actions/runs/10011688901/job/27675682526?pr=3645

Removal is blocked by a @metamask/snaps-sdk release with @metamask/providers bumped to >=17.1.1.

See: Fix regressions introduced by @metamask/providers@17.1.1

MajorLift · 2024-07-19T18:38:59Z

packages/accounts-controller/CHANGELOG.md

+### Uncategorized
+
+- Please hold off on new releases of this package until the yarn resolution for `@metamask/providers` is removed.
+  - This is blocked by a `@metamask/snaps-sdk` release with `@metamask/providers` bumped to `>=17.1.1`.
+    - See: [Fix regressions introduced by @metamask/providers@17.1.1](https://github.com/MetaMask/snaps/pull/2579)
+  - Build error fixed by yarn resolution: [MetaMask/core/actions/runs/10011688901/job/27675682526?pr=3645](https://github.com/MetaMask/core/actions/runs/10011688901/job/27675682526?pr=3645)
+


The packages affected by the yarn resolution are {accounts,chain,profile-sync}-controller. It's safe to temporarily merge the yarn resolution into main, as long as these packages aren't released (and even if they are @metamask/providers is not a direct dependency of any of these packages).

In any case, these three packages will be excluded from the core releases that follow this PR, since they are blocked by a snaps-controllers release as well.

mcmire

I obviously underestimated how long this would take. Fantastic work on keeping track and staying on top of all of the downstream upgrades in order to unlock this PR 🎉

MajorLift · 2024-07-19T21:23:18Z

packages/transaction-controller/package.json

    "@metamask/eth-json-rpc-provider": "^4.1.1",
    "@metamask/ethjs-provider-http": "^0.3.0",
-    "@metamask/keyring-api": "^8.0.0",
+    "@metamask/keyring-api": "^8.0.1",


For the @MetaMask/confirmations team: this is the only change in this PR to a confirmations-owned package (aside from the typescript bumps and test build script updates on every package).

…tion for `@metamask/providers` (#4547) ## References - Contributes to #3651 - Follows from #3645 ## Changelog ### `@metamask/accounts-controller` ```md ### Changed - Bump `@metamask/snaps-sdk` from `^6.1.0` to `^6.1.1` ([#4547](#4547)) - Bump `@metamask/snaps-utils` from `^7.8.0` to `^7.8.1` ([#4547](#4547)) ``` ### `@metamask/chain-controller` ```md ### Changed - Bump `@metamask/snaps-controllers` from `^9.3.0` to `^9.3.1` ([#4547](#4547)) - Bump `@metamask/snaps-sdk` from `^6.1.0` to `^6.1.1` ([#4547](#4547)) - Bump `@metamask/snaps-utils` from `^7.8.0` to `^7.8.1` ([#4547](#4547)) ``` ### `@metamask/profile-sync-controller` ```md ### Changed - Bump `@metamask/snaps-controllers` from `^9.3.0` to `^9.3.1` ([#4547](#4547)) - Bump `@metamask/snaps-sdk` from `^6.1.0` to `^6.1.1` ([#4547](#4547)) - Bump `@metamask/snaps-utils` from `^7.8.0` to `^7.8.1` ([#4547](#4547)) ``` ## Checklist - [x] I've updated the test suite for new or updated code as appropriate - [x] I've updated documentation (JSDoc, Markdown, etc.) for new or updated code as appropriate - [x] I've highlighted breaking changes using the "BREAKING" category above as appropriate

Update the template used by the `create-package` script. The previous template was no longer compatible with our Yarn constraints. Here are the PRs related to these specific changes: * #4648 * #3645 * #1390 * #3668

## Explanation Update the template used by the `create-package` script. The previous template was no longer compatible with our Yarn constraints. ## References Here are the PRs related to these specific changes: * #4648 * #3645 * #1390 * #3668 ## Changelog N/A ## Checklist - [x] I've updated the test suite for new or updated code as appropriate - [x] I've updated documentation (JSDoc, Markdown, etc.) for new or updated code as appropriate - [x] I've communicated my changes to consumers by [updating changelogs for packages I've changed](https://github.com/MetaMask/core/tree/main/docs/contributing.md#updating-changelogs), highlighting breaking changes as necessary - [x] I've prepared draft pull requests for clients and consumer packages to resolve any breaking changes

MajorLift force-pushed the 231208-typescript-module-resolution branch from f14624e to 0029d3b Compare December 9, 2023 00:37

MajorLift added the team-wallet-framework Deprecated: Please use `team-core-platform` instead. label Dec 12, 2023

mcmire mentioned this pull request Mar 14, 2024

Cannot use this in a project with module and moduleResolution of nodenext/node16 MetaMask/utils#163

Closed

MajorLift force-pushed the 231208-typescript-module-resolution branch from ee34373 to 5b2d07d Compare May 9, 2024 15:30

MajorLift force-pushed the 231208-typescript-module-resolution branch from 3ee4229 to 26937bc Compare May 22, 2024 00:57

MajorLift self-assigned this May 22, 2024

MajorLift changed the title ~~Update tsconfig module, moduleResolution options to Node16~~ Update tsconfig module, moduleResolution options to NodeNext May 22, 2024

MajorLift force-pushed the 231208-typescript-module-resolution branch 3 times, most recently from 49aa7ea to 92db09e Compare May 22, 2024 21:32

MajorLift force-pushed the 231208-typescript-module-resolution branch from 92db09e to 94b220f Compare May 31, 2024 20:31

MajorLift changed the title ~~Update tsconfig module, moduleResolution options to NodeNext~~ Bump TypeScript to ^5.0.4 and set module, moduleResolution to NodeNext May 31, 2024

MajorLift force-pushed the 231208-typescript-module-resolution branch 2 times, most recently from f5d774c to e0487db Compare June 3, 2024 01:27

MetaMask deleted a comment from socket-security bot Jun 3, 2024

MajorLift force-pushed the 231208-typescript-module-resolution branch 3 times, most recently from b459ff2 to 5cc2651 Compare June 3, 2024 20:06

This was referenced Jun 5, 2024

Upgrade to TypeScript v5.1 #4370

Closed

9.0.0 MetaMask/utils#190

Closed

MajorLift force-pushed the 231208-typescript-module-resolution branch from 0454e0e to 903468e Compare June 25, 2024 19:45

MajorLift and others added 7 commits June 27, 2024 13:52

Bump TypeScript to ~5.0.4

bf479b4

Set tsconfig module, moduleResolution options to NodeNext

5084b8f

'yarn build:types' works

1d77aa8

Apply yarn patches to replace superstruct imports with `@metamask/sup…

dd45b15

…erstruct` in all core nested dependencies

Linter fixes

6e73a35

Bump TypeScript to ~5.0.4 in remaining packages

cad9be5

Fix missing yarn patch for @metamask/keyring-api@^6.1.1

a014d03

MajorLift removed the DO-NOT-MERGE label Jul 18, 2024

MajorLift force-pushed the 231208-typescript-module-resolution branch 2 times, most recently from c4cc6dd to 9ebfa84 Compare July 19, 2024 01:57

Bump @metamask/snaps-sdk to ^6.1.0, @metamask/snaps-utils to `^…

59fe394

…7.8.0`, `@metamask/snaps-controllers` to `^9.3.0`

MajorLift force-pushed the 231208-typescript-module-resolution branch from 9ebfa84 to 59fe394 Compare July 19, 2024 17:02

Merge branch 'main' into 231208-typescript-module-resolution

6b0075d

MajorLift force-pushed the 231208-typescript-module-resolution branch from c273024 to 6b0075d Compare July 19, 2024 17:05

Set yarn resolution for @metamask/snaps-sdk@6.1.0 dependency `@meta…

f50b209

…mask/providers` from `17.0.0` to `17.1.1`

MajorLift force-pushed the 231208-typescript-module-resolution branch from 986f16f to f50b209 Compare July 19, 2024 18:03

MajorLift requested a review from a team July 19, 2024 18:11

MajorLift commented Jul 19, 2024

View reviewed changes

Mrtenz approved these changes Jul 19, 2024

View reviewed changes

mcmire approved these changes Jul 19, 2024

View reviewed changes

This was referenced Jul 19, 2024

Bump @metamask/snaps-sdk from ^6.0.0 to ^6.1.0 MetaMask/keyring-api#358

Merged

Bump @metamask/snaps-{sdk,utils,controllers} MetaMask/eth-snap-keyring#363

Merged

MajorLift commented Jul 19, 2024

View reviewed changes

MajorLift requested a review from a team July 19, 2024 21:51

MajorLift mentioned this pull request Jul 20, 2024

Bump TypeScript module output options target and lib to ES2022 #4507

Open

matthewwalsh0 approved these changes Jul 22, 2024

View reviewed changes

Merge branch 'main' into 231208-typescript-module-resolution

5f0b948

MajorLift merged commit 668e96a into main Jul 22, 2024

MajorLift deleted the 231208-typescript-module-resolution branch July 22, 2024 13:17

MajorLift mentioned this pull request Jul 23, 2024

Bump @metamask/snaps-{controllers,sdk,utils} and remove yarn resolution for @metamask/providers #4547

Merged

3 tasks

Gudahtt mentioned this pull request Jul 26, 2024

Release 180.0.0 #4548

Merged

3 tasks

MajorLift mentioned this pull request Oct 8, 2024

feat: upgrade assets-controllers to v38.2.0 MetaMask/metamask-extension#27629

Merged

7 tasks

Gudahtt mentioned this pull request Jul 16, 2025

fix: Update create-package template #6130

Merged

4 tasks

Uh oh!

Upgrade to TypeScript v5.0 and set module{,Resolution} option to Node16 #3645

Upgrade to TypeScript v5.0 and set module{,Resolution} option to Node16 #3645

Uh oh!

Conversation

MajorLift commented Dec 8, 2023 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Explanation

TypeScript v5.0

Node16

Motivation

Interop: CJS modules referencing ESM modules

Reasons for moving away from node/node10

Node16 vs. NodeNext

Relationship with other tsconfig options

Description

Release order roadmap

References

Changelog

@metamask/accounts-controller

@metamask/assets-controllers (major)

@metamask/chain-controller

@metamask/keyring-controller

@metamask/network-controller (minor)

@metamask/profile-sync-controller

@metamask/transaction-controller

@metamask/user-operation-controller

Checklist

Uh oh!

mcmire commented Mar 14, 2024

Uh oh!

socket-security bot commented Jun 25, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Next steps

Uh oh!

MajorLift left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

MajorLift Jul 19, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

mcmire left a comment

Choose a reason for hiding this comment

Uh oh!

MajorLift Jul 19, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

Upgrade to TypeScript v5.0 and set `module{,Resolution}` option to `Node16` #3645

Upgrade to TypeScript v5.0 and set `module{,Resolution}` option to `Node16` #3645

MajorLift commented Dec 8, 2023 •

edited

Loading

`Node16`

Reasons for moving away from `node`/`node10`

`Node16` vs. `NodeNext`

`@metamask/accounts-controller`

`@metamask/assets-controllers` (major)

`@metamask/chain-controller`

`@metamask/keyring-controller`

`@metamask/network-controller` (minor)

`@metamask/profile-sync-controller`

`@metamask/transaction-controller`

`@metamask/user-operation-controller`

socket-security bot commented Jun 25, 2024 •

edited

Loading

MajorLift Jul 19, 2024 •

edited

Loading

MajorLift Jul 19, 2024 •

edited

Loading