Merge pull request #133 from ty-ras/issue/129-remove-qol-improvement

#129 Looks like 'never' type just behaves very ba…

Author: stazz

endpoint-spec/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ty-ras/endpoint-spec",
-  "version": "2.2.2",
+  "version": "2.2.3",
   "author": {
     "name": "Stanislav Muhametsin",
     "email": "[email protected]",

endpoint-spec/src/api.types/url.types.ts

@@ -394,10 +394,10 @@ export type GetMethodArgsGeneric<
   TStateSpec,
 > = {
   state: dataBE.MaterializeRuntimeState<TStateHKT, TStateSpec>;
-} & (never extends TServerContext ? {} : { context: TServerContext }) &
-  (TProtocolSpec extends protocol.ProtocolSpecURL<infer TURLData>
-    ? { url: protocol.RuntimeOf<TURLData> }
-    : {}) &
+  context: TServerContext;
+} & (TProtocolSpec extends protocol.ProtocolSpecURL<infer TURLData>
+  ? { url: protocol.RuntimeOf<TURLData> }
+  : {}) &
   (TProtocolSpec extends protocol.ProtocolSpecQuery<infer TQuery>
     ? { query: protocol.RuntimeOf<TQuery> }
     : {}) &

endpoint-spec/src/implementation/app.ts

@@ -303,7 +303,7 @@ function newBuilderGenericImpl<
     ? processMethodArg
     : (pArgs) => {
         const boundMethod = processMethodArg(pArgs) ?? pArgs.boundMethod;
-        return (args) => boundMethod(data.omit(args, "context" as any) as any); // eslint-disable-line @typescript-eslint/no-unsafe-argument, @typescript-eslint/no-unused-vars
+        return (args) => boundMethod(data.omit(args, "context") as any); // eslint-disable-line @typescript-eslint/no-unsafe-argument, @typescript-eslint/no-unused-vars
       };
   // Define resetMetadataProviders separately, as defining it inline causes problems with function overload having different argument count.
   const resetMetadataProviders: api.ApplicationBuilderGeneric<