chore: use Node protocol imports (remix-run#7000)

Co-authored-by: Mark Dalgleish <[email protected]>

docs/guides/migrating-react-router-app.md

@@ -46,7 +46,7 @@ Let's start by creating two new files:
 <docs-info>All of your app code in Remix will live in an `app` directory by convention. If your existing app uses a directory with the same name, rename it to something like `src` or `old-app` to differentiate as we migrate to Remix.</docs-info>
 
 ```tsx filename=app/entry.server.tsx
-import { PassThrough } from "stream";
+import { PassThrough } from "node:stream";
 
 import type {
   AppLoadContext,

docs/guides/resource-routes.md

@@ -124,7 +124,7 @@ export const action = async ({ request }: ActionArgs) => {
 Resource routes can be used to handle webhooks. For example, you can create a webhook that receives notifications from GitHub when a new commit is pushed to a repository:
 
 ```tsx
-import crypto from "crypto";
+import crypto from "node:crypto";
 
 import type { ActionArgs } from "@remix-run/node"; // or cloudflare/deno
 import { json } from "@remix-run/node"; // or cloudflare/deno

docs/guides/streaming.md

@@ -81,7 +81,7 @@ Let's take a dive into how to accomplish this.
 First, to enable streaming with React 18, you'll update your `entry.server.tsx` file to use `renderToPipeableStream`. Here's a simple (and incomplete) version of that:
 
 ```tsx filename=app/entry.server.tsx lines=[1,10,20,27,32,37]
-import { PassThrough } from "stream";
+import { PassThrough } from "node:stream";
 
 import { Response } from "@remix-run/node"; // or cloudflare/deno
 import type {
@@ -131,7 +131,7 @@ export default function handleRequest(
 This handles errors and properly disables streaming for bots which you typically want to force waiting, so you can display all the content for SEO purposes.
 
 ```tsx filename=app/entry.server.tsx
-import { PassThrough } from "stream";
+import { PassThrough } from "node:stream";
 
 import { Response } from "@remix-run/node"; // or cloudflare/deno
 import type {

docs/other-api/adapter.md

@@ -97,7 +97,7 @@ module.exports = createRequestHandler({
 Here's an example with Netlify:
 
 ```ts
-const path = require("path");
+const path = require("node:path");
 
 const {
   createRequestHandler,

integration/cf-compiler-test.ts

@@ -1,6 +1,6 @@
 import { test, expect } from "@playwright/test";
-import fs from "fs/promises";
-import path from "path";
+import fs from "node:fs/promises";
+import path from "node:path";
 import shell from "shelljs";
 import glob from "glob";
 

integration/compiler-test.ts

@@ -1,7 +1,7 @@
-import path from "path";
+import path from "node:path";
 import fse from "fs-extra";
 import { test, expect } from "@playwright/test";
-import { PassThrough } from "stream";
+import { PassThrough } from "node:stream";
 
 import {
   createFixture,
@@ -55,7 +55,7 @@ test.describe("compiler", () => {
         `,
         "app/routes/built-ins.jsx": js`
           import { useLoaderData } from "@remix-run/react";
-          import * as path from "path";
+          import * as path from "node:path";
 
           export let loader = () => {
             return path.join("test", "file.txt");
@@ -67,7 +67,7 @@ test.describe("compiler", () => {
         `,
         "app/routes/built-ins-polyfill.jsx": js`
           import { useLoaderData } from "@remix-run/react";
-          import * as path from "path";
+          import * as path from "node:path";
 
           export default function BuiltIns() {
             return <div id="built-ins-polyfill">{path.join("test", "file.txt")}</div>;
@@ -228,7 +228,7 @@ test.describe("compiler", () => {
     let routeModule = await fixture.getBrowserAsset(
       fixture.build.assets.routes["routes/built-ins"].module
     );
-    // does not include `import bla from "path"` in the output bundle
+    // does not include `import bla from "node:path"` in the output bundle
     expect(routeModule).not.toMatch(/from\s*"path/);
   });
 
@@ -247,7 +247,7 @@ test.describe("compiler", () => {
     let routeModule = await fixture.getBrowserAsset(
       fixture.build.assets.routes["routes/built-ins-polyfill"].module
     );
-    // does not include `import bla from "path"` in the output bundle
+    // does not include `import bla from "node:path"` in the output bundle
     expect(routeModule).not.toMatch(/from\s*"path/);
   });
 

integration/defer-test.ts

@@ -981,7 +981,7 @@ test.describe("aborted", () => {
       ////////////////////////////////////////////////////////////////////////////
       files: {
         "app/entry.server.tsx": js`
-          import { PassThrough } from "stream";
+          import { PassThrough } from "node:stream";
           import type { AppLoadContext, EntryContext } from "@remix-run/node";
           import { Response } from "@remix-run/node";
           import { RemixServer } from "@remix-run/react";

integration/deno-compiler-test.ts

@@ -1,6 +1,6 @@
 import { test, expect } from "@playwright/test";
 import * as fse from "fs-extra";
-import path from "path";
+import path from "node:path";
 import shell from "shelljs";
 import glob from "glob";
 

integration/deterministic-build-output-test.ts

@@ -1,7 +1,7 @@
 import { test, expect } from "@playwright/test";
 import globby from "globby";
-import fs from "fs";
-import path from "path";
+import fs from "node:fs";
+import path from "node:path";
 
 import type { FixtureInit } from "./helpers/create-fixture";
 import { createFixtureProject, js, css } from "./helpers/create-fixture";

integration/file-uploads-test.ts

@@ -1,5 +1,5 @@
-import * as fs from "fs/promises";
-import * as path from "path";
+import * as fs from "node:fs/promises";
+import * as path from "node:path";
 import { test, expect } from "@playwright/test";
 
 import { createFixture, createAppFixture, js } from "./helpers/create-fixture";
@@ -14,7 +14,7 @@ test.describe("file-uploads", () => {
     fixture = await createFixture({
       files: {
         "app/fileUploadHandler.js": js`
-          import * as path from "path";
+          import * as path from "node:path";
           import {
             unstable_composeUploadHandlers as composeUploadHandlers,
             unstable_createFileUploadHandler as createFileUploadHandler,

integration/helpers/cleanup.mjs

@@ -1,4 +1,4 @@
-import * as path from "path";
+import * as path from "node:path";
 import spawn from "cross-spawn";
 
 if (process.env.CI) {

integration/helpers/playwright-fixture.ts

@@ -1,4 +1,4 @@
-import cp from "child_process";
+import cp from "node:child_process";
 import type { Page, Response, Request } from "@playwright/test";
 import { test } from "@playwright/test";
 import cheerio from "cheerio";

integration/hmr-log-test.ts

@@ -47,7 +47,7 @@ let fixture = (options: { appPort: number; devPort: number }): FixtureInit => ({
     }),
 
     "server.js": js`
-      let path = require("path");
+      let path = require("node:path");
       let express = require("express");
       let { createRequestHandler } = require("@remix-run/express");
       let { logDevReady } = require("@remix-run/node");

integration/hmr-test.ts

@@ -48,7 +48,7 @@ let fixture = (options: { appPort: number; devPort: number }): FixtureInit => ({
     }),
 
     "server.js": js`
-      let path = require("path");
+      let path = require("node:path");
       let express = require("express");
       let { createRequestHandler } = require("@remix-run/express");
       let { broadcastDevReady } = require("@remix-run/node");

integration/server-source-maps-test.ts

@@ -1,6 +1,6 @@
 import { test, expect } from "@playwright/test";
-import path from "path";
-import fsp from "fs/promises";
+import path from "node:path";
+import fsp from "node:fs/promises";
 
 import { createFixture, js } from "./helpers/create-fixture";
 import type { Fixture } from "./helpers/create-fixture";

integration/upload-test.ts

@@ -1,4 +1,4 @@
-import * as path from "path";
+import * as path from "node:path";
 import { test, expect } from "@playwright/test";
 
 import { PlaywrightFixture } from "./helpers/playwright-fixture";

jest/buildRemix.js

@@ -1,4 +1,4 @@
-import * as path from "path";
+import * as path from "node:path";
 import { spawn } from "cross-spawn";
 
 function buildRemix(dir) {

packages/create-remix/__tests__/create-remix-test.ts

@@ -1,8 +1,8 @@
-import type { ChildProcessWithoutNullStreams } from "child_process";
-import { spawn } from "child_process";
-import { tmpdir } from "os";
-import path from "path";
-import { pathToFileURL } from "url";
+import type { ChildProcessWithoutNullStreams } from "node:child_process";
+import { spawn } from "node:child_process";
+import { tmpdir } from "node:os";
+import path from "node:path";
+import { pathToFileURL } from "node:url";
 import fse from "fs-extra";
 import semver from "semver";
 import stripAnsi from "strip-ansi";

packages/create-remix/__tests__/fixtures/successful-remix-init/remix.init/index.js

@@ -1,5 +1,5 @@
-const fs = require("fs");
-const path = require("path");
+const fs = require("node:fs");
+const path = require("node:path");
 
 module.exports = ({ rootDirectory }) => {
   fs.writeFileSync(

packages/create-remix/__tests__/fixtures/tar.js

@@ -1,6 +1,6 @@
 const tar = require("tar-fs");
-const fs = require("fs");
-const path = require("path");
+const fs = require("node:fs");
+const path = require("node:path");
 
 let files = fs.readdirSync(__dirname);
 let dirs = files.filter((file) =>

packages/create-remix/__tests__/github-mocks.ts

@@ -1,5 +1,5 @@
-import * as nodePath from "path";
-import fsp from "fs/promises";
+import * as nodePath from "node:path";
+import fsp from "node:fs/promises";
 import invariant from "tiny-invariant";
 import type { setupServer } from "msw/node";
 import { rest } from "msw";

packages/create-remix/__tests__/msw.ts

@@ -1,5 +1,5 @@
-import path from "path";
-import fsp from "fs/promises";
+import path from "node:path";
+import fsp from "node:fs/promises";
 import { setupServer } from "msw/node";
 import { rest } from "msw";
 

packages/create-remix/rollup.config.js

@@ -1,4 +1,4 @@
-const path = require("path");
+const path = require("node:path");
 const json = require("@rollup/plugin-json").default;
 const babel = require("@rollup/plugin-babel").default;
 const nodeResolve = require("@rollup/plugin-node-resolve").default;

packages/remix-architect/__tests__/server-test.ts

@@ -1,5 +1,5 @@
-import fsp from "fs/promises";
-import path from "path";
+import fsp from "node:fs/promises";
+import path from "node:path";
 import lambdaTester from "lambda-tester";
 import type { APIGatewayProxyEventV2 } from "aws-lambda";
 import {

packages/remix-architect/sessions/arcTableSessionStorage.ts

@@ -1,4 +1,4 @@
-import * as crypto from "crypto";
+import * as crypto from "node:crypto";
 import type {
   SessionData,
   SessionStorage,

packages/remix-cloudflare-pages/rollup.config.js

@@ -1,4 +1,4 @@
-const path = require("path");
+const path = require("node:path");
 const babel = require("@rollup/plugin-babel").default;
 const nodeResolve = require("@rollup/plugin-node-resolve").default;
 

packages/remix-cloudflare-workers/rollup.config.js

@@ -1,4 +1,4 @@
-const path = require("path");
+const path = require("node:path");
 const babel = require("@rollup/plugin-babel").default;
 const nodeResolve = require("@rollup/plugin-node-resolve").default;
 

packages/remix-cloudflare/rollup.config.js

@@ -1,4 +1,4 @@
-const path = require("path");
+const path = require("node:path");
 const babel = require("@rollup/plugin-babel").default;
 const nodeResolve = require("@rollup/plugin-node-resolve").default;
 const copy = require("rollup-plugin-copy");

packages/remix-css-bundle/rollup.config.js

@@ -1,4 +1,4 @@
-const path = require("path");
+const path = require("node:path");
 const babel = require("@rollup/plugin-babel").default;
 const nodeResolve = require("@rollup/plugin-node-resolve").default;
 const copy = require("rollup-plugin-copy");

packages/remix-dev/__tests__/cli-test.ts

@@ -1,7 +1,7 @@
-import childProcess from "child_process";
-import os from "os";
-import path from "path";
-import util from "util";
+import childProcess from "node:child_process";
+import os from "node:os";
+import path from "node:path";
+import util from "node:util";
 import fse from "fs-extra";
 import semver from "semver";
 

packages/remix-dev/__tests__/fixtures/indie-stack/app/entry.server.tsx

@@ -1,4 +1,4 @@
-import { PassThrough } from "stream";
+import { PassThrough } from "node:stream";
 import type { AppLoadContext, EntryContext } from "@remix-run/node";
 import { Response } from "@remix-run/node";
 import { RemixServer } from "@remix-run/react";

packages/remix-dev/__tests__/fixtures/successful-remix-init/remix.init/index.js

@@ -1,5 +1,5 @@
-const fs = require("fs");
-const path = require("path");
+const fs = require("node:fs");
+const path = require("node:path");
 
 module.exports = ({ rootDirectory }) => {
   fs.writeFileSync(

packages/remix-dev/__tests__/fixtures/tar.js

@@ -1,6 +1,6 @@
 const tar = require("tar-fs");
-const fs = require("fs");
-const path = require("path");
+const fs = require("node:fs");
+const path = require("node:path");
 
 let files = fs.readdirSync(__dirname);
 let dirs = files.filter((file) =>

packages/remix-dev/__tests__/github-mocks.ts

@@ -1,5 +1,5 @@
-import * as nodePath from "path";
-import fsp from "fs/promises";
+import * as nodePath from "node:path";
+import fsp from "node:fs/promises";
 import invariant from "tiny-invariant";
 import type { setupServer } from "msw/node";
 import { rest } from "msw";

packages/remix-dev/__tests__/init-test.ts

@@ -1,5 +1,5 @@
-import * as os from "os";
-import * as path from "path";
+import * as os from "node:os";
+import * as path from "node:path";
 import * as fse from "fs-extra";
 import stripAnsi from "strip-ansi";
 

packages/remix-dev/__tests__/msw.ts

@@ -1,5 +1,5 @@
-import path from "path";
-import fsp from "fs/promises";
+import path from "node:path";
+import fsp from "node:fs/promises";
 import { setupServer } from "msw/node";
 import { rest } from "msw";
 

packages/remix-dev/__tests__/readConfig-test.ts

@@ -1,4 +1,4 @@
-import path from "path";
+import path from "node:path";
 
 import type { RemixConfig } from "../config";
 import { readConfig } from "../config";

packages/remix-dev/__tests__/useJavascript-test.ts

@@ -1,5 +1,5 @@
-import { tmpdir } from "os";
-import path from "path";
+import { tmpdir } from "node:os";
+import path from "node:path";
 import glob from "fast-glob";
 import fs from "fs-extra";
 

packages/remix-dev/__tests__/utils/cli.ts

@@ -1,5 +1,5 @@
 import execa from "execa";
-import path from "path";
+import path from "node:path";
 import glob from "fast-glob";
 import fse from "fs-extra";
 

packages/remix-dev/__tests__/utils/withApp.ts

@@ -1,5 +1,5 @@
-import os from "os";
-import path from "path";
+import os from "node:os";
+import path from "node:path";
 import fse from "fs-extra";
 
 const retry = async (

packages/remix-dev/cli/commands.ts

@@ -1,5 +1,5 @@
-import * as path from "path";
-import { execSync } from "child_process";
+import * as path from "node:path";
+import { execSync } from "node:child_process";
 import * as fse from "fs-extra";
 import getPort, { makeRange } from "get-port";
 import prettyMs from "pretty-ms";

packages/remix-dev/compiler/compiler.ts

@@ -1,4 +1,4 @@
-import * as path from "path";
+import * as path from "node:path";
 
 import type { Context } from "./context";
 import * as CSS from "./css";

packages/remix-dev/compiler/css/bundle.ts

@@ -1,4 +1,4 @@
-import * as path from "path";
+import * as path from "node:path";
 import * as fse from "fs-extra";
 import type * as esbuild from "esbuild";
 import postcss from "postcss";

packages/remix-dev/compiler/css/compiler.ts

@@ -1,4 +1,4 @@
-import { builtinModules as nodeBuiltins } from "module";
+import { builtinModules as nodeBuiltins } from "node:module";
 import * as esbuild from "esbuild";
 
 import type { RemixConfig } from "../../config";