test(e2e): workflow example 17 — explicit container with directory mount (trusted profile) #819
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!819
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "test/e2e-wf17-explicit-container"
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 17 — Explicit Container with Directory Mount using the trusted automation profile. Exercises the full container workflow: resource registration (git-checkout + container-instance), project linking via separate
link-resourcecalls (following spec's prescribed command sequence), execution environment setting via project context set, plan-level execution-env-priority override via plan use, and complete plan lifecycle with dynamic actor selection.Additionally includes TDD bug-capture tests for two deferred acceptance criteria where CLI features are not yet implemented, and a regression guard for the now-fixed dual mount registration (bug #1078).
Closes #763
ISSUES CLOSED: #763
What This PR Does
git-checkoutfor the code repo andcontainer-instance(with--image) for the container environment.project createthenproject link-resourcecalls for each resource, matching the spec's prescribed command sequence.project context set --execution-environment containerat the project level, with best-effort persistence verification viaproject context show(field extraction with graceful fallback if not exposed).plan use --execution-env-priority override(plan-level override, precedence level 1, used as a workaround sinceproject context setdoes not yet support--execution-env-priority).plan use→ strategize → execute → diff → lifecycle-apply → status with--automation-profile trusted.applyafter lifecycle-apply, matching M6'sFull Flow Apply Steppattern.execution_environmentfield from plan status JSON and asserts it equalscontainerwhen present; logs gap warning when absent.plan_idmatches canonical Crockford Base32 ULID pattern^[0-9A-HJKMNP-TV-Z]{26}$.ANTHROPIC_API_KEYis available, falls back toopenai/gpt-4o(consistent with M6 pattern).Skip If No LLM Keysguard on main test, workspace initialization viaagents init --force --yes.--mountflag implemented). Test converted fromtdd_expected_failto permanent regression guard withtdd_issue+tdd_issue_1078tags.WF17 TDD Project Level Execution Env Priority Override— AC #3, tracked in #1079 (tdd_expected_fail)WF17 TDD Precedence Level 2 Project Override Resolution— AC #5, tracked in #1080 (tdd_expected_fail)Acceptance Criteria Status
E2EForce Tags E2Ein Settings--mountnow on CLI.tdd_expected_failremoved, regression guard active.plan use. TDD test added, tracked in #1079execution_environmentfield extracted from plan status JSON and asserted ascontainer; full lifecycle completes successfullySkip If No LLM KeysguardKey Design Decisions
--mountnow available on CLI — Bug #1078 is fixed. Theresource add container-instance --mountcommand works. TDD test converted to regression guard.--yesflag onlifecycle-apply— Required for non-interactive execution in E2E tests. All other E2E tests use this flag. Added to WF17 to match the established pattern.--execution-env-priority overrideonproject context set, but the CLI only implements this flag onplan use. Tracked in #1079, #1080.--execution-environmentaccepts enum values (host/container), not resource names per spec — Known spec divergence documented inline.execution_environmentfield is extracted from plan status and asserted ascontainerwhen present. If absent, a warning is logged documenting the gap.project context showdoes not exposeexecution_environment— Persistence verification uses field extraction with graceful fallback.project createthenproject link-resourcecalls.Should Contain ${final_phase.lower()} apply.tdd_issue+tdd_issue_<N>per CONTRIBUTING.md (nottdd_bug).Manual Verification
Prerequisites
ANTHROPIC_API_KEYorOPENAI_API_KEYenvironment variable setCommands
What to Look For
--automation-profile trustedand--execution-env-priority overrideplan statusshows terminal apply phase andexecution_environment=containerTracebackorINTERNALin any command's stderrf82bbc85af2a1c15b60f2a1c15b60f81a1c6f7b781a1c6f7b7135891ec45135891ec45097cd71c9d097cd71c9d03e3481154PM Review — Day 34
Status: Mergeable, 0 reviews, M8 (v3.7.0)
Author: @freemo
E2E test for WF17 (explicit container with directory mount, trusted profile). M8 — low 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
03e3481154e89174e503Code Review — PR #819
(Cannot submit formal approval — self-authored PR.)
E2E test for WF17. Well-structured with proper labels, milestone, and issue linkage. No issues found.
e89174e503b0968d27b9b0968d27b9e720b34c57e720b34c5778f98c8e2478f98c8e24693f002555693f00255533293811fc33293811fc6b876e638a6b876e638a9a8a27241c9a8a27241c5eca78c6f25eca78c6f29c36c217229c36c217225ddc04bcc7