jzombie / term-wm / 28688331903

75%

Build Type

push

github

Committed by web-flow

Commit Message

feat(core): Formalize app window state machine (#81)

* feat(core): formalize window lifecycle as a state machine with
event-driven teardown

Replace ad-hoc `minimized: bool` with a canonical `WindowState` enum
(Realized/Mapped/Unmapped/Iconic/Shaded) and a single
`transition_window()`
method on WindowManager that atomically mutates state and applies all
z-order/layout/focus side-effects without holding borrows across calls.

Add `AppExited(WindowKey)` variant to `UnifiedEvent` for event-driven
async teardown. Replace `Arc<dyn Fn()>` wakeup callbacks with a
`PtyStatus` enum (Wakeup/Exited) and a `Box<dyn Fn(PtyStatus)>` status
callback, keeping the pty-engine crate agnostic of UI types. The
callback
is shared between reader thread and main thread via `Arc<Mutex<>>` so
it can be set after the reader thread starts.

Remove `has_exited()` polling from `enumerate_windows()` — windows now
close via the event-driven `AppExited` path. Fix `EventSource` trait
dispatch for `take_exited_windows()` so generic runner code actually
receives accumulated exit keys.

Add 10 tests covering all 8 legal state transitions and their
side-effects,
illegal transitions, close_window variants, and shade/unshade.

Coverage Stats

361 of 457 new or added lines in 11 files covered. (78.99%)

8 existing lines in 3 files now uncovered.

13422 of 19616 relevant lines covered (68.42%)

143.69 hits per line