Merge pull request #236 from talentlessguy/remove-abort-controller

Remove redundant AbortController polyfill
qgustavor · Feb 7, 2025 · 8af67f9 · 8af67f9
2 parents cf94ba2 + 39f43cb
commit 8af67f9
Show file tree

Hide file tree

Showing 8 changed files with 5 additions and 12 deletions.
diff --git a/build.js b/build.js
@@ -72,7 +72,6 @@ async function doBundle (format) {
     external: format.bundleExternals
       ? []
       : [
-          'abort-controller',
           'agentkeepalive',
           'multistream',
           'node-fetch',
@@ -89,7 +88,6 @@ async function doBundle (format) {
       : [alias({
           http: require.resolve('./browser/noop.mjs'),
           https: require.resolve('./browser/noop.mjs'),
-          'abort-controller': require.resolve('./browser/noop.mjs'),
           'node-fetch': require.resolve('./browser/fetch.mjs'),
           './crypto/rsa.mjs': require.resolve('./browser/rsa.mjs'),
           './aes.mjs': require.resolve('./browser/aes.mjs'),

diff --git a/lib/abort-controller-polyfill.mjs b/lib/abort-controller-polyfill.mjs
diff --git a/lib/api.mjs b/lib/api.mjs
@@ -1,7 +1,6 @@
 import { EventEmitter } from 'events'
 import { Agent as HttpAgent } from 'http'
 import { Agent as HttpsAgent } from 'https'
-import AbortController from './abort-controller-polyfill.mjs'
 import { createPromise } from './util.mjs'
 
 const MAX_RETRIES = 4

diff --git a/lib/file.mjs b/lib/file.mjs
@@ -4,7 +4,6 @@ import { EventEmitter } from 'events'
 import { streamToCb, createPromise } from './util.mjs'
 import { PassThrough } from 'stream'
 import StreamSkip from 'stream-skip'
-import AbortController from './abort-controller-polyfill.mjs'
 
 class File extends EventEmitter {
   constructor (opt) {

diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -69,7 +69,6 @@
     "provenance": true
   },
   "dependencies": {
-    "abort-controller": "^3.0.0",
     "pumpify": "^2.0.1",
     "stream-skip": "^1.0.3"
   },

diff --git a/test/helpers/test-runner.mjs b/test/helpers/test-runner.mjs
@@ -91,7 +91,6 @@ if (testedPlatform === 'node') {
       // The below aliases are copied from build.js
       http: require.resolve('../../browser/noop.mjs'),
       https: require.resolve('../../browser/noop.mjs'),
-      'abort-controller': require.resolve('../../browser/noop.mjs'),
       'node-fetch': require.resolve('../../browser/fetch.mjs'),
       './crypto/rsa.mjs': require.resolve('../../browser/rsa.mjs'),
       './aes.mjs': require.resolve('../../browser/aes.mjs'),

diff --git a/types/cjs.d.ts b/types/cjs.d.ts
@@ -4,7 +4,6 @@ import { EventEmitter } from 'events'
 import { Agent as HttpAgent } from 'http'
 import { Agent as HttpsAgent } from 'https'
 
-import AbortController from 'abort-controller'
 import type * as fetch from 'node-fetch'
 
 declare function megajs (options: megajs.StorageOpts, cb?: megajs.errorCb): megajs.Storage