Skip to content

Commit f38d8c9

Browse files
committed
refactor!: replace is-promise dependency with instanceof check
BREAKING: drops support for non-native promises Closes #136 Signed-off-by: Jon Koops <[email protected]>
1 parent 276d40d commit f38d8c9

File tree

4 files changed

+8
-6
lines changed

4 files changed

+8
-6
lines changed

HISTORY.md

+5
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,11 @@ unreleased
44
* Remove `setImmediate` support check
55
* Restore `debug` dependency
66

7+
3.0.0 / ????-??-??
8+
==================
9+
10+
* Replace `is-promise` dependency with `instanceof` check (drops support for non-native promises)
11+
712
2.1.0 / 2025-02-10
813
==================
914

index.js

+1-2
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,6 @@
1212
* @private
1313
*/
1414

15-
const isPromise = require('is-promise')
1615
const Layer = require('./lib/layer')
1716
const { METHODS } = require('node:http')
1817
const parseUrl = require('parseurl')
@@ -646,7 +645,7 @@ function processParams (params, layer, called, req, res, done) {
646645

647646
try {
648647
const ret = fn(req, res, paramCallback, paramVal, key)
649-
if (isPromise(ret)) {
648+
if (ret instanceof Promise) {
650649
ret.then(null, function (error) {
651650
paramCallback(error || new Error('Rejected promise'))
652651
})

lib/layer.js

+2-3
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,6 @@
1212
* @private
1313
*/
1414

15-
const isPromise = require('is-promise')
1615
const pathRegexp = require('path-to-regexp')
1716
const debug = require('debug')('router:layer')
1817

@@ -115,7 +114,7 @@ Layer.prototype.handleError = function handleError (error, req, res, next) {
115114
const ret = fn(error, req, res, next)
116115

117116
// wait for returned promise
118-
if (isPromise(ret)) {
117+
if (ret instanceof Promise) {
119118
ret.then(null, function (error) {
120119
next(error || new Error('Rejected promise'))
121120
})
@@ -147,7 +146,7 @@ Layer.prototype.handleRequest = function handleRequest (req, res, next) {
147146
const ret = fn(req, res, next)
148147

149148
// wait for returned promise
150-
if (isPromise(ret)) {
149+
if (ret instanceof Promise) {
151150
ret.then(null, function (error) {
152151
next(error || new Error('Rejected promise'))
153152
})

package.json

-1
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,6 @@
1010
"repository": "pillarjs/router",
1111
"dependencies": {
1212
"debug": "^4.4.0",
13-
"is-promise": "^4.0.0",
1413
"parseurl": "^1.3.3",
1514
"path-to-regexp": "^8.0.0"
1615
},

0 commit comments

Comments
 (0)