boost-skill-cli 1.0.74__tar.gz → 1.0.76__tar.gz

{boost_skill_cli-1.0.74 → boost_skill_cli-1.0.76}/PKG-INFO

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

{boost_skill_cli-1.0.74 → boost_skill_cli-1.0.76}/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.74'
-__version_tuple__ = version_tuple = (1, 0, 74)
+__version__ = version = '1.0.76'
+__version_tuple__ = version_tuple = (1, 0, 76)
 
 __commit_id__ = commit_id = None

{boost_skill_cli-1.0.74 → boost_skill_cli-1.0.76}/boost_cli/commands/configuration.py

@@ -25,15 +25,7 @@ from ..core import output as out
 from ..errors import BoostError
 
 
-def _tilde(p) -> str:
-    """Contract $HOME to ~ in a path-ish string for display."""
-    s = str(p)
-    for h in {str(paths.home()), str(paths.home().resolve())}:
-        if s == h:
-            return "~"
-        if s.startswith(h + os.sep):
-            return "~" + s[len(h):]
-    return s
+_tilde = paths.tilde
 
 
 def _user() -> str:

{boost_skill_cli-1.0.74 → boost_skill_cli-1.0.76}/boost_cli/commands/discovery.py

@@ -41,13 +41,7 @@ def _positive_int(s: str) -> int:
     return v
 
 
-def _tilde(p) -> str:
-    """Show a path with $HOME contracted to ~."""
-    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 _read_text(p: Path) -> str:

{boost_skill_cli-1.0.74 → boost_skill_cli-1.0.76}/boost_cli/commands/info.py

@@ -23,14 +23,7 @@ from ..errors import BoostError
 
 # ---------------------------------------------------------------- helpers
 
-def _tilde(p) -> str:
-    """Show a path with $HOME contracted to ~."""
-    s, h = str(p), str(paths.home())
-    if s == h:
-        return "~"
-    if s.startswith(h + os.sep):
-        return "~" + s[len(h):]
-    return s
+_tilde = paths.tilde
 
 
 def _read(p: Path) -> str:

{boost_skill_cli-1.0.74 → boost_skill_cli-1.0.76}/boost_cli/commands/intelligence.py

@@ -35,12 +35,7 @@ def _note_fallback() -> None:
         _warned_fallback = True
 
 
-def _tilde(p) -> str:
-    s = str(p)
-    for h in {str(paths.home()), str(paths.home().resolve())}:
-        if s.startswith(h):
-            return "~" + s[len(h):]
-    return s
+_tilde = paths.tilde
 
 
 def _skill_text(name: str) -> Tuple[str, str]:

{boost_skill_cli-1.0.74 → boost_skill_cli-1.0.76}/boost_cli/commands/quality.py

@@ -73,9 +73,7 @@ _STACK_MARKERS = [
 
 # --- shared helpers ---------------------------------------------------------
 
-def _tilde(p) -> str:
-    s, h = str(p), str(paths.home())
-    return "~" + s[len(h):] if s.startswith(h) else s
+_tilde = paths.tilde
 
 
 def _s(n: int) -> str:

{boost_skill_cli-1.0.74 → boost_skill_cli-1.0.76}/boost_cli/commands/taps.py

@@ -3,7 +3,6 @@ from __future__ import annotations
 
 import argparse
 import json
-import os
 
 from .. import cliparse, spin
 from ..core import catalog, config, gitutil, journal, lockfile, paths, registry, store, util
@@ -11,15 +10,7 @@ from ..core import output as out
 from ..errors import BoostError
 
 
-def _tilde(p) -> str:
-    """Contract $HOME to ~ in a path-ish string for display."""
-    s = str(p)
-    for h in {str(paths.home()), str(paths.home().resolve())}:
-        if s == h:
-            return "~"
-        if s.startswith(h + os.sep):
-            return "~" + s[len(h):]
-    return s
+_tilde = paths.tilde
 
 
 def _tap_catalog(args) -> int:

{boost_skill_cli-1.0.74 → boost_skill_cli-1.0.76}/boost_cli/commands/team.py

@@ -4,7 +4,6 @@ from __future__ import annotations
 import getpass
 import hashlib
 import json
-import os
 import platform
 import shutil
 import stat
@@ -16,15 +15,7 @@ from ..core import output as out
 from ..errors import BoostError
 
 
-def _tilde(p) -> str:
-    """Contract $HOME to ~ in a path-ish string for display."""
-    s = str(p)
-    for h in {str(paths.home()), str(paths.home().resolve())}:
-        if s == h:
-            return "~"
-        if s.startswith(h + os.sep):
-            return "~" + s[len(h):]
-    return s
+_tilde = paths.tilde
 
 
 def _user() -> str:

{boost_skill_cli-1.0.74 → boost_skill_cli-1.0.76}/boost_cli/core/paths.py

@@ -32,6 +32,23 @@ def expand(p: str) -> Path:
     return Path(p)
 
 
+def tilde(p) -> str:
+    """Contract $HOME to ``~`` in a path-ish string for display.
+
+    Only contracts a real path boundary: the string must equal home() or sit
+    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.
+    """
+    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
+
+
 def boost_home() -> Path:
     override = os.environ.get("BOOST_HOME")
     return Path(override) if override else home() / ".boost"

{boost_skill_cli-1.0.74 → boost_skill_cli-1.0.76}/boost_skill_cli.egg-info/PKG-INFO

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

{boost_skill_cli-1.0.74 → boost_skill_cli-1.0.76}/docs/roadmap/items/unify-tilde-two-copies-have-a-boundary-bug.md

@@ -2,15 +2,15 @@
 id: unify-tilde-two-copies-have-a-boundary-bug
 board: code
 section: internals
-status: planned
+status: shipped
 category: Correctness
 complexity: S
 impact: Med
 wow: 2
 note: 
 order: 9
-owner:
-pr:
+owner: loop/unify-tilde
+pr: 86
 title: Unify <code>_tilde()</code> — two copies have a boundary bug
 ---
 Eight command modules each define <code>_tilde</code>; the <code>quality</code> and <code>intelligence</code> versions use <code>startswith(home)</code> with no separator check (<code>quality.py:78</code>), so <code>/Users/bob-backup</code> wrongly contracts to <code>~-backup</code>. Collapse to one helper in <code>core/paths</code>.

{boost_skill_cli-1.0.74 → boost_skill_cli-1.0.76}/docs/roadmap.html

@@ -476,7 +476,7 @@
       </article>
 
       <article class="cap rcard">
-        <div class="head"><span class="pill planned">Planned</span><span class="cat">Correctness</span></div>
+        <div class="head"><span class="pill shipped">Shipped</span><span class="cat">Correctness</span></div>
         <h3>Unify <code>_tilde()</code> — two copies have a boundary bug</h3>
         <p>Eight command modules each define <code>_tilde</code>; the <code>quality</code> and <code>intelligence</code> versions use <code>startswith(home)</code> with no separator check (<code>quality.py:78</code>), so <code>/Users/bob-backup</code> wrongly contracts to <code>~-backup</code>. Collapse to one helper in <code>core/paths</code>.</p>
         <div class="meta">

{boost_skill_cli-1.0.74 → boost_skill_cli-1.0.76}/tests/unit/test_paths.py

@@ -1,6 +1,7 @@
 """Unit tests: boost_cli/core/paths.py — all locations derive from $HOME."""
 from __future__ import annotations
 
+import os
 from pathlib import Path
 
 from boost_cli.core import paths
@@ -81,6 +82,40 @@ class TestExpand:
         assert paths.expand("~other") == Path("~other")
 
 
+class TestTilde:
+    def test_exact_home_is_tilde(self, sandbox):
+        assert paths.tilde(str(sandbox)) == "~"
+
+    def test_path_under_home_contracts(self, sandbox):
+        p = sandbox / "x" / "y"
+        assert paths.tilde(str(p)) == "~" + os.sep + "x" + os.sep + "y"
+
+    def test_sibling_prefix_not_contracted(self, sandbox):
+        # regression: /Users/bob-backup must NOT contract to ~-backup when
+        # home is /Users/bob. A separator boundary is required, not a bare
+        # startswith(home).
+        sibling = str(sandbox) + "-backup"
+        assert paths.tilde(sibling) == sibling
+
+    def test_path_outside_home_unchanged(self, sandbox):
+        assert paths.tilde("/etc/hosts") == "/etc/hosts"
+
+    def test_accepts_path_object(self, sandbox):
+        assert paths.tilde(sandbox / "cache") == "~" + os.sep + "cache"
+
+    def test_resolved_home_contracts_through_symlink(self, tmp_path, monkeypatch):
+        # HOME is a symlink; a path expressed against the *resolved* home must
+        # still contract (the helper tries both raw and resolved home).
+        real = tmp_path / "real_home"
+        real.mkdir()
+        link = tmp_path / "link_home"
+        link.symlink_to(real)
+        monkeypatch.setenv("HOME", str(link))
+        monkeypatch.delenv("BOOST_HOME", raising=False)
+        resolved = str(paths.home().resolve())
+        assert paths.tilde(resolved + os.sep + "sub") == "~" + os.sep + "sub"
+
+
 class TestEnsureDirs:
     EXPECTED = ("repos", "cache", "logs", "state", "state/snapshots",
                 "state/lock-history", "state/profiles")