fix: use_aws printing default profile when its not used, minor updates to agent docs (#2617)

Author: brandonskiser

crates/chat-cli/src/cli/chat/parser.rs

@@ -681,8 +681,10 @@ mod tests {
     use super::*;
 
     #[tokio::test]
-    async fn test_parse() {
+    async fn test_response_parser_ignores_licensed_code() {
         // let _ = tracing_subscriber::fmt::try_init();
+
+        let content_to_ignore = "IGNORE ME PLEASE";
         let tool_use_id = "TEST_ID".to_string();
         let tool_name = "execute_bash".to_string();
         let tool_args = serde_json::json!({
@@ -698,7 +700,7 @@ mod tests {
                 content: " there".to_string(),
             },
             ChatResponseStream::AssistantResponseEvent {
-                content: "IGNORE ME PLEASE".to_string(),
+                content: content_to_ignore.to_string(),
             },
             ChatResponseStream::CodeReferenceEvent(()),
             ChatResponseStream::ToolUseEvent {
@@ -741,8 +743,14 @@ mod tests {
             Arc::new(Mutex::new(None)),
         );
 
+        let mut output = String::new();
         for _ in 0..5 {
-            println!("{:?}", parser.recv().await.unwrap());
+            output.push_str(&format!("{:?}", parser.recv().await.unwrap()));
         }
+
+        assert!(
+            !output.contains(content_to_ignore),
+            "assistant text preceding a code reference should be ignored as this indicates licensed code is being returned"
+        );
     }
 }

crates/chat-cli/src/cli/chat/tools/use_aws.rs

@@ -136,8 +136,6 @@ impl UseAws {
 
         if let Some(ref profile_name) = self.profile_name {
             queue!(output, style::Print(format!("Profile name: {}\n", profile_name)))?;
-        } else {
-            queue!(output, style::Print("Profile name: default\n".to_string()))?;
         }
 
         queue!(output, style::Print(format!("Region: {}", self.region)))?;

docs/agent-format.md

@@ -233,7 +233,7 @@ The `useLegacyMcpJson` field determines whether to include MCP servers defined i
 }
 ```
 
-When set to `true`, the agent will have access to all MCP servers defined in the global configuration in addition to those defined in the agent's `mcpServers` field.
+When set to `true`, the agent will have access to all MCP servers defined in the global and local configurations in addition to those defined in the agent's `mcpServers` field.
 
 ## Complete Example
 

docs/built-in-tools.md

@@ -159,7 +159,7 @@ Tools can be explicitly allowed in the `allowedTools` section of the agent confi
 }
 ```
 
-If a tool is not in the `allowedTools` list, the user will be prompted for permission when the tool is used.
+If a tool is not in the `allowedTools` list, the user will be prompted for permission when the tool is used unless an allowed `toolSettings` configuration is set.
 
 Some tools have default permission behaviors:
 - `fs_read` and `report_issue` are trusted by default