From 39ab42539cece98042798a1832a5b22e088f3d12 Mon Sep 17 00:00:00 2001 From: Kydoimos97 Date: Wed, 10 Jun 2026 16:42:26 -0600 Subject: [PATCH] fix(incident): use lowercase needswrenchbotcheck field key MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Datadog stores the field as 'needswrenchbotcheck' (all lowercase). The previous casing 'NeedsWrenchBotCheck' was silently discarded by the API — the PATCH returned 200 but the field never landed on the incident, which is why the notification rule never fired. --- puppy_kit/commands/incident.py | 2 +- tests/commands/test_incident.py | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/puppy_kit/commands/incident.py b/puppy_kit/commands/incident.py index b567b9b..d71b762 100644 --- a/puppy_kit/commands/incident.py +++ b/puppy_kit/commands/incident.py @@ -728,7 +728,7 @@ def update_incident( "value": is_duplicate.capitalize(), } if needs_wrench_bot_check is not None: - field_data["NeedsWrenchBotCheck"] = { + field_data["needswrenchbotcheck"] = { "type": "dropdown", "value": needs_wrench_bot_check.capitalize(), } diff --git a/tests/commands/test_incident.py b/tests/commands/test_incident.py index f57e2a8..9e27cdc 100644 --- a/tests/commands/test_incident.py +++ b/tests/commands/test_incident.py @@ -454,7 +454,7 @@ def test_update_incident_needs_wrench_bot_check(self, mock_client, runner): assert "updated" in result.output _, kwargs = mock_patch.call_args sent_fields = kwargs["json"]["data"]["attributes"]["fields"] - assert sent_fields["NeedsWrenchBotCheck"] == {"type": "dropdown", "value": "Yes"} + assert sent_fields["needswrenchbotcheck"] == {"type": "dropdown", "value": "Yes"} class TestSetStatusIncident: