Conversation
…s omitted - Ensure `message_delta` is emitted on tool call usage chunks and stream completion even if `finish_reason` is missing or null. - Add `terminalOpenAIFinishReason` fallback to map missing finish reasons to standard stop reasons. - Consolidate content block finalization into `finalizeOpenAIAnthropicContentBlocks` and delta emission into `emitAnthropicMessageDelta`. Closes: router-for-me#5308
- Fall back to array index in tool calls accumulator when the `index` field is missing. - Ensure parallel tool calls without explicit indices are accumulated properly during streaming. Closes: router-for-me#5058
…sification (router-for-me#5313) * feat(executor): measure effective TTFT with protocol-aware token classification - Introduce protocol-aware streaming token classification under internal/runtime/executor/helps - Implement responses_ttft_helpers.go for OpenAI Responses / Codex WebSocket & SSE streaming - Filter out container metadata, rate limits, handshake headers, and tool execution outputs - Implement dual-track firstPacketDuration fallback for abnormal / zero-token stream terminations - Add TrackHTTPClientRoundTripOnly to bypass false TTFT triggers on first response body reads - Optimize UsageReporter.ObserveTokenEvent hot path with fast-path RLock short-circuiting (0 B/op) - Align terminal event completions across SSE, WebSocket streaming, and non-streaming executions - Add EnsurePublished metric recording fallback for successful terminal turns lacking usage blocks - Add stub helpers with comprehensive specifications for Chat, Claude, and Gemini protocols * fix(executor): normalize SSE response.done and terminate incomplete WS streams * fix(usage): capture first-packet fallback on initial body reads with TrackHTTPClientRoundTripOnly
- Inline local `$ref` pointers and strip `$defs` and `definitions` from tool parameter schemas for Moonshot compatibility. - Ensure tool parameter root objects declare an explicit `type: "object"`. Closes: router-for-me#5316
|
This pull request targeted The base branch has been automatically changed to |
…/upstream-minimal-quota
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 5bbd8de32f
ℹ️ 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".
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 0fafc126e0
ℹ️ 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".
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 5f4068ca5a
ℹ️ 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".
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 076694fba0
ℹ️ 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".
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 225c586908
ℹ️ 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".
3 participants
Summary
Expose one explicitly public, bounded
plugin_quotaobject on management auth-file entries so native plugins can publish subscription quota data without adding provider-specific host code or exposing their private metadata.The host deliberately does not understand provider names, contract versions, quota windows, currencies, or freshness. Plugins and consumers own those semantics, so future subscription providers can use the same seam without another CLIProxyAPI patch.
Why this matters beyond one plugin
CLIProxyAPI's plugin system can add authentication and model providers, but subscription quota information currently has no safe path into management clients. A plugin author must otherwise choose between losing quota visibility, requesting a provider-specific host patch, or exposing a broader metadata channel.
This change creates a small provider-neutral capability:
plugin_quotakey;The accompanying CPA Manager Plus draft demonstrates an independent generic consumer, but the transport is not tied to that UI or to Cursor.
Scope
Auth.Metadata["plugin_quota"].plugin_quotawhen dynamic-plugin auth responses are decoded, including when metadata is subsequently sent back through the plugin boundary; all existing numeric metadata retains its historicalfloat64representation.tokens,latest_tokens, andperiod_tokensnames are accepted only for finite numeric counter values.plugin_quota; sibling plugin metadata remains private.No route, refresh lifecycle, provider branch, plugin ABI, auth availability, built-in quota, or generic metadata projection is added.
Upstream sync
devat5208aec7on 2026-09-05.Review surface
Verification
go test ./internal/api/handlers/managementgo test ./...go build -o <temporary-output> ./cmd/serverSecurity notes
The field is an explicit public namespace, not a general metadata escape hatch. Invalid, oversized, non-object, non-JSON, credential-shaped, or unsafe payloads are omitted without changing the auth entry's availability. The 16 KiB limit is enforced again after HTML sanitization, and token-counter exemptions are value-type aware. Rejection diagnostics contain the auth ID only and never include payload data.
Related