Add model availability NUX tooltips#13021
Conversation
|
@codex review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 6bd76b27aa
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
@codex review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 4d4897c47b
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| let mut edits = vec![ConfigEdit::ClearPath { | ||
| segments: vec!["tui".to_string(), "model_availability_nux".to_string()], | ||
| }]; |
There was a problem hiding this comment.
Avoid rewriting the full NUX map from stale state
model_availability_nux_count_edits clears tui.model_availability_nux and rewrites every key. Combined with startup code that builds updates from an in-memory snapshot, concurrent Codex launches can clobber each other's increments (lost update). This breaks the 4-exposure cap by regressing or dropping per-model counters.
Useful? React with 👍 / 👎.
|
@codex can you add e2e integration test? |
Summary
Testing
|
Uh oh!
There was an error while loading. Please reload this page.