test(e2e): workflow example 18 — container with remote repo clone (trusted profile) #820
No reviewers
Labels
No labels
auto/needs-reevaluation
controller-managed
auto/blocked-by-deps
auto/ci-timeout
auto/claimed-implementer
auto/claimed-merge
auto/claimed-reviewer
auto/driver-down
auto/invariant-violation
auto/last-attempt-tier-0
auto/last-attempt-tier-1
auto/last-attempt-tier-2
auto/last-attempt-tier-min
Automation Tracking
auto/needs-conflict-resolution
auto/needs-implementer
auto/postmortem
auto/ready-to-merge
auto/restart-throttled
auto/revert
auto/sentinel
auto/stale-inactivity
auto/unstable
Blocked
Bounty
$100
Bounty
$1000
Bounty
$10000
Bounty
$20
Bounty
$2000
Bounty
$250
Bounty
$50
Bounty
$500
Bounty
$5000
Bounty
$750
MoSCoW
Could have
MoSCoW
Must have
MoSCoW
Should have
Needs Feedback
Points
1
Points
13
Points
2
Points
21
Points
3
Points
34
Points
5
Points
55
Points
8
Points
88
Priority
Backlog
Priority
CI Blocker
Priority
Critical
Priority
High
Priority
Low
Priority
Medium
Signed-off: Owner
Signed-off: Scrum Master
Signed-off: Tech Lead
Spike
State
Completed
State
Duplicate
State
In Progress
State
In Review
State
Paused
State
Unverified
State
Verified
State
Wont Do
Type
Automation
Type
Bug
Type
Discussion
Type
Documentation
Type
Epic
Type
Feature
Type
Legendary
Type
Refactor
Type
Support
Type
Task
Type
Testing
No project
No assignees
2 participants
Notifications
Due date
No due date set.
Depends on
#627 Implement @tdd_expected_fail tag handling in Behave environment
cleveragents/cleveragents-core
#628 Implement @tdd_expected_fail tag handling in Robot Framework
cleveragents/cleveragents-core
Reference
cleveragents/cleveragents-core!820
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "test/e2e-wf18-container-clone"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Summary
E2E test for Specification Workflow Example 18: Container with Remote Repo Clone using the
trustedautomation profile. Tests the full container-instance workflow including--clone-into, plan-level execution environment withfallbackpriority, and the complete plan lifecycle.Closes #764
ISSUES CLOSED: #764
Changes
E2E Test (
robot/e2e/wf18_container_clone.robot)--clone-into: Registers acontainer-instanceresource with the--clone-into REPO_URL:CONTAINER_PATHflag per the spec (notgit-checkout).project createthenproject link-resourceto match the spec's demonstrated flow for Example 18.ANTHROPIC_API_KEYvsOPENAI_API_KEYand selects the appropriate actor (followingm6_acceptance.robotpattern).--execution-environment containerand--execution-env-priority fallbackwith--automation-profile trusted. Note: the spec shows--execution-environment cloud/build-env(a resource name), but the current CLI only supports enum values (hostorcontainer); resource-name-based resolution is future work.lifecycle-apply --yes: Uses--yesflag to skip the confirmation prompt in non-interactive (CI) execution, consistent with all other E2E tests.Run Keyword And Warn On Failureto handle LLM output non-determinism.TracebackandINTERNALerror checks on every step.deploy_manager.py,Dockerfile) still exist after the full lifecycle.[0-9A-HJKMNP-TV-Z]{26}.${RUN_SUFFIX}).Force Tags E2E: Suite-level tag instead of per-test[Tags].[Timeout] 20 minutes: Comfortable margin over cumulative step timeouts.[Teardown]: Per-test teardown that logs completion.plan executecalls (Strategize then Execute).Skip If No LLM Keys: Graceful skip when no LLM API keys are available.--format plain: All CLI commands use--format plainfor consistency.CLI Enhancement (
src/cleveragents/cli/commands/resource.py)--clone-intoflag onresource addcommand. AcceptsREPO_URL:CONTAINER_PATHformat, stores the value as aclone_intoproperty on the resource.--clone-intois only accepted forcontainer-instanceanddevcontainer-instanceresource types. Using it with other types emits a validation error.--clone-intovalue contains a:separator with non-empty segments on both sides. Usesrsplit(":", 1)to handle URLs with ports.Behave Tests (
features/resource_cli_coverage.feature)--clone-intoCLI coverage:resource add container-instancewith--clone-intostoresclone_intoproperty successfully.--clone-intoon non-container type (git-checkout) fails with validation error.--clone-intowith invalid format (no colon separator) fails with validation error.--clone-intowith missing path segment fails with validation error.CHANGELOG (
CHANGELOG.md)--clone-intois a new CLI flag introduced in this PR, with input validation and type restriction.Rebase and Fix Notes
master(commit5ddc04bc), resolving merge conflicts inCHANGELOG.mdandsrc/cleveragents/cli/commands/resource.py. Both the new--mountflag (from WF17/PR merged to master) and--clone-intoflag (from this PR) are preserved.--yesflag toplan lifecycle-applycommand call. The--yesflag was introduced to skip the confirmation prompt in non-interactive mode; all other E2E tests already use it. Without--yes, the command failed with rc=1 when run in CI (no terminal available for prompting).Manual Verification
Prerequisites
ANTHROPIC_API_KEYorOPENAI_API_KEYenvironment variable setCommands
What to Look For
nox -e lint,nox -e typecheck,nox -e unit_tests,nox -e integration_tests,nox -e coverage_reportall pass5505f1669d4e307bc57d4e307bc57d602cfbfd2d602cfbfd2dd840408f49d840408f490eccc8f3970eccc8f39770d9d17532PM Review — Day 34
Status: Mergeable, 0 reviews, M9 (v3.8.0)
Author: @freemo
E2E test for WF18 (container with remote repo clone, trusted profile). M9 — lowest urgency.
Action Items
PM Status — Day 36 (2026-03-16)
Day 34 review assignment deadline check. This PR has 0 reviewer activity after 2 days.
Priority note: M3 PRs take precedence. Reviewers should complete M3 reviews first, then address M4+ PRs in milestone order.
Assigned reviewer: Please acknowledge and provide an ETA for your review, or flag if reassignment is needed.
@hurui200320 I am going to have you take over this PR, it is mostly completed but is waiting on #628 and #966 One is yours and one is Brent's. Please be sure to get this PR and the two blocking PRs I listed in asap, thanks.
PM Status — Day 37
Reviewers assigned. This PR needs at least 2 approving reviews per
CONTRIBUTING.mdbefore merge.Author: Please ensure this PR is rebased on latest
masterand all quality gates pass before requesting merge.PM status — Day 37
70d9d17532310ae288f7Code Review — PR #820
(Cannot submit formal approval — self-authored PR.)
E2E test for WF18. Well-structured with proper labels, milestone, and issue linkage. No issues found.
310ae288f75e839505335e83950533c658b928c3c658b928c34f12abdf204f12abdf2013a51b149413a51b1494be37ef3dcbbe37ef3dcb92cb31306d92cb31306d7f5c376a647f5c376a643b47eccb433b47eccb431613d6f4ad1613d6f4ad9cace15d6e