chore: consolidate common

src/commands/league_data.rs

@@ -1,11 +1,35 @@
 //! League data command implementation
 
 use crate::{
-    core::league_settings_path, espn::cache_settings::load_or_fetch_league_settings, LeagueId,
-    Result, Season,
+    core::league_settings_path, error::EspnError,
+    espn::cache_settings::load_or_fetch_league_settings, LeagueId, Result, Season,
+    LEAGUE_ID_ENV_VAR,
 };
 
-use super::resolve_league_id;
+/// Resolve league ID from option or environment variable
+pub fn resolve_league_id(league_id: Option<LeagueId>) -> Result<LeagueId> {
+    match league_id {
+        Some(id) => Ok(id),
+        None => match std::env::var(LEAGUE_ID_ENV_VAR) {
+            Ok(env_id) => {
+                let parsed_id: u32 = env_id.parse().map_err(|_| EspnError::MissingLeagueId {
+                    env_var: LEAGUE_ID_ENV_VAR.to_string(),
+                })?;
+
+                if parsed_id == 0 {
+                    return Err(EspnError::MissingLeagueId {
+                        env_var: LEAGUE_ID_ENV_VAR.to_string(),
+                    });
+                }
+
+                Ok(LeagueId::new(parsed_id))
+            }
+            Err(_) => Err(EspnError::MissingLeagueId {
+                env_var: LEAGUE_ID_ENV_VAR.to_string(),
+            }),
+        },
+    }
+}
 
 /// Handle the league data command
 pub async fn handle_league_data(

src/commands/mod.rs

@@ -1,34 +1,8 @@
 //! Command implementations for ESPN Fantasy Football CLI
 
+pub mod common;
 pub mod league_data;
 pub mod player_data;
 pub mod player_filters;
 pub mod projection_analysis;
 pub mod update_all_data;
-
-use crate::{error::EspnError, LeagueId, Result, LEAGUE_ID_ENV_VAR};
-
-/// Resolve league ID from option or environment variable
-pub fn resolve_league_id(league_id: Option<LeagueId>) -> Result<LeagueId> {
-    match league_id {
-        Some(id) => Ok(id),
-        None => match std::env::var(LEAGUE_ID_ENV_VAR) {
-            Ok(env_id) => {
-                let parsed_id: u32 = env_id.parse().map_err(|_| EspnError::MissingLeagueId {
-                    env_var: LEAGUE_ID_ENV_VAR.to_string(),
-                })?;
-
-                if parsed_id == 0 {
-                    return Err(EspnError::MissingLeagueId {
-                        env_var: LEAGUE_ID_ENV_VAR.to_string(),
-                    });
-                }
-
-                Ok(LeagueId::new(parsed_id))
-            }
-            Err(_) => Err(EspnError::MissingLeagueId {
-                env_var: LEAGUE_ID_ENV_VAR.to_string(),
-            }),
-        },
-    }
-}