Skip to content

Phase 9D: extract clip export controller - #75

Merged
FernandoAbishai merged 1 commit into
mainfrom
phase/9d-clip-export-controller
Sep 7, 2026
Merged

Phase 9D: extract clip export controller#75
FernandoAbishai merged 1 commit into
mainfrom
phase/9d-clip-export-controller

Conversation

@FernandoAbishai

Copy link
Copy Markdown
Owner

Summary

  • extract clip export/job orchestration from AIPanel into useClipExportController
  • move transient export jobs/outputs, directory ownership, polling, cancel/retry, direct export, batch sequencing and manifest orchestration behind one vertical controller
  • keep Review/Prepare, draft editing, AI discovery, publishing copy and UI composition in AIPanel
  • preserve backend, project schema, provider contracts and 9C.3 lifecycle semantics

Preserved safety invariants

  • durable status=exporting is claimed before the first /jobs/export await
  • direct export and retry re-read the current durable draft
  • batch re-reads every planned draft JIT and pauses if it was invalidated
  • cancel does not terminalize or release busy before polling observes terminal state
  • export folder remains frozen while busy
  • runtime file capabilities remain transient only
  • project v1 save/reopen behavior is unchanged

Architecture boundary

  • AIPanel no longer owns clip export endpoints, polling, job/output state or batch stop state
  • controller does not own AI provider, clip-metadata, publishing-copy or social-publishing orchestration

Validation

  • all 22 frontend test:* scripts PASS
  • npm run smoke:golden-creator-path PASS
  • npm run lint PASS
  • npm run build:frontend PASS
  • git diff --check PASS
  • two independent CURRENT blocker/high reviews: no blocker/high findings

Copilot AI lite review requested due to automatic review settings September 7, 2026 09:16

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 e8c76f4 into main Sep 7, 2026
2 checks passed
@FernandoAbishai
FernandoAbishai deleted the phase/9d-clip-export-controller branch September 7, 2026 09:30
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