strictcli 0.20.0__tar.gz → 0.20.1__tar.gz

strictcli-0.20.1/.rlsbl/changes/.validated

@@ -0,0 +1 @@
+5b2b40dead26752424a198c76f3634593b1cdb9b

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

@@ -0,0 +1,18 @@
+## 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.

{strictcli-0.20.0 → strictcli-0.20.1}/.rlsbl/changes/0.15.1.md

@@ -1,5 +1,14 @@
 ## 0.15.1
 
+Config show bool parity and negative values in config set
+
+<details>
+<summary>Context</summary>
+
+config show now outputs lowercase true/false matching Go. config set accepts negative numeric values.
+
+</details>
+
 ### Breaking
 
 - **Breaking.** Config set now accepts negative numeric values. Unknown single-dash tokens produce 'unexpected argument' instead of 'unknown flag'.

{strictcli-0.20.0 → strictcli-0.20.1}/.rlsbl/changes/0.16.0.md

@@ -1,5 +1,14 @@
 ## 0.16.0
 
+unique and env_separator flag fields, config array coercion, config set repeatable support
+
+<details>
+<summary>Context</summary>
+
+Breaking: config show now requires --plain or --json. New features: unique field enforces no duplicate values on repeatable flags, env_separator controls how env vars are split into arrays, config arrays are coerced to declared types, config set supports repeatable flags with --clear and --default.
+
+</details>
+
 ### Breaking
 
 - **Config show.** `config show` now requires explicit `--plain` or `--json` flag instead of defaulting to plain text.

{strictcli-0.20.0 → strictcli-0.20.1}/.rlsbl/changes/0.16.1.md

@@ -1,5 +1,7 @@
 ## 0.16.1
 
+Error message fixes and help text improvements
+
 ### Fixes
 
 - **Fix.** `config set` mutex error messages now include the `config set:` prefix for consistency.

{strictcli-0.20.0 → strictcli-0.20.1}/.rlsbl/changes/0.17.0.md

@@ -1,5 +1,7 @@
 ## 0.17.0
 
+Repeatable flag default validation and help text display
+
 ### Breaking
 
 - **Breaking.** Repeatable flag defaults are now validated at registration: must be a list, `default=[]` is rejected as redundant, and element types must match the flag type.

strictcli-0.20.1/.rlsbl/changes/0.18.0.md

@@ -0,0 +1,14 @@
+## 0.18.0
+
+Public API for running checks programmatically
+
+<details>
+<summary>Context</summary>
+
+Adds App.run_checks(), format_check_results(), format_check_results_json(), and CheckRunResult — a stable public API replacing the private _filter_checks/_resolve_check_order/_run_checks/_check_format_human functions that consumers like rlsbl were forced to use.
+
+</details>
+
+### Features
+
+- **New.** Public API for running checks programmatically: `App.run_checks()`, `format_check_results()`, `format_check_results_json()`, `CheckRunResult`.

strictcli-0.20.1/.rlsbl/changes/0.20.0.md

@@ -0,0 +1,19 @@
+## 0.20.0
+
+Frozen Command, command tags, tag contracts
+
+<details>
+<summary>Context</summary>
+
+Breaking: Command dataclass is now frozen=True with tuple fields (was mutable list). New: command tags (string labels with group inheritance) and tag contracts (registration-time validation that tagged commands have required flags).
+
+</details>
+
+### Breaking
+
+- **Breaking.** Command dataclass is now `frozen=True` with `tuple` fields instead of `list`.
+
+### Features
+
+- **New.** Command tags: string labels on commands and groups with group-to-command inheritance.
+- **New.** Tag contracts: registration-time validation that tagged commands have required flags.

strictcli-0.20.1/.rlsbl/changes/0.20.1.jsonl

@@ -0,0 +1,3 @@
+{"commits":["88874b77625f9f096a9e7c38212cb65ab7bd4188"],"user_facing":false}
+{"commits":["0e5647f165e7b5ebb4bb135bd664f072306b282a"],"user_facing":true,"description":"**Fix.** Flags named after Python keywords (`--global`, `--class`, `--import`) now produce valid parameter names with a trailing underscore (e.g., `global_`).","type":"fix"}
+{"commits":["8e769b6aa17c51d30edae5a074f4a6f31c624853"],"user_facing":false}

strictcli-0.20.1/.rlsbl/changes/0.20.1.md

@@ -0,0 +1,14 @@
+## 0.20.1
+
+Fix keyword collision in flag parameter names
+
+<details>
+<summary>Context</summary>
+
+Flags named after Python keywords (--global, --class, --import, etc.) now produce valid parameter names with a trailing underscore per PEP 8 convention.
+
+</details>
+
+### Fixes
+
+- **Fix.** Flags named after Python keywords (`--global`, `--class`, `--import`) now produce valid parameter names with a trailing underscore (e.g., `global_`).

strictcli-0.20.1/.rlsbl/releases/v0.20.1.toml

@@ -0,0 +1,8 @@
+# Version bump type: patch, minor, or major
+bump = "patch"
+# Short description of this release (required)
+description = "Fix keyword collision in flag parameter names"
+# Optional context explaining why these changes were made
+context = "Flags named after Python keywords (--global, --class, --import, etc.) now produce valid parameter names with a trailing underscore per PEP 8 convention."
+include = ["pypi", "npm"]
+exclude = []

{strictcli-0.20.0 → strictcli-0.20.1}/CHANGELOG.md

@@ -2,6 +2,21 @@
 
 # Changelog
 
+## 0.20.1
+
+Fix keyword collision in flag parameter names
+
+<details>
+<summary>Context</summary>
+
+Flags named after Python keywords (--global, --class, --import, etc.) now produce valid parameter names with a trailing underscore per PEP 8 convention.
+
+</details>
+
+### Fixes
+
+- **Fix.** Flags named after Python keywords (`--global`, `--class`, `--import`) now produce valid parameter names with a trailing underscore (e.g., `global_`).
+
 ## 0.20.0
 
 Frozen Command, command tags, tag contracts

{strictcli-0.20.0 → strictcli-0.20.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: strictcli
-Version: 0.20.0
+Version: 0.20.1
 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.20.0 → strictcli-0.20.1}/package-lock.json

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

{strictcli-0.20.0 → strictcli-0.20.1}/package.json

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

{strictcli-0.20.0 → strictcli-0.20.1}/pyproject.toml

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

{strictcli-0.20.0 → strictcli-0.20.1}/strictcli/__init__.py

@@ -2,7 +2,7 @@
 
 from __future__ import annotations
 
-__version__ = "0.20.0"
+__version__ = "0.20.1"
 
 __all__ = [
     "App", "Flag", "Arg", "FlagSet", "MutexGroup", "CoRequired", "Requires",
@@ -12,6 +12,7 @@ __all__ = [
 ]
 
 import contextlib
+import keyword
 import fnmatch
 import importlib.metadata
 import inspect
@@ -2553,8 +2554,15 @@ def _parse_command(
 
 
 def _flag_param_name(flag_name: str) -> str:
-    """Convert a flag name like '--dry-run' to a Python parameter name 'dry_run'."""
-    return flag_name.lstrip("-").replace("-", "_")
+    """Convert a flag name like '--dry-run' to a Python parameter name 'dry_run'.
+
+    If the result is a Python keyword (e.g. 'global', 'class'), appends '_'
+    per PEP 8 convention (e.g. 'global_', 'class_').
+    """
+    name = flag_name.lstrip("-").replace("-", "_")
+    if keyword.iskeyword(name):
+        name += "_"
+    return name
 
 
 def _build_and_validate_command(

strictcli-0.20.1/tests/test_keyword_flags.py

@@ -0,0 +1,117 @@
+"""Tests for Python keyword collision handling in flag parameter names."""
+
+import json
+import os
+
+import strictcli
+from strictcli import _flag_param_name
+
+
+# --- Unit tests for _flag_param_name ---
+
+
+def test_non_keyword_unchanged():
+    """Non-keyword flag names convert normally (dashes to underscores)."""
+    assert _flag_param_name("--dry-run") == "dry_run"
+
+
+def test_keyword_global():
+    """'global' is a Python keyword, should get trailing underscore."""
+    assert _flag_param_name("--global") == "global_"
+
+
+def test_keyword_class():
+    """'class' is a Python keyword, should get trailing underscore."""
+    assert _flag_param_name("--class") == "class_"
+
+
+def test_keyword_import():
+    """'import' is a Python keyword, should get trailing underscore."""
+    assert _flag_param_name("--import") == "import_"
+
+
+def test_keyword_for():
+    """'for' is a Python keyword, should get trailing underscore."""
+    assert _flag_param_name("--for") == "for_"
+
+
+def test_keyword_in():
+    """'in' is a Python keyword, should get trailing underscore."""
+    assert _flag_param_name("--in") == "in_"
+
+
+def test_non_keyword_globally():
+    """'globally' is NOT a keyword, should NOT get trailing underscore."""
+    assert _flag_param_name("--globally") == "globally"
+
+
+# --- Integration test: command with --global flag ---
+
+
+def test_command_with_global_flag():
+    """A command with a --global bool flag works end-to-end."""
+    app = strictcli.App(name="testapp", version="1.0.0", help="test app")
+    received = {}
+
+    @app.command("cmd", help="a command")
+    @strictcli.flag("global", type=bool, help="apply globally")
+    def cmd(**kwargs):
+        received.update(kwargs)
+
+    r = app.test(["cmd", "--global"])
+    assert r.exit_code == 0
+    assert received["global_"] is True
+
+
+def test_command_with_global_flag_default():
+    """When --global is not passed, its default (False) uses the global_ key."""
+    app = strictcli.App(name="testapp", version="1.0.0", help="test app")
+    received = {}
+
+    @app.command("cmd", help="a command")
+    @strictcli.flag("global", type=bool, help="apply globally")
+    def cmd(**kwargs):
+        received.update(kwargs)
+
+    r = app.test(["cmd"])
+    assert r.exit_code == 0
+    assert received["global_"] is False
+
+
+# --- Integration test: config with keyword flag ---
+
+
+def test_config_key_uses_suffixed_name(tmp_path):
+    """Config keys for keyword flags use the suffixed name (global_)."""
+    # Write a config file with the suffixed key
+    config_dir = tmp_path / "kwtest"
+    config_dir.mkdir(parents=True, exist_ok=True)
+    config_file = config_dir / "config.json"
+    config_file.write_text(json.dumps({"global_": True}) + "\n")
+
+    # Set XDG_CONFIG_HOME BEFORE creating the app, since config is loaded
+    # during App.__init__
+    old_xdg = os.environ.get("XDG_CONFIG_HOME")
+    os.environ["XDG_CONFIG_HOME"] = str(tmp_path)
+    try:
+        app = strictcli.App(
+            name="kwtest",
+            version="1.0.0",
+            help="test app",
+            config=True,
+        )
+        received = {}
+
+        @app.command("cmd", help="a command")
+        @strictcli.flag("global", type=bool, help="apply globally")
+        def cmd(**kwargs):
+            received.update(kwargs)
+
+        r = app.test(["cmd"])
+        assert r.exit_code == 0
+        assert received["global_"] is True
+    finally:
+        if old_xdg is None:
+            del os.environ["XDG_CONFIG_HOME"]
+        else:
+            os.environ["XDG_CONFIG_HOME"] = old_xdg

strictcli-0.20.1/todo/keyword-collision-in-flag-param-name.md

@@ -0,0 +1,18 @@
+# Keyword collision in `_flag_param_name`
+
+## Problem
+
+`_flag_param_name` does not handle Python keyword collisions. When a CLI flag name matches a Python keyword (e.g., `--global`), the generated parameter name is an invalid Python identifier.
+
+rlsbl currently monkey-patches `_flag_param_name` to work around this by appending `_` for keywords like `global`.
+
+## Fix
+
+The fix is 2 lines in `_flag_param_name`:
+
+1. Add `import keyword` at the top of the module
+2. Add `if keyword.iskeyword(name): return name + "_"` to `_flag_param_name`
+
+## Downstream impact
+
+Once this ships in a strictcli release, the rlsbl monkey-patch can be removed.

{strictcli-0.20.0 → strictcli-0.20.1}/uv.lock

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

strictcli-0.20.0/.rlsbl/changes/.validated

@@ -1 +0,0 @@
-019cae063cca2c44324d0c76260c72e72334b7c8

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

@@ -1,9 +0,0 @@
-## 0.15.0
-
-### 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.

strictcli-0.20.0/.rlsbl/changes/0.18.0.md

@@ -1,5 +0,0 @@
-## 0.18.0
-
-### Features
-
-- **New.** Public API for running checks programmatically: `App.run_checks()`, `format_check_results()`, `format_check_results_json()`, `CheckRunResult`.