From 26f0e6ad7e0103dd4aa5f908411f362bfe54074d Mon Sep 17 00:00:00 2001 From: John Dvorak Date: Fri, 22 May 2026 14:06:06 -0700 Subject: [PATCH] chore: remove delay no-op handlers from chaos-v3, document name collision --- src/quality/chaos-v3.ts | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/src/quality/chaos-v3.ts b/src/quality/chaos-v3.ts index 2497d71..03fecdb 100644 --- a/src/quality/chaos-v3.ts +++ b/src/quality/chaos-v3.ts @@ -56,7 +56,7 @@ export interface ChaosApplicationResult { type InboundChaosHandler = (ctx: EvalContext, event: ChaosEvent) => EvalContext const inboundHandlers: Record = { - 'inbound-delay': (ctx) => ctx, + // inbound-delay: handled at transport level via sleep(), not here 'inbound-error': (ctx, event) => ({ ...ctx, response: { @@ -203,7 +203,7 @@ const outboundHandlers: Record = { }), 'outbound-corruption': (response, event) => applyCorruptionToDependencyResponse(response, event), - 'outbound-delay': (response) => response, + // outbound-delay: handled at transport level via sleep(), not here } export function applyChaosToDependencyResponse( @@ -239,6 +239,9 @@ function applyCorruptionToDependencyResponse( } /** * Apply all outbound chaos events to a set of dependency responses. + * Note: triple-boundary-testing.ts defines a separate applyChaosToAllResponses + * with different types (DependencyResponseSample). This chaos-v3 version operates + * on DependencyResponse and is used by tests and the chaos handler. */ export function applyChaosToAllResponses( responses: ReadonlyArray,