fix: improve server error messages to include HTTP status code

crates/goose/src/providers/utils.rs

@@ -68,6 +68,17 @@ fn check_context_length_exceeded(text: &str) -> bool {
         .any(|phrase| text_lower.contains(phrase))
 }
 
+fn format_server_error_message(status_code: u16, payload: Option<&Value>) -> String {
+    if let Some(p) = payload {
+        format!("HTTP {}: {:?}", status_code, p)
+    } else {
+        format!(
+            "HTTP {}: No response body received from server",
+            status_code
+        )
+    }
+}
+
 pub fn map_http_error_to_provider_error(
     status: StatusCode,
     payload: Option<Value>,
@@ -116,7 +127,10 @@ pub fn map_http_error_to_provider_error(
             details: format!("{:?}", payload),
             retry_delay: None,
         },
-        _ if status.is_server_error() => ProviderError::ServerError(format!("{:?}", payload)),
+        _ if status.is_server_error() => ProviderError::ServerError(format_server_error_message(
+            status.as_u16(),
+            payload.as_ref(),
+        )),
         _ => ProviderError::RequestFailed(format!("Request failed with status: {}", status)),
     };
 
@@ -295,12 +309,9 @@ pub async fn handle_response_google_compat(response: Response) -> Result<Value,
                 retry_delay,
             })
         }
-        _ if final_status.is_server_error() => {
-            Err(ProviderError::ServerError(format!("{:?}", payload)))
-        }
-        StatusCode::INTERNAL_SERVER_ERROR | StatusCode::SERVICE_UNAVAILABLE => {
-            Err(ProviderError::ServerError(format!("{:?}", payload)))
-        }
+        _ if final_status.is_server_error() => Err(ProviderError::ServerError(
+            format_server_error_message(final_status.as_u16(), payload.as_ref()),
+        )),
         _ => {
             tracing::debug!(
                 "{}", format!("Provider request failed with status: {}. Payload: {:?}", final_status, payload)
@@ -1112,13 +1123,13 @@ mod tests {
             (
                 StatusCode::INTERNAL_SERVER_ERROR,
                 None,
-                ProviderError::ServerError("None".to_string()),
+                ProviderError::ServerError("HTTP 500: No response body received from server".to_string()),
             ),
             // is_server_error() with payload
             (
                 StatusCode::BAD_GATEWAY,
                 Some(json!({"error": "upstream error"})),
-                ProviderError::ServerError("Some(Object {\"error\": String(\"upstream error\")})".to_string()),
+                ProviderError::ServerError("HTTP 502: Object {\"error\": String(\"upstream error\")}".to_string()),
             ),
             // Default - any other status code
             (