Skip to content

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

Merged
Yarden-zamir merged 11 commits into
mainfrom
fork-sync/2026-10-26-1509
Oct 26, 2026
Merged

Yarden-zamir merged 11 commits into
mainfrom
fork-sync/2026-10-26-1509

Conversation

@Yarden-zamir

Copy link
Copy Markdown
Owner

Cause

Fork Sync run 40346250614 stopped on merge conflicts in 3 files. The batch has 10 upstream commits:

Conflicts and resolution

  • useEnvironmentTheme.ts: upstream's file. The fork useActiveThemeFonts hook stays.
  • themePalette.ts (no conflict, but the fork style fields were lost): upstream's environmentThemeDefinition now builds the definitions. One fork line adds style: publishedThemeStyle(theme) to it.
  • __root.tsx: upstream drops SlowRpcRequestToastCoordinator and BrowserChromeThemeSync. The fork ForkAccessChangeNotices stays. The fork machine paused and client pending screens no longer mount their own chrome sync, because the app now syncs it on every route render.
  • Mobile ThreadComposer.tsx: upstream's useDerivedValue border radius. The fork theme radius and raised style stay as its input.

Fork fixes

  • Theme style fields: the fork style code (applyThemeStyle, active theme fonts and icons, user CSS, publishedThemeStyle) moves from themePalette.ts into the new fork file apps/web/src/forkThemeStyle.ts. themePalette.ts keeps one import and two calls. appInit.tsx, AppIcon.tsx and useEnvironmentTheme.ts import from the fork file.
  • themeStyle.browser.test.ts: upstream removed setEnvironmentThemes. The test now publishes themes through the server themes atom, as upstream's useEnvironmentTheme.test.ts does.

Upstream merge

Merges pingdotgg/t3code main at 3f94487 "fix(web): switching back to a thread no longer restores its tab's older title (pingdotgg#25521)".

Adopted upstream pull requests

All 7 entries are unchanged at their listed heads. No new merges.

Upstream changes in fork areas

16 files changed on both sides. Apart from the conflicts above, none affects a fork feature:

  • Mobile queue (ThreadQueueControl.tsx, use-thread-composer-state.ts): upstream changes the drag state and the end of queued edits. The fork usage limit guard on Resume and Send merged without a change.
  • Mobile composer editor: upstream settles view commands. The fork theme font hook is on other lines.
  • Mobile thread settings: upstream removes the model and runtime mode callbacks from the composer. No fork code uses them.
  • appInit.tsx: upstream adds the theme repaint and chrome sync next to the fork theme font sync.
  • entities.ts: upstream removes the active environment id atom. No fork code uses it.
  • Preview chrome row and hosted browser: the address bar focus and resize target change. No fork code depends on the old props.

Checks

  • checks.py: typecheck, migration check, adopted pull request check, format and lint pass. Knip passes when run alone.
  • fork_tests.py: 179 fork test files pass locally (117 root and 62 web). The apps/server fork test files are left to Fork CI.
  • Upstream's useEnvironmentTheme.test.ts and themePalette.test.ts pass.

Fork repair skill changes

None.

juliusmarminge and others added 11 commits October 26, 2026 06:34
…ger throws (pingdotgg#25503)

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

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
…er title (pingdotgg#25521)

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
…6-1509

# Conflicts:
#	apps/mobile/src/features/threads/ThreadComposer.tsx
#	apps/web/src/hooks/useEnvironmentTheme.ts
#	apps/web/src/routes/__root.tsx
@Yarden-zamir
Yarden-zamir merged commit 012cc8f into main Oct 26, 2026
11 of 12 checks passed
@Yarden-zamir
Yarden-zamir deleted the fork-sync/2026-10-26-1509 branch October 26, 2026 16:04
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.

2 participants