fix(resources): add overlay to ResourceTypeConfigSchema _VALID_STRATEGIES #3235

Merged
freemo merged 1 commits from fix/resource-schema-overlay-strategy-validation into master 2026-04-05 21:11:59 +00:00

1 Commits

Author SHA1 Message Date
freemo dbf3b8d7f7 fix(resources): remove overlay from SandboxStrategy enum - not in spec
CI / lint (pull_request) Successful in 27s
CI / typecheck (pull_request) Successful in 51s
CI / quality (pull_request) Successful in 38s
CI / security (pull_request) Successful in 58s
CI / build (pull_request) Successful in 29s
CI / helm (pull_request) Successful in 23s
CI / unit_tests (pull_request) Successful in 6m46s
CI / e2e_tests (pull_request) Successful in 18m2s
CI / integration_tests (pull_request) Successful in 22m24s
CI / docker (pull_request) Successful in 1m20s
CI / coverage (pull_request) Successful in 11m11s
CI / status-check (pull_request) Successful in 1s
CI / benchmark-publish (pull_request) Has been skipped
CI / benchmark-regression (pull_request) Successful in 57m8s
Add 'overlay' to _VALID_STRATEGIES frozenset in ResourceTypeConfigSchema
validator so that resource type YAML configs specifying
sandbox_strategy: overlay are accepted without raising ValueError.

The overlay strategy is defined in the specification for fs-mount
resources using OverlayFS. The domain model SandboxStrategy enum
correctly includes OVERLAY = 'overlay', but the YAML schema validator
in src/cleveragents/resource/schema.py was missing it from
_VALID_STRATEGIES, causing a validation error on valid configurations.

Changes:
- Add 'overlay' to _VALID_STRATEGIES in src/cleveragents/resource/schema.py
- Add BDD scenario: 'Schema accepts overlay sandbox strategy'
- Add step definitions for the new scenario

ISSUES CLOSED: #2827
2026-04-05 08:04:27 +00:00