chore: use our own http methods object instead of the enum inside the types package

Author: maxholman

lib/index.ts

@@ -1,8 +1,7 @@
 /* eslint-disable max-classes-per-file */
 import { Construct } from 'constructs';
-import { OpenAPIV3 } from 'openapi-types';
 
-export const { HttpMethods } = OpenAPIV3;
+export { HttpMethods } from './http-methods.js';
 
 // export the version of Constructs we are using to ensure we're
 // everyone is using the same version

lib/operation.ts

@@ -1,5 +1,6 @@
 import { Construct } from 'constructs';
 import type { OpenAPIV3_1 } from 'openapi-types';
+import type { HttpMethods } from './http-methods.js';
 import type { Parameter } from './parameter.js';
 import { RequestBody, type RequestBodyOptions } from './request-body.js';
 import type { Response } from './response.js';
@@ -24,7 +25,7 @@ export interface OperationOptions<TPath extends string = '/'> {
 export class Operation<TPath extends string = '/'> extends Construct {
   private readonly options: OperationOptions<TPath>;
 
-  public readonly method: OpenAPIV3_1.HttpMethods;
+  public readonly method: HttpMethods;
 
   private requestBody?: RequestBody;
 
@@ -34,7 +35,7 @@ export class Operation<TPath extends string = '/'> extends Construct {
 
   constructor(
     scope: Construct,
-    method: OpenAPIV3_1.HttpMethods,
+    method: HttpMethods,
     options: OperationOptions<TPath>,
   ) {
     super(scope, method);

lib/path.ts

@@ -2,6 +2,7 @@ import { strict } from 'node:assert';
 import { Construct } from 'constructs';
 import type { OpenAPIV3_1 } from 'openapi-types';
 import type { Api } from './api.js';
+import { HttpMethods } from './http-methods.js';
 import { Operation, type OperationOptions } from './operation.js';
 import type { Parameter } from './parameter.js';
 import type { Server } from './server.js';
@@ -25,7 +26,7 @@ export class Path<TPath extends string = '/'> extends Construct {
   }
 
   public addOperation(
-    method: OpenAPIV3_1.HttpMethods,
+    method: HttpMethods,
     options: OperationOptions<TPath>,
   ): this {
     // make sure we are not duplicating tags
@@ -43,7 +44,7 @@ export class Path<TPath extends string = '/'> extends Construct {
     return this.options.path;
   }
 
-  public synth(): OpenAPIV3_1.PathItemObject {
+  public synth() {
     return {
       ...Object.fromEntries(
         this.node