Use lenient resolver return type

[mdmower-csnw]

In #921, a stronger type was applied to OperationHandlerOptions['resolver'] so that end users would have an idea of what the parameters are for their custom resolvers. It went too far in stipulating a return type. Set the return type to unknown and let users decide how much type safety they need in their resolver.

Fixes #952

[mdmower-csnw]

@csuermann - could you confirm that this would fix the issue you ran into in #952? A quick way to test would be to manually edit node_modules/express-openapi-validator/dist/framework/types.d.ts in your project, changing

export declare type OperationHandlerOptions = {
    basePath: string;
    resolver: (handlersPath: string, route: RouteMetadata, apiDoc: OpenAPIV3.Document) => RequestHandler | Promise<RequestHandler>;
};

to

export declare type OperationHandlerOptions = {
    basePath: string;
    resolver: (handlersPath: string, route: RouteMetadata, apiDoc: OpenAPIV3.Document) => unknown;
};

When you're done testing, that file could be restored.

[csuermann]

Hi @mdmower-csnw! Thank you! That seems to work!

[mdmower-csnw]

@cdimascio - this is ready for review. It fixes an issue I introduced.

[cdimascio]

Thanks!