jzombie / term-wm / 28817147915

75%

Build Type

push

github

Committed by web-flow

Commit Message

refactor(component): Unify component trait, window ownership, and action-based event flow (#83)

* refactor(component): Unify component trait, window ownership, and
action-based event flow

Major architectural refactor that advances the component system toward
the shared Component<T> trait design:

- Generalize Component trait to Component<TermWmAction> with associated
  action type. render() takes &self (immutable) instead of &mut self.
- Add new lifecycle methods: update() for action processing, destroy()
  for cleanup. Replace handle_event returning bool with on_mouse/on_key
  returning EventResult<TermWmAction>, enabling declarative action flow.
- Introduce interior mutability (RefCell/Cell) in TextRendererComponent
  to support immutable render() contract.
- Move component ownership into WindowManager via create_window(Box<dyn
  Component<TermWmAction>>), eliminating the App-level
  BTreeMap<WindowKey,
  ScrollViewComponent<TerminalComponent>> and simplifying lifecycle
  mgmt.
- Bridge WindowKey gap with Arc<Mutex<Option<WindowKey>>> in PTY status
  callbacks so wakeup/exit events can reference the correct window.
- Add HitboxRegistry parameter to render() for hit-testing support.
- Update SelectionViewport trait: selection_viewport() and
  logical_position_from_point() now take area parameter instead of
  using stored content_area.
- Remove WmMenuAction alias; use TermWmAction directly throughout.
- Relocate research docs from .opencode/plans/ to docs/research/.
- Update all examples, integration tests, and downstream components.

* Replace bare subtraction with `checked_sub` call

* Patch doc formatting

* Update comments

* cargo fmt --all

Coverage Stats

2783 of 4276 new or added lines in 46 files covered. (65.08%)

57 existing lines in 22 files now uncovered.

15409 of 22175 relevant lines covered (69.49%)

167.74 hits per line