feat!: use native fetch, update all deps

Author: SgtPooki

.aegir.js

@@ -1,14 +1,32 @@
-import { MockServerController } from './dist/test/MockServerController.js'
 
 /** @type {import('aegir').PartialOptions} */
-const config = {
+export default {
+  typescript: true,
+  dependencyCheck: {
+    ignore: [
+      '@swc/cli',
+      '@swc/core',
+      '@swc/helpers',
+      'cors',
+      'dotenvrc',
+      'express',
+      'express-promise-router',
+      'mock-ipfs-pinning-service',
+      'portscanner',
+      'regenerator-runtime',
+      'winston',
+    ]
+  },
   docs: {
     publish: true,
     entryPoint: './'
   },
   build: {
+    types: true,
     config: {
-      platform: 'node'
+      format: 'esm',
+      platform: 'node',
+      external: ['electron', '#ansi-styles', 'yargs/yargs', '#supports-color']
     },
     bundlesizeMax: '44KB'
   },
@@ -17,6 +35,7 @@ const config = {
     progress: true,
     cov: false,
     async before () {
+      const { MockServerController } = await import('./dist/test/MockServerController.js')
       return {
         env: {
           MOCK_PINNING_SERVER_SECRET: 'ci',
@@ -26,12 +45,10 @@ const config = {
     },
     /**
      * @param {import('aegir').GlobalOptions & import('aegir').TestOptions} _
-     * @param { {controller: MockServerController} } beforeResult
+     * @param { { controller: import('./test/MockServerController.js').MockServerController } } beforeResult
      */
     async after (_, {controller}) {
       await controller.shutdown()
     }
   }
 }
-
-export default config

.eslintrc.json

@@ -1,8 +1,10 @@
 {
   "root": true,
-  "parser": "@typescript-eslint/parser",
-  "plugins": ["@typescript-eslint", "import"],
   "extends": ["ipfs"],
+  "parserOptions": {
+    "project": true,
+    "sourceType": "module"
+  },
   "overrides": [
     {
       "parserOptions": {
@@ -13,7 +15,6 @@
         "./types/**/*.ts"
       ],
       "rules": {
-        "@typescript-eslint/strict-boolean-expressions": "off"
       }
     },
     {
@@ -32,8 +33,7 @@
         "semi": "off",
         "indent": "off",
         "no-unused-vars": "off",
-        "no-undef": "off",
-        "@typescript-eslint/strict-boolean-expressions": "off"
+        "no-undef": "off"
       }
     }
   ],

.gitignore

@@ -24,3 +24,4 @@ logs
 .tsbuildinfo*
 # generated by nektos/act cli
 workflow
+.coverage

.vscode/settings.json

@@ -20,7 +20,7 @@ import type { PinsGetRequest, PinResults } from '@ipfs-shipyard/pinning-service-
 const config = new Configuration({
   endpointUrl, // the URI for your pinning provider, e.g. `http://localhost:3000`
   accessToken, // the secret token/key given to you by your pinning provider
-  // fetchApi: fetch, // You can pass your own fetchApi implementation, but we use fetch-ponyfill by default.
+  // fetchApi: fetch, // You can pass your own fetchApi implementation, but we use NodeJS fetch by default.
 })
 
 const client = new RemotePinningServiceClient(config)

README.md

@@ -1,4 +1,3 @@
-
 // @ts-check
 // export async function mochaGlobalSetup () {
 

fixtures.js

@@ -58,12 +58,6 @@
       "require": "./dist/dist.generated/apis"
     }
   },
-  "eslintConfig": {
-    "extends": "ipfs",
-    "parserOptions": {
-      "sourceType": "module"
-    }
-  },
   "release": {
     "branches": [
       "main"
@@ -147,7 +141,7 @@
   },
   "scripts": {
     "ci:test": "run-s test:*",
-    "dep-check": "aegir dep-check dist/**/*.js -- --ignore @openapitools/openapi-generator-cli @typescript-eslint/eslint-plugin @typescript-eslint/parser",
+    "dep-check": "aegir dep-check dist/**/*.js",
     "fix": "run-s fix:*",
     "fix:lint": "aegir lint --fix",
     "release": "aegir release",
@@ -165,49 +159,39 @@
     "test:firefox": "aegir test -t browser -- --browser firefox",
     "test:firefox-webworker": "aegir test -t webworker -- --browser firefox",
     "test:electron-main": "aegir test -t electron-main",
-    "posttest": "npx nyc report",
+    "posttest": "npx nyc report || echo 'coverage report temporarily broken'",
     "pregen": "openapi-generator-cli validate -i https://raw.githubusercontent.com/ipfs/pinning-services-api-spec/v1.0.0/ipfs-pinning-service.yaml",
     "gen": "run-p gen:fetch",
     "postgen": "run-s build:generated",
     "gen:fetch": "openapi-generator-cli generate --generator-key fetch",
     "gen:node": "openapi-generator-cli generate --generator-key node",
     "gen:ts": "openapi-generator-cli generate --generator-key ts",
     "clean": "aegir clean",
+    "reset": "aegir clean node_modules dist dist.generated",
     "lint": "aegir lint"
   },
-  "dependencies": {
-    "fetch-ponyfill": "^7.1.0"
-  },
   "devDependencies": {
-    "@openapitools/openapi-generator-cli": "^2.4.26",
-    "@swc/cli": "^0.1.55",
-    "@swc/core": "^1.2.157",
-    "@swc/helpers": "^0.3.3",
-    "@types/cors": "^2.8.12",
-    "@types/eslint": "^8.4.2",
-    "@types/express": "^4.17.13",
-    "@types/mocha": "^10.0.0",
-    "@types/node": "^18.8.2",
-    "@types/portscanner": "^2.1.1",
+    "@openapitools/openapi-generator-cli": "^2.13.5",
+    "@swc/cli": "^0.4.0",
+    "@swc/core": "^1.7.18",
+    "@swc/helpers": "^0.5.12",
+    "@types/cors": "^2.8.17",
+    "@types/express": "^4.17.21",
+    "@types/mocha": "^10.0.7",
+    "@types/node": "^22.5.0",
+    "@types/portscanner": "^2.1.4",
     "@types/winston": "^2.4.4",
-    "@typescript-eslint/eslint-plugin": "^5.22.0",
-    "@typescript-eslint/parser": "^5.22.0",
-    "aegir": "^44.1.1",
-    "check-aegir-project": "^1.0.3",
+    "aegir": "^43.0.3",
     "cors": "^2.8.5",
     "cp-cli": "^2.0.0",
     "dotenvrc": "^1.0.1",
-    "eslint": "^8.14.0",
-    "eslint-plugin-import": "^2.26.0",
-    "express": "^4.17.3",
+    "express": "^4.19.2",
     "express-promise-router": "^4.1.1",
-    "mock-ipfs-pinning-service": "^0.4.0",
+    "mock-ipfs-pinning-service": "^0.4.2",
     "npm-run-all": "^4.1.5",
     "portscanner": "^2.2.0",
-    "regenerator-runtime": "^0.13.9",
-    "ts-node": "^10.7.0",
-    "tsc-silent": "^1.2.1",
-    "typescript": "^5.5.4",
-    "winston": "^3.6.0"
+    "regenerator-runtime": "^0.14.1",
+    "tsc-silent": "^1.2.2",
+    "winston": "^3.14.2"
   }
 }

package.json

@@ -1,10 +1,8 @@
-import fetchPonyfill from 'fetch-ponyfill'
-
+import { PinsApi as RemotePinningServiceClient } from '../dist.generated/apis/index.js'
 import { Configuration as GeneratedConfiguration } from '../dist.generated/index.js'
 import type { ConfigurationParameters as GeneratedConfigurationParameters } from '../dist.generated/index.js'
-import { PinsApi as RemotePinningServiceClient } from '../dist.generated/apis/index.js'
 
-interface ConfigurationParameters extends Omit<GeneratedConfigurationParameters, 'basePath'>{
+interface ConfigurationParameters extends Omit<GeneratedConfigurationParameters, 'basePath'> {
   endpointUrl?: string
 }
 class Configuration extends GeneratedConfiguration {
@@ -14,7 +12,7 @@ class Configuration extends GeneratedConfiguration {
      * Prevent the need for everyone to have to override the fetch API...
      */
     if (options.fetchApi == null) {
-      finalOptions.fetchApi = fetchPonyfill().fetch
+      finalOptions.fetchApi = fetch
     }
 
     // @see https://github.com/ipfs-shipyard/js-pinning-service-http-client/issues/3

src/index.ts

@@ -1,14 +1,12 @@
-import type { Application } from 'express'
-import type { Server } from 'http'
-import portscanner from 'portscanner'
 import cors from 'cors'
-
 import { setup } from 'mock-ipfs-pinning-service'
-
+import portscanner from 'portscanner'
 import { logger } from './logger.js'
+import type { Application } from 'express'
+import type { Server } from 'http'
 
 try {
-  import('dotenvrc')
+  await import('dotenvrc')
 } catch (err) {
   // no dotenvrc.. that's okay
   // eslint-disable-next-line no-console
@@ -23,7 +21,7 @@ export class MockServer {
 
   public basePath: string = ''
 
-  constructor (private readonly config: Parameters<typeof import('mock-ipfs-pinning-service')['setup']>[0] = { token: process.env.MOCK_PINNING_SERVER_SECRET }) {
+  constructor (private readonly config: Parameters<typeof setup>[0] = { token: process.env.MOCK_PINNING_SERVER_SECRET }) {
     process.on('uncaughtException', MockServer.onEADDRINUSE)
   }
 
@@ -84,14 +82,16 @@ export class MockServer {
    * Ensure the port set for this instance is not already in use by another MockServer
    */
   private async getAvailablePort (): Promise<string> {
-    return await new Promise((resolve, reject) => portscanner.findAPortNotInUse(3000, 3099, '127.0.0.1', (error, port) => {
-      if (error != null) {
-        return reject(error)
-      }
-      this.port = port.toString()
-      this.setBasePath()
-      resolve(this.port)
-    }))
+    return new Promise((resolve, reject) => {
+      portscanner.findAPortNotInUse(3000, 3099, '127.0.0.1', (error, port) => {
+        if (error != null) {
+          reject(error); return
+        }
+        this.port = port.toString()
+        this.setBasePath()
+        resolve(this.port)
+      })
+    })
   }
 
   private async service (): Promise<Application> {
@@ -133,7 +133,7 @@ export class MockServer {
     })
   }
 
-  private static onEADDRINUSE (err: Error & { code: string }) {
+  private static onEADDRINUSE (err: Error & { code: string }): void {
     if (err.code === 'EADDRINUSE') {
       logger.error('Unexpected conflict with port usage')
     } else {

test/MockServer.ts

@@ -1,9 +1,9 @@
+import cors from 'cors'
 import express from 'express'
 import Router from 'express-promise-router'
-import cors from 'cors'
-
 import { MockServer } from './MockServer.js'
 import { logger } from './logger.js'
+import type { Server } from 'http'
 
 /**
  * MockServerController stands up a server on port 3000
@@ -14,11 +14,12 @@ class MockServerController {
   private readonly router = Router()
 
   private readonly port = 3000
-  server: import('http').Server
+  server: Server
   constructor () {
     this.shutdown.bind(this)
     this.shutdownSync.bind(this)
-    this.router.get<'/start', {port?: string}>('/start', async (req, res, next) => { // eslint-disable-line @typescript-eslint/no-misused-promises
+    // eslint-disable-next-line @typescript-eslint/no-misused-promises
+    this.router.get<'/start', { port?: string }>('/start', async (req, res, next) => {
       logger.debug('MockServerController: /start GET request received')
       const { port } = req.params
 
@@ -44,7 +45,8 @@ class MockServerController {
     /**
      * A client will request to shut down it's mockServer by port, which it should have received upon calling '/start'
      */
-    this.router.get<'/stop/:port', {port: string}>('/stop/:port', async (req, res, next) => { // eslint-disable-line @typescript-eslint/no-misused-promises
+    // eslint-disable-next-line @typescript-eslint/no-misused-promises
+    this.router.get<'/stop/:port', { port: string }>('/stop/:port', async (req, res, next) => {
       const { port } = req.params
       logger.debug(`MockServerController: /stop/${port.toString()} GET request received`)
 
@@ -82,13 +84,13 @@ class MockServerController {
     })
   }
 
-  private shutdownSync () {
+  private shutdownSync (): void {
     this.shutdown().catch((err) => {
       logger.error(err)
     })
   }
 
-  async shutdown () {
+  async shutdown (): Promise<void> {
     // To prevent duplicated cleanup, remove the process listeners on server close.
     process.off('beforeExit', this.shutdownSync)
     process.off('SIGTERM', this.shutdownSync)
@@ -115,7 +117,7 @@ class MockServerController {
     }
   }
 
-  private async startIpfsPinningServer (port?: string) {
+  private async startIpfsPinningServer (port?: string): Promise<MockServer> {
     const mockServer = new MockServer({
       token: process.env.MOCK_PINNING_SERVER_SECRET
     })