Repository navigation
[AI-9939] Chore: Video proof takes its task from the PR description - #123
Conversation
Review Summary✅ Focused CI cleanup: removes the repo-specific videoproof task file and related workflow wiring so Videoproof-CI relies on the angie-dev-tools kit defaults. No application code or testable product paths changed. 🤖 Code Style Review Agent |
🎬 Video Proof Test ResultsBranch: Clicked Ask Angie to open the loadSidebarV2 sidebar on the product editor POC. 📸 ScreenshotsArtifacts are also available on the workflow run. |
PR Approver EvaluationPre-checks did not pass — auto-merge evaluation stopped. Reasons to auto-merge
Reasons not to auto-merge
VerdictWould auto-merge: No — pre-checks blocked evaluation. Fix the issues above, then push new commits to re-run. 🤖 PR Approver Agent |
Review Summary✅ Focused CI workflow updates: videoproof no longer uses a repo-local task file or manual dispatch, and PR reviewer dispatch passes a draft flag. No application code changes; config-only diff with no code-quality issues in scope. 🤖 Code Style Review Agent |
🎬 Video Proof Test ResultsBranch: Clicked Ask Angie to open the loadSidebarV2 sidebar on the product editor POC. 📸 ScreenshotsArtifacts are also available on the workflow run. |
PR Approver EvaluationPre-checks did not pass — auto-merge evaluation stopped. Reasons to auto-merge
Reasons not to auto-merge
VerdictWould auto-merge: No — pre-checks blocked evaluation. Fix the issues above, then push new commits to re-run. 🤖 PR Approver Agent |
…eviewers" This reverts commit a78594b.
🎬 Video Proof Test ResultsBranch: Clicked Ask Angie to open the loadSidebarV2 sidebar on the product editor POC. 📸 ScreenshotsArtifacts are also available on the workflow run. |
PR Approver EvaluationPre-checks did not pass — auto-merge evaluation stopped. Reasons to auto-merge
Reasons not to auto-merge
VerdictWould auto-merge: No — pre-checks blocked evaluation. Fix the issues above, then push new commits to re-run. 🤖 PR Approver Agent |
PR Approver EvaluationPR removes the videoproof task file and its CI workflow reference, plus removes workflow_dispatch trigger — CI workflow changes require human review per escalation policy. Reasons to auto-merge
Reasons not to auto-merge
VerdictWould auto-merge: No — human review is required. Please request review from rnd-ai-advanced@elementor.com. 🤖 PR Approver Agent |








Problem
angie-dev-tools#33 makes the PR description the only video proof task and requires a PR number. angie-sdk still passes
.github/videoproof-task.txtand allows video proof on workflow_dispatch, where there is no PR.Solution
Delete
.github/videoproof-task.txt, itsVIDEO_PROOF_TASK_FILEenv line, and theworkflow_dispatchtrigger ofvideoproof-ci.yml.Test
On http://127.0.0.1:8080/demo/load-sidebar-v2-full-config/ click Ask Angie (
#demo-toggle). Pass if the right-hand Angie sidebar opens, including a "Sign in to Angie" screen.Video proof fails on this PR until angie-dev-tools#33 merges, because angie-dev-tools master still requires the task file.
Jira: https://elementor.atlassian.net/browse/AI-9939