specfact-cli 0.46.4__tar.gz → 0.46.16__tar.gz

{specfact_cli-0.46.4 → specfact_cli-0.46.16}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: specfact-cli
-Version: 0.46.4
+Version: 0.46.16
 Summary: The swiss knife CLI for agile DevOps teams. Keep backlog, specs, tests, and code in sync with validation and contract enforcement for new projects and long-lived codebases.
 Project-URL: Homepage, https://github.com/nold-ai/specfact-cli
 Project-URL: Repository, https://github.com/nold-ai/specfact-cli.git

{specfact_cli-0.46.4 → specfact_cli-0.46.16}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "specfact-cli"
-version = "0.46.4"
+version = "0.46.16"
 description = "The swiss knife CLI for agile DevOps teams. Keep backlog, specs, tests, and code in sync with validation and contract enforcement for new projects and long-lived codebases."
 readme = "README.md"
 requires-python = ">=3.11"
@@ -221,6 +221,7 @@ test-cov = "pytest --cov=src --cov-report=term-missing {args}"
 type-check = "basedpyright --pythonpath $(python -c 'import sys; print(sys.executable)') {args}"
 # basedpyright --level error: suppress warning noise in pre-commit (Block 1 runs `hatch run lint`).
 lint = "ruff format . --check && basedpyright --level error --pythonpath $(python -c 'import sys; print(sys.executable)') && ruff check . && pylint src tests tools && python scripts/verify_safe_project_writes.py"
+lint-changed = "python scripts/run_changed_lint.py {args}"
 governance = "pylint src tests tools --reports=y --output-format=parseable"
 format = "ruff check . --fix && ruff format ."
 

{specfact_cli-0.46.4 → specfact_cli-0.46.16}/src/__init__.py

@@ -3,4 +3,4 @@ SpecFact CLI - Spec→Contract→Sentinel tool for contract-driven development.
 """
 
 # Package version: keep in sync with pyproject.toml, setup.py, src/specfact_cli/__init__.py
-__version__ = "0.46.4"
+__version__ = "0.46.16"

{specfact_cli-0.46.4 → specfact_cli-0.46.16}/src/specfact_cli/__init__.py

@@ -45,6 +45,6 @@ def _bootstrap_bundle_paths() -> None:
 
 _bootstrap_bundle_paths()
 
-__version__ = "0.46.4"
+__version__ = "0.46.16"
 
 __all__ = ["__version__"]

{specfact_cli-0.46.4 → specfact_cli-0.46.16}/src/specfact_cli/analyzers/code_analyzer.py

@@ -32,6 +32,24 @@ from specfact_cli.utils.feature_keys import to_classname_key, to_sequential_key
 console = Console()
 
 
+def _ensure_semgrep_runtime_dir(repo_path: Path, relative: str) -> str:
+    """Create and return a stable repo-local runtime directory for Semgrep."""
+    path = (repo_path / relative).resolve()
+    path.mkdir(parents=True, exist_ok=True)
+    return str(path)
+
+
+def _build_semgrep_env(repo_path: Path) -> dict[str, str]:
+    """Build a repo-local Semgrep runtime env so CLI startup stays deterministic."""
+    env = dict(os.environ)
+    env["XDG_CONFIG_HOME"] = _ensure_semgrep_runtime_dir(repo_path, ".specfact/config")
+    env["XDG_CACHE_HOME"] = _ensure_semgrep_runtime_dir(repo_path, ".specfact/cache")
+    env["SEMGREP_VERSION_CACHE_PATH"] = _ensure_semgrep_runtime_dir(repo_path, ".specfact/cache/semgrep")
+    semgrep_log_dir = _ensure_semgrep_runtime_dir(repo_path, ".specfact/logs")
+    env["SEMGREP_LOG_FILE"] = str((Path(semgrep_log_dir) / "semgrep.log").resolve())
+    return env
+
+
 @dataclass
 class _SemgrepFeatureBuckets:
     api_endpoints: list[str] = field(default_factory=list)
@@ -457,6 +475,7 @@ class CodeAnalyzer:
                 capture_output=True,
                 text=True,
                 timeout=5,  # Increased timeout to 5s (Semgrep may need time to initialize)
+                env=_build_semgrep_env(self.repo_path),
             )
             return result.returncode == 0
         except (FileNotFoundError, subprocess.TimeoutExpired, OSError):
@@ -566,6 +585,7 @@ class CodeAnalyzer:
                 capture_output=True,
                 text=True,
                 timeout=timeout,
+                env=_build_semgrep_env(self.repo_path),
             )
 
             # Semgrep may return non-zero for valid findings

{specfact_cli-0.46.4 → specfact_cli-0.46.16}/src/specfact_cli/cli.py

@@ -14,7 +14,7 @@ from collections.abc import Callable, Mapping
 from dataclasses import dataclass
 from datetime import datetime
 from pathlib import Path
-from typing import Annotated, Any, cast
+from typing import Annotated, Any, NoReturn, cast
 
 
 _DetectShellFn = Callable[..., tuple[str | None, str | None]]
@@ -66,6 +66,7 @@ from specfact_cli.registry import CommandRegistry
 from specfact_cli.registry.alias_manager import resolve_command
 from specfact_cli.registry.bootstrap import register_builtin_commands
 from specfact_cli.registry.metadata import CommandMetadata
+from specfact_cli.registry.module_availability import ModuleAvailabilityStatus, classify_module_availability
 from specfact_cli.runtime import get_configured_console, init_debug_log_file, set_debug_mode
 from specfact_cli.utils.progressive_disclosure import ProgressiveDisclosureGroup
 from specfact_cli.utils.structured_io import StructuredFormat
@@ -124,6 +125,28 @@ def _print_missing_bundle_command_help(invoked: str) -> None:
     module_id = _INVOKED_TO_MARKETPLACE_MODULE.get(invoked)
     console = get_configured_console()
     if module_id is not None:
+        availability = classify_module_availability(module_id=module_id, command_name=invoked)
+        if availability.status is ModuleAvailabilityStatus.DISABLED:
+            console.print(
+                f"[bold red]Module '{availability.module_id or module_id}' is installed but disabled.[/bold red]\n"
+                f"The [bold]{invoked}[/bold] command group is provided by that module. "
+                f"Enable with [bold]{availability.recovery_command}[/bold]."
+            )
+            return
+        if availability.status is ModuleAvailabilityStatus.SKIPPED:
+            console.print(
+                f"[bold red]Module '{availability.module_id or module_id}' is installed but skipped.[/bold red]\n"
+                f"Reason: {availability.reason}. "
+                "Inspect with [bold]specfact module list --show-origin[/bold]."
+            )
+            return
+        if availability.status is ModuleAvailabilityStatus.SHADOWED:
+            console.print(
+                f"[bold red]Module '{availability.module_id or module_id}' is shadowed in this workspace.[/bold red]\n"
+                f"Shadowed by: {availability.shadowed_by}. "
+                "Inspect with [bold]specfact module list --show-origin[/bold]."
+            )
+            return
         console.print(
             f"[bold red]Module '{module_id}' is not installed.[/bold red]\n"
             f"The [bold]{invoked}[/bold] command group is provided by that module. "
@@ -509,6 +532,84 @@ def _lazy_delegate_cmd_name_ready(self: _LazyDelegateGroup) -> bool:
     return len(self._lazy_cmd_name) > 0
 
 
+def _args_request_help(args: tuple[str, ...] | list[str]) -> bool:
+    """Return True when delegated args are asking only for command help."""
+    return any(arg in ("--help", "-h", "--help-advanced", "-ha") for arg in args)
+
+
+def _delegated_help_path(cmd_name: str, args: tuple[str, ...] | list[str]) -> str:
+    """Build a stable command path for fallback help output."""
+    path_parts = [cmd_name]
+    for arg in args:
+        if arg.startswith("-"):
+            continue
+        path_parts.append(arg)
+    return " ".join(path_parts)
+
+
+def _print_lazy_help_fallback(cmd_name: str, args: tuple[str, ...] | list[str]) -> None:
+    """Print minimal help when Typer cannot materialize a command for a loaded bundle."""
+    command_path = _delegated_help_path(cmd_name, args)
+    get_configured_console().print(
+        f"[bold]{command_path}[/bold]\n\n"
+        "Help is available for this installed command path, but the command metadata could not be "
+        "materialized in this runtime. Reinstall the providing module or run the command without "
+        "`--help` to execute it."
+    )
+
+
+def _raise_lazy_delegate_click_exception(exc: Exception) -> NoReturn:
+    raise click.ClickException(str(exc)) from exc
+
+
+def _load_lazy_delegate_typer(cmd_name: str) -> typer.Typer:
+    resolved_name = resolve_command(cmd_name)
+    try:
+        return CommandRegistry.get_typer(resolved_name)
+    except ValueError as exc:
+        if cmd_name in KNOWN_BUNDLE_GROUP_OR_SHIM_NAMES:
+            _print_missing_bundle_command_help(cmd_name)
+            raise SystemExit(1) from None
+        _raise_lazy_delegate_click_exception(exc)
+        raise AssertionError("unreachable") from None
+
+
+def _build_lazy_delegate_click_command(cmd_name: str, args: tuple[str, ...], real_typer: typer.Typer) -> click.Command:
+    from typer.main import get_command
+
+    try:
+        return get_command(real_typer)
+    except (RuntimeError, ValueError) as exc:
+        if _args_request_help(args):
+            _print_lazy_help_fallback(cmd_name, args)
+            raise SystemExit(0) from None
+        _raise_lazy_delegate_click_exception(exc)
+        raise AssertionError("unreachable") from None
+
+
+def _lazy_delegate_prog_name(ctx: click.Context, cmd_name: str) -> str:
+    parts: list[str] = []
+    parent = ctx.parent
+    while parent and getattr(parent, "command", None):
+        name = getattr(parent.command, "name", None)
+        if name and name != "__delegate__":
+            parts.append(name)
+        parent = getattr(parent, "parent", None)
+    if parts:
+        return " ".join(reversed(parts))
+    original_prog_name = ctx.meta.get("original_prog_name")
+    if isinstance(original_prog_name, str) and original_prog_name:
+        return original_prog_name
+    return cmd_name
+
+
+def _strip_redundant_single_command_arg(click_cmd: click.Command, args: tuple[str, ...]) -> list[str]:
+    args_list = list(args)
+    if not isinstance(click_cmd, click.Group) and args_list and args_list[0] == getattr(click_cmd, "name", None):
+        return args_list[1:]
+    return args_list
+
+
 class _LazyDelegateGroup(click.Group):
     """Click Group that delegates all args to the real command (lazy-loaded)."""
 
@@ -521,6 +622,8 @@ class _LazyDelegateGroup(click.Group):
             name=name or cmd_name,
             help=help or help_str,
             context_settings={"ignore_unknown_options": True},
+            invoke_without_command=True,
+            no_args_is_help=False,
         )
         self._lazy_cmd_name = cmd_name
         self._lazy_help_str = help_str
@@ -530,31 +633,15 @@ class _LazyDelegateGroup(click.Group):
         cmd_name = self._lazy_cmd_name
 
         def _invoke(args: tuple[str, ...]) -> None:
-            from typer.main import get_command
-
             ctx = click.get_current_context()
-            resolved_name = resolve_command(cmd_name)
-            real_typer = CommandRegistry.get_typer(resolved_name)
-            click_cmd = get_command(real_typer)
+            real_typer = _load_lazy_delegate_typer(cmd_name)
+            click_cmd = _build_lazy_delegate_click_command(cmd_name, args, real_typer)
             # Build full prog name from root (e.g. "specfact sync") so usage shows "specfact sync bridge", not "sync sync bridge"
-            parts: list[str] = []
-            p = ctx.parent
-            while p and getattr(p, "command", None):
-                name = getattr(p.command, "name", None)
-                if name and name != "__delegate__":
-                    parts.append(name)
-                p = getattr(p, "parent", None)
-            prog_name = " ".join(reversed(parts)) if parts else cmd_name
-            args_list = list(args)
+            prog_name = _lazy_delegate_prog_name(ctx, cmd_name)
             # When the real app is a single command (e.g. drift has only "detect"), Typer
             # builds a TyperCommand, not a Group. Then args are ["detect", "bundle", "--repo", ...]
             # and the command expects ["bundle", "--repo", ...] (no leading "detect").
-            if (
-                not isinstance(click_cmd, click.Group)
-                and args_list
-                and args_list[0] == getattr(click_cmd, "name", None)
-            ):
-                args_list = args_list[1:]
+            args_list = _strip_redundant_single_command_arg(click_cmd, args)
             exit_code = click_cmd.main(args=args_list, prog_name=prog_name, standalone_mode=False)
             if exit_code and exit_code != 0:
                 raise SystemExit(exit_code)
@@ -567,6 +654,14 @@ class _LazyDelegateGroup(click.Group):
             add_help_option=False,  # Pass --help through to real Typer so "specfact backlog daily ado --help" shows correct usage
         )
 
+    @require(lambda ctx: ctx is not None, "ctx must not be None")
+    @ensure(lambda result: result is None or isinstance(result, int), "result must be None or an exit code")
+    def invoke(self, ctx: click.Context) -> Any:
+        if ctx.invoked_subcommand is None and not ctx.args:
+            ctx.meta["original_prog_name"] = ctx.command_path
+            return self._delegate_cmd.main(args=[], prog_name=ctx.command_path, standalone_mode=False)
+        return super().invoke(ctx)
+
     @require(_lazy_delegate_cmd_name_ready, "lazy command name must be set")
     @ensure(lambda result: isinstance(result, tuple) and len(result) == 3, "result must be a 3-tuple")
     def resolve_command(
@@ -574,7 +669,7 @@ class _LazyDelegateGroup(click.Group):
     ) -> tuple[str | None, click.Command | None, list[str]]:
         # Pass through all args to the delegate so "plan init bundle" becomes args for the real plan Typer.
         if not args:
-            return None, None, []
+            return self._delegate_cmd.name, self._delegate_cmd, []
         return self._delegate_cmd.name, self._delegate_cmd, list(args)
 
     @ensure(lambda result: isinstance(result, list), "result must be a list of command names")
@@ -598,8 +693,11 @@ class _LazyDelegateGroup(click.Group):
         from typer.main import get_command
 
         resolved_name = resolve_command(self._lazy_cmd_name)
-        real_typer = CommandRegistry.get_typer(resolved_name)
-        click_cmd = get_command(real_typer)
+        try:
+            real_typer = CommandRegistry.get_typer(resolved_name)
+            click_cmd = get_command(real_typer)
+        except (RuntimeError, ValueError):
+            return None
         if isinstance(click_cmd, click.Group):
             return click_cmd
         return None
@@ -610,8 +708,11 @@ class _LazyDelegateGroup(click.Group):
         from typer.main import get_command
 
         resolved_name = resolve_command(self._lazy_cmd_name)
-        real_typer = CommandRegistry.get_typer(resolved_name)
-        click_cmd = get_command(real_typer)
+        try:
+            real_typer = CommandRegistry.get_typer(resolved_name)
+            click_cmd = get_command(real_typer)
+        except (RuntimeError, ValueError):
+            return
         prog_name = (
             f"{ctx.parent.command.name} {self._lazy_cmd_name}"
             if ctx.parent and ctx.parent.command

{specfact_cli-0.46.4 → specfact_cli-0.46.16}/src/specfact_cli/modules/init/module-package.yaml

@@ -1,5 +1,5 @@
 name: init
-version: 0.1.30
+version: 0.1.31
 commands:
   - init
 category: core
@@ -17,5 +17,5 @@ publisher:
 description: Initialize SpecFact workspace and bootstrap local configuration.
 license: Apache-2.0
 integrity:
-  checksum: sha256:9e03421972d3254082307834b474e7673957de8c11ffacc563f2da3f35e7cf05
-  signature: ikUYhUJ8AFU9RkB+ZBnp0lKrDLT7ZIqkauRYUMRY/swrIjRCTRzHIpNcvCmujK6h1w6EtT/+wGG1v5bZtZB8CQ==
+  checksum: sha256:0f7bc54a823bea14033fcb143ecb6c83d2bca2b5da661f03a0b545100acebe5b
+  signature: dTatkqgBUtti4tL/pmcFBZY9bsJ61gY/V0lP9gZU8Y5W3YWK+wpgRx1oewlAmfKzkxca2NhalKcjLACQjTNvAA==

{specfact_cli-0.46.4 → specfact_cli-0.46.16}/src/specfact_cli/modules/init/src/commands.py

@@ -106,7 +106,7 @@ def _resolve_field_mapping_templates_dir(repo_path: Path) -> Path | None:
         package_templates_dir = Path(str(templates_ref)).resolve()
         if package_templates_dir.exists():
             return package_templates_dir
-    except Exception:
+    except (ImportError, OSError, ValueError):
         try:
             import importlib.util
 
@@ -118,8 +118,8 @@ def _resolve_field_mapping_templates_dir(repo_path: Path) -> Path | None:
                 ).resolve()
                 if package_templates_dir.exists():
                     return package_templates_dir
-        except Exception:
-            pass
+        except (ImportError, OSError, ValueError):
+            return None
     return None
 
 
@@ -434,7 +434,15 @@ def _is_valid_repo_path(repo: Path) -> bool:
 
 
 @beartype
-def _install_profile_bundles(profile: str, install_root: Path, non_interactive: bool) -> None:
+def _marketplace_ids_for_bundles(bundle_ids: list[str]) -> list[str]:
+    return [
+        first_run_selection.MARKETPLACE_ONLY_BUNDLES[bid]
+        for bid in bundle_ids
+        if bid in first_run_selection.MARKETPLACE_ONLY_BUNDLES
+    ]
+
+
+def _install_profile_bundles(profile: str, install_root: Path, non_interactive: bool) -> list[str]:
     """Resolve profile to bundle list and install via module installer."""
     bundle_ids = first_run_selection.resolve_profile_bundles(profile)
     if bundle_ids:
@@ -444,10 +452,11 @@ def _install_profile_bundles(profile: str, install_root: Path, non_interactive:
             install_root,
             non_interactive=non_interactive,
         )
+    return _marketplace_ids_for_bundles(bundle_ids)
 
 
 @beartype
-def _install_bundle_list(install_arg: str, install_root: Path, non_interactive: bool) -> None:
+def _install_bundle_list(install_arg: str, install_root: Path, non_interactive: bool) -> list[str]:
     """Parse comma-separated or 'all' and install bundles via module installer."""
     bundle_ids = first_run_selection.resolve_install_bundles(install_arg)
     if bundle_ids:
@@ -457,20 +466,32 @@ def _install_bundle_list(install_arg: str, install_root: Path, non_interactive:
             install_root,
             non_interactive=non_interactive,
         )
+    return _marketplace_ids_for_bundles(bundle_ids)
 
 
-def _apply_profile_or_install_bundles(profile: str | None, install: str | None) -> None:
+def _apply_profile_or_install_bundles(profile: str | None, install: str | None) -> list[str]:
     try:
         non_interactive = is_non_interactive()
         if profile is not None:
-            _install_profile_bundles(profile, INIT_USER_MODULES_ROOT, non_interactive=non_interactive)
-        else:
-            _install_bundle_list(install or "", INIT_USER_MODULES_ROOT, non_interactive=non_interactive)
+            return _install_profile_bundles(profile, INIT_USER_MODULES_ROOT, non_interactive=non_interactive)
+        return _install_bundle_list(install or "", INIT_USER_MODULES_ROOT, non_interactive=non_interactive)
     except ValueError as e:
         console.print(f"[red]Error:[/red] {e}")
         raise typer.Exit(1) from e
 
 
+def _refresh_init_module_state(repo_path: Path, enabled_module_ids: list[str]) -> list[dict[str, Any]]:
+    modules_list = get_discovered_modules_for_state(
+        enable_ids=enabled_module_ids,
+        disable_ids=[],
+        base_path=repo_path,
+        preserve_existing=True,
+    )
+    if modules_list:
+        write_modules_state(modules_list)
+    return modules_list
+
+
 def _run_interactive_first_run_install() -> None:
     try:
         bundle_ids = _interactive_first_run_bundle_selection()
@@ -702,8 +723,9 @@ def init(
 
         repo_path = repo.resolve()
 
+        enabled_module_ids: list[str] = []
         if profile is not None or install is not None:
-            _apply_profile_or_install_bundles(profile, install)
+            enabled_module_ids = _apply_profile_or_install_bundles(profile, install)
         elif is_first_run(user_root=INIT_USER_MODULES_ROOT) and is_non_interactive():
             console.print(
                 "[red]Error:[/red] In CI/CD (non-interactive) mode, first-run init requires "
@@ -718,9 +740,7 @@ def init(
             _run_interactive_first_run_install()
 
         _init_user_visible_step("[cyan]→[/cyan] Discovering installed modules and writing registry state…")
-        modules_list = get_discovered_modules_for_state(enable_ids=[], disable_ids=[])
-        if modules_list:
-            write_modules_state(modules_list)
+        modules_list = _refresh_init_module_state(repo_path, enabled_module_ids)
 
         _init_user_visible_step("[cyan]→[/cyan] Indexing CLI commands for help cache…")
         run_discovery_and_write_cache(__version__)

{specfact_cli-0.46.4 → specfact_cli-0.46.16}/src/specfact_cli/modules/module_registry/module-package.yaml

@@ -1,5 +1,5 @@
 name: module-registry
-version: 0.1.20
+version: 0.1.23
 commands:
   - module
 category: core
@@ -17,5 +17,5 @@ publisher:
 description: 'Manage modules: search, list, show, install, and upgrade.'
 license: Apache-2.0
 integrity:
-  checksum: sha256:a92afa757a54ee63b84ae4a5f5b232cb5dbddacfcb31fcde3abcdb4927eada8c
-  signature: jelDGPZyCLLpyzqH4+S2t7V9ICy/puYEzdUu/GX0oPiWgMZRg8aZlnECZGy+m+sHvS0XX3hPAXCSp3IJf6hHDg==
+  checksum: sha256:f500281d2249d712be23a1b25b5660374694dfa47634f60ae4378eb2cdb753ca
+  signature: cunuat95bD44IcNUBs35NTlHYOR8atydVz4ZyZTjIzwJL6Wz5YOfBYGq/WtgDnOFv3KplfvhRZbTo3CwmS/KBQ==

{specfact_cli-0.46.4 → specfact_cli-0.46.16}/src/specfact_cli/modules/module_registry/src/commands.py

@@ -20,12 +20,14 @@ from packaging.version import InvalidVersion, Version
 from rich.console import Console
 from rich.table import Table
 
+from specfact_cli import __version__
 from specfact_cli.models.module_package import ModulePackageMetadata
 from specfact_cli.modules import module_io_shim
 from specfact_cli.registry.alias_manager import create_alias, list_aliases, remove_alias
 from specfact_cli.registry.custom_registries import add_registry, fetch_all_indexes, list_registries, remove_registry
+from specfact_cli.registry.help_cache import run_discovery_and_write_cache
 from specfact_cli.registry.marketplace_client import fetch_registry_index
-from specfact_cli.registry.module_discovery import discover_all_modules
+from specfact_cli.registry.module_discovery import discover_all_modules, discover_all_modules_for_project
 from specfact_cli.registry.module_installer import (
     REGISTRY_ID_FILE,
     USER_MODULES_ROOT,
@@ -42,7 +44,9 @@ from specfact_cli.registry.module_lifecycle import (
     render_modules_table,
     select_module_ids_interactive,
 )
+from specfact_cli.registry.module_packages import get_discovered_modules_for_state
 from specfact_cli.registry.module_security import ensure_publisher_trusted, is_official_publisher
+from specfact_cli.registry.module_state import read_modules_state, write_modules_state
 from specfact_cli.registry.registry import CommandRegistry
 from specfact_cli.runtime import is_non_interactive
 
@@ -178,15 +182,64 @@ def _resolve_install_target_root(scope_normalized: str, repo: Path | None) -> Pa
     return USER_MODULES_ROOT if scope_normalized == "user" else repo_path / ".specfact" / "modules"
 
 
+def _normalize_project_repo(repo: Path | None) -> Path | None:
+    """Resolve a project-scoped repo argument to the nearest workspace root."""
+    if repo is None:
+        return None
+    repo_path = repo.resolve()
+    for candidate in [repo_path, *repo_path.parents]:
+        if (candidate / ".git").exists():
+            return candidate
+    return repo_path
+
+
+def _read_installed_manifest_id(module_dir: Path, fallback_name: str) -> str:
+    manifest_path = module_dir / "module-package.yaml"
+    try:
+        raw = yaml.safe_load(manifest_path.read_text(encoding="utf-8"))
+    except (OSError, yaml.YAMLError):
+        return fallback_name
+    if isinstance(raw, dict):
+        manifest = cast(dict[str, Any], raw)
+        if manifest.get("name"):
+            return str(manifest["name"])
+    return fallback_name
+
+
+def _enable_if_disabled(module_id: str, base_path: Path | None = None) -> bool:
+    state = read_modules_state()
+    if state.get(module_id, {}).get("enabled", True) is not False:
+        return False
+    modules = get_discovered_modules_for_state(
+        enable_ids=[module_id],
+        disable_ids=[],
+        base_path=base_path,
+        preserve_existing=True,
+    )
+    write_modules_state(modules)
+    run_discovery_and_write_cache(__version__)
+    return any(str(row.get("id", "")) == module_id and bool(row.get("enabled", True)) for row in modules)
+
+
 def _install_skip_if_already_satisfied(
     scope_normalized: str,
     requested_name: str,
     target_root: Path,
+    repo: Path | None,
     reinstall: bool,
     discovered_by_name: dict[str, Any],
 ) -> bool:
-    if (target_root / requested_name / "module-package.yaml").exists() and not reinstall:
-        console.print(f"[yellow]Module '{requested_name}' is already installed in {target_root}.[/yellow]")
+    installed_dir = target_root / requested_name
+    if (installed_dir / "module-package.yaml").exists() and not reinstall:
+        module_id = _read_installed_manifest_id(installed_dir, requested_name)
+        enabled = _enable_if_disabled(module_id, base_path=repo if scope_normalized == "project" else None)
+        if enabled:
+            console.print(
+                f"[yellow]Module '{module_id}' is already installed in {target_root}; "
+                "enabled it in module state.[/yellow]"
+            )
+        else:
+            console.print(f"[yellow]Module '{module_id}' is already installed in {target_root}.[/yellow]")
         return True
     skip_sources = {"builtin", "project", "user", "custom"}
     if scope_normalized == "project":
@@ -195,9 +248,14 @@ def _install_skip_if_already_satisfied(
         skip_sources.discard("project")
     existing = discovered_by_name.get(requested_name)
     if existing is not None and existing.source in skip_sources:
+        enabled = _enable_if_disabled(
+            existing.metadata.name,
+            base_path=repo if scope_normalized == "project" else None,
+        )
+        state_hint = " Enabled it in module state." if enabled else ""
         console.print(
-            f"[yellow]Module '{requested_name}' is already available from source '{existing.source}'. "
-            "No marketplace install needed.[/yellow]"
+            f"[yellow]Module '{existing.metadata.name}' is already available from source '{existing.source}'. "
+            f"No marketplace install needed.{state_hint}[/yellow]"
         )
         return True
     return False
@@ -274,6 +332,7 @@ class _InstallOneParams:
     scope_normalized: str
     source_normalized: str
     target_root: Path
+    repo: Path | None
     version: str | None
     reinstall: bool
     trust_non_official: bool
@@ -289,6 +348,7 @@ def _install_one(module_id: str, params: _InstallOneParams) -> bool:
         params.scope_normalized,
         requested_name,
         params.target_root,
+        params.repo,
         params.reinstall,
         params.discovered_by_name,
     ):
@@ -392,12 +452,17 @@ def _install_impl(module_ids: list[str], **kwargs: Any) -> None:
         )
         raise typer.Exit(1)
     scope_normalized, source_normalized = _parse_install_scope_and_source(scope, source)
-    target_root = _resolve_install_target_root(scope_normalized, repo)
-    discovered_by_name = {entry.metadata.name: entry for entry in discover_all_modules()}
+    normalized_repo = _normalize_project_repo(repo) if scope_normalized == "project" else None
+    target_root = _resolve_install_target_root(scope_normalized, normalized_repo)
+    discovered = (
+        discover_all_modules_for_project(normalized_repo) if normalized_repo is not None else discover_all_modules()
+    )
+    discovered_by_name = {entry.metadata.name: entry for entry in discovered}
     params = _InstallOneParams(
         scope_normalized=scope_normalized,
         source_normalized=source_normalized,
         target_root=target_root,
+        repo=normalized_repo,
         version=version,
         reinstall=reinstall,
         trust_non_official=trust_non_official,

{specfact_cli-0.46.4 → specfact_cli-0.46.16}/src/specfact_cli/modules/upgrade/module-package.yaml

@@ -1,5 +1,5 @@
 name: upgrade
-version: 0.1.4
+version: 0.1.12
 commands:
   - upgrade
 category: core
@@ -17,5 +17,4 @@ publisher:
 description: Check and apply SpecFact CLI version upgrades.
 license: Apache-2.0
 integrity:
-  checksum: sha256:0648be45eb877287ebef717d38c71d48c1e17191dfb24b0c8dde57015f7ba144
-  signature: ZMw8ljS+0f4TYg2WVAqQCpgaae1d8z7wT/1r2yxuM6ZeZjMejhgeBuOyXopda5LOXjioxTxOlWZmGN94cCC3Ag==
+  checksum: sha256:5bc01c11370e696da4fa365188c3d223a7bd3a99ffc36e878b841a4081d6bb62