@common.js/p-timeout 6.1.1 → 6.1.3

package/README.md

@@ -2,4 +2,4 @@
 
 The [p-timeout](https://www.npmjs.com/package/p-timeout) package exported as CommonJS modules.
 
-Exported from [p-timeout@6.1.1](https://www.npmjs.com/package/p-timeout/v/6.1.1) using https://github.com/etienne-martin/common.js.
+Exported from [p-timeout@6.1.3](https://www.npmjs.com/package/p-timeout/v/6.1.3) using https://github.com/etienne-martin/common.js.

package/index.d.ts

@@ -3,12 +3,12 @@ export class TimeoutError extends Error {
 	constructor(message?: string);
 }
 
-export interface ClearablePromise<T> extends Promise<T> {
+export type ClearablePromise<T> = {
 	/**
 	Clear the timeout.
 	*/
 	clear: () => void;
-}
+} & Promise<T>;
 
 export type Options<ReturnType> = {
 	/**

package/index.js

@@ -322,23 +322,29 @@ function pTimeout(promise, options) {
         clearTimeout: clearTimeout
     } : _customTimers;
     var timer;
-    var cancelablePromise = new Promise(function(resolve, reject) {
+    var wrappedPromise = new Promise(function(resolve, reject) {
         if (typeof milliseconds !== "number" || Math.sign(milliseconds) !== 1) {
             throw new TypeError("Expected `milliseconds` to be a positive number, got `".concat(milliseconds, "`"));
         }
-        if (milliseconds === Number.POSITIVE_INFINITY) {
-            resolve(promise);
-            return;
-        }
         if (options.signal) {
             var signal = options.signal;
             if (signal.aborted) {
                 reject(getAbortedReason(signal));
             }
-            signal.addEventListener("abort", function() {
+            var abortHandler = function() {
                 reject(getAbortedReason(signal));
+            };
+            signal.addEventListener("abort", abortHandler, {
+                once: true
+            });
+            promise.finally(function() {
+                signal.removeEventListener("abort", abortHandler);
             });
         }
+        if (milliseconds === Number.POSITIVE_INFINITY) {
+            promise.then(resolve, reject);
+            return;
+        }
         // We create the error outside of `setTimeout` to preserve the stack trace.
         var timeoutError = new TimeoutError();
         timer = customTimers.setTimeout.call(undefined, function() {
@@ -370,8 +376,8 @@ function pTimeout(promise, options) {
                         _state.trys.push([
                             0,
                             2,
-                            3,
-                            4
+                            ,
+                            3
                         ]);
                         return [
                             4,
@@ -383,21 +389,16 @@ function pTimeout(promise, options) {
                         ]);
                         return [
                             3,
-                            4
+                            3
                         ];
                     case 2:
                         error = _state.sent();
                         reject(error);
                         return [
                             3,
-                            4
+                            3
                         ];
                     case 3:
-                        customTimers.clearTimeout.call(undefined, timer);
-                        return [
-                            7
-                        ];
-                    case 4:
                         return [
                             2
                         ];
@@ -405,6 +406,9 @@ function pTimeout(promise, options) {
             });
         })();
     });
+    var cancelablePromise = wrappedPromise.finally(function() {
+        cancelablePromise.clear();
+    });
     cancelablePromise.clear = function() {
         customTimers.clearTimeout.call(undefined, timer);
         timer = undefined;

package/package.json

@@ -1,12 +1,13 @@
 {
   "name": "@common.js/p-timeout",
-  "version": "6.1.1",
+  "version": "6.1.3",
   "description": "p-timeout package exported as CommonJS modules",
   "license": "MIT",
   "repository": "etienne-martin/common.js",
   "funding": "https://github.com/sponsors/sindresorhus",
   "type": "commonjs",
   "types": "./index.d.ts",
+  "sideEffects": false,
   "engines": {
     "node": ">=14.16"
   },
@@ -22,9 +23,10 @@
     "delay": "^5.0.0",
     "in-range": "^3.0.0",
     "p-cancelable": "^4.0.1",
+    "sinon": "^19.0.2",
     "time-span": "^5.1.0",
     "tsd": "^0.22.0",
-    "xo": "^0.51.0"
+    "xo": "^0.54.2"
   },
   "homepage": "https://github.com/etienne-martin/common.js#readme",
   "dependencies": {},