ci(ui-e2e-gate): wire the orphaned slop-removal cloud e2e runner (ratchet red on develop) - #11702
Conversation
The ui-e2e-runner-coverage ratchet went red on develop again right after #11646 merged: #11657 (cloud-surface unification, merged one minute earlier) added packages/ui/src/cloud/__e2e__/run-slop-removal-e2e.mjs with no packages/ui package.json script and no CI leg. Add test:slop-removal-e2e and a matching ui-e2e-gate.yml leg + artifact upload path. Runner verified green locally via the exact CI invocation (bun run --cwd packages/ui test:slop-removal-e2e): 30/30 checks, exit 0. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Your trial has ended. Reactivate Greptile to resume code reviews.
|
Claude encountered an error —— View job I'll analyze this and get back to you. |
Found during #11628 verification (follow-up to #11646).
Problem
The
packages/scripts/__tests__/ui-e2e-runner-coverage.test.tsratchet is red on develop again. #11646 (merged 2026-07-02 23:05:30Z) wired the credentials + frontend-hosting runners, but #11657 (merged 2026-07-02 23:04:19Z — one minute earlier, in-flight at the same time) added a third runner,packages/ui/src/cloud/__e2e__/run-slop-removal-e2e.mjs, with nopackages/uipackage.json script and no CI leg:Change (8 insertions, 2 files)
packages/ui/package.json: addtest:slop-removal-e2e(samebun run --cwd ../.. <runner>convention as the neighboring scripts)..github/workflows/ui-e2e-gate.yml: add a "Cloud settings surface (slop-removal) e2e" leg after the credentials leg, and addoutput-slop-removalto the artifact upload paths.Verification (all local, this branch on top of current develop)
bun test packages/scripts/__tests__/ui-e2e-runner-coverage.test.tsbun run --cwd packages/ui test:slop-removal-e2e(exact CI invocation)ALL CHECKS PASSED, 30/30 ✓ assertionsbun run --cwd packages/ui test:credentials-e2e(re-validated #11646 leg)ALL GREEN, 11/11 ✓ assertionsbun run --cwd packages/ui test:frontend-hosting-e2e(re-validated #11646 leg)ALL GREEN, 7/7 ✓ assertionsactionlint .github/workflows/ui-e2e-gate.ymlCommitted evidence outputs (
output-slop-removal/, RESULT.json, walkthrough.webm) were regenerated locally by these runs and deliberately restored to HEAD to keep the diff to the 8-line wiring change; the checked-in evidence from #11657 remains canonical.No other orphaned runners: the ratchet enumerates every
run-*.mjsunderpackages/ui/src/**/__e2e__and passes 1/1 with this change.🤖 Generated with Claude Code