strictcli 0.16.0__tar.gz → 0.17.0__tar.gz

strictcli-0.17.0/.rlsbl/changes/.validated

@@ -0,0 +1 @@
+912fca00b39e373782d5c0b88d65d2067f89a1b1

strictcli-0.17.0/.rlsbl/changes/0.16.0.md

@@ -0,0 +1,18 @@
+## 0.16.0
+
+### Breaking
+
+- **Config show.** `config show` now requires explicit `--plain` or `--json` flag instead of defaulting to plain text.
+
+### Features
+
+- **New flag field.** `unique` field on repeatable flags enforces no duplicate values at CLI parse time and in config arrays.
+- **New flag field.** `env_separator` on repeatable flags controls how environment variable values are split into arrays.
+- **Config arrays.** Config values for repeatable flags are now coerced from strings to the declared type.
+- **Config set.** Repeatable flags can now be managed with `config set`, including `--clear` to reset and `--default` to restore default values.
+
+### Fixes
+
+- **Fix.** Float NaN/Inf env var error messages now include the environment variable suffix.
+- **Fix.** Config error messages now use consistent type names and array formatting.
+- **Fix.** Flag collection no longer includes config group pseudo-flags, preventing incorrect behavior in commands that enumerate all flags.

strictcli-0.17.0/.rlsbl/changes/0.16.1.jsonl

@@ -0,0 +1,3 @@
+{"commits":["cdc010316ce356628e2592b493ccc368dd782b48"],"user_facing":true,"description":"**Fix.** `config set` mutex error messages now include the `config set:` prefix for consistency.","type":"fix"}
+{"commits":["98bfea68b1b971b42a31ecb1d08ecc3b55df86f2","1d9ee273ce05153c7c9e23b8297c6fccdb342c21"],"user_facing":false}
+{"commits":["67b8d456d3e491b3d398ee7b0b7a4f3ba116cab1"],"user_facing":true,"description":"**Fix.** Config set help text now mentions backslash escaping for values containing commas in repeatable flags.","type":"fix"}

strictcli-0.17.0/.rlsbl/changes/0.16.1.md

@@ -0,0 +1,6 @@
+## 0.16.1
+
+### Fixes
+
+- **Fix.** `config set` mutex error messages now include the `config set:` prefix for consistency.
+- **Fix.** Config set help text now mentions backslash escaping for values containing commas in repeatable flags.

strictcli-0.17.0/.rlsbl/changes/0.17.0.jsonl

@@ -0,0 +1,3 @@
+{"commits":["5660d554d8a5dbb47bb6872d8513ba497fb04e7f"],"user_facing":false}
+{"commits":["ba0ca0dd03644fa63f6ca6525ce81d814100a7a1"],"user_facing":true,"description":"**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.","type":"breaking"}
+{"commits":["47de2e49ebd7248a5b16a54860ec9cdf1e5e98e9"],"user_facing":true,"description":"**New.** Help text now displays non-empty repeatable flag defaults as `[default: a, b, c]`.","type":"feature"}

strictcli-0.17.0/.rlsbl/releases/v0.16.1.toml

@@ -0,0 +1,8 @@
+# Version bump type: patch, minor, or major
+bump = "patch"
+# Short description of this release (required)
+description = "Error message fixes and help text improvements"
+# Optional context explaining why these changes were made
+context = ""
+include = ["pypi", "npm"]
+exclude = []

strictcli-0.17.0/.rlsbl/releases/v0.17.0.toml

@@ -0,0 +1,8 @@
+# Version bump type: patch, minor, or major
+bump = "minor"
+# Short description of this release (required)
+description = "Repeatable flag default validation and help text display"
+# Optional context explaining why these changes were made
+context = ""
+include = ["pypi", "npm"]
+exclude = []

{strictcli-0.16.0 → strictcli-0.17.0}/CHANGELOG.md

@@ -2,16 +2,26 @@
 
 # Changelog
 
-## 0.16.0
+## 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.
 
-unique and env_separator flag fields, config array coercion, config set repeatable support
+### Features
+
+- **New.** Help text now displays non-empty repeatable flag defaults as `[default: a, b, c]`.
 
-<details>
-<summary>Context</summary>
+## 0.16.1
 
-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.
+### Fixes
 
-</details>
+- **Fix.** `config set` mutex error messages now include the `config set:` prefix for consistency.
+- **Fix.** Config set help text now mentions backslash escaping for values containing commas in repeatable flags.
+
+## 0.16.0
 
 ### Breaking
 

{strictcli-0.16.0 → strictcli-0.17.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: strictcli
-Version: 0.16.0
+Version: 0.17.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.16.0 → strictcli-0.17.0}/package-lock.json

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

{strictcli-0.16.0 → strictcli-0.17.0}/package.json

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

{strictcli-0.16.0 → strictcli-0.17.0}/pyproject.toml

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

{strictcli-0.16.0 → strictcli-0.17.0}/strictcli/__init__.py

@@ -2,7 +2,7 @@
 
 from __future__ import annotations
 
-__version__ = "0.16.0"
+__version__ = "0.17.0"
 
 __all__ = [
     "App", "Flag", "Arg", "Tag", "MutexGroup", "CoRequired", "Requires",
@@ -588,6 +588,38 @@ class Flag:
                     raise ValueError(
                         f'Flag "{self.name}": choice {c!r} is not of type {self.type.__name__}'
                     )
+        # Validate repeatable flag defaults
+        if self.repeatable and not isinstance(self.default, _MissingSentinel):
+            if self.default is not None:
+                if not isinstance(self.default, list):
+                    raise ValueError(
+                        f'Flag "{self.name}": repeatable flag default must be a list'
+                    )
+                if len(self.default) == 0:
+                    raise ValueError(
+                        f'Flag "{self.name}": explicit empty default is redundant '
+                        f"for repeatable flags, omit the default"
+                    )
+                # Validate element types
+                type_name = {str: "str", int: "int", float: "float"}[self.type]
+                for i, elem in enumerate(self.default):
+                    if self.type is str:
+                        if not isinstance(elem, str):
+                            raise ValueError(
+                                f'Flag "{self.name}": default element {i} is not of type {type_name}'
+                            )
+                    elif self.type is int:
+                        if not isinstance(elem, int) or isinstance(elem, bool):
+                            raise ValueError(
+                                f'Flag "{self.name}": default element {i} is not of type {type_name}'
+                            )
+                    elif self.type is float:
+                        if not isinstance(elem, (int, float)) or isinstance(elem, bool):
+                            raise ValueError(
+                                f'Flag "{self.name}": default element {i} is not of type {type_name}'
+                            )
+                        if isinstance(elem, int):
+                            self.default[i] = float(elem)
         # Validate default type for int flags
         if self.type is int and not isinstance(self.default, _MissingSentinel) and self.default is not None:
             if not self.repeatable and not isinstance(self.default, int):
@@ -1163,11 +1195,7 @@ class App:
 
         # config show
         def _config_show_handler(**_kw) -> int:
-            use_plain = _kw.get("plain", False)
             use_json = _kw.get("json", False)
-            if not use_plain and not use_json:
-                print("config show: specify --plain or --json", file=sys.stderr)
-                return 1
             config_data = _load_config(
                 app_ref.name,
                 config_path_override=app_ref.config_path,
@@ -1251,7 +1279,7 @@ class App:
             # Validate: exactly one of (value, --clear, --default)
             has_value = value is not None
             if use_clear and use_default:
-                print("--clear and --default are mutually exclusive",
+                print("config set: --clear and --default are mutually exclusive",
                       file=sys.stderr)
                 return 1
             if has_value and use_clear:
@@ -1369,7 +1397,7 @@ class App:
             args=[
                 Arg(name="key", help="Config key to set"),
                 Arg(name="value",
-                    help="Value to set (comma-separated for repeatable flags)",
+                    help="Value to set (comma-separated for repeatable flags, use backslash to escape commas)",
                     required=False),
             ],
             flags=[
@@ -2908,8 +2936,10 @@ def _build_flag_meta(f: Flag) -> str:
     if f.type is bool:
         meta_parts.append(f"default: {'true' if f.default else 'false'}")
     elif f.repeatable:
-        # Repeatable flags are never required; no default shown
-        pass
+        # Repeatable flags are never required; show default only if non-empty
+        if f.default:
+            joined = ", ".join(_format_value_for_error(elem) for elem in f.default)
+            meta_parts.append(f"default: {joined}")
     elif f.default is not None:
         meta_parts.append(f"default: {f.default}")
     else:

{strictcli-0.16.0 → strictcli-0.17.0}/tests/test_config.py

@@ -1114,14 +1114,13 @@ def _make_config_set_app(config_path=None, config_format="json"):
     )
 
     @app.command("run", help="run something")
-    @strictcli.flag("tags", type=str, help="tags", repeatable=True, unique=False,
-                    default=[])
+    @strictcli.flag("tags", type=str, help="tags", repeatable=True, unique=False)
     @strictcli.flag("counts", type=int, help="counts", repeatable=True,
-                    unique=False, default=[])
+                    unique=False)
     @strictcli.flag("rates", type=float, help="rates", repeatable=True,
-                    unique=False, default=[])
+                    unique=False)
     @strictcli.flag("ids", type=int, help="unique ids", repeatable=True,
-                    unique=True, default=[])
+                    unique=True)
     @strictcli.flag("name", type=str, help="name", default="default")
     def run(tags, counts, rates, ids, name):
         print(f"tags={tags} counts={counts} rates={rates} ids={ids} name={name}")
@@ -1153,6 +1152,18 @@ def test_config_set_repeatable_int(tmp_path, monkeypatch):
     assert data["counts"] == [1, 2, 3]
 
 
+def test_config_set_repeatable_float(tmp_path, monkeypatch):
+    """Config set writes a float array for a repeatable flag."""
+    monkeypatch.setenv("XDG_CONFIG_HOME", str(tmp_path))
+    app = _make_config_set_app()
+    r = app.test(["config", "set", "rates", "1.5,2.5,3.0"])
+    assert r.exit_code == 0
+
+    config_file = tmp_path / "repapp" / "config.json"
+    data = json.loads(config_file.read_text())
+    assert data["rates"] == [1.5, 2.5, 3.0]
+
+
 def test_config_set_escaped_comma(tmp_path, monkeypatch):
     r"""Config set handles escaped commas: a\,b,c -> ["a,b", "c"]."""
     monkeypatch.setenv("XDG_CONFIG_HOME", str(tmp_path))
@@ -1297,4 +1308,4 @@ def test_config_set_clear_and_default_error(tmp_path, monkeypatch):
     app = _make_config_set_app()
     r = app.test(["config", "set", "tags", "--clear", "--default"])
     assert r.exit_code == 1
-    assert "--clear and --default are mutually exclusive" in r.stderr
+    assert "config set: --clear and --default are mutually exclusive" in r.stderr

{strictcli-0.16.0 → strictcli-0.17.0}/tests/test_repeatable.py

@@ -534,3 +534,104 @@ def test_env_separator_float_nan_error(monkeypatch):
     r = app.test(["cmd"])
     assert r.exit_code == 1
     assert "--rate: NaN is not allowed (from env var 'RATES')" in r.stderr
+
+
+# --- Repeatable flag default validation ---
+
+
+def test_repeatable_default_must_be_list():
+    """Repeatable flag with non-list default raises ValueError."""
+    with pytest.raises(ValueError, match='Flag "record": repeatable flag default must be a list'):
+        _make_app_with_repeatable(type=str, default="not a list")
+
+
+def test_repeatable_default_empty_list_error():
+    """Repeatable flag with explicit default=[] raises ValueError."""
+    with pytest.raises(
+        ValueError,
+        match='Flag "record": explicit empty default is redundant for repeatable flags, omit the default',
+    ):
+        _make_app_with_repeatable(type=str, default=[])
+
+
+def test_repeatable_default_wrong_element_type_str():
+    """Repeatable str flag with default=[1] raises ValueError at element 0."""
+    with pytest.raises(ValueError, match='Flag "record": default element 0 is not of type str'):
+        _make_app_with_repeatable(type=str, default=[1])
+
+
+def test_repeatable_default_wrong_element_type_int():
+    """Repeatable int flag with default=["x"] raises ValueError."""
+    with pytest.raises(ValueError, match='Flag "record": default element 0 is not of type int'):
+        _make_app_with_repeatable(type=int, default=["x"])
+
+
+def test_repeatable_default_int_coerced_to_float():
+    """Repeatable float flag with default=[1, 2] coerces to [1.0, 2.0]."""
+    app = strictcli.App(name="test", version="1.0.0", help="test app")
+
+    @app.command("cmd", help="a command")
+    @strictcli.flag("val", type=float, help="a value", repeatable=True, unique=False,
+                    default=[1, 2])
+    def cmd(val):
+        print(f"val={val!r}")
+
+    # The flag object should have coerced defaults
+    flag_obj = app._commands["cmd"].flags[0]
+    assert flag_obj.default == [1.0, 2.0]
+    assert all(isinstance(x, float) for x in flag_obj.default)
+
+
+def test_repeatable_default_valid():
+    """Repeatable flag with valid non-empty default succeeds."""
+    app = _make_app_with_repeatable(type=str, default=["a", "b"])
+    r = app.test(["cmd"])
+    assert r.exit_code == 0
+    assert "record=['a', 'b']" in r.stdout
+
+
+def test_repeatable_default_applied():
+    """Regression guard: repeatable flag default is applied when no CLI/env value."""
+    app = strictcli.App(name="test", version="1.0.0", help="test app")
+
+    @app.command("cmd", help="a command")
+    @strictcli.flag("tag", help="a tag", repeatable=True, unique=False,
+                    default=["a", "b"])
+    def cmd(tag):
+        print(f"tag={tag!r}")
+
+    r = app.test(["cmd"])
+    assert r.exit_code == 0
+    assert "tag=['a', 'b']" in r.stdout
+
+
+def test_repeatable_default_shown_in_help():
+    """Repeatable flag with non-empty default shows it in help text."""
+    app = _make_app_with_repeatable(type=str, default=["x", "y"])
+    r = app.test(["cmd", "--help"])
+    assert r.exit_code == 0
+    assert "[default: x, y]" in r.stdout
+
+
+def test_repeatable_no_default_not_in_help():
+    """Repeatable flag with no explicit default shows no default in help."""
+    app = _make_app_with_repeatable(type=str)
+    r = app.test(["cmd", "--help"])
+    assert r.exit_code == 0
+    assert "[default" not in r.stdout
+
+
+def test_repeatable_default_int_in_help():
+    """Repeatable int flag with default shows formatted ints in help."""
+    app = _make_app_with_repeatable(type=int, default=[1, 2, 3])
+    r = app.test(["cmd", "--help"])
+    assert r.exit_code == 0
+    assert "[default: 1, 2, 3]" in r.stdout
+
+
+def test_repeatable_default_float_in_help():
+    """Repeatable float flag with default shows formatted floats in help."""
+    app = _make_app_with_repeatable(type=float, default=[1.5, 2.5])
+    r = app.test(["cmd", "--help"])
+    assert r.exit_code == 0
+    assert "[default: 1.5, 2.5]" in r.stdout

{strictcli-0.16.0 → strictcli-0.17.0}/uv.lock

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

strictcli-0.16.0/.rlsbl/changes/.validated

@@ -1 +0,0 @@
-6e78d5bbca2712fb8e63e4316ea118fff59b992f