Skip to content

chore: sync upstream main - #127

Merged
tarik02 merged 39 commits into
mainfrom
sync-upstream-main-20260812
Aug 12, 2026
Merged

chore: sync upstream main#127
tarik02 merged 39 commits into
mainfrom
sync-upstream-main-20260812

Conversation

@tarik02

@tarik02 tarik02 commented Aug 12, 2026

Copy link
Copy Markdown
Collaborator

What Changed

  • Merged 38 upstream-only commits through pingdotgg/t3code@5a8461480.
  • Added upstream pull request browsing and review for GitHub, GitLab, Azure DevOps, and Bitbucket across the server and web client.
  • Brought in mobile live-follow, thread action menus, pending-input layout updates, usage fixes, Open VSX theme search, and the latest desktop and web fixes.
  • Preserved fork goal controls, progressive history, canary updater notes, GitHub-hosted release jobs, web release archives, and fork LegendList behavior across overlapping files.

Why

Actualizes the fork to current upstream while keeping intentional fork behavior where the same release, mobile, and right-panel code changed.

UI Changes

This includes upstream UI changes. Screenshots were not captured because browser verification was not requested.

Checklist

  • This PR is small and focused
  • I explained what changed and why
  • I included before/after screenshots for any UI changes
  • I included a video for animation/interaction changes

Made with Codex using GPT-5.6.

Bil0000 and others added 30 commits August 10, 2026 17:47
Co-authored-by: Julius Marminge <julius0216@outlook.com>
Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
Co-authored-by: maria <maria@kuuro.net>
Co-authored-by: maria-rcks <254055478+maria-rcks@users.noreply.github.com>
Co-authored-by: codex <codex@users.noreply.github.com>
Co-authored-by: tsouth89 <tsouth89@users.noreply.github.com>
Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
…JSON body (#5072)

Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Co-authored-by: Illia Panasenko <hello@ipanasenko.me>
Co-authored-by: t3-code[bot] <269035359+t3-code[bot]@users.noreply.github.com>
…shift+click and show shortcut in tooltip (#5994)

Co-authored-by: shivam <91240327+shivamhwp@users.noreply.github.com>
…ontext menu (#5574)

Co-authored-by: shivam <91240327+shivamhwp@users.noreply.github.com>
Co-authored-by: codex <codex@users.noreply.github.com>
Co-authored-by: Thuong Tin <thuongtin@gmail.com>
Co-authored-by: Kapish14 <kapishnarang01@gmail.com>
Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
Co-authored-by: codex <codex@users.noreply.github.com>
Co-authored-by: codex <codex@users.noreply.github.com>
…/ (#6189)

Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
…the sidebar footer (#6031)

Co-authored-by: shivam <91240327+shivamhwp@users.noreply.github.com>
Co-authored-by: Rodrigo Brechard <rodrigo@clubtidy.fr>
Co-authored-by: Claude Opus 5 (1M context) <noreply@anthropic.com>
@github-actions github-actions Bot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:XXL 📱 Native Change Changes the native fingerprint; merging blocks production OTAs until a new store build ships. labels Aug 12, 2026

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 8b4233a1fc

ℹ️ 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".

Comment on lines +846 to +847
needs: [preflight, build, build_web_dist, publish_cli]
if: ${{ !failure() && !cancelled() && needs.preflight.result == 'success' && needs.build.result == 'success' && needs.build_web_dist.result == 'success' && needs.publish_cli.result == 'success' }}

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Remove the disabled CLI gate from releases

The publish_cli job is explicitly disabled with if: false at line 782, so its result is always skipped, never success. Requiring needs.publish_cli.result == 'success' therefore skips the GitHub release job for every stable, nightly, and canary run, preventing release artifacts from being published; keep the release independent of this fork-disabled job or explicitly accept a skipped result.

AGENTS.md reference: AGENTS.md:L7-L7

Useful? React with 👍 / 👎.

Comment on lines +540 to +543
const handleScrollToEnd = useCallback(() => {
void Haptics.selectionAsync();
void scrollMessageToEnd({ animated: true, closeKeyboard: false }).catch(() => {
freeze.set(false);

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Jump to the live tail when newer history exists

When progressive history is displaying a segment with hasNextMessages, pressing the new scroll-to-end button only scrolls to the end of the currently loaded segment. That programmatic scroll re-enables follow and hides the button even though newer pages—and the live tail—still exist; use the route's existing showLatestMessages command when hasNextMessages is true, or keep the affordance visible until the live window is restored.

AGENTS.md reference: AGENTS.md:L75-L75

Useful? React with 👍 / 👎.

Comment on lines +6371 to +6372
activeThreadPr?.number === activeRightPanelSurface.number &&
threadRepository === activeRightPanelSurface.repository

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Include the project when identifying the thread's PR

If another checked-out project or source-control host has the same repository path and PR number as the active thread's PR, this comparison incorrectly selects context="thread" because it ignores activeRightPanelSurface.projectId. PullRequestDetailPanel then hides its checkout control on the assumption that this foreign PR is already under the thread's feet; require the surface project ID to match activeProject.id as well.

Useful? React with 👍 / 👎.

@tarik02
tarik02 merged commit 74de36b into main Aug 12, 2026
11 checks passed
@github-actions

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 10.5 KiB 10.7 KiB +214 B (+2.0%) 15.1 KiB
Codex Thread snapshot wire 4.8 KiB 4.8 KiB +9 B (+0.2%) 7.3 KiB
Codex Live turn WebSocket wire 5.7 KiB 5.9 KiB +205 B (+3.5%) 7.8 KiB
Codex Live turn WebSocket decoded 50.3 KiB 51.7 KiB +1.5 KiB (+2.9%) 66.4 KiB
Codex Live turn messages 10 11 +1 (+10.0%) 21
Claude Total thread wire 10.5 KiB 10.5 KiB −44 B (−0.4%) 15.1 KiB
Claude Thread snapshot wire 4.8 KiB 4.8 KiB +8 B (+0.2%) 7.3 KiB
Claude Live turn WebSocket wire 5.7 KiB 5.7 KiB −52 B (−0.9%) 7.8 KiB
Claude Live turn WebSocket decoded 51.1 KiB 51.1 KiB −44 B (−0.1%) 66.4 KiB
Claude Live turn messages 10 9 −1 (−10.0%) 21

Baseline: a8311ef · PR result: 8b4233a · 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: 76.2 KiB
  • Claude decoded thread snapshot: 76.6 KiB

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

📱 Native Change Changes the native fingerprint; merging blocks production OTAs until a new store build ships. size:XXL 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.