refactor: implement DataProvider pattern for system screens

Implement DataProvider pattern for:
- VersionCheckScreen: No data needed (empty provider)
- PanicScreen: No data needed (empty provider)

These system screens don't require initialization data and use
empty providers.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>

Author: unhappychoice

src/presentation/game/screens/panic_screen.rs

@@ -1,8 +1,7 @@
 use crate::domain::events::EventBus;
-use crate::domain::models::{SessionResult, TotalResult};
 use crate::infrastructure::logging::get_current_log_file_path;
 use crate::presentation::game::events::NavigateTo;
-use crate::presentation::game::{Screen, UpdateStrategy};
+use crate::presentation::game::{RenderBackend, Screen, ScreenDataProvider, ScreenType, UpdateStrategy};
 use crate::presentation::ui::Colors;
 use crate::Result;
 use ratatui::{
@@ -154,11 +153,35 @@ impl PanicScreen {
     }
 }
 
+pub struct PanicScreenDataProvider;
+
+impl ScreenDataProvider for PanicScreenDataProvider {
+    fn provide(&self) -> Result<Box<dyn std::any::Any>> {
+        Ok(Box::new(()))
+    }
+}
+
 impl Screen for PanicScreen {
-    fn init(&mut self) -> Result<()> {
+    fn get_type(&self) -> ScreenType {
+        ScreenType::Panic
+    }
+
+    fn default_provider() -> Box<dyn ScreenDataProvider>
+    where
+        Self: Sized,
+    {
+        Box::new(PanicScreenDataProvider)
+    }
+
+    fn get_render_backend(&self) -> RenderBackend {
+        RenderBackend::Ratatui
+    }
+
+    fn init_with_data(&mut self, _data: Box<dyn std::any::Any>) -> Result<()> {
         Ok(())
     }
 
+
     fn handle_key_event(
         &mut self,
         key_event: crossterm::event::KeyEvent,
@@ -176,8 +199,6 @@ impl Screen for PanicScreen {
     fn render_crossterm_with_data(
         &mut self,
         _stdout: &mut Stdout,
-        _session_result: Option<&SessionResult>,
-        _total_result: Option<&TotalResult>,
     ) -> Result<()> {
         // This is a fallback - panic screen should use ratatui
         Ok(())

src/presentation/game/screens/version_check_screen.rs

@@ -1,8 +1,7 @@
 use crate::domain::events::EventBus;
-use crate::domain::models::{SessionResult, TotalResult};
 use crate::presentation::game::events::NavigateTo;
 use crate::presentation::game::views::VersionCheckView;
-use crate::presentation::game::{Screen, UpdateStrategy};
+use crate::presentation::game::{RenderBackend, Screen, ScreenDataProvider, ScreenType, UpdateStrategy};
 use crate::Result;
 use crossterm::{
     event::{self, Event, KeyCode},
@@ -69,7 +68,34 @@ impl VersionCheckScreen {
     }
 }
 
+pub struct VersionCheckScreenDataProvider;
+
+impl ScreenDataProvider for VersionCheckScreenDataProvider {
+    fn provide(&self) -> Result<Box<dyn std::any::Any>> {
+        Ok(Box::new(()))
+    }
+}
+
 impl Screen for VersionCheckScreen {
+    fn get_type(&self) -> ScreenType {
+        ScreenType::VersionCheck
+    }
+
+    fn default_provider() -> Box<dyn ScreenDataProvider>
+    where
+        Self: Sized,
+    {
+        Box::new(VersionCheckScreenDataProvider)
+    }
+
+    fn get_render_backend(&self) -> RenderBackend {
+        RenderBackend::Ratatui
+    }
+
+    fn init_with_data(&mut self, _data: Box<dyn std::any::Any>) -> Result<()> {
+        Ok(())
+    }
+
     fn handle_key_event(&mut self, key_event: event::KeyEvent) -> Result<()> {
         use crossterm::event::{KeyCode, KeyModifiers};
         match key_event.code {
@@ -88,8 +114,6 @@ impl Screen for VersionCheckScreen {
     fn render_crossterm_with_data(
         &mut self,
         _stdout: &mut Stdout,
-        _session_result: Option<&SessionResult>,
-        _total_result: Option<&TotalResult>,
     ) -> crate::Result<()> {
         // Version check is now handled by ScreenManager
         // let current_version = env!("CARGO_PKG_VERSION");