Commit Graph
1582 Commits
Author SHA1 Message Date
Michael Bolin 5c714eef19 Merge f6a0a3afe8 into sapling-pr-archive-bolinfest 2025-05-30 13:49:06 -07:00
Michael Bolin f6a0a3afe8 fix: chat completions API now also passes tools along 2025-05-30 13:48:46 -07:00
Michael Bolin 20e4866870 fix: introduce create_tools_json() and share it with chat_completions.rs 2025-05-30 13:48:39 -07:00
Michael Bolin 187a54944a Merge ca2c197a85 into sapling-pr-archive-bolinfest 2025-05-30 13:13:23 -07:00
Michael Bolin ca2c197a85 fix: chat completions API now also passes tools along 2025-05-30 13:13:11 -07:00
Michael Bolin e207f20f64 fix: add extra debugging to GitHub Action (#1173)
https://github.com/openai/codex/actions/runs/15352839832/job/43205041563
appeared to fail around `postComment()`, but I don't see the output from
`fail()` in the logs. Adding a bit more info.
2025-05-30 11:16:30 -07:00
Michael Bolin d586261d12 Merge 354ceaf365 into sapling-pr-archive-bolinfest 2025-05-30 11:15:37 -07:00
Michael Bolin 354ceaf365 fix: add extra debugging to GitHub Action 2025-05-30 11:15:32 -07:00
Michael Bolin f2598c9132 Merge 534d998e31 into sapling-pr-archive-bolinfest 2025-05-30 11:15:26 -07:00
Michael Bolin 534d998e31 fix: add extra debugging to GitHub Action 2025-05-30 11:15:19 -07:00
Michael Bolin 0f40ef5a10 fix: missed a step in #1171 for codex.yml (#1172)
Missed in my copy/paste.
2025-05-30 11:04:41 -07:00
Michael Bolin 865072b8b0 merge commit for archive created by Sapling 2025-05-30 11:04:14 -07:00
Michael Bolin 7c6e2fc86c fix: missed a step in #1171 for codex.yml 2025-05-30 11:04:05 -07:00
Michael Bolin 8676185389 fix: update outdated repo setup in codex.yml (#1171)
We should do some work to share the setup logic across `codex.yml`,
`ci.yml`, and `rust-ci.yml`.
2025-05-30 10:58:57 -07:00
Michael Bolin a2c8119d7e merge commit for archive created by Sapling 2025-05-30 10:58:00 -07:00
Michael Bolin faaac85967 fix: update outdated repo setup in codex.yml 2025-05-30 10:57:50 -07:00
Michael Bolin baa92f37e0 feat: initial import of experimental GitHub Action (#1170)
This is a first cut at a GitHub Action that lets you define prompt
templates in `.md` files under `.github/codex/labels` that will run
Codex with the associated prompt when the label is added to a GitHub
pull request.

For example, this PR includes these files:

```
.github/codex/labels/codex-attempt.md
.github/codex/labels/codex-code-review.md
.github/codex/labels/codex-investigate-issue.md
```

And the new `.github/workflows/codex.yml` workflow declares the
following triggers:

```yaml
on:
  issues:
    types: [opened, labeled]
  pull_request:
    branches: [main]
    types: [labeled]
```

as well as the following expression to gate the action:

```
jobs:
  codex:
    if: |
      (github.event_name == 'issues' && (
        (github.event.action == 'labeled' && (github.event.label.name == 'codex-attempt' || github.event.label.name == 'codex-investigate-issue'))
      )) ||
      (github.event_name == 'pull_request' && github.event.action == 'labeled' && github.event.label.name == 'codex-code-review')
```

Note the "actor" who added the label must have write access to the repo
for the action to take effect.

After adding a label, the action will "ack" the request by replacing the
original label (e.g., `codex-review`) with an `-in-progress` suffix
(e.g., `codex-review-in-progress`). When it is finished, it will swap
the `-in-progress` label with a `-completed` one (e.g.,
`codex-review-completed`).

Users of the action are responsible for providing an `OPENAI_API_KEY`
and making it available as a secret to the action.
2025-05-30 10:55:28 -07:00
Michael Bolin a90acbbcaa merge commit for archive created by Sapling 2025-05-30 10:49:48 -07:00
Michael Bolin 0bac2fcbc3 feat: initial import of experimental GitHub Action 2025-05-30 10:49:41 -07:00
Michael Bolin e7eed45dbc merge commit for archive created by Sapling 2025-05-30 10:43:45 -07:00
Michael Bolin 78f116e2f9 feat: initial import of experimental GitHub Action 2025-05-30 10:43:40 -07:00
Michael Bolin df67b60f83 merge commit for archive created by Sapling 2025-05-30 10:40:48 -07:00
Michael Bolin 54869c1503 feat: initial import of experimental GitHub Action 2025-05-30 10:40:40 -07:00
Michael Bolin b0d1bfacd3 merge commit for archive created by Sapling 2025-05-30 10:40:18 -07:00
Michael Bolin 6be57c201d feat: initial import of experimental GitHub Action 2025-05-30 10:40:12 -07:00
Michael Bolin 95a2760021 Merge 7da44b1143 into sapling-pr-archive-bolinfest 2025-05-30 10:38:25 -07:00
Michael Bolin 7da44b1143 feat: initial import of experimental GitHub Action 2025-05-30 10:38:18 -07:00
Michael Bolin a0239c3cd6 fix: enable set positional-arguments in justfile (#1169)
The way these definitions worked before, they did not handle quoted args
with spaces properly.

For example, if you had `/tmp/test-just/printlen.py` as:

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

import sys

print(len(sys.argv))
```

and your `justfile` was:

```
printlen *args:
    /tmp/test-just/printlen.py {{args}}
```

Then:

```shell
$ just printlen foo bar
3
$ just printlen 'foo bar'
3
```

which is not what we want: `'foo bar'` should be treated as one
argument.

The fix is to use
[positional-arguments](https://github.com/casey/just/blob/515e806b5121a4696113ef15b5f0b12e69854570/README.md#L1131):

```
set positional-arguments

printlen *args:
    /tmp/test-just/printlen.py "$@"
```
2025-05-30 09:11:53 -07:00
Michael Bolin 056bf5f32d Merge 91cc3f642f into sapling-pr-archive-bolinfest 2025-05-30 09:06:24 -07:00
Michael Bolin 91cc3f642f fix: enable set positional-arguments in justfile 2025-05-30 09:06:18 -07:00
Michael Bolin 1dafac9f15 Merge 3931f3f91e into sapling-pr-archive-bolinfest 2025-05-30 09:06:10 -07:00
Michael Bolin 3931f3f91e fix: enable set positional-arguments in justfile 2025-05-30 09:06:03 -07:00
Michael Bolin c7048da53c merge commit for archive created by Sapling 2025-05-30 00:59:00 -07:00
Michael Bolin eb9b72365f fix: chat completions API now also passes tools along 2025-05-30 00:58:55 -07:00
Michael Bolin b8cc6b0cfd Merge 63943d608b into sapling-pr-archive-bolinfest 2025-05-30 00:56:33 -07:00
Michael Bolin 63943d608b fix: chat completions API now also passes tools along 2025-05-30 00:56:25 -07:00
Michael Bolin 205b5ccbd8 Merge 9db52a6cee into sapling-pr-archive-bolinfest 2025-05-29 17:07:15 -07:00
Michael Bolin 9db52a6cee fix: chat completions API now also passes tools along 2025-05-29 17:07:07 -07:00
Michael Bolin bdfa95ed31 docs: split the config-related portion of codex-rs/README.md into its own config.md file (#1165)
Also updated the overview on `codex-rs/README.md` while here.
codex-rs-dfac02b343605ce61154ab2e075ac6c38f533916-1-rust-v0.0.2505291659
2025-05-29 16:59:35 -07:00
Michael Bolin d186f6bdc8 merge commit for archive created by Sapling 2025-05-29 16:57:35 -07:00
Michael Bolin fa4dfba064 docs: split the config-related portion of codex-rs/README.md into its own config.md file 2025-05-29 16:57:29 -07:00
Fouad Matin 828e2062c2 fix(codex-rs): use codex-mini-latest as default (#1164) 2025-05-29 16:55:19 -07:00
Michael Bolin d2fda98479 Merge 59466c66d6 into sapling-pr-archive-bolinfest 2025-05-29 16:52:39 -07:00
Michael Bolin 59466c66d6 docs: split the config-related portion of codex-rs/README.md into its own config.md file 2025-05-29 16:52:35 -07:00
Michael Bolin e7b9f79ec0 merge commit for archive created by Sapling 2025-05-29 16:11:46 -07:00
Michael Bolin cc6497aa68 fix: chat completions API now also passes tools along 2025-05-29 16:11:34 -07:00
Michael Bolin 5652569acd Merge 14c28b5f21 into sapling-pr-archive-bolinfest 2025-05-29 15:59:00 -07:00
Michael Bolin 14c28b5f21 docs: split the config-related portion of codex-rs/README.md into its own config.md file 2025-05-29 15:58:54 -07:00
Michael Bolin 92957c47fb fix: update justfile to facilitate running CLIs from source and formatting source code (#1163) 2025-05-29 15:35:14 -07:00
Michael Bolin 8c1902b562 chore: update GitHub workflow for native artifacts for npm release (#1162)
Among other things, this picks up this UI treatment fix:

https://github.com/openai/codex/pull/1161
2025-05-29 15:34:06 -07:00