ref(aci): remove detector usage in WorkflowFireHistory and action firing

[cathteng]

Title

Should merge #102918 first. Also removes need for get_detectors_by_groupevents_bulk!

[codecov]

❌ 1 Tests Failed:

Tests completed	Failed	Passed	Skipped
29559	1	29558	236

View the top 1 failed test(s) by shortest run time

tests.sentry.workflow_engine.processors.test_delayed_workflow.TestFireActionsForGroups::test_fire_actions_for_groups__fire_actions

Stack Traces | 33s run time

#x1B[1m#x1B[.../workflow_engine/processors/test_delayed_workflow.py#x1B[0m:926: in test_fire_actions_for_groups__fire_actions
    assert first_call_kwargs["detector_id"] == self.detector.id
#x1B[1m#x1B[31mE   KeyError: 'detector_id'#x1B[0m

To view more test analytics, go to the Test Analytics Dashboard
_{📋 Got 3 mins? Take this short survey to help us improve Test Analytics.}

[cursor]

Bug: Invalid Events Trigger Failing Actions

Removing the detector existence check allows workflow fire histories to be created and actions to be triggered for events without detectors. Previously, events missing detectors were skipped with a warning. Now, trigger_action tasks will be queued and fail when get_detector_by_event raises Detector.DoesNotExist, causing unnecessary task failures and retries instead of gracefully handling missing detectors upfront.

 

[cursor]

Bug: Activity Actions Fail: Detector Type Mismatch

The unconditional call to get_detector_by_event raises TypeError for Activity events since it only supports GroupEvent. Previously, the detector was passed via detector_id parameter, avoiding this call for Activity-triggered actions. Now Activity events that trigger actions will fail when the async task attempts to fetch the detector.