boost-skill-cli 1.0.78__tar.gz → 1.0.79__tar.gz

{boost_skill_cli-1.0.78 → boost_skill_cli-1.0.79}/PKG-INFO

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

{boost_skill_cli-1.0.78 → boost_skill_cli-1.0.79}/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.78'
-__version_tuple__ = version_tuple = (1, 0, 78)
+__version__ = version = '1.0.79'
+__version_tuple__ = version_tuple = (1, 0, 79)
 
 __commit_id__ = commit_id = None

{boost_skill_cli-1.0.78 → boost_skill_cli-1.0.79}/boost_cli/core/catalog.py

@@ -67,10 +67,12 @@ def _description(meta: dict, body: str) -> str:
 def _make_entry(root: Path, defining_file: Path, kind: str, default_name: str,
                 tap_name: str, curated: bool, meta: dict, body: str) -> dict:
     name = str(meta.get("name") or "").strip() or default_name
+    slug = util.slugify(name) if " " in name else name
+    description = _description(meta, body)
     item_dir = defining_file.parent
     return {
-        "name": util.slugify(name) if " " in name else name,
-        "description": _description(meta, body),
+        "name": slug,
+        "description": description,
         "version": str(meta.get("version") or "0.0.0"),
         "tap": tap_name,
         "curated": curated,
@@ -78,6 +80,9 @@ def _make_entry(root: Path, defining_file: Path, kind: str, default_name: str,
         "rel_dir": str(item_dir.relative_to(root)) if item_dir != root else ".",
         "skill_md": str(defining_file.relative_to(root)),
         "meta": meta,
+        # Lowercased substring-search blob, flattened once at index time so
+        # search() never re-walks the frontmatter per query (see _search_blob).
+        "search_blob": _search_blob(slug, description, meta),
     }
 
 
@@ -232,6 +237,13 @@ def _meta_text(meta) -> str:
     return " ".join(parts).lower()
 
 
+def _search_blob(name: str, description: str, meta) -> str:
+    """The full lowercased substring-search text for an entry: its name,
+    description and flattened frontmatter. Built once at index time and cached
+    on the entry as ``search_blob`` so search() never rebuilds it per query."""
+    return " ".join([name.lower(), (description or "").lower(), _meta_text(meta)])
+
+
 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
@@ -241,7 +253,12 @@ 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, _meta_text(e.get("meta", {}))])
+        # Precomputed at index time; fall back for older caches / entries
+        # constructed without a blob (e.g. tests passing raw dicts).
+        blob = e.get("search_blob")
+        if blob is None:
+            blob = _search_blob(e["name"], e.get("description", ""),
+                                e.get("meta", {}))
         score = 0
         if q == name:
             score += 100

{boost_skill_cli-1.0.78 → boost_skill_cli-1.0.79}/boost_skill_cli.egg-info/PKG-INFO

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

{boost_skill_cli-1.0.78 → boost_skill_cli-1.0.79}/docs/roadmap/items/stop-re-serializing-entry-meta-on-every-search.md

@@ -2,15 +2,15 @@
 id: stop-re-serializing-entry-meta-on-every-search
 board: code
 section: internals
-status: planned
+status: shipped
 category: Performance
 complexity: S
 impact: Med
 wow: 2
 note: 
 order: 11
-owner:
-pr:
+owner: loop/precompute-search-blob
+pr: 93
 title: Stop re-serializing entry meta on every search
 ---
 <code>catalog.search</code> computes <code>json.dumps(meta).lower()</code> for <b>every entry on every query</b> just to substring-match (<code>catalog.py:222</code>). Precompute a lowercased search blob at index time, or match structured fields directly.

{boost_skill_cli-1.0.78 → boost_skill_cli-1.0.79}/docs/roadmap.html

@@ -498,7 +498,7 @@
       </article>
 
       <article class="cap rcard">
-        <div class="head"><span class="pill planned">Planned</span><span class="cat">Performance</span></div>
+        <div class="head"><span class="pill shipped">Shipped</span><span class="cat">Performance</span></div>
         <h3>Stop re-serializing entry meta on every search</h3>
         <p><code>catalog.search</code> computes <code>json.dumps(meta).lower()</code> for <b>every entry on every query</b> just to substring-match (<code>catalog.py:222</code>). Precompute a lowercased search blob at index time, or match structured fields directly.</p>
         <div class="meta">

{boost_skill_cli-1.0.78 → boost_skill_cli-1.0.79}/tests/unit/test_catalog.py

@@ -477,3 +477,37 @@ class TestMetaText:
     def test_curated_entry_with_no_match_is_excluded(self):
         res = catalog.search("xyzzy", entries=[_entry("unrelated", "t", curated=True)])
         assert res == []
+
+
+class TestSearchBlobPrecompute:
+    def test_search_blob_flattens_name_desc_meta_lowercased(self):
+        blob = catalog._search_blob(
+            "Foo-Bar", "Desc Here", {"tags": ["Alpha"], "x": "On"})
+        assert blob == "foo-bar desc here tags alpha x on"
+
+    def test_search_blob_tolerates_empty_description(self):
+        assert catalog._search_blob("name", "", {}) == "name  "
+        assert catalog._search_blob("name", None, {}) == "name  "
+
+    def test_make_entry_precomputes_blob(self, tmp_path):
+        f = tmp_path / "SKILL.md"
+        f.write_text("body")
+        e = catalog._make_entry(
+            tmp_path, f, "skill", "fallback-name", "tap", False,
+            {"tags": ["Kubernetes"]}, "Handles clusters")
+        assert e["search_blob"] == "fallback-name handles clusters tags kubernetes"
+
+    def test_search_reads_precomputed_blob_not_live_meta(self):
+        # Token lives only in search_blob, not in meta -> a match proves search
+        # consumed the precomputed blob rather than re-walking meta.
+        e = _entry("x", "t", meta={})
+        e["search_blob"] = "x  zzsentinel"
+        res = catalog.search("zzsentinel", entries=[e])
+        assert [(m["name"], s) for m, s in res] == [("x", 2)]
+
+    def test_search_falls_back_when_blob_absent(self):
+        # Older caches / raw entries have no blob -> recompute from meta.
+        e = _entry("x", "t", meta={"tags": ["fallbackword"]})
+        assert "search_blob" not in e
+        res = catalog.search("fallbackword", entries=[e])
+        assert [(m["name"], s) for m, s in res] == [("x", 2)]