specfact-cli 0.37.3__tar.gz → 0.37.4__tar.gz

{specfact_cli-0.37.3 → specfact_cli-0.37.4}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: specfact-cli
-Version: 0.37.3
+Version: 0.37.4
 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.37.3 → specfact_cli-0.37.4}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "specfact-cli"
-version = "0.37.3"
+version = "0.37.4"
 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"

{specfact_cli-0.37.3 → specfact_cli-0.37.4}/src/specfact_cli/__init__.py

@@ -8,6 +8,6 @@ This package provides command-line tools for:
 - Supporting agile ceremonies and team workflows
 """
 
-__version__ = "0.37.3"
+__version__ = "0.37.4"
 
 __all__ = ["__version__"]

{specfact_cli-0.37.3 → specfact_cli-0.37.4}/src/specfact_cli/modules/backlog/module-package.yaml

@@ -1,5 +1,5 @@
 name: backlog
-version: 0.1.0
+version: 0.1.2
 commands:
   - backlog
 command_help:
@@ -28,5 +28,5 @@ publisher:
 description: Manage backlog ceremonies, refinement, and dependency insights.
 license: Apache-2.0
 integrity:
-  checksum: sha256:1ab19fd9dd206ea5bb12af2f785fda4b56c5e3324902154f31ca24cc88334b4f
-  signature: UDxeFiE6DafWglCWREHuSZz4ezApHwygKI4wqkHiFIcHFk5CRqnH9KPUdTVyB+Agiys4k3kKNfGb/0Ljit//DA==
+  checksum: sha256:ef3febe90c62fa24b3da6b6b2f8a7b605c5b544db01aac10e5cfacc807bcca26
+  signature: 2b/UNbFlgRQiwJ+LrEQKyoYUi8lxeLGc4hrcTUdCinU9yXa1h6GvYpRkAkzRW8kxNNNsetphuf7hLZ3II3bNDQ==

{specfact_cli-0.37.3 → specfact_cli-0.37.4}/src/specfact_cli/modules/backlog/src/commands.py

@@ -4224,6 +4224,12 @@ def map_fields(
             if key and value:
                 cli_option_map[key] = value
 
+        issue_type_id_map = {
+            issue_type: repo_issue_types.get(issue_type, "")
+            for issue_type in ["epic", "feature", "story", "task", "bug"]
+            if repo_issue_types.get(issue_type)
+        }
+
         # Fast-path for fully specified non-interactive invocations.
         if project_ref and (github_type_field_id or "").strip() and cli_option_map:
             github_custom_mapping_file = _persist_github_custom_mapping_file(repo_issue_types)
@@ -4247,6 +4253,33 @@ def map_fields(
             console.print(f"[green]Custom mapping:[/green] {github_custom_mapping_file}")
             return
 
+        if not project_ref:
+            if cli_option_map or (github_type_field_id or "").strip():
+                console.print(
+                    "[yellow]⚠[/yellow] GitHub ProjectV2 Type options/field-id were provided, but no ProjectV2 "
+                    "number/ID was set. Skipping ProjectV2 mapping."
+                )
+            github_custom_mapping_file = _persist_github_custom_mapping_file(repo_issue_types)
+            initial_settings_update: dict[str, Any] = {
+                "github_issue_types": {"type_ids": issue_type_id_map},
+                # Clear stale ProjectV2 mapping when user explicitly skips ProjectV2 input.
+                "provider_fields": {"github_project_v2": None},
+                "field_mapping_file": ".specfact/templates/backlog/field_mappings/github_custom.yaml",
+            }
+            config_path = _upsert_backlog_provider_settings(
+                "github",
+                initial_settings_update,
+                project_id=project_context,
+                adapter="github",
+            )
+            console.print(f"[green]✓[/green] GitHub mapping saved to {config_path}")
+            console.print(f"[green]Custom mapping:[/green] {github_custom_mapping_file}")
+            console.print(
+                "[dim]ProjectV2 Type field mapping skipped; repository issue types were captured "
+                "(ProjectV2 is optional).[/dim]"
+            )
+            return
+
         project_id = ""
         project_title = ""
         fields_nodes: list[dict[str, Any]] = []
@@ -4445,12 +4478,6 @@ def map_fields(
                 if option_name and option_name in option_name_to_id:
                     option_map[issue_type] = option_name_to_id[option_name]
 
-        issue_type_id_map = {
-            issue_type: repo_issue_types.get(issue_type, "")
-            for issue_type in ["epic", "feature", "story", "task", "bug"]
-            if repo_issue_types.get(issue_type)
-        }
-
         settings_update: dict[str, Any] = {}
         if issue_type_id_map:
             settings_update["github_issue_types"] = {"type_ids": issue_type_id_map}

{specfact_cli-0.37.3 → specfact_cli-0.37.4}/src/specfact_cli/modules/module_registry/module-package.yaml

@@ -1,5 +1,5 @@
 name: module-registry
-version: 0.1.0
+version: 0.1.3
 commands:
   - module
 command_help:
@@ -15,5 +15,5 @@ publisher:
 description: 'Manage modules: search, list, show, install, and upgrade.'
 license: Apache-2.0
 integrity:
-  checksum: sha256:933aeb8adb353565167a3a63dfda9dd62c15ce4f5574a85198c8007e992780e3
-  signature: 8bn3S2Unl2sLaGIF7f0MlLkH4AoplU5h2VY/uDoJBwmO3rl3xweYXuoJftPKmFASBUWcVxxoK+YoGvOzY6fpDA==
+  checksum: sha256:92a3673edb39974791afcc357a1e931b6e4152d1ad7af0e4a4d722c29431b90d
+  signature: eynkV+88hx9PPGlOnB5rwWHkiEtgFe8wNIU+5I0C8PZR+WWoRIcTcvTe5TYYAyiHi6JmJZZLvrDsTIpe3K3VDQ==

{specfact_cli-0.37.3 → specfact_cli-0.37.4}/src/specfact_cli/modules/module_registry/src/commands.py

@@ -509,13 +509,14 @@ def list_modules(
     ),
 ) -> None:
     """List installed modules with trust labels and optional origin details."""
-    modules = get_modules_with_state()
+    all_modules = get_modules_with_state()
+    modules = all_modules
     if source:
         modules = [m for m in modules if str(m.get("source", "")) == source]
     render_modules_table(console, modules, show_origin=show_origin)
 
     bundled = get_bundled_module_metadata()
-    installed_ids = {str(module.get("id", "")).strip() for module in get_modules_with_state()}
+    installed_ids = {str(module.get("id", "")).strip() for module in all_modules}
     available = [meta for name, meta in bundled.items() if name not in installed_ids]
     if not show_bundled_available:
         if available:

{specfact_cli-0.37.3 → specfact_cli-0.37.4}/src/specfact_cli/registry/marketplace_client.py

@@ -96,5 +96,5 @@ def download_module(
     file_name = Path(parsed.path).name or f"{module_id.replace('/', '-')}.tar.gz"
     target_path = target_dir / file_name
     target_path.write_bytes(content)
-    logger.info("Downloaded module '%s' to '%s'", module_id, target_path)
+    logger.debug("Downloaded module '%s' to '%s'", module_id, target_path)
     return target_path

{specfact_cli-0.37.3 → specfact_cli-0.37.4}/src/specfact_cli/registry/module_discovery.py

@@ -10,11 +10,13 @@ from icontract import ensure
 
 from specfact_cli.common import get_bridge_logger
 from specfact_cli.models.module_package import ModulePackageMetadata
+from specfact_cli.utils.prompts import print_warning
 
 
 USER_MODULES_ROOT = Path.home() / ".specfact" / "modules"
 MARKETPLACE_MODULES_ROOT = Path.home() / ".specfact" / "marketplace-modules"
 CUSTOM_MODULES_ROOT = Path.home() / ".specfact" / "custom-modules"
+_SHADOW_HINT_KEYS: set[tuple[str, str, str, str]] = set()
 
 
 @dataclass(frozen=True)
@@ -45,7 +47,7 @@ def discover_all_modules(
 
     logger = get_bridge_logger(__name__)
     discovered: list[DiscoveredModule] = []
-    seen_names: set[str] = set()
+    seen_by_name: dict[str, DiscoveredModule] = {}
 
     effective_builtin_root = builtin_root or get_modules_root()
     effective_project_root = get_workspace_modules_root()
@@ -87,23 +89,40 @@ def discover_all_modules(
         entries = discover_package_metadata(root, source=source)
         for package_dir, metadata in entries:
             module_name = metadata.name
-            if module_name in seen_names:
+            if module_name in seen_by_name:
+                existing = seen_by_name[module_name]
+                if source == "user" and existing.source == "project":
+                    warning_key = (
+                        module_name,
+                        existing.source,
+                        source,
+                        str(existing.package_dir.resolve()),
+                    )
+                    if warning_key not in _SHADOW_HINT_KEYS:
+                        _SHADOW_HINT_KEYS.add(warning_key)
+                        print_warning(
+                            f"Module '{module_name}' from project scope ({existing.package_dir}) takes precedence over "
+                            f"user-scoped module ({package_dir}) in this workspace. The user copy is ignored here. "
+                            f"Inspect origins with `specfact module list --show-origin`; if stale, clean user scope "
+                            f"with `specfact module uninstall {module_name} --scope user`."
+                        )
                 if source in {"user", "marketplace", "custom"}:
-                    logger.warning(
-                        "Module '%s' from %s at '%s' is shadowed by higher-priority source.",
+                    logger.debug(
+                        "Module '%s' from %s at '%s' is shadowed by higher-priority source %s at '%s'.",
                         module_name,
                         source,
                         package_dir,
+                        existing.source,
+                        existing.package_dir,
                     )
                 continue
 
-            seen_names.add(module_name)
-            discovered.append(
-                DiscoveredModule(
-                    package_dir=package_dir,
-                    metadata=metadata,
-                    source=source,
-                )
+            entry = DiscoveredModule(
+                package_dir=package_dir,
+                metadata=metadata,
+                source=source,
             )
+            seen_by_name[module_name] = entry
+            discovered.append(entry)
 
     return discovered

{specfact_cli-0.37.3 → specfact_cli-0.37.4}/src/specfact_cli/registry/module_installer.py

@@ -409,7 +409,7 @@ def install_module(
     manifest_path = final_path / "module-package.yaml"
 
     if manifest_path.exists() and not reinstall:
-        logger.info("Module already installed (%s)", module_name)
+        logger.debug("Module already installed (%s)", module_name)
         return final_path
 
     archive_path = download_module(module_id, version=version)
@@ -481,7 +481,7 @@ def install_module(
                 shutil.rmtree(staged_path)
             raise
 
-    logger.info("Installed marketplace module '%s' to '%s'", module_id, final_path)
+    logger.debug("Installed marketplace module '%s' to '%s'", module_id, final_path)
     return final_path
 
 
@@ -519,7 +519,7 @@ def uninstall_module(
         if not module_path.exists():
             continue
         shutil.rmtree(module_path)
-        logger.info("Uninstalled module '%s' from '%s'", module_name, root)
+        logger.debug("Uninstalled module '%s' from '%s'", module_name, root)
         return
 
     roots_str = ", ".join(str(root) for root in candidate_roots)

{specfact_cli-0.37.3 → specfact_cli-0.37.4}/src/specfact_cli/registry/module_lifecycle.py

@@ -15,7 +15,6 @@ from specfact_cli.registry.module_packages import (
     discover_all_package_metadata,
     expand_disable_with_dependents,
     expand_enable_with_dependencies,
-    get_discovered_modules_for_state,
     merge_module_state,
     validate_disable_safe,
     validate_enable_safe,
@@ -34,24 +33,24 @@ def get_modules_with_state(
     disable_ids: list[str] | None = None,
 ) -> list[dict[str, Any]]:
     """Return discovered modules with version, enabled state, source, and trust metadata."""
-    modules_list = get_discovered_modules_for_state(enable_ids=enable_ids or [], disable_ids=disable_ids or [])
     discovered = discover_all_modules()
-    source_map = {entry.metadata.name: entry.source for entry in discovered}
-    official_map = {
-        entry.metadata.name: bool(
-            entry.metadata.publisher and entry.metadata.publisher.name.strip().lower() == "nold-ai"
+    discovered_list: list[tuple[str, str]] = [(entry.metadata.name, entry.metadata.version) for entry in discovered]
+    state = read_modules_state()
+    enabled_map = merge_module_state(discovered_list, state, enable_ids or [], disable_ids or [])
+
+    modules_list: list[dict[str, Any]] = []
+    for entry in discovered:
+        publisher_name = entry.metadata.publisher.name if entry.metadata.publisher else "unknown"
+        modules_list.append(
+            {
+                "id": entry.metadata.name,
+                "version": entry.metadata.version,
+                "enabled": enabled_map.get(entry.metadata.name, True),
+                "source": entry.source,
+                "official": bool(publisher_name.strip().lower() == "nold-ai"),
+                "publisher": publisher_name,
+            }
         )
-        for entry in discovered
-    }
-    publisher_map = {
-        entry.metadata.name: entry.metadata.publisher.name if entry.metadata.publisher else "unknown"
-        for entry in discovered
-    }
-    for item in modules_list:
-        module_id = str(item.get("id", ""))
-        item["source"] = source_map.get(module_id, "unknown")
-        item["official"] = official_map.get(module_id, False)
-        item["publisher"] = publisher_map.get(module_id, "unknown")
     return modules_list
 
 
@@ -80,7 +79,11 @@ def apply_module_state_update(*, enable_ids: list[str], disable_ids: list[str],
             for module_id, dependents in blocked_disable.items():
                 lines.append(f"Cannot disable '{module_id}': required by enabled modules: {', '.join(dependents)}")
             raise ValueError("\n".join(lines))
-    modules_list = get_discovered_modules_for_state(enable_ids=enable_ids, disable_ids=disable_ids)
+    final_enabled_map = merge_module_state(discovered_list, state, enable_ids, disable_ids)
+    modules_list = [
+        {"id": meta.name, "version": meta.version, "enabled": final_enabled_map.get(meta.name, True)}
+        for _package_dir, meta in packages
+    ]
     write_modules_state(modules_list)
     run_discovery_and_write_cache(__version__)
     return get_modules_with_state()

{specfact_cli-0.37.3 → specfact_cli-0.37.4}/src/specfact_cli/registry/module_packages.py

@@ -831,7 +831,7 @@ def register_module_package_commands(
                     f"schema version {meta.schema_version} required, current is {CURRENT_PROJECT_SCHEMA_VERSION}",
                 )
             )
-            logger.warning(
+            logger.debug(
                 "Module %s: Schema version %s required, but current is %s (skipped)",
                 meta.name,
                 meta.schema_version,
@@ -841,7 +841,7 @@ def register_module_package_commands(
         if meta.schema_version is None:
             logger.debug("Module %s: No schema version declared (assuming current)", meta.name)
         else:
-            logger.info("Module %s: Schema version %s (compatible)", meta.name, meta.schema_version)
+            logger.debug("Module %s: Schema version %s (compatible)", meta.name, meta.schema_version)
 
         if meta.schema_extensions:
             try:
@@ -892,7 +892,7 @@ def register_module_package_commands(
             elif operations:
                 partial_modules.append((meta.name, operations))
                 if is_debug_mode():
-                    logger.warning("Module %s: ModuleIOContract partial (%s)", meta.name, ", ".join(operations))
+                    logger.info("Module %s: ModuleIOContract partial (%s)", meta.name, ", ".join(operations))
                 protocol_partial += 1
             else:
                 legacy_modules.append(meta.name)