boost-skill-cli 1.0.60__tar.gz → 1.0.62__tar.gz

{boost_skill_cli-1.0.60 → boost_skill_cli-1.0.62}/PKG-INFO

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

{boost_skill_cli-1.0.60 → boost_skill_cli-1.0.62}/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.60'
-__version_tuple__ = version_tuple = (1, 0, 60)
+__version__ = version = '1.0.62'
+__version_tuple__ = version_tuple = (1, 0, 62)
 
 __commit_id__ = commit_id = None

{boost_skill_cli-1.0.60 → boost_skill_cli-1.0.62}/boost_skill_cli.egg-info/PKG-INFO

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

{boost_skill_cli-1.0.60 → boost_skill_cli-1.0.62}/docs/roadmap.html

@@ -114,7 +114,7 @@
   <!-- ================= SNAPSHOT ================= -->
   <section class="block reveal">
     <div class="stats">
-      <div class="stat"><b>5</b><span>Shipped</span></div>
+      <div class="stat"><b>6</b><span>Shipped</span></div>
       <div class="stat"><b>3</b><span>Next up</span></div>
       <div class="stat"><b>4</b><span>Planned</span></div>
       <div class="stat"><b>48</b><span>Loop finds</span></div>
@@ -193,6 +193,24 @@
         </div>
       </article>
 
+      <article class="cap rcard">
+        <div class="head"><span class="pill shipped">Shipped</span><span class="cat">Testing · Resilience</span></div>
+        <h3>Crash-recorder error paths — <code>core/logs.py</code></h3>
+        <p>The black-box diagnostic logger swallows every filesystem/handler
+           failure so a broken log can never break the CLI — but those
+           <code>except</code> arms (the parts that matter most <em>during</em> a
+           crash) were the least exercised. New tests force each failure —
+           handler <code>close()</code> raising, a read-only log dir, an
+           unresolvable version, a wedged logger, <code>unlink</code> denied —
+           taking the module to <strong>100%</strong> coverage.</p>
+        <div class="meta">
+          <span class="m">Complexity <b>S</b></span>
+          <span class="m">Impact <b>Med</b></span>
+          <span class="m">Wow <b class="wow">★★★</b></span>
+          <span class="m">7 branches · 88%→100%</span>
+        </div>
+      </article>
+
       <article class="cap rcard">
         <div class="head"><span class="pill shipped">Shipped</span><span class="cat">Infra · DX</span></div>
         <h3>Autonomous ship-workflow &amp; isolated worktree</h3>
@@ -856,6 +874,23 @@
     </div>
     <div class="rgrid">
 
+      <article class="cap rcard">
+        <div class="head"><span class="pill planned">Planned</span><span class="cat">DX · Design system</span></div>
+        <h3>Promote nav / footer into the shared style system</h3>
+        <p>Canonical <code>style/boost.css</code> ships tokens + primitives but
+           <strong>no nav, logo, or footer</strong>, so every page (the guide and
+           both roadmaps) redefines that chrome inline — and the Design Roadmap
+           currently renders its nav <strong>unstyled</strong> because of the gap.
+           Add <code>.site-nav</code>/<code>.logo</code>/<code>footer</code>
+           primitives to the one sheet so the chrome is defined once and can't drift.</p>
+        <div class="meta">
+          <span class="m">Complexity <b>S</b></span>
+          <span class="m">Impact <b>Med</b></span>
+          <span class="m">Wow <b class="wow">★★★</b></span>
+          <span class="m"><code>style/boost.css</code></span>
+        </div>
+      </article>
+
       <article class="cap rcard">
         <div class="head"><span class="pill planned">Planned</span><span class="cat">DX · Infra</span></div>
         <h3>Shift-left gate — <code>pre-commit</code> + pre-commit.ci</h3>
@@ -992,6 +1027,22 @@
     </div>
     <div class="rgrid">
 
+      <article class="cap rcard">
+        <div class="head"><span class="pill next">Next</span><span class="cat">Docs · Discoverability</span></div>
+        <h3>Make the roadmaps discoverable</h3>
+        <p>Neither <code>roadmap.html</code> nor <code>design-roadmap.html</code> is
+           linked from <code>docs/index.html</code> or the README — the living
+           roadmaps are unreachable unless you already know the URL. Add nav entries
+           on the Visual Guide and a "Roadmap" link in the README so the two boards
+           are one click away and the work stays visible.</p>
+        <div class="meta">
+          <span class="m">Complexity <b>S</b></span>
+          <span class="m">Impact <b>Med</b></span>
+          <span class="m">Wow <b class="wow">★★★</b></span>
+          <span class="m">2 nav links</span>
+        </div>
+      </article>
+
       <article class="cap rcard">
         <div class="head"><span class="pill planned">Planned</span><span class="cat">Docs · Perf</span></div>
         <h3>Lighthouse CI on the Pages site</h3>

{boost_skill_cli-1.0.60 → boost_skill_cli-1.0.62}/tests/unit/test_logs.py

@@ -2,6 +2,7 @@
 from __future__ import annotations
 
 import logging
+import pathlib
 
 import pytest
 
@@ -151,6 +152,102 @@ def test_console_level_resolution(env, monkeypatch):
     assert logs._console_level(False, False, False) is None
 
 
+# --- defensive / best-effort error branches --------------------------------
+# logs.py is the black-box recorder: a broken log must never break the CLI, so
+# every filesystem/handler touchpoint swallows its own errors. Those `except`
+# arms are the parts that matter most in a crash, yet are the least exercised —
+# these tests force each failure so the swallow is real, not just declared.
+
+def test_reset_swallows_a_handler_that_fails_to_close(env):
+    logs.configure()
+    logger = logs.get_logger()
+
+    class BadHandler(logging.Handler):
+        def close(self):
+            raise RuntimeError("cannot close")
+
+        def emit(self, record):
+            pass
+
+    bad = BadHandler()
+    logger.addHandler(bad)
+    logs.reset()                       # must not raise despite close() blowing up
+    assert bad not in logger.handlers  # …and still detaches the handler
+    assert not logger.handlers
+
+
+def test_configure_survives_file_handler_oserror(env, monkeypatch):
+    def _boom(*a, **k):
+        raise OSError("read-only filesystem")
+
+    monkeypatch.setattr(logs.logging.handlers, "RotatingFileHandler", _boom)
+    logs.reset()
+    logger = logs.configure()          # the OSError is swallowed, CLI proceeds
+    assert logger is logs.get_logger()
+    # no file handler was attached, and logging still works without raising
+    assert not any(isinstance(h, logging.handlers.RotatingFileHandler)
+                   for h in logger.handlers)
+    logger.info("still-alive")
+
+
+def test_boost_version_falls_back_to_unknown(env, monkeypatch):
+    import boost_cli
+    # simulate a build with no resolvable version (the `from .. import` fails)
+    monkeypatch.delattr(boost_cli, "__version__", raising=False)
+    monkeypatch.setitem(__import__("sys").modules, "boost_cli._version", None)
+    assert logs._boost_version() == "unknown"
+
+
+def test_crash_report_swallows_trail_write_failure(env, monkeypatch):
+    """A broken logger must not stop the on-disk crash file from being written."""
+    real = logs.get_logger()
+
+    class BadLogger:
+        def error(self, *a, **k):
+            raise RuntimeError("logger is wedged")
+
+    monkeypatch.setattr(logs, "get_logger", lambda: BadLogger())
+    report = logs.write_crash_report(ValueError("boom"), ["x"])
+    monkeypatch.setattr(logs, "get_logger", lambda: real)   # restore for teardown
+    assert report is not None and report.exists()
+    assert "boom" in report.read_text()
+
+
+def test_crash_report_returns_none_when_file_write_fails(env, monkeypatch):
+    class BadDir:
+        def mkdir(self, *a, **k):
+            raise OSError("no space left on device")
+
+    monkeypatch.setattr(logs.paths, "logs_dir", lambda: BadDir())
+    assert logs.write_crash_report(ValueError("boom"), ["x"]) is None
+
+
+def test_prune_returns_early_when_glob_fails(env, monkeypatch):
+    class BadDir:
+        def glob(self, pattern):
+            raise OSError("cannot list directory")
+
+    monkeypatch.setattr(logs.paths, "logs_dir", lambda: BadDir())
+    logs._prune_crash_reports()        # must return quietly, not raise
+
+
+def test_prune_swallows_unlink_failure_and_keeps_files(env, monkeypatch):
+    logs.configure()
+    monkeypatch.setattr(logs, "KEEP_CRASH_REPORTS", 1)
+    d = paths.logs_dir()
+    d.mkdir(parents=True, exist_ok=True)
+    for i in range(4):
+        (d / ("crash-t%02d.log" % i)).write_text("x", encoding="utf-8")
+
+    def _locked(self, *a, **k):
+        raise OSError("file is locked")
+
+    monkeypatch.setattr(pathlib.Path, "unlink", _locked)
+    logs._prune_crash_reports()        # unlink raises for each stale file…
+    # …swallowed, so all four files survive (nothing deleted)
+    assert len(sorted(d.glob("crash-*.log"))) == 4
+
+
 # --- CLI integration -------------------------------------------------------
 
 def test_cli_logs_invocation(boost):