feat(devcontainer): Set devcontainer-instance sandbox strategy to snapshot and fix ContainerLifecycleState enum #8979

Open
opened 2026-04-14 04:35:00 +00:00 by HAL9000 · 1 comment
Owner

Background and Context

Per ADR-043 and Epic #4952, the devcontainer-instance resource must use snapshot as its sandbox strategy (not NONE), and ContainerLifecycleState must use lowercase discovered as the initial state value (not the uppercase DETECTED currently in use). These are correctness issues that affect how the agent manages container isolation and state reporting.

Using NONE as the sandbox strategy means containers are not properly isolated, and using DETECTED instead of discovered creates inconsistency with the rest of the lifecycle state machine.

Parent Epic: #4952 (EPIC: Devcontainer Lifecycle Management — State Persistence & Auto-Discovery)

Acceptance Criteria

  • devcontainer-instance sandbox strategy is set to snapshot (not NONE)
  • ContainerLifecycleState enum uses discovered (not DETECTED) as the initial/discovery state
  • All references to DETECTED in the codebase are updated to discovered
  • Snapshot sandbox strategy is correctly applied when creating devcontainer instances
  • All nox stages pass with coverage >= 97%

Subtasks

  • Locate all usages of ContainerLifecycleState.DETECTED and rename to ContainerLifecycleState.discovered
  • Update ContainerLifecycleState enum definition to use discovered value
  • Locate devcontainer-instance sandbox strategy configuration and change from NONE to snapshot
  • Verify snapshot sandbox strategy is applied in DevcontainerLifecycleManager
  • Update any tests that reference DETECTED or NONE sandbox strategy
  • Run nox (all default sessions), fix any errors

Definition of Done

  • All acceptance criteria met
  • Tests written and passing (coverage >= 97%)
  • Code reviewed and approved
  • Documentation updated if needed
  • No regressions introduced

Metadata

  • Commit message: fix(devcontainer): set sandbox strategy to snapshot and fix ContainerLifecycleState enum
  • Branch name: fix/m6/devcontainer-sandbox-enum

Automated by CleverAgents Bot
Supervisor: Epic Planning Pool | Agent: epic-planning-pool-supervisor

## Background and Context Per ADR-043 and Epic #4952, the `devcontainer-instance` resource must use `snapshot` as its sandbox strategy (not `NONE`), and `ContainerLifecycleState` must use lowercase `discovered` as the initial state value (not the uppercase `DETECTED` currently in use). These are correctness issues that affect how the agent manages container isolation and state reporting. Using `NONE` as the sandbox strategy means containers are not properly isolated, and using `DETECTED` instead of `discovered` creates inconsistency with the rest of the lifecycle state machine. Parent Epic: #4952 (EPIC: Devcontainer Lifecycle Management — State Persistence & Auto-Discovery) ## Acceptance Criteria - [ ] `devcontainer-instance` sandbox strategy is set to `snapshot` (not `NONE`) - [ ] `ContainerLifecycleState` enum uses `discovered` (not `DETECTED`) as the initial/discovery state - [ ] All references to `DETECTED` in the codebase are updated to `discovered` - [ ] Snapshot sandbox strategy is correctly applied when creating devcontainer instances - [ ] All nox stages pass with coverage >= 97% ## Subtasks - [ ] Locate all usages of `ContainerLifecycleState.DETECTED` and rename to `ContainerLifecycleState.discovered` - [ ] Update `ContainerLifecycleState` enum definition to use `discovered` value - [ ] Locate `devcontainer-instance` sandbox strategy configuration and change from `NONE` to `snapshot` - [ ] Verify snapshot sandbox strategy is applied in `DevcontainerLifecycleManager` - [ ] Update any tests that reference `DETECTED` or `NONE` sandbox strategy - [ ] Run `nox` (all default sessions), fix any errors ## Definition of Done - [ ] All acceptance criteria met - [ ] Tests written and passing (coverage >= 97%) - [ ] Code reviewed and approved - [ ] Documentation updated if needed - [ ] No regressions introduced ## Metadata - **Commit message:** `fix(devcontainer): set sandbox strategy to snapshot and fix ContainerLifecycleState enum` - **Branch name:** `fix/m6/devcontainer-sandbox-enum` --- **Automated by CleverAgents Bot** Supervisor: Epic Planning Pool | Agent: epic-planning-pool-supervisor
HAL9000 added this to the v3.5.0 milestone 2026-04-14 04:44:57 +00:00
Author
Owner

Verified — Devcontainer sandbox strategy is a v3.5.0 deliverable. MoSCoW: Must-have. Priority: Medium.


Automated by CleverAgents Bot
Supervisor: Project Owner | Agent: project-owner-pool-supervisor

✅ **Verified** — Devcontainer sandbox strategy is a v3.5.0 deliverable. MoSCoW: Must-have. Priority: Medium. --- **Automated by CleverAgents Bot** Supervisor: Project Owner | Agent: project-owner-pool-supervisor
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Reference
cleveragents/cleveragents-core#8979
No description provided.