boost-skill-cli 1.0.153__tar.gz → 1.0.155__tar.gz

{boost_skill_cli-1.0.153 → boost_skill_cli-1.0.155}/Makefile

@@ -43,6 +43,7 @@ lint:
 	@command -v actionlint >/dev/null 2>&1 && actionlint || echo "actionlint not on PATH — skipping (CI enforces it)"
 	$(PY) scripts/build_registries.py --check
 	$(PY) scripts/build_roadmap.py --check
+	$(PY) scripts/import_budget.py
 
 # Supply-chain CVE gate: fail on a known OSV/PyPI advisory in the project's
 # dependency closure. Mirrors the pip-audit CI workflow; run before releasing.

{boost_skill_cli-1.0.153 → boost_skill_cli-1.0.155}/PKG-INFO

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

{boost_skill_cli-1.0.153 → boost_skill_cli-1.0.155}/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.153'
-__version_tuple__ = version_tuple = (1, 0, 153)
+__version__ = version = '1.0.155'
+__version_tuple__ = version_tuple = (1, 0, 155)
 
 __commit_id__ = commit_id = None

{boost_skill_cli-1.0.153 → boost_skill_cli-1.0.155}/boost_cli/cli.py

@@ -17,6 +17,7 @@ from .core import logs
 from .core import nethttp
 from .core import output as out
 from .errors import BoostError
+import contextlib
 
 # Global flags handled before dispatch, stripped from the command's own argv.
 _GLOBAL_FLAGS = {
@@ -174,7 +175,7 @@ def print_command_help(name: str) -> int:
     meta = resolve(name)
     if not meta:
         return _unknown(name)
-    _group, module, summary = meta
+    _group, _module, summary = meta
     print(out.c("boost %s" % name, out.BOLD) + " — " + summary)
     # delegate to the command's own --help for flags
     return _dispatch(name, ["--help"], soft=True)
@@ -196,7 +197,7 @@ def _unknown(name: str) -> int:
 
 
 def _dispatch(name: str, argv: List[str], soft: bool = False) -> int:
-    group, module, _summary = _BY_NAME[name]
+    _group, module, _summary = _BY_NAME[name]
     mod = importlib.import_module("boost_cli.commands.%s" % module)
     func = getattr(mod, "cmd_%s" % name.replace("-", "_"), None)
     if func is None:
@@ -264,13 +265,11 @@ def main(argv: List[str] | None = None) -> int:
         print()
         return rc
     except BrokenPipeError:
-        try:
+        with contextlib.suppress(Exception):
             sys.stdout.close()
-        except Exception:
-            pass
         rc = 0
         return rc
-    except Exception as e:  # noqa: BLE001 — top-level safety net
+    except Exception as e:
         report = logs.write_crash_report(e, [name, *rest])
         if logs.is_debug():
             raise

{boost_skill_cli-1.0.153 → boost_skill_cli-1.0.155}/boost_cli/cliparse.py

@@ -19,7 +19,7 @@ from .core import output as out
 
 
 class BoostArgumentParser(argparse.ArgumentParser):
-    def error(self, message: str):  # noqa: D401 - argparse hook
+    def error(self, message: str):
         """Print a branded error + dimmed usage, then exit 2 (argparse's code)."""
         out.err(message)
         sys.stderr.write(out.c(self.format_usage(), out.DIM))

{boost_skill_cli-1.0.153 → boost_skill_cli-1.0.155}/boost_cli/commands/bmad.py

@@ -119,7 +119,7 @@ def _run_installer(directory: Path, modules: str) -> "subprocess.CompletedProces
         proc = subprocess.run(cmd, capture_output=True, text=True, timeout=900)
     except (OSError, subprocess.TimeoutExpired) as e:
         raise BoostError("bmad install failed: %s" % e,
-                         hint="run it yourself: " + " ".join(cmd))
+                         hint="run it yourself: " + " ".join(cmd)) from e
     if proc.returncode != 0:
         tail = (proc.stderr or proc.stdout or "").strip().splitlines()
         raise BoostError(

{boost_skill_cli-1.0.153 → boost_skill_cli-1.0.155}/boost_cli/commands/configuration.py

@@ -72,7 +72,7 @@ def cmd_config(argv) -> int:
         except TypeError as e:
             raise BoostError(str(e),
                             hint="the parent key holds a plain value — "
-                                 "`boost config unset` it first")
+                                 "`boost config unset` it first") from e
         val = config.get(args.key)
         journal.log("config", args.key, op="set")
         out.ok("set %s = %s"
@@ -101,25 +101,31 @@ def cmd_clean(argv) -> int:
                    help="also remove snapshots older than 90 days")
     args = p.parse_args(argv)
 
-    items = []  # (path, kind, bytes)
-    for _agent, spec in agents.known_agents().items():
+    items: list[tuple[Path, str, int]] = []  # (path, kind, bytes)
+    for spec in agents.known_agents().values():
         adir = spec["dir"]
         if not adir.is_dir():
             continue
-        for link in sorted(adir.iterdir()):
-            if link.is_symlink() and not link.exists():
-                items.append((link, "broken symlink", 0))
+        items.extend(
+            (link, "broken symlink", 0)
+            for link in sorted(adir.iterdir())
+            if link.is_symlink() and not link.exists()
+        )
 
     configured = {t.safe_name for t in registry.list_taps()}
     if paths.cache_dir().is_dir():
-        for f in sorted(paths.cache_dir().glob("*.json")):
-            if f.stem not in configured:
-                items.append((f, "stale tap cache", f.stat().st_size))
+        items.extend(
+            (f, "stale tap cache", f.stat().st_size)
+            for f in sorted(paths.cache_dir().glob("*.json"))
+            if f.stem not in configured
+        )
 
     if paths.lock_history_dir().is_dir():
         snaps = sorted(paths.lock_history_dir().glob("lock-*.json"))
-        for old in snaps[:-50]:
-            items.append((old, "old lock history", old.stat().st_size))
+        items.extend(
+            (old, "old lock history", old.stat().st_size)
+            for old in snaps[:-50]
+        )
 
     if paths.store_dir().is_dir():
         for pth in sorted(paths.store_dir().rglob("*")):
@@ -442,7 +448,7 @@ def cmd_onboard(argv) -> int:
                                   text=True, timeout=120)
         except (subprocess.TimeoutExpired, OSError) as e:
             raise BoostError("gh pr create failed: %s" % e,
-                            hint="branch %s is committed — push it and open the PR manually" % branch)
+                            hint="branch %s is committed — push it and open the PR manually" % branch) from e
         if proc.returncode != 0:
             tail = (proc.stderr or proc.stdout or "").strip().splitlines()
             raise BoostError("gh pr create failed: %s" % (tail[-1] if tail else "unknown error"),
@@ -669,7 +675,7 @@ def cmd_schedule(argv) -> int:
                 kept = [ln for ln in lines if not ln.rstrip().endswith(_CRON_MARK)]
                 try:
                     proc = subprocess.run(["crontab", "-"],
-                                          input="\n".join(kept + [entry]) + "\n",
+                                          input="\n".join([*kept, entry]) + "\n",
                                           capture_output=True, text=True, timeout=30)
                 except (OSError, subprocess.TimeoutExpired):
                     proc = None
@@ -943,7 +949,7 @@ def cmd_mcp(argv) -> int:
             proc = subprocess.run(cmd, capture_output=True, text=True, timeout=60)
         except (OSError, subprocess.TimeoutExpired) as e:
             raise BoostError("claude mcp %s failed: %s" % (args.action, e),
-                            hint="run it yourself: " + " ".join(cmd))
+                            hint="run it yourself: " + " ".join(cmd)) from e
         for ln in (proc.stdout or "").strip().splitlines():
             out.info(ln)
         if proc.returncode != 0:

{boost_skill_cli-1.0.153 → boost_skill_cli-1.0.155}/boost_cli/commands/discovery.py

@@ -17,11 +17,12 @@ import urllib.parse
 from pathlib import Path
 
 from .. import cliparse, spin
-from ..core import (agents, ai, catalog, config, dense, embed, gitutil,
+from ..core import (agents, ai, catalog, config, gitutil,
                     journal, lockfile, paths, rag, registry, store, util)
 from ..core import output as out
 from ..core.stackprobe import detect_stack  # re-exported: shared with Quality
 from ..errors import BoostError
+import contextlib
 
 __all__ = ["detect_stack"]
 
@@ -31,7 +32,7 @@ def _positive_int(s: str) -> int:
     try:
         v = int(s)
     except ValueError:
-        raise argparse.ArgumentTypeError("invalid int value: %r" % s)
+        raise argparse.ArgumentTypeError("invalid int value: %r" % s) from None
     if v < 1:
         raise argparse.ArgumentTypeError("must be >= 1")
     return v
@@ -164,6 +165,11 @@ def cmd_reindex(argv):
         stats = rag.build(force=args.force)
     dense_stats = None
     if args.dense:
+        # Imported lazily (only on `--dense`) so the optional dense/embedding
+        # stack never loads for the common offline commands this module also
+        # holds — count, stats, trending, plain search. See the import-budget
+        # gate (scripts/import_budget.py).
+        from ..core import embed
         with spin.Spinner("embedding chunks into the dense store"):
             dense_stats = _reindex_dense(args.force)
     if args.as_json:
@@ -191,6 +197,9 @@ def cmd_reindex(argv):
 
 def _reindex_dense(force):
     """Build the dense vector store; returns stats or None when unavailable."""
+    # Local import: the dense/embedding engines are opt-in (`search --dense`),
+    # so they stay out of startup for every other command in this module.
+    from ..core import dense, embed
     if not dense.have_backend() or not embed.available():
         return None
     return dense.build(force=force)
@@ -219,7 +228,7 @@ def cmd_index(argv):
                  "search/code?q=filename:SKILL.md&per_page=100&page=%d" % page],
                 capture_output=True, text=True, timeout=120)
         except (subprocess.TimeoutExpired, OSError) as e:
-            raise BoostError("gh api timed out on page %d" % page, hint=str(e))
+            raise BoostError("gh api timed out on page %d" % page, hint=str(e)) from e
         if proc.returncode != 0:
             tail = "\n".join((proc.stderr or proc.stdout or "").strip()
                              .splitlines()[-3:])
@@ -233,7 +242,7 @@ def cmd_index(argv):
             data = json.loads(proc.stdout)
         except json.JSONDecodeError:
             raise BoostError("gh api returned unparseable JSON",
-                            hint="try `boost index` again, or `gh auth status`")
+                            hint="try `boost index` again, or `gh auth status`") from None
         if page == 1:
             total = int(data.get("total_count") or 0)
         batch = data.get("items") or []
@@ -328,7 +337,7 @@ def cmd_discover(argv):
         data = json.loads(dpath.read_text(encoding="utf-8"))
     except (json.JSONDecodeError, OSError):
         raise BoostError("the discovery index is corrupt",
-                        hint="rebuild it with `boost index`")
+                        hint="rebuild it with `boost index`") from None
     all_items = data.get("items") or []
     tokens = [t.lower() for t in args.query if t.strip()]
     items = [it for it in all_items
@@ -598,7 +607,7 @@ def _aurora_pairs(curses):
     """Allocate Aurora curses colour pairs; token-name -> attr int (0 if none)."""
     names = ("cyan", "violet", "pink", "green", "yellow")
     try:
-        custom = curses.can_change_color() and curses.COLORS >= 16 + len(names)
+        custom = curses.can_change_color() and 16 + len(names) <= curses.COLORS
     except curses.error:
         custom = False
     fg = dict(_nearest_base(curses))
@@ -695,19 +704,15 @@ def _browse_tui(curses, entries):
         return "loading description…"
 
     def ui(scr):
-        try:
+        with contextlib.suppress(curses.error):
             curses.curs_set(0)
-        except curses.error:
-            pass
         scr.timeout(80)  # short poll so a finished lazy-load redraws promptly
         th = _aurora_theme(curses)
 
         def put(y, x, s, attr=0):  # clip-safe cell writer
             if 0 <= y < _h and 0 <= x < _w - 1 and s:
-                try:
+                with contextlib.suppress(curses.error):
                     scr.addnstr(y, x, s, _w - 1 - x, attr)
-                except curses.error:
-                    pass
 
         filt, sel, detail = "", 0, False
         selected: set = set()

{boost_skill_cli-1.0.153 → boost_skill_cli-1.0.155}/boost_cli/commands/info.py

@@ -31,7 +31,7 @@ def _read(p: Path) -> str:
     try:
         return Path(p).read_text(encoding="utf-8", errors="replace")
     except OSError as e:
-        raise BoostError("cannot read %s: %s" % (_tilde(p), e))
+        raise BoostError("cannot read %s: %s" % (_tilde(p), e)) from e
 
 
 def _resolve_skill_md(name: str):
@@ -115,7 +115,7 @@ def _kind_table(heading, items, extra=None):
             row.append(str(e.get(extra[1], "") or ""))
         rows.append(tuple(row))
     if extra:
-        headers = headers + (extra[0],)
+        headers = (*headers, extra[0])
     out.table(rows, headers=headers)
     noun = heading.split()[-1][:-1]  # "installed rules" -> "rule"
     print("  " + out.aurora("%d %s%s installed"
@@ -319,10 +319,10 @@ def cmd_edit(argv):
     editor = os.environ.get("VISUAL") or os.environ.get("EDITOR") or "vi"
     cmd = shlex.split(editor) or ["vi"]   # support EDITOR="code -w" etc.
     try:
-        rc = subprocess.call(cmd + [str(path)])
+        rc = subprocess.call([*cmd, str(path)])
     except OSError as e:
         raise BoostError("cannot launch editor %r: %s" % (editor, e),
-                        hint="set $VISUAL or $EDITOR to a valid command")
+                        hint="set $VISUAL or $EDITOR to a valid command") from e
     if rc != 0:
         out.warn("editor exited with status %d" % rc)
     sha = util.sha256_dir(sdir)
@@ -595,9 +595,11 @@ def cmd_deps(argv):
     unmet, pairs, seen = [], [], set()
     for name in sorted(inst):
         meta = _skill_meta(name) or {}
-        for r in _as_list(meta.get("requires")):
-            if r not in inst:
-                unmet.append({"skill": name, "requires": r})
+        unmet.extend(
+            {"skill": name, "requires": r}
+            for r in _as_list(meta.get("requires"))
+            if r not in inst
+        )
         for c_name in _as_list(meta.get("conflicts")):
             if c_name in inst:
                 key = tuple(sorted((name, c_name)))

{boost_skill_cli-1.0.153 → boost_skill_cli-1.0.155}/boost_cli/commands/intelligence.py

@@ -22,6 +22,7 @@ from ..core import (ai, catalog, frontmatter, gitutil, imperative, journal,
                     lockfile, paths, store, util)
 from ..core import output as out
 from ..errors import BoostError
+import contextlib
 
 # ---------------------------------------------------------------- helpers
 
@@ -777,9 +778,8 @@ def cmd_context(argv: List[str]) -> int:
         inst = lockfile.installed()
         for name in sorted(_mentioned_skills(state)):
             entry = inst.get(name)
-            if entry and not entry.get("quarantined"):
-                if store.link_agents(name).linked:
-                    restored += 1
+            if entry and not entry.get("quarantined") and store.link_agents(name).linked:
+                restored += 1
         state["enabled"] = False
         _save_state(_CONTEXT_STATE, state)
         journal.log("context", "disable", restored=restored)
@@ -1005,10 +1005,8 @@ def _repo_activity(since: str) -> Tuple[Optional[int], Optional[int]]:
     proc = gitutil.run(["rev-list", "--count", "--since=" + since, "HEAD"],
                        cwd=Path.cwd(), check=False)
     if proc.returncode == 0:
-        try:
+        with contextlib.suppress(ValueError):
             commits = int(proc.stdout.strip() or 0)
-        except ValueError:
-            pass
     proc = gitutil.run(["log", "--name-only", "--since=" + since,
                         "--pretty=format:"], cwd=Path.cwd(), check=False)
     if proc.returncode == 0:

{boost_skill_cli-1.0.153 → boost_skill_cli-1.0.155}/boost_cli/commands/pkg.py

@@ -568,7 +568,7 @@ def _bundle_dump(file: Optional[str]) -> int:
         dest.write_text(text, encoding="utf-8")
     except OSError as e:
         raise BoostError("cannot write %s: %s" % (_tilde(dest), e.strerror or e),
-                        hint="check the path exists and is writable")
+                        hint="check the path exists and is writable") from e
     n_taps = sum(1 for ln in text.splitlines() if ln.startswith("tap "))
     n_skills = sum(1 for ln in text.splitlines() if ln.startswith("skill "))
     out.ok("wrote %s (%s, %s)" % (_tilde(dest), _plural(n_taps, "tap"),
@@ -591,7 +591,7 @@ def _bundle_install(file: Optional[str]) -> int:
         try:
             text, label = path.read_text(encoding="utf-8"), str(path)
         except OSError as e:
-            raise BoostError("cannot read %s: %s" % (_tilde(path), e.strerror or e))
+            raise BoostError("cannot read %s: %s" % (_tilde(path), e.strerror or e)) from e
     taps_added = installed_n = present = failed = 0
     have_taps = {t.name for t in registry.list_taps()}
     have_skills = set(lockfile.installed())
@@ -929,18 +929,18 @@ def _snapshot_restore(snap_id: str) -> int:
     # Read the whole archive BEFORE touching the store, so a corrupt
     # snapshot can never leave us with an emptied environment.
     try:
-        tf = tarfile.open(str(tar_path), "r:gz")
+        tf = tarfile.open(str(tar_path), "r:gz")  # noqa: SIM115  (opened in a try to translate the error; used via `with tf:` below)
     except (tarfile.TarError, OSError, EOFError) as e:
         raise BoostError("snapshot %s is unreadable: %s" % (snap_id, e),
                         hint="the store was left untouched — "
-                             "see `boost snapshot list` for other snapshots")
+                             "see `boost snapshot list` for other snapshots") from e
     with tf:
         try:
             members = tf.getmembers()
         except (tarfile.TarError, OSError, EOFError) as e:
             raise BoostError("snapshot %s is corrupt: %s" % (snap_id, e),
                             hint="the store was left untouched — "
-                                 "see `boost snapshot list` for other snapshots")
+                                 "see `boost snapshot list` for other snapshots") from e
         for child in root.iterdir():
             if child.is_dir() and not child.is_symlink():
                 util.rmtree(child)
@@ -1012,7 +1012,7 @@ def cmd_export(argv: List[str]) -> int:
                     tf.add(str(store.skill_store_dir(name)), arcname=name)
     except OSError as e:
         raise BoostError("cannot write %s: %s" % (_tilde(dest), e.strerror or e),
-                        hint="check the output path exists and is writable")
+                        hint="check the output path exists and is writable") from e
     out.ok("exported %s → %s (%s)" % (_plural(len(chosen), "skill"),
                                       _tilde(dest.resolve()),
                                       util.human_size(dest.stat().st_size)))
@@ -1063,7 +1063,7 @@ def cmd_adapt(argv: List[str]) -> int:
             util.atomic_write_text(dest, source)
         except OSError as e:
             raise BoostError("cannot write %s: %s" % (_tilde(dest), e.strerror or e),
-                            hint="check the output path exists and is writable")
+                            hint="check the output path exists and is writable") from e
         out.ok("adapted %s → %s (%s · %s)" % (display, _tilde(dest.resolve()),
                                               args.to, model or "framework default"))
     else:

{boost_skill_cli-1.0.153 → boost_skill_cli-1.0.155}/boost_cli/commands/quality.py

@@ -96,9 +96,10 @@ def _broken_links() -> List[Path]:
     for adir in agents.enabled_agents().values():
         if not adir.is_dir():
             continue
-        for link in sorted(adir.iterdir()):
-            if link.is_symlink() and not link.exists():
-                broken.append(link)
+        broken.extend(
+            link for link in sorted(adir.iterdir())
+            if link.is_symlink() and not link.exists()
+        )
     return broken
 
 
@@ -353,7 +354,7 @@ def cmd_doctor(argv):
         bad("%d broken symlink%s in agent dirs — run `boost heal`"
             % (len(broken), _s(len(broken))))
 
-    for agent, adir in enabled.items():
+    for adir in enabled.values():
         if adir.is_dir() and not os.access(str(adir), os.W_OK):
             bad("agent dir %s is not writable" % _tilde(adir))
 
@@ -776,10 +777,7 @@ def cmd_heal(argv):
     dry = args.dry_run
     actions: List[str] = []
 
-    wanted = [paths.boost_home(), paths.repos_dir(), paths.cache_dir(),
-              paths.logs_dir(), paths.state_dir(), paths.snapshots_dir(),
-              paths.lock_history_dir(), paths.profiles_dir(),
-              paths.store_dir()] + list(agents.enabled_agents().values())
+    wanted = [paths.boost_home(), paths.repos_dir(), paths.cache_dir(), paths.logs_dir(), paths.state_dir(), paths.snapshots_dir(), paths.lock_history_dir(), paths.profiles_dir(), paths.store_dir(), *list(agents.enabled_agents().values())]
     missing = [d for d in wanted if not d.is_dir()]
     if missing:
         if dry:
@@ -859,9 +857,11 @@ def cmd_conflict(argv):
     for name, _entry in installed:
         meta, body = _read_skill(store.skill_store_dir(name))
         conflicts = meta.get("conflicts") or []
-        for other in (conflicts if isinstance(conflicts, list) else [conflicts]):
-            if str(other) in installed_names and str(other) != name:
-                declared.append((name, str(other)))
+        declared.extend(
+            (name, str(other))
+            for other in (conflicts if isinstance(conflicts, list) else [conflicts])
+            if str(other) in installed_names and str(other) != name
+        )
         for raw in body.splitlines():
             m = imperative.RULE_RE.match(raw)
             if not m:
@@ -879,7 +879,7 @@ def cmd_conflict(argv):
 
     pairs, seen = [], set()
     for da, db in declared:
-        key = tuple(sorted((da, db))) + ("declared",)
+        key = (*tuple(sorted((da, db))), "declared")
         if key in seen:
             continue
         seen.add(key)

{boost_skill_cli-1.0.153 → boost_skill_cli-1.0.155}/boost_cli/commands/team.py

@@ -183,7 +183,7 @@ def _load_profile(name: str) -> dict:
     try:
         return json.loads(p.read_text(encoding="utf-8"))
     except (json.JSONDecodeError, OSError) as e:
-        raise BoostError("profile %s is unreadable: %s" % (name, e))
+        raise BoostError("profile %s is unreadable: %s" % (name, e)) from e
 
 
 def _profile_diff(profile: dict):

{boost_skill_cli-1.0.153 → boost_skill_cli-1.0.155}/boost_cli/core/claude_settings.py

@@ -26,6 +26,7 @@ from typing import List, Optional
 
 from . import paths, util
 from ..errors import BoostError
+import contextlib
 
 SCOPES = ("global", "project")
 MARKER = "# boost:"
@@ -87,10 +88,8 @@ def _prune_history() -> None:
     snaps = sorted(_history_dir().glob("*.json"),
                    key=lambda f: (f.stat().st_mtime, f.name))
     for old in snaps[:-HISTORY_KEEP]:
-        try:
+        with contextlib.suppress(OSError):
             old.unlink()
-        except OSError:
-            pass
 
 
 # --------------------------------------------------------------- marker helpers

{boost_skill_cli-1.0.153 → boost_skill_cli-1.0.155}/boost_cli/core/frontmatter.py

@@ -150,8 +150,7 @@ def dump(meta: dict) -> str:
     for k, v in meta.items():
         if isinstance(v, list):
             lines.append("%s:" % k)
-            for item in v:
-                lines.append("  - %s" % item)
+            lines.extend("  - %s" % item for item in v)
         elif isinstance(v, bool):
             lines.append("%s: %s" % (k, "true" if v else "false"))
         elif v is None:

{boost_skill_cli-1.0.153 → boost_skill_cli-1.0.155}/boost_cli/core/gitutil.py

@@ -20,11 +20,11 @@ def run(args: List[str], cwd: Optional[Path] = None, check: bool = True,
                         hint="install git, e.g. `xcode-select --install` or `brew install git`")
     try:
         proc = subprocess.run(
-            ["git"] + args, cwd=str(cwd) if cwd else None,
+            ["git", *args], cwd=str(cwd) if cwd else None,
             capture_output=True, text=True, timeout=timeout,
         )
     except subprocess.TimeoutExpired:
-        raise BoostError("git %s timed out after %ds" % (args[0], timeout))
+        raise BoostError("git %s timed out after %ds" % (args[0], timeout)) from None
     if check and proc.returncode != 0:
         detail = (proc.stderr or proc.stdout or "").strip().splitlines()
         raise BoostError("git %s failed: %s" % (args[0], detail[-1] if detail else "unknown error"))

{boost_skill_cli-1.0.153 → boost_skill_cli-1.0.155}/boost_cli/core/injectscan.py

@@ -107,10 +107,12 @@ def scan_text(text: str) -> List[Finding]:
     """
     findings: List[Finding] = []
     for lineno, raw in enumerate(text.splitlines(), start=1):
-        for rule in RULES:
-            if rule.pattern.search(raw):
-                findings.append(Finding(rule.id, rule.severity, lineno,
-                                        raw.strip()[:200], rule.description))
+        findings.extend(
+            Finding(rule.id, rule.severity, lineno,
+                    raw.strip()[:200], rule.description)
+            for rule in RULES
+            if rule.pattern.search(raw)
+        )
     findings.sort(key=lambda f: (-_SEVERITY_RANK[f.severity], f.line,
                                  f.rule_id))
     return findings

{boost_skill_cli-1.0.153 → boost_skill_cli-1.0.155}/boost_cli/core/logs.py

@@ -43,6 +43,7 @@ from pathlib import Path
 from typing import List, Optional
 
 from . import paths
+import contextlib
 
 LOGGER_NAME = "boost"
 MAX_BYTES = 1_000_000  # ~1 MB per file …
@@ -117,10 +118,8 @@ def reset() -> None:
     global _configured, _debug_console
     logger = logging.getLogger(LOGGER_NAME)
     for h in list(logger.handlers):
-        try:
+        with contextlib.suppress(Exception):
             h.close()
-        except Exception:
-            pass
         logger.removeHandler(h)
     _configured = False
     _debug_console = False
@@ -236,10 +235,8 @@ def write_crash_report(exc: BaseException, argv: List[str]) -> Optional[Path]:
         "",
     ])
     # Always try to get it into the rotating trail, even if the file write fails.
-    try:
+    with contextlib.suppress(Exception):
         get_logger().error("crash: %s: %s", type(exc).__name__, exc)
-    except Exception:
-        pass
     try:
         paths.logs_dir().mkdir(parents=True, exist_ok=True)
         report = paths.logs_dir() / ("crash-%s.log" % _file_stamp())
@@ -257,7 +254,5 @@ def _prune_crash_reports() -> None:
     except OSError:
         return
     for stale in reports[:-KEEP_CRASH_REPORTS]:
-        try:
+        with contextlib.suppress(OSError):
             stale.unlink()
-        except OSError:
-            pass

{boost_skill_cli-1.0.153 → boost_skill_cli-1.0.155}/boost_cli/core/mcp.py

@@ -110,7 +110,7 @@ def handle_request(req: dict, *, version: str,
         except BoostError as e:
             text = "Error: %s" % e.message + ("\nhint: %s" % e.hint if e.hint else "")
             is_err = True
-        except Exception as e:  # noqa: BLE001 — server must not die
+        except Exception as e:
             text, is_err = "Error: %s" % e, True
         if text is None:
             resp["error"] = {"code": -32602, "message": "unknown tool %r" % tool}

{boost_skill_cli-1.0.153 → boost_skill_cli-1.0.155}/boost_cli/core/output.py

@@ -13,6 +13,7 @@ import os
 import re
 import shutil
 import sys
+import contextlib
 
 RESET = "\033[0m"
 BOLD = "\033[1m"
@@ -36,10 +37,8 @@ def harden_console_encoding() -> None:
     """
     for stream in (sys.stdout, sys.stderr):
         if hasattr(stream, "reconfigure"):
-            try:
+            with contextlib.suppress(OSError, ValueError):
                 stream.reconfigure(encoding="utf-8", errors="replace")
-            except (OSError, ValueError):
-                pass
 
 
 def use_color(stream=None) -> bool:
@@ -273,7 +272,7 @@ def panel(lines, title: str | None = None, hue: str = "cyan") -> str:
         lines = [lines]
     widths = [visible_len(x) for x in lines]
     tw = visible_len(title) if title else 0
-    inner = max(widths + [tw + 2])  # a titled rule needs a space each side
+    inner = max([*widths, tw + 2])  # a titled rule needs a space each side
 
     def b(s: str) -> str:
         return aurora(s, hue)

{boost_skill_cli-1.0.153 → boost_skill_cli-1.0.155}/boost_cli/core/policy.py

@@ -9,6 +9,7 @@ import json
 from typing import List
 
 from . import config, paths
+import contextlib
 
 DEFAULTS = {
     "blocked_skills": [],      # names never allowed
@@ -26,10 +27,8 @@ def load() -> dict:
     p = paths.policy_path()
     base = dict(DEFAULTS)
     if p.exists():
-        try:
+        with contextlib.suppress(json.JSONDecodeError, OSError):
             base.update(json.loads(p.read_text(encoding="utf-8")))
-        except (json.JSONDecodeError, OSError):
-            pass
     return base