strictcli 0.32.0__tar.gz → 0.32.1__tar.gz

{strictcli-0.32.0 → strictcli-0.32.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: strictcli
-Version: 0.32.0
+Version: 0.32.1
 Summary: A strict CLI framework for Python
 Project-URL: Homepage, https://github.com/smm-h/strictcli
 Project-URL: Repository, https://github.com/smm-h/strictcli

{strictcli-0.32.0 → strictcli-0.32.1}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "strictcli"
-version = "0.32.0"
+version = "0.32.1"
 description = "A strict CLI framework for Python"
 readme = "README.md"
 license = "MIT"

{strictcli-0.32.0 → strictcli-0.32.1}/strictcli/__init__.py

@@ -2,7 +2,7 @@
 
 from __future__ import annotations
 
-__version__ = "0.32.0"
+__version__ = "0.32.1"
 
 __all__ = [
     "App", "Flag", "Arg", "FlagSet", "MutexGroup", "CoRequired", "Requires",
@@ -2905,6 +2905,27 @@ class App:
             coverage_dir = self._coverage_dir
             manifest_path = self._coverage_manifest_path
 
+            # Subject-matter gating (the sanctioned skip class, mirroring
+            # project-type gating): when the anchored coverage root holds NEITHER
+            # a committed manifest NOR any shard files, this is not the app's own
+            # development tree -- e.g. an installed app running its checks from a
+            # foreign project's cwd, where the construction-anchored root points
+            # at a directory with no coverage state. Report a visible SKIP instead
+            # of failing with the app's entire command surface listed as
+            # uncovered. When EITHER exists, behavior is unchanged: a partial
+            # manifest still fails honestly, and an empty-manifest file present
+            # still means "coverage configured but empty" = fail listing all.
+            manifest_exists = bool(manifest_path) and os.path.isfile(manifest_path)
+            shards_exist = bool(coverage_dir) and os.path.isdir(coverage_dir) and any(
+                fname.endswith(".jsonl") for fname in os.listdir(coverage_dir)
+            )
+            if not manifest_exists and not shards_exist:
+                anchor = os.path.dirname(manifest_path) if manifest_path else coverage_dir
+                return reporter.skipped(
+                    f"no coverage state at {anchor} -- cli-test-coverage applies "
+                    "to the app's own development tree"
+                )
+
             covered: set[str] = set()
 
             # Seed from the committed manifest -- this is what makes the verdict

{strictcli-0.32.0 → strictcli-0.32.1}/tests/test_coverage.py

@@ -175,16 +175,36 @@ class TestCoverageCheck:
         cov_result = next(r for r in results if r.name == "cli-test-coverage")
         assert cov_result.status == "pass"
 
-    def test_empty_coverage_fails_naming_all(self, tmp_path):
-        """No shards and no manifest -> deterministic FAIL naming every command.
-
-        The verdict is derived from committed state (manifest union shards), so an
-        empty everything state fails by listing every uncovered command rather than
-        the old machine-dependent "no coverage data" error.
+    def test_zero_coverage_state_skips(self, tmp_path):
+        """No shards and no manifest -> SKIP with a subject-matter reason.
+
+        Subject-matter gating: when the anchored coverage root holds NEITHER a
+        committed manifest NOR any shard files, this is not the app's own
+        development tree (e.g. an installed app running its checks from a foreign
+        project's cwd). The check reports a visible SKIP naming the anchored path
+        rather than failing with the whole command surface listed as uncovered.
         """
         app = _make_app(tmp_path)
         # Don't run any test() or call() -- no shards, no manifest
 
+        results, _, code = app.run_checks(
+            SimpleCtx(project_root=tmp_path),
+            run_all=True,
+        )
+        cov_result = next(r for r in results if r.name == "cli-test-coverage")
+        assert cov_result.status == "skip"
+        assert "development tree" in cov_result.message
+        # Reason names the anchored .strictcli path, not the foreign cwd.
+        assert str(tmp_path / ".strictcli") in cov_result.message
+
+    def test_empty_manifest_file_present_fails_listing_all(self, tmp_path):
+        """An empty-manifest file present means "coverage configured but empty"
+        -> FAIL listing every command, NOT a skip. The skip class only triggers
+        when NEITHER a manifest NOR any shards exist."""
+        app = _make_app(tmp_path)
+        manifest_path = tmp_path / ".strictcli" / "test-coverage.json"
+        manifest_path.write_text("[]\n")
+
         results, _, code = app.run_checks(
             SimpleCtx(project_root=tmp_path),
             run_all=True,

{strictcli-0.32.0 → strictcli-0.32.1}/uv.lock

@@ -232,7 +232,7 @@ wheels = [
 
 [[package]]
 name = "strictcli"
-version = "0.32.0"
+version = "0.32.1"
 source = { editable = "." }
 dependencies = [
     { name = "tomlkit" },