perf: skip handler return value validation

core/src/router/base.ts

@@ -362,7 +362,7 @@ export abstract class BaseActionRouter<K extends ActionKind> extends BaseRouter
     handler: GetActionTypeHandler<ActionTypeClasses<K>[T], any>
   ) {
     const pluginName = plugin.name
-    const schema = this.handlerDescriptions[handlerType].resultSchema
+    // const schema = this.handlerDescriptions[handlerType].resultSchema
 
     // Wrap the handler with identifying attributes
     const wrapped = Object.assign(
@@ -375,10 +375,11 @@ export abstract class BaseActionRouter<K extends ActionKind> extends BaseRouter
             )} handler on ${pluginName} provider. Called with ${args.length} arguments.`,
           })
         }
-        const kind = this.kind
-        return validateSchema(result, schema, {
-          context: `${String(handlerType)} handler output from provider ${pluginName} for ${kind} type ${actionType}`,
-        })
+        // const kind = this.kind
+        // return validateSchema(result, schema, {
+        //   context: `${String(handlerType)} handler output from provider ${pluginName} for ${kind} type ${actionType}`,
+        // })
+        return result
       })),
       { handlerType, pluginName, actionType }
     )

core/src/router/module.ts

@@ -169,7 +169,7 @@ export class ModuleRouter extends BaseRouter {
     handler: ModuleActionHandlers[T]
   ) {
     const pluginName = plugin.name
-    const schema = this.moduleHandlerDescriptions[handlerType].resultSchema
+    // const schema = this.moduleHandlerDescriptions[handlerType].resultSchema
 
     // Wrap the handler with identifying attributes
     const wrapped = Object.assign(
@@ -183,9 +183,10 @@ export class ModuleRouter extends BaseRouter {
             message: `Got empty response from ${moduleType}.${handlerType} handler (called with ${args.length} args) on ${pluginName} provider.`,
           })
         }
-        return validateSchema(result, schema, {
-          context: `${handlerType} handler output from provider ${pluginName} for module type ${moduleType} `,
-        })
+        // return validateSchema(result, schema, {
+        //   context: `${handlerType} handler output from provider ${pluginName} for module type ${moduleType} `,
+        // })
+        return result
       }),
       { handlerType, pluginName, moduleType, wrapped: handler }
     )

core/src/router/provider.ts

@@ -241,7 +241,7 @@ export class ProviderRouter extends BaseRouter {
     handler: ProviderHandlers[T]
   ) {
     const pluginName = plugin.name
-    const schema = this.pluginHandlerDescriptions[handlerType].resultSchema
+    // const schema = this.pluginHandlerDescriptions[handlerType].resultSchema
 
     // Wrap the handler with identifying attributes
     const wrapped: WrappedPluginHandlers[T] = Object.assign(
@@ -255,7 +255,8 @@ export class ProviderRouter extends BaseRouter {
             message: `Got empty response from ${handlerType} handler on ${pluginName} provider. Called with ${args.length} args.`,
           })
         }
-        return validateSchema(result, schema, { context: `${handlerType} output from plugin ${pluginName}` })
+        // return validateSchema(result, schema, { context: `${handlerType} output from plugin ${pluginName}` })
+        return result
       },
       { handlerType, pluginName }
     ) as WrappedPluginHandlers[T]

core/test/integ/src/plugins/kubernetes/helm/config.ts

@@ -122,7 +122,6 @@ describe("configureHelmModule", () => {
           dependencies: [],
           disabled: false,
           sourceModuleName: "api-image",
-          timeout: DEFAULT_DEPLOY_TIMEOUT_SEC,
           spec: expectedSpec,
         },
       ],

core/test/integ/src/plugins/kubernetes/kubernetes-type/config.ts

@@ -133,7 +133,6 @@ describe("configureKubernetesModule", () => {
           tasks: [taskSpec],
           timeout: 300,
         },
-        timeout: DEFAULT_DEPLOY_TIMEOUT_SEC,
       },
     ])