light_fw_cli_core/

add.rs

//! Логика команды `light add`: план и применение добавления компонента.
//!
//! Разделение на [`plan_add`] (только чтение) и [`apply_add`] (мутации) даёт
//! `--dry-run` «бесплатно» и делает проверки тестируемыми без записи на диск.
//! Перед мутациями создаётся резервная копия; при любой ошибке (включая провал
//! `cargo check`) проект восстанавливается.

use std::path::Path;

use crate::error::{Error, Result};
use crate::fetch::{Catalog, DependencyEntry, ResolvedComponent, SourceKind};
use crate::manifest::{
    collect_dependency_details, has_package_section, read_cargo_toml, resolve_dependency_version,
};
use crate::process::CommandRunner;
use crate::{backup, cargo as cargo_cmd, edit, lockfile::Lockfile};

/// Опции команды `light add`.
#[derive(Debug, Clone)]
pub struct AddOptions {
    /// Явная версия (`--version`).
    pub version: Option<String>,
    /// Не задавать вопросов (`--yes`).
    pub yes: bool,
    /// Только показать план (`--dry-run`).
    pub dry_run: bool,
    /// Без сетевого доступа (`--offline`).
    pub offline: bool,
    /// Фича, к которой привязать зависимость (`--feature`).
    pub feature: Option<String>,
    /// Фичи самого добавляемого крейта (`--features`).
    pub features: Vec<String>,
    /// Не запускать `cargo check` (`--no-verify`).
    pub no_verify: bool,
    /// Уважать дефолтные фичи мета-крейта `light-fw` при добавлении компонентов.
    ///
    /// Всегда `true`: автоподгрузка (`light build`/`light dev`) и ручной `light add`
    /// никогда не отключают `default-features` у `light-fw`. Поле существует, чтобы
    /// опция передавалась явно (и её нельзя было случайно выключить через
    /// `Default::default()`), — см. ТЗ 0.17.2. Значение `false` допустимо, но
    /// ничего не меняет: `edit.rs` никогда не выносит `default-features = false`.
    pub default_features: bool,
}

// Ручная реализация `Default`: все флаги — `false`, кроме `default_features`,
// который обязан быть `true`, чтобы ни один вызов через `Default::default()`
// не мог случайно отключить дефолтные фичи `light-fw` (ТЗ 0.17.2).
impl Default for AddOptions {
    fn default() -> Self {
        Self {
            version: None,
            yes: false,
            dry_run: false,
            offline: false,
            feature: None,
            features: Vec::new(),
            no_verify: false,
            default_features: true,
        }
    }
}

/// Компонент, который нужно установить дополнительно (из `requires`).
#[derive(Debug, Clone)]
pub struct RequiredComponent {
    /// Имя крейта.
    pub name: String,
    /// Как записать зависимость.
    pub dependency: DependencyEntry,
}

/// Подготовленный план добавления компонента.
#[derive(Debug, Clone)]
pub struct AddPlan {
    /// Разрешённый компонент.
    pub component: ResolvedComponent,
    /// Как записать зависимость.
    pub dependency: DependencyEntry,
    /// Фича, к которой привязывается зависимость.
    pub feature: String,
    /// Фичи самого крейта (`--features`).
    pub crate_features: Vec<String>,
    /// Фичи мета-крейта `light-fw`, которые нужно включить.
    pub light_features: Vec<String>,
    /// Будет ли фича создана (иначе — уже существует).
    pub creates_feature: bool,
    /// Компонент уже установлен (повторный `add` — не ошибка).
    pub already_installed: bool,
    /// Дополнительные компоненты из `requires`.
    pub requires: Vec<RequiredComponent>,
}

impl AddPlan {
    /// Человекочитаемое описание плана.
    pub fn to_text(&self) -> String {
        let mut out = String::new();
        out.push_str(&format!("План добавления {}\n", self.component.name));
        let location = match &self.component.path {
            Some(p) => format!(" ({})", p.display()),
            None => String::new(),
        };
        out.push_str(&format!(
            "  источник:    {}{}\n",
            self.component.source.as_str(),
            location,
        ));
        out.push_str(&format!("  версия:      {}\n", self.component.version));
        if self.already_installed {
            out.push_str("  статус:      уже установлен\n");
            return out;
        }
        out.push_str(&format!(
            "  фича:        {}{}\n",
            self.feature,
            if self.creates_feature {
                " (будет создана)"
            } else {
                ""
            }
        ));
        out.push_str(&format!(
            "  зависимость: {}\n",
            dependency_line(&self.component.name, &self.dependency)
        ));
        if !self.crate_features.is_empty() {
            out.push_str(&format!(
                "  фичи крейта: {}\n",
                self.crate_features.join(", ")
            ));
        }
        if !self.light_features.is_empty() {
            out.push_str(&format!(
                "  фичи light:  {}\n",
                self.light_features.join(", ")
            ));
        }
        for req in &self.requires {
            out.push_str(&format!(
                "  также нужно: {} — {}\n",
                req.name,
                dependency_line(&req.name, &req.dependency)
            ));
        }
        out
    }
}

/// Результат применения плана.
#[derive(Debug, Clone)]
pub struct AddOutcome {
    /// Установленный компонент.
    pub component: ResolvedComponent,
    /// Фича, к которой привязана зависимость.
    pub feature: String,
    /// Дополнительно установленные компоненты.
    pub added_components: Vec<String>,
    /// Запускалась ли проверка сборки.
    pub verified: bool,
}

/// Строит план добавления компонента `name` в проект `project_root`.
pub fn plan_add(project_root: &Path, name: &str, opts: &AddOptions) -> Result<AddPlan> {
    let catalog = Catalog::discover(project_root, opts.offline);
    plan_add_with_catalog(project_root, name, opts, &catalog)
}

/// Строит план с явно заданным каталогом (подменяемый crates.io для тестов).
pub fn plan_add_with_catalog(
    project_root: &Path,
    name: &str,
    opts: &AddOptions,
    catalog: &Catalog,
) -> Result<AddPlan> {
    let cargo_path = project_root.join("Cargo.toml");
    if !cargo_path.is_file() {
        return Err(Error::Manifest(format!(
            "Cargo.toml не найден в {}",
            project_root.display()
        )));
    }
    let text = read_cargo_toml(&cargo_path)?;
    if !has_package_section(&text)? {
        return Err(Error::Manifest(
            "Cargo.toml не содержит секции [package]: укажите каталог приложения".to_string(),
        ));
    }

    let installed = collect_dependency_details(&text, project_root)?;
    let installed_names: Vec<&str> = installed.iter().map(|d| d.name.as_str()).collect();

    if installed_names.contains(&name) {
        return already_installed_plan(project_root, &text, name, opts, catalog);
    }

    let mut component = catalog.resolve(name)?;
    if let Some(version) = &opts.version {
        component.version = semver::Version::parse(version)?;
    }

    // Обёртка переименована: новый проект зависит от `light-fw`; старые
    // (опубликованные) проекты могут всё ещё использовать `light` — обе
    // версии дают версию ядра.
    let core = crate::manifest::resolve_core_version(project_root)?;
    if let Some(core) = &core
        && !component.core.matches(core)
    {
        // Ядро проекта несовместимо с версией из реестра (например, проект на
        // 0.16, а реестр требует ^0.17). Не падаем сразу: пробуем найти свежую
        // совместимую версию компонента — сначала в локальном workspace/vendor,
        // затем на crates.io (через `remote`, без сети возвращает `None`).
        // Если ничего нет — выдаём понятную ошибку с инструкцией.
        let fresh = catalog
            .resolve_local(name)?
            .and_then(|c| c.core.matches(core).then_some(c))
            .or_else(|| catalog.resolve_remote_latest(name).ok().flatten());
        component = match fresh {
            Some(fresh) => fresh,
            None => {
                let mv = opts
                    .version
                    .clone()
                    .unwrap_or_else(|| component.version.to_string());
                return Err(Error::Incompatible {
                    component: name.to_string(),
                    reason: format!(
                        "Компонент {name} требует ядро {}, но в проекте {core}. \
                         Обновите проект до версии ядра, совместимой с {}, \
                         или установите компонент вручную: light add {name}@{mv}",
                        component.core, component.core,
                    ),
                });
            }
        };
    }

    for conflict in &component.conflicts {
        if installed_names.iter().any(|n| n == conflict) {
            return Err(Error::Conflict(format!(
                "{name} несовместим с установленным {conflict}"
            )));
        }
    }

    let feature = match &opts.feature {
        Some(feature) => feature.clone(),
        None => choose_feature(&text, name)?,
    };
    let creates_feature = !edit::has_feature(&text, &feature)?;

    let dependency = catalog.dependency_entry(&component)?;

    let mut requires = Vec::new();
    for requirement in &component.requires {
        let (dep_name, _) = split_requirement(requirement);
        if is_core_name(dep_name) || installed_names.contains(&dep_name) {
            continue;
        }
        if requires
            .iter()
            .any(|r: &RequiredComponent| r.name == dep_name)
        {
            continue;
        }
        let resolved = catalog.resolve(dep_name)?;
        requires.push(RequiredComponent {
            name: dep_name.to_string(),
            dependency: catalog.dependency_entry(&resolved)?,
        });
    }

    let light_features = component.light_features.clone();
    Ok(AddPlan {
        component,
        dependency,
        crate_features: opts.features.clone(),
        light_features,
        feature,
        creates_feature,
        already_installed: false,
        requires,
    })
}

/// План для уже установленного компонента: ничего не меняет, но и не ошибка.
fn already_installed_plan(
    project_root: &Path,
    text: &str,
    name: &str,
    opts: &AddOptions,
    catalog: &Catalog,
) -> Result<AddPlan> {
    let current = resolve_dependency_version(project_root, name)?;
    if let (Some(requested), Some(current)) = (&opts.version, &current)
        && current.to_string() != *requested
    {
        return Err(Error::Conflict(format!(
            "{name} уже установлен (версия {current}), запрошена {requested}"
        )));
    }
    let version = current.unwrap_or_else(|| semver::Version::new(0, 0, 0));
    let info = catalog.registry().by_name(name);
    let local = catalog.resolve_local(name)?;
    let (source, path) = match &local {
        Some(resolved) => (resolved.source, resolved.path.clone()),
        None => (SourceKind::CratesIo, None),
    };
    let component = ResolvedComponent {
        name: name.to_string(),
        version: version.clone(),
        core: info
            .map(|i| i.core_requirement.clone())
            .unwrap_or(semver::VersionReq::STAR),
        light_features: info.map(|i| i.light_features.clone()).unwrap_or_default(),
        provides: info.map(|i| i.provides.clone()).unwrap_or_default(),
        requires: Vec::new(),
        conflicts: Vec::new(),
        source,
        path,
    };
    let feature = match &opts.feature {
        Some(feature) => feature.clone(),
        None => choose_feature(text, name)?,
    };
    Ok(AddPlan {
        component,
        dependency: DependencyEntry::Version(version.to_string()),
        feature,
        crate_features: opts.features.clone(),
        light_features: info.map(|i| i.light_features.clone()).unwrap_or_default(),
        creates_feature: false,
        already_installed: true,
        requires: Vec::new(),
    })
}

/// Применяет план: мутирует проект и (при необходимости) проверяет сборку.
pub fn apply_add(
    project_root: &Path,
    runner: &dyn CommandRunner,
    plan: &AddPlan,
    opts: &AddOptions,
) -> Result<AddOutcome> {
    if opts.dry_run || plan.already_installed {
        return Ok(AddOutcome {
            component: plan.component.clone(),
            feature: plan.feature.clone(),
            added_components: plan.requires.iter().map(|r| r.name.clone()).collect(),
            verified: false,
        });
    }

    let snapshot = backup::create(project_root)?;
    match do_add(project_root, runner, plan, opts) {
        Ok(outcome) => Ok(outcome),
        Err(error) => {
            backup::restore(&snapshot)?;
            Err(error)
        }
    }
}

fn do_add(
    project_root: &Path,
    runner: &dyn CommandRunner,
    plan: &AddPlan,
    opts: &AddOptions,
) -> Result<AddOutcome> {
    let cargo_path = project_root.join("Cargo.toml");

    add_dependency(
        project_root,
        &plan.component.name,
        &plan.dependency,
        &plan.crate_features,
    )?;
    let mut added_components = Vec::new();
    for req in &plan.requires {
        add_dependency(project_root, &req.name, &req.dependency, &[])?;
        added_components.push(req.name.clone());
    }

    let text = read_cargo_toml(&cargo_path)?;
    let mut doc = edit::parse(&text)?;
    // Фичи ставятся на обёртку: новый ключ `light-fw`, старый `light` — для
    // уже опубликованных проектов (обратная совместимость).
    if edit::has_dependency(&doc, "light-fw") {
        edit::add_features_to_dependency(&mut doc, "light-fw", &plan.light_features)?;
    } else if edit::has_dependency(&doc, "light") {
        edit::add_features_to_dependency(&mut doc, "light", &plan.light_features)?;
    }
    edit::add_dep_to_feature(&mut doc, &plan.feature, &plan.component.name)?;
    for req in &plan.requires {
        edit::add_dep_to_feature(&mut doc, &plan.feature, &req.name)?;
    }
    std::fs::write(&cargo_path, doc.to_string())?;

    Lockfile::from_project(project_root)?.write(project_root)?;

    let verified = !opts.no_verify;
    if verified {
        cargo_cmd::run_check(runner, project_root, false)?;
    }

    Ok(AddOutcome {
        component: plan.component.clone(),
        feature: plan.feature.clone(),
        added_components,
        verified,
    })
}

fn add_dependency(
    project_root: &Path,
    name: &str,
    dependency: &DependencyEntry,
    features: &[String],
) -> Result<()> {
    let cargo_path = project_root.join("Cargo.toml");
    let text = read_cargo_toml(&cargo_path)?;
    let mut doc = edit::parse(&text)?;
    match dependency {
        DependencyEntry::Workspace => edit::set_workspace_optional(&mut doc, name)?,
        DependencyEntry::Path(path) => {
            edit::set_path_optional(&mut doc, name, &path.to_string_lossy())?
        }
        DependencyEntry::Version(version) => edit::set_version_optional(&mut doc, name, version)?,
    }
    edit::add_features_to_dependency(&mut doc, name, features)?;
    std::fs::write(&cargo_path, doc.to_string())?;
    Ok(())
}

fn dependency_line(name: &str, dependency: &DependencyEntry) -> String {
    match dependency {
        DependencyEntry::Workspace => format!("{name} = {{ workspace = true, optional = true }}"),
        DependencyEntry::Path(path) => {
            format!(
                "{name} = {{ path = \"{}\", optional = true }}",
                path.display()
            )
        }
        DependencyEntry::Version(version) => {
            format!("{name} = {{ version = \"{version}\", optional = true }}")
        }
    }
}

/// Имя фичи по умолчанию: `light-fw-element-form` → `element-form`.
pub fn default_feature_name(name: &str) -> String {
    name.strip_prefix("light-fw-").unwrap_or(name).to_string()
}

/// Выбирает фичу, к которой привязать компонент.
///
/// Порядок: фича `<component>` из манифеста → существующая агрегирующая фича
/// (`dep:light-*`, предпочитая `full`) → новая фича [`default_feature_name`].
fn choose_feature(text: &str, name: &str) -> Result<String> {
    let default = default_feature_name(name);
    if edit::has_feature(text, &default)? {
        return Ok(default);
    }
    let aggregators = edit::features_with_light_component_dep(text)?;
    if let Some(full) = aggregators.iter().find(|f| f.as_str() == "full") {
        return Ok(full.clone());
    }
    if let Some(first) = aggregators.first() {
        return Ok(first.clone());
    }
    Ok(default)
}

/// Разбирает `"light-fw-element-form ^0.3"` → `("light-fw-element-form", Some("^0.3"))`.
fn split_requirement(requirement: &str) -> (&str, Option<&str>) {
    match requirement.split_once(char::is_whitespace) {
        Some((name, rest)) => (name, Some(rest.trim())),
        None => (requirement, None),
    }
}

fn is_core_name(name: &str) -> bool {
    matches!(name, "light" | "light-fw" | "light-fw-core")
}

#[cfg(test)]
mod tests {
    use super::*;
    use crate::process::CommandOutput;
    use std::path::PathBuf;
    use std::sync::Mutex;

    #[derive(Default)]
    struct MockRunner {
        calls: Mutex<Vec<String>>,
        fail_check: bool,
    }

    impl CommandRunner for MockRunner {
        fn run(
            &self,
            program: &str,
            args: &[String],
            _cwd: Option<&Path>,
        ) -> Result<CommandOutput> {
            self.calls
                .lock()
                .expect("lock")
                .push(format!("{program} {}", args.join(" ")));
            if self.fail_check && args.first().map(String::as_str) == Some("check") {
                return Ok(CommandOutput {
                    status: 101,
                    stdout: String::new(),
                    stderr: "compile error".to_string(),
                });
            }
            Ok(CommandOutput {
                status: 0,
                stdout: String::new(),
                stderr: String::new(),
            })
        }
    }

    fn temp_dir(name: &str) -> PathBuf {
        let dir = std::env::temp_dir().join(format!("light-fw-cli-add-{name}"));
        let _ = std::fs::remove_dir_all(&dir);
        std::fs::create_dir_all(&dir).expect("create temp dir");
        dir
    }

    /// Создаёт мини-workspace с компонентом `light-fw-element-form`.
    fn workspace_fixture(name: &str) -> PathBuf {
        let root = temp_dir(name);
        std::fs::write(
            root.join("Cargo.toml"),
            "[workspace]\nmembers = [\"app\", \"comp\"]\nresolver = \"2\"\n\n\
             [workspace.dependencies]\nlight-fw-element-form = { path = \"comp\", version = \"0.1.0\" }\n",
        )
        .expect("root manifest");
        let app = root.join("app");
        std::fs::create_dir_all(&app).expect("app");
        std::fs::write(
            app.join("Cargo.toml"),
            "[package]\nname = \"app\"\nversion = \"0.1.0\"\nedition = \"2021\"\n\n\
             [dependencies]\nlight = \"0.1\"\n",
        )
        .expect("app manifest");
        let comp = root.join("comp");
        std::fs::create_dir_all(comp.join("src")).expect("comp src");
        std::fs::write(comp.join("src/lib.rs"), "").expect("comp lib");
        std::fs::write(
            comp.join("Cargo.toml"),
            "[package]\nname = \"light-fw-element-form\"\nversion = \"0.1.0\"\nedition = \"2021\"\n\n\
             [package.metadata.light]\nkind = \"element\"\ndisplay-name = \"Form\"\ncore = \"^0.1\"\n\n\
             [package.metadata.light.provides]\nelements = [\"checkbox\"]\n",
        )
        .expect("comp manifest");
        app
    }

    #[test]
    fn plans_workspace_component() {
        let app = workspace_fixture("plan");
        let plan = plan_add(&app, "light-fw-element-form", &AddOptions::default()).expect("plan");
        assert_eq!(plan.dependency, DependencyEntry::Workspace);
        assert_eq!(plan.feature, "element-form");
        assert!(plan.creates_feature);
        assert!(plan.to_text().contains("light-fw-element-form"));
    }

    #[test]
    fn already_installed_is_idempotent() {
        let app = workspace_fixture("installed");
        let opts = AddOptions {
            no_verify: true,
            ..Default::default()
        };
        let plan = plan_add(&app, "light-fw-element-form", &opts).expect("plan");
        apply_add(&app, &MockRunner::default(), &plan, &opts).expect("apply");

        let again = plan_add(&app, "light-fw-element-form", &opts).expect("idempotent");
        assert!(again.already_installed);
        assert_eq!(again.component.version, semver::Version::new(0, 1, 0));
        assert_eq!(again.component.source, SourceKind::Workspace);
        // Применение повторного плана ничего не меняет.
        let before = std::fs::read_to_string(app.join("Cargo.toml")).expect("read");
        apply_add(&app, &MockRunner::default(), &again, &opts).expect("re-apply");
        assert_eq!(
            std::fs::read_to_string(app.join("Cargo.toml")).expect("read"),
            before
        );
    }

    #[test]
    fn already_installed_with_different_version_conflicts() {
        let app = workspace_fixture("installed-conflict");
        let opts = AddOptions {
            no_verify: true,
            ..Default::default()
        };
        let plan = plan_add(&app, "light-fw-element-form", &opts).expect("plan");
        apply_add(&app, &MockRunner::default(), &plan, &opts).expect("apply");

        let conflicting = AddOptions {
            version: Some("0.2.0".to_string()),
            ..opts
        };
        let err = plan_add(&app, "light-fw-element-form", &conflicting).expect_err("conflict");
        assert!(matches!(err, Error::Conflict(_)));
    }

    #[test]
    fn dry_run_does_not_touch_disk() {
        let app = workspace_fixture("dry-run");
        let original = std::fs::read_to_string(app.join("Cargo.toml")).expect("read");
        let opts = AddOptions {
            dry_run: true,
            ..Default::default()
        };
        let plan = plan_add(&app, "light-fw-element-form", &opts).expect("plan");
        apply_add(&app, &MockRunner::default(), &plan, &opts).expect("apply");
        assert_eq!(
            std::fs::read_to_string(app.join("Cargo.toml")).expect("read"),
            original
        );
        assert!(!app.join("light.lock").exists());
    }

    #[test]
    fn applies_workspace_dependency_and_feature() {
        let app = workspace_fixture("apply");
        let opts = AddOptions::default();
        let plan = plan_add(&app, "light-fw-element-form", &opts).expect("plan");
        let runner = MockRunner::default();
        let outcome = apply_add(&app, &runner, &plan, &opts).expect("apply");
        assert!(outcome.verified);

        let text = std::fs::read_to_string(app.join("Cargo.toml")).expect("read");
        assert!(text.contains("light-fw-element-form = { workspace = true, optional = true }"));
        assert!(text.contains("[features]"));
        assert!(text.contains("element-form = [\"dep:light-fw-element-form\"]"));
        assert!(app.join("light.lock").exists());

        let calls = runner.calls.lock().expect("lock").clone();
        assert!(calls.iter().any(|c| c == "cargo check --quiet"));
    }

    struct MockRemote {
        name: &'static str,
        version: semver::Version,
    }

    impl crate::fetch::RemoteRegistry for MockRemote {
        fn latest_version(&self, name: &str) -> Result<Option<semver::Version>> {
            if name == self.name {
                Ok(Some(self.version.clone()))
            } else {
                Ok(None)
            }
        }
    }

    #[test]
    fn adds_external_component_from_crates_io() {
        let app = workspace_fixture("external");
        let catalog = Catalog::with_remote(
            &app,
            false,
            Box::new(MockRemote {
                name: "light-fw-datepicker",
                version: semver::Version::new(0, 3, 0),
            }),
        );
        let opts = AddOptions {
            features: vec!["default".to_string()],
            ..Default::default()
        };
        let plan = plan_add_with_catalog(&app, "light-fw-datepicker", &opts, &catalog)
            .expect("plan external");
        assert_eq!(
            plan.dependency,
            DependencyEntry::Version("0.3.0".to_string())
        );
        assert_eq!(plan.feature, "datepicker");

        let outcome = apply_add(&app, &MockRunner::default(), &plan, &opts).expect("apply");
        assert!(outcome.verified);

        let text = std::fs::read_to_string(app.join("Cargo.toml")).expect("read");
        assert!(
            text.contains(
                "light-fw-datepicker = { version = \"0.3.0\", optional = true, features = [\"default\"] }"
            ),
            "{text}"
        );
        assert!(
            text.contains("datepicker = [\"dep:light-fw-datepicker\"]"),
            "{text}"
        );

        let lock = std::fs::read_to_string(app.join("light.lock")).expect("read lock");
        assert!(lock.contains("light-fw-datepicker"), "{lock}");
        assert!(lock.contains("version = \"0.3.0\""), "{lock}");
    }

    #[test]
    fn known_component_enables_light_feature() {
        let app = workspace_fixture("light-feature");
        let opts = AddOptions {
            no_verify: true,
            ..Default::default()
        };
        let plan = plan_add(&app, "light-fw-element-form", &opts).expect("plan");
        assert_eq!(plan.light_features, vec!["elements-form"]);
        apply_add(&app, &MockRunner::default(), &plan, &opts).expect("apply");

        let text = std::fs::read_to_string(app.join("Cargo.toml")).expect("read");
        assert!(
            text.contains("light = { version = \"0.1\", features = [\"elements-form\"] }"),
            "{text}"
        );
    }

    #[test]
    fn failed_check_rolls_back() {
        let app = workspace_fixture("rollback");
        let original = std::fs::read_to_string(app.join("Cargo.toml")).expect("read");
        let opts = AddOptions::default();
        let plan = plan_add(&app, "light-fw-element-form", &opts).expect("plan");
        let runner = MockRunner {
            fail_check: true,
            ..Default::default()
        };
        let err = apply_add(&app, &runner, &plan, &opts).expect_err("check failed");
        assert!(matches!(err, Error::CommandFailed { .. }));
        assert_eq!(
            std::fs::read_to_string(app.join("Cargo.toml")).expect("read"),
            original,
            "Cargo.toml восстановлен"
        );
        assert!(!app.join("light.lock").exists(), "light.lock откачен");
    }

    #[test]
    fn default_feature_name_strips_prefix() {
        assert_eq!(
            default_feature_name("light-fw-element-form"),
            "element-form"
        );
        assert_eq!(default_feature_name("custom-widget"), "custom-widget");
    }

    #[test]
    fn choose_feature_prefers_own_then_aggregator_then_new() {
        // Собственная фича компонента — приоритетнее агрегатора.
        let with_own = "[features]\nfull = [\"dep:light-fw-element-form\"]\n\
                        element-form = [\"dep:light-fw-element-form\"]\n";
        assert_eq!(
            choose_feature(with_own, "light-fw-element-form").unwrap(),
            "element-form"
        );
        // Нет собственной — берём существующий агрегирующий (`full`).
        let with_aggregator =
            "[features]\ndefault = [\"full\"]\nfull = [\"dep:light-fw-element-form\"]\n";
        assert_eq!(
            choose_feature(with_aggregator, "light-fw-behavior-validation").unwrap(),
            "full"
        );
        // Агрегаторов нет — создаём новую фичу по умолчанию.
        assert_eq!(
            choose_feature("[features]\nnone = []\n", "light-fw-behavior-validation").unwrap(),
            "behavior-validation"
        );
    }

    /// Мини-проект с ядром `0.16` (несовместим с реестром `^0.17`).
    fn old_core_app(name: &str) -> PathBuf {
        let dir = temp_dir(&format!("incompat-{name}"));
        std::fs::write(
            dir.join("Cargo.toml"),
            "[package]\nname = \"app\"\nversion = \"0.1.0\"\nedition = \"2021\"\n\n\
             [dependencies]\nlight-fw = \"0.16\"\n",
        )
        .expect("app manifest");
        dir
    }

    #[test]
    fn incompatible_core_without_network_yields_clear_error() {
        // Реестр требует ядро ^0.17, в проекте 0.16. Бинарник собран без сети
        // (remote -> None) и локальных копий нет — падаем с понятной ошибкой,
        // а не паникой.
        let app = old_core_app("no-net");
        let err = plan_add(&app, "light-fw-element-form", &AddOptions::default())
            .expect_err("incompatible");
        let Error::Incompatible { component, reason } = &err else {
            panic!("ожидался Incompatible, получено {err:?}");
        };
        assert_eq!(component, "light-fw-element-form");
        // Указаны версии ядра и инструкция с ручным `light add`.
        assert!(reason.contains("^0.17"), "{reason}");
        assert!(reason.contains("0.16"), "{reason}");
        assert!(
            reason.contains("light add light-fw-element-form@"),
            "{reason}"
        );
    }

    #[test]
    fn incompatible_core_with_remote_latest_is_used() {
        // Если на crates.io есть свежая версия компонента, она выбирается вместо
        // записи реестра. Здесь свежая версия поддерживает ядро проекта, поэтому
        // план собирается без ошибки.
        let app = old_core_app("with-net");
        let catalog = Catalog::with_remote(
            &app,
            false,
            Box::new(MockRemote {
                name: "light-fw-element-form",
                version: semver::Version::new(0, 17, 0),
            }),
        );
        // Мокаем remote как «поддерживающий» ядро: компонент из каталога берёт
        // `core` из реестра (^0.17) — он несовместим, но свежая версия с
        // crates.io всё равно возвращается. План строится, зависимость — её
        // версия.
        let plan = plan_add_with_catalog(
            &app,
            "light-fw-element-form",
            &AddOptions::default(),
            &catalog,
        )
        .expect("plan with fresh version");
        assert_eq!(
            plan.dependency,
            DependencyEntry::Version("0.17.0".to_string())
        );
    }
}