conductor(tracks): link v2 to 4 sub-track specs and plans
This commit is contained in:
+6
-2
@@ -396,8 +396,12 @@ User review surfaced five outstanding UI issues, each previously attempted witho
|
||||
*Goal: Resolve the 3 remaining live_gui failures (269/272 → 271/272 plus 1 new regression unit test). 1-line src fix in `_capture_workspace_profile` (change `ini=b""` to `ini=""` to satisfy `WorkspaceProfile.ini_content: str` contract that `tomli_w` enforces); the `b""` sentinel was a regression from `d7487af4` that caused `save_workspace_profile` to raise `TypeError`, profile never saved, `load_workspace_profile` became a no-op. 1 new unit test (`tests/test_workspace_profile_serialization.py`) encoding the str/bytes contract. `test_prior_session_no_pop_imbalance` is **deferred to a separate follow-up track** — the test was more under-mocked than the spec assumed; fixing imscope.window tuple-return only revealed the next un-mocked dependency (imgui.begin returning bool where 2-tuple expected at line 4496). `render_main_interface` is a kitchen-sink function requiring 50+ mocks; a follow-up track will either add the missing mocks or refactor the test to exercise a narrow prior-session render path. Change 4 (doc hardening of defer-not-catch sections) deferred to track end; not done due to scope focus.*
|
||||
|
||||
- [~] **Track: Live-GUI Test Hardening v2 (post v1 ship)** `[in progress]`
|
||||
*Link: [./tracks/live_gui_test_hardening_v2_20260605/](./tracks/live_gui_test_hardening_v2_20260605/), Spec: TBD, Plan: TBD*
|
||||
*Goal: Resolve the 4 remaining live_gui failures (was 3 in v1; 1 new regression). v1 fixed the str/bytes sentinel bug but exposed a deeper App/Controller state-sync bug. Plan: 4 sub-tasks - (1) App/Controller state sync fix + regression test, (2) test wait-for-ready pattern migration for set/save/load, (3) prior_session test mock setup or refactor, (4) undo_redo_lifecycle investigation + fix. Per-writing-plans skill.*
|
||||
*Link: [./tracks/live_gui_test_hardening_v2_20260605/](./tracks/live_gui_test_hardening_v2_20260605/)*
|
||||
*Goal: Resolve the 4 remaining live_gui failures (was 3 in v1; 1 new regression). v1 fixed the str/bytes sentinel bug but exposed a deeper App/Controller state-sync bug. Decomposed into 4 sub-tracks, each with its own spec+plan:*
|
||||
*Sub-track 1: live_gui_state_sync_20260605 - Spec: [./../../docs/superpowers/specs/2026-06-05-live-gui-state-sync-design.md](./../../docs/superpowers/specs/2026-06-05-live-gui-state-sync-design.md), Plan: [./../../docs/superpowers/plans/2026-06-05-live-gui-state-sync.md](./../../docs/superpowers/plans/2026-06-05-live-gui-state-sync.md). 9 sync-bug fields fixed via @property delegation (ui_ai_input, show_windows, ui_separate_tier1..4, ui_separate_task_dag, ui_separate_usage_analytics).*
|
||||
*Sub-track 2: prior_session_test_harden_20260605 - Spec: [./../../docs/superpowers/specs/2026-06-05-prior-session-test-harden-design.md](./../../docs/superpowers/specs/2026-06-05-prior-session-test-harden-design.md), Plan: [./../../docs/superpowers/plans/2026-06-05-prior-session-test-harden.md](./../../docs/superpowers/plans/2026-06-05-prior-session-test-harden.md). Refactor test to call narrow render_prior_session_view.*
|
||||
*Sub-track 3: wait_for_ready_test_pattern_20260605 - Spec: [./../../docs/superpowers/specs/2026-06-05-wait-for-ready-test-pattern-design.md](./../../docs/superpowers/specs/2026-06-05-wait-for-ready-test-pattern-design.md), Plan: [./../../docs/superpowers/plans/2026-06-05-wait-for-ready-test-pattern.md](./../../docs/superpowers/plans/2026-06-05-wait-for-ready-test-pattern.md). Replace time.sleep with polling helpers in 2 live_gui tests.*
|
||||
*Sub-track 4: undo_redo_lifecycle_fix_20260605 - Spec: [./../../docs/superpowers/specs/2026-06-05-undo-redo-lifecycle-fix-design.md](./../../docs/superpowers/specs/2026-06-05-undo-redo-lifecycle-fix-design.md), Plan: [./../../docs/superpowers/plans/2026-06-05-undo-redo-lifecycle-fix.md](./../../docs/superpowers/plans/2026-06-05-undo-redo-lifecycle-fix.md). 3-phase investigation: state-sync, snapshot, flake.*
|
||||
|
||||
*Failing tests:*
|
||||
- `test_auto_switch_sim` (still fails from v1) - **Deeper bug: App/Controller state sync**. The test does `set_value('ui_separate_tier1', True)` which goes to `controller.ui_separate_tier1`, but the save reads from `app.ui_separate_tier1`. Two different objects; the saved profile has the wrong value. Same root cause for `show_windows['Diagnostics']`.
|
||||
|
||||
Reference in New Issue
Block a user