Files
cleveragents-core/features
HAL9000 4768d6d6dd
CI / lint (pull_request) Successful in 58s
CI / typecheck (pull_request) Successful in 1m7s
CI / security (pull_request) Successful in 1m8s
CI / push-validation (pull_request) Successful in 37s
CI / build (pull_request) Successful in 51s
CI / helm (pull_request) Successful in 53s
CI / quality (pull_request) Successful in 1m17s
CI / unit_tests (pull_request) Successful in 5m31s
CI / docker (pull_request) Successful in 1m42s
CI / integration_tests (pull_request) Successful in 9m8s
CI / coverage (pull_request) Successful in 9m28s
CI / status-check (pull_request) Successful in 4s
fix(decisions): refactor ExecutePhaseDecisionHook to shared _record helper
The seven record_* methods previously each duplicated the validate +
snapshot + record + log + except pattern, inflating uncovered-line
counts and producing ruff-format violations. Centralise that boilerplate
in a single _record helper; each public method now delegates with just
the decision type + log label.

Also realigns features/execute_decision_recording.feature with its
steps: split combined Then steps, fix alt=/alts= mismatch, change
mentions "X" to mentions="X", and add a missing space in the
res_select scenario. Adds hardcoded steps for the empty-string error
cases (behave's `{q}` placeholder needs >=1 char), plus scenarios for
plan_id construction validation and whitespace-only inputs.

Fixes the CI / lint failure (ruff format) and the CI / unit_tests
failure (11 errored scenarios in execute_decision_recording.feature).

ISSUES CLOSED: #8477
2026-06-13 18:50:14 -04:00
..