refactor: move find_project_root to the hm crate, drop DirProvider re-export

Delete crates/hm-common/src/dirs.rs. Its sole remaining function,
find_project_root, discovers the `.hm/` project root and is used only by
the hm crate (context.rs and the config_layered test), so it moves to
crates/hm/src/project.rs where it belongs.

Also drop the `pub use dir_provider::DirProvider` crate-root re-export:
every other hm-common type is module-qualified (hm_common::git::Git,
hm_common::fs::write_atomic), so callers now use
hm_common::dir_provider::DirProvider for consistency.

Author: markovejnovic

crates/hm-common/src/lib.rs

@@ -3,13 +3,10 @@
 #[cfg(feature = "sys-runtime")]
 pub mod sys_runtime;
 pub mod dir_provider;
-pub mod dirs;
 pub mod format;
 pub mod fs;
 pub mod git;
 pub mod process;
 pub mod python;
 pub mod string;
 pub mod time;
-
-pub use dir_provider::DirProvider;

crates/hm-config/src/creds.rs

@@ -1,7 +1,7 @@
 //! File-backed credential store at `~/.config/hm/credentials.toml`.
 //!
 //! The file is written with [`Privacy::Private`] (0o600, parent dir 0o700)
-//! via [`hm_common::fs::blocking::write_atomic`], keyed by `(service, account)`.
+//! via [`hm_common::fs::write_atomic`], keyed by `(service, account)`.
 
 use anyhow::{Context, Result};
 use serde::{Deserialize, Serialize};
@@ -15,7 +15,7 @@ struct CredentialFile {
 }
 
 fn path() -> Result<PathBuf> {
-    let dirs = hm_common::DirProvider::new().context("could not determine config directory")?;
+    let dirs = hm_common::dir_provider::DirProvider::new().context("could not determine config directory")?;
     Ok(dirs.config().join("hm").join("credentials.toml"))
 }
 

crates/hm-config/src/lib.rs

@@ -119,7 +119,7 @@ impl Config {
     ///
     /// Returns an error if the platform config directory cannot be determined.
     pub fn user_config_path() -> Result<PathBuf> {
-        let dirs = hm_common::DirProvider::new().context("could not determine config directory")?;
+        let dirs = hm_common::dir_provider::DirProvider::new().context("could not determine config directory")?;
         Ok(dirs.config().join("hm").join("config.toml"))
     }
 

crates/hm-exec/src/local/backend.rs

@@ -50,7 +50,7 @@ impl LocalBackend {
     /// (VM backend + snapshot registry) and registering the [`VmRunner`] as
     /// the default runner.
     fn build_registry(&self) -> Result<RunnerRegistry> {
-        let dirs = hm_common::DirProvider::new().ok_or_else(|| {
+        let dirs = hm_common::dir_provider::DirProvider::new().ok_or_else(|| {
             BackendError::Local("cannot resolve the Harmont cache directory".into())
         })?;
         let registry =

crates/hm/src/context.rs

@@ -28,7 +28,7 @@ impl RunContext {
     /// Returns an error if the config file is unreadable or malformed.
     pub fn from_cli(cli: &Cli) -> Result<Self> {
         let start_dir = std::env::current_dir().context("cannot determine current directory")?;
-        let project_root = hm_common::dirs::find_project_root(&start_dir);
+        let project_root = crate::project::find_project_root(&start_dir);
         let config = Config::load(project_root.as_deref())?;
 
         let output = OutputMode::Human {

crates/hm/src/lib.rs

@@ -3,10 +3,9 @@
     reason = "transitive dependency version conflicts in rand/windows-sys/thiserror chains; not fixable without upstream updates"
 )]
 // The `dirs` crate must NOT be added as a direct dependency of this
-// crate. All directory resolution goes through `hm_common::dirs`, which
-// owns the `dirs` dependency and provides both platform primitives and
-// Harmont-specific discovery. Adding `dirs` here would bypass that
-// single source of truth.
+// crate. Platform-directory resolution goes through
+// `hm_common::DirProvider`, which owns the `dirs` dependency. Adding
+// `dirs` here would bypass that single source of truth.
 
 #[allow(
     clippy::print_stdout,
@@ -24,5 +23,6 @@ pub use hm_config as config;
 /// `harmont_cli::creds_store` path.
 pub use hm_config::creds as creds_store;
 pub mod context;
+pub mod project;
 pub mod error;
 pub(crate) mod signal;

crates/hm-common/src/dirs.rs crates/hm/src/project.rscrates/hm-common/src/dirs.rs renamed to crates/hm/src/project.rs

@@ -1,8 +1,7 @@
-//! Project-directory discovery.
+//! The current Harmont project on disk.
 //!
-//! The `hm`-namespaced directory *roots* (config, cache, …) live in
-//! [`crate::dir_provider`]. This module handles the orthogonal problem of
-//! locating the current project by walking up from a starting path.
+//! A project is any directory tree rooted at a directory that contains `.hm/`;
+//! this module locates that root.
 
 #![allow(clippy::must_use_candidate)]
 

crates/hm/tests/config_layered.rs

@@ -85,7 +85,7 @@ fn load_resolves_project_root() {
     )
     .unwrap();
 
-    let found = hm_common::dirs::find_project_root(project_dir.path());
+    let found = harmont_cli::project::find_project_root(project_dir.path());
     assert_eq!(found, Some(project_dir.path().to_path_buf()));
 
     let config_path = harmont_cli::config::Config::project_config_path(project_dir.path());