Skip to content

Commit 10ff060

Browse files
committed
docs: record the 2026-08-11 fifth upstream batch
1 parent 532a4fb commit 10ff060

1 file changed

Lines changed: 24 additions & 1 deletion

File tree

.agents/upstream-review.md

Lines changed: 24 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
---
22
remote: t3code-upstream
33
branch: main
4-
reviewed-through: "35172010b131510d36d0cef54e174926e38a3013"
4+
reviewed-through: "2db08457f2f4eaaa713a067b2ea480ca2b583025"
55
reviewed-through-date: "2026-08-11"
66
---
77

@@ -591,6 +591,29 @@ Setup notes discovered during that pass, since they will bite the next session:
591591
that the accessibility tree does not expose are unreachable by gesture; deep
592592
links such as `pylon-code-dev://new/draft` are the reliable way in.
593593

594+
## 2026-08-11 (fifth batch) — `35172010b131510d36d0cef54e174926e38a3013..2db08457f2f4eaaa713a067b2ea480ca2b583025`
595+
596+
Two upstream commits, both adopted. **First batch to land through the pull
597+
request workflow** rather than a fast-forward onto `pylon`, per the `Landing
598+
changes` section added to `AGENTS.md` in PR #1. Each change set is its own PR,
599+
so `L1` and `L2` may merge in either order.
600+
601+
DEF-1 and DEF-2 were re-evaluated against this head and both **stay deferred**.
602+
Neither gate holds: the date is before 2026-08-25, and the fourteen-day churn
603+
check on the pull-requests files still returns `cad2c9361` and `a7b0366cb`.
604+
605+
| Change set | Upstream | Decision | Pylon reference | Rationale or revisit condition |
606+
| ---------- | --------------------- | -------- | --------------- | ------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- |
607+
| L1 | `2db08457f` / `#6207` | adopted | PR #2 | The Push quick action had no icon case and fell through to the generic `InfoIcon`; it now uses `CloudUploadIcon`, matching how push is already drawn elsewhere in the same file. Direct follow-up to K6 / `#6194`. |
608+
| L2 | `083fa4ab2` / `#6036` | adopted | PR #3 | Palette generation moves to OKLCH, adding `culori` and `@types/culori`. Clean cherry-pick despite five diverged files. **K11 / `#6183` is an ancestor of this commit**, which is why it applies against the theme-aware artwork adopted in the fourth batch — taking them out of order would have fought. **`index.html` verified Pylon-first:** the diff adds no visible T3 branding and leaves `<title>Pylon (Alpha)</title>` untouched; the `t3-chat-dark` id and boot-script comments are pre-existing compatibility identifiers. |
609+
610+
Verification: `themePalette`, `themeBoot`, `vscodeThemeImport`, and
611+
`SidebarStageBackdrop` pass (78 tests). Web typecheck, lint, and format clean.
612+
613+
**Not proven by tests:** whether the five built-in themes still _look_ right
614+
after the color-space change. Palette math that computes cleanly can still shift
615+
perceptibly, so L2 wants one pass in a real client after merge.
616+
594617
## Deferred register
595618

596619
Upstream work that has been reviewed and consciously _not_ adopted yet, with

0 commit comments

Comments
 (0)