Skip to content

fix(server): fork Codex threads with active writers - #1

Closed
f-trycua wants to merge 326 commits into
mainfrom
t3code/fix-active-thread-writer
Closed

fix(server): fork Codex threads with active writers#1
f-trycua wants to merge 326 commits into
mainfrom
t3code/fix-active-thread-writer

Conversation

@f-trycua

Copy link
Copy Markdown

Problem

Codex rejects thread/resume when another app-server process already owns the thread writer. T3 currently stops the session and surfaces the provider stack trace, even though Codex can branch the conversation safely.

Fix

Detect the specific active-writer response and call thread/fork with the requested working directory, model, service tier, approval policy, and sandbox mode. T3 then stores the forked thread ID and continues from the persisted conversation history. Missing-thread fallback and unrelated resume failures keep their existing behavior.

Tests

  • vp test run apps/server/src/provider/Layers/CodexSessionRuntime.test.ts
  • vp run --filter ./apps/server typecheck
  • targeted lint and format checks for both changed files

Model: gpt-5.6-sol
Harness: T3 Code Codex

t3dotgg and others added 30 commits August 8, 2026 19:45
…ep (pingdotgg#5776)

Co-authored-by: Claude Opus 5 (1M context) <noreply@anthropic.com>
…data (pingdotgg#5773)

Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
…g#5777)

Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
…server (pingdotgg#5788)

Co-authored-by: Claude Opus 5 (1M context) <noreply@anthropic.com>
Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
pingdotgg#5782)

Co-authored-by: tsouth89 <tsouth89@users.noreply.github.com>
Co-authored-by: Claude Opus 5 (1M context) <noreply@anthropic.com>
…stalled providers are found (pingdotgg#5074)

Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
…thread (pingdotgg#5901)

Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
Co-authored-by: t3-code[bot] <269035359+t3-code[bot]@users.noreply.github.com>
Co-authored-by: codex <codex@users.noreply.github.com>
…5928)

Co-authored-by: Simone <185146821+Lucenx9@users.noreply.github.com>
Co-authored-by: Claude Opus 5 (1M context) <noreply@anthropic.com>
Bil0000 and others added 22 commits August 15, 2026 14:34
Co-authored-by: maria <maria@kuuro.net>
…#6392)

Co-authored-by: Shivam Sharma <91240327+shivamhwp@users.noreply.github.com>
Co-authored-by: shivam <91240327+shivamhwp@users.noreply.github.com>
…ingdotgg#7082)

Co-authored-by: Claude Opus 5 (1M context) <noreply@anthropic.com>
…g#7083)

Co-authored-by: Claude Opus 5 (1M context) <noreply@anthropic.com>
@github-actions github-actions Bot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:M labels Aug 18, 2026
@f-trycua
f-trycua force-pushed the t3code/fix-active-thread-writer branch from ab88b25 to e884bb3 Compare August 18, 2026 15:44
@f-trycua

Copy link
Copy Markdown
Author

Superseded by pingdotgg#7414, which targets the original repository.

@f-trycua f-trycua closed this Aug 18, 2026
@github-actions github-actions Bot added size:XXL and removed size:M labels Aug 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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