Skip to content

fix(server): sync three shared migrations with upstream fixes - #86

Merged
wianvdm merged 2 commits into
mainfrom
fix/upstream-migration-sync
Nov 13, 2026
Merged

wianvdm merged 2 commits into
mainfrom
fix/upstream-migration-sync

Conversation

@wianvdm

@wianvdm wianvdm commented Nov 13, 2026

Copy link
Copy Markdown
Owner

Shared Persistence Compat compares our migrations with pingdotgg/t3code main, and upstream has since fixed three shared migrations. Every fork PR fails that check until our copies match.

Cherry-picks the two upstream commits unchanged, tests included:

Tests: vp test run src/persistence/Migrations.test.ts src/persistence/Migrations/064_CanonicalizeThreadPullRequestLinks.test.ts 11/11; check-shared-persistence-compat.cjs against upstream main passes (79 shared migrations).

Claude Opus 5.5 in Claude Code, via T3 Code.

🤖 Generated with Claude Code

@github-actions

Copy link
Copy Markdown

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

ℹ️ The exact PR base did not have a successful artifact. Baseline uses the latest successful main measurement shown below.

Provider Metric Main baseline This PR Impact PR ceiling
Codex Total thread wire 5.0 KiB 5.0 KiB 0 B (0.0%) 6.8 KiB ✅
Codex Thread snapshot wire 3.8 KiB 3.8 KiB 0 B (0.0%) 4.9 KiB ✅
Codex Live turn WebSocket wire 1.2 KiB 1.2 KiB 0 B (0.0%) 2.0 KiB ✅
Codex Live turn WebSocket decoded 21.5 KiB 21.5 KiB 0 B (0.0%) 24.4 KiB ✅
Codex Live turn messages 1 1 0 (0.0%) 8 ✅
Claude Total thread wire 5.0 KiB 5.0 KiB 0 B (0.0%) 6.8 KiB ✅
Claude Thread snapshot wire 3.8 KiB 3.8 KiB 0 B (0.0%) 4.9 KiB ✅
Claude Live turn WebSocket wire 1.2 KiB 1.2 KiB 0 B (0.0%) 2.0 KiB ✅
Claude Live turn WebSocket decoded 21.9 KiB 21.9 KiB 0 B (0.0%) 24.4 KiB ✅
Claude Live turn messages 1 1 0 (0.0%) 8 ✅

Baseline: f5929a0 · PR result: db1f74b · 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: 108.5 KiB
  • Claude decoded thread snapshot: 108.8 KiB

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

@wianvdm
wianvdm merged commit da74e9a into main Nov 13, 2026
21 checks passed
@wianvdm
wianvdm deleted the fix/upstream-migration-sync branch November 14, 2026 16:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants