simple-agents-py 0.1.7__tar.gz → 0.1.8__tar.gz

{simple_agents_py-0.1.7 → simple_agents_py-0.1.8}/Cargo.lock

@@ -2026,7 +2026,7 @@ dependencies = [
 
 [[package]]
 name = "simple-agent-type"
-version = "0.1.7"
+version = "0.1.8"
 dependencies = [
  "async-trait",
  "blake3",
@@ -2041,7 +2041,7 @@ dependencies = [
 
 [[package]]
 name = "simple-agents-cache"
-version = "0.1.7"
+version = "0.1.8"
 dependencies = [
  "async-trait",
  "simple-agent-type",
@@ -2050,7 +2050,7 @@ dependencies = [
 
 [[package]]
 name = "simple-agents-cli"
-version = "0.1.7"
+version = "0.1.8"
 dependencies = [
  "clap",
  "serde",
@@ -2067,7 +2067,7 @@ dependencies = [
 
 [[package]]
 name = "simple-agents-core"
-version = "0.1.7"
+version = "0.1.8"
 dependencies = [
  "async-trait",
  "serde",
@@ -2094,7 +2094,7 @@ dependencies = [
 
 [[package]]
 name = "simple-agents-ffi"
-version = "0.1.7"
+version = "0.1.8"
 dependencies = [
  "async-trait",
  "simple-agent-type",
@@ -2105,7 +2105,7 @@ dependencies = [
 
 [[package]]
 name = "simple-agents-healing"
-version = "0.1.7"
+version = "0.1.8"
 dependencies = [
  "criterion",
  "proptest",
@@ -2121,7 +2121,7 @@ dependencies = [
 
 [[package]]
 name = "simple-agents-macros"
-version = "0.1.7"
+version = "0.1.8"
 dependencies = [
  "proc-macro2",
  "quote",
@@ -2133,7 +2133,7 @@ dependencies = [
 
 [[package]]
 name = "simple-agents-napi"
-version = "0.1.7"
+version = "0.1.8"
 dependencies = [
  "napi",
  "napi-derive",
@@ -2145,7 +2145,7 @@ dependencies = [
 
 [[package]]
 name = "simple-agents-providers"
-version = "0.1.7"
+version = "0.1.8"
 dependencies = [
  "async-trait",
  "bytes",
@@ -2173,7 +2173,7 @@ dependencies = [
 
 [[package]]
 name = "simple-agents-py"
-version = "0.1.7"
+version = "0.1.8"
 dependencies = [
  "pyo3",
  "reqwest",
@@ -2186,7 +2186,7 @@ dependencies = [
 
 [[package]]
 name = "simple-agents-router"
-version = "0.1.7"
+version = "0.1.8"
 dependencies = [
  "async-trait",
  "rand 0.8.5",

{simple_agents_py-0.1.7 → simple_agents_py-0.1.8}/Cargo.toml

@@ -3,7 +3,7 @@ members = ["crates/*"]
 resolver = "2"
 
 [workspace.package]
-version = "0.1.7"
+version = "0.1.8"
 edition = "2021"
 rust-version = "1.75"
 authors = ["SimpleAgents Contributors"]

{simple_agents_py-0.1.7 → simple_agents_py-0.1.8}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: simple-agents-py
-Version: 0.1.7
+Version: 0.1.8
 Classifier: Programming Language :: Python :: 3
 Classifier: Programming Language :: Rust
 Classifier: License :: OSI Approved :: MIT License

{simple_agents_py-0.1.7 → simple_agents_py-0.1.8}/crates/simple-agents-py/src/lib.rs

@@ -169,39 +169,48 @@ fn build_request_with_messages(
 }
 
 fn parse_messages(messages: &Bound<'_, PyAny>) -> Result<Vec<Message>> {
-    let list: &PyList = messages.downcast().map_err(|_| {
+    let list: &Bound<'_, PyList> = messages.downcast().map_err(|_| {
         SimpleAgentsError::Config("messages must be a list of dicts".to_string())
     })?;
     let mut result = Vec::with_capacity(list.len());
 
     for (idx, item) in list.iter().enumerate() {
-        let dict: &PyDict = item
+        let dict: &Bound<'_, PyDict> = item
             .downcast()
             .map_err(|_| SimpleAgentsError::Config(format!("message[{idx}] must be a dict")))?;
 
-        let role_obj = dict.get_item("role").ok_or_else(|| {
-            SimpleAgentsError::Config(format!("message[{idx}] missing 'role'"))
-        })?;
-        let role: &str = role_obj.extract().map_err(|_| {
+        let role_obj = dict
+            .get_item("role")
+            .map_err(|_| SimpleAgentsError::Config(format!("message[{idx}] missing 'role'")))?
+            .ok_or_else(|| SimpleAgentsError::Config(format!("message[{idx}] missing 'role'")))?;
+        let role: String = role_obj.extract().map_err(|_| {
             SimpleAgentsError::Config(format!("message[{idx}].role must be a string"))
         })?;
 
-        let content_obj = dict.get_item("content").ok_or_else(|| {
-            SimpleAgentsError::Config(format!("message[{idx}] missing 'content'"))
-        })?;
-        let content: &str = content_obj.extract().map_err(|_| {
+        let content_obj = dict
+            .get_item("content")
+            .map_err(|_| SimpleAgentsError::Config(format!("message[{idx}] missing 'content'")))?
+            .ok_or_else(|| {
+                SimpleAgentsError::Config(format!("message[{idx}] missing 'content'"))
+            })?;
+        let content: String = content_obj.extract().map_err(|_| {
             SimpleAgentsError::Config(format!(
                 "message[{idx}].content must be a string"
             ))
         })?;
 
-        let mut message = match role {
-            "user" => Message::user(content),
-            "assistant" => Message::assistant(content),
-            "system" => Message::system(content),
+        let mut message = match role.as_str() {
+            "user" => Message::user(&content),
+            "assistant" => Message::assistant(&content),
+            "system" => Message::system(&content),
             "tool" => {
                 let tool_call_id = dict
                     .get_item("tool_call_id")
+                    .map_err(|_| {
+                        SimpleAgentsError::Config(format!(
+                            "message[{idx}] missing 'tool_call_id' for tool role"
+                        ))
+                    })?
                     .ok_or_else(|| {
                         SimpleAgentsError::Config(format!(
                             "message[{idx}] missing 'tool_call_id' for tool role"
@@ -213,7 +222,7 @@ fn parse_messages(messages: &Bound<'_, PyAny>) -> Result<Vec<Message>> {
                             "message[{idx}].tool_call_id must be a string"
                         ))
                     })?;
-                Message::tool(content, tool_call_id)
+                Message::tool(&content, tool_call_id)
             }
             _ => {
                 return Err(SimpleAgentsError::Config(format!(
@@ -222,7 +231,10 @@ fn parse_messages(messages: &Bound<'_, PyAny>) -> Result<Vec<Message>> {
             }
         };
 
-        if let Some(name_obj) = dict.get_item("name") {
+        if let Some(name_obj) = dict
+            .get_item("name")
+            .map_err(|_| SimpleAgentsError::Config(format!("message[{idx}].name must be a string")))?
+        {
             if !name_obj.is_none() {
                 let name: String = name_obj.extract().map_err(|_| {
                     SimpleAgentsError::Config(format!("message[{idx}].name must be a string"))

{simple_agents_py-0.1.7 → simple_agents_py-0.1.8}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "maturin"
 
 [project]
 name = "simple-agents-py"
-version = "0.1.7"
+version = "0.1.8"
 description = "Python bindings for SimpleAgents"
 readme = "README.md"
 requires-python = ">=3.9"