Skip to content

Commit 7676290

Browse files
committed
Clarify agent-to-agent update limitations
1 parent 77dd039 commit 7676290

4 files changed

Lines changed: 12 additions & 8 deletions

File tree

plugins/thousandeyes/skills/thousandeyes-synthetic-monitoring/SKILL.md

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -68,7 +68,7 @@ Load [reference.md](reference.md) for product-language mapping, supported tools,
6868
- Network and Application Synthetics:
6969
- `http-server` for URL availability and response timing
7070
- `agent-to-server` for network path and reachability
71-
- `agent-to-agent` for inter-site connectivity between enterprise agents
71+
- `agent-to-agent` for inter-site connectivity between enterprise agents; currently create/get/delete only
7272
- `api` for multi-step API request workflows
7373
- `dns-server`, `dns-trace`, or `dnssec` for DNS monitoring
7474
- `bgp` when the user is monitoring route reachability for a prefix
@@ -97,7 +97,7 @@ Load [reference.md](reference.md) for product-language mapping, supported tools,
9797
5. For Browser Synthetics `web-transactions`, require an async-function style script and both `url` and `transaction_script`.
9898
6. For Browser Synthetics transaction creation, prefer existing examples from the ThousandEyes transaction scripting examples repository before writing a script from scratch.
9999
7. For API test creation, prefer a short, explicit step sequence with stable request names, full URLs, and only the headers/body fields the workflow needs.
100-
8. For `update`, do not guess the current test configuration. If required update fields are unclear, inspect the current test first.
100+
8. For `update`, do not guess the current test configuration. If required update fields are unclear, inspect the current test first. If `test_type` is `agent-to-agent`, explain that `update_synthetic_test` does not currently support that type and recommend recreating the test after confirmation.
101101

102102
### 5) Confirm before execution
103103

@@ -117,7 +117,7 @@ Do not execute until the user confirms, unless the user explicitly asked to run
117117
- `list_network_app_synthetics_tests` for discovery
118118
- `get_network_app_synthetics_test` for current-state inspection
119119
- `create_synthetic_test` for new scheduled tests
120-
- `update_synthetic_test` for changes to existing scheduled tests
120+
- `update_synthetic_test` for changes to existing scheduled tests, except `agent-to-agent`
121121
- `delete_synthetic_test` for removals
122122
- the matching `run_*_instant_test` tool for immediate validation
123123
- `get_templates` and `deploy_template` for template-based synthetic monitoring of an application
@@ -140,6 +140,7 @@ Always return:
140140
- Never delete a test unless both the exact `test_id` and `test_type` are known.
141141
- Never fabricate type-specific fields such as `url`, `server`, `domain`, `prefix`, or `target_agent_id`.
142142
- If the user asks for a partial update but the current test details are unclear, inspect the test first instead of guessing.
143+
- If the user wants to update an `agent-to-agent` test, explain that the current MCP tool does not support that update path and offer delete-and-recreate guidance instead.
143144
- If multiple tests match a name or target, stop and ask the user to choose.
144145
- Keep sample transaction scripts minimal and point to [reference.md](reference.md) for the required async structure.
145146
- Keep API test payloads minimal and readable; do not invent unsupported step-builder features beyond the MCP schema.

plugins/thousandeyes/skills/thousandeyes-synthetic-monitoring/reference.md

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ Use the narrowest valid test type for the target, but explain it in product lang
4141
| Test Type | Best For | Required Fields |
4242
| --- | --- | --- |
4343
| `agent-to-server` | Network and Application Synthetics for reachability, path, and latency | `test_name`, `server`, `agent_ids`; often `protocol` |
44-
| `agent-to-agent` | Network and Application Synthetics for inter-site connectivity | `test_name`, `agent_ids`, `target_agent_id`; often `protocol`, `port` |
44+
| `agent-to-agent` | Network and Application Synthetics for inter-site connectivity; currently create/get/delete only | `test_name`, `agent_ids`, `target_agent_id`; often `protocol`, `port` |
4545
| `http-server` | Network and Application Synthetics for URL availability and response timing | `test_name`, `url`, `agent_ids` |
4646
| `page-load` | Browser Synthetics for page performance and waterfalls | `test_name`, `url`, `agent_ids` |
4747
| `dns-server` | Network and Application Synthetics for DNS response time and availability | `test_name`, `domain`, `dns_servers`, `agent_ids` |
@@ -66,6 +66,7 @@ Additional update rules:
6666
- Include only the fields you intend to change, but do not guess unknown values.
6767
- If the exact current test is unclear, inspect it first with `get_network_app_synthetics_test`.
6868
- Keep `test_type` aligned with the existing test. Do not use update as a type conversion.
69+
- `agent-to-agent` is not currently supported by `update_synthetic_test`. If the user needs to change one, explain the limitation and recommend delete-and-recreate after inspection and confirmation.
6970

7071
Delete rules:
7172

skills/thousandeyes-synthetic-monitoring/SKILL.md

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -68,7 +68,7 @@ Load [reference.md](reference.md) for product-language mapping, supported tools,
6868
- Network and Application Synthetics:
6969
- `http-server` for URL availability and response timing
7070
- `agent-to-server` for network path and reachability
71-
- `agent-to-agent` for inter-site connectivity between enterprise agents
71+
- `agent-to-agent` for inter-site connectivity between enterprise agents; currently create/get/delete only
7272
- `api` for multi-step API request workflows
7373
- `dns-server`, `dns-trace`, or `dnssec` for DNS monitoring
7474
- `bgp` when the user is monitoring route reachability for a prefix
@@ -97,7 +97,7 @@ Load [reference.md](reference.md) for product-language mapping, supported tools,
9797
5. For Browser Synthetics `web-transactions`, require an async-function style script and both `url` and `transaction_script`.
9898
6. For Browser Synthetics transaction creation, prefer existing examples from the ThousandEyes transaction scripting examples repository before writing a script from scratch.
9999
7. For API test creation, prefer a short, explicit step sequence with stable request names, full URLs, and only the headers/body fields the workflow needs.
100-
8. For `update`, do not guess the current test configuration. If required update fields are unclear, inspect the current test first.
100+
8. For `update`, do not guess the current test configuration. If required update fields are unclear, inspect the current test first. If `test_type` is `agent-to-agent`, explain that `update_synthetic_test` does not currently support that type and recommend recreating the test after confirmation.
101101

102102
### 5) Confirm before execution
103103

@@ -117,7 +117,7 @@ Do not execute until the user confirms, unless the user explicitly asked to run
117117
- `list_network_app_synthetics_tests` for discovery
118118
- `get_network_app_synthetics_test` for current-state inspection
119119
- `create_synthetic_test` for new scheduled tests
120-
- `update_synthetic_test` for changes to existing scheduled tests
120+
- `update_synthetic_test` for changes to existing scheduled tests, except `agent-to-agent`
121121
- `delete_synthetic_test` for removals
122122
- the matching `run_*_instant_test` tool for immediate validation
123123
- `get_templates` and `deploy_template` for template-based synthetic monitoring of an application
@@ -140,6 +140,7 @@ Always return:
140140
- Never delete a test unless both the exact `test_id` and `test_type` are known.
141141
- Never fabricate type-specific fields such as `url`, `server`, `domain`, `prefix`, or `target_agent_id`.
142142
- If the user asks for a partial update but the current test details are unclear, inspect the test first instead of guessing.
143+
- If the user wants to update an `agent-to-agent` test, explain that the current MCP tool does not support that update path and offer delete-and-recreate guidance instead.
143144
- If multiple tests match a name or target, stop and ask the user to choose.
144145
- Keep sample transaction scripts minimal and point to [reference.md](reference.md) for the required async structure.
145146
- Keep API test payloads minimal and readable; do not invent unsupported step-builder features beyond the MCP schema.

skills/thousandeyes-synthetic-monitoring/reference.md

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ Use the narrowest valid test type for the target, but explain it in product lang
4141
| Test Type | Best For | Required Fields |
4242
| --- | --- | --- |
4343
| `agent-to-server` | Network and Application Synthetics for reachability, path, and latency | `test_name`, `server`, `agent_ids`; often `protocol` |
44-
| `agent-to-agent` | Network and Application Synthetics for inter-site connectivity | `test_name`, `agent_ids`, `target_agent_id`; often `protocol`, `port` |
44+
| `agent-to-agent` | Network and Application Synthetics for inter-site connectivity; currently create/get/delete only | `test_name`, `agent_ids`, `target_agent_id`; often `protocol`, `port` |
4545
| `http-server` | Network and Application Synthetics for URL availability and response timing | `test_name`, `url`, `agent_ids` |
4646
| `page-load` | Browser Synthetics for page performance and waterfalls | `test_name`, `url`, `agent_ids` |
4747
| `dns-server` | Network and Application Synthetics for DNS response time and availability | `test_name`, `domain`, `dns_servers`, `agent_ids` |
@@ -66,6 +66,7 @@ Additional update rules:
6666
- Include only the fields you intend to change, but do not guess unknown values.
6767
- If the exact current test is unclear, inspect it first with `get_network_app_synthetics_test`.
6868
- Keep `test_type` aligned with the existing test. Do not use update as a type conversion.
69+
- `agent-to-agent` is not currently supported by `update_synthetic_test`. If the user needs to change one, explain the limitation and recommend delete-and-recreate after inspection and confirmation.
6970

7071
Delete rules:
7172

0 commit comments

Comments
 (0)