chore: sync public mirror from internal - #1304
Conversation
|
Resolved 0 review-bot thread(s) on this generated public mirror PR. Review of mirrored code belongs on the mono source PR. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 0abefce62c
ℹ️ About Codex in GitHub
Codex has been enabled to automatically 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 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
|
Resolved 1 review-bot thread(s) on this generated public mirror PR (chatgpt-codex-connector). Review of mirrored code belongs on the mono source PR. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: ea207e893f
ℹ️ About Codex in GitHub
Codex has been enabled to automatically 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 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
|
Resolved 2 review-bot thread(s) on this generated public mirror PR (chatgpt-codex-connector). Review of mirrored code belongs on the mono source PR. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: ef9ef026eb
ℹ️ About Codex in GitHub
Codex has been enabled to automatically 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 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| input_tokens, | ||
| output_tokens, | ||
| cache_read_input_tokens: cache_read_tokens.unwrap_or_default(), | ||
| cache_creation_input_tokens: cache_creation_tokens.unwrap_or_default(), |
There was a problem hiding this comment.
Include cached input in the token budget
When AiRsModel receives OpenAI-compatible or Google usage, StreamEvent::input_tokens is already the uncached prompt count (see packages/ai-rs/src/openai.rs and packages/ai-rs/src/google.rs), but this mapping retains that value as Usage.input_tokens and stores cache counts separately. Usage::tokens() only sums input_tokens + output_tokens, so a largely cached request can consume far more than Budget::max_tokens without exhausting it, contrary to the new Usage.input_tokens contract. Fold the cache buckets into the budget total (or make input_tokens total prompt tokens) before emitting the chunk.
Useful? React with 👍 / 👎.
Summary
dx-corp/monodx-corp/codeas a generated public mirror of the private source of truthc30dc2f64dce3300144e0e0d0dc97eb97b8744eeea207e893f2111f9aee0b41265d5ec1240dbc5e117file(s) to copy/update and0stale file(s) to delete0sync-holdlabel to this PR; the sync workflow skips every push while it is setSource-of-truth status
Projection: deixic-code
Source: dx-corp/mono@c30dc2f64dce3300144e0e0d0dc97eb97b8744ee
Prior destination base: 358d47f
Content SHA-256: fc3edcce20cf8d94ea1cd95fce135752740dc77c7bd79715e242599e385cb806
Result: drift_detected; 17 changed, 0 deleted
Destination-owned content is preserved. Destination CI is a separate health signal.
copy/update .repository-projection.json
copy/update packages/dex-host-rs/src/model.rs
copy/update packages/runtime-rs/src/agent/credential_store.rs
copy/update packages/runtime-rs/src/agent/native.rs
copy/update packages/runtime-rs/src/agent/native/provider_payload.rs
copy/update packages/runtime-rs/src/agent/native/tests.rs
copy/update vendor/dex-loop/src/budget.rs
copy/update vendor/dex-loop/src/compaction.rs
copy/update vendor/dex-loop/src/engine.rs
copy/update vendor/dex-loop/src/event.rs
copy/update vendor/dex-loop/src/lib.rs
copy/update vendor/dex-loop/src/ports.rs
copy/update vendor/dex-loop/tests/client_tool_replay.rs
copy/update vendor/dex-loop/tests/prefetch.rs
copy/update vendor/dex-loop/tests/read_wave_progress.rs
copy/update vendor/dex-loop/tests/scenarios.rs
copy/update vendor/dex-loop/tests/support/mod.rs
Drift sample
Public-only commits since last generated sync
Validation
Test Plan
maestro-sync-public-release-mirrorworkflow inpublic-treemoderequire-internal-prconfirms internal source PR lineageStaged Rollout
dx-corp/mono@c30dc2f64dce3300144e0e0d0dc97eb97b8744ee, including existing hidden/evaluation surfaces, and keeps public package parity behind the established public-source-provenance gate.Supersedes