Skip to content

fix: avoid stale translation session reuse#108

Merged
2sem merged 1 commit into
mainfrom
fix/translation-session-crash
Apr 4, 2026
Merged

fix: avoid stale translation session reuse#108
2sem merged 1 commit into
mainfrom
fix/translation-session-crash

Conversation

@2sem

@2sem 2sem commented Apr 4, 2026

Copy link
Copy Markdown
Owner

Summary

  • Fix a crash caused by reusing a TranslationSession after the .translationTask view lifecycle ended.
  • Ensure each translate action uses a fresh session delivered by .translationTask, especially after language changes.

Test Status

  • Ran mise x -- tuist test

Flow

flowchart TD
    A[User taps Translate] --> B[ViewModel resets translation configuration]
    B --> C[Visible view receives fresh .translationTask session]
    C --> D[Translation runs only with fresh session]
    D --> E[Language change + retranslate no longer crashes]
Loading

@tuist

tuist Bot commented Apr 4, 2026

Copy link
Copy Markdown

🛠️ Tuist Run Report 🛠️

Tests 🧪

Scheme Status Cache hit rate Tests Skipped Ran Commit
talktrans-Workspace 0 % 1 0 1 e159ceb0b

Builds 🔨

Scheme Status Duration Commit
talktrans-Workspace 1m 15s e159ceb0b

@2sem 2sem merged commit 75f18a8 into main Apr 4, 2026
1 check passed
@2sem 2sem deleted the fix/translation-session-crash branch April 4, 2026 12:03
@2sem 2sem mentioned this pull request Apr 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant