refactor: use node: specifier imports and full relative paths imports, update tests for new type errors

package.json

@@ -36,7 +36,7 @@
     "@types/node": "^20.0.0",
     "@types/once": "^1.4.0",
     "esbuild": "^0.19.0",
-    "fetch-mock": "^9.3.1",
+    "fetch-mock": "npm:@gr2m/fetch-mock@^9.11.0-pull-request-644.1",
     "glob": "^10.2.4",
     "jest": "^29.0.0",
     "lolex": "^6.0.0",
@@ -65,7 +65,10 @@
     },
     "modulePathIgnorePatterns": [
       "<rootDir>/pkg"
-    ]
+    ],
+    "moduleNameMapper": {
+      "^(.+)\\.jsx?$": "$1"
+    }
   },
   "release": {
     "branches": [

src/fetch-wrapper.ts

@@ -1,8 +1,8 @@
-import { isPlainObject } from "./is-plain-object";
+import { isPlainObject } from "./is-plain-object.js";
 import { RequestError } from "@octokit/request-error";
 import type { EndpointInterface } from "@octokit/types";
 
-import getBuffer from "./get-buffer-response";
+import getBuffer from "./get-buffer-response.js";
 
 export default function fetchWrapper(
   requestOptions: ReturnType<EndpointInterface>,

src/index.ts

@@ -1,8 +1,8 @@
 import { endpoint } from "@octokit/endpoint";
 import { getUserAgent } from "universal-user-agent";
 
-import { VERSION } from "./version";
-import withDefaults from "./with-defaults";
+import { VERSION } from "./version.js";
+import withDefaults from "./with-defaults.js";
 
 export const request = withDefaults(endpoint, {
   headers: {

src/with-defaults.ts

@@ -1,4 +1,4 @@
-import fetchWrapper from "./fetch-wrapper";
+import fetchWrapper from "./fetch-wrapper.js";
 import type {
   EndpointOptions,
   EndpointInterface,

test/defaults.test.ts

@@ -1,6 +1,6 @@
 import fetchMock from "fetch-mock";
 
-import { request } from "../src";
+import { request } from "../src/index.ts";
 
 describe("endpoint.defaults()", () => {
   it("is a function", () => {

test/is-plain-object.test.ts

@@ -1,4 +1,4 @@
-import { isPlainObject } from "../src/is-plain-object";
+import { isPlainObject } from "../src/is-plain-object.ts";
 
 describe("isPlainObject", () => {
   function Foo() {

test/request.test.ts

@@ -1,5 +1,6 @@
-import fs from "fs";
-import stream, { Stream } from "stream";
+import fs from "node:fs";
+import stream from "node:stream";
+import { ReadableStream } from "node:stream/web";
 
 import { getUserAgent } from "universal-user-agent";
 import fetchMock from "fetch-mock";
@@ -11,7 +12,7 @@ import type {
   ResponseHeaders,
 } from "@octokit/types";
 
-import { request } from "../src";
+import { request } from "../src/index.ts";
 
 const userAgent = `octokit-request.js/0.0.0-development ${getUserAgent()}`;
 const stringToArrayBuffer = require("string-to-arraybuffer");
@@ -1117,7 +1118,7 @@ x//0u+zd/R/QRUzLOw4N72/Hu+UG6MNt5iDZFCtapRaKt6OvSBwy8w==
     }).then((response) => {
       expect(response.status).toEqual(200);
       expect(response.headers["content-type"]).toEqual("application/x-gzip");
-      expect(response.data).toBeInstanceOf(Stream);
+      expect(response.data).toBeInstanceOf(ReadableStream);
       expect(mock.done()).toBe(true);
     });
   });

test/tsconfig.test.json

@@ -3,7 +3,8 @@
   "compilerOptions": {
     "emitDeclarationOnly": false,
     "noEmit": true,
-    "verbatimModuleSyntax": false
+    "verbatimModuleSyntax": false,
+    "allowImportingTsExtensions": true
   },
   "include": ["src/**/*"]
 }