boost-skill-cli 1.0.152__tar.gz → 1.0.154__tar.gz

{boost_skill_cli-1.0.152 → boost_skill_cli-1.0.154}/PKG-INFO

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

{boost_skill_cli-1.0.152 → boost_skill_cli-1.0.154}/boost

@@ -12,6 +12,12 @@ while [ -h "$SOURCE" ]; do
   [[ "$SOURCE" != /* ]] && SOURCE="$DIR/$SOURCE"
 done
 ROOT="$(cd -P "$(dirname "$SOURCE")" >/dev/null 2>&1 && pwd)"
+# Under Git Bash on Windows, $ROOT is an MSYS path (/d/a/boost/boost); native
+# Windows Python can't resolve that, so convert to a mixed path (D:/a/...)
+# before it's embedded in the -c script below. No-op on Linux/macOS.
+if command -v cygpath >/dev/null 2>&1; then
+  ROOT="$(cygpath -m "$ROOT")"
+fi
 
 find_python() {
   for py in python3 python; do

{boost_skill_cli-1.0.152 → boost_skill_cli-1.0.154}/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.152'
-__version_tuple__ = version_tuple = (1, 0, 152)
+__version__ = version = '1.0.154'
+__version_tuple__ = version_tuple = (1, 0, 154)
 
 __commit_id__ = commit_id = None

{boost_skill_cli-1.0.152 → boost_skill_cli-1.0.154}/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:
@@ -224,6 +225,7 @@ def _extract_globals(argv: List[str]) -> tuple[dict, List[str]]:
 
 def main(argv: List[str] | None = None) -> int:
     argv = list(sys.argv[1:] if argv is None else argv)
+    out.harden_console_encoding()
     # Belt-and-suspenders fork safety: a host may fork() into `boost mcp
     # --stdio` without the no_proxy env boost injects at registration, leaving
     # the process one default getproxies() away from a macOS _scproxy SIGABRT.
@@ -263,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.152 → boost_skill_cli-1.0.154}/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.152 → boost_skill_cli-1.0.154}/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(
@@ -192,7 +192,7 @@ def _copy_global_skills(modules) -> int:
                 continue
             t = dest / d.name
             if t.exists():
-                shutil.rmtree(t)
+                util.rmtree(t)
             shutil.copytree(d, t)
             n += 1
         return n
@@ -278,7 +278,7 @@ def _disable(scope) -> int:
     for d in sorted(_skills_dir(scope).glob("bmad-*")):
         target = qdir / d.name
         if target.exists():
-            shutil.rmtree(target)
+            util.rmtree(target)
         shutil.move(str(d), str(target))
         moved += 1
     _set_scope_state(scope, startup=False, disabled=True)
@@ -298,7 +298,7 @@ def _enable(scope) -> int:
         for d in sorted(qdir.glob("bmad-*")):
             target = dest / d.name
             if target.exists():
-                shutil.rmtree(target)
+                util.rmtree(target)
             shutil.move(str(d), str(target))
             restored += 1
     _set_scope_state(scope, disabled=False)
@@ -366,7 +366,7 @@ def _state_read() -> dict:
     if not p.exists():
         return {"global": {}, "projects": {}}
     try:
-        d = json.loads(p.read_text())
+        d = json.loads(p.read_text(encoding="utf-8"))
     except (json.JSONDecodeError, OSError):
         return {"global": {}, "projects": {}}
     d.setdefault("global", {})
@@ -376,7 +376,7 @@ def _state_read() -> dict:
 
 def _state_write(d: dict) -> None:
     paths.ensure_dirs()
-    _state_path().write_text(json.dumps(d, indent=2) + "\n")
+    _state_path().write_text(json.dumps(d, indent=2) + "\n", encoding="utf-8")
 
 
 def _proj_key() -> str:

{boost_skill_cli-1.0.152 → boost_skill_cli-1.0.154}/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("*")):
@@ -152,7 +158,7 @@ def cmd_clean(argv) -> int:
                 if pth.is_symlink() or pth.is_file():
                     pth.unlink()
                 elif pth.is_dir():
-                    shutil.rmtree(pth)
+                    util.rmtree(pth)
             except OSError as e:
                 out.warn("could not remove %s: %s" % (_tilde(pth), e))
                 continue
@@ -224,7 +230,7 @@ def cmd_create(argv) -> int:
     title = name.replace("-", " ").title()
     target.mkdir(parents=True, exist_ok=True)
     skill_md.write_text(frontmatter.dump(meta) + "\n\n"
-                        + _CREATE_BODY % {"title": title})
+                        + _CREATE_BODY % {"title": title}, encoding="utf-8")
     journal.log("create", name, path=str(target))
     out.ok("created %s" % _tilde(skill_md))
     if args.install:
@@ -426,7 +432,7 @@ def cmd_onboard(argv) -> int:
     for rel, content in files:
         fp = repo / rel
         fp.parent.mkdir(parents=True, exist_ok=True)
-        fp.write_text(content)
+        fp.write_text(content, encoding="utf-8")
         out.ok("created %s" % _tilde(fp))
     journal.log("onboard", _tilde(repo), pr=args.pr or None)
 
@@ -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"),
@@ -599,7 +605,7 @@ def cmd_schedule(argv) -> int:
             if plist.exists():
                 present = True
                 m = re.search(r"<key>StartInterval</key>\s*<integer>(\d+)</integer>",
-                              plist.read_text())
+                              plist.read_text(encoding="utf-8"))
                 if m:
                     secs = int(m.group(1))
                     interval = _interval_label(secs)
@@ -641,7 +647,7 @@ def cmd_schedule(argv) -> int:
         if darwin:
             plist = _plist_path()
             plist.parent.mkdir(parents=True, exist_ok=True)
-            plist.write_text(_plist_body(shim, seconds))
+            plist.write_text(_plist_body(shim, seconds), encoding="utf-8")
             out.ok("wrote %s" % _tilde(plist))
             try:
                 subprocess.run(["launchctl", "unload", str(plist)],
@@ -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.152 → boost_skill_cli-1.0.154}/boost_cli/commands/discovery.py

@@ -22,6 +22,7 @@ from ..core import (agents, ai, catalog, config, dense, embed, gitutil,
 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
@@ -219,7 +220,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 +234,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 []
@@ -252,7 +253,7 @@ def cmd_index(argv):
     paths.ensure_dirs()
     _discovery_path().write_text(json.dumps(
         {"generated": util.now_iso(), "github_total": total, "items": items},
-        indent=1))
+        indent=1), encoding="utf-8")
     repos = len({it["repo"] for it in items})
     out.ok("indexed %d skill files across %d repos (GitHub reports %d total)"
            % (len(items), repos, total))
@@ -325,10 +326,10 @@ def cmd_discover(argv):
                      "gh auth login`), then run `boost index`")
         return 0
     try:
-        data = json.loads(dpath.read_text())
+        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 +599,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 +696,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()
@@ -979,7 +976,7 @@ def cmd_count(argv):
     dpath = _discovery_path()
     if dpath.exists():
         try:
-            discovery = len(json.loads(dpath.read_text()).get("items") or [])
+            discovery = len(json.loads(dpath.read_text(encoding="utf-8")).get("items") or [])
         except (json.JSONDecodeError, OSError):
             discovery = None
     if args.as_json:

{boost_skill_cli-1.0.152 → boost_skill_cli-1.0.154}/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.152 → boost_skill_cli-1.0.154}/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
 
@@ -60,14 +61,14 @@ def _load_state(fname: str, default: dict) -> dict:
     if not p.exists():
         return json.loads(json.dumps(default))
     try:
-        return json.loads(p.read_text())
+        return json.loads(p.read_text(encoding="utf-8"))
     except (json.JSONDecodeError, OSError):
         return json.loads(json.dumps(default))
 
 
 def _save_state(fname: str, data: dict) -> None:
     paths.ensure_dirs()
-    (paths.state_dir() / fname).write_text(json.dumps(data, indent=2) + "\n")
+    (paths.state_dir() / fname).write_text(json.dumps(data, indent=2) + "\n", encoding="utf-8")
 
 
 def _install_generated(name: str, text: str) -> None:
@@ -75,7 +76,7 @@ def _install_generated(name: str, text: str) -> None:
     with tempfile.TemporaryDirectory(prefix="boost-gen-") as td:
         src = Path(td) / name
         src.mkdir()
-        (src / "SKILL.md").write_text(text)
+        (src / "SKILL.md").write_text(text, encoding="utf-8")
         res = store.install_from_path(src, name=name, tap_label="local")
     out.ok("installed %s → %s" % (name, _tilde(res.dest)))
     if res.linked:
@@ -88,7 +89,7 @@ def _write_generated(dest: Path, text: str) -> bool:
         out.info("aborted — %s left untouched" % _tilde(dest))
         return False
     dest.parent.mkdir(parents=True, exist_ok=True)
-    dest.write_text(text)
+    dest.write_text(text, encoding="utf-8")
     out.ok("wrote %s" % _tilde(dest))
     return True
 
@@ -649,7 +650,7 @@ def cmd_evolve(argv: List[str]) -> int:
     if not args.apply:
         out.info(out.role("re-run with --apply to write these changes", "muted"))
         return 0
-    skill_md.write_text(new)
+    skill_md.write_text(new, encoding="utf-8")
     entry["sha256"] = util.sha256_dir(store.skill_store_dir(args.name))
     entry["updated_at"] = util.now_iso()
     entry["version"] = new_ver
@@ -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.152 → boost_skill_cli-1.0.154}/boost_cli/commands/pkg.py

@@ -22,14 +22,7 @@ from ..core import output as out
 from ..errors import BoostError
 
 
-def _tilde(p) -> str:
-    """Contract $HOME to ~ for display (also matches the resolved home,
-    so /private/var vs /var symlinks on macOS still contract)."""
-    s = str(p)
-    for h in (str(paths.home()), str(paths.home().resolve())):
-        if s == h or s.startswith(h + os.sep):
-            return "~" + s[len(h):]
-    return s
+_tilde = paths.tilde
 
 
 def _plural(n: int, word: str) -> str:
@@ -338,7 +331,7 @@ def cmd_sync(argv: List[str]) -> int:
             for name in orphans:
                 target = store.skill_store_dir(name)
                 if target.is_dir():
-                    shutil.rmtree(target)
+                    util.rmtree(target)
                 journal.log("prune", name)
                 pruned.append(name)
     left = [n for n in orphans if n not in pruned]
@@ -572,10 +565,10 @@ def _bundle_dump(file: Optional[str]) -> int:
         return 0
     dest = paths.expand(file)
     try:
-        dest.write_text(text)
+        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"),
@@ -596,9 +589,9 @@ def _bundle_install(file: Optional[str]) -> int:
                             hint="point at the Boostfile itself, "
                                  "or use `boost import` for skill directories")
         try:
-            text, label = path.read_text(), str(path)
+            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())
@@ -880,7 +873,7 @@ def _snapshot_save(label: Optional[str]) -> int:
     manifest = {"id": snap_id, "label": label or "", "created": util.now_iso(),
                 "skills": skill_count}
     (paths.snapshots_dir() / (snap_id + ".json")).write_text(
-        json.dumps(manifest, indent=2) + "\n")
+        json.dumps(manifest, indent=2) + "\n", encoding="utf-8")
     journal.log("snapshot", snap_id, label=label)
     out.ok("saved %s (%s, %s)" % (snap_id, _plural(skill_count, "skill"),
                                   util.human_size(tar_path.stat().st_size)))
@@ -900,7 +893,7 @@ def _snapshot_list(as_json: bool) -> int:
         meta = {}
         if side.exists():
             try:
-                meta = json.loads(side.read_text())
+                meta = json.loads(side.read_text(encoding="utf-8"))
             except (json.JSONDecodeError, OSError):
                 meta = {}
         snaps.append({"id": snap_id, "created": meta.get("created", ""),
@@ -936,21 +929,21 @@ 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():
-                shutil.rmtree(child)
+                util.rmtree(child)
             else:
                 child.unlink()
         try:
@@ -1006,7 +999,7 @@ def cmd_export(argv: List[str]) -> int:
                 for name in chosen:
                     sdir = store.skill_store_dir(name)
                     for f in sorted(p for p in sdir.rglob("*") if p.is_file()):
-                        zf.write(str(f), arcname="%s/%s" % (name, f.relative_to(sdir)))
+                        zf.write(str(f), arcname="%s/%s" % (name, f.relative_to(sdir).as_posix()))
         else:
             with tarfile.open(str(dest), "w:gz") as tf:
                 data = manifest.encode()
@@ -1019,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)))
@@ -1070,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: