From a0aa37bb9bf487d3a9c7b05bd0f6a1b95ac54457 Mon Sep 17 00:00:00 2001 From: 2 * r + 2 * t <61896496+soramanew@users.noreply.github.com> Date: Wed, 17 Jun 2026 21:05:46 +1000 Subject: [PATCH] fix: remove duplicate resolve component --- src/caelestia/subcommands/install.py | 17 ++++++----------- 1 file changed, 6 insertions(+), 11 deletions(-) diff --git a/src/caelestia/subcommands/install.py b/src/caelestia/subcommands/install.py index 11b2f9b..6364b20 100644 --- a/src/caelestia/subcommands/install.py +++ b/src/caelestia/subcommands/install.py @@ -107,10 +107,14 @@ class Command: disable = _parse_list_arg(self.args.disable_components) try: manifest = source.manifest_at(tip) - manifest.resolve_components(enable=enable, disable=disable) + # No flags given, prompt user for non-default components if enable is None and disable is None: - self.prompt_optional_components(manifest) + optional = [name for name, comp in manifest.components.items() if not comp.default] + if optional: + enable = prompt_selection(optional, "Components to enable?") + + manifest.resolve_components(enable=enable, disable=disable) except (SourceError, ManifestError, ComponentError) as e: fatal(e) @@ -119,15 +123,6 @@ class Command: return source, tip, manifest - def prompt_optional_components(self, manifest: Manifest) -> None: - comp_arr = manifest.disabled_components - if not comp_arr: - return - - selected = prompt_selection(comp_arr, "Components to enable?") - if selected: - manifest.resolve_components(enable=selected) - def deploy_configs(self, source: DotsSource, manifest: Manifest) -> dict[str, str]: print() log("Installing configs...")