Skip to content

fix(cua): show call titles and align app icons - #13307

Merged
Yash-Singh1 merged 1 commit into
t3code/codex-turn-mappingfrom
fix-cua-tool-call-title-icon
Sep 23, 2026
Merged

Yash-Singh1 merged 1 commit into
t3code/codex-turn-mappingfrom
fix-cua-tool-call-title-icon

Conversation

@Yash-Singh1

Copy link
Copy Markdown
Collaborator

CUA tool calls displayed Cua_repl.js instead of the action title supplied to the tool. Native app icons also looked too small and sat high in the activity row.

Use the CUA title argument when projecting new calls and when rendering existing saved calls in web and mobile. Center tool icons in the row and size native app icons to match the surrounding text. Desktop uses the web timeline.

Before

CUA tool call before

After

CUA tool call after

Checks

  • 307 focused tests passed across the server, web, and mobile.
  • Server, web, mobile, and shared package type checks passed.
  • Targeted formatting passed. Targeted lint passed with existing warnings outside these edits.

Model: GPT-6. Harness: Codex.

@github-actions github-actions Bot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:M 30-99 changed lines (additions + deletions). labels Sep 23, 2026
@macroscopeapp

macroscopeapp Bot commented Sep 23, 2026

Copy link
Copy Markdown
Contributor

Approvability

Verdict: Approved at 9d2f934

Macroscope's review found this PR approvable — This is a narrowly scoped CUA display fix with focused tests: it populates and renders action titles while preserving existing fallbacks, and adjusts icon alignment and sizing. It does not alter tool execution, product defaults, schemas, or sensitive infrastructure.

You can add or adjust custom eligibility rules. Learn more.

@github-actions

Copy link
Copy Markdown
Contributor

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

ℹ️ No successful main baseline artifact is available yet. This run establishes the initial measurement.

Provider Metric Main baseline This PR Impact PR ceiling
Codex Total thread wire — 4.9 KiB — 6.8 KiB ✅
Codex Thread snapshot wire — 3.7 KiB — 4.9 KiB ✅
Codex Live turn WebSocket wire — 1.2 KiB — 2.0 KiB ✅
Codex Live turn WebSocket decoded — 20.4 KiB — 29.3 KiB ✅
Codex Live turn messages — 2 — 8 ✅
Claude Total thread wire — 4.9 KiB — 6.8 KiB ✅
Claude Thread snapshot wire — 3.7 KiB — 4.9 KiB ✅
Claude Live turn WebSocket wire — 1.2 KiB — 2.0 KiB ✅
Claude Live turn WebSocket decoded — 20.8 KiB — 29.3 KiB ✅
Claude Live turn messages — 2 — 8 ✅

Baseline: unavailable · PR result: 9d2f934 · Source CI: success

Scenario and decoded snapshot size

10 historical turns, 5 command tools per turn, 878.9 KiB retained MCP result per historical turn, and a 1.05 MiB retained result in the measured turn.

  • Codex decoded thread snapshot: 106.1 KiB
  • Claude decoded thread snapshot: 106.4 KiB

Updated in place by a trusted workflow. PR artifacts are strictly validated and never executed.

@Yash-Singh1
Yash-Singh1 merged commit a36d656 into t3code/codex-turn-mapping Sep 23, 2026
24 checks passed
@Yash-Singh1
Yash-Singh1 deleted the fix-cua-tool-call-title-icon branch September 23, 2026 20:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:M 30-99 changed lines (additions + deletions). vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant