Skip to content

feat: retry the last user event#3373

Closed
Frozair wants to merge 1 commit into
tailcallhq:mainfrom
Frozair:codex/retry-api-389
Closed

feat: retry the last user event#3373
Frozair wants to merge 1 commit into
tailcallhq:mainfrom
Frozair:codex/retry-api-389

Conversation

@Frozair
Copy link
Copy Markdown

@Frozair Frozair commented May 21, 2026

Summary

  • add a first-class API::retry(conversation_id) path instead of routing /retry through an empty follow-up message
  • rebuild retry requests from the last saved raw user event in conversation history, skipping synthetic user messages that do not preserve raw content
  • wire both retry entry points in forge_main to the new API method and add focused regression tests for retry event extraction

Why

The current interactive retry flow is reachable, but it retries by sending an empty follow-up into the conversation. This PR keeps /retry tied to the last actual user event, which is a closer semantic match for a real retry and avoids depending on a synthetic empty message.

Verification

  • cargo test -p forge_domain test_last_retryable_event
  • cargo test -p forge_main retry
  • cargo check -p forge_api -p forge_main
  • git diff --check

/claim #389

@github-actions github-actions Bot added the type: feature Brand new functionality, features, pages, workflows, endpoints, etc. label May 21, 2026
@Frozair
Copy link
Copy Markdown
Author

Frozair commented May 21, 2026

Closing this out because the linked issue was already closed before submission, so I’m not planning to spend more review/follow-up time on this lane.

@Frozair Frozair closed this May 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: feature Brand new functionality, features, pages, workflows, endpoints, etc.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant