chore: sync upstream main - #127
Conversation
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>
Co-authored-by: Julius Marminge <julius0216@outlook.com> Co-authored-by: codex <codex@users.noreply.github.com>
Co-authored-by: t3-code[bot] <269035359+t3-code[bot]@users.noreply.github.com> Co-authored-by: maria <254055478+maria-rcks@users.noreply.github.com>
There was a problem hiding this comment.
💡 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".
| 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' }} |
There was a problem hiding this comment.
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 👍 / 👎.
| const handleScrollToEnd = useCallback(() => { | ||
| void Haptics.selectionAsync(); | ||
| void scrollMessageToEnd({ animated: true, closeKeyboard: false }).catch(() => { | ||
| freeze.set(false); |
There was a problem hiding this comment.
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 👍 / 👎.
| activeThreadPr?.number === activeRightPanelSurface.number && | ||
| threadRepository === activeRightPanelSurface.repository |
There was a problem hiding this comment.
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 👍 / 👎.
Thread transfer impact✅ Thread transfer remains within every enforced ceiling.
Baseline: Scenario and decoded snapshot size10 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.
Updated in place by a trusted workflow. PR artifacts are strictly validated and never executed. |
What Changed
pingdotgg/t3code@5a8461480.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
Made with Codex using GPT-5.6.