strictcli 0.11.0__tar.gz → 0.12.0__tar.gz

strictcli-0.12.0/.rlsbl/changes/.validated

@@ -0,0 +1 @@
+c161283293520cc94f39d9cfec589f6f4cf5a352

strictcli-0.12.0/.rlsbl/changes/0.12.0.jsonl

@@ -0,0 +1,2 @@
+{"commits":["495cf93ada5d5870e1fa83594b7c856f2f493f19","d604e97d59fa0e252bfd179a2e4d0f0f62757677"],"user_facing":true,"description":"**Breaking.** CWD auto-discovery of checks.toml removed; checks must be explicitly enabled via `checks_path=`. checks.toml now requires a top-level `app` field matching the app name.","type":"breaking"}
+{"commits":["fb4899a3e17880d31ac56b42aca24d359528c8af"],"user_facing":false}

strictcli-0.12.0/.rlsbl/changes/0.12.0.md

@@ -0,0 +1,5 @@
+## 0.12.0
+
+### Breaking
+
+- **Breaking.** CWD auto-discovery of checks.toml removed; checks must be explicitly enabled via `checks_path=`. checks.toml now requires a top-level `app` field matching the app name.

strictcli-0.12.0/.rlsbl/releases/v0.12.0.toml

@@ -0,0 +1,4 @@
+# Version bump type: patch, minor, or major
+bump = "minor"
+include = ["pypi", "npm"]
+exclude = []

{strictcli-0.11.0 → strictcli-0.12.0}/CHANGELOG.md

@@ -2,6 +2,12 @@
 
 # Changelog
 
+## 0.12.0
+
+### Breaking
+
+- **Breaking.** CWD auto-discovery of checks.toml removed; checks must be explicitly enabled via `checks_path=`. checks.toml now requires a top-level `app` field matching the app name.
+
 ## 0.11.0
 
 ### Features

{strictcli-0.11.0 → strictcli-0.12.0}/PKG-INFO

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

{strictcli-0.11.0 → strictcli-0.12.0}/package-lock.json

@@ -1,12 +1,12 @@
 {
   "name": "strictcli",
-  "version": "0.11.0",
+  "version": "0.12.0",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {
     "": {
       "name": "strictcli",
-      "version": "0.11.0",
+      "version": "0.12.0",
       "hasInstallScript": true,
       "license": "MIT"
     }

{strictcli-0.11.0 → strictcli-0.12.0}/package.json

@@ -1,6 +1,6 @@
 {
   "name": "strictcli",
-  "version": "0.11.0",
+  "version": "0.12.0",
   "description": "A strict, zero-dependency CLI framework for Python (npm wrapper)",
   "main": "index.js",
   "scripts": {

{strictcli-0.11.0 → strictcli-0.12.0}/pyproject.toml

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

{strictcli-0.11.0 → strictcli-0.12.0}/strictcli/__init__.py

@@ -2,7 +2,7 @@
 
 from __future__ import annotations
 
-__version__ = "0.11.0"
+__version__ = "0.12.0"
 
 __all__ = [
     "App", "Flag", "Arg", "Tag", "MutexGroup", "CoRequired", "Requires",
@@ -130,8 +130,8 @@ _CHECK_REQUIRED_FIELDS = {"tags", "severity", "fast", "pure", "needs_network", "
 _CHECK_VALID_SEVERITIES = {"error", "warn"}
 
 
-def _load_checks_toml(path: str | Path) -> dict[str, _CheckDef]:
-    """Parse and validate a checks.toml file, returning check definitions.
+def _load_checks_toml(path: str | Path) -> tuple[str, dict[str, _CheckDef]]:
+    """Parse and validate a checks.toml file, returning (app_name, check_defs).
 
     Raises ValueError on any schema violation, file error, or invalid TOML.
     """
@@ -145,13 +145,20 @@ def _load_checks_toml(path: str | Path) -> dict[str, _CheckDef]:
     except (tomllib.TOMLDecodeError, UnicodeDecodeError) as exc:
         raise ValueError(f"checks.toml: {exc}") from exc
 
-    # Only [checks] is allowed at the top level
+    # Only "app" and [checks] are allowed at the top level
     for key in data:
-        if key != "checks":
+        if key not in ("app", "checks"):
             raise ValueError(f'checks.toml: unknown top-level key "{key}"')
 
+    # Validate required "app" field
+    if "app" not in data:
+        raise ValueError('checks.toml: missing required top-level key "app"')
+    if not isinstance(data["app"], str) or not data["app"]:
+        raise ValueError('checks.toml: "app" must be a non-empty string')
+    app_name = data["app"]
+
     if "checks" not in data:
-        return {}
+        return (app_name, {})
 
     checks_section = data["checks"]
     if not isinstance(checks_section, dict):
@@ -243,7 +250,7 @@ def _load_checks_toml(path: str | Path) -> dict[str, _CheckDef]:
                     f'unknown check "{dep}"'
                 )
 
-    return result
+    return (app_name, result)
 
 
 class _HelpRequested(Exception):
@@ -709,14 +716,15 @@ class App:
             checks_toml_path = Path(self.checks_path).resolve()
             if not checks_toml_path.is_file():
                 raise ValueError(f"checks_path does not exist: {self.checks_path}")
-        else:
-            checks_toml_path = Path.cwd() / ".strictcli" / "checks.toml"
-        if checks_toml_path.is_file():
-            self._check_defs: dict[str, _CheckDef] = _load_checks_toml(checks_toml_path)
+            app_name, self._check_defs = _load_checks_toml(checks_toml_path)
+            if app_name != self.name:
+                raise ValueError(
+                    f'checks.toml: app "{app_name}" does not match app name "{self.name}"'
+                )
             self._checks_enabled = True
             self._register_check_command()
         else:
-            self._check_defs = {}
+            self._check_defs: dict[str, _CheckDef] = {}
             self._checks_enabled = False
 
     @property
@@ -730,12 +738,12 @@ class App:
             if not self._checks_enabled:
                 raise ValueError(
                     f'cannot register check "{name}": '
-                    f"no .strictcli/checks.toml found"
+                    f"checks not enabled"
                 )
             if name not in self._check_defs:
                 raise ValueError(
                     f'cannot register check "{name}": '
-                    f"not declared in .strictcli/checks.toml"
+                    f"not declared in checks.toml"
                 )
             if self._check_defs[name].impl is not None:
                 raise ValueError(f'check "{name}": duplicate registration')
@@ -756,7 +764,7 @@ class App:
         )
         if missing:
             return (
-                "checks declared in .strictcli/checks.toml but not registered: "
+                "checks declared in checks.toml but not registered: "
                 + ", ".join(missing)
             )
         return None

{strictcli-0.11.0 → strictcli-0.12.0}/tests/test_check_command.py

@@ -11,6 +11,8 @@ from strictcli import CheckResult
 
 
 TWO_CHECKS_TOML = """\
+app = "testapp"
+
 [checks.version-check]
 tags = ["release"]
 severity = "error"
@@ -29,6 +31,8 @@ depends_on = []
 """
 
 THREE_CHECKS_WITH_DEP_TOML = """\
+app = "testapp"
+
 [checks.base-check]
 tags = ["infra"]
 severity = "error"
@@ -66,11 +70,13 @@ def _setup_checks_app(tmp_path, monkeypatch, toml_content, register_impls=True,
 
     pass_results: dict mapping check name to CheckResult. If None, all return pass.
     """
-    (tmp_path / ".strictcli").mkdir(exist_ok=True)
-    (tmp_path / ".strictcli" / "checks.toml").write_text(toml_content)
-    monkeypatch.chdir(tmp_path)
+    toml_file = tmp_path / "checks.toml"
+    toml_file.write_text(toml_content)
 
-    app = strictcli.App(name="testapp", version="1.0.0", help="test app")
+    app = strictcli.App(
+        name="testapp", version="1.0.0", help="test app",
+        checks_path=str(toml_file),
+    )
 
     if register_impls:
         if pass_results is None:
@@ -317,13 +323,15 @@ class TestCheckIgnoreWarnings:
 
 
 class TestCheckNoContextFactory:
-    def test_no_context_factory_error(self, tmp_path, monkeypatch):
+    def test_no_context_factory_error(self, tmp_path):
         """Running checks without set_check_context produces error."""
-        (tmp_path / ".strictcli").mkdir(exist_ok=True)
-        (tmp_path / ".strictcli" / "checks.toml").write_text(TWO_CHECKS_TOML)
-        monkeypatch.chdir(tmp_path)
+        toml_file = tmp_path / "checks.toml"
+        toml_file.write_text(TWO_CHECKS_TOML)
 
-        app = strictcli.App(name="testapp", version="1.0.0", help="test app")
+        app = strictcli.App(
+            name="testapp", version="1.0.0", help="test app",
+            checks_path=str(toml_file),
+        )
         for name in app._check_defs:
             app._check_defs[name].impl = (
                 lambda ctx, n=name: CheckResult(status="pass", message=f"{n} OK")

{strictcli-0.11.0 → strictcli-0.12.0}/tests/test_check_discovery.py

@@ -1,13 +1,13 @@
 """Tests for check discovery and double-entry registration (Phase 3)."""
 
-import os
-
 import pytest
 
 import strictcli
 
 
 VALID_TOML = """\
+app = "testapp"
+
 [checks.lint-code]
 tags = ["code", "fast"]
 severity = "error"
@@ -26,6 +26,8 @@ depends_on = ["lint-code"]
 """
 
 INVALID_TOML = """\
+app = "testapp"
+
 [checks.BadName]
 tags = ["a"]
 severity = "error"
@@ -37,15 +39,15 @@ depends_on = []
 
 
 class TestDiscovery:
-    def test_discovers_checks_toml_in_cwd(self, tmp_path, monkeypatch):
+    def test_no_checks_path_ignores_cwd(self, tmp_path, monkeypatch):
+        """Without checks_path, CWD checks.toml is NOT discovered."""
         (tmp_path / ".strictcli").mkdir()
         (tmp_path / ".strictcli" / "checks.toml").write_text(VALID_TOML)
         monkeypatch.chdir(tmp_path)
 
         app = strictcli.App(name="testapp", version="1.0.0", help="test app")
-        assert app._checks_enabled is True
-        assert "lint-code" in app._check_defs
-        assert "check-deps" in app._check_defs
+        assert app._checks_enabled is False
+        assert app._check_defs == {}
 
     def test_no_toml_sets_checks_disabled(self, tmp_path, monkeypatch):
         monkeypatch.chdir(tmp_path)
@@ -54,13 +56,15 @@ class TestDiscovery:
         assert app._checks_enabled is False
         assert app._check_defs == {}
 
-    def test_invalid_toml_raises_valueerror(self, tmp_path, monkeypatch):
-        (tmp_path / ".strictcli").mkdir()
-        (tmp_path / ".strictcli" / "checks.toml").write_text(INVALID_TOML)
-        monkeypatch.chdir(tmp_path)
+    def test_invalid_toml_raises_valueerror(self, tmp_path):
+        toml_file = tmp_path / "checks.toml"
+        toml_file.write_text(INVALID_TOML)
 
         with pytest.raises(ValueError, match='invalid check name "BadName"'):
-            strictcli.App(name="testapp", version="1.0.0", help="test app")
+            strictcli.App(
+                name="testapp", version="1.0.0", help="test app",
+                checks_path=str(toml_file),
+            )
 
 
 class TestChecksPath:
@@ -116,8 +120,8 @@ class TestChecksPath:
                 checks_path=str(bad_path),
             )
 
-    def test_none_uses_cwd_discovery(self, tmp_path, monkeypatch):
-        """checks_path=None (default) falls back to CWD-based discovery."""
+    def test_none_means_disabled(self, tmp_path, monkeypatch):
+        """checks_path=None (default) means checks are disabled."""
         (tmp_path / ".strictcli").mkdir()
         (tmp_path / ".strictcli" / "checks.toml").write_text(VALID_TOML)
         monkeypatch.chdir(tmp_path)
@@ -126,17 +130,19 @@ class TestChecksPath:
             name="testapp", version="1.0.0", help="test app",
             checks_path=None,
         )
-        assert app._checks_enabled is True
-        assert "lint-code" in app._check_defs
+        assert app._checks_enabled is False
+        assert app._check_defs == {}
 
 
 class TestCheckDecorator:
-    def test_registers_implementation(self, tmp_path, monkeypatch):
-        (tmp_path / ".strictcli").mkdir()
-        (tmp_path / ".strictcli" / "checks.toml").write_text(VALID_TOML)
-        monkeypatch.chdir(tmp_path)
+    def test_registers_implementation(self, tmp_path):
+        toml_file = tmp_path / "checks.toml"
+        toml_file.write_text(VALID_TOML)
 
-        app = strictcli.App(name="testapp", version="1.0.0", help="test app")
+        app = strictcli.App(
+            name="testapp", version="1.0.0", help="test app",
+            checks_path=str(toml_file),
+        )
 
         @app.check("lint-code")
         def lint_impl(ctx):
@@ -144,24 +150,28 @@ class TestCheckDecorator:
 
         assert app._check_defs["lint-code"].impl is lint_impl
 
-    def test_undeclared_name_raises(self, tmp_path, monkeypatch):
-        (tmp_path / ".strictcli").mkdir()
-        (tmp_path / ".strictcli" / "checks.toml").write_text(VALID_TOML)
-        monkeypatch.chdir(tmp_path)
+    def test_undeclared_name_raises(self, tmp_path):
+        toml_file = tmp_path / "checks.toml"
+        toml_file.write_text(VALID_TOML)
 
-        app = strictcli.App(name="testapp", version="1.0.0", help="test app")
+        app = strictcli.App(
+            name="testapp", version="1.0.0", help="test app",
+            checks_path=str(toml_file),
+        )
 
         with pytest.raises(ValueError, match='cannot register check "nonexistent"'):
             @app.check("nonexistent")
             def bad_impl(ctx):
                 pass
 
-    def test_duplicate_registration_raises(self, tmp_path, monkeypatch):
-        (tmp_path / ".strictcli").mkdir()
-        (tmp_path / ".strictcli" / "checks.toml").write_text(VALID_TOML)
-        monkeypatch.chdir(tmp_path)
+    def test_duplicate_registration_raises(self, tmp_path):
+        toml_file = tmp_path / "checks.toml"
+        toml_file.write_text(VALID_TOML)
 
-        app = strictcli.App(name="testapp", version="1.0.0", help="test app")
+        app = strictcli.App(
+            name="testapp", version="1.0.0", help="test app",
+            checks_path=str(toml_file),
+        )
 
         @app.check("lint-code")
         def first(ctx):
@@ -177,19 +187,33 @@ class TestCheckDecorator:
 
         app = strictcli.App(name="testapp", version="1.0.0", help="test app")
 
-        with pytest.raises(ValueError, match="no .strictcli/checks.toml found"):
+        with pytest.raises(ValueError, match="checks not enabled"):
             @app.check("anything")
             def bad(ctx):
                 pass
 
+    def test_register_check_not_enabled_message(self, tmp_path):
+        """Verify exact error message for registering check when not enabled."""
+        app = strictcli.App(name="testapp", version="1.0.0", help="test app")
+
+        with pytest.raises(
+            ValueError,
+            match=r'cannot register check "my-check": checks not enabled',
+        ):
+            @app.check("my-check")
+            def impl(ctx):
+                pass
+
 
 class TestDoubleEntryValidation:
-    def test_missing_impl_error_in_test(self, tmp_path, monkeypatch):
-        (tmp_path / ".strictcli").mkdir()
-        (tmp_path / ".strictcli" / "checks.toml").write_text(VALID_TOML)
-        monkeypatch.chdir(tmp_path)
+    def test_missing_impl_error_in_test(self, tmp_path):
+        toml_file = tmp_path / "checks.toml"
+        toml_file.write_text(VALID_TOML)
 
-        app = strictcli.App(name="testapp", version="1.0.0", help="test app")
+        app = strictcli.App(
+            name="testapp", version="1.0.0", help="test app",
+            checks_path=str(toml_file),
+        )
 
         # Register a dummy command so the app has something to run
         @app.command("hello", help="say hello")
@@ -204,15 +228,17 @@ class TestDoubleEntryValidation:
         # check-deps is not registered -- should fail validation
         result = app.test(["hello"])
         assert result.exit_code == 1
-        assert "checks declared in .strictcli/checks.toml but not registered" in result.stderr
+        assert "checks declared in checks.toml but not registered" in result.stderr
         assert "check-deps" in result.stderr
 
-    def test_all_registered_passes_validation(self, tmp_path, monkeypatch):
-        (tmp_path / ".strictcli").mkdir()
-        (tmp_path / ".strictcli" / "checks.toml").write_text(VALID_TOML)
-        monkeypatch.chdir(tmp_path)
+    def test_all_registered_passes_validation(self, tmp_path):
+        toml_file = tmp_path / "checks.toml"
+        toml_file.write_text(VALID_TOML)
 
-        app = strictcli.App(name="testapp", version="1.0.0", help="test app")
+        app = strictcli.App(
+            name="testapp", version="1.0.0", help="test app",
+            checks_path=str(toml_file),
+        )
 
         @app.command("hello", help="say hello")
         def hello(**kw):
@@ -242,3 +268,28 @@ class TestDoubleEntryValidation:
         result = app.test(["hello"])
         assert result.exit_code == 0
         assert "hello" in result.stdout
+
+    def test_app_mismatch_raises(self, tmp_path):
+        """checks_path with app='wrong' but App name='testapp' raises ValueError."""
+        toml = """\
+app = "wrong"
+
+[checks.my-check]
+tags = ["a"]
+severity = "error"
+fast = true
+pure = true
+needs_network = false
+depends_on = []
+"""
+        toml_file = tmp_path / "checks.toml"
+        toml_file.write_text(toml)
+
+        with pytest.raises(
+            ValueError,
+            match=r'checks.toml: app "wrong" does not match app name "testapp"',
+        ):
+            strictcli.App(
+                name="testapp", version="1.0.0", help="test app",
+                checks_path=str(toml_file),
+            )

{strictcli-0.11.0 → strictcli-0.12.0}/tests/test_check_runner.py

@@ -113,7 +113,7 @@ class TestRunChecks:
     def _make_app_with_checks(self, check_defs, tmp_path, monkeypatch):
         """Build an App with pre-populated check definitions."""
         # Write a minimal TOML so the app discovers checks
-        toml_lines = []
+        toml_lines = ['app = "testapp"', ""]
         for name, cdef in check_defs.items():
             tags_str = ", ".join(f'"{t}"' for t in cdef.tags)
             deps_str = ", ".join(f'"{d}"' for d in cdef.depends_on)
@@ -126,11 +126,13 @@ class TestRunChecks:
             toml_lines.append(f"depends_on = [{deps_str}]")
             toml_lines.append("")
 
-        (tmp_path / ".strictcli").mkdir(exist_ok=True)
-        (tmp_path / ".strictcli" / "checks.toml").write_text("\n".join(toml_lines))
-        monkeypatch.chdir(tmp_path)
+        toml_file = tmp_path / "checks.toml"
+        toml_file.write_text("\n".join(toml_lines))
 
-        app = strictcli.App(name="testapp", version="1.0.0", help="test app")
+        app = strictcli.App(
+            name="testapp", version="1.0.0", help="test app",
+            checks_path=str(toml_file),
+        )
         # Inject the impl functions from our check_defs
         for name, cdef in check_defs.items():
             if cdef.impl is not None:

{strictcli-0.11.0 → strictcli-0.12.0}/tests/test_check_schema.py

@@ -6,6 +6,8 @@ import strictcli
 
 
 CHECKS_TOML = """\
+app = "testapp"
+
 [checks.lint-code]
 tags = ["code", "fast"]
 severity = "error"
@@ -28,11 +30,14 @@ class TestSchemaWithChecks:
     """--dump-schema includes checks key when checks are enabled."""
 
     def test_checks_key_present(self, tmp_path, monkeypatch):
-        (tmp_path / ".strictcli").mkdir()
-        (tmp_path / ".strictcli" / "checks.toml").write_text(CHECKS_TOML)
+        toml_file = tmp_path / "checks.toml"
+        toml_file.write_text(CHECKS_TOML)
         monkeypatch.chdir(tmp_path)
 
-        app = strictcli.App(name="testapp", version="1.0.0", help="test app")
+        app = strictcli.App(
+            name="testapp", version="1.0.0", help="test app",
+            checks_path=str(toml_file),
+        )
 
         @app.check("lint-code")
         def lint_impl(ctx):
@@ -54,11 +59,14 @@ class TestSchemaWithChecks:
         assert len(data["checks"]) == 2
 
     def test_check_metadata_correct(self, tmp_path, monkeypatch):
-        (tmp_path / ".strictcli").mkdir()
-        (tmp_path / ".strictcli" / "checks.toml").write_text(CHECKS_TOML)
+        toml_file = tmp_path / "checks.toml"
+        toml_file.write_text(CHECKS_TOML)
         monkeypatch.chdir(tmp_path)
 
-        app = strictcli.App(name="testapp", version="1.0.0", help="test app")
+        app = strictcli.App(
+            name="testapp", version="1.0.0", help="test app",
+            checks_path=str(toml_file),
+        )
 
         @app.check("lint-code")
         def lint_impl(ctx):

{strictcli-0.11.0 → strictcli-0.12.0}/tests/test_toml_loading.py

@@ -7,6 +7,8 @@ from strictcli import _load_checks_toml
 
 
 VALID_TOML = """\
+app = "testapp"
+
 [checks.lint-code]
 tags = ["code", "fast"]
 severity = "error"
@@ -29,7 +31,8 @@ class TestLoadChecksToml:
     def test_valid_toml_multiple_checks(self, tmp_path):
         f = tmp_path / "checks.toml"
         f.write_text(VALID_TOML)
-        result = _load_checks_toml(f)
+        app_name, result = _load_checks_toml(f)
+        assert app_name == "testapp"
         assert len(result) == 2
         assert "lint-code" in result
         assert "check-deps" in result
@@ -51,6 +54,8 @@ class TestLoadChecksToml:
 
     def test_missing_required_field(self, tmp_path):
         toml = """\
+app = "testapp"
+
 [checks.my-check]
 tags = ["a"]
 fast = true
@@ -65,6 +70,8 @@ depends_on = []
 
     def test_wrong_type_string_where_bool_expected(self, tmp_path):
         toml = """\
+app = "testapp"
+
 [checks.my-check]
 tags = ["a"]
 severity = "error"
@@ -80,6 +87,8 @@ depends_on = []
 
     def test_unknown_field(self, tmp_path):
         toml = """\
+app = "testapp"
+
 [checks.my-check]
 tags = ["a"]
 severity = "error"
@@ -96,6 +105,8 @@ extra = "nope"
 
     def test_unknown_top_level_key(self, tmp_path):
         toml = """\
+app = "testapp"
+
 [metadata]
 version = "1.0"
 
@@ -114,6 +125,8 @@ depends_on = []
 
     def test_invalid_check_name_uppercase(self, tmp_path):
         toml = """\
+app = "testapp"
+
 [checks.MyCheck]
 tags = ["a"]
 severity = "error"
@@ -129,6 +142,8 @@ depends_on = []
 
     def test_invalid_check_name_dots(self, tmp_path):
         toml = """\
+app = "testapp"
+
 [checks."my.check"]
 tags = ["a"]
 severity = "error"
@@ -144,6 +159,8 @@ depends_on = []
 
     def test_invalid_check_name_spaces(self, tmp_path):
         toml = """\
+app = "testapp"
+
 [checks."my check"]
 tags = ["a"]
 severity = "error"
@@ -159,6 +176,8 @@ depends_on = []
 
     def test_depends_on_nonexistent_check(self, tmp_path):
         toml = """\
+app = "testapp"
+
 [checks.my-check]
 tags = ["a"]
 severity = "error"
@@ -172,11 +191,11 @@ depends_on = ["nonexistent"]
         with pytest.raises(ValueError, match='depends_on references unknown check "nonexistent"'):
             _load_checks_toml(f)
 
-    def test_empty_file_returns_empty_dict(self, tmp_path):
+    def test_empty_file_raises_missing_app(self, tmp_path):
         f = tmp_path / "checks.toml"
         f.write_text("")
-        result = _load_checks_toml(f)
-        assert result == {}
+        with pytest.raises(ValueError, match='missing required top-level key "app"'):
+            _load_checks_toml(f)
 
     def test_invalid_toml_syntax(self, tmp_path):
         f = tmp_path / "checks.toml"
@@ -186,6 +205,8 @@ depends_on = ["nonexistent"]
 
     def test_invalid_severity_value(self, tmp_path):
         toml = """\
+app = "testapp"
+
 [checks.my-check]
 tags = ["a"]
 severity = "critical"
@@ -201,6 +222,8 @@ depends_on = []
 
     def test_tags_empty_list(self, tmp_path):
         toml = """\
+app = "testapp"
+
 [checks.my-check]
 tags = []
 severity = "error"
@@ -211,11 +234,13 @@ depends_on = []
 """
         f = tmp_path / "checks.toml"
         f.write_text(toml)
-        defs = _load_checks_toml(f)
+        _, defs = _load_checks_toml(f)
         assert defs["my-check"].tags == []
 
     def test_tags_with_empty_string(self, tmp_path):
         toml = """\
+app = "testapp"
+
 [checks.my-check]
 tags = [""]
 severity = "error"
@@ -233,3 +258,67 @@ depends_on = []
         f = tmp_path / "nonexistent.toml"
         with pytest.raises(ValueError, match="checks.toml:"):
             _load_checks_toml(f)
+
+    def test_missing_app_field(self, tmp_path):
+        toml = """\
+[checks.my-check]
+tags = ["a"]
+severity = "error"
+fast = true
+pure = true
+needs_network = false
+depends_on = []
+"""
+        f = tmp_path / "checks.toml"
+        f.write_text(toml)
+        with pytest.raises(ValueError, match='missing required top-level key "app"'):
+            _load_checks_toml(f)
+
+    def test_app_field_wrong_type(self, tmp_path):
+        toml = """\
+app = 42
+
+[checks.my-check]
+tags = ["a"]
+severity = "error"
+fast = true
+pure = true
+needs_network = false
+depends_on = []
+"""
+        f = tmp_path / "checks.toml"
+        f.write_text(toml)
+        with pytest.raises(ValueError, match='"app" must be a non-empty string'):
+            _load_checks_toml(f)
+
+    def test_app_field_empty_string(self, tmp_path):
+        toml = """\
+app = ""
+
+[checks.my-check]
+tags = ["a"]
+severity = "error"
+fast = true
+pure = true
+needs_network = false
+depends_on = []
+"""
+        f = tmp_path / "checks.toml"
+        f.write_text(toml)
+        with pytest.raises(ValueError, match='"app" must be a non-empty string'):
+            _load_checks_toml(f)
+
+    def test_app_field_only_no_checks_section(self, tmp_path):
+        toml = 'app = "testapp"\n'
+        f = tmp_path / "checks.toml"
+        f.write_text(toml)
+        app_name, result = _load_checks_toml(f)
+        assert app_name == "testapp"
+        assert result == {}
+
+    def test_app_name_returned(self, tmp_path):
+        f = tmp_path / "checks.toml"
+        f.write_text(VALID_TOML)
+        app_name, result = _load_checks_toml(f)
+        assert app_name == "testapp"
+        assert len(result) == 2

{strictcli-0.11.0 → strictcli-0.12.0}/uv.lock

@@ -232,7 +232,7 @@ wheels = [
 
 [[package]]
 name = "strictcli"
-version = "0.11.0"
+version = "0.12.0"
 source = { editable = "." }
 
 [package.dev-dependencies]

strictcli-0.11.0/.rlsbl/changes/.validated

@@ -1 +0,0 @@
-228f7f6e4e51965e7f3cc19ea1d112e915b585c8