Skip to content

chore(upstream): adopt the second 2026-08-21 T3 Code batch (#66) - #66

Merged
rynfar merged 25 commits into
pylonfrom
upstream/2026-08-21-second-batch
Aug 21, 2026
Merged

chore(upstream): adopt the second 2026-08-21 T3 Code batch (#66)#66
rynfar merged 25 commits into
pylonfrom
upstream/2026-08-21-second-batch

Conversation

@rynfar

@rynfar rynfar commented Aug 21, 2026

Copy link
Copy Markdown
Collaborator

Upstream moved 23 commits past our cursor overnight, and one of them unblocked
work we had already deferred. This adopts 20 of the 23 and retires DEF-6.

DEF-6 is closed

#7089 (enable Cursor by default) was deferred yesterday because upstream's
own main was red on it: the commit added a test expecting every provider
enabled while the schema still had Grok and OpenCode off, contradicting an
existing test. #7725 reconciles them by deleting the contradictory test.
Re-probed exactly as the register instructed — both commits onto a scratch
branch, 39/39 pass in settings.test.ts, which is the only place in the repo
asserting a cursor default. Cursor is probed on fresh installs again; #7459's
substance survives because Grok and OpenCode stay false.

What else came in

Pinned-thread drag fixes (#7103, #7676) land on a Pylon feature. Sidebar
provider icons now resolve from the thread's own environment (#7292) —
default instance ids are driver slugs, so the old flat map collided across
environments, which our remote/multi-environment emphasis makes likelier to
hit. #7697 is a real security tightening: we accepted any vscode: or
cursor: URL for the OS handler, and it narrows that to a vscode-remote
host with an /ssh-remote+ path and no embedded credentials. #7561 moves the
preview loading bar off a rAF/React-state loop onto a compositor-driven CSS
animation. Plus slash-menu skills, terminal shifted-character encoding,
stream-following at the live edge, a longer git worktree add timeout, and
HTTPS-by-default GitHub clones.

Skipped three: #7283 (we already solved macOS lint gating better in #51, as
a path-filtered workflow rather than a gate job), #7728 and #7658 (fork
governance and upstream-specific prose).

The two that needed hands

#7286 CI parallelization could not be cherry-picked — our ci.yml had
diverged. Ported by hand: --parallel for non-server suites, three shards for
apps/server (278 files at fileParallelism: false), Rust into its own job.
Kept continue-on-error: true from #9 and added upstream's presence gate,
because they guard different failures — the gate stops the two shards that
legitimately produce nothing from racing for the artifact name, the flag stops
an exhausted org artifact quota from failing a run whose tests passed. Both
test jobs also pass --fail-if-no-match, which upstream does not use: without
it a filter matching no package only warns and exits 0, which is exactly the
silent no-op AGENTS.md warns about.

#7665 plans-out-of-repo is bigger than its title — it also adds a policy
doc, an AGENTS.md section, and config changes. Upstream deleted 32 files; we
tracked 34. The two survivors are live design docs for a shipping Pylon
provider, so under the policy this same commit adopts, they moved to
docs/internals/ rather than being deleted or stranded as tracked files
inside a now-gitignored directory.

Verification

  • 19 test files / 374 tests across web, contracts, client-runtime, desktop,
    mobile, and the two touched server suites — all pass.
  • Typecheck clean across 6 packages, each confirmed to have actually run.
  • actionlint on the rewritten ci.yml: nothing beyond the pre-existing
    unknown Blacksmith runner labels.
  • The sharded server command was run locally first: shard 1/3 took 95 of 278
    files, 900 passed. Its one failure is accountDrainEndToEnd.test.ts, which
    the previous batch's ledger already investigated as a local-only failure that
    reproduces on unmodified origin/pylon.

No UI screenshots: every visual change here is upstream's own, adopted
unmodified, and the composer/sidebar edits are structural rather than
appearance changes.

Written by Claude Opus 5 (1M context) in Claude Code.


View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith with what you need. Autofix is disabled.

Rishet11 and others added 24 commits August 21, 2026 09:52
…rovider (#7089)

(cherry picked from commit 730ce9edd9873144c1d2b01e5f1c85414c3760ad)
(cherry picked from commit 1afe5545b7d0397bbd6515981372072d146002ad)
(cherry picked from commit 4bdbd8ce15fbeeb6514623704a203c43ad37d2cd)
Co-authored-by: t3-code[bot] <269035359+t3-code[bot]@users.noreply.github.com>
Co-authored-by: Utkarsh Patil <73941998+UtkarshUsername@users.noreply.github.com>
(cherry picked from commit fe87502084ad99209a22f801347102ebb29cc65b)
…580)

(cherry picked from commit f0fb83aff1be5d093225cda511df1216cef748d2)
(cherry picked from commit 68966c1e6603d6f5d52a4ee227e2b0ee03b7109f)
Co-authored-by: maria-rcks <254055478+maria-rcks@users.noreply.github.com>
Co-authored-by: t3-code[bot] <269035359+t3-code[bot]@users.noreply.github.com>
(cherry picked from commit e723501227b7d1d0b960eb8fcfd9c8d6099d805e)
Co-authored-by: John William Creighton (s243a) <s243a@users.noreply.github.com>
Co-authored-by: Claude Opus 5 (1M context) <noreply@anthropic.com>
(cherry picked from commit 6d3bf01b4f021eb3fbd2feaf07fcacaa4f55881d)
(cherry picked from commit 820e5639c30be83a9433f696613760f8ee42075a)
…326)

Co-authored-by: Claude Opus 5 (1M context) <noreply@anthropic.com>
(cherry picked from commit 12c497083e6ed2d6f90cc7775b5ecea232926b4a)
Co-authored-by: t3-code[bot] <269035359+t3-code[bot]@users.noreply.github.com>
Co-authored-by: Alex Southwell <4596216+saphid@users.noreply.github.com>
(cherry picked from commit 9167622a4ec1e540938adbb1dec70d255fdeb041)
Co-authored-by: Simone <185146821+Lucenx9@users.noreply.github.com>
(cherry picked from commit 20e5a3396858f93157a4ad6b1f3a2f565c032fee)
(cherry picked from commit 6d5c6c4a671c7b6fc9723c63c1003916fb3b8ab0)
(cherry picked from commit 18f6d034897d9cc5429dcb88c6a9795d183f1687)
…nment (#7292)

Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
(cherry picked from commit f3fcfe1f63ac58cc7bdad2b459931a52ae5f6fa2)
(cherry picked from commit ce8ca5bb3d005d8653d1b09c1e7c9e02d3ef8ae4)
…edge (#6519)

(cherry picked from commit e2697d63ec52e719ebd2d4e25baee2cb58243c83)
Co-authored-by: t3-code[bot] <269035359+t3-code[bot]@users.noreply.github.com>
Co-authored-by: maria <254055478+maria-rcks@users.noreply.github.com>
(cherry picked from commit 9f12eab383ab0fd348c9d35cd248cd939a2c5796)
Co-authored-by: t3-code[bot] <269035359+t3-code[bot]@users.noreply.github.com>
Co-authored-by: maria <254055478+maria-rcks@users.noreply.github.com>
(cherry picked from commit 549201fcfd3baa6cf92f09e5d868c536c80d2d55)
(cherry picked from commit be7d35aaeb49a04483ec5e0d2284e8b5b70a3b6e)
#7665 gitignores `.plans/` and rules that durable architecture belongs in
`docs/internals/`. Pylon's two Prime Agent documents were the only files in
`.plans/` upstream did not delete, and they describe a shipping Pylon
provider rather than an abandoned intention, so they move rather than go.

The integration document keeps its own "historical investigation" banner:
it records why the daemon-first direction won over the ACP spike, which is
the part still worth reading.
Ports #7286. `apps/server` sets `fileParallelism: false`, so its 278 test
files ran strictly one at a time on a single runner; sharding spreads them
over three runners without ever putting two server test files on the same
machine, so the isolation that flag buys survives intact. Everything else
runs with `--parallel`, which drops package dependency ordering these test
tasks never needed. Rust moves to its own 4vcpu job because Check and Test
each paid 7-9s installing a toolchain for checks that take under 3s.

Two Pylon-first departures from upstream's version:

- the transfer-result upload keeps `continue-on-error: true` from #9. It
  now also carries upstream's presence gate, because those guard different
  failures: the gate stops the two shards that legitimately produce nothing
  from racing for the artifact name, while `continue-on-error` stops an
  exhausted org artifact quota from failing a run whose tests passed.
- both test jobs pass `--fail-if-no-match`. Without it a filter matching no
  package only warns and exits 0, which would turn a renamed package into a
  green job that tested nothing.
#7760 switched GitHub clones from the SSH URL to the HTTPS one, which is a
change users notice: `gh auth login` is now sufficient and no SSH key setup
is needed. The other providers still clone over SSH, and pasting a full
`git@` URL still forces SSH anywhere.
Twenty of twenty-three upstream commits adopted, three skipped, cursor to
be7d35aae. DEF-6 came due when #7725 reconciled the contradictory provider
default tests and was adopted after re-probing exactly as the register
instructed, so the deferred register is empty again.
@github-actions github-actions Bot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:XXL labels Aug 21, 2026
@github-actions

github-actions Bot commented Aug 21, 2026

Copy link
Copy Markdown

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

Provider Metric Main baseline This PR Impact PR ceiling
Codex Total thread wire 13.4 KiB 13.4 KiB −3 B (−0.0%) 15.1 KiB
Codex Thread snapshot wire 6.9 KiB 6.9 KiB −6 B (−0.1%) 7.3 KiB
Codex Live turn WebSocket wire 6.5 KiB 6.5 KiB +3 B (+0.0%) 7.8 KiB
Codex Live turn WebSocket decoded 55.0 KiB 55.0 KiB 0 B (0.0%) 66.4 KiB
Codex Live turn messages 16 16 0 (0.0%) 21
Claude Total thread wire 13.5 KiB 13.5 KiB +18 B (+0.1%) 15.1 KiB
Claude Thread snapshot wire 6.9 KiB 6.9 KiB +3 B (+0.0%) 7.3 KiB
Claude Live turn WebSocket wire 6.6 KiB 6.6 KiB +15 B (+0.2%) 7.8 KiB
Claude Live turn WebSocket decoded 55.8 KiB 55.8 KiB 0 B (0.0%) 66.4 KiB
Claude Live turn messages 16 16 0 (0.0%) 21

Baseline: 4f01bcf · PR result: 5c907dc · Source CI: success

Scenario and decoded snapshot size

10 historical turns, 5 command tools per turn, 878.9 KiB retained MCP result per historical turn, and a 1.05 MiB retained result in the measured turn.

  • Codex decoded thread snapshot: 109.4 KiB
  • Claude decoded thread snapshot: 110.0 KiB

Updated in place by a trusted workflow. PR artifacts are strictly validated and never executed.

A high-effort review of #66 found one issue: the search effect's comment
claims `installingId` is deliberately not a dependency while it is in the
array, above a dead eslint directive in an oxlint repo. `vp check` is clean
and the file is byte-identical to upstream, so it is recorded rather than
fixed.
@rynfar
rynfar merged commit 35217db into pylon Aug 21, 2026
14 checks passed
@rynfar
rynfar deleted the upstream/2026-08-21-second-batch branch August 21, 2026 17:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XXL vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.