chore(native): switch to Python::detach for blocking section; simplify bool conversion; remove deprecation warnings where possible

Author: gersmann

.gitignore

@@ -51,6 +51,11 @@ uv.lock
 # Ignore new markdown files by default
 *.md
 
+# Rust build outputs
+target/
+crates/**/target/
+crates/**/Cargo.lock
+
 # Local environment files
 .env
 .env.*

crates/codex_native/src/lib.rs

@@ -4,7 +4,7 @@ use codex_core::protocol::{AskForApproval, EventMsg, InputItem};
 use codex_core::{AuthManager, ConversationManager};
 use codex_protocol::config_types::SandboxMode;
 use pyo3::prelude::*;
-use pyo3::types::{PyDict, PyFloat, PyList, PyModule, PyString, PyBoolMethods};
+use pyo3::types::{PyDict, PyFloat, PyList, PyModule, PyString};
 use serde_json::Value as JsonValue;
 use std::path::PathBuf;
 use std::sync::{mpsc, Arc, Mutex};
@@ -111,7 +111,8 @@ impl CodexEventStream {
     fn __iter__(slf: PyRefMut<'_, Self>) -> PyRefMut<'_, Self> { slf }
 
     fn __next__(&mut self, py: Python<'_>) -> Option<Py<PyAny>> {
-        let res = py.allow_threads(|| self.rx.lock().ok()?.recv().ok());
+        // Run the blocking recv without holding the GIL
+        let res = py.detach(|| self.rx.lock().ok()?.recv().ok());
         match res {
             Some(v) => json_to_py(py, &v).ok(),
             None => None,
@@ -142,12 +143,12 @@ fn codex_native(_py: Python<'_>, m: &Bound<PyModule>) -> PyResult<()> {
     Ok(())
 }
 
-fn json_to_py(py: Python<'_>, v: &JsonValue) -> PyResult<PyObject> {
+fn json_to_py(py: Python<'_>, v: &JsonValue) -> PyResult<Py<PyAny>> {
     let obj = match v {
         JsonValue::Null => py.None().into_pyobject(py)?.unbind().into_any(),
         JsonValue::Bool(b) => {
-            let iu: i64 = if *b { 1 } else { 0 };
-            iu.into_pyobject(py)?.unbind().into_any()
+            let i: i64 = if *b { 1 } else { 0 };
+            i.into_pyobject(py)?.unbind().into_any()
         }
         JsonValue::Number(n) => {
             if let Some(i) = n.as_i64() {