boost-skill-cli 1.0.80__tar.gz → 1.0.82__tar.gz

{boost_skill_cli-1.0.80 → boost_skill_cli-1.0.82}/PKG-INFO

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

{boost_skill_cli-1.0.80 → boost_skill_cli-1.0.82}/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.80'
-__version_tuple__ = version_tuple = (1, 0, 80)
+__version__ = version = '1.0.82'
+__version_tuple__ = version_tuple = (1, 0, 82)
 
 __commit_id__ = commit_id = None

{boost_skill_cli-1.0.80 → boost_skill_cli-1.0.82}/boost_cli/core/catalog.py

@@ -18,9 +18,10 @@ that ship in the same GitHub registries:
 from __future__ import annotations
 
 import json
+import os
 import re
 from pathlib import Path
-from typing import List, Optional, Tuple
+from typing import List, Optional, Set, Tuple
 
 from ..errors import BoostError
 from . import frontmatter, gitutil, paths, registry, util
@@ -42,10 +43,6 @@ DOC_STEMS = {"readme", "contributing", "license", "changelog", "security",
              "code_of_conduct", "index", "_index", "authors", "notice"}
 
 
-def _ignored(path: Path) -> bool:
-    return any(part in util.IGNORED for part in path.parts)
-
-
 def _read(path: Path) -> Optional[Tuple[dict, str]]:
     try:
         return frontmatter.parse(path.read_text(encoding="utf-8", errors="replace"))
@@ -107,46 +104,51 @@ def _classify_workflow(path: Path, meta: dict) -> bool:
 def scan_dir(root: Path, tap_name: str = "local", curated: bool = False) -> List[dict]:
     root = Path(root)
     entries: List[dict] = []
-    skill_dirs: List[Path] = []
-    for skill_md in sorted(root.rglob("SKILL.md")):
-        if _ignored(skill_md):
-            continue
-        parsed = _read(skill_md)
-        if parsed is None:
-            continue
-        meta, body = parsed
-        skill_dir = skill_md.parent
-        default = skill_dir.name if skill_dir != root else root.name
-        entries.append(_make_entry(root, skill_md, KIND_SKILL, default,
-                                   tap_name, curated, meta, body))
-        skill_dirs.append(skill_dir)
-
-    for path in sorted(root.rglob("*")):
-        if not path.is_file() or _ignored(path):
-            continue
-        # never re-index a file that belongs to a scanned skill directory
-        if any(path == d or d in path.parents for d in skill_dirs):
-            continue
-        is_rule = _classify_rule(path)
-        is_md = path.suffix.lower() == ".md" and path.name != "SKILL.md"
-        if not is_rule and not is_md:
-            continue
-        parsed = _read(path)
-        if parsed is None:
-            continue
-        meta, body = parsed
-        if is_rule:
-            kind = KIND_RULE
-        elif _classify_workflow(path, meta):
-            kind = KIND_WORKFLOW
-        else:
+    skill_dirs: Set[Path] = set()
+    # One top-down walk instead of two rglob passes: prune ignored dirs in place
+    # so we never descend into .git/__pycache__, and test skill-dir membership
+    # against a set via each dir's own ancestor chain rather than O(files × dirs).
+    # Top-down order guarantees a file's governing SKILL.md dir is seen first, and
+    # the final entries.sort() normalizes output order regardless of walk order.
+    for dirpath, dirnames, filenames in os.walk(root):
+        dirnames[:] = [d for d in dirnames if d not in util.IGNORED]
+        here = Path(dirpath)
+        # A directory holding a SKILL.md is a skill: index it and remember the
+        # dir so its other files are not re-indexed as loose rules/workflows.
+        if "SKILL.md" in filenames:
+            parsed = _read(here / "SKILL.md")
+            if parsed is not None:
+                meta, body = parsed
+                default = here.name if here != root else root.name
+                entries.append(_make_entry(root, here / "SKILL.md", KIND_SKILL,
+                                           default, tap_name, curated,
+                                           meta, body))
+                skill_dirs.add(here)
+        # Files inside any skill directory belong to that skill, never re-indexed.
+        if here in skill_dirs or any(a in skill_dirs for a in here.parents):
             continue
-        if path.name in RULE_FILENAMES:
-            default = path.parent.name if path.parent != root else root.name
-        else:
-            default = path.stem
-        entries.append(_make_entry(root, path, kind, default,
-                                   tap_name, curated, meta, body))
+        for name in filenames:
+            path = here / name
+            is_rule = _classify_rule(path)
+            is_md = path.suffix.lower() == ".md" and path.name != "SKILL.md"
+            if not is_rule and not is_md:
+                continue
+            parsed = _read(path)
+            if parsed is None:
+                continue
+            meta, body = parsed
+            if is_rule:
+                kind = KIND_RULE
+            elif _classify_workflow(path, meta):
+                kind = KIND_WORKFLOW
+            else:
+                continue
+            if path.name in RULE_FILENAMES:
+                default = path.parent.name if path.parent != root else root.name
+            else:
+                default = path.stem
+            entries.append(_make_entry(root, path, kind, default,
+                                       tap_name, curated, meta, body))
 
     entries.sort(key=lambda e: (e["skill_md"], e["name"]))
     return entries

{boost_skill_cli-1.0.80 → boost_skill_cli-1.0.82}/boost_skill_cli.egg-info/PKG-INFO

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

{boost_skill_cli-1.0.80 → boost_skill_cli-1.0.82}/docs/roadmap/items/prune-ignored-dirs-during-scan-dir-walk.md

@@ -2,15 +2,15 @@
 id: prune-ignored-dirs-during-scan-dir-walk
 board: code
 section: internals
-status: planned
+status: shipped
 category: Performance
 complexity: M
 impact: Low-Med
 wow: 2
 note: 
 order: 12
-owner:
-pr:
+owner: loop/prune-scan-dir
+pr: 92
 title: Prune ignored dirs during <code>scan_dir</code> walk
 ---
 Two full <code>rglob</code> passes descend <b>into</b> <code>.git</code>/<code>node_modules</code> and only filter afterward, and the skill-dir membership test is O(files × skill_dirs) (<code>catalog.py:106,119–123</code>). Switch to an <code>os.walk</code> that prunes ignored dirs in place; index skill dirs in a set.

{boost_skill_cli-1.0.80 → boost_skill_cli-1.0.82}/docs/roadmap.html

@@ -528,7 +528,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>Prune ignored dirs during <code>scan_dir</code> walk</h3>
         <p>Two full <code>rglob</code> passes descend <b>into</b> <code>.git</code>/<code>node_modules</code> and only filter afterward, and the skill-dir membership test is O(files × skill_dirs) (<code>catalog.py:106,119–123</code>). Switch to an <code>os.walk</code> that prunes ignored dirs in place; index skill dirs in a set.</p>
         <div class="meta">

{boost_skill_cli-1.0.80 → boost_skill_cli-1.0.82}/tests/unit/test_catalog.py

@@ -239,6 +239,27 @@ class TestScanRulesAndWorkflows:
         assert [e["name"] for e in entries] == ["s"]
         assert entries[0]["kind"] == "skill"
 
+    def test_rule_entry_carries_tap_and_curated(self, tmp_path):
+        root = tmp_path / "tap"
+        (root / "nextjs").mkdir(parents=True)
+        (root / "nextjs" / ".cursorrules").write_text("Next.js expert.\n")
+        (e,) = catalog.scan_dir(root, "mytap", curated=True)
+        assert e["kind"] == "rule"
+        assert e["tap"] == "mytap"
+        assert e["curated"] is True
+
+    def test_ignored_dirs_pruned_for_loose_rules_and_workflows(self, tmp_path):
+        root = tmp_path / "tap"
+        (root / ".git").mkdir(parents=True)
+        (root / "sub" / "__pycache__").mkdir(parents=True)
+        (root / ".git" / "hook.mdc").write_text("---\ndescription: x\n---\n")
+        (root / "sub" / "__pycache__" / "commands").mkdir()
+        (root / "sub" / "__pycache__" / "commands" / "c.md").write_text(
+            "---\ndescription: y\n---\nx\n")
+        (root / "real").mkdir()
+        (root / "real" / ".cursorrules").write_text("real rule\n")
+        assert [e["name"] for e in catalog.scan_dir(root)] == ["real"]
+
     def test_unreadable_rule_file_skipped(self, tmp_path):
         root = tmp_path / "tap"
         (root / "a").mkdir(parents=True)