From 76a5322000fc262ae6c780350e702548dfcc78e5 Mon Sep 17 00:00:00 2001 From: Yong Tao Date: Mon, 24 Nov 2025 14:07:07 -0800 Subject: [PATCH] fix batch-get api prefix in use_aws's READONLY_OPS list --- crates/chat-cli/src/cli/chat/tools/use_aws.rs | 2 +- docs/built-in-tools.md | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/crates/chat-cli/src/cli/chat/tools/use_aws.rs b/crates/chat-cli/src/cli/chat/tools/use_aws.rs index b7390744cd..05aacbde89 100644 --- a/crates/chat-cli/src/cli/chat/tools/use_aws.rs +++ b/crates/chat-cli/src/cli/chat/tools/use_aws.rs @@ -31,7 +31,7 @@ use crate::cli::agent::{ use crate::os::Os; use crate::util::tool_permission_checker::is_tool_in_allowlist; -const READONLY_OPS: [&str; 6] = ["get", "describe", "list", "ls", "search", "batch_get"]; +const READONLY_OPS: [&str; 6] = ["get", "describe", "list", "ls", "search", "batch-get"]; // TODO: we should perhaps composite this struct with an interface that we can use to mock the // actual cli with. That will allow us to more thoroughly test it. diff --git a/docs/built-in-tools.md b/docs/built-in-tools.md index a0f56305cf..4144975144 100644 --- a/docs/built-in-tools.md +++ b/docs/built-in-tools.md @@ -155,7 +155,7 @@ Make AWS CLI API calls with the specified service, operation, and parameters. |--------|------|---------|-------------| | `allowedServices` | array of strings | `[]` | List of AWS services that can be accessed without prompting | | `deniedServices` | array of strings | `[]` | List of AWS services to deny. Deny rules are evaluated before allow rules | -| `autoAllowReadonly` | boolean | `false` | Whether to automatically allow read-only operations (get, describe, list, ls, search, batch_get) without prompting | +| `autoAllowReadonly` | boolean | `false` | Whether to automatically allow read-only operations (get, describe, list, ls, search, batch-get) without prompting | ## Using Tool Settings in Agent Configuration