chore: dependency maintenance + removing dead code (#287)

## 🧰 Changes

* [x] Drops `@apidevtools/swagger-mehtods` in favor of a hardcoded list
of HTTP methods. This list never changes so there's really no need to
pull in a separate library for it.
* [x] Removing a `node:util.format` export that was never being used.

Author: erunion

Diff for: lib/util.js

@@ -3,7 +3,6 @@ const util = require('util');
 
 const url = require('@readme/json-schema-ref-parser/lib/util/url');
 
-exports.format = util.format;
 exports.inherits = util.inherits;
 
 /**

Diff for: lib/validators/spec/openapi.js

@@ -1,8 +1,12 @@
-const swaggerMethods = require('@apidevtools/swagger-methods');
 const { ono } = require('@jsdevtools/ono');
 
 const util = require('../../util');
 
+/**
+ * @see {@link https://github.com/OAI/OpenAPI-Specification/blob/main/versions/3.1.0.md#path-item-object}
+ */
+const supportedHTTPMethods = ['get', 'put', 'post', 'delete', 'options', 'head', 'patch', 'trace'];
+
 module.exports = validateSpec;
 
 /**
@@ -58,8 +62,7 @@ function validateSpec(api) {
  * @param {string}        operationIds  - An array of collected operationIds found in other paths
  */
 function validatePath(api, path, pathId, operationIds) {
-  // `@apidevtools/swagger-methods` doesn't ship a `trace` method so we need to improvise.
-  [...swaggerMethods, 'trace'].forEach(operationName => {
+  supportedHTTPMethods.forEach(operationName => {
     const operation = path[operationName];
     const operationId = `${pathId}/${operationName}`;
 

Diff for: lib/validators/spec/swagger.js

@@ -1,8 +1,12 @@
-const swaggerMethods = require('@apidevtools/swagger-methods');
 const { ono } = require('@jsdevtools/ono');
 
 const util = require('../../util');
 
+/**
+ * @see {@link https://github.com/OAI/OpenAPI-Specification/blob/main/versions/2.0.md#path-item-object}
+ */
+const supportedHTTPMethods = ['get', 'put', 'post', 'delete', 'options', 'head', 'patch'];
+
 const primitiveTypes = ['array', 'boolean', 'integer', 'number', 'string'];
 const schemaTypes = ['array', 'boolean', 'integer', 'number', 'string', 'object', 'null', undefined];
 
@@ -47,7 +51,7 @@ function validateSpec(api) {
  * @param {string}        operationIds  - An array of collected operationIds found in other paths
  */
 function validatePath(api, path, pathId, operationIds) {
-  swaggerMethods.forEach(operationName => {
+  supportedHTTPMethods.forEach(operationName => {
     const operation = path[operationName];
     const operationId = `${pathId}/${operationName}`;
 

Diff for: package-lock.json

@@ -46,7 +46,6 @@
     "test": "vitest run --coverage"
   },
   "dependencies": {
-    "@apidevtools/swagger-methods": "^3.0.2",
     "@jsdevtools/ono": "^7.1.3",
     "@readme/better-ajv-errors": "^2.1.2",
     "@readme/json-schema-ref-parser": "^1.2.0",
@@ -60,7 +59,7 @@
   },
   "devDependencies": {
     "@readme/eslint-config": "^14.1.2",
-    "@types/node": "^22.10.10",
+    "@types/node": "^22.12.0",
     "@vitest/coverage-v8": "^3.0.4",
     "eslint": "^8.56.0",
     "openapi-types": "^12.1.3",