Skip to content

Commit 735e138

Browse files
committed
fix(datadog): add SEV-0, document page-size caps, drop unsourced output defaults
- The severity dropdown omitted SEV-0, which IncidentSeverity allows and both incident tool descriptions already advertised. - Page-size descriptions now state Datadog's documented default of 10 and cap of 100 instead of an arbitrary example, so an agent does not request an out-of-range page. - trigger_synthetics_tests emitted an explicit null for a string-typed optional output, and update_synthetics_status reported 'live' on the error path regardless of what the caller actually requested.
1 parent 462b379 commit 735e138

9 files changed

Lines changed: 11 additions & 10 deletions

File tree

‎apps/docs/content/docs/en/integrations/datadog.mdx‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -368,7 +368,7 @@ List incidents for the organization. Requires the Incident Management `incident_
368368
| Parameter | Type | Required | Description |
369369
| --------- | ---- | -------- | ----------- |
370370
| `include` | string | No | Comma-separated related resources to include: "users" and/or "attachments" |
371-
| `pageSize` | number | No | Number of incidents to return per page \(e.g., 10\) |
371+
| `pageSize` | number | No | Number of incidents to return per page \(default: 10, max: 100\) |
372372
| `pageOffset` | number | No | Index of the first incident to return \(e.g., 0, 10, 20\) |
373373
| `apiKey` | string | Yes | Datadog API key |
374374
| `applicationKey` | string | Yes | Datadog Application key |
@@ -1140,7 +1140,7 @@ List Cloud SIEM detection rules. Requires the `security_monitoring_rules_read` p
11401140
| --------- | ---- | -------- | ----------- |
11411141
| `query` | string | No | Search query filtering rules by attributes such as type, source, or tags \(e.g., "type:log_detection source:cloudtrail"\) |
11421142
| `sort` | string | No | Sort attribute, prefix with "-" for descending: name, creation_date, update_date, enabled, type, highest_severity, or source |
1143-
| `pageSize` | number | No | Number of rules per page \(e.g., 25\) |
1143+
| `pageSize` | number | No | Number of rules per page \(default: 10, max: 100\) |
11441144
| `pageNumber` | number | No | Page to retrieve, starting at zero |
11451145
| `apiKey` | string | Yes | Datadog API key |
11461146
| `applicationKey` | string | Yes | Datadog Application key |
@@ -1210,7 +1210,7 @@ List service definitions from the Datadog Service Catalog. Requires the `apm_ser
12101210

12111211
| Parameter | Type | Required | Description |
12121212
| --------- | ---- | -------- | ----------- |
1213-
| `pageSize` | number | No | Number of service definitions per page \(e.g., 25\) |
1213+
| `pageSize` | number | No | Number of service definitions per page \(default: 10, max: 100\) |
12141214
| `pageNumber` | number | No | Page to retrieve, starting at zero |
12151215
| `schemaVersion` | string | No | Schema version to return \(e.g., "v2", "v2.1", "v2.2"\) |
12161216
| `apiKey` | string | Yes | Datadog API key |

‎apps/sim/blocks/blocks/datadog.ts‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -784,6 +784,7 @@ Return ONLY the comma-separated tag list - no explanations, no extra text.`,
784784
title: 'Severity',
785785
type: 'dropdown',
786786
options: [
787+
{ label: 'SEV-0', id: 'SEV-0' },
787788
{ label: 'SEV-1', id: 'SEV-1' },
788789
{ label: 'SEV-2', id: 'SEV-2' },
789790
{ label: 'SEV-3', id: 'SEV-3' },

‎apps/sim/tools/datadog/list_incidents.ts‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@ export const listIncidentsTool: ToolConfig<ListIncidentsParams, ListIncidentsRes
3030
type: 'number',
3131
required: false,
3232
visibility: 'user-or-llm',
33-
description: 'Number of incidents to return per page (e.g., 10)',
33+
description: 'Number of incidents to return per page (default: 10, max: 100)',
3434
},
3535
pageOffset: {
3636
type: 'number',

‎apps/sim/tools/datadog/list_security_rules.ts‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@ export const listSecurityRulesTool: ToolConfig<ListSecurityRulesParams, ListSecu
3333
type: 'number',
3434
required: false,
3535
visibility: 'user-or-llm',
36-
description: 'Number of rules per page (e.g., 25)',
36+
description: 'Number of rules per page (default: 10, max: 100)',
3737
},
3838
pageNumber: {
3939
type: 'number',

‎apps/sim/tools/datadog/list_services.ts‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ export const listServicesTool: ToolConfig<ListServicesParams, ListServicesRespon
1919
type: 'number',
2020
required: false,
2121
visibility: 'user-or-llm',
22-
description: 'Number of service definitions per page (e.g., 25)',
22+
description: 'Number of service definitions per page (default: 10, max: 100)',
2323
},
2424
pageNumber: {
2525
type: 'number',

‎apps/sim/tools/datadog/trigger_synthetics_tests.ts‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -71,7 +71,7 @@ export const triggerSyntheticsTestsTool: ToolConfig<
7171
return {
7272
success: true,
7373
output: {
74-
batchId: data.batch_id ?? null,
74+
batchId: data.batch_id,
7575
triggeredCheckIds: data.triggered_check_ids ?? [],
7676
results: data.results ?? [],
7777
locations: data.locations ?? [],

‎apps/sim/tools/datadog/types.ts‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -768,7 +768,7 @@ export interface UpdateSyntheticsStatusParams extends DatadogBaseParams {
768768

769769
interface UpdateSyntheticsStatusOutput {
770770
success: boolean
771-
status: SyntheticsTestPauseStatus
771+
status?: SyntheticsTestPauseStatus
772772
}
773773

774774
export interface UpdateSyntheticsStatusResponse extends ToolResponse {

‎apps/sim/tools/datadog/update_synthetics_status.ts‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,7 @@ export const updateSyntheticsStatusTool: ToolConfig<
5959
},
6060

6161
transformResponse: async (response: Response, params) => {
62-
const status = params?.newStatus ?? 'live'
62+
const status = params?.newStatus
6363

6464
if (!response.ok) {
6565
return {

‎apps/sim/tools/generated/tool-metadata.ts‎

Lines changed: 1 addition & 1 deletion
Large diffs are not rendered by default.

0 commit comments

Comments
 (0)