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

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

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: boost-skill-cli
-Version: 1.0.152
+Version: 1.0.153
 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.153}/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.153}/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.153'
+__version_tuple__ = version_tuple = (1, 0, 153)
 
 __commit_id__ = commit_id = None

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

@@ -224,6 +224,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.

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

@@ -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.153}/boost_cli/commands/configuration.py

@@ -152,7 +152,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 +224,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 +426,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)
 
@@ -599,7 +599,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 +641,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)],

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

@@ -252,7 +252,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,7 +325,7 @@ 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`")
@@ -979,7 +979,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.153}/boost_cli/commands/intelligence.py

@@ -60,14 +60,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 +75,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 +88,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 +649,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

{boost_skill_cli-1.0.152 → boost_skill_cli-1.0.153}/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,7 +565,7 @@ 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")
@@ -596,7 +589,7 @@ 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))
     taps_added = installed_n = present = failed = 0
@@ -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", ""),
@@ -950,7 +943,7 @@ def _snapshot_restore(snap_id: str) -> int:
                                  "see `boost snapshot list` for other snapshots")
         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()

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

@@ -269,7 +269,7 @@ def cmd_doctor(argv):
     lp = paths.lockfile_path()
     if lp.exists():
         try:
-            raw = json.loads(lp.read_text())
+            raw = json.loads(lp.read_text(encoding="utf-8"))
             if raw.get("version") != lockfile.SCHEMA_VERSION:
                 bad("lock file schema is v%s, expected v%d"
                     % (raw.get("version"), lockfile.SCHEMA_VERSION))
@@ -499,7 +499,7 @@ def cmd_audit(argv):
                 text = f.read_text(encoding="utf-8", errors="replace")
             except OSError:
                 continue
-            rel = str(f.relative_to(sdir))
+            rel = f.relative_to(sdir).as_posix()
             for pat, severity, label in _AUDIT_PATTERNS:
                 for m in pat.finditer(text):
                     line_no = text.count("\n", 0, m.start()) + 1

{boost_skill_cli-1.0.152 → boost_skill_cli-1.0.153}/boost_cli/commands/taps.py

@@ -157,7 +157,7 @@ def _tap_updated(tap: "registry.Tap") -> str:
         except BoostError:
             pass
     try:
-        data = json.loads(tap.cache_file.read_text())
+        data = json.loads(tap.cache_file.read_text(encoding="utf-8"))
         return util.rel_time(data.get("generated", ""))
     except (OSError, ValueError):
         return "?"

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

@@ -40,14 +40,14 @@ def _load_cohorts() -> dict:
     if not p.exists():
         return {}
     try:
-        return json.loads(p.read_text())
+        return json.loads(p.read_text(encoding="utf-8"))
     except (json.JSONDecodeError, OSError):
         return {}
 
 
 def _save_cohorts(cohorts: dict) -> None:
     paths.ensure_dirs()
-    _cohorts_path().write_text(json.dumps(cohorts, indent=2) + "\n")
+    _cohorts_path().write_text(json.dumps(cohorts, indent=2) + "\n", encoding="utf-8")
 
 
 def _is_member(user: str, cohort_name: str, percent: int) -> bool:
@@ -181,7 +181,7 @@ def _load_profile(name: str) -> dict:
         raise BoostError("no profile named %s" % name,
                         hint="list profiles with `boost profile list`")
     try:
-        return json.loads(p.read_text())
+        return json.loads(p.read_text(encoding="utf-8"))
     except (json.JSONDecodeError, OSError) as e:
         raise BoostError("profile %s is unreadable: %s" % (name, e))
 
@@ -217,7 +217,7 @@ def cmd_profile(argv) -> int:
         profiles = []
         for f in sorted(paths.profiles_dir().glob("*.json")):
             try:
-                data = json.loads(f.read_text())
+                data = json.loads(f.read_text(encoding="utf-8"))
             except (json.JSONDecodeError, OSError):
                 continue
             profiles.append({"name": data.get("name", f.stem),
@@ -242,7 +242,7 @@ def cmd_profile(argv) -> int:
                                   "version": e.get("version", "0.0.0")}
                               for n, e in installed.items()}}
         paths.ensure_dirs()
-        _profile_path(args.name).write_text(json.dumps(profile, indent=2) + "\n")
+        _profile_path(args.name).write_text(json.dumps(profile, indent=2) + "\n", encoding="utf-8")
         journal.log("profile", args.name, op="save", skills=len(installed))
         out.ok("saved profile %s (%d skills)" % (args.name, len(installed)))
         return 0
@@ -390,7 +390,7 @@ def cmd_protocol(argv) -> int:
         script = _handler_script()
         script.write_text("#!/usr/bin/env bash\n"
                           "# boost:// URL handler — invoked with the URL as $1\n"
-                          'exec "%s" protocol open "$1"\n' % shim)
+                          'exec "%s" protocol open "$1"\n' % shim, encoding="utf-8")
         script.chmod(script.stat().st_mode | stat.S_IEXEC)
         out.ok("wrote handler script %s" % _tilde(script))
         if system == "Darwin":
@@ -404,7 +404,7 @@ def cmd_protocol(argv) -> int:
             desktop.parent.mkdir(parents=True, exist_ok=True)
             desktop.write_text("[Desktop Entry]\nType=Application\nName=boost protocol handler\n"
                                "Exec=%s %%u\nMimeType=x-scheme-handler/boost;\nNoDisplay=true\n"
-                               % script)
+                               % script, encoding="utf-8")
             out.ok("wrote %s" % _tilde(desktop))
             if shutil.which("xdg-mime"):
                 import subprocess

{boost_skill_cli-1.0.152 → boost_skill_cli-1.0.153}/boost_cli/core/catalog.py

@@ -74,8 +74,8 @@ def _make_entry(root: Path, defining_file: Path, kind: str, default_name: str,
         "tap": tap_name,
         "curated": curated,
         "kind": kind,
-        "rel_dir": str(item_dir.relative_to(root)) if item_dir != root else ".",
-        "skill_md": str(defining_file.relative_to(root)),
+        "rel_dir": item_dir.relative_to(root).as_posix() if item_dir != root else ".",
+        "skill_md": defining_file.relative_to(root).as_posix(),
         "meta": meta,
         # Lowercased substring-search blob, flattened once at index time so
         # search() never re-walks the frontmatter per query (see _search_blob).
@@ -166,7 +166,7 @@ def rebuild_tap(tap: "registry.Tap") -> List[dict]:
         "generated": util.now_iso(),
         "commit": gitutil.head_commit(tap.path),
         "skills": entries,
-    }, indent=1))
+    }, indent=1), encoding="utf-8")
     # Drop the mtime-keyed cache so a rebuild is visible to an immediately
     # following load even when the filesystem mtime granularity is coarse.
     _ENTRY_CACHE.pop(str(tap.cache_file), None)
@@ -197,7 +197,7 @@ def _cached_tap(tap: "registry.Tap") -> Optional[List[dict]]:
     if cached is not None and cached[0] == stamp:
         return cached[1]
     try:
-        skills = json.loads(p.read_text()).get("skills", [])
+        skills = json.loads(p.read_text(encoding="utf-8")).get("skills", [])
     except (json.JSONDecodeError, OSError):
         _ENTRY_CACHE.pop(key, None)
         return None

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

@@ -59,7 +59,7 @@ def load(scope: str, project_dir: Optional[Path] = None) -> dict:
     if not p.exists():
         return {}
     try:
-        data = json.loads(p.read_text())
+        data = json.loads(p.read_text(encoding="utf-8"))
     except (json.JSONDecodeError, OSError):
         return {}
     return data if isinstance(data, dict) else {}
@@ -78,9 +78,9 @@ def save(scope: str, data: dict, project_dir: Optional[Path] = None) -> None:
         while dest.exists():
             dest = hist / ("%s-%s-%d.json" % (scope, stamp, n))
             n += 1
-        dest.write_text(p.read_text())
+        dest.write_text(p.read_text(encoding="utf-8"), encoding="utf-8")
         _prune_history()
-    p.write_text(json.dumps(data, indent=2) + "\n")
+    p.write_text(json.dumps(data, indent=2) + "\n", encoding="utf-8")
 
 
 def _prune_history() -> None:

{boost_skill_cli-1.0.152 → boost_skill_cli-1.0.153}/boost_cli/core/config.py

@@ -61,7 +61,7 @@ REGISTRY_CATALOG = paths.package_root() / "data" / "registries.json"
 def load_registry_catalog() -> list:
     """The bundled curated registries, or [] if the data file is missing."""
     try:
-        data = json.loads(REGISTRY_CATALOG.read_text())
+        data = json.loads(REGISTRY_CATALOG.read_text(encoding="utf-8"))
     except (OSError, json.JSONDecodeError):
         return []
     return data.get("registries", [])
@@ -105,7 +105,7 @@ def _read() -> dict:
     if not p.exists():
         return deepcopy(DEFAULTS)
     try:
-        user = json.loads(p.read_text())
+        user = json.loads(p.read_text(encoding="utf-8"))
     except (json.JSONDecodeError, OSError):
         return deepcopy(DEFAULTS)
     return _merge(DEFAULTS, user)

{boost_skill_cli-1.0.152 → boost_skill_cli-1.0.153}/boost_cli/core/dense.py

@@ -230,10 +230,18 @@ def retrieve(query: str, k: int = 60, kind: Optional[str] = None,
         return None
     try:
         pool = max(k * _POOL, 200)
-        knn = con.execute(
-            "SELECT rowid, distance FROM vec_chunks "
-            "WHERE embedding MATCH ? ORDER BY distance LIMIT ?",
-            (mod.serialize_float32(qv[0]), pool)).fetchall()
+        try:
+            knn = con.execute(
+                "SELECT rowid, distance FROM vec_chunks "
+                "WHERE embedding MATCH ? ORDER BY distance LIMIT ?",
+                (mod.serialize_float32(qv[0]), pool)).fetchall()
+        except sqlite3.OperationalError:
+            # Some sqlite3/sqlite-vec combinations (observed on Windows)
+            # reject this exact query shape ("a LIMIT or 'k = ?' constraint
+            # is required") despite the bound LIMIT above — dense retrieval
+            # is genuinely unusable here, so fall back to BM25 rather than
+            # crash the search command.
+            return None
         if not knn:
             return []
         by_id = {r[0]: (r[1], r[2], r[3], r[4]) for r in con.execute(

{boost_skill_cli-1.0.152 → boost_skill_cli-1.0.153}/boost_cli/core/journal.py

@@ -31,7 +31,7 @@ def events(n: Optional[int] = None, action: Optional[str] = None,
     if not p.exists():
         return []
     out = []
-    for line in p.read_text().splitlines():
+    for line in p.read_text(encoding="utf-8").splitlines():
         try:
             e = json.loads(line)
         except json.JSONDecodeError:
@@ -55,7 +55,7 @@ def rotation_healthy() -> bool:
 def _maybe_rotate() -> None:
     p = paths.pulse_path()
     try:
-        lines = p.read_text().splitlines()
+        lines = p.read_text(encoding="utf-8").splitlines()
     except OSError:
         return
     if len(lines) > ROTATE_AT:

{boost_skill_cli-1.0.152 → boost_skill_cli-1.0.153}/boost_cli/core/lockfile.py

@@ -29,7 +29,7 @@ def read() -> dict:
     if not p.exists():
         return _skeleton()          # empty: never installed anything yet
     try:
-        lock = json.loads(p.read_text())
+        lock = json.loads(p.read_text(encoding="utf-8"))
     except (json.JSONDecodeError, OSError):
         # Corrupt (present but unparseable) is NOT the same as empty: returning
         # a bare skeleton here would let the next write() overwrite the only
@@ -169,7 +169,7 @@ def history_list() -> List[dict]:
     out = []
     for p in _history_files():
         try:
-            data = json.loads(p.read_text())
+            data = json.loads(p.read_text(encoding="utf-8"))
         except (json.JSONDecodeError, OSError):
             continue
         out.append({
@@ -187,4 +187,4 @@ def history_read(hist_id: str) -> dict:
         from ..errors import BoostError
         raise BoostError("no lock history entry %s" % hist_id,
                         hint="list entries with `boost replay`")
-    return json.loads(p.read_text())
+    return json.loads(p.read_text(encoding="utf-8"))

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

@@ -25,6 +25,23 @@ MAGENTA = "\033[35m"
 CYAN = "\033[36m"
 
 
+def harden_console_encoding() -> None:
+    """Reconfigure stdout/stderr to UTF-8.
+
+    boost's own output uses non-ASCII (checkmarks, em dashes, arrows); a
+    console whose codepage isn't UTF-8 — chiefly Windows' legacy codepages —
+    raises UnicodeEncodeError on that output otherwise. reconfigure() is
+    missing on some wrapped/piped streams (e.g. under certain test harnesses),
+    so skip a stream that doesn't have it rather than fail startup over it.
+    """
+    for stream in (sys.stdout, sys.stderr):
+        if hasattr(stream, "reconfigure"):
+            try:
+                stream.reconfigure(encoding="utf-8", errors="replace")
+            except (OSError, ValueError):
+                pass
+
+
 def use_color(stream=None) -> bool:
     # BOOST_COLOR is boost's own explicit override and wins over everything
     # else (most-specific-wins): always/never force it on/off regardless of

{boost_skill_cli-1.0.152 → boost_skill_cli-1.0.153}/boost_cli/core/paths.py

@@ -39,14 +39,18 @@ def tilde(p) -> str:
     directly beneath it (``home + os.sep``). A bare ``startswith(home)`` would
     wrongly turn a sibling like ``/Users/bob-backup`` into ``~-backup``. Both
     the raw and resolved forms of home() are tried so symlinked homes contract.
+
+    Always returns ``/``-separated output (even for a path outside $HOME) so
+    boost's display text is stable across platforms — Windows accepts forward
+    slashes natively, so nothing is lost for a Windows reader either.
     """
     s = str(p)
     for h in (str(home()), str(home().resolve())):
         if s == h:
             return "~"
         if s.startswith(h + os.sep):
-            return "~" + s[len(h):]
-    return s
+            return ("~" + s[len(h):]).replace(os.sep, "/")
+    return s.replace(os.sep, "/")
 
 
 def boost_home() -> Path:

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

@@ -27,7 +27,7 @@ def load() -> dict:
     base = dict(DEFAULTS)
     if p.exists():
         try:
-            base.update(json.loads(p.read_text()))
+            base.update(json.loads(p.read_text(encoding="utf-8")))
         except (json.JSONDecodeError, OSError):
             pass
     return base
@@ -36,7 +36,7 @@ def load() -> dict:
 def save(pol: dict) -> None:
     paths.ensure_dirs()
     known = {k: pol.get(k, DEFAULTS[k]) for k in DEFAULTS}
-    paths.policy_path().write_text(json.dumps(known, indent=2) + "\n")
+    paths.policy_path().write_text(json.dumps(known, indent=2) + "\n", encoding="utf-8")
 
 
 def check_install(entry: dict, installed_count: int) -> List[str]:

{boost_skill_cli-1.0.152 → boost_skill_cli-1.0.153}/boost_cli/core/rag.py

@@ -122,7 +122,7 @@ def _tap_commits() -> Dict[str, str]:
     for t in registry.list_taps():
         commit = ""
         try:
-            data = json.loads(t.cache_file.read_text())
+            data = json.loads(t.cache_file.read_text(encoding="utf-8"))
             commit = str(data.get("commit") or "")
         except (OSError, json.JSONDecodeError):
             pass
@@ -288,7 +288,7 @@ def _load_raw() -> Optional[dict]:
     if isinstance(cached, tuple) and cached[0] == mtime:
         return cached[1]  # type: ignore[return-value]
     try:
-        data = json.loads(p.read_text())
+        data = json.loads(p.read_text(encoding="utf-8"))
     except (OSError, json.JSONDecodeError):
         return None
     if not isinstance(data, dict) or data.get("version") != INDEX_VERSION:

{boost_skill_cli-1.0.152 → boost_skill_cli-1.0.153}/boost_cli/core/registry.py

@@ -2,13 +2,12 @@
 from __future__ import annotations
 
 import difflib
-import shutil
 from dataclasses import dataclass
 from pathlib import Path
 from typing import List, Optional
 
 from ..errors import BoostError
-from . import config, gitutil, paths
+from . import config, gitutil, paths, util
 
 
 @dataclass
@@ -40,7 +39,10 @@ def parse_spec(spec: str):
     Accepts: owner/repo | full git URL | existing local path.
     """
     spec = spec.strip().rstrip("/")
-    p = Path(spec).expanduser()
+    # paths.expand(), not Path.expanduser(): expanduser() consults the OS's
+    # own home-dir lookup (USERPROFILE on Windows), which ignores the $HOME
+    # override the whole test suite (and BOOST_HOME sandboxing) relies on.
+    p = paths.expand(spec)
     if p.exists() and p.is_dir():
         return (p.resolve().name, str(p.resolve()))
     if spec.startswith(("http://", "https://", "git@", "ssh://")):
@@ -78,7 +80,7 @@ def add(spec: str, curated: bool = False) -> Tap:
     tap = Tap(name=name, url=url, curated=curated)
     paths.ensure_dirs()
     if tap.path.exists():
-        shutil.rmtree(tap.path)
+        util.rmtree(tap.path)
     gitutil.clone_shallow(url, tap.path)
     cfg = config.load()
     cfg.setdefault("taps", []).append(
@@ -93,7 +95,7 @@ def remove(name: str) -> Tap:
     cfg["taps"] = [t for t in cfg.get("taps", []) if t["name"] != tap.name]
     config.save(cfg)
     if tap.path.exists():
-        shutil.rmtree(tap.path)
+        util.rmtree(tap.path)
     if tap.cache_file.exists():
         tap.cache_file.unlink()
     return tap