refactor(common): enforce utils:: path, remove top-level re-exports

- Remove pub use utils::{id, path, text, debug} from lib.rs
- All call sites now use crab_common::utils::id::new_ulid() etc.
- 20 files updated across auth, bridge, cli, config, daemon, fs,
  process, session, telemetry, tui crates
- Follows Rust convention: module path reflects logical grouping

Author: lingcoder

crates/auth/src/oauth.rs

@@ -110,7 +110,7 @@ impl TokenStore {
 /// Return the default token file path: `~/.crab/auth/tokens.json`.
 #[must_use]
 pub fn default_token_path() -> PathBuf {
-    crab_common::path::home_dir()
+    crab_common::utils::path::home_dir()
         .join(".crab")
         .join(TOKEN_DIR)
         .join(TOKEN_FILE)

crates/bridge/src/repl_bridge.rs

@@ -75,7 +75,7 @@ impl ReplBridge {
             ));
         }
 
-        let id = ConnectionId::new(crab_common::id::new_ulid());
+        let id = ConnectionId::new(crab_common::utils::id::new_ulid());
         self.clients.push(ClientHandle {
             id: id.clone(),
             info,

crates/bridge/src/trusted_device.rs

@@ -37,7 +37,7 @@ impl TrustedDeviceRegistry {
 
     /// Load the registry from the config directory.
     pub fn load() -> crab_common::Result<Self> {
-        let path = crab_common::path::home_dir()
+        let path = crab_common::utils::path::home_dir()
             .join(".crab")
             .join("trusted_devices.json");
         if !path.exists() {
@@ -52,7 +52,7 @@ impl TrustedDeviceRegistry {
 
     /// Save the registry to the config directory.
     pub fn save(&self) -> crab_common::Result<()> {
-        let path = crab_common::path::home_dir()
+        let path = crab_common::utils::path::home_dir()
             .join(".crab")
             .join("trusted_devices.json");
         if let Some(parent) = path.parent() {

crates/cli/src/commands/chat.rs

@@ -167,7 +167,7 @@ pub async fn run_chat(config: ChatConfig) -> anyhow::Result<()> {
         None
     };
 
-    let session_id = crab_common::id::new_ulid();
+    let session_id = crab_common::utils::id::new_ulid();
 
     // 7. Build system prompt
     let registry = crab_tools::builtin::create_default_registry();

crates/cli/src/commands/run.rs

@@ -165,7 +165,7 @@ pub async fn run_once(config: RunConfig) -> anyhow::Result<()> {
     );
 
     let global_dir = settings::global_config_dir();
-    let session_id = crab_common::id::new_ulid();
+    let session_id = crab_common::utils::id::new_ulid();
 
     // 6. Build session config
     let session_config = SessionConfig {

crates/cli/src/commands/update.rs

@@ -81,7 +81,7 @@ struct VersionCheckCache {
 
 /// Path to the version check cache file.
 fn cache_path() -> PathBuf {
-    crab_common::path::home_dir()
+    crab_common::utils::path::home_dir()
         .join(".crab")
         .join("update-check-cache.json")
 }

crates/cli/src/main.rs

@@ -531,13 +531,13 @@ fn resolve_system_prompt(
 #[allow(clippy::too_many_lines)]
 async fn run(cli: &Cli, resume_session_id: Option<String>) -> anyhow::Result<()> {
     // Initialise debug/tracing if requested
-    let debug_filter = crab_common::debug::resolve_debug_filter(cli.debug.as_deref());
-    let debug_config = crab_common::debug::DebugConfig {
+    let debug_filter = crab_common::utils::debug::resolve_debug_filter(cli.debug.as_deref());
+    let debug_config = crab_common::utils::debug::DebugConfig {
         enabled: debug_filter.is_some() || cli.verbose,
         filter: debug_filter,
         file: cli.debug_file.clone(),
     };
-    crab_common::debug::init_debug(&debug_config);
+    crab_common::utils::debug::init_debug(&debug_config);
 
     let working_dir = std::env::current_dir().unwrap_or_else(|_| PathBuf::from("."));
 
@@ -678,9 +678,9 @@ async fn run(cli: &Cli, resume_session_id: Option<String>) -> anyhow::Result<()>
     let session_id = if let Some(ref id) = cli.session_id {
         id.clone()
     } else if cli.fork_session && effective_resume_id.is_some() {
-        crab_common::id::new_ulid()
+        crab_common::utils::id::new_ulid()
     } else {
-        crab_common::id::new_ulid()
+        crab_common::utils::id::new_ulid()
     };
 
     // Build allowed/denied tool lists from CLI flags

crates/cli/src/setup.rs

@@ -129,14 +129,14 @@ pub fn install_panic_hook() {
 
 /// Initialize the tracing/logging system.
 ///
-/// This delegates to `crab_common::debug::init_debug` which supports:
+/// This delegates to `crab_common::utils::debug::init_debug` which supports:
 /// - Console output to stderr (with level filtering)
 /// - Optional file output to `~/.crab/logs/`
 ///
 /// When `verbose` is true, enables `crab=debug` level logging.
 /// Otherwise, tracing stays at its default (warn+error only).
 pub fn init_logging(verbose: bool) {
-    let config = crab_common::debug::DebugConfig {
+    let config = crab_common::utils::debug::DebugConfig {
         enabled: verbose,
         filter: if verbose {
             Some("crab=debug".to_string())
@@ -145,7 +145,7 @@ pub fn init_logging(verbose: bool) {
         },
         file: if verbose { Some(log_file_path()) } else { None },
     };
-    crab_common::debug::init_debug(&config);
+    crab_common::utils::debug::init_debug(&config);
 }
 
 /// Run all initialization steps.
@@ -161,15 +161,15 @@ pub fn initialize(verbose: bool) {
 
 /// Return the crash log path: `~/.crab/logs/crash.log`.
 fn crash_log_path() -> PathBuf {
-    crab_common::path::home_dir()
+    crab_common::utils::path::home_dir()
         .join(".crab")
         .join("logs")
         .join("crash.log")
 }
 
 /// Return the debug log file path: `~/.crab/logs/debug.log`.
 fn log_file_path() -> PathBuf {
-    crab_common::path::home_dir()
+    crab_common::utils::path::home_dir()
         .join(".crab")
         .join("logs")
         .join("debug.log")

crates/common/src/lib.rs

@@ -4,9 +4,3 @@ pub mod utils;
 
 pub use error::Error;
 pub use result::Result;
-
-// Re-export utils at top level for backwards compatibility
-pub use utils::debug;
-pub use utils::id;
-pub use utils::path;
-pub use utils::text;

crates/config/src/config_toml.rs

@@ -49,7 +49,7 @@ pub struct ProviderProfile {
 /// `~/.config/crab-code/config.toml` (XDG-style).
 #[must_use]
 pub fn config_toml_path() -> PathBuf {
-    crab_common::path::home_dir()
+    crab_common::utils::path::home_dir()
         .join(".config")
         .join("crab-code")
         .join("config.toml")