[codex] clarify web search action labels#23
Merged
Conversation
Lay the protocol + sizing groundwork for the composer Context gauge: ContextBreakdown / AiContextUpdateMessage on the wire, and SessionPromptLayerSizes + emptyPromptLayerSizes() so the Pi-SDK path can report per-layer char budgets without re-running layer builders. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
In-composer circular gauge plus floating popover with per-category token usage (System prompt, System tools, MCP tools, Skills, Memory files, Messages, Autocompact buffer, Free space). Pi-SDK sessions report the full split; harness sessions report Messages + Free space. Wires breakdown emission on session_created, every turn_end, model switch, connector refresh, conversation-context load, and surface change. Calibrates char/4 estimates against the model's reported input_tokens via a pre-turn snapshot captured at turn_start, with the resulting scale factor persisted in PersistedSession so resumed sessions keep their calibration. Codex harness emits per-turn last.inputTokens (not the cumulative total) capped at the model context window. Replaces the old memory-count ContextIndicator badge; the "View memory details" link in the popover footer keeps the side panel reachable from the composer. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
anton:web_searchbefore choosing labels, favicons, metadata, and citation source handling.Root Cause
Search tools can arrive at the desktop renderer with provider or MCP prefixes, so the previous exact
web_search/web_researchlabel cases were skipped and the UI fell back to generic tool names without the query target.Impact
Users can now see what the agent searched for directly in collapsed action rows, for example
Web Search "NVIDIA Vera Rubin roadmap", making web lookup activity less ambiguous.Validation
git diff --checkpnpm --filter @anton/desktop typecheckNote: the typecheck emits existing engine warnings for packages that want Node >=22 while this environment is on Node v20.18.1, but the desktop typecheck passed.