Skip to content

Commit f4afe79

Browse files
adinauerclaude
andcommitted
docs(options): Clarify conditional signal options
State that options such as sampling should only be added when the signal supports them. Avoid implying that Logs and Metrics currently expose sampling controls. Co-Authored-By: Claude <noreply@anthropic.com>
1 parent cb65d6e commit f4afe79

1 file changed

Lines changed: 3 additions & 2 deletions

File tree

‎.cursor/rules/options.mdc‎

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -19,8 +19,9 @@ Each namespaced options class is a `public static final class` inside `SentryOpt
1919

2020
Do not assume that a namespaced signal needs an aggregate `enabled` field. In particular, Logs and
2121
Metrics are available through deliberate API calls whenever the SDK is enabled. Their namespaced
22-
options contain behavior such as sampling, `beforeSend`, limits, and processor factories. Automatic
23-
capture integrations use source-local enable options that default to `false`.
22+
options contain signal-specific behavior such as `beforeSend`, limits, and processor factories. Add
23+
options such as sampling only when the signal supports them. Automatic capture integrations use
24+
source-local enable options that default to `false`.
2425

2526
To add a new namespaced options class:
2627
1. Create the `public static final class` inside `SentryOptions` with fields, getters/setters, and any callback interfaces

0 commit comments

Comments
 (0)