strictcli 0.16.1__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.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.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.1 → strictcli-0.17.0}/CHANGELOG.md

@@ -2,9 +2,19 @@
 
 # Changelog
 
-## 0.16.1
+## 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.
 
-Error message fixes and help text improvements
+### Features
+
+- **New.** Help text now displays non-empty repeatable flag defaults as `[default: a, b, c]`.
+
+## 0.16.1
 
 ### Fixes
 

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

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

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

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

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

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

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

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

@@ -2,7 +2,7 @@
 
 from __future__ import annotations
 
-__version__ = "0.16.1"
+__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):
@@ -2904,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.1 → 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}")

{strictcli-0.16.1 → 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.1 → strictcli-0.17.0}/uv.lock

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

strictcli-0.16.1/.rlsbl/changes/.validated

@@ -1 +0,0 @@
-2a117a0d39aebe26c1f73df78aecb99c81ee2128