Commit Graph
1362 Commits
Author SHA1 Message Date
Michael Bolin f670a8267d Merge ce126481ce into sapling-pr-archive-bolinfest 2025-05-17 08:50:53 -07:00
Michael Bolin ce126481ce fix: ensure the first user message always displays after the session info 2025-05-17 08:50:46 -07:00
Michael Bolin f9143d0361 fix: do not let Tab keypress flow through to composer when used to toggle focus (#977)
One line fix from Codex!
2025-05-16 19:27:49 -07:00
Fouad Matin 2880925a44 bump(version): 0.1.2505161800 (#978)
## `0.1.2505161800`

- Sign in with chatgpt credits (#974)
- Add support for OpenAI tool type, local_shell (#961)
2025-05-16 18:18:15 -07:00
Michael Bolin 5058dde1ca Merge 25d55ac285 into sapling-pr-archive-bolinfest 2025-05-16 18:17:20 -07:00
Michael Bolin 25d55ac285 fix: do not let Tab keypress flow through to composer when used to toggle focus 2025-05-16 18:17:13 -07:00
Fouad Matin 3e19e8fd59 add: sign in with chatgpt credits (#974) 2025-05-16 17:55:08 -07:00
Trevor Creech c7312c9d52 Fix CLA link in workflow (#964)
## Summary
- fix the CLA link posted by the bot
- docs suggest using an absolute URL:
https://github.com/marketplace/actions/cla-assistant-lite
codex-rs-b5257992b06373acef8b20a4ca25ffc1b96688e2-1-rust-v0.0.2505161708
2025-05-16 17:11:57 -07:00
Michael Bolin 1dc14cefa1 fix: make codex-mini-latest the default model in the Rust TUI (#972)
It's time to make `codex-mini-latest` the new default, as this should be
an "evergreen" model pointer.

* Equivalent change in TypeScript
https://github.com/openai/codex/pull/951
* See some notes about using `codex-mini-latest` with MCP in
https://github.com/openai/codex/pull/961
2025-05-16 17:08:18 -07:00
Michael Bolin 4f313b77ec Merge 8f1aebe69c into sapling-pr-archive-bolinfest 2025-05-16 16:23:00 -07:00
Michael Bolin 8f1aebe69c fix: make codex-mini-latest the default model in the Rust TUI 2025-05-16 16:22:54 -07:00
Michael Bolin b99c58fa5b Merge 09d3b3063d into sapling-pr-archive-bolinfest 2025-05-16 16:22:50 -07:00
Michael Bolin 09d3b3063d fix: make codex-mini-latest the default model in the Rust TUI 2025-05-16 16:22:43 -07:00
Michael Bolin 7ca84087e6 feat: make it possible to toggle mouse mode in the Rust TUI (#971)
I did a bit of research to understand why I could not use my mouse to
drag to select text to copy to the clipboard in iTerm.

Apparently https://github.com/openai/codex/pull/641 to enable mousewheel
scrolling broke this functionality. It seems that, unless we put in a
bit of effort, we can have drag-to-select or scrolling, but not both.
Though if you know the trick to hold down `Option` will dragging with
the mouse in iTerm, you can probably get by with this. (I did not know
about this option prior to researching this issue.)

Nevertheless, users may still prefer to disable mouse capture
altogether, so this PR introduces:

* the ability to set `tui.disable_mouse_capture = true` in `config.toml`
to disable mouse capture
* a new command, `/toggle-mouse-mode` to toggle mouse capture
2025-05-16 16:16:50 -07:00
Michael Bolin 83f369f8e1 merge commit for archive created by Sapling 2025-05-16 16:11:19 -07:00
Michael Bolin 80546e052e feat: make it possible to toggle mouse mode in the Rust TUI 2025-05-16 16:11:13 -07:00
Michael Bolin deb4e85006 Merge 1111e304c8 into sapling-pr-archive-bolinfest 2025-05-16 16:09:38 -07:00
Michael Bolin 1111e304c8 feat: make it possible to toggle mouse mode in the Rust TUI 2025-05-16 16:09:34 -07:00
Michael Bolin b18efeadcc Merge 628e3fbb9a into sapling-pr-archive-bolinfest 2025-05-16 16:02:58 -07:00
Michael Bolin 628e3fbb9a feat: make it possible to toggle mouse mode in the Rust TUI 2025-05-16 16:02:55 -07:00
Michael Bolin 4d300d5103 Merge 04a069b938 into sapling-pr-archive-bolinfest 2025-05-16 16:01:16 -07:00
Michael Bolin 04a069b938 feat: make it possible to toggle mouse mode in the Rust TUI 2025-05-16 16:01:11 -07:00
Michael Bolin aec7895ca4 merge commit for archive created by Sapling 2025-05-16 15:58:53 -07:00
Michael Bolin eaef3fe1ed feat: make it possible to toggle mouse mode in the Rust TUI 2025-05-16 15:58:47 -07:00
Michael Bolin 2cb18c4f6c Merge 29583cfea6 into sapling-pr-archive-bolinfest 2025-05-16 15:52:10 -07:00
Michael Bolin 29583cfea6 feat: make it possible to toggle mouse mode in the Rust TUI 2025-05-16 15:52:00 -07:00
Michael Bolin 67ac8ef605 fix: use text other than 'TODO' as test example (#969)
I casually `rg TODO` to look for TODOs, so the use of TODO in a sample
string in test output was throwing things off.
2025-05-16 14:51:03 -07:00
Michael Bolin b0e2cb8273 Merge 6195de0c5e into sapling-pr-archive-bolinfest 2025-05-16 14:45:12 -07:00
Michael Bolin 6195de0c5e fix: use text other than 'TODO' as test example 2025-05-16 14:45:03 -07:00
Michael Bolin f48dd99f22 feat: add support for OpenAI tool type, local_shell (#961)
The new `codex-mini-latest` model expects a new tool with `{"type":
"local_shell"}`. Its contract is similar to the existing `function` tool
with `"name": "shell"`, so this takes the `local_shell` tool call into
`ExecParams` and sends it through the existing
`handle_container_exec_with_params()` code path.

This also adds the following logic when adding the default set of tools
to a request:

```rust
let default_tools = if self.model.starts_with("codex") {
    &DEFAULT_CODEX_MODEL_TOOLS
} else {
    &DEFAULT_TOOLS
};
```

That is, if the model name starts with `"codex"`, we add `{"type":
"local_shell"}` to the list of tools; otherwise, we add the
aforementioned `shell` tool.

To test this, I ran the TUI with `-m codex-mini-latest` and verified
that it used the `local_shell` tool. Though I also had some entries in
`[mcp_servers]` in my personal `config.toml`. The `codex-mini-latest`
model seemed eager to try the tools from the MCP servers first, so I
have personally commented them out for now, so keep an eye out if you're
testing `codex-mini-latest`!

Perhaps we should include more details with `{"type": "local_shell"}` or
update the following:


https://github.com/openai/codex/blob/fd0b1b020818dfe8aaf7eb68425f09e86ab1b819/codex-rs/core/prompt.md

For reference, the corresponding change in the TypeScript CLI is
https://github.com/openai/codex/pull/951.
2025-05-16 14:38:08 -07:00
Michael Bolin f7e0c51a20 Merge 2040d45947 into sapling-pr-archive-bolinfest 2025-05-16 14:18:26 -07:00
Michael Bolin 2040d45947 feat: add support for OpenAI tool type, local_shell 2025-05-16 14:18:19 -07:00
Michael Bolin dfd54e1433 chore: refactor handle_function_call() into smaller functions (#965)
Overall, `codex.rs` is still far too large, but at least there's less
indenting now that things have been moved into smaller functions.

This will also make it easier to introduce the `local_shell` tool in
https://github.com/openai/codex/pull/961.

---
[//]: # (BEGIN SAPLING FOOTER)
Stack created with [Sapling](https://sapling-scm.com). Best reviewed
with [ReviewStack](https://reviewstack.dev/openai/codex/pull/965).
* #961
* __->__ #965
2025-05-16 14:17:10 -07:00
Michael Bolin 6cf503fefb Merge 0698fb23d3 into sapling-pr-archive-bolinfest 2025-05-16 14:08:44 -07:00
Michael Bolin 0698fb23d3 feat: add support for OpenAI tool type, local_shell 2025-05-16 14:08:38 -07:00
Michael Bolin ef8ff3b1b4 chore: refactor handle_function_call() into smaller functions 2025-05-16 14:08:38 -07:00
Michael Bolin 9739820366 fix: remove file named ">" in the codex-cli folder (#968)
This file appears to have been accidentally added in
https://github.com/openai/codex/pull/912. Its presence makes the repo
impossible to clone on Windows.
2025-05-16 14:08:23 -07:00
Michael Bolin f8c94926fd Merge 26200e4942 into sapling-pr-archive-bolinfest 2025-05-16 14:07:11 -07:00
Michael Bolin 26200e4942 fix: remove file named ">" in the codex-cli folder 2025-05-16 14:07:03 -07:00
Michael Bolin 486c85bb0d Merge 0347022c20 into sapling-pr-archive-bolinfest 2025-05-16 13:57:06 -07:00
Michael Bolin 0347022c20 chore: refactor handle_function_call() into smaller functions 2025-05-16 13:56:59 -07:00
Michael Bolin 114cbd691b merge commit for archive created by Sapling 2025-05-16 13:53:28 -07:00
Michael Bolin e2b08eee8b feat: add support for OpenAI tool type, local_shell 2025-05-16 13:53:20 -07:00
Michael Bolin 7af8e04bdc Merge 493203d976 into sapling-pr-archive-bolinfest 2025-05-16 13:47:44 -07:00
Michael Bolin e0291c89f6 chore: refactor handle_function_call() into smaller functions 2025-05-16 13:47:36 -07:00
Michael Bolin 493203d976 feat: add support for OpenAI tool type, local_shell 2025-05-16 13:47:36 -07:00
Michael Bolin 4e66934828 Merge 8470d27eeb into sapling-pr-archive-bolinfest 2025-05-16 13:44:33 -07:00
Michael Bolin 8470d27eeb feat: add support for OpenAI tool type, local_shell 2025-05-16 13:44:27 -07:00
Fouad Matin fd0b1b0208 bump(version): 0.1.2505161243 (#967)
## `0.1.2505161243`

- Sign in with chatgpt (#963)
- Session history viewer (#912)
- Apply patch issue when using different cwd (#942)
- Diff command for filenames with special characters (#954)
2025-05-16 12:46:52 -07:00
Michael Bolin f09f62e1e2 merge commit for archive created by Sapling 2025-05-16 12:43:55 -07:00