codex-python 0.2.14__tar.gz → 0.2.16__tar.gz

{codex_python-0.2.14 → codex_python-0.2.16}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: codex-python
-Version: 0.2.14
+Version: 0.2.16
 Classifier: Programming Language :: Python :: 3
 Classifier: Programming Language :: Python :: 3 :: Only
 Classifier: Programming Language :: Python :: 3.13
@@ -128,3 +128,14 @@ Links
 - uv: https://docs.astral.sh/uv/
 - maturin: https://www.maturin.rs/
 
+## GitHub Workflow: Autonomous Review
+
+This repo includes a workflow that runs autonomous code review on pull requests and can
+apply edits on demand via a slash command.
+
+- Automatic review runs on PR open, synchronize, reopen, and when marked ready for review.
+- To request autonomous edits on a PR, comment `/codex` in either a PR thread or a
+  review comment. The workflow listens for `/codex` and triggers the "act" job.
+
+See `.github/workflows/codex-autoreview.yml` for configuration.
+

{codex_python-0.2.14 → codex_python-0.2.16}/README.md

@@ -108,3 +108,14 @@ Links
 - Codex repo: https://github.com/openai/codex
 - uv: https://docs.astral.sh/uv/
 - maturin: https://www.maturin.rs/
+
+## GitHub Workflow: Autonomous Review
+
+This repo includes a workflow that runs autonomous code review on pull requests and can
+apply edits on demand via a slash command.
+
+- Automatic review runs on PR open, synchronize, reopen, and when marked ready for review.
+- To request autonomous edits on a PR, comment `/codex` in either a PR thread or a
+  review comment. The workflow listens for `/codex` and triggers the "act" job.
+
+See `.github/workflows/codex-autoreview.yml` for configuration.

{codex_python-0.2.14 → codex_python-0.2.16}/codex/__init__.py

@@ -31,4 +31,4 @@ __all__ = [
 ]
 
 # Package version. Kept in sync with Cargo.toml via CI before builds.
-__version__ = "0.2.14"
+__version__ = "0.2.16"

{codex_python-0.2.14 → codex_python-0.2.16}/codex/api.py

@@ -16,10 +16,13 @@ class CodexError(Exception):
 class CodexNativeError(CodexError):
     """Raised when the native extension is not available or fails."""
 
-    def __init__(self) -> None:
+    def __init__(self, message: str | None = None) -> None:
         super().__init__(
-            "codex_native extension not installed or failed to run. "
-            "Run `make dev-native` or ensure native wheels are installed."
+            message
+            or (
+                "codex_native extension not installed or failed to run. "
+                "Run `make dev-native` or ensure native wheels are installed."
+            )
         )
 
 
@@ -31,7 +34,14 @@ class Conversation:
 
     def __iter__(self) -> Iterator[Event]:
         """Yield `Event` objects from the native stream."""
-        for item in self._stream:
+        iterator = iter(self._stream)
+        while True:
+            try:
+                item = next(iterator)
+            except StopIteration:
+                return
+            except RuntimeError as exc:  # surfaced from native iterator
+                raise CodexNativeError(str(exc)) from exc
             try:
                 yield Event.model_validate(item)
             except Exception:
@@ -76,7 +86,7 @@ class CodexClient:
             )
             return Conversation(_stream=stream)
         except RuntimeError as e:
-            raise CodexNativeError() from e
+            raise CodexNativeError(str(e)) from e
 
 
 def run_exec(
@@ -97,7 +107,7 @@ def run_exec(
             load_default_config=load_default_config,
         )
     except RuntimeError as e:
-        raise CodexNativeError() from e
+        raise CodexNativeError(str(e)) from e
 
     out: list[Event] = []
     for item in events:

{codex_python-0.2.14 → codex_python-0.2.16}/codex/config.py

@@ -218,7 +218,8 @@ class CodexConfig(BaseModel):
     project_doc_max_bytes: int | None = None
     profile: str | None = None
     profiles: dict[str, ProfileConfig] | None = None
-    tools: ToolsConfig | None = None
+    # Accept either a structured ToolsConfig or a plain mapping (common in tests/CLI overrides)
+    tools: ToolsConfig | dict[str, Any] | None = None
     projects: dict[str, ProjectConfig] | None = None
 
     # Experimental / internal

{codex_python-0.2.14 → codex_python-0.2.16}/crates/codex_native/Cargo.lock

@@ -311,7 +311,7 @@ checksum = "b94f61472cee1439c0b966b47e3aca9ae07e45d070759512cd390ea2bebc6675"
 [[package]]
 name = "codex-apply-patch"
 version = "0.0.0"
-source = "git+https://github.com/openai/codex?branch=main#fdf4a686463f13b56ee83f807bccbe3457542564"
+source = "git+https://github.com/openai/codex?branch=main#a8026d3846486e790827b26e9abf13e7f96e48bf"
 dependencies = [
  "anyhow",
  "once_cell",
@@ -324,7 +324,7 @@ dependencies = [
 [[package]]
 name = "codex-core"
 version = "0.0.0"
-source = "git+https://github.com/openai/codex?branch=main#fdf4a686463f13b56ee83f807bccbe3457542564"
+source = "git+https://github.com/openai/codex?branch=main#a8026d3846486e790827b26e9abf13e7f96e48bf"
 dependencies = [
  "anyhow",
  "askama",
@@ -374,7 +374,7 @@ dependencies = [
 [[package]]
 name = "codex-file-search"
 version = "0.0.0"
-source = "git+https://github.com/openai/codex?branch=main#fdf4a686463f13b56ee83f807bccbe3457542564"
+source = "git+https://github.com/openai/codex?branch=main#a8026d3846486e790827b26e9abf13e7f96e48bf"
 dependencies = [
  "anyhow",
  "clap",
@@ -388,7 +388,7 @@ dependencies = [
 [[package]]
 name = "codex-mcp-client"
 version = "0.0.0"
-source = "git+https://github.com/openai/codex?branch=main#fdf4a686463f13b56ee83f807bccbe3457542564"
+source = "git+https://github.com/openai/codex?branch=main#a8026d3846486e790827b26e9abf13e7f96e48bf"
 dependencies = [
  "anyhow",
  "mcp-types",
@@ -402,7 +402,7 @@ dependencies = [
 [[package]]
 name = "codex-protocol"
 version = "0.0.0"
-source = "git+https://github.com/openai/codex?branch=main#fdf4a686463f13b56ee83f807bccbe3457542564"
+source = "git+https://github.com/openai/codex?branch=main#a8026d3846486e790827b26e9abf13e7f96e48bf"
 dependencies = [
  "base64",
  "icu_decimal",
@@ -422,12 +422,13 @@ dependencies = [
 
 [[package]]
 name = "codex_native"
-version = "0.2.14"
+version = "0.2.16"
 dependencies = [
  "anyhow",
  "clap",
  "codex-core",
  "codex-protocol",
+ "dotenvy",
  "openssl-sys",
  "pathdiff",
  "pyo3",
@@ -601,6 +602,12 @@ dependencies = [
  "syn",
 ]
 
+[[package]]
+name = "dotenvy"
+version = "0.15.7"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "1aaf95b3e5c8f23aa320147307562d361db0ae0d51242340f558153b4eb2439b"
+
 [[package]]
 name = "downcast-rs"
 version = "1.2.1"
@@ -885,7 +892,7 @@ dependencies = [
  "cfg-if",
  "libc",
  "r-efi",
- "wasi 0.14.5+wasi-0.2.4",
+ "wasi 0.14.7+wasi-0.2.4",
 ]
 
 [[package]]
@@ -919,7 +926,7 @@ dependencies = [
  "futures-core",
  "futures-sink",
  "http",
- "indexmap 2.11.1",
+ "indexmap 2.11.3",
  "slab",
  "tokio",
  "tokio-util",
@@ -1064,9 +1071,9 @@ dependencies = [
 
 [[package]]
 name = "hyper-util"
-version = "0.1.16"
+version = "0.1.17"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "8d9b05277c7e8da2c93a568989bb6207bef0112e8d17df7a6eda4a3cf143bc5e"
+checksum = "3c6995591a8f1380fcb4ba966a252a4b29188d51d2b89e3a252f5305be65aea8"
 dependencies = [
  "base64",
  "bytes",
@@ -1299,13 +1306,14 @@ dependencies = [
 
 [[package]]
 name = "indexmap"
-version = "2.11.1"
+version = "2.11.3"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "206a8042aec68fa4a62e8d3f7aa4ceb508177d9324faf261e1959e495b7a1921"
+checksum = "92119844f513ffa41556430369ab02c295a3578af21cf945caa3e9e0c2481ac3"
 dependencies = [
  "equivalent",
  "hashbrown 0.15.5",
  "serde",
+ "serde_core",
 ]
 
 [[package]]
@@ -1448,7 +1456,7 @@ dependencies = [
 [[package]]
 name = "mcp-types"
 version = "0.0.0"
-source = "git+https://github.com/openai/codex?branch=main#fdf4a686463f13b56ee83f807bccbe3457542564"
+source = "git+https://github.com/openai/codex?branch=main#a8026d3846486e790827b26e9abf13e7f96e48bf"
 dependencies = [
  "serde",
  "serde_json",
@@ -1748,12 +1756,12 @@ checksum = "7edddbd0b52d732b21ad9a5fab5c704c14cd949e5e9a1ec5929a24fded1b904c"
 
 [[package]]
 name = "plist"
-version = "1.7.4"
+version = "1.8.0"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "3af6b589e163c5a788fab00ce0c0366f6efbb9959c2f9874b224936af7fce7e1"
+checksum = "740ebea15c5d1428f910cd1a5f52cebf8d25006245ed8ade92702f4943d91e07"
 dependencies = [
  "base64",
- "indexmap 2.11.1",
+ "indexmap 2.11.3",
  "quick-xml",
  "serde",
  "time",
@@ -2122,9 +2130,9 @@ dependencies = [
 
 [[package]]
 name = "rustls-webpki"
-version = "0.103.5"
+version = "0.103.6"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "b5a37813727b78798e53c2bec3f5e8fe12a6d6f8389bf9ca7802add4c9905ad8"
+checksum = "8572f3c2cb9934231157b45499fc41e1f58c589fdfb81a844ba873265e80f8eb"
 dependencies = [
  "ring",
  "rustls-pki-types",
@@ -2225,9 +2233,9 @@ dependencies = [
 
 [[package]]
 name = "serde"
-version = "1.0.223"
+version = "1.0.225"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "a505d71960adde88e293da5cb5eda57093379f64e61cf77bf0e6a63af07a7bac"
+checksum = "fd6c24dee235d0da097043389623fb913daddf92c76e9f5a1db88607a0bcbd1d"
 dependencies = [
  "serde_core",
  "serde_derive",
@@ -2235,18 +2243,18 @@ dependencies = [
 
 [[package]]
 name = "serde_core"
-version = "1.0.223"
+version = "1.0.225"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "20f57cbd357666aa7b3ac84a90b4ea328f1d4ddb6772b430caa5d9e1309bb9e9"
+checksum = "659356f9a0cb1e529b24c01e43ad2bdf520ec4ceaf83047b83ddcc2251f96383"
 dependencies = [
  "serde_derive",
 ]
 
 [[package]]
 name = "serde_derive"
-version = "1.0.223"
+version = "1.0.225"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "3d428d07faf17e306e699ec1e91996e5a165ba5d6bce5b5155173e91a8a01a56"
+checksum = "0ea936adf78b1f766949a4977b91d2f5595825bd6ec079aa9543ad2685fc4516"
 dependencies = [
  "proc-macro2",
  "quote",
@@ -2259,7 +2267,7 @@ version = "1.0.145"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "402a6f66d8c709116cf22f558eab210f5a50187f702eb4d7e5ef38d9a7f1c79c"
 dependencies = [
- "indexmap 2.11.1",
+ "indexmap 2.11.3",
  "itoa",
  "memchr",
  "ryu",
@@ -2269,11 +2277,11 @@ dependencies = [
 
 [[package]]
 name = "serde_spanned"
-version = "1.0.0"
+version = "1.0.1"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "40734c41988f7306bb04f0ecf60ec0f3f1caa34290e4e8ea471dcd3346483b83"
+checksum = "2789234a13a53fc4be1b51ea1bab45a3c338bdb884862a257d10e5a74ae009e6"
 dependencies = [
- "serde",
+ "serde_core",
 ]
 
 [[package]]
@@ -2298,7 +2306,7 @@ dependencies = [
  "chrono",
  "hex",
  "indexmap 1.9.3",
- "indexmap 2.11.1",
+ "indexmap 2.11.3",
  "schemars 0.9.0",
  "schemars 1.0.4",
  "serde",
@@ -2698,12 +2706,12 @@ dependencies = [
 
 [[package]]
 name = "toml"
-version = "0.9.5"
+version = "0.9.6"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "75129e1dc5000bfbaa9fee9d1b21f974f9fbad9daec557a521ee6e080825f6e8"
+checksum = "ae2a4cf385da23d1d53bc15cdfa5c2109e93d8d362393c801e87da2f72f0e201"
 dependencies = [
- "indexmap 2.11.1",
- "serde",
+ "indexmap 2.11.3",
+ "serde_core",
  "serde_spanned",
  "toml_datetime",
  "toml_parser",
@@ -2713,20 +2721,20 @@ dependencies = [
 
 [[package]]
 name = "toml_datetime"
-version = "0.7.0"
+version = "0.7.1"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "bade1c3e902f58d73d3f294cd7f20391c1cb2fbcb643b73566bc773971df91e3"
+checksum = "a197c0ec7d131bfc6f7e82c8442ba1595aeab35da7adbf05b6b73cd06a16b6be"
 dependencies = [
- "serde",
+ "serde_core",
 ]
 
 [[package]]
 name = "toml_edit"
-version = "0.23.4"
+version = "0.23.5"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "7211ff1b8f0d3adae1663b7da9ffe396eabe1ca25f0b0bee42b0da29a9ddce93"
+checksum = "c2ad0b7ae9cfeef5605163839cb9221f453399f15cfb5c10be9885fcf56611f9"
 dependencies = [
- "indexmap 2.11.1",
+ "indexmap 2.11.3",
  "toml_datetime",
  "toml_parser",
  "toml_writer",
@@ -3032,18 +3040,18 @@ checksum = "ccf3ec651a847eb01de73ccad15eb7d99f80485de043efb2f370cd654f4ea44b"
 
 [[package]]
 name = "wasi"
-version = "0.14.5+wasi-0.2.4"
+version = "0.14.7+wasi-0.2.4"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "a4494f6290a82f5fe584817a676a34b9d6763e8d9d18204009fb31dceca98fd4"
+checksum = "883478de20367e224c0090af9cf5f9fa85bed63a95c1abf3afc5c083ebc06e8c"
 dependencies = [
  "wasip2",
 ]
 
 [[package]]
 name = "wasip2"
-version = "1.0.0+wasi-0.2.4"
+version = "1.0.1+wasi-0.2.4"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "03fa2761397e5bd52002cd7e73110c71af2109aca4e521a9f40473fe685b0a24"
+checksum = "0562428422c63773dad2c345a1882263bbf4d65cf3f42e90921f787ef5ad58e7"
 dependencies = [
  "wit-bindgen",
 ]
@@ -3477,9 +3485,9 @@ checksum = "d135d17ab770252ad95e9a872d365cf3090e3be864a34ab46f48555993efc904"
 
 [[package]]
 name = "wit-bindgen"
-version = "0.45.1"
+version = "0.46.0"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "5c573471f125075647d03df72e026074b7203790d41351cd6edc96f46bcccd36"
+checksum = "f17a85883d4e6d00e8a97c586de764dabcc06133f7f1d55dce5cdc070ad7fe59"
 
 [[package]]
 name = "writeable"

{codex_python-0.2.14 → codex_python-0.2.16}/crates/codex_native/Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "codex_native"
-version = "0.2.14"
+version = "0.2.16"
 edition = "2021"
 
 [lib]
@@ -22,6 +22,7 @@ tracing = "0.1"
 clap = { version = "4", features = ["derive"] }
 which = "6"
 pathdiff = "0.2"
+dotenvy = "0.15.7"
 
 # Upstream Codex crates from the monorepo (use git deps; pin to main for now)
 # Pin to a specific commit of the upstream Codex monorepo to avoid breaking API changes

{codex_python-0.2.14 → codex_python-0.2.16}/crates/codex_native/src/bin/protocol_schema.rs

@@ -15,7 +15,11 @@ struct Args {
     ts_out: PathBuf,
 
     /// Output directory for the generated JSON Schema (.generated/schema by default)
-    #[arg(long = "schema-out", value_name = "DIR", default_value = ".generated/schema")]
+    #[arg(
+        long = "schema-out",
+        value_name = "DIR",
+        default_value = ".generated/schema"
+    )]
     schema_out: PathBuf,
 
     /// Optional path to the Prettier binary to format emitted TS (forwarded to upstream generator)
@@ -63,9 +67,12 @@ fn ensure_ts_generated(ts_out: &Path, prettier: Option<&Path>) -> Result<()> {
             .arg(relative_path_from(&monorepo, ts_out)?)
             .current_dir(&monorepo);
         if let Some(bin) = prettier {
-            cmd.arg("--prettier").arg(relative_path_from(&monorepo, bin)?);
+            cmd.arg("--prettier")
+                .arg(relative_path_from(&monorepo, bin)?);
         }
-        let status = cmd.status().context("Failed to run codex-protocol-ts generator")?;
+        let status = cmd
+            .status()
+            .context("Failed to run codex-protocol-ts generator")?;
         if status.success() {
             return Ok(());
         }

{codex_python-0.2.14 → codex_python-0.2.16}/crates/codex_native/src/lib.rs

@@ -1,16 +1,16 @@
 use anyhow::{Context, Result};
 use codex_core::config::{find_codex_home, Config, ConfigOverrides, ConfigToml};
 use codex_core::protocol::{EventMsg, InputItem};
-use codex_core::{AuthManager, ConversationManager};
+use codex_core::{AuthManager, CodexAuth, ConversationManager};
 // use of SandboxMode is handled within core::config; not needed here
 use pyo3::prelude::*;
 use pyo3::types::{PyDict, PyFloat, PyList, PyModule, PyString};
 use serde_json::Value as JsonValue;
-use toml::value::Value as TomlValue;
-use toml::value::Table as TomlTable;
 use std::path::PathBuf;
 use std::sync::{mpsc, Arc, Mutex};
 use std::thread;
+use toml::value::Table as TomlTable;
+use toml::value::Value as TomlValue;
 
 #[pyfunction]
 fn run_exec_collect(
@@ -38,9 +38,12 @@ fn run_exec_collect(
 }
 
 async fn run_exec_impl(prompt: String, config: Config) -> Result<Vec<JsonValue>> {
-    let conversation_manager = ConversationManager::new(AuthManager::shared(
-        config.codex_home.clone(),
-    ));
+    let conversation_manager = match std::env::var("OPENAI_API_KEY") {
+        Ok(val) if !val.trim().is_empty() => {
+            ConversationManager::with_auth(CodexAuth::from_api_key(&val))
+        }
+        _ => ConversationManager::new(AuthManager::shared(config.codex_home.clone())),
+    };
     let new_conv = conversation_manager.new_conversation(config).await?;
     let conversation = new_conv.conversation.clone();
 
@@ -61,13 +64,15 @@ async fn run_exec_impl(prompt: String, config: Config) -> Result<Vec<JsonValue>>
                 out.push(serde_json::to_value(&ev)?);
                 if is_complete {
                     // Ask the agent to shutdown; collect remaining events
-                    let _ = conversation.submit(codex_core::protocol::Op::Shutdown).await;
+                    let _ = conversation
+                        .submit(codex_core::protocol::Op::Shutdown)
+                        .await;
                 }
                 if is_shutdown {
                     break;
                 }
             }
-            Err(_) => break,
+            Err(err) => return Err(err.into()),
         }
     }
     Ok(out)
@@ -79,7 +84,7 @@ fn to_py<E: std::fmt::Display>(e: E) -> PyErr {
 
 #[pyclass]
 struct CodexEventStream {
-    rx: Arc<Mutex<mpsc::Receiver<JsonValue>>>,
+    rx: Arc<Mutex<mpsc::Receiver<Result<JsonValue, String>>>>,
 }
 
 #[pymethods]
@@ -88,12 +93,13 @@ impl CodexEventStream {
         slf
     }
 
-    fn __next__(&mut self, py: Python<'_>) -> Option<Py<PyAny>> {
+    fn __next__(&mut self, py: Python<'_>) -> PyResult<Option<Py<PyAny>>> {
         // Run the blocking recv without holding the GIL
-        let res = py.detach(|| self.rx.lock().ok()?.recv().ok());
+        let res = py.detach(|| self.rx.lock().ok().and_then(|rx| rx.recv().ok()));
         match res {
-            Some(v) => json_to_py(py, &v).ok(),
-            None => None,
+            Some(Ok(v)) => Ok(Some(json_to_py(py, &v)?)),
+            Some(Err(msg)) => Err(pyo3::exceptions::PyRuntimeError::new_err(msg)),
+            None => Ok(None),
         }
     }
 }
@@ -104,15 +110,18 @@ fn start_exec_stream(
     config_overrides: Option<Bound<'_, PyDict>>,
     load_default_config: bool,
 ) -> PyResult<CodexEventStream> {
-    let (tx, rx) = mpsc::channel::<JsonValue>();
+    let (tx, rx) = mpsc::channel::<Result<JsonValue, String>>();
 
     // Build a pure-Rust Config on the Python thread
     let config = build_config(config_overrides, load_default_config).map_err(to_py)?;
     let prompt_clone = prompt.clone();
 
     thread::spawn(move || {
-        if let Err(e) = run_exec_stream_impl(prompt_clone, config, tx) {
-            eprintln!("codex_native stream error: {e}");
+        let tx_for_impl = tx.clone();
+        if let Err(e) = run_exec_stream_impl(prompt_clone, config, tx_for_impl) {
+            let msg = e.to_string();
+            let _ = tx.send(Err(msg.clone()));
+            eprintln!("codex_native stream error: {msg}");
         }
     });
     Ok(CodexEventStream {
@@ -169,10 +178,11 @@ fn json_to_py(py: Python<'_>, v: &JsonValue) -> PyResult<Py<PyAny>> {
     Ok(obj)
 }
 
-fn build_config(
-    overrides: Option<Bound<'_, PyDict>>,
-    load_default_config: bool,
-) -> Result<Config> {
+fn build_config(overrides: Option<Bound<'_, PyDict>>, load_default_config: bool) -> Result<Config> {
+    // Match CLI behavior: import env vars from ~/.codex/.env (if present)
+    // before reading config/auth so OPENAI_API_KEY and friends are visible.
+    // Security: filter out CODEX_* variables just like the CLI does.
+    load_dotenv();
     let mut overrides_struct = ConfigOverrides::default();
     let mut cli_overrides: Vec<(String, TomlValue)> = Vec::new();
 
@@ -272,7 +282,10 @@ fn build_config(
 
     if load_default_config {
         // Start from built-in defaults and apply CLI + typed overrides.
-        Ok(Config::load_with_cli_overrides(cli_overrides, overrides_struct)?)
+        Ok(Config::load_with_cli_overrides(
+            cli_overrides,
+            overrides_struct,
+        )?)
     } else {
         // Do NOT read any on-disk config. Build a TOML value purely from CLI-style overrides
         // and then apply the strongly-typed overrides on top. We still resolve CODEX_HOME to
@@ -287,7 +300,38 @@ fn build_config(
 
         let root_value = TomlValue::Table(base_tbl);
         let cfg: ConfigToml = root_value.try_into().map_err(|e| anyhow::anyhow!(e))?;
-        Ok(Config::load_from_base_config_with_overrides(cfg, overrides_struct, codex_home)?)
+        Ok(Config::load_from_base_config_with_overrides(
+            cfg,
+            overrides_struct,
+            codex_home,
+        )?)
+    }
+}
+
+const ILLEGAL_ENV_VAR_PREFIX: &str = "CODEX_";
+
+/// Load env vars from ~/.codex/.env, filtering out any keys that start with
+/// CODEX_ (reserved for internal use). This mirrors the behavior in the
+/// `codex-arg0` crate used by the CLI so python users get the same DX.
+fn load_dotenv() {
+    if let Ok(codex_home) = find_codex_home() {
+        let env_path = codex_home.join(".env");
+        if let Ok(iter) = dotenvy::from_path_iter(env_path) {
+            set_filtered(iter);
+        }
+    }
+}
+
+/// Helper to set vars from a dotenvy iterator while filtering out `CODEX_` keys.
+fn set_filtered<I>(iter: I)
+where
+    I: IntoIterator<Item = Result<(String, String), dotenvy::Error>>,
+{
+    for (key, value) in iter.into_iter().flatten() {
+        if !key.to_ascii_uppercase().starts_with(ILLEGAL_ENV_VAR_PREFIX) {
+            // Safe to modify env here – we do it up front before we spawn runtimes/threads.
+            unsafe { std::env::set_var(&key, &value) };
+        }
     }
 }
 
@@ -394,12 +438,19 @@ fn insert_parts(current: &mut TomlTable, parts: &[&str], val: TomlValue) {
     }
 }
 
-fn run_exec_stream_impl(prompt: String, config: Config, tx: mpsc::Sender<JsonValue>) -> Result<()> {
+fn run_exec_stream_impl(
+    prompt: String,
+    config: Config,
+    tx: mpsc::Sender<Result<JsonValue, String>>,
+) -> Result<()> {
     let rt = tokio::runtime::Runtime::new()?;
     rt.block_on(async move {
-        let conversation_manager = ConversationManager::new(AuthManager::shared(
-            config.codex_home.clone(),
-        ));
+        let conversation_manager = match std::env::var("OPENAI_API_KEY") {
+            Ok(val) if !val.trim().is_empty() => {
+                ConversationManager::with_auth(CodexAuth::from_api_key(&val))
+            }
+            _ => ConversationManager::new(AuthManager::shared(config.codex_home.clone())),
+        };
         let new_conv = conversation_manager.new_conversation(config).await?;
         let conversation = new_conv.conversation.clone();
 
@@ -415,15 +466,20 @@ fn run_exec_stream_impl(prompt: String, config: Config, tx: mpsc::Sender<JsonVal
                 Ok(ev) => {
                     let is_shutdown = matches!(ev.msg, EventMsg::ShutdownComplete);
                     let is_complete = matches!(ev.msg, EventMsg::TaskComplete(_));
-                    let _ = tx.send(serde_json::to_value(&ev)?);
+                    let event_json = serde_json::to_value(&ev)?;
+                    if tx.send(Ok(event_json)).is_err() {
+                        break;
+                    }
                     if is_complete {
-                        let _ = conversation.submit(codex_core::protocol::Op::Shutdown).await;
+                        let _ = conversation
+                            .submit(codex_core::protocol::Op::Shutdown)
+                            .await;
                     }
                     if is_shutdown {
                         break;
                     }
                 }
-                Err(_) => break,
+                Err(err) => return Err(err.into()),
             }
         }
         Ok::<(), anyhow::Error>(())

{codex_python-0.2.14 → codex_python-0.2.16}/pyproject.toml

@@ -64,3 +64,9 @@ disallow_untyped_defs = true
 no_implicit_optional = true
 check_untyped_defs = true
 ignore_missing_imports = true
+exclude = [
+  "^codex-proj/",
+  "^\\.generated/",
+  "^crates/.*/target/",
+  "^dist/",
+]