Skip to content

Tags: openai/codex

Tags

rust-v0.0.2505061740

Toggle rust-v0.0.2505061740's commit message
Release 0.0.2505061740

codex-rs-132146b6d4e133d014f763a0d8dabd853f3fc0c0-1-rust-v0.0.2505061740

Toggle codex-rs-132146b6d4e133d014f763a0d8dabd853f3fc0c0-1-rust-v0.0.2505061740's commit message

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
chore: introduce codex-common crate (#843)

I started this PR because I wanted to share the `format_duration()`
utility function in `codex-rs/exec/src/event_processor.rs` with the TUI.
The question was: where to put it?

`core` should have as few dependencies as possible, so moving it there
would introduce a dependency on `chrono`, which seemed undesirable.
`core` already had this `cli` feature to deal with a similar situation
around sharing common utility functions, so I decided to:

* make `core` feature-free
* introduce `common`
* `common` can have as many "special interest" features as it needs,
each of which can declare their own deps
* the first two features of common are `cli` and `elapsed`

In practice, this meant updating a number of `Cargo.toml` files,
replacing this line:

```toml
codex-core = { path = "../core", features = ["cli"] }
```

with these:

```toml
codex-core = { path = "../core" }
codex-common = { path = "../common", features = ["cli"] }
```

Moving `format_duration()` into its own file gave it some "breathing
room" to add a unit test, so I had Codex generate some tests and new
support for durations over 1 minute.

rust-v0.0.2505021951

Toggle rust-v0.0.2505021951's commit message
Release 0.0.2505021951

codex-rs-5915a59c8290765d6097caf4074aae93a85380fa-1-rust-v0.0.2505021951

Toggle codex-rs-5915a59c8290765d6097caf4074aae93a85380fa-1-rust-v0.0.2505021951's commit message

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
feat: configurable notifications in the Rust CLI (#793)

With this change, you can specify a program that will be executed to get
notified about events generated by Codex. The notification info will be
packaged as a JSON object. The supported notification types are defined
by the `UserNotification` enum introduced in this PR. Initially, it
contains only one variant, `AgentTurnComplete`:

```rust
pub(crate) enum UserNotification {
    #[serde(rename_all = "kebab-case")]
    AgentTurnComplete {
        turn_id: String,

        /// Messages that the user sent to the agent to initiate the turn.
        input_messages: Vec<String>,

        /// The last message sent by the assistant in the turn.
        last_assistant_message: Option<String>,
    },
}
```

This is intended to support the common case when a "turn" ends, which
often means it is now your chance to give Codex further instructions.

For example, I have the following in my `~/.codex/config.toml`:

```toml
notify = ["python3", "/Users/mbolin/.codex/notify.py"]
```

I created my own custom notifier script that calls out to
[terminal-notifier](https://github.com/julienXX/terminal-notifier) to
show a desktop push notification on macOS. Contents of `notify.py`:

```python
#!/usr/bin/env python3

import json
import subprocess
import sys


def main() -> int:
    if len(sys.argv) != 2:
        print("Usage: notify.py <NOTIFICATION_JSON>")
        return 1

    try:
        notification = json.loads(sys.argv[1])
    except json.JSONDecodeError:
        return 1

    match notification_type := notification.get("type"):
        case "agent-turn-complete":
            assistant_message = notification.get("last-assistant-message")
            if assistant_message:
                title = f"Codex: {assistant_message}"
            else:
                title = "Codex: Turn Complete!"
            input_messages = notification.get("input_messages", [])
            message = " ".join(input_messages)
            title += message
        case _:
            print(f"not sending a push notification for: {notification_type}")
            return 0

    subprocess.check_output(
        [
            "terminal-notifier",
            "-title",
            title,
            "-message",
            message,
            "-group",
            "codex",
            "-ignoreDnD",
            "-activate",
            "com.googlecode.iterm2",
        ]
    )

    return 0


if __name__ == "__main__":
    sys.exit(main())
```

For reference, here are related PRs that tried to add this functionality
to the TypeScript version of the Codex CLI:

* #160
* #498

rust-v0.0.2504301327

Toggle rust-v0.0.2504301327's commit message
Release 0.0.2504301327

rust-v0.0.2504301219

Toggle rust-v0.0.2504301219's commit message
Release 0.0.2504301219

rust-v0.0.2504301132

Toggle rust-v0.0.2504301132's commit message
Release v0.0.2504301132

rust-v.0.0.2504292236

Toggle rust-v.0.0.2504292236's commit message
Release v0.0.2504292236

rust-v.0.0.2504292006

Toggle rust-v.0.0.2504292006's commit message
Release v0.0.2504292006

rust-v.0.0.2504291954

Toggle rust-v.0.0.2504291954's commit message
Release v0.0.2504291954