Repository navigation
chore: sync fork with upstream main (1295 commits) - #20
Merged
Merged
Conversation
Co-authored-by: maria-rcks <254055478+maria-rcks@users.noreply.github.com>
Co-authored-by: maria-rcks <254055478+maria-rcks@users.noreply.github.com>
Co-authored-by: maria-rcks <254055478+maria-rcks@users.noreply.github.com>
Co-authored-by: maria-rcks <254055478+maria-rcks@users.noreply.github.com>
Co-authored-by: maria-rcks <254055478+maria-rcks@users.noreply.github.com>
pingdotgg#9433) Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: maria-rcks <254055478+maria-rcks@users.noreply.github.com>
…9466) Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
…pingdotgg#9282) Co-authored-by: Claude Fable 5.1 <noreply@anthropic.com> Co-authored-by: Julius Marminge <julius0216@outlook.com>
Co-authored-by: maria-rcks <254055478+maria-rcks@users.noreply.github.com>
…ingdotgg#9469) Co-authored-by: Claude Code <noreply@anthropic.com>
Co-authored-by: maria-rcks <254055478+maria-rcks@users.noreply.github.com>
…ingdotgg#9479) Co-authored-by: Claude Code <noreply@anthropic.com>
Co-authored-by: maria-rcks <254055478+maria-rcks@users.noreply.github.com>
…9492) Co-authored-by: maria-rcks <254055478+maria-rcks@users.noreply.github.com>
Co-authored-by: maria-rcks <254055478+maria-rcks@users.noreply.github.com>
…ngdotgg#9486) Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
…gg#9493) Co-authored-by: Claude Code <noreply@anthropic.com>
Co-authored-by: Simone <185146821+Lucenx9@users.noreply.github.com>
Co-authored-by: Claude Opus 5 (1M context) <noreply@anthropic.com>
…en reader (pingdotgg#5521) Co-authored-by: Julius Marminge <51714798+juliusmarminge@users.noreply.github.com>
…gg#4685) Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
Long runs display total minutes, such as `419m 50s`, which is hard to read. Show `6h 59m 50s` instead. Web, desktop, and mobile use the shared formatter for long live, completed, and stopped runs. ## Verification - 210 tests pass with `vp test run packages/shared/src/orchestrationTiming.test.ts apps/web/src/components/chat/MessagesTimeline.logic.test.ts apps/mobile/src/lib/threadActivity.test.ts`. - Shared, web, and mobile typechecks pass. - Changed-file lint and formatting pass, with existing lint warnings. - Browser verification and screenshots skipped at Theo's request. Created with GPT-6 Astra (preview) in Codex.
…s the binary (pingdotgg#9325) Co-authored-by: Claude Code <noreply@anthropic.com>
Press ArrowUp in an empty composer to get the last prompt sent in this thread. ArrowUp again goes further back, ArrowDown comes forward, and going past the newest prompt clears the composer. History is per thread and derived from the thread's user messages on the keypress, so nothing new is stored or synced. Send-time appends (terminal and element context, preview annotations, review comments, the ultrathink prefix) and app-composed sends are stripped. With text in the composer, arrows move the caret unless the caret is on the first or last visual line of an unedited recalled prompt. Modifiers, IME composition, menus, approvals, and pending questions take priority. Attachments are not restored. Prior art: pingdotgg#1778 by @PratyushChauhan, pingdotgg#4336 by @mfazekas, pingdotgg#7952 by @sethwebster. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Sync the fork with pingdotgg/t3code after six weeks of drift. All 12 fork commits are preserved; three resolutions changed shape because upstream solved the same problem differently or removed the ground the fork patch stood on. - CLAUDE.md: take upstream's regular file containing `@AGENTS.md`. It achieves what the fork's symlink repair (#12) intended and does not break on Windows, where symlinks do not survive checkout. - index.css: upstream built a runtime theming system (`data-theme-id`, `--app-theme-*`) and expanded the token set with roles the fork's palette never defined. Take upstream's structure whole and append Dimidium as a default-palette override, so every new role resolves and a theme selected in Settings still wins on specificity. - index.css: drop the `[data-sidebar-version]` overrides. Upstream removed those selectors, so the rules were dead. - ClaudeProvider.ts: upstream moved the capabilities probe from `timeoutOption` to `Effect.timeout`, making a timeout a failure rather than `Option.none`. The fork's two-branch warning collapses to one. - manifest: merge both. Upstream's `id`/`scope`/dark background plus the fork's 192/512/maskable icons, which upstream's 32px/180px set lacks for installability. Drop the now-duplicate manifest.json. - workflows: reapply the Blacksmith -> GitHub-hosted runner swap across upstream's five new workflows, and re-disable release.yml's tag and schedule triggers. - brand assets: regenerate from upstream's updated icon sources via the fork's own no-macOS pipeline rather than picking a side. Verified: typecheck exit 0 across 15 packages; 61/61 targeted tests including providerInstances.test.ts, which covers the warning-status fix; web build exit 0 emitting dist/sw.js and the merged manifest, with the Dimidium palette and the standalone-mode inset present in the built CSS. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
github-actions
Bot
added
vouch:trusted
The upstream merge brought in ModelPickerContent.test.ts, a test added during the six-week gap that asserts a provider collapses to its active synthetic row in both "error" and "warning" status. It merged cleanly as text but conflicts semantically with 1b96885, which deliberately stopped hiding a provider's models while it is merely in warning status. Drop the two "warning" rows from that table, leaving "error" as the only status that still collapses the picker. This is the same migration 1b96885 applied to the fixtures it touched at the time; this test did not exist yet to receive it. Also fix formatting in index.css flagged by `vp check`. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
This was referenced Sep 5, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
20 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
pingdotgg/t3code@maininto the fork after six weeks of drift (merge-base 2026-07-23, upstream tip 2026-09-04): 1295 commits behind, 12 ahead.upstreamremote (the clone only hadorigin).Why
The fork had drifted far enough that it no longer picked up upstream fixes, and every conflict-prone file was one the fork had customized. Syncing now — rather than at 2000+ commits — keeps the PWA, theme, and CI customizations rebasable at all.
Notable resolutions
CLAUDE.md@AGENTS.mdwins over the fork's symlink (#12). Same intent, and it survives checkout on Windows.apps/web/src/index.cssdata-theme-id,--app-theme-*) and added token roles the fork's palette never defined. Took upstream's structure whole; Dimidium reapplied as an appended default-palette override, so new roles resolve and a theme picked in Settings → Appearance still wins on specificity.apps/web/src/index.css[data-sidebar-version]overrides — upstream removed those selectors, so the rules were dead code.ClaudeProvider.tstimeoutOptiontoEffect.timeout, so a timeout is now a failure rather thanOption.none; the fork's two-branch warning collapses to one. The behavioural fix inproviderInstances.tsmerged cleanly.id/scope/darkbackground_color+ the fork's 192/512/maskable icons (upstream ships only 32px/180px, which is not installable on Android). Duplicatemanifest.jsonremoved.release.ymltag/schedule triggers re-disabled..iconsources via the fork's own no-macOS pipeline (37 assets), rather than picking a side.Upstream added its own PWA manifest in this window but still ships no service worker, so the fork's
vite-plugin-pwaintegration continues to do real work.Test plan
typecheck— exit 0, 0 TS errors, 15/15 packages (output was Effect language-service suggestions only)providerInstances.test.ts,brand-assets.test.ts,icon-composite.test.ts;providerInstances.test.tsspecifically covers the warning-status fix, confirming it survived behaviourally@t3tools/web build— exit 0, emitsdist/sw.js+ workbox with 2194 precache entriesmanifest.webmanifest+ all PWA icons shipped todist/; Dimidium values (#f5f3f1/#1a8e46light,#141414/#37e57bdark) compiled into the CSS;display-mode: standaloneinset rule presentMerge note
Merge this with a real merge commit, not a squash. Squashing 1295 upstream commits would make them non-ancestors of
main, so the next sync would re-conflict on everything resolved here.🤖 Generated with Claude Code