Skip to content

Add video step-overlay walkthrough report#247

Merged
JE-Chen merged 1 commit into
devfrom
feat/video-overlay-batch
Jun 19, 2026
Merged

Add video step-overlay walkthrough report#247
JE-Chen merged 1 commit into
devfrom
feat/video-overlay-batch

Conversation

@JE-Chen

@JE-Chen JE-Chen commented Jun 19, 2026

Copy link
Copy Markdown
Member

Reporting batch — the visual companion to the HTML/JSON reports. Full layers + tests + EN/Zh v39 docs + README.

Feature (utils/video_report, cv2 only on the real path)

  • write_step_video(steps, output, fps, seconds_per_step): renders per-step screenshots into a walkthrough video where each frame is held for a few seconds with its caption and a pass/fail colour banner burned in. build_overlay_plan (per-step frame plan, no I/O) and render_overlay_frame (single banner) are usable on their own.
  • Testable without cv2: the loader / drawer / writer_factory hooks are injectable, so the assembly logic is unit-tested with fakes and no cv2/numpy dependency (runs in quality.yml). The real path lazily imports cv2 only when those hooks are absent; a dedicated test exercises it under importorskip. The writer is released in a finally.
  • Executor AC_write_step_video (steps as list or builder JSON string); MCP ac_write_step_video; Builder under Report.

Verification

  • 8 tests pass (frame plan, min-1-frame, injected drawer, full assembly with fakes, writer released on error, real OpenCV path under importorskip, wiring, facade); ruff clean; radon no CC≥C; bandit clean; PySide6-free.

@codacy-production

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

🟢 Metrics 43 complexity · 0 duplication

Metric Results
Complexity 43
Duplication 0

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

@JE-Chen JE-Chen merged commit 4fe0e48 into dev Jun 19, 2026
16 checks passed
@JE-Chen JE-Chen deleted the feat/video-overlay-batch branch June 19, 2026 15:40
@sonarqubecloud

Copy link
Copy Markdown

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