Michael Bolin
7f999fa955
merge commit for archive created by Sapling
2025-08-07 17:21:33 -07:00
Michael Bolin
a473f7a75f
fix: default to credits from ChatGPT auth, when possible
2025-08-07 17:21:28 -07:00
Michael Bolin
0265ea0486
merge commit for archive created by Sapling
2025-08-07 17:18:13 -07:00
Michael Bolin
08bfb12600
fix: default to credits from ChatGPT auth, when possible
2025-08-07 17:18:06 -07:00
Michael Bolin
9992db09f4
merge commit for archive created by Sapling
2025-08-07 17:17:08 -07:00
Michael Bolin
5fc9ecc601
fix: default to credits from ChatGPT auth, when possible
2025-08-07 17:17:02 -07:00
Michael Bolin
295abf3e51
chore: change CodexAuth::from_api_key() to take &str instead of String ( #1970 )
...
Good practice and simplifies some of the call sites.
---
[//]: # (BEGIN SAPLING FOOTER)
Stack created with [Sapling](https://sapling-scm.com ). Best reviewed
with [ReviewStack](https://reviewstack.dev/openai/codex/pull/1970 ).
* #1971
* __->__ #1970
* #1966
* #1965
* #1962
2025-08-07 16:55:33 -07:00
Michael Bolin
52926cd1b2
merge commit for archive created by Sapling
2025-08-07 16:50:09 -07:00
Michael Bolin
3b12816016
chore: change CodexAuth::from_api_key() to take &str instead of String
2025-08-07 16:50:02 -07:00
Michael Bolin
8d2411c001
fix: default to credits from ChatGPT auth, when possible
2025-08-07 16:50:02 -07:00
Michael Bolin
b991c04f86
chore: move top-level load_auth() to CodexAuth::from_codex_home() ( #1966 )
...
There are two valid ways to create an instance of `CodexAuth`:
`from_api_key()` and `from_codex_home()`. Now both are static methods of
`CodexAuth` and are listed first in the implementation.
---
[//]: # (BEGIN SAPLING FOOTER)
Stack created with [Sapling](https://sapling-scm.com ). Best reviewed
with [ReviewStack](https://reviewstack.dev/openai/codex/pull/1966 ).
* #1971
* #1970
* __->__ #1966
* #1965
* #1962
2025-08-07 16:49:37 -07:00
Michael Bolin
02c9c2ecad
chore: make CodexAuth::api_key a private field ( #1965 )
...
Force callers to access this information via `get_token()` rather than
messing with it directly.
---
[//]: # (BEGIN SAPLING FOOTER)
Stack created with [Sapling](https://sapling-scm.com ). Best reviewed
with [ReviewStack](https://reviewstack.dev/openai/codex/pull/1965 ).
* #1971
* #1970
* #1966
* __->__ #1965
* #1962
2025-08-07 16:40:01 -07:00
Michael Bolin
5e2799a3a6
Merge 77e14c5019 into sapling-pr-archive-bolinfest
2025-08-07 16:33:53 -07:00
Michael Bolin
77e14c5019
fix: default to credits from ChatGPT auth, when possible
2025-08-07 16:33:45 -07:00
Michael Bolin
c0b22b9c78
chore: change CodexAuth::from_api_key() to take &str instead of String
2025-08-07 16:33:45 -07:00
Michael Bolin
5c1bfa660c
chore: move top-level load_auth() to CodexAuth::from_codex_home()
2025-08-07 16:33:45 -07:00
Michael Bolin
df37e7e33d
chore: make CodexAuth::api_key a private field
2025-08-07 16:33:45 -07:00
Michael Bolin
db76f32888
chore: rename CodexAuth::new() to create_dummy_codex_auth_for_testing() because it is not for general consumption ( #1962 )
...
`CodexAuth::new()` was the first method listed in `CodexAuth`, but it is
only meant to be used by tests. Rename it to
`create_dummy_chatgpt_auth_for_testing()` and move it to the end of the
implementation.
---
[//]: # (BEGIN SAPLING FOOTER)
Stack created with [Sapling](https://sapling-scm.com ). Best reviewed
with [ReviewStack](https://reviewstack.dev/openai/codex/pull/1962 ).
* #1971
* #1970
* #1966
* #1965
* __->__ #1962
2025-08-07 16:33:29 -07:00
Michael Bolin
81a549c685
merge commit for archive created by Sapling
2025-08-07 16:21:41 -07:00
Michael Bolin
dcab238fd1
fix: default to credits from ChatGPT auth, when possible
2025-08-07 16:21:37 -07:00
Michael Bolin
280c98ed79
merge commit for archive created by Sapling
2025-08-07 16:20:54 -07:00
Michael Bolin
9cd3a45f3e
fix: default to credits from ChatGPT auth, when possible
2025-08-07 16:20:48 -07:00
Michael Bolin
8631d73b4e
Merge 4f2436f271 into sapling-pr-archive-bolinfest
2025-08-07 16:16:25 -07:00
Michael Bolin
4f2436f271
fix: default to credits from ChatGPT auth, when possible
2025-08-07 16:16:21 -07:00
Michael Bolin
c45329f1bc
merge commit for archive created by Sapling
2025-08-07 15:53:57 -07:00
Michael Bolin
be1efb2575
fix: default to credits from ChatGPT auth, when possible
2025-08-07 15:53:51 -07:00
Michael Bolin
ae5a78f686
Merge f464733e2d into sapling-pr-archive-bolinfest
2025-08-07 15:49:08 -07:00
Michael Bolin
f464733e2d
fix: default to credits from ChatGPT auth, when possible
2025-08-07 15:48:58 -07:00
Michael Bolin
5dc812d738
chore: change CodexAuth::from_api_key() to take &str instead of String
2025-08-07 15:43:11 -07:00
Dylan
548466df09
[client] Tune retries and backoff ( #1956 )
...
## Summary
10 is a bit excessive 😅 Also updates our backoff factor to space out
requests further.
2025-08-07 15:23:31 -07:00
Michael Bolin
1e9bd5af5c
merge commit for archive created by Sapling
2025-08-07 15:10:24 -07:00
Michael Bolin
4366fa898a
chore: move top-level load_auth() to CodexAuth::from_codex_home()
2025-08-07 15:09:00 -07:00
Michael Bolin
333c49bfe7
chore: make CodexAuth::api_key a private field
2025-08-07 15:09:00 -07:00
Michael Bolin
da391eb8f4
chore: rename CodexAuth::new() to create_dummy_codex_auth_for_testing() because it is not for general consumption
2025-08-07 15:09:00 -07:00
Michael Bolin
33c1f6edbe
Merge b343138c6a into sapling-pr-archive-bolinfest
2025-08-07 14:56:25 -07:00
Michael Bolin
b343138c6a
chore: move top-level load_auth() to CodexAuth::from_codex_home()
2025-08-07 14:56:17 -07:00
Michael Bolin
f4ab56b8ee
merge commit for archive created by Sapling
2025-08-07 14:50:22 -07:00
Michael Bolin
922d516f58
chore: make CodexAuth::api_key a private field
2025-08-07 14:50:14 -07:00
Michael Bolin
fde883d740
merge commit for archive created by Sapling
2025-08-07 14:49:26 -07:00
Michael Bolin
7c1871e2df
chore: rename CodexAuth::new() to create_dummy_codex_auth_for_testing() because it is not for general consumption
2025-08-07 14:49:14 -07:00
Michael Bolin
0a0e482141
chore: make CodexAuth::api_key a private field
2025-08-07 14:49:14 -07:00
Michael Bolin
29450fbdca
merge commit for archive created by Sapling
2025-08-07 14:47:14 -07:00
Michael Bolin
fe2d786d5b
chore: make CodexAuth::api_key a private field
2025-08-07 14:47:05 -07:00
Michael Bolin
1f18d6bcbf
chore: rename CodexAuth::new() to create_dummy_codex_auth_for_testing() because it is not for general consumption
2025-08-07 14:47:05 -07:00
Michael Bolin
d2c804a956
merge commit for archive created by Sapling
2025-08-07 14:45:59 -07:00
Michael Bolin
5c19391ea6
chore: make CodexAuth::api_key a private field
2025-08-07 14:45:46 -07:00
Michael Bolin
fc329350b9
chore: rename CodexAuth::new() to create_dummy_codex_auth_for_testing() because it is not for general consumption
2025-08-07 14:45:46 -07:00
Michael Bolin
5d0d56b320
Merge 6db81d73d2 into sapling-pr-archive-bolinfest
2025-08-07 14:19:46 -07:00
Michael Bolin
6db81d73d2
chore: rename CodexAuth::new() to create_dummy_codex_auth_for_testing() because it is not for general consumption
2025-08-07 14:19:41 -07:00
Michael Bolin
7d67159587
fix: public load_auth() fn always called with include_env_var=true ( #1961 )
...
Apparently `include_env_var=false` was only used for testing, so clean
up the API a little to make that clear.
---
[//]: # (BEGIN SAPLING FOOTER)
Stack created with [Sapling](https://sapling-scm.com ). Best reviewed
with [ReviewStack](https://reviewstack.dev/openai/codex/pull/1961 ).
* #1962
* __->__ #1961
2025-08-07 14:19:30 -07:00
Michael Bolin