fix(aci): Cache feature checks to keep span counts low (#95305)

delayed_workflows traces that are timed out or close to it are
frequently at the 1000 span limit, which means they're incomplete.
Caching features.has calls when they aren't expected to change avoids
logging 1 (or potentially more) spans per group that ultimately isn't
interesting to our performance work, making investigations easier.

Author: kcons

src/sentry/workflow_engine/processors/delayed_workflow.py

@@ -628,6 +628,14 @@ def fire_actions_for_groups(
         ).filter(workflow__in=workflows)
     }
 
+    # Feature check caching to keep us within the trace budget.
+    should_trigger_actions = features.has(
+        "organizations:workflow-engine-trigger-actions", organization
+    )
+    should_trigger_actions_async = features.has(
+        "organizations:workflow-engine-action-trigger-async", organization
+    )
+
     total_actions = 0
     with track_batch_performance(
         "workflow_engine.delayed_workflow.fire_actions_for_groups.loop",
@@ -701,15 +709,9 @@ def fire_actions_for_groups(
                 )
                 total_actions += len(filtered_actions)
 
-                if features.has(
-                    "organizations:workflow-engine-trigger-actions",
-                    organization,
-                ):
+                if should_trigger_actions:
                     for action in filtered_actions:
-                        if features.has(
-                            "organizations:workflow-engine-action-trigger-async",
-                            organization,
-                        ):
+                        if should_trigger_actions_async:
                             task_params = build_trigger_action_task_params(
                                 action, detector, workflow_event_data
                             )