boost-skill-cli 1.0.58__tar.gz → 1.0.60__tar.gz

boost_skill_cli-1.0.60/.gitattributes

@@ -0,0 +1,4 @@
+# Generated build outputs — never hand-edit; regenerate from source (see
+# CONTRIBUTING.md "Generated files"). Marked linguist-generated so GitHub
+# collapses their diffs and excludes them from language stats.
+boost_cli/data/registries.json linguist-generated=true

{boost_skill_cli-1.0.58 → boost_skill_cli-1.0.60}/CONTRIBUTING.md

@@ -33,6 +33,20 @@ agent configs.
 - Anything under `boost_cli/core/` is mutation-tested; expect to add unit
   tests that actually kill your mutants.
 
+## Generated files — never hand-edit
+
+Some checked-in files are build outputs. Edit the source, regenerate, and commit
+the result **as the final step before opening the PR** so the artifact never
+drifts from its source (CI fails the build otherwise):
+
+| Generated file | Source of truth | Regenerate with |
+|---|---|---|
+| `boost_cli/data/registries.json` | `SKILLS`/`RULES`/`WORKFLOWS` tuples in `scripts/build_registries.py` | `python3 scripts/build_registries.py` |
+
+CI runs `python3 scripts/build_registries.py --check` and fails on drift; the
+same guard lives in `tests/unit/test_registries_fresh.py`. Regenerating last also
+keeps line-adjacent JSON conflicts between parallel branches rare.
+
 ## Pull requests
 
 Branch from `main`, keep PRs focused, and write the PR description in the

{boost_skill_cli-1.0.58 → boost_skill_cli-1.0.60}/PKG-INFO

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

{boost_skill_cli-1.0.58 → boost_skill_cli-1.0.60}/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.58'
-__version_tuple__ = version_tuple = (1, 0, 58)
+__version__ = version = '1.0.60'
+__version_tuple__ = version_tuple = (1, 0, 60)
 
 __commit_id__ = commit_id = None

{boost_skill_cli-1.0.58 → boost_skill_cli-1.0.60}/boost_cli/core/catalog.py

@@ -210,6 +210,28 @@ def resolve_one(name: str) -> dict:
     return matches[0]
 
 
+def _meta_text(meta) -> str:
+    """Lowercased, space-joined flatten of a frontmatter dict for substring
+    search. Cheaper than ``json.dumps(meta).lower()`` — which built and threw
+    away a JSON string on every entry of every query just to substring-match —
+    while matching the same words (tags, keys, scalar values)."""
+    parts: List[str] = []
+
+    def walk(v) -> None:
+        if isinstance(v, dict):
+            for k, val in v.items():
+                parts.append(str(k))
+                walk(val)
+        elif isinstance(v, (list, tuple)):
+            for val in v:
+                walk(val)
+        elif v is not None:
+            parts.append(str(v))
+
+    walk(meta or {})
+    return " ".join(parts).lower()
+
+
 def search(query: str, entries: Optional[List[dict]] = None):
     """Rank entries against a query -> [(entry, score)] best-first."""
     entries = all_entries() if entries is None else entries
@@ -219,7 +241,7 @@ def search(query: str, entries: Optional[List[dict]] = None):
     for e in entries:
         name = e["name"].lower()
         desc = (e["description"] or "").lower()
-        blob = " ".join([name, desc, json.dumps(e.get("meta", {})).lower()])
+        blob = " ".join([name, desc, _meta_text(e.get("meta", {}))])
         score = 0
         if q == name:
             score += 100

{boost_skill_cli-1.0.58 → boost_skill_cli-1.0.60}/boost_skill_cli.egg-info/PKG-INFO

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

{boost_skill_cli-1.0.58 → boost_skill_cli-1.0.60}/boost_skill_cli.egg-info/SOURCES.txt

@@ -1,3 +1,4 @@
+.gitattributes
 .gitignore
 CLAUDE.md
 CONTRIBUTING.md
@@ -106,6 +107,7 @@ tests/unit/test_output.py
 tests/unit/test_paths.py
 tests/unit/test_policy.py
 tests/unit/test_rag.py
+tests/unit/test_registries_fresh.py
 tests/unit/test_registry.py
 tests/unit/test_spin.py
 tests/unit/test_store.py

{boost_skill_cli-1.0.58 → boost_skill_cli-1.0.60}/scripts/build_registries.py

@@ -6,13 +6,18 @@ entry is deduped by owner/repo, classified by type + category, and tagged with
 `list_only` when the repo is an awesome-list that mostly links out (so its
 scannable item count is far below its advertised total).
 
-Run:  python3 scripts/build_registries.py
+Run:    python3 scripts/build_registries.py            # regenerate the JSON
+Verify: python3 scripts/build_registries.py --check    # fail if it has drifted
 """
 from __future__ import annotations
 
+import argparse
 import json
+import sys
 from pathlib import Path
 
+DEST = Path(__file__).resolve().parent.parent / "boost_cli" / "data" / "registries.json"
+
 # Repos that are index/awesome READMEs — they *point at* many items but ship
 # few or no scannable item files themselves. Kept for discovery, flagged so
 # count math stays honest.
@@ -163,7 +168,12 @@ def rows(pairs, typ):
         }
 
 
-def main() -> int:
+def build_payload() -> dict:
+    """Assemble the registries payload from the SKILLS/RULES/WORKFLOWS tuples.
+
+    Pure function of the source tuples — no I/O — so both the writer and the
+    ``--check`` verifier render byte-for-byte identical output.
+    """
     by_name: dict = {}
     for entry in list(rows(SKILLS, "skill")) + list(rows(RULES, "rule")) + \
             list(rows(WORKFLOWS, "workflow")):
@@ -181,7 +191,7 @@ def main() -> int:
 
     entries = sorted(by_name.values(), key=lambda e: (e["type"], e["name"].lower()))
     scannable = [e for e in entries if not e["list_only"]]
-    payload = {
+    return {
         "generated_note": "Curated skill/rule/workflow registries for boost. "
                           "est_items are research estimates; verify by tapping. "
                           "list_only repos are index/awesome lists that link out.",
@@ -190,10 +200,16 @@ def main() -> int:
         "est_items_scannable": sum(e["est_items"] for e in scannable),
         "registries": entries,
     }
-    dest = Path(__file__).resolve().parent.parent / "boost_cli" / "data" / "registries.json"
-    dest.parent.mkdir(parents=True, exist_ok=True)
-    dest.write_text(json.dumps(payload, indent=2) + "\n")
-    print("wrote %s" % dest)
+
+
+def render(payload: dict) -> str:
+    """Serialize a payload to the exact on-disk representation."""
+    return json.dumps(payload, indent=2) + "\n"
+
+
+def _summary(payload: dict) -> None:
+    entries = payload["registries"]
+    scannable = [e for e in entries if not e["list_only"]]
     print("registries: %d  (scannable %d, list-only %d)"
           % (len(entries), len(scannable), len(entries) - len(scannable)))
     print("est items total: %d   scannable: %d"
@@ -205,6 +221,37 @@ def main() -> int:
         by_type[e["type"]][1] += e["est_items"]
     for typ, (n, est) in sorted(by_type.items()):
         print("  %-9s %3d repos  ~%d items" % (typ, n, est))
+
+
+def main(argv: list[str] | None = None) -> int:
+    parser = argparse.ArgumentParser(description=__doc__)
+    parser.add_argument(
+        "--check", action="store_true",
+        help="verify the committed JSON matches a fresh build; exit 1 on drift "
+             "(does not write). Used in CI to catch un-regenerated edits.",
+    )
+    args = parser.parse_args(argv)
+
+    payload = build_payload()
+    fresh = render(payload)
+
+    if args.check:
+        current = DEST.read_text() if DEST.exists() else ""
+        if current != fresh:
+            print(
+                "ERROR: %s is out of date — regenerate it with\n"
+                "    python3 scripts/build_registries.py\n"
+                "and commit the result (see CONTRIBUTING.md)." % DEST,
+                file=sys.stderr,
+            )
+            return 1
+        print("%s is up to date." % DEST)
+        return 0
+
+    DEST.parent.mkdir(parents=True, exist_ok=True)
+    DEST.write_text(fresh)
+    print("wrote %s" % DEST)
+    _summary(payload)
     return 0
 
 

{boost_skill_cli-1.0.58 → boost_skill_cli-1.0.60}/tests/unit/test_catalog.py

@@ -443,6 +443,32 @@ class TestSearchRanking:
         res = catalog.search("security", entries=[e])
         assert [(m["name"], s) for m, s in res] == [("x", 2)]
 
+    def test_meta_key_still_matches(self):
+        # keys are part of the search text, just as json.dumps(meta) included
+        # them — a query for a frontmatter key name still scores.
+        e = _entry("x", "t", meta={"framework": "react"})
+        assert [(m["name"], s) for m, s in catalog.search("framework", entries=[e])] \
+            == [("x", 2)]
+
+    def test_meta_nested_value_matches(self):
+        e = _entry("x", "t", meta={"requires": [{"skill": "planning"}]})
+        assert [(m["name"], s) for m, s in catalog.search("planning", entries=[e])] \
+            == [("x", 2)]
+
+
+class TestMetaText:
+    def test_flattens_keys_and_values_lowercased(self):
+        assert catalog._meta_text({"Tags": ["AI", "ML"]}) == "tags ai ml"
+
+    def test_nested_dicts_and_lists(self):
+        out = catalog._meta_text({"a": {"b": [1, "Two"]}})
+        assert out == "a b 1 two"
+
+    def test_skips_none_and_empty(self):
+        assert catalog._meta_text({}) == ""
+        assert catalog._meta_text(None) == ""
+        assert catalog._meta_text({"k": None}) == "k"
+
     def test_description_only_match(self):
         e = _entry("styler", "t", desc="handles jira tickets")
         res = catalog.search("jira", entries=[e])

boost_skill_cli-1.0.60/tests/unit/test_registries_fresh.py

@@ -0,0 +1,49 @@
+"""Committed registries.json stays in lockstep with its generator.
+
+``boost_cli/data/registries.json`` is a generated artifact (source of truth is
+the SKILLS/RULES/WORKFLOWS tuples in ``scripts/build_registries.py``). If a repo
+row is edited without regenerating, the committed JSON drifts. This test — the
+in-suite twin of the ``build_registries.py --check`` CI step — fails on drift so
+a stale artifact can't merge.
+
+Skips when the repo-root files aren't reachable (e.g. the mutation sandbox,
+which only copies ``boost_cli/``).
+"""
+from __future__ import annotations
+
+import importlib.util
+from pathlib import Path
+
+import pytest
+
+_ROOT = Path(__file__).resolve().parents[2]
+_SCRIPT = _ROOT / "scripts" / "build_registries.py"
+_JSON = _ROOT / "boost_cli" / "data" / "registries.json"
+
+
+def _load_builder():
+    spec = importlib.util.spec_from_file_location("build_registries", _SCRIPT)
+    assert spec and spec.loader
+    module = importlib.util.module_from_spec(spec)
+    spec.loader.exec_module(module)
+    return module
+
+
+@pytest.mark.skipif(not (_SCRIPT.exists() and _JSON.exists()),
+                    reason="repo-root files not reachable (e.g. mutation sandbox)")
+def test_registries_json_is_regenerated():
+    builder = _load_builder()
+    fresh = builder.render(builder.build_payload())
+    committed = _JSON.read_text()
+    assert committed == fresh, (
+        "boost_cli/data/registries.json is out of date — regenerate with\n"
+        "    python3 scripts/build_registries.py\n"
+        "and commit the result (see CONTRIBUTING.md)."
+    )
+
+
+@pytest.mark.skipif(not _SCRIPT.exists(),
+                    reason="repo-root files not reachable (e.g. mutation sandbox)")
+def test_check_flag_passes_on_fresh_tree():
+    builder = _load_builder()
+    assert builder.main(["--check"]) == 0