Skip to content

chore(sync): merge pingdotgg/t3code main - #326

Merged
Yarden-zamir merged 13 commits into
mainfrom
fork-sync/2026-10-24-0108
Oct 24, 2026
Merged

Yarden-zamir merged 13 commits into
mainfrom
fork-sync/2026-10-24-0108

Conversation

@Yarden-zamir

Copy link
Copy Markdown
Owner

Cause

Fork Sync run 40048214300 stopped on a merge conflict in apps/mobile/src/features/threads/ThreadFeed.tsx. Upstream pingdotgg#22585 "fix(mobile): chat and composer chips stay readable in light mode" shows t3-thread:// links in chat as thread chips, with the project icon of the thread. The fork already shows thread links as chips with its "Thread link chips" feature (server setting threadLinkChips, off by default).

Conflicts and resolution

  • ThreadFeed.tsx imports: kept upstream's useThreadChipIcons and the fork imports of forkThreadLinks and forkMessageLinks.
  • ThreadFeed.tsx markdownLinkHandlers: both sides added linkCustomization. It now calls the fork pill first, then upstream's enrichedThreadLinkCustomization. With threadLinkChips off, the fork pill returns nothing, so links look as upstream shows them. With it on, the fork pill also covers thread web addresses, app links and message links, as before.

Upstream merge

Merges pingdotgg/t3code main at 6c21798 "fix(mobile): Home and pull request lists let UIKit own their top inset (pingdotgg#24284)".

Adopted upstream pull requests

All 7 entries (pingdotgg#16653, pingdotgg#16654, pingdotgg#16655, pingdotgg#16684, pingdotgg#16695, pingdotgg#16698, pingdotgg#16719) are unchanged at their listed heads. No new merges.

Upstream changes in fork areas

22 files changed on both sides. Effect on fork features:

  • Thread link chips / Message links (mobile): upstream withChipIcon replaces a thread chip's icon with the project icon of the thread in chat messages. With messageLinks on, this also replaces the message icon of a message link when the project has an icon. I kept upstream's icon and wrote this limit in docs/fork/features.md. If the message icon matters, a fork guard in AssistantMarkdownContent and UserMessageContent can keep it.
  • Pull request filters (web): upstream adds a pullRequests.filterOptions request (contracts, ws.ts, scopes, RPC instrumentation, GitHub queries). Author and label pickers now search the host, and PullRequestFiltersMenu takes a new optionScope prop. The fork owner filter and filter chips render through PullRequestsPage only, which passes the prop. No effect. The regenerated GraphQL types equal the merged file.
  • Mobile theme: upstream adds OKLab helpers, isDarkThemeColor and themeAccentWithContrast. The fork published-theme branches in getMobileThemeColors stay. No effect.
  • Home screen (mobile): upstream lets UIKit own the top inset. The fork sidebar tabs and card layout lines stay. No effect.
  • Composer editor, ProjectFavicon, AppSymbol, MenuPopup, App.tsx, Stack.tsx, config.ts, vite.config.ts: additive upstream changes next to fork lines (font family hook, request headers, settings screens). No effect.

Checks

  • checks.py: typecheck, migration check and adopted pull request check pass.
  • vp lint on ThreadFeed.tsx: no errors.
  • fork_tests.py: 154 of 245 fork test files pass locally (desktop, machine map, packages, scripts, web). The 91 apps/server fork test files are left to Fork CI, because these tests stop the agent session on this VPS. This merge changes server code only to add the new pull request request.

Fork repair skill changes

  • fork_tests.py takes --only <prefix> and --exclude <prefix> (repeatable), with a test in test_scripts.py. SKILL.md lists them and says when to use them on the VPS. This replaces the hand-made file lists of the last two sync repairs.

yordis and others added 13 commits October 23, 2026 16:11
…build output (pingdotgg#23965)

Signed-off-by: Yordis Prieto <yordis.prieto@gmail.com>
Co-authored-by: Theo Browne <me@t3.gg>
Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
Co-authored-by: Victor Vogel <satsdev@only21mil.xyz>
Co-authored-by: Julius Marminge <julius0216@outlook.com>
Co-authored-by: Yash Singh <saiansh2525@gmail.com>
Co-authored-by: Joseph Vidal <135430578+josephv123@users.noreply.github.com>
Co-authored-by: Theo Browne <me@theo.li>
Co-authored-by: shivam <91240327+shivamhwp@users.noreply.github.com>
Co-authored-by: PR Batch Tester <agent@local.test>
Co-authored-by: fixfon <12512757+fixfon@users.noreply.github.com>
Co-authored-by: Utkarsh Patil <73941998+UtkarshUsername@users.noreply.github.com>
Co-authored-by: Yukun Shan <92423096+nateEc@users.noreply.github.com>
Co-authored-by: Akshar Patel <123344143+AksharP5@users.noreply.github.com>
Co-authored-by: Ben Davis <45952064+bmdavis419@users.noreply.github.com>
…otgg#23922)

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
… for a scroll (pingdotgg#23923)

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
…tgg#23924)

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
…gdotgg#22585)

Co-authored-by: Julius Marminge <julius0216@outlook.com>
Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
…dotgg#22628)

Co-authored-by: Claude Fable 5.1 <noreply@anthropic.com>
… other services (pingdotgg#23975)

Signed-off-by: Yordis Prieto <yordis.prieto@gmail.com>
…4-0108

# Conflicts:
#	apps/mobile/src/features/threads/ThreadFeed.tsx
…fixes

On the VPS the apps/server fork tests stop the agent session, and a foreground command stops after 10 minutes. The filters replace the hand-made file lists of the last two sync repairs.
@Yarden-zamir
Yarden-zamir merged commit bddc341 into main Oct 24, 2026
18 of 19 checks passed
@Yarden-zamir
Yarden-zamir deleted the fork-sync/2026-10-24-0108 branch October 24, 2026 01:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants