This ExecutionMetricsOnWebhooksFlag flag is always enabled in production and there is no known use case for disabling it. Keeping the flag adds unnecessary branching and cognitive overhead.
This issue proposes to:
- Remove the
ExecutionMetricsOnWebhooksFlag feature flag
- Always attach execution metrics to webhook events
- Delete the flag definition and related constants
This is a cleanup / simplification change with no user-facing behavior change.
Motivation
- Reduce feature flag surface area
- Make webhook behavior deterministic
- Improve maintainability and readability
Request for Approval
Since this is a straightforward cleanup, I’d like to ask if it’s okay to open a PR for this.
Happy to follow whatever direction you prefer — just wanted to check before starting the work 🙂
/cc @jakubno
This ExecutionMetricsOnWebhooksFlag flag is always enabled in production and there is no known use case for disabling it. Keeping the flag adds unnecessary branching and cognitive overhead.
This issue proposes to:
ExecutionMetricsOnWebhooksFlagfeature flagThis is a cleanup / simplification change with no user-facing behavior change.
Motivation
Request for Approval
Since this is a straightforward cleanup, I’d like to ask if it’s okay to open a PR for this.
Happy to follow whatever direction you prefer — just wanted to check before starting the work 🙂
/cc @jakubno