build: migrate to pnpm 12, oxc tools, native test runner, and upgrade dependencies

🤖 Generated with [OpenCode](https://opencode.ai) (Smart-router)

Author: daquinoaldo

.eslintrc.json

@@ -23,29 +23,32 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@3d3c42e5aac5ba805825da76410c181273ba90b1 # v7.0.1
+      - uses: pnpm/action-setup@d9184bf108216479bc5a137cc391f4d7b14c870b # v6.1.0
+        with:
+          version: 12.3.4
       - uses: actions/setup-node@820762786026740c76f36085b0efc47a31fe5020 # v7.0.0
         with:
           node-version: 26
           registry-url: https://registry.npmjs.org/
-          package-manager-cache: false
+          cache: pnpm
 
       - name: Bump and push version
         if: inputs.upgrade != 'existing'
         run: |-
           git config --global user.name "github-actions[bot]"
           git config --global user.email "41898282+github-actions[bot]@users.noreply.github.com"
-          npm version ${{ inputs.upgrade }} -m "chore: release %s"
+          pnpm version ${{ inputs.upgrade }} -m "chore: release %s"
           git push --follow-tags
 
-      - run: npm publish
+      - run: pnpm publish
 
       - name: Build and create release
         env:
           GH_TOKEN: ${{ secrets.GITHUB_TOKEN }}
         run: |-
           name=$(node -p "require('./package.json').name")
           version=$(node -p "require('./package.json').version")
-          npm i -g pkg
+          pnpm add --global pkg
           pkg --out-path build .
           gh release create "v$version" \
             --title "Release v$version" \

.github/workflows/release.yml

@@ -12,6 +12,6 @@ jobs:
   stale:
     runs-on: ubuntu-latest
     steps:
-    - uses: actions/stale@4391f3da665fdf50b6810c1a66712fb9ba21aa93 # v11.0.0
-      with:
-        delete-branch: true
+      - uses: actions/stale@4391f3da665fdf50b6810c1a66712fb9ba21aa93 # v11.0.0
+        with:
+          delete-branch: true

.github/workflows/stale.yml

@@ -25,13 +25,13 @@ jobs:
 
     steps:
       - uses: actions/checkout@3d3c42e5aac5ba805825da76410c181273ba90b1 # v7.0.1
+      - uses: pnpm/action-setup@d9184bf108216479bc5a137cc391f4d7b14c870b # v6.1.0
+        with:
+          version: 12.3.4
       - uses: actions/setup-node@820762786026740c76f36085b0efc47a31fe5020 # v7.0.0
         with:
           node-version: ${{ matrix.node }}
-      - run: npm ci
-      - run: npm test
-      - uses: actions/upload-artifact@043fb46d1a93c77aae656e7c1c64a875d1fc6a0a # v7.0.1
-        if: matrix.os == 'ubuntu-latest' && matrix.node == 26
-        with:
-          name: coverage-report
-          path: coverage/lcov.info
+          cache: pnpm
+      - run: pnpm install --frozen-lockfile
+      - run: pnpm run lint
+      - run: pnpm test

.github/workflows/test.yml

@@ -0,0 +1,6 @@
+{
+  "$schema": "./node_modules/oxfmt/configuration_schema.json",
+  "semi": false,
+  "arrowParens": "avoid",
+  "singleQuote": false
+}

.oxfmtrc.json

@@ -0,0 +1,18 @@
+# Contributing
+
+Thanks for reading this document!
+
+Contributions are welcome, but I have little time to invest in this project. I ask you to test the change extensively to make my review as easier and possible, and I apologize in advance if it will take me some time to review it.
+
+## Helpful commands
+
+```sh
+# install dependencies
+pnpm install
+
+# type checking and style fixes
+pnpm lint
+
+# run tests
+pnpm test
+```

CONTRIBUTING.md

@@ -17,19 +17,24 @@ function checkUpdates() {
       host: "api.github.com",
       path: "/repos/daquinoaldo/https-localhost/releases/latest",
       method: "GET",
-      headers: { "User-Agent": "node.js" }
+      headers: { "User-Agent": "node.js" },
     }
-    https.request(options, res => {
-      let body = ""
-      res.on("data", chunk => { body += chunk.toString("utf8") })
-      res.on("end", () => {
-        const currentVersion = JSON.parse(fs.readFileSync(
-          path.resolve(__dirname, "package.json"))).version
-        const latestVersion = JSON.parse(body).tag_name.replace("v", "")
-        if (currentVersion !== latestVersion)
-          console.warn("[https-localhost] New update available.")
+    https
+      .request(options, res => {
+        let body = ""
+        res.on("data", chunk => {
+          body += chunk.toString("utf8")
+        })
+        res.on("end", () => {
+          const currentVersion = JSON.parse(
+            fs.readFileSync(path.resolve(__dirname, "package.json")),
+          ).version
+          const latestVersion = JSON.parse(body).tag_name.replace("v", "")
+          if (currentVersion !== latestVersion)
+            console.warn("[https-localhost] New update available.")
+        })
       })
-    }).end()
+      .end()
   } catch (e) {
     // Just catch everything, this is not a critic part and can fail.
     // It is important to not affect the script behavior.
@@ -50,8 +55,10 @@ function getExe() {
     case "win32":
       return "mkcert-" + MKCERT_VERSION + "-windows-amd64.exe"
     default:
-      console.warn("Cannot generate the localhost certificate on your " +
-        "platform. Please, consider contacting the developer if you can help.")
+      console.warn(
+        "Cannot generate the localhost certificate on your " +
+          "platform. Please, consider contacting the developer if you can help.",
+      )
       process.exit(0)
   }
 }
@@ -62,7 +69,7 @@ function download(url, path) {
   const file = fs.createWriteStream(path)
   return new Promise(resolve => {
     function get(url, file) {
-      https.get(url, (response) => {
+      https.get(url, response => {
         if (response.statusCode === 302) get(response.headers.location, file)
         else response.pipe(file).on("finish", resolve)
       })
@@ -75,26 +82,23 @@ function download(url, path) {
 async function mkcert(appDataPath, exe, domain) {
   // fix problems with spaces
   /* istanbul ignore next: platform dependent */
-  const escapeSpaces = function(path) {
+  const escapeSpaces = function (path) {
     // escape spaces (not already escaped)
     if (process.platform === "darwin" || process.platform === "linux")
       return path.replace(/(?<!\\) /g, "\\ ")
     // use apex on Windows
-    if (process.platform === "win32")
-      return "\"" + path + "\""
+    if (process.platform === "win32") return '"' + path + '"'
     return path
   }
 
   const exePath = escapeSpaces(path.join(appDataPath, exe))
   const crtPath = escapeSpaces(path.join(appDataPath, domain + ".crt"))
   const keyPath = escapeSpaces(path.join(appDataPath, domain + ".key"))
-  const cmd = `${exePath} -install -cert-file ${crtPath}` +
-    ` -key-file ${keyPath} ${domain}`
+  const cmd = `${exePath} -install -cert-file ${crtPath}` + ` -key-file ${keyPath} ${domain}`
 
   // sleep on windows due to issue #28
   /* istanbul ignore if: cannot be tested */
-  if (process.platform === "win32")
-    await new Promise(resolve => setTimeout(resolve, 3000))
+  if (process.platform === "win32") await new Promise(resolve => setTimeout(resolve, 3000))
 
   return new Promise((resolve, reject) => {
     console.log("Running mkcert to generate certificates...")
@@ -117,15 +121,12 @@ async function mkcert(appDataPath, exe, domain) {
 async function generate(appDataPath = CERT_PATH, customDomain = undefined) {
   const domain = customDomain || "localhost"
   console.info("Generating certificates...")
-  console.log("Certificates path: " + appDataPath +
-    ". Never modify nor share this files.")
+  console.log("Certificates path: " + appDataPath + ". Never modify nor share this files.")
   // mkdir if not exists
   /* istanbul ignore else: not relevant */
-  if (!fs.existsSync(appDataPath))
-    fs.mkdirSync(appDataPath, { recursive: true })
+  if (!fs.existsSync(appDataPath)) fs.mkdirSync(appDataPath, { recursive: true })
   // build the executable url and path
-  const url = "https://github.com/FiloSottile/mkcert/releases/download/" +
-    MKCERT_VERSION + "/"
+  const url = "https://github.com/FiloSottile/mkcert/releases/download/" + MKCERT_VERSION + "/"
   const exe = getExe()
   const exePath = path.join(appDataPath, exe)
   // download the executable
@@ -144,19 +145,19 @@ async function getCerts(customDomain = undefined) {
   /* istanbul ignore if: cannot test pkg */
   if (process.pkg) checkUpdates()
   // check if a reinstall is forced or needed by a mkcert update
-  if (process.env.REINSTALL ||
-    !fs.existsSync(path.join(certPath, getExe())))
+  if (process.env.REINSTALL || !fs.existsSync(path.join(certPath, getExe())))
     await generate(certPath, domain)
   try {
     return {
       key: fs.readFileSync(path.join(certPath, domain + ".key")),
-      cert: fs.readFileSync(path.join(certPath, domain + ".crt"))
+      cert: fs.readFileSync(path.join(certPath, domain + ".crt")),
     }
   } catch (e) {
     /* istanbul ignore else: should never occur */
     if (certPath !== CERT_PATH) {
-      console.error("Cannot find localhost.key and localhost.crt in the" +
-        " specified path: " + certPath)
+      console.error(
+        "Cannot find localhost.key and localhost.crt in the" + " specified path: " + certPath,
+      )
       process.exit(1)
     } else {
       // Missing certificates (first run)
@@ -171,8 +172,7 @@ async function getCerts(customDomain = undefined) {
 // delete a folder and the file inside it
 function remove(appDataPath = CERT_PATH) {
   if (fs.existsSync(appDataPath)) {
-    fs.readdirSync(appDataPath)
-      .forEach(file => fs.unlinkSync(path.join(appDataPath, file)))
+    fs.readdirSync(appDataPath).forEach(file => fs.unlinkSync(path.join(appDataPath, file)))
     fs.rmdirSync(appDataPath)
   }
 }
@@ -181,17 +181,23 @@ function remove(appDataPath = CERT_PATH) {
 /* istanbul ignore if: cannot be tested */
 if (require.main === module)
   // if run with -u or --uninstall
-  if (process.argv.length === 3 &&
-    (process.argv[2] === "-u" || process.argv[2] === "--uninstall")) {
+  if (
+    process.argv.length === 3 &&
+    (process.argv[2] === "-u" || process.argv[2] === "--uninstall")
+  ) {
     remove()
     console.info("Certificates removed.")
-  } else try { // install
-    generate()
-  } catch (err) { console.error("\nExec error: " + err) }
+  } else
+    try {
+      // install
+      generate()
+    } catch (err) {
+      console.error("\nExec error: " + err)
+    }
 
 // export as module
 module.exports = {
   getCerts,
   generate,
-  remove
+  remove,
 }

certs.js

@@ -22,10 +22,10 @@ const createServer = (domain = process.env.HOST || "localhost") => {
   app.getCerts = getCerts
 
   // override the default express listen method to use our server
-  app.listen = async function(port = process.env.PORT ||
-  /* istanbul ignore next: cannot be tested on Travis */ 443) {
-    app.server = https.createServer(await getCerts(domain), app)
-      .listen(port)
+  app.listen = async function (
+    port = process.env.PORT || /* istanbul ignore next: cannot be tested on Travis */ 443,
+  ) {
+    app.server = https.createServer(await getCerts(domain), app).listen(port)
     console.info("Server running on port " + port + ".")
     return app.server
   }
@@ -42,36 +42,41 @@ const createServer = (domain = process.env.HOST || "localhost") => {
   /* SETUP USEFUL FUNCTIONS */
 
   // redirect http to https, usage `app.redirect()`
-  app.redirect = function(
+  app.redirect = function (
     /* istanbul ignore next: cannot be tested on Travis */ httpPort = 80,
-    httpsPort = process.env.PORT || 443) {
-    app.http = http.createServer((req, res) => {
-      const reqHost = req.headers.host
-        ? req.headers.host.replace(":" + httpPort, "")
-        : /* istanbul ignore next: cannot be tested */ "localhost"
-      res.writeHead(301, {
-        Location: "https://" + reqHost +
-          (httpsPort !== 443 ? ":" + httpsPort : "") + (req.url ||
-        /* istanbul ignore next: cannot be tested */ "")
+    httpsPort = process.env.PORT || 443,
+  ) {
+    app.http = http
+      .createServer((req, res) => {
+        const reqHost = req.headers.host
+          ? req.headers.host.replace(":" + httpPort, "")
+          : /* istanbul ignore next: cannot be tested */ "localhost"
+        res.writeHead(301, {
+          Location:
+            "https://" +
+            reqHost +
+            (httpsPort !== 443 ? ":" + httpsPort : "") +
+            (req.url || /* istanbul ignore next: cannot be tested */ ""),
+        })
+        res.end()
       })
-      res.end()
-    }).listen(httpPort)
+      .listen(httpPort)
     console.info("http to https redirection active.")
   }
 
   // serve static content, usage `app.serve([path])`
-  app.serve = function(staticPath = process.cwd(), port = process.env.PORT ||
-  /* istanbul ignore next: cannot be tested on Travis */ 443) {
+  app.serve = function (
+    staticPath = process.cwd(),
+    port = process.env.PORT || /* istanbul ignore next: cannot be tested on Travis */ 443,
+  ) {
     app.use(express.static(staticPath))
     // redirect 404 to 404.html or to index.html
     app.use((req, res) => {
       const p404 = staticPath + "/404.html"
       const index = staticPath + "/index.html"
       // istanbul ignore else: not interesting
-      if (fs.existsSync(p404))
-        res.status(404).sendFile(path.resolve(p404))
-      else if (fs.existsSync(index))
-        res.status(200).sendFile(path.resolve(index))
+      if (fs.existsSync(p404)) res.status(404).sendFile(path.resolve(p404))
+      else if (fs.existsSync(index)) res.status(200).sendFile(path.resolve(index))
       else res.status(404).send(req.path + " not found.")
     })
     console.info("Serving static path: " + staticPath)
@@ -94,17 +99,20 @@ if (require.main === module) {
   if (!process.env.PORT) app.redirect()
 
   /* istanbul ignore next: cannot be tested */
-  process.on("uncaughtException", function(err) {
+  process.on("uncaughtException", function (err) {
     switch (err.errno) {
       case "EACCES":
         console.error(
           "EACCES: run as administrator to use the default ports 443 and 80. " +
-          "You can also change port with: `PORT=4433 serve ~/myproj`.")
+            "You can also change port with: `PORT=4433 serve ~/myproj`.",
+        )
         break
       case "EADDRINUSE":
-        console.error("EADDRINUSE: another service on your machine is using " +
-        "the current port.\nStop it or change port with:" +
-        "`PORT=4433 serve ~/myproj`.")
+        console.error(
+          "EADDRINUSE: another service on your machine is using " +
+            "the current port.\nStop it or change port with:" +
+            "`PORT=4433 serve ~/myproj`.",
+        )
         break
       default:
         console.error("Unexpected error " + err.errno + ":\n\n" + err)