fix #1294: update the list of native node modules

Author: evanw

CHANGELOG.md

@@ -1,5 +1,11 @@
 # Changelog
 
+## Unreleased
+
+* Update the list of built-in node modules ([#1294](https://github.com/evanw/esbuild/issues/1294))
+
+    The list of built-in modules that come with node was outdated, so it has been updated. It now includes new modules such as `wasi` and `_http_common`. Modules in this list are automatically marked as external when esbuild's platform is configured to `node`.
+
 ## 0.12.0
 
 **This release contains backwards-incompatible changes.** Since esbuild is before version 1.0.0, these changes have been released as a new minor version to reflect this (as [recommended by npm](https://docs.npmjs.com/cli/v6/using-npm/semver/)). You should either be pinning the exact version of `esbuild` in your `package.json` file or be using a version range syntax that only accepts patch upgrades such as `~0.11.0`. See the documentation about [semver](https://docs.npmjs.com/cli/v6/using-npm/semver/) for more information.

internal/resolver/resolver.go

@@ -1681,44 +1681,60 @@ func IsPackagePath(path string) bool {
 }
 
 var BuiltInNodeModules = map[string]bool{
-	"assert":         true,
-	"async_hooks":    true,
-	"buffer":         true,
-	"child_process":  true,
-	"cluster":        true,
-	"console":        true,
-	"constants":      true,
-	"crypto":         true,
-	"dgram":          true,
-	"dns":            true,
-	"domain":         true,
-	"events":         true,
-	"fs":             true,
-	"http":           true,
-	"http2":          true,
-	"https":          true,
-	"inspector":      true,
-	"module":         true,
-	"net":            true,
-	"os":             true,
-	"path":           true,
-	"perf_hooks":     true,
-	"process":        true,
-	"punycode":       true,
-	"querystring":    true,
-	"readline":       true,
-	"repl":           true,
-	"stream":         true,
-	"string_decoder": true,
-	"sys":            true,
-	"timers":         true,
-	"tls":            true,
-	"trace_events":   true,
-	"tty":            true,
-	"url":            true,
-	"util":           true,
-	"v8":             true,
-	"vm":             true,
-	"worker_threads": true,
-	"zlib":           true,
+	"_http_agent":         true,
+	"_http_client":        true,
+	"_http_common":        true,
+	"_http_incoming":      true,
+	"_http_outgoing":      true,
+	"_http_server":        true,
+	"_stream_duplex":      true,
+	"_stream_passthrough": true,
+	"_stream_readable":    true,
+	"_stream_transform":   true,
+	"_stream_wrap":        true,
+	"_stream_writable":    true,
+	"_tls_common":         true,
+	"_tls_wrap":           true,
+	"assert":              true,
+	"async_hooks":         true,
+	"buffer":              true,
+	"child_process":       true,
+	"cluster":             true,
+	"console":             true,
+	"constants":           true,
+	"crypto":              true,
+	"dgram":               true,
+	"diagnostics_channel": true,
+	"dns":                 true,
+	"domain":              true,
+	"events":              true,
+	"fs":                  true,
+	"http":                true,
+	"http2":               true,
+	"https":               true,
+	"inspector":           true,
+	"module":              true,
+	"net":                 true,
+	"os":                  true,
+	"path":                true,
+	"perf_hooks":          true,
+	"process":             true,
+	"punycode":            true,
+	"querystring":         true,
+	"readline":            true,
+	"repl":                true,
+	"stream":              true,
+	"string_decoder":      true,
+	"sys":                 true,
+	"timers":              true,
+	"tls":                 true,
+	"trace_events":        true,
+	"tty":                 true,
+	"url":                 true,
+	"util":                true,
+	"v8":                  true,
+	"vm":                  true,
+	"wasi":                true,
+	"worker_threads":      true,
+	"zlib":                true,
 }