Revert "Drop node: protocol for importing Node.js modules (#720)"

Revert c805cb3

Originally, we thought that the `node:` prefix might cause problems when bundling tus-js-client. However, it turned out that this was not the case (see linked PR), so we can now add them again.

biome.json

@@ -19,7 +19,7 @@
         "noParameterAssign": "off",
         "noDefaultExport": "error",
         "useCollapsedElseIf": "error",
-        "useNodejsImportProtocol": "off"
+        "useNodejsImportProtocol": "error"
       },
       "suspicious": {
         "noExplicitAny": "error",

demos/cordova/hooks/before_prepare/copy_tus_files.js

@@ -1,7 +1,7 @@
 #!/usr/bin/env node
 
-const fs = require('fs')
-const path = require('path')
+const fs = require('node:fs')
+const path = require('node:path')
 
 const rootDir = path.join(__dirname, '..', '..')
 const tusJsPath = path.join(rootDir, '..', '..', 'dist', 'tus.js')

demos/nodejs/index.js

@@ -1,4 +1,4 @@
-import { createReadStream } from 'fs'
+import { createReadStream } from 'node:fs'
 import { Upload } from 'tus-js-client'
 
 const path = `${import.meta.dirname}/../../README.md`

lib/node/FileUrlStorage.ts

@@ -1,4 +1,4 @@
-import { readFile, writeFile } from 'fs/promises'
+import { readFile, writeFile } from 'node:fs/promises'
 import { lock } from 'proper-lockfile'
 import type { PreviousUpload, UrlStorage } from '../options.js'
 

lib/node/NodeFileReader.ts

@@ -1,4 +1,4 @@
-import { ReadStream } from 'fs'
+import { ReadStream } from 'node:fs'
 import isStream from 'is-stream'
 
 import type { FileReader, UploadInput } from '../options.js'

lib/node/NodeHttpStack.ts

@@ -1,9 +1,9 @@
 // The url.parse method is superseeded by the url.URL constructor,
 // but it is still included in Node.js
-import * as http from 'http'
-import * as https from 'https'
-import { Readable, Transform, type Writable } from 'stream'
-import { parse } from 'url'
+import * as http from 'node:http'
+import * as https from 'node:https'
+import { Readable, Transform, type Writable } from 'node:stream'
+import { parse } from 'node:url'
 import throttle from 'lodash.throttle'
 import type { HttpProgressHandler, HttpRequest, HttpResponse, HttpStack } from '../options.js'
 import type { FileSliceTypes } from './index.js'

lib/node/fileSignature.ts

@@ -1,6 +1,6 @@
-import { createHash } from 'crypto'
-import * as fs from 'fs'
-import * as path from 'path'
+import { createHash } from 'node:crypto'
+import * as fs from 'node:fs'
+import * as path from 'node:path'
 import type { UploadInput, UploadOptions } from '../options.js'
 
 export function fingerprint(file: UploadInput, options: UploadOptions): Promise<string | null> {

lib/node/sources/NodeFileSource.ts

@@ -1,4 +1,4 @@
-import { type ReadStream, createReadStream, promises as fsPromises } from 'fs'
+import { type ReadStream, createReadStream, promises as fsPromises } from 'node:fs'
 import type { FileSource } from '../../options.js'
 
 export async function getFileSource(stream: ReadStream): Promise<NodeFileSource> {

scripts/setup-exports.js

@@ -1,5 +1,5 @@
-import path, { relative } from 'path'
-import { mkdir, readFile, writeFile } from 'fs/promises'
+import { mkdir, readFile, writeFile } from 'node:fs/promises'
+import path, { relative } from 'node:path'
 
 /**
  * Each entry is a "subpackage" that users should be able to import as `tus-js-client/${entry}`.

test/spec/test-node-specific.js

@@ -1,9 +1,9 @@
-import crypto from 'crypto'
-import { once } from 'events'
-import fs from 'fs'
-import http from 'http'
-import https from 'https'
-import stream from 'stream'
+import crypto from 'node:crypto'
+import { once } from 'node:events'
+import fs from 'node:fs'
+import http from 'node:http'
+import https from 'node:https'
+import stream from 'node:stream'
 import intoStream from 'into-stream'
 import temp from 'temp'
 import { Upload, canStoreURLs } from 'tus-js-client'