Skip to content

fix(scripts): the reconnect verifier finds a box's saved pairing over its routes - #187

Merged
andrewcai8 merged 2 commits into
mainfrom
fix/verifier-route-credentials
Oct 7, 2026
Merged

andrewcai8 merged 2 commits into
mainfrom
fix/verifier-route-credentials

Conversation

@andrewcai8

Copy link
Copy Markdown
Owner

verify-box-reconnect.ts --second-client failed live with "client B: no pairing saved for the box" although the box connected: since upstream's multi-route connections (pingdotgg#15467) pairings are saved per route, not under bearer:<environmentId>. The check now looks up each of the box's routes. With this, the full second-client run passes live on andrew.megpt.app (list → host wakes → open → reopen with the saved pairing → dispose).

🤖 Generated with Claude Code

… its routes

Pairings are saved per route since upstream's multi-route connections, so the
old bearer:<environmentId> key no longer finds them. Passed live on andrew.megpt.app.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@github-actions github-actions Bot added size:S vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. labels Oct 7, 2026
@github-actions

github-actions Bot commented Oct 7, 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 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 20.9 KiB 20.9 KiB 0 B (0.0%) 29.3 KiB ✅
Codex Live turn messages 2 2 0 (0.0%) 8 ✅
Claude Total thread wire 5.0 KiB 5.0 KiB +24 B (+0.5%) 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 +24 B (+2.0%) 2.0 KiB ✅
Claude Live turn WebSocket decoded 21.2 KiB 21.2 KiB +41 B (+0.2%) 29.3 KiB ✅
Claude Live turn messages 1 2 +1 (+100.0%) 8 ✅

Baseline: e60139d · PR result: d1d0e36 · 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.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@andrewcai8
andrewcai8 merged commit b674962 into main Oct 7, 2026
25 checks passed
@andrewcai8
andrewcai8 deleted the fix/verifier-route-credentials branch October 7, 2026 09:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:S 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.

1 participant