bug(cli): automation_profile._get_service() bypasses DI container #1181

Merged
freemo merged 1 commits from bugfix/m4-automation-profile-di-bypass into master 2026-04-02 17:54:14 +00:00

1 Commits

Author SHA1 Message Date
brent.edwards 601f408860 bug(cli): automation_profile._get_service() bypasses DI container
CI / lint (pull_request) Successful in 23s
CI / typecheck (pull_request) Successful in 54s
CI / quality (pull_request) Successful in 3m44s
CI / security (pull_request) Successful in 4m5s
CI / build (pull_request) Successful in 17s
CI / helm (pull_request) Successful in 23s
CI / unit_tests (pull_request) Successful in 6m39s
CI / docker (pull_request) Successful in 1m23s
CI / coverage (pull_request) Successful in 13m11s
CI / e2e_tests (pull_request) Successful in 17m23s
CI / integration_tests (pull_request) Successful in 21m36s
CI / status-check (pull_request) Successful in 1s
CI / benchmark-publish (pull_request) Has been skipped
CI / benchmark-regression (pull_request) Failing after 36m14s
Route automation profile CLI service resolution through the DI container by using container.automation_profile_service() and wiring a dedicated automation_profile_service provider in the application container. This removes ad-hoc repository/session construction from the CLI path so dependency lifecycle and configuration stay centralized and consistent with specification DI rules.

Add integration-style, non-mocked Behave proof that _get_service() resolves through real container/provider wiring, while keeping focused delegation assertions in the existing coverage feature. Remove the unrelated robot/resource_dag.robot changes from this PR and clean up duplicate _get_service test overlap by dropping the extra TDD-only feature file pair.

ISSUES CLOSED: #990
2026-04-02 06:21:49 +00:00