Skip to content

fix(mobile): wait for deep-link thread hydration - #764

Merged
rynfar merged 1 commit into
pylonfrom
upstream/2026-09-24-mobile-deeplink-hydration
Sep 25, 2026
Merged

rynfar merged 1 commit into
pylonfrom
upstream/2026-09-24-mobile-deeplink-hydration

Conversation

@rynfar

@rynfar rynfar commented Sep 24, 2026 •

Copy link
Copy Markdown
Collaborator

Opening a thread deep link could show "Thread unavailable" while that environment's shell and thread detail were still synchronizing. The route now waits for its own shell/detail state, while deleted threads and sync errors leave loading promptly. A deleted or missing thread returns to the thread list; connection failures can retry the environment; unknown environments open environment settings.

Adapted from T3 commit caf8b5d798909780a7b197cb264e782888b919c1 (#11502). The terminal recovery action is Pylon-specific: reconnecting cannot restore a deleted or missing thread. Existing shell-first thread rendering, drafts, and local/remote environment selection remain intact.

Validation: 7 focused mobile/shared shell tests passed, mobile typecheck and scoped lint passed, and independent source review cleared the final patch. Lint reports pre-existing ThreadRouteScreen warnings. Browser/device verification was not run because AGENTS.md requires explicit approval.

Model: GPT-6 Sol (low). Harness: Codex in Pylon, with GPT-6 root oversight and independent review.


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

@vercel

vercel Bot commented Sep 24, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated
pylon-marketing Ignored Ignored Preview Sep 24, 2026 8:41am UTC

@github-actions github-actions Bot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:L labels Sep 24, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

Provider Metric Main baseline This PR Impact PR ceiling
Codex Total thread wire 14.0 KiB 14.0 KiB −16 B (−0.1%) 15.1 KiB ✅
Codex Thread snapshot wire 7.2 KiB 7.2 KiB +7 B (+0.1%) 7.3 KiB ✅
Codex Live turn WebSocket wire 6.8 KiB 6.7 KiB −23 B (−0.3%) 7.8 KiB ✅
Codex Live turn WebSocket decoded 58.0 KiB 58.0 KiB −44 B (−0.1%) 66.4 KiB ✅
Codex Live turn messages 10 9 −1 (−10.0%) 21 ✅
Claude Total thread wire 14.0 KiB 14.0 KiB +10 B (+0.1%) 15.1 KiB ✅
Claude Thread snapshot wire 7.2 KiB 7.2 KiB −10 B (−0.1%) 7.3 KiB ✅
Claude Live turn WebSocket wire 6.7 KiB 6.8 KiB +20 B (+0.3%) 7.8 KiB ✅
Claude Live turn WebSocket decoded 58.9 KiB 58.9 KiB 0 B (0.0%) 66.4 KiB ✅
Claude Live turn messages 10 10 0 (0.0%) 21 ✅

Baseline: 7ed3f71 · PR result: 745fe8f · 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: 115.6 KiB
  • Claude decoded thread snapshot: 116.3 KiB

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

@rynfar
rynfar merged commit c6a2aca into pylon Sep 25, 2026
19 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:L 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