Skip to content

fix(app): recover failed turn retractions without duplicating drafts - #61

Merged
asfires merged 1 commit into
mainfrom
t3code/fix-failed-turn-retraction
Aug 30, 2026
Merged

fix(app): recover failed turn retractions without duplicating drafts#61
asfires merged 1 commit into
mainfrom
t3code/fix-failed-turn-retraction

Conversation

@asfires

@asfires asfires commented Aug 30, 2026

Copy link
Copy Markdown
Owner

What Changed

  • Use Codex thread/revert when reverting a specific turn, with fallback support for older app-server versions.
  • Verify rollback results from the mutation response to avoid failures caused by lagging thread reads.
  • Prevent unresolved mid-thread retractions from being moved into new-thread drafts.
  • Add focused server and web regression tests.

Why

Failed or delayed retraction acknowledgements could leave recovery pending, repeatedly roll back the same provider turn, or duplicate a message as a new-thread draft. This change uses the provider’s direct revert response when available, preserves compatibility with older Codex versions, and limits draft recovery to first-message retractions where it is appropriate.

UI Changes

No visible UI changes. The web change updates retraction recovery behavior only.

Checklist

  • This PR is small and focused
  • I explained what changed and why
  • I included before/after screenshots for any UI changes
  • I included a video for animation/interaction changes

- Use Codex revert responses to avoid stale-read failures
- Keep unresolved mid-thread retractions out of new-thread drafts
@github-actions github-actions Bot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:M labels Aug 30, 2026
@asfires
asfires merged commit c093de1 into main Aug 30, 2026
6 of 14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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