9 months ago
be68a59The previous fix incorrectly treated all null extraction results as zero, which broke the test expecting unparseable data (like 'NaN') to fail. Now we distinguish between: - Empty structures (empty arrays, null, undefined) → treat as 0 - Unparseable data with content (invalid strings, NaN) → fail the check This fixes the observability.slo.test.ts test while still addressing the original P1 issue of empty log query results causing false SLO failures. Co-authored-by: mweinbach <mweinbach@users.noreply.github.com>
Parent49191b7