fix(uipath-platform): run queue/trigger seed e2e under docker, not tempdir#2113
Conversation
…mpdir queue-progress-e2e and trigger-api-e2e run seed.py, which calls `uip or folders`. Under `driver: tempdir` the CLI has no pre-baked tools and must auto-install @uipath/orchestrator-tool at runtime — which fails on the prerelease CLI the eval installs (resolves on `stable`, no stable build on its line; UiPath/cli#3083), so the seed dies before the task starts and both score 0 every run. role-lifecycle-e2e runs the same seed under docker and passes. Switch both to `driver: docker` so orchestrator-tool is pre-baked, matching the established pattern (the ixp e2e tasks use docker for the same reason). Fixes the two seed failures without depending on the CLI-side fix landing/publishing. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
|
Claude finished @alexenica's task in 1m 58s —— View job PR Review: fix(uipath-platform): run queue/trigger seed e2e under docker, not tempdir
SummarySwitches Change-by-Change Review1.
|
| Area | Status | Notes |
|---|---|---|
| Frontmatter | OK | No frontmatter changes; existing fields valid |
| E2E Tests | OK | Driver fix restores these two tasks to runnable state |
| Skill Body | N/A | No skill body changes |
| References & Assets | N/A | No reference/asset changes |
| Repo Hygiene | OK | No secrets, no unrelated changes, scoped to the two affected files |
Issues for Manual Review
None found.
Conclusion
Clean, well-scoped fix. The root cause analysis in the PR description is thorough — confirmed by checking that seed.py calls uip_json("or", "folders", ...) and that the sibling role_lifecycle_e2e (which runs the same seed under the default docker driver) already passes. The inline comment block documenting why docker is required is a nice touch for future maintainers. Approve.
|
Claude finished @alexenica's task in 2m 17s —— View job Coder-eval task lint (advisory)2 task YAMLs changed; verdict counts: 0 Critical, 1 High, 0 Medium, 0 Low, 2 OK. Rubric: .claude/commands/lint-task.md. This check is advisory and never blocks merge. Evidence of passing run❌ High — PR body does not claim the changed tasks have been run and passed under the new
Per-task lint
|
…mpdir (#2113) queue-progress-e2e and trigger-api-e2e run seed.py, which calls `uip or folders`. Under `driver: tempdir` the CLI has no pre-baked tools and must auto-install @uipath/orchestrator-tool at runtime — which fails on the prerelease CLI the eval installs (resolves on `stable`, no stable build on its line; UiPath/cli#3083), so the seed dies before the task starts and both score 0 every run. role-lifecycle-e2e runs the same seed under docker and passes. Switch both to `driver: docker` so orchestrator-tool is pre-baked, matching the established pattern (the ixp e2e tasks use docker for the same reason). Fixes the two seed failures without depending on the CLI-side fix landing/publishing. Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com>
What & why
skill-platform-queue-progress-e2eandskill-platform-trigger-api-e2efail their pre-run (seed.py) every run with:Root cause is the
sandbox.driver, not the seed. Both tasks setdriver: tempdir, which runs on a bare host with no pre-baked tool plugins.seed.pycallsuip or folders, so the CLI tries to auto-install@uipath/orchestrator-toolat runtime — which fails on the prerelease CLI the eval installs (it resolves tools onstable, and there's no stable build on its prerelease-only line; see UiPath/cli#3083). The pre-run dies before the task starts → score 0, deterministically.skill-platform-role-lifecycle-e2eruns the sameseed.pybut under the default docker driver, whereorchestrator-toolis pre-baked — and it passes. That's the tell.The change
Switch both tasks to
driver: docker, so the pre-bakedskills-image:latestprovidesorchestrator-tooland the seed'suip orcalls resolve with no runtime install. This matches the existing convention — theuipath-ixpe2e tasks already usedriver: dockerfor exactly this reason (uip ixpis pre-baked, not resolvable in tempdir).Why this over the CLI fix
UiPath/cli#3083 fixes the underlying prerelease resolution bug, but it only reaches these tasks after it merges and a fixed build is published and the eval installs it. This change fixes the two seed failures directly, now, with no cross-repo dependency. (#3083 is still worth landing as a real product bug.)
Verified by inspecting the 2026-07-15 run: of 778 tasks, only these two hit the error — the only two that both (a) run
seed.py'suip orin pre-run and (b) usedriver: tempdir.🤖 Generated with Claude Code