12714633836
100%

Ran 10 Jan 2025 05:55PM UTC

Jobs 1

Files 1

Run time 1min

Badge

Embed ▾

Committed 10 Jan 2025 05:53PM UTC coverage: 100.0%. Remained the same

Build # 12714633836

Build Type

push

github

Committed by

wesleytodd

Commit Message

fix(refactor): prefix built-in node module imports

Since v5 relies on node >= 18, this is now possible (since v16, v14.18.0
[^1][^2]).

It's functionally irrelevant:
1. It's not required for CJS nor ESM (with a few exceptions [^3])
2. It has no performance promises

However, there are upsides to this approach:
1. It brings clear boundaries to what's a built-in and what's an
external dependency
2. It reduces the risk of importing unwanted deps where a built-in is
expected
3. It's slightly more interoperable with other JS runtimes that provide
node compatibility[^4], albeit only during development. Once imported
from npm, built-ins are assumed.

[^1]:https://nodejs.org/docs/latest-v22.x/api/modules.html#built-in-modules
[^2]:https://github.com/nodejs/node/pull/37246
[^3]:https://nodejs.org/api/modules.html#built-in-modules-with-mandatory-node-prefix
[^4]:https://docs.deno.com/runtime/fundamentals/node/#using-node's-built-in-modules

Run Details

1 of 1 relevant lines covered (100.0%)

6.0 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
1	12714633836.1	10 Jan 2025 05:55PM UTC	1	100.0	GitHub Action Run

expressjs / express / 12714633836
100%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 12714633836

expressjs / express / 12714633836 100%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 12714633836

expressjs / express / 12714633836
100%

README BADGES
x