Skip to content

sync: integrate upstream wave 6 through 827345a07a - #257

Merged
aaditagrawal merged 2 commits into
mainfrom
sync/upstream-20260902-wave-6
Sep 2, 2026
Merged

sync: integrate upstream wave 6 through 827345a07a#257
aaditagrawal merged 2 commits into
mainfrom
sync/upstream-20260902-wave-6

Conversation

@aaditagrawal

@aaditagrawal aaditagrawal commented Sep 2, 2026

Copy link
Copy Markdown
Owner

What changed

Integrates the single upstream commit that landed while PR #256 was in CI, through upstream commit 827345a.

The provider model details affordance now uses a click-capable popover while retaining hover behavior, so model details work for mouse and touch/click interaction across fork provider instances.

Verification

  • node_modules/.bin/vp check
  • node_modules/.bin/vp run typecheck
  • Independent focused review found no actionable defects
  • Authenticated web verification opened Settings, selected the Codex Models panel, clicked Details for GPT-5.6-Sol, and confirmed the slug/capability popover remained visible
  • Clean browser console and network log

Scope

Fork-only maintenance PR targeting aaditagrawal/t3code main. Upstream remains read-only.

@aaditagrawal aaditagrawal changed the title fix(web): model info button opens its details on click (#9177) sync: integrate upstream wave 6 through 827345a07a Sep 2, 2026

Copy link
Copy Markdown
Owner Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@github-actions github-actions Bot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:S 10-29 effective changed lines (test files excluded in mixed PRs). labels Sep 2, 2026
@github-actions

github-actions Bot commented Sep 2, 2026

Copy link
Copy Markdown

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

Provider Metric Main baseline This PR Impact PR ceiling
Codex Total thread wire 13.2 KiB 13.2 KiB −80 B (−0.6%) 15.1 KiB
Codex Thread snapshot wire 6.9 KiB 6.9 KiB +7 B (+0.1%) 7.3 KiB
Codex Live turn WebSocket wire 6.3 KiB 6.3 KiB −87 B (−1.3%) 7.8 KiB
Codex Live turn WebSocket decoded 55.5 KiB 54.7 KiB −822 B (−1.4%) 66.4 KiB
Codex Live turn messages 8 8 0 (0.0%) 21
Claude Total thread wire 13.3 KiB 13.3 KiB −49 B (−0.4%) 15.1 KiB
Claude Thread snapshot wire 6.9 KiB 6.9 KiB +3 B (+0.0%) 7.3 KiB
Claude Live turn WebSocket wire 6.4 KiB 6.3 KiB −52 B (−0.8%) 7.8 KiB
Claude Live turn WebSocket decoded 56.4 KiB 56.3 KiB −44 B (−0.1%) 66.4 KiB
Claude Live turn messages 9 8 −1 (−11.1%) 21

Baseline: 5c6d36e · PR result: d1e300d · 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: 109.5 KiB
  • Claude decoded thread snapshot: 110.2 KiB

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

Copy link
Copy Markdown
Owner Author

Merge activity

  • Sep 2, 7:04 AM UTC: A user started a stack merge that includes this pull request via Graphite.

@aaditagrawal
aaditagrawal merged commit 64b96f9 into main Sep 2, 2026
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:S 10-29 effective changed lines (test files excluded in mixed PRs). 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.

2 participants