fix: drop merge-descriptors in favor of a native mixin helper

Author: Abdel-Monaam-Aouini

lib/express.js

@@ -14,7 +14,6 @@
 
 var bodyParser = require('body-parser')
 var EventEmitter = require('node:events').EventEmitter;
-var mixin = require('merge-descriptors');
 var proto = require('./application');
 var Router = require('router');
 var req = require('./request');
@@ -79,3 +78,27 @@ exports.raw = bodyParser.raw
 exports.static = require('serve-static');
 exports.text = bodyParser.text
 exports.urlencoded = bodyParser.urlencoded
+
+/**
+ * Copy the named property descriptors from `source` to `target`.
+ * Overwrites existing properties by default unless `overwrite` is `false`.
+ * @private
+ */
+function mixin(target, source, overwrite) {
+  if (overwrite === undefined) {
+    overwrite = true;
+  }
+
+  var names = Object.getOwnPropertyNames(source);
+  for (var i = 0; i < names.length; i++) {
+    var name = names[i];
+    if (overwrite === false && Object.prototype.hasOwnProperty.call(target, name)) {
+      continue;
+    }
+
+    var descriptor = Object.getOwnPropertyDescriptor(source, name);
+    Object.defineProperty(target, name, descriptor);
+  }
+
+  return target;
+}

package.json

@@ -46,7 +46,6 @@
     "finalhandler": "^2.1.0",
     "fresh": "^2.0.0",
     "http-errors": "^2.0.0",
-    "merge-descriptors": "^2.0.0",
     "mime-types": "^3.0.0",
     "on-finished": "^2.4.1",
     "once": "^1.4.0",