Skip to content

Conversation

@OliverJAsh
Copy link

Motivation

This will blow up at runtime, but there's no TypeScript error:

beforeSend: (event, ctx) => {
  // ❌ No TypeScript error
  'foo' in ctx;

This PR fixes the types so we do get a TypeScript error.

Changes

Test instructions

Checklist

  • Tested locally
  • Tested on staging
  • Added unit tests for this change.
  • Added e2e/integration tests for this change.

@OliverJAsh OliverJAsh requested a review from a team as a code owner October 17, 2025 13:39
@datadog-datadog-prod-us1
Copy link

🎯 Code Coverage
Patch Coverage: 100.00%
Total Coverage: 92.63% (+0.00%)

View detailed report

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 8f52631 | Docs | Was this helpful? Give us feedback!

@thomas-lebeau
Copy link
Collaborator

@OliverJAsh, Thank you for your contribution.

Could you accept the license/cla so we can merge your PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants