Skip to content

chore: sync upstream pingdotgg/t3code - #184

Merged
adamfgr merged 6 commits into
mainfrom
chore/upstream-sync-20260716
Jul 16, 2026
Merged

chore: sync upstream pingdotgg/t3code#184
adamfgr merged 6 commits into
mainfrom
chore/upstream-sync-20260716

Conversation

@wizzoapp

@wizzoapp wizzoapp Bot commented Jul 16, 2026

Copy link
Copy Markdown

Nightly sync of new commits from pingdotgg/t3code (3513fa04f..ecb35f758, 6 commits), cherry-picked because this fork squash-merges to main.

Synced commits

upstream title
2fdc704bb perf(web): duty-cycle status animations and remove fixed noise overlay (pingdotgg#3978)
7b928d548 fix(docs): correct CI task-runner commands in ci.md (pingdotgg#3990)
c69120f48 fix(docs): repair broken source links in architecture overview (pingdotgg#3991)
462442667 fix(docs): replace stale codething-mvp absolute paths with repo-relative links (pingdotgg#3992)
d359789ed docs: Add T3 Code Legal Docs (pingdotgg#3972)
ecb35f758 Fix Legal modal header crash (pingdotgg#4000)

Conflict resolution

Exactly one file conflicted — docs/operations/ci.md (docs only; the rest of the range applied cleanly).

Upstream pingdotgg#3990 only rewrote the first bullet (bun run lint/typecheck/test → the Vite+ task-runner commands). The fork had independently diverged on the other bullets, so the hunk collided.

Resolution keeps the fork's description of the fork's real CI, and adopts upstream's corrected command names:

  • Took upstream's command names — verified against .github/workflows/ci.yml: it really does run vp check (lint + typecheck), vpr typecheck, and vp run test. The fork's old text said vp run typecheck, which was stale.
  • Kept fork-local reality for the remaining bullets. Taking upstream verbatim would have made these docs wrong for this fork: upstream claims macOS/Linux/Windows release artifacts, but release.yml here builds Windows x64 only (windows_signing: true, mobile builds deprecated). The mobile-deprecated and Windows-signing bullets are fork-local and were preserved.
  • Also retained the fork's extra CI stages (desktop build verification, desktop launch smoke, release smoke).

Verification

  • vp run typecheck — pass
  • vp check — 0 errors (1 pre-existing lint warning in ConnectionsSettings.tsx, untouched by this sync)
  • vp test run657 files / 5737 tests passed, 7 skipped

No prod-code changes were needed; the fork's model routing is registry-driven.

🤖 Generated with Claude Code

@wizzoapp

wizzoapp Bot commented Jul 16, 2026

Copy link
Copy Markdown
Author

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Swish!

Reviewed commit: 9cd277ec73

ℹ️ 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".

@adamfgr adamfgr left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Auto-approved by wizzo-approve: all policy conditions verified against HEAD 9cd277e.

@adamfgr
adamfgr merged commit 6a16add into main Jul 16, 2026
8 checks passed
@adamfgr
adamfgr deleted the chore/upstream-sync-20260716 branch July 16, 2026 02:40
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