feat(aci): Add missing anomaly detection conditions

[scttcper]

Apparently we were missing these entirely. Also fixes some issues when editing an existing anomaly detector and cleans up an unused config value.

[codecov]

❌ 1 Tests Failed:

Tests completed	Failed	Passed	Skipped
12410	1	12409	10

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

DetectorEdit Metric Detector creates detector with dynamic detection and no resolution thresholds

Stack Traces | 1.93s run time

Error: expect(jest.fn()).toHaveBeenCalledWith(...expected)

Expected: "....../organizations/org-slug/detectors/", ObjectContaining {"data": ObjectContaining {"conditionGroup": {"conditions": [{"comparison": {"seasonality": "auto", "sensitivity": "high", "thresholdType": 0}, "conditionResult": 75, "type": "anomaly_detection"}], "logicType": "any"}, "config": {"detectionType": "dynamic", "sensitivity": "high"}, "dataSources": [{"aggregate": "count(span.duration)", "dataset": "events_analytics_platform", "environment": null, "eventTypes": ["trace_item_span"], "query": "", "queryType": 1, "timeWindow": 3600}], "name": "Dynamic", "owner": null, "projectId": "2", "type": "metric_issue", "workflowIds": []}}
Received: "....../organizations/org-slug/detectors/", {"data": {"conditionGroup": {"conditions": [{"comparison": {"seasonality": "auto", "sensitivity": "high", "thresholdType": 0}, "conditionResult": 75, "type": "anomaly_detection"}], "logicType": "any"}, "config": {"detectionType": "dynamic"}, "dataSources": [{"aggregate": "count(span.duration)", "dataset": "events_analytics_platform", "environment": null, "eventTypes": ["trace_item_span"], "query": "", "queryType": 1, "timeWindow": 3600}], "description": null, "name": "Dynamic", "owner": null, "projectId": "2", "type": "metric_issue", "workflowIds": []}, "error": [Function error], "method": "POST", "success": [Function success]}

Number of calls: 1

Ignored nodes: comments, script, style
...
    at toHaveBeenCalledWith (.../views/detectors/new-setting.spec.tsx:563:36)
    at runWithExpensiveErrorDiagnosticsDisabled (.../sentry/node_modules/.pnpm/@testing-library+dom@10.4.0/node_modules/@.../dom/dist/config.js:47:12)
    at checkCallback (.../sentry/node_modules/.pnpm/@testing-library+dom@10.4.0/node_modules/@.../dom/dist/wait-for.js:124:77)
    at checkRealTimersCallback (.../sentry/node_modules/.pnpm/@testing-library+dom@10.4.0/node_modules/@.../dom/dist/wait-for.js:118:16)
    at Timeout.task [as _onTimeout] (.../sentry/node_modules/.pnpm/jsdom@26.1..../jsdom/browser/Window.js:579:19)
    at listOnTimeout (node:internal/timers:588:17)
    at processTimers (node:internal/timers:523:7)

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

[mifu67]

Should also be removed here

[scttcper]

this one is sensitivity

[mifu67]

Sensitivity needs to be removed too 😅