chore: sync upstream pingdotgg/t3code#73
Conversation
… computer switching (pingdotgg#3687) Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com>
Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
|
@codex review |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: b78cd75989
ℹ️ About Codex in GitHub
Codex has been enabled to automatically 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 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 01bceeaefd
ℹ️ About Codex in GitHub
Codex has been enabled to automatically 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 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
2 participants
Nightly upstream sync of
pingdotgg/t3code→agriffiths-bots/t3code.Synced commits (2)
Cherry-picked
4ac094fef..upstream/main(6009720844f053e048e18ca1abe4f429559ab9fc):753b12198fix(mobile): combined test branch — scroll, back-swipe, thread lists, computer switching (fix(mobile): combined test branch — scroll, back-swipe, thread lists, computer switching pingdotgg/t3code#3687)b78cd7598Add repo-root favicon.svg so t3 code shows its own icon (Add repo-root favicon.svg so t3 code shows its own icon pingdotgg/t3code#3683)Conflict resolution
Conflicted-file set (the only surface where this merge could introduce a bug; everything else applied cleanly from upstream):
apps/mobile/src/features/threads/thread-navigation-groups.test.ts(modify/delete) — upstream deleted thethread-navigation-groups.tsmodule and itsThreadNavigationDrawer.tsx; the fork had only upstream-origin content in this test (last touched by a prior sync). Resolved by deleting the orphaned test (module it imported no longer exists). Verified no remaining references tothread-navigation-groupsin the tree.pnpm-lock.yaml(content) — upstream bumpedreact-native-keyboard-controller1.21.6 → 1.21.13. Took upstream's lockfile and regenerated withpnpm install --lockfile-only(pnpm 10.24.0) to reconcile against the mergedpackage.jsonset.Verification (all green on this branch)
vp run typecheck— 0 errors (suggestions only)vp check— 0 errors, 12 pre-existing warnings in files not touched by this syncvp test run— 616 test files / 4974 tests passed, 7 skipped, exit 0Merge gated by CI + the fork's upstream-sync policy (CI-only). Upstream-origin Codex findings outside the conflicted-file set are accepted per policy.