refactor: use node: specifier imports and full relative paths (#561)

* refactor: replace NodeJS internal module imports with `node:` specifier imports

* refactor: use full relative paths in imports

* build: fix `test:typescript` script

package.json

@@ -11,7 +11,7 @@
     "lint:fix": "prettier --write '{src,test}/**/*.{js,ts,css,less,scss,vue,json,gql,md}' README.md package.json",
     "pretest": "npm run -s lint",
     "test": "jest --coverage",
-    "test:typescript": "npx tsc --noEmit --declaration --noUnusedLocals --esModuleInterop test/typescript-validate.ts"
+    "test:typescript": "npx tsc --noEmit --declaration --noUnusedLocals --esModuleInterop --allowImportingTsExtensions test/typescript-validate.ts"
   },
   "repository": "github:octokit/action.js",
   "keywords": [
@@ -60,6 +60,9 @@
         "functions": 100,
         "lines": 100
       }
+    },
+    "moduleNameMapper": {
+      "^(.+)\\.jsx?$": "$1"
     }
   },
   "release": {

src/index.ts

@@ -4,8 +4,8 @@ import { paginateRest } from "@octokit/plugin-paginate-rest";
 import { legacyRestEndpointMethods } from "@octokit/plugin-rest-endpoint-methods";
 export type { RestEndpointMethodTypes } from "@octokit/plugin-rest-endpoint-methods";
 
-import { VERSION } from "./version";
-import type { OctokitOptions } from "@octokit/core/dist-types/types";
+import { VERSION } from "./version.js";
+import type { OctokitOptions } from "@octokit/core/dist-types/types.js";
 import { fetch as undiciFetch, ProxyAgent } from "undici";
 
 const DEFAULTS = {

test/smoke.test.ts

@@ -1,6 +1,6 @@
 import fetchMock from "fetch-mock";
-import { createServer, type Server } from "https";
-import { Octokit, getProxyAgent, customFetch } from "../src";
+import { createServer, type Server } from "node:https";
+import { Octokit, getProxyAgent, customFetch } from "../src/index.ts";
 import { ProxyAgent } from "undici";
 
 // mock undici such that we can substitute our own fetch implementation

test/tsconfig.test.json

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

test/typescript-validate.ts

@@ -1,4 +1,4 @@
-import { Octokit } from "../src";
+import { Octokit } from "../src/index.ts";
 
 // ************************************************************
 // THIS CODE IS NOT EXECUTED. IT IS JUST FOR TYPECHECKING