Skip to content

fix: isolate transcription results by media - #50

Merged
FernandoAbishai merged 1 commit into
mainfrom
fix/transcription-media-lifecycle
Aug 25, 2026
Merged

fix: isolate transcription results by media#50
FernandoAbishai merged 1 commit into
mainfrom
fix/transcription-media-lifecycle

Conversation

@FernandoAbishai

Copy link
Copy Markdown
Owner

Phase 7D.1 P1 corrective

  • Isolates asynchronous transcription runs by monotonic generation and expected media path.
  • Prevents stale transcript, active-panel, message/log/progress, error, finally, and job-id writes after media/project replacement.
  • Preserves cancellation, retry, current-run completion, full-video/short workflow intent, Electron picker cancellation, and failed-upload behavior.
  • Adds deterministic lifecycle regression coverage for stale/current results, polling UI, cleanup, errors, job-id races, transitions, restore, retry, and panel intent.
  • No backend, project schema, AI clip workspace, release, or version changes.

Verification

  • All 21 frontend smoke scripts: PASS
  • Backend smoke: PASS (121 tests, loopback permission enabled)
  • npm run lint: PASS
  • npm run build:frontend: PASS
  • git diff --check: PASS

Do not merge.

Copilot AI lite review requested due to automatic review settings August 25, 2026 03:33

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@FernandoAbishai
FernandoAbishai merged commit 6d4ad9c into main Aug 25, 2026
2 checks passed
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.

2 participants