strictcli 0.13.0__tar.gz → 0.15.0__tar.gz

strictcli-0.15.0/.github/workflows/publish.yml

@@ -0,0 +1,27 @@
+name: Publish
+on:
+  release:
+    types: [published]
+  workflow_dispatch:
+permissions:
+  contents: read
+  id-token: write
+jobs:
+  pypi:
+    runs-on: ubuntu-latest
+    steps:
+    - uses: actions/checkout@v6
+    - uses: astral-sh/setup-uv@v7
+    - run: uv build
+    - uses: pypa/gh-action-pypi-publish@release/v1
+  npm:
+    runs-on: ubuntu-latest
+    steps:
+    - uses: actions/checkout@v6
+    - uses: actions/setup-node@v6
+      with:
+        node-version: 24
+        registry-url: https://registry.npmjs.org
+    - run: npm publish --provenance --access public
+      env:
+        NODE_AUTH_TOKEN: ${{ secrets.NPM_TOKEN }}

strictcli-0.15.0/.npmignore

@@ -0,0 +1,34 @@
+# Development and CI
+tests/
+test/
+__tests__/
+.github/
+scripts/
+docs/
+todo/
+e2e/
+
+# rlsbl metadata
+.rlsbl/
+.rlsbl-monorepo/
+
+# Python artifacts (for multi-target projects)
+__pycache__/
+*.pyc
+*.py
+*.toml
+.venv/
+
+# Build/cache
+coverage/
+.nyc_output/
+
+# Editor/OS
+.DS_Store
+*.log
+.env
+.env.local
+
+# Selfdoc
+.selfdoc/
+selfdoc.json

strictcli-0.15.0/.rlsbl/bases/.github/workflows/publish.yml

@@ -0,0 +1,27 @@
+name: Publish
+on:
+  release:
+    types: [published]
+  workflow_dispatch:
+permissions:
+  contents: read
+  id-token: write
+jobs:
+  pypi:
+    runs-on: ubuntu-latest
+    steps:
+    - uses: actions/checkout@v6
+    - uses: astral-sh/setup-uv@v7
+    - run: uv build
+    - uses: pypa/gh-action-pypi-publish@release/v1
+  npm:
+    runs-on: ubuntu-latest
+    steps:
+    - uses: actions/checkout@v6
+    - uses: actions/setup-node@v6
+      with:
+        node-version: 24
+        registry-url: https://registry.npmjs.org
+    - run: npm publish --provenance --access public
+      env:
+        NODE_AUTH_TOKEN: ${{ secrets.NPM_TOKEN }}

strictcli-0.15.0/.rlsbl/bases/.npmignore

@@ -0,0 +1,34 @@
+# Development and CI
+tests/
+test/
+__tests__/
+.github/
+scripts/
+docs/
+todo/
+e2e/
+
+# rlsbl metadata
+.rlsbl/
+.rlsbl-monorepo/
+
+# Python artifacts (for multi-target projects)
+__pycache__/
+*.pyc
+*.py
+*.toml
+.venv/
+
+# Build/cache
+coverage/
+.nyc_output/
+
+# Editor/OS
+.DS_Store
+*.log
+.env
+.env.local
+
+# Selfdoc
+.selfdoc/
+selfdoc.json

strictcli-0.15.0/.rlsbl/changes/.validated

@@ -0,0 +1 @@
+a6dff298b7a2dad7fa928b088eeb4f3f9c0b2cf5

strictcli-0.15.0/.rlsbl/changes/0.13.0.md

@@ -0,0 +1,5 @@
+## 0.13.0
+
+### Features
+
+- **New feature.** Schema dump (--dump-schema) now includes a project_id field read from pyproject.toml, providing provenance for schema validation.

strictcli-0.15.0/.rlsbl/changes/0.14.0.jsonl

@@ -0,0 +1,3 @@
+{"commits":["f2f2de83edc0b7fe24f46744201e8be1407f28a3"],"user_facing":false}
+{"commits":["a2e1d64fb5f695169fafa63e5d1b69fe5c4e043d"],"user_facing":true,"description":"**New feature.** `checks_embed` parameter accepts raw TOML bytes, enabling inline checks configuration without a file on disk.","type":"feature"}
+{"commits":["09799755dc2ef93a088f739a46001102bcb2aa5e"],"user_facing":false}

strictcli-0.15.0/.rlsbl/changes/0.14.0.md

@@ -0,0 +1,5 @@
+## 0.14.0
+
+### Features
+
+- **New feature.** `checks_embed` parameter accepts raw TOML bytes, enabling inline checks configuration without a file on disk.

strictcli-0.15.0/.rlsbl/changes/0.15.0.jsonl

@@ -0,0 +1,6 @@
+{"commits":["057edc5200703fe2a2d37d28f488833b35254a06"],"user_facing":false}
+{"commits":["e6af4e228227d55585c09ec9012cca846eff2651"],"user_facing":false}
+{"commits":["e5dc999d0d920ba0737d2c565be818af8465de1c"],"user_facing":true,"description":"**Breaking.** `config set` now validates keys against registered flags and rejects unknown keys.","type":"breaking"}
+{"commits":["e5dc999d0d920ba0737d2c565be818af8465de1c"],"user_facing":true,"description":"**New feature.** `config set` coerces values to the flag's declared type (int, bool, float) before writing to config.","type":"feature"}
+{"commits":["4cf2df20f76491a8d9b504307de8d78ca514e7ab"],"user_facing":false}
+{"commits":["050a53f9af245a308f6d5895307beb7f479c9c27"],"user_facing":false}

{strictcli-0.13.0 → strictcli-0.15.0}/.rlsbl/hashes.json

@@ -1,6 +1,6 @@
 {
   ".github/workflows/ci.yml": "b4e075171b131d25f9cce55a3e3a30732e95a2a794af00f354ee3200626d6a04",
-  ".github/workflows/publish.yml": "8befe0164321eb68e9465c634aa15d6ccb30d46fa669f7d30bf3ed064ad258ba",
+  ".github/workflows/publish.yml": "e77dcd0b5d1c1abae5424e5f3b73862f44f06b5d46209cbe71bc04d6baa6b5be",
   "CHANGELOG.md": "7a6ec46141007c18090a0ff325e2dd68cd51588c4e0b5af7b580365e5eaaca85",
   ".gitignore": "c89d76c6c43e7f0897b3472a15b7c0103e0be5b3a04c2b44e585fa48bdcda2bb",
   "LICENSE": "e958892abc1dd5d991fca6d528e695d335f25773d373d2e0be5f4a9507d8f8d5",
@@ -11,5 +11,6 @@
   ".rlsbl/hooks/pre-checks.sh": "60c57d4563263124530e6f33aca77550ea2161ad3b33b3ba6ad1c03fea9eb6e2",
   ".rlsbl/lint/python.toml": "9cfbcef2e010d5fad243437bf2fd8df54a1fae70ee0762f9d11da70fe207501c",
   ".rlsbl/lint/go.toml": "807d5cf760baad6d571a5bfd43cb163efe356f7462ba9f1918d72de45f62339f",
-  ".rlsbl/lint/npm.toml": "baa36014e65923d8cc5c8b83c77fa651da79351627570476402da66840027645"
+  ".rlsbl/lint/npm.toml": "baa36014e65923d8cc5c8b83c77fa651da79351627570476402da66840027645",
+  ".npmignore": "7c0eea9785c0318aeb39bf8b670ce8384f4a5b812fde69ce23d7d018057325a3"
 }

strictcli-0.15.0/.rlsbl/releases/v0.15.0.toml

@@ -0,0 +1,8 @@
+# Version bump type: patch, minor, or major
+bump = "minor"
+# Short description of this release (required)
+description = "Config set type coercion and key validation"
+# Optional context explaining why these changes were made
+context = "config set now validates keys against registered flags and coerces string values to the flag's declared type before writing. This is a breaking change: unknown keys are rejected."
+include = ["pypi", "npm"]
+exclude = []

strictcli-0.15.0/.rlsbl/version

@@ -0,0 +1 @@
+0.60.1

{strictcli-0.13.0 → strictcli-0.15.0}/CHANGELOG.md

@@ -2,6 +2,31 @@
 
 # Changelog
 
+## 0.15.0
+
+Config set type coercion and key validation
+
+<details>
+<summary>Context</summary>
+
+config set now validates keys against registered flags and coerces string values to the flag's declared type before writing. This is a breaking change: unknown keys are rejected.
+
+</details>
+
+### Breaking
+
+- **Breaking.** `config set` now validates keys against registered flags and rejects unknown keys.
+
+### Features
+
+- **New feature.** `config set` coerces values to the flag's declared type (int, bool, float) before writing to config.
+
+## 0.14.0
+
+### Features
+
+- **New feature.** `checks_embed` parameter accepts raw TOML bytes, enabling inline checks configuration without a file on disk.
+
 ## 0.13.0
 
 ### Features

{strictcli-0.13.0 → strictcli-0.15.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: strictcli
-Version: 0.13.0
+Version: 0.15.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.13.0 → strictcli-0.15.0}/package-lock.json

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

{strictcli-0.13.0 → strictcli-0.15.0}/package.json

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

{strictcli-0.13.0 → strictcli-0.15.0}/pyproject.toml

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

{strictcli-0.13.0 → strictcli-0.15.0}/strictcli/__init__.py

@@ -2,7 +2,7 @@
 
 from __future__ import annotations
 
-__version__ = "0.13.0"
+__version__ = "0.15.0"
 
 __all__ = [
     "App", "Flag", "Arg", "Tag", "MutexGroup", "CoRequired", "Requires",
@@ -130,37 +130,32 @@ _CHECK_REQUIRED_FIELDS = {"tags", "severity", "fast", "pure", "needs_network", "
 _CHECK_VALID_SEVERITIES = {"error", "warn"}
 
 
-def _load_checks_toml(path: str | Path) -> tuple[str, dict[str, _CheckDef]]:
-    """Parse and validate a checks.toml file, returning (app_name, check_defs).
+def _parse_checks_toml(data: bytes) -> tuple[str, dict[str, _CheckDef]]:
+    """Parse and validate checks TOML data, returning (app_name, check_defs).
 
-    Raises ValueError on any schema violation, file error, or invalid TOML.
+    Raises ValueError on any schema violation or invalid TOML.
     """
-    path = Path(path)
     try:
-        raw = path.read_bytes()
-    except OSError as exc:
-        raise ValueError(f"checks.toml: {exc}") from exc
-    try:
-        data = tomllib.loads(raw.decode())
+        parsed = tomllib.loads(data.decode())
     except (tomllib.TOMLDecodeError, UnicodeDecodeError) as exc:
         raise ValueError(f"checks.toml: {exc}") from exc
 
     # Only "app" and [checks] are allowed at the top level
-    for key in data:
+    for key in parsed:
         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:
+    if "app" not in parsed:
         raise ValueError('checks.toml: missing required top-level key "app"')
-    if not isinstance(data["app"], str) or not data["app"]:
+    if not isinstance(parsed["app"], str) or not parsed["app"]:
         raise ValueError('checks.toml: "app" must be a non-empty string')
-    app_name = data["app"]
+    app_name = parsed["app"]
 
-    if "checks" not in data:
+    if "checks" not in parsed:
         return (app_name, {})
 
-    checks_section = data["checks"]
+    checks_section = parsed["checks"]
     if not isinstance(checks_section, dict):
         raise ValueError("checks.toml: [checks] must be a table")
 
@@ -253,6 +248,19 @@ def _load_checks_toml(path: str | Path) -> tuple[str, dict[str, _CheckDef]]:
     return (app_name, result)
 
 
+def _load_checks_toml(path: str | Path) -> tuple[str, dict[str, _CheckDef]]:
+    """Read and parse a checks.toml file, returning (app_name, check_defs).
+
+    Raises ValueError on any file error, schema violation, or invalid TOML.
+    """
+    path = Path(path)
+    try:
+        raw = path.read_bytes()
+    except OSError as exc:
+        raise ValueError(f"checks.toml: {exc}") from exc
+    return _parse_checks_toml(raw)
+
+
 class _HelpRequested(Exception):
     """Raised when --help or -h is encountered."""
 
@@ -277,6 +285,21 @@ class _ParseError(Exception):
         self.command_prefix = command_prefix
 
 
+def _strict_bool(s: str) -> bool:
+    """Parse a boolean string strictly.
+
+    Accepts: 1, true, yes (case-insensitive) -> True
+    Accepts: 0, false, no (case-insensitive) -> False
+    Everything else raises ValueError.
+    """
+    lower = s.lower()
+    if lower in ("1", "true", "yes"):
+        return True
+    if lower in ("0", "false", "no"):
+        return False
+    raise ValueError(f"expected boolean, got '{s}'")
+
+
 def _strict_int(s: str) -> int:
     """Parse an integer string strictly -- no leading/trailing whitespace allowed.
 
@@ -675,6 +698,7 @@ class App:
     config_path: str | None = None
     config_format: str = "json"
     checks_path: str | Path | None = None
+    checks_embed: bytes | None = None
     flags: list[Flag] = field(default_factory=list)
     _commands: dict[str, Command] = field(default_factory=dict)
     _groups: dict[str, Group] = field(default_factory=dict)
@@ -712,6 +736,8 @@ class App:
             self._register_config_group()
         # Discover checks TOML
         self._check_context_factory: Callable | None = None
+        if self.checks_path is not None and self.checks_embed is not None:
+            raise ValueError("cannot use both checks_path and checks_embed")
         if self.checks_path is not None:
             checks_toml_path = Path(self.checks_path).resolve()
             if not checks_toml_path.is_file():
@@ -723,6 +749,14 @@ class App:
                 )
             self._checks_enabled = True
             self._register_check_command()
+        elif self.checks_embed is not None:
+            app_name, self._check_defs = _parse_checks_toml(self.checks_embed)
+            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: dict[str, _CheckDef] = {}
             self._checks_enabled = False
@@ -995,7 +1029,7 @@ class App:
         )
 
         # config set
-        def _config_set_handler(key, value, **_kw) -> None:
+        def _config_set_handler(key, value, **_kw) -> int:
             path = _config_path(
                 app_ref.name,
                 override=app_ref.config_path,
@@ -1004,26 +1038,53 @@ class App:
             dir_path = os.path.dirname(path)
             os.makedirs(dir_path, exist_ok=True)
             # Read existing config
-            existing: dict = {}
-            if os.path.isfile(path):
-                if app_ref.config_format == "toml":
+            existing = _load_config(
+                app_ref.name,
+                config_path_override=app_ref.config_path,
+                config_format=app_ref.config_format,
+            )
+
+            # Look up the key against registered flags
+            all_flags = app_ref._collect_all_flags()
+            matched_flag = None
+            for f in all_flags:
+                if _flag_param_name(f.name) == key:
+                    matched_flag = f
+                    break
+            if matched_flag is None:
+                print(f"config set: unknown key '{key}'", file=sys.stderr)
+                return 1
+
+            # Coerce the string value to the flag's type
+            try:
+                if matched_flag.type == bool:
+                    typed_value = _strict_bool(value)
+                elif matched_flag.type == int:
                     try:
-                        with open(path, "rb") as fh:
-                            existing = tomllib.load(fh)
-                    except (tomllib.TOMLDecodeError, UnicodeDecodeError):
-                        existing = {}
-                else:
+                        typed_value = _strict_int(value)
+                    except ValueError:
+                        raise ValueError(f"expected integer, got '{value}'")
+                elif matched_flag.type == float:
                     try:
-                        with open(path) as fh:
-                            existing = json.loads(fh.read())
-                    except (json.JSONDecodeError, ValueError):
-                        existing = {}
-            existing[key] = value
+                        typed_value = _strict_float(value)
+                    except ValueError as fe:
+                        msg = str(fe)
+                        if msg in ("NaN is not allowed", "Inf is not allowed"):
+                            raise
+                        raise ValueError(f"expected float, got '{value}'") from fe
+                else:  # str
+                    typed_value = value
+            except ValueError as e:
+                print(f"config set: key '{key}': {e}", file=sys.stderr)
+                return 1
+
+            existing[key] = typed_value
             if app_ref.config_format == "toml":
                 _write_toml_flat(existing, path)
             else:
                 with open(path, "w") as fh:
                     fh.write(json.dumps(existing, indent=2) + "\n")
+            return 0
 
         config_grp.commands["set"] = Command(
             name="set",
@@ -1343,12 +1404,9 @@ class App:
                 env_val = os.environ.get(f.env)
                 if env_val is not None:
                     if f.type is bool:
-                        lower = env_val.lower()
-                        if lower in ("1", "true", "yes"):
-                            cli_set[f.name] = True
-                        elif lower in ("0", "false", "no"):
-                            cli_set[f.name] = False
-                        else:
+                        try:
+                            cli_set[f.name] = _strict_bool(env_val)
+                        except ValueError:
                             raise _ParseError(
                                 f"invalid boolean value {env_val!r} for env var "
                                 f"'{f.env}' (flag '--{f.name}')"
@@ -1741,12 +1799,9 @@ def _parse_command(
             env_val = os.environ.get(f.env)
             if env_val is not None:
                 if f.type is bool:
-                    lower = env_val.lower()
-                    if lower in ("1", "true", "yes"):
-                        cli_set[f.name] = True
-                    elif lower in ("0", "false", "no"):
-                        cli_set[f.name] = False
-                    else:
+                    try:
+                        cli_set[f.name] = _strict_bool(env_val)
+                    except ValueError:
                         raise _ParseError(
                             f"invalid boolean value {env_val!r} for env var "
                             f"'{f.env}' (flag '--{f.name}')"

{strictcli-0.13.0 → strictcli-0.15.0}/tests/test_check_discovery.py

@@ -293,3 +293,54 @@ depends_on = []
                 name="testapp", version="1.0.0", help="test app",
                 checks_path=str(toml_file),
             )
+
+
+class TestChecksEmbed:
+    def test_checks_embed_enables_checks(self, tmp_path, monkeypatch):
+        """checks_embed with valid TOML bytes enables checks."""
+        monkeypatch.chdir(tmp_path)
+
+        app = strictcli.App(
+            name="testapp", version="1.0.0", help="test app",
+            checks_embed=VALID_TOML.encode(),
+        )
+        assert app._checks_enabled is True
+        assert "lint-code" in app._check_defs
+        assert "check-deps" in app._check_defs
+
+    def test_checks_embed_and_checks_path_raises(self, tmp_path, monkeypatch):
+        """Using both checks_path and checks_embed raises ValueError."""
+        monkeypatch.chdir(tmp_path)
+        toml_file = tmp_path / "checks.toml"
+        toml_file.write_text(VALID_TOML)
+
+        with pytest.raises(ValueError, match="cannot use both checks_path and checks_embed"):
+            strictcli.App(
+                name="testapp", version="1.0.0", help="test app",
+                checks_path=str(toml_file),
+                checks_embed=VALID_TOML.encode(),
+            )
+
+    def test_checks_embed_invalid_toml(self, tmp_path, monkeypatch):
+        """checks_embed with garbage bytes raises ValueError."""
+        monkeypatch.chdir(tmp_path)
+
+        with pytest.raises(ValueError, match="checks.toml:"):
+            strictcli.App(
+                name="testapp", version="1.0.0", help="test app",
+                checks_embed=b"\x80\x81 not valid toml {{{{",
+            )
+
+    def test_checks_embed_wrong_app_name(self, tmp_path, monkeypatch):
+        """checks_embed with mismatched app name raises ValueError."""
+        monkeypatch.chdir(tmp_path)
+        wrong_toml = 'app = "wrong"\n'
+
+        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_embed=wrong_toml.encode(),
+            )