Skip to content

perf(server): speed up cold desktop launches - #13320

Merged
Yash-Singh1 merged 1 commit into
t3code/codex-turn-mappingfrom
t3code/fix-legacy-import-startup-query
Sep 23, 2026
Merged

Yash-Singh1 merged 1 commit into
t3code/codex-turn-mappingfrom
t3code/fix-legacy-import-startup-query

Conversation

@Yash-Singh1

Copy link
Copy Markdown
Collaborator

Cold desktop launches spent time scanning orchestration events during the legacy-thread checks. Use the existing created-thread index for both startup queries.

In four isolated cold launches against the same V2 database snapshot, time to first window reveal improved by 7.3 seconds (66%). This measurement covers cold launches only.

Validation: focused importer tests, lint, and typecheck.

Model and harness: GPT-6 in Codex.

- Pin startup migration checks to the created-thread index
- Assert both importer queries use that index
@github-actions github-actions Bot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:XS 0-9 changed lines (additions + deletions). labels Sep 23, 2026
@Yash-Singh1 Yash-Singh1 changed the title fix(server): speed up cold desktop launches perf(server): speed up cold desktop launches Sep 23, 2026
@macroscopeapp

macroscopeapp Bot commented Sep 23, 2026

Copy link
Copy Markdown
Contributor

Approvability

Verdict: Approved at 9a5679b

Macroscope's review found this PR approvable — This is a small, self-contained SQLite query-plan optimization that forces an existing, migration-created index for two startup import checks. It preserves the queries' logical behavior and has plan coverage confirming the intended index is used.

You can add or adjust custom eligibility rules. Learn more.

@github-actions

Copy link
Copy Markdown
Contributor

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

ℹ️ No successful main baseline artifact is available yet. This run establishes the initial measurement.

Provider Metric Main baseline This PR Impact PR ceiling
Codex Total thread wire — 4.9 KiB — 6.8 KiB ✅
Codex Thread snapshot wire — 3.7 KiB — 4.9 KiB ✅
Codex Live turn WebSocket wire — 1.2 KiB — 2.0 KiB ✅
Codex Live turn WebSocket decoded — 20.4 KiB — 29.3 KiB ✅
Codex Live turn messages — 2 — 8 ✅
Claude Total thread wire — 4.9 KiB — 6.8 KiB ✅
Claude Thread snapshot wire — 3.7 KiB — 4.9 KiB ✅
Claude Live turn WebSocket wire — 1.2 KiB — 2.0 KiB ✅
Claude Live turn WebSocket decoded — 20.7 KiB — 29.3 KiB ✅
Claude Live turn messages — 1 — 8 ✅

Baseline: unavailable · PR result: 9a5679b · 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.

@Yash-Singh1
Yash-Singh1 merged commit 6ca6a24 into t3code/codex-turn-mapping Sep 23, 2026
25 checks passed
@Yash-Singh1
Yash-Singh1 deleted the t3code/fix-legacy-import-startup-query branch September 23, 2026 23:21
@Yash-Singh1
Yash-Singh1 restored the t3code/fix-legacy-import-startup-query branch September 23, 2026 23:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XS 0-9 changed lines (additions + deletions). 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