feat(tui): implement PersonaRegistry with YAML persistence and cycle_persona() method [RETRY] #10630

Open
HAL9000 wants to merge 4 commits from feat/tui-v370/persona-registry-merge-v2 into master

4 Commits

Author SHA1 Message Date
controller-ci-rerun 75a715ca2f chore: re-trigger CI [controller]
CI / lint (pull_request) Failing after 58s
CI / typecheck (pull_request) Successful in 1m12s
CI / push-validation (pull_request) Successful in 20s
CI / helm (pull_request) Successful in 27s
CI / quality (pull_request) Successful in 47s
CI / build (pull_request) Successful in 37s
CI / security (pull_request) Successful in 1m17s
CI / coverage (pull_request) Has been skipped
CI / unit_tests (pull_request) Failing after 4m33s
CI / docker (pull_request) Has been skipped
CI / e2e_tests (pull_request) Failing after 13m19s
CI / integration_tests (pull_request) Failing after 13m27s
CI / status-check (pull_request) Has been cancelled
2026-06-11 00:07:28 -04:00
HAL9000 5947510746 fix(tests): remove duplicate step definitions and fix quoted step pattern in tui_persona_cycle_steps
CI / lint (pull_request) Failing after 1m6s
CI / quality (pull_request) Successful in 1m24s
CI / typecheck (pull_request) Successful in 1m27s
CI / security (pull_request) Successful in 1m30s
CI / coverage (pull_request) Has been skipped
CI / helm (pull_request) Successful in 28s
CI / push-validation (pull_request) Successful in 24s
CI / build (pull_request) Successful in 49s
CI / integration_tests (pull_request) Successful in 4m0s
CI / e2e_tests (pull_request) Successful in 3m33s
CI / unit_tests (pull_request) Failing after 5m58s
CI / docker (pull_request) Has been skipped
CI / status-check (pull_request) Failing after 15s
Resolves AmbiguousStep errors caused by duplicate step definitions in
tui_persona_cycle_steps.py that conflicted with tui_persona_system_steps.py:
- Removed duplicate "a temporary TUI persona registry" Given step
- Removed duplicate 'I set active persona to ...' When step
- Removed duplicate 'active persona for session ...' Then step

Also fixed the step pattern for "the registry last persona should be set to"
to use quoted capture group {persona_name} matching the feature file syntax.
2026-04-23 12:08:37 +00:00
HAL9000 77b48a76df fix(tests): resolve ambiguous step definition in persona state coverage tests
CI / push-validation (pull_request) Successful in 36s
CI / helm (pull_request) Successful in 40s
CI / lint (pull_request) Failing after 1m8s
CI / build (pull_request) Successful in 3m52s
CI / quality (pull_request) Successful in 4m28s
CI / typecheck (pull_request) Successful in 4m40s
CI / security (pull_request) Successful in 4m55s
CI / coverage (pull_request) Has been skipped
CI / unit_tests (pull_request) Failing after 5m12s
CI / docker (pull_request) Has been skipped
CI / e2e_tests (pull_request) Successful in 7m52s
CI / integration_tests (pull_request) Successful in 7m57s
CI / status-check (pull_request) Failing after 4s
- Rename duplicate step 'the registry last persona should be set to' to 'the mock registry last persona should be set to' in tui_persona_state_coverage_steps.py
- Update corresponding feature file to use the new step name
- Fixes AmbiguousStep error that was preventing unit tests from running
2026-04-18 19:54:26 +00:00
freemo a650d307e1 feat(tui): implement PersonaRegistry with YAML load/save/list/cycle and PersonaState.cycle_persona()
CI / lint (pull_request) Failing after 59s
CI / push-validation (pull_request) Successful in 24s
CI / helm (pull_request) Successful in 55s
CI / build (pull_request) Successful in 3m41s
CI / quality (pull_request) Successful in 4m14s
CI / unit_tests (pull_request) Failing after 4m20s
CI / typecheck (pull_request) Successful in 4m33s
CI / security (pull_request) Successful in 5m13s
CI / coverage (pull_request) Has been skipped
CI / docker (pull_request) Has been skipped
CI / e2e_tests (pull_request) Successful in 7m2s
CI / integration_tests (pull_request) Successful in 7m44s
CI / status-check (pull_request) Failing after 4s
2026-04-18 18:40:43 +00:00