Skip to content

chore: sync with upstream pingdotgg/t3code main - #222

Merged
yordis merged 17 commits into
mainfrom
yordis/chore-sync-upstream-20261109-2
Nov 10, 2026
Merged

yordis merged 17 commits into
mainfrom
yordis/chore-sync-upstream-20261109-2

Conversation

@yordis

@yordis yordis commented Nov 10, 2026 •

Copy link
Copy Markdown
Member
  • Merge applied without textual conflicts except the agent context budget file: took upstream's values, then re-recorded the measured MCP tool list size (48359) so the budget test passes and stays below main's recorded value.
  • Upstream added Blacksmith-only mobile workflows and unpinned action tags: pinned every tag to a full commit SHA (reusing existing pins) so the org policy does not cause startup failures.
  • useblacksmith/stickydisk is not on the org allowlist: removed the cache mount steps and disabled the Android and iOS build jobs (dev client, preview, production), since this fork does not ship mobile builds.
  • No fork ledger entry was made redundant by the new upstream commits; migrations are untouched and identical to upstream.

Note

Medium Risk
Touches OAuth CSRF binding, preview tab selection for agents, and a large mobile layout refactor; mobile build jobs are disabled here but merged workflow/scripts still expand CI surface.

Overview
Brings in Blacksmith-backed mobile automation: a new Mobile Dev Client workflow and split Android/iOS EAS preview pipelines (local Android builds, shared PR comment script with tests, CI wiring). Docs and agent skills now describe dispatching that workflow when mobile:install misses cache. This fork keeps the workflow files but disables Android/iOS build jobs, drops sticky-disk mounts, and pins workflow action SHAs per org policy.

Mobile app splits adaptive workspace context into stable actions vs pane state (useWorkspacePaneState) to cut re-renders on sidebar toggles, and removes the Android composer-keyboard sidebar slide/overhang behavior. Thread lists and the composer show ACP registry instance names when icons don’t distinguish them; Android drops compact single-line headers; the thread feed stops Android focus-scroll jumps and drops unused initial content inset helpers.

Server: MCP OAuth CSRF binds to DPoP thumbprint when the hosted app renews credentials on the same browser key; preview tab visibility and default tab pick follow the desktop-shown tab, not only stream viewers; Antigravity usage gets richer model resolution/pricing and strips inherited Crashpad pipe env; preview tool copy tightens write vs display guidance. Web: image preview uses Ctrl/pinch to zoom and normal wheel to pan.

Reviewed by Cursor Bugbot for commit b0782c9. Bugbot is set up for automated code reviews on this repo. Configure here.

juliusmarminge and others added 17 commits November 9, 2026 17:42
…approve" (pingdotgg#33645)

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
New threads started without an active thread or draft used the first project in sidebar order, even after working in another project. This was visible when creating a draft from Settings or delivering a pending snapshot after reopening the app.

Remember the project from the current thread or draft, including its environment, and prefer it when a new draft needs a project. Removed or archived projects fall back to sidebar order. The new-thread keyboard shortcut reads the same preference when invoked. Pending snapshots wait while the remembered environment loads and retry when its project state becomes available.

Validation: regression coverage for project selection across environments, missing and archived projects, and a preference changed after shortcut registration. Verified Settings → reload → New thread in the client. `vp run check:changed` covers formatting, lint, affected typechecks, tests, and React compiler checks.

Model: GPT 6.1 Sol via the Codex harness.
Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
…s that build (pingdotgg#33617)

Co-authored-by: Claude Fable 5.2 <noreply@anthropic.com>
…w at rest (pingdotgg#33599)

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

Co-authored-by: Julius Marminge <julius0216@outlook.com>
Co-authored-by: Claude Fable 5.1 <noreply@anthropic.com>
Co-authored-by: maria-rcks <maria@kuuro.net>
…bar changes (pingdotgg#33611)

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
…rashpad pipe (pingdotgg#26374)

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

Co-authored-by: Claude Fable 5.2 <noreply@anthropic.com>
…tgg#26630)

Co-authored-by: PR Batch Tester <agent@local.test>
Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
Co-authored-by: maria-rcks <maria@kuuro.net>
…ab (pingdotgg#23589)

Co-authored-by: PR Batch Tester <agent@local.test>
Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
Co-authored-by: PR Batch Tester <agent@local.test>
Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
Co-authored-by: maria-rcks <maria@kuuro.net>
…pstream-20261109-2

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

# Conflicts:
#	apps/server/src/provider/agentContextBudget.json
Signed-off-by: Yordis Prieto <yordis.prieto@gmail.com>
Signed-off-by: Yordis Prieto <yordis.prieto@gmail.com>
Signed-off-by: Yordis Prieto <yordis.prieto@gmail.com>

@cursor cursor Bot 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.

Cursor Bugbot has reviewed your changes using high effort and found 2 potential issues.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit b0782c9. Configure here.

android-build:
name: Android preview client
needs: preview
if: needs.preview.outputs.build == 'true'

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Fork Android preview build still enabled

Medium Severity

The fork Android preview android-build job still runs when needs.preview.outputs.build is true. The other native build jobs (mobile-dev-client, same-repo preview android-build, production store builds) are gated with if: false because this fork does not ship mobile builds, but this pull_request_target job was left enabled. A labeled fork PR with no existing client still starts a 32-vCPU Blacksmith eas build --local using the reviewed SHA and EXPO_PREVIEW_TOKEN.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit b0782c9. Configure here.

name: Android preview client
needs: android
# disabled: Android and iOS builds are not used by this fork
if: false

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Preview comment waits for disabled build

Medium Severity

The same-repo Android preview job still comments that a Blacksmith client is building and will receive an install link when that finishes. android-build is now if: false, so when no cached client exists the comment stays on “Building on Blacksmith” with a run link that never posts the APK. Testers cannot tell the native build was intentionally skipped.

Additional Locations (1)
Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit b0782c9. Configure here.

@yordis
yordis merged commit d96351d into main Nov 10, 2026
6 of 24 checks passed
@yordis
yordis deleted the yordis/chore-sync-upstream-20261109-2 branch November 10, 2026 02:35
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.

5 participants