boost-skill-cli 1.0.261__tar.gz → 1.0.263__tar.gz

{boost_skill_cli-1.0.261 → boost_skill_cli-1.0.263}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: boost-skill-cli
-Version: 1.0.261
+Version: 1.0.263
 Summary: boost — Homebrew for AI coding skills
 Author: Jonathan Reyes
 License: GNU GENERAL PUBLIC LICENSE

{boost_skill_cli-1.0.261 → boost_skill_cli-1.0.263}/boost_cli/_version.py

@@ -18,7 +18,7 @@ version_tuple: tuple[int | str, ...]
 commit_id: str | None
 __commit_id__: str | None
 
-__version__ = version = '1.0.261'
-__version_tuple__ = version_tuple = (1, 0, 261)
+__version__ = version = '1.0.263'
+__version_tuple__ = version_tuple = (1, 0, 263)
 
 __commit_id__ = commit_id = None

{boost_skill_cli-1.0.261 → boost_skill_cli-1.0.263}/boost_skill_cli.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: boost-skill-cli
-Version: 1.0.261
+Version: 1.0.263
 Summary: boost — Homebrew for AI coding skills
 Author: Jonathan Reyes
 License: GNU GENERAL PUBLIC LICENSE

{boost_skill_cli-1.0.261 → boost_skill_cli-1.0.263}/tests/unit/test_toolchain_lock.py

@@ -21,7 +21,12 @@ REQS = ROOT / "requirements"
 GROUPS = ("lint-tools", "test-tools", "mutation-tools", "coverage-tools",
           "release-tools")
 
+# The extras group is not optional decoration: without it, `coverage[toml]==7.15.2`
+# matches `name=coverage`, then `spec` fails because the next character is `[`,
+# and the pin invariant below reports a perfectly pinned line as unpinned.
+# uv and pip-tools emit `name[extra]==version` routinely.
 _REQ = re.compile(r"^(?P<name>[A-Za-z0-9][A-Za-z0-9._-]*)"
+                  r"(?P<extras>\[[^\]]+\])?"
                   r"(?P<spec>==[^\s;\\]+)?(?P<rest>.*)$")
 
 
@@ -91,7 +96,8 @@ class TestLockInvariants:
                 m = _REQ.match(line.strip())
                 assert m, "unparseable requirement in %s: %r" % (stem, line)
                 assert m.group("spec"), \
-                    "%s is not pinned with == in %s" % (m.group("name"), stem)
+                    "%s is not pinned with == in %s: %r" % (
+                        m.group("name"), stem, line)
 
     def test_every_pin_carries_hashes(self):
         # This is the whole point: pip implies --require-hashes when any hash is
@@ -124,6 +130,60 @@ class TestLockInvariants:
             assert str(ROOT) not in text, stem
 
 
+class TestRequirementPattern:
+    """The pattern itself, against literal lines.
+
+    Everything else in this file runs `_REQ` over whatever `requirements/*.txt`
+    happens to contain today, so a form the lock does not currently use is a
+    form the pattern is never tested against. That is exactly how the extras
+    case got here: nothing in the lock carries an extra, so `coverage[toml]==…`
+    reading as *unpinned* stayed invisible until a Dependabot PR regenerated
+    the lock and the gate rejected a valid pin with a message naming the wrong
+    problem.
+    """
+
+    @staticmethod
+    def _parsed(line):
+        m = _REQ.match(line)
+        assert m, "unparseable: %r" % line
+        return m
+
+    def test_a_plain_pin(self):
+        m = self._parsed("hypothesis==6.140.3")
+        assert m.group("name") == "hypothesis"
+        assert m.group("extras") is None
+        assert m.group("spec") == "==6.140.3"
+
+    def test_a_pin_carrying_an_extra(self):
+        # THE BUG: this used to come back with spec=None.
+        m = self._parsed("coverage[toml]==7.15.2")
+        assert m.group("name") == "coverage"
+        assert m.group("extras") == "[toml]"
+        assert m.group("spec") == "==7.15.2"
+
+    def test_a_pin_carrying_several_extras(self):
+        m = self._parsed("ruff[all,dev]==0.16.0")
+        assert m.group("extras") == "[all,dev]"
+        assert m.group("spec") == "==0.16.0"
+
+    def test_a_pin_with_an_environment_marker(self):
+        m = self._parsed("coverage[toml]==7.10.7 ; python_full_version < '3.10'")
+        assert m.group("name") == "coverage"
+        assert m.group("spec") == "==7.10.7", "the marker must not eat the pin"
+
+    def test_names_with_dots_dashes_and_digits(self):
+        for line, name in (("zope.interface==7.0", "zope.interface"),
+                           ("ruff-lsp==0.1", "ruff-lsp"),
+                           ("s3transfer==0.10", "s3transfer")):
+            assert self._parsed(line).group("name") == name
+
+    def test_an_unpinned_requirement_still_reports_unpinned(self):
+        # The fix must not make everything look pinned.
+        for line in ("hypothesis", "hypothesis>=6", "coverage[toml]>=7.15",
+                     "coverage[toml]~=7.15"):
+            assert self._parsed(line).group("spec") is None, line
+
+
 class TestEvalExtraStaysOut:
     def test_langchain_stack_is_not_locked(self):
         # The [eval] extra deliberately pins the old langchain 0.3 stack (ragas