Skip to content

Fix: Retry Summarization skips re-transcription#19

Open
kimjune01 wants to merge 2 commits intoRecapAI:mainfrom
kimjune01:fix/retry-summarization-skip-transcription
Open

Fix: Retry Summarization skips re-transcription#19
kimjune01 wants to merge 2 commits intoRecapAI:mainfrom
kimjune01:fix/retry-summarization-skip-transcription

Conversation

@kimjune01
Copy link

Summary

  • Retry Summarization now reuses the existing transcription from the database instead of re-running Whisper
  • Adds retrySummarization(recordingID:) to ProcessingCoordinator that goes straight to the summarization phase
  • Updates SummaryViewModel.retryProcessing() to route summarizationFailed and completed states through the new path

Problem

When clicking "Retry Summarization", the app ran the full pipeline: transcription → summarization. For long recordings this meant waiting several minutes for Whisper to re-transcribe audio that was already transcribed and stored in the database.

Changes

  • ProcessingCoordinatorType.swift — added retrySummarization(recordingID:) to protocol
  • ProcessingCoordinator.swift — implemented retrySummarization and resumeSummarization which skip transcription and use stored transcript text
  • SummaryViewModel.swift — routes retry for summarizationFailed/completed states through retrySummarization instead of startProcessing

Test plan

  • Trigger a recording and let it complete transcription + summarization
  • Click "Retry Summarization" and verify it skips transcription (no Whisper loading/processing)
  • Verify summarization completes and updates the UI
  • Test retry after a summarization failure
  • Verify retrying a failed transcription still re-runs the full pipeline

Fixes #12

kimjune01 and others added 2 commits February 7, 2026 22:02
When clicking "Retry Summarization", the app previously re-ran the
entire Whisper transcription before re-summarizing. This was very slow
for long recordings. Now it reuses the existing transcription text from
the database and only re-runs the summarization step.

Adds `retrySummarization(recordingID:)` to ProcessingCoordinator which
skips the transcription phase and goes straight to summarization using
the already-stored transcript.

Fixes RecapAI#12

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Verifies that summarizationFailed and completed states route through
retrySummarization instead of retryProcessing, avoiding redundant
re-transcription.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
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.

"Retry Summarization" triggers an entirely new transcription prior to an attempted re-summarization

1 participant