Skip to content

chore(upstream): track the compaction message queue fix - #789

Merged
incognitojam merged 1 commit into
mainfrom
styal/track-compaction-queue-fix
Oct 13, 2026
Merged

incognitojam merged 1 commit into
mainfrom
styal/track-compaction-queue-fix

Conversation

@incognitojam

Copy link
Copy Markdown
Owner

Problem

Messages sent during context compaction can be rejected as steering instead of queued. Upstream #12824 addresses this regression and is still open.

Change

Add an ordinary watch entry explaining the regression and why we are waiting for upstream merge and chronological intake. The lag and intake reports keep the fix visible without changing queue order.

Scope and approval

Maintainer-requested update to the existing upstream watch list, limited to .github/upstream-tracked-prs.json.

Verification

  • vp test run scripts/upstream/lib/tracked-prs.test.ts: all six tests pass, covering watch-entry validation and report/status behavior.
  • The updated JSON passes decodeTrackedPRs and vp fmt --check.
  • node scripts/upstream/tracked-prs-report.ts --fork-ref HEAD shows fix(v2): queue messages sent while context compaction runs pingdotgg/t3code#12824 as open, with the expected watch reason and no provisional snapshot.

Written by an agent (Codex, gpt-6.1-sol).

@github-actions

github-actions Bot commented Oct 13, 2026 •

Copy link
Copy Markdown

Upstream divergence

Both sides are compared with upstream at the in-order tip 263097a8da. Excludes .repos/ and pnpm-lock.yaml.

Measure Base This PR Change
Upstream files the fork changes 1,327 1,327 0
Lines changed in them 59,638 59,638 0
Upstream files the fork removes 37 37 0
Fork-only files 410 410 0
Fork-only lines 71,567 71,571 +4

No upstream file changes how much it differs from upstream.

@github-actions

github-actions Bot commented Oct 13, 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 4.9 KiB 4.9 KiB −40 B (−0.8%) 6.8 KiB ✅
Codex Thread snapshot wire 3.7 KiB 3.7 KiB 0 B (0.0%) 4.9 KiB ✅
Codex Live turn WebSocket wire 1.2 KiB 1.1 KiB −40 B (−3.3%) 2.0 KiB ✅
Codex Live turn WebSocket decoded 20.4 KiB 20.4 KiB −41 B (−0.2%) 29.3 KiB ✅
Codex Live turn messages 2 1 −1 (−50.0%) 8 ✅
Claude Total thread wire 4.9 KiB 4.9 KiB +41 B (+0.8%) 6.8 KiB ✅
Claude Thread snapshot wire 3.7 KiB 3.7 KiB 0 B (0.0%) 4.9 KiB ✅
Claude Live turn WebSocket wire 1.2 KiB 1.2 KiB +41 B (+3.5%) 2.0 KiB ✅
Claude Live turn WebSocket decoded 20.7 KiB 20.8 KiB +41 B (+0.2%) 29.3 KiB ✅
Claude Live turn messages 1 2 +1 (+100.0%) 8 ✅

Baseline: 8d2ebd6 · PR result: 9033105 · 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: 106.1 KiB
  • Claude decoded thread snapshot: 106.4 KiB

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

@incognitojam
incognitojam force-pushed the styal/track-compaction-queue-fix branch from e088027 to 9033105 Compare October 13, 2026 11:03
@incognitojam
incognitojam enabled auto-merge (squash) October 13, 2026 11:13
@incognitojam
incognitojam merged commit 19c9e14 into main Oct 13, 2026
32 checks passed
@incognitojam
incognitojam deleted the styal/track-compaction-queue-fix branch October 13, 2026 11:14
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.

1 participant