boost-skill-cli 1.0.32__tar.gz → 1.0.33__tar.gz

{boost_skill_cli-1.0.32 → boost_skill_cli-1.0.33}/PKG-INFO

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

{boost_skill_cli-1.0.32 → boost_skill_cli-1.0.33}/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.32'
-__version_tuple__ = version_tuple = (1, 0, 32)
+__version__ = version = '1.0.33'
+__version_tuple__ = version_tuple = (1, 0, 33)
 
 __commit_id__ = commit_id = None

boost_skill_cli-1.0.33/boost_cli/cliparse.py

@@ -0,0 +1,31 @@
+"""A branded ArgumentParser.
+
+Argparse's default failure output is a raw, unstyled ``usage:`` block with a
+bare ``prog: error: …`` line — nothing like boost's branded top-level dispatch
+(which routes through :func:`output.err` and offers a "did you mean"). This
+subclass routes a parser's own errors through the same Aurora output layer, so
+a wrong invocation looks as designed as a right one.
+
+Adopt it via :func:`parser` in place of ``argparse.ArgumentParser``. Sub-parsers
+created with ``add_subparsers()`` inherit this class automatically, so a single
+top-level swap brands a command's whole tree.
+"""
+from __future__ import annotations
+
+import argparse
+import sys
+
+from .core import output as out
+
+
+class BoostArgumentParser(argparse.ArgumentParser):
+    def error(self, message: str):  # noqa: D401 - argparse hook
+        """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))
+        self.exit(2)
+
+
+def parser(*args, **kwargs) -> BoostArgumentParser:
+    """Drop-in replacement for ``argparse.ArgumentParser(...)``."""
+    return BoostArgumentParser(*args, **kwargs)

{boost_skill_cli-1.0.32 → boost_skill_cli-1.0.33}/boost_cli/commands/pkg.py

@@ -2,7 +2,6 @@
 reinstall, bundle, import, migrate, pin, unpin, snapshot, export."""
 from __future__ import annotations
 
-import argparse
 import io
 import json
 import os
@@ -15,6 +14,7 @@ from datetime import datetime, timezone
 from pathlib import Path
 from typing import Dict, List, Optional
 
+from .. import cliparse
 from ..core import (agents, catalog, gitutil, journal, lockfile, paths,
                     registry, store, util)
 from ..core import output as out
@@ -83,7 +83,7 @@ def _boostfile_text(skills: Dict[str, dict], via: str = "boost bundle dump") ->
 # ── install ──────────────────────────────────────────────────────────────
 
 def cmd_install(argv: List[str]) -> int:
-    ap = argparse.ArgumentParser(prog="boost install",
+    ap = cliparse.parser(prog="boost install",
                                  description="Install a skill from a tap registry")
     ap.add_argument("names", nargs="+", metavar="NAME",
                     help="skill name, optionally qualified as tap:skill")
@@ -155,7 +155,7 @@ def cmd_install(argv: List[str]) -> int:
 # ── uninstall ────────────────────────────────────────────────────────────
 
 def cmd_uninstall(argv: List[str]) -> int:
-    ap = argparse.ArgumentParser(
+    ap = cliparse.parser(
         prog="boost uninstall",
         description="Remove an installed skill, rule, workflow, or config")
     ap.add_argument("names", nargs="+", metavar="NAME")
@@ -194,7 +194,7 @@ _PLAN_LABELS = [
 
 
 def cmd_sync(argv: List[str]) -> int:
-    ap = argparse.ArgumentParser(
+    ap = cliparse.parser(
         prog="boost sync",
         description="Reconcile installed skills & symlinks against the lock file")
     ap.add_argument("--diff", action="store_true",
@@ -257,7 +257,7 @@ def cmd_sync(argv: List[str]) -> int:
 # ── update ───────────────────────────────────────────────────────────────
 
 def cmd_update(argv: List[str]) -> int:
-    ap = argparse.ArgumentParser(prog="boost update",
+    ap = cliparse.parser(prog="boost update",
                                  description="Sync taps or update installed skills")
     ap.add_argument("tap", nargs="?", metavar="TAP", help="refresh only this tap")
     ap.add_argument("--taps-only", action="store_true",
@@ -317,7 +317,7 @@ def cmd_update(argv: List[str]) -> int:
 # ── reinstall ────────────────────────────────────────────────────────────
 
 def cmd_reinstall(argv: List[str]) -> int:
-    ap = argparse.ArgumentParser(prog="boost reinstall",
+    ap = cliparse.parser(prog="boost reinstall",
                                  description="Reinstall a skill or all skills (force)")
     ap.add_argument("names", nargs="*", metavar="NAME")
     ap.add_argument("--all", action="store_true",
@@ -369,7 +369,7 @@ def cmd_reinstall(argv: List[str]) -> int:
 # ── bundle ───────────────────────────────────────────────────────────────
 
 def cmd_bundle(argv: List[str]) -> int:
-    ap = argparse.ArgumentParser(prog="boost bundle",
+    ap = cliparse.parser(prog="boost bundle",
                                  description="Export/install skill sets via a Boostfile")
     ap.add_argument("action", choices=("dump", "install"))
     ap.add_argument("file", nargs="?", metavar="FILE",
@@ -481,7 +481,7 @@ def _bundle_install(file: Optional[str]) -> int:
 # ── import ───────────────────────────────────────────────────────────────
 
 def cmd_import(argv: List[str]) -> int:
-    ap = argparse.ArgumentParser(
+    ap = cliparse.parser(
         prog="boost import",
         description="Import skills from a GitHub URL or local path")
     ap.add_argument("source", metavar="URL_OR_PATH")
@@ -552,7 +552,7 @@ def _import_root(root: Path, name: Optional[str], do_all: bool,
 # ── migrate ──────────────────────────────────────────────────────────────
 
 def cmd_migrate(argv: List[str]) -> int:
-    ap = argparse.ArgumentParser(
+    ap = cliparse.parser(
         prog="boost migrate",
         description="Migrate skills between agents or from Skills CLI")
     ap.add_argument("--from", dest="src", metavar="AGENT",
@@ -625,7 +625,7 @@ def cmd_migrate(argv: List[str]) -> int:
 # ── pin / unpin ──────────────────────────────────────────────────────────
 
 def cmd_pin(argv: List[str]) -> int:
-    ap = argparse.ArgumentParser(prog="boost pin",
+    ap = cliparse.parser(prog="boost pin",
                                  description="Pin a skill to its current version")
     ap.add_argument("name", metavar="NAME")
     args = ap.parse_args(argv)
@@ -633,7 +633,7 @@ def cmd_pin(argv: List[str]) -> int:
 
 
 def cmd_unpin(argv: List[str]) -> int:
-    ap = argparse.ArgumentParser(prog="boost unpin",
+    ap = cliparse.parser(prog="boost unpin",
                                  description="Allow a pinned skill to update again")
     ap.add_argument("name", metavar="NAME")
     args = ap.parse_args(argv)
@@ -663,7 +663,7 @@ def _set_pin(name: str, pinned: bool) -> int:
 # ── snapshot ─────────────────────────────────────────────────────────────
 
 def cmd_snapshot(argv: List[str]) -> int:
-    ap = argparse.ArgumentParser(prog="boost snapshot",
+    ap = cliparse.parser(prog="boost snapshot",
                                  description="Save & restore whole skill environments")
     ap.add_argument("action", choices=("save", "list", "restore"))
     ap.add_argument("arg", nargs="?", metavar="LABEL|ID",
@@ -784,7 +784,7 @@ def _snapshot_restore(snap_id: str) -> int:
 # ── export ───────────────────────────────────────────────────────────────
 
 def cmd_export(argv: List[str]) -> int:
-    ap = argparse.ArgumentParser(
+    ap = cliparse.parser(
         prog="boost export",
         description="Package skills as shareable zip/tar archives")
     ap.add_argument("names", nargs="*", metavar="NAME",

{boost_skill_cli-1.0.32 → boost_skill_cli-1.0.33}/boost_skill_cli.egg-info/PKG-INFO

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

{boost_skill_cli-1.0.32 → boost_skill_cli-1.0.33}/boost_skill_cli.egg-info/SOURCES.txt

@@ -13,6 +13,7 @@ boost_cli/__init__.py
 boost_cli/__main__.py
 boost_cli/_version.py
 boost_cli/cli.py
+boost_cli/cliparse.py
 boost_cli/errors.py
 boost_cli/commands/__init__.py
 boost_cli/commands/bmad.py
@@ -84,6 +85,7 @@ tests/unit/test_agents.py
 tests/unit/test_ai.py
 tests/unit/test_catalog.py
 tests/unit/test_claude_settings.py
+tests/unit/test_cliparse.py
 tests/unit/test_config.py
 tests/unit/test_dense.py
 tests/unit/test_embed.py

{boost_skill_cli-1.0.32 → boost_skill_cli-1.0.33}/docs/design-roadmap.html

@@ -436,11 +436,11 @@
         </article>
 
         <article class="ritem" data-track="commands" data-impact="med">
-          <div class="ritem-top"><span class="rid">BOOST-D25</span><span class="rstatus proposed">Proposed</span></div>
+          <div class="ritem-top"><span class="rid">BOOST-D25</span><span class="rstatus progress">In progress</span></div>
           <h4>Branded usage &amp; error output</h4>
           <p>Grading confirmed this hits <b>every command with a required argument</b> — <code>boost uninstall</code>, <code>boost snapshot</code> and more all dump raw, unstyled <code>argparse</code>: a multi-line usage block with a bare <code>error:</code>, nothing like the branded top-level dispatch (which already routes through <code>Error:</code> + a difflib "did you mean"). The durable fix is a shared <code>BoostArgumentParser</code> subclass whose <code>error()</code>/usage route through the Aurora output layer, adopted across the ~50 per-command parsers — a broad, test-first sweep, so logged rather than rushed.</p>
           <div class="rmeta"><span class="m-cat">ux</span><span>Complexity L</span><span class="m-imp med">Impact Med</span><span class="m-wow">★★★☆☆</span></div>
-          <div class="rref"><code>core · new BoostArgumentParser · commands/*.py</code></div>
+          <div class="rref"><code>cliparse.py (BoostArgumentParser) · pkg.py done · 9 files to go</code></div>
         </article>
 
       </div>

boost_skill_cli-1.0.33/tests/unit/test_cliparse.py

@@ -0,0 +1,43 @@
+"""Unit tests: boost_cli/cliparse.BoostArgumentParser — branded errors."""
+from __future__ import annotations
+
+import pytest
+
+from boost_cli import cliparse
+
+
+@pytest.fixture(autouse=True)
+def plain(monkeypatch):
+    monkeypatch.setenv("NO_COLOR", "1")
+
+
+def test_parser_returns_boost_subclass():
+    p = cliparse.parser(prog="boost demo")
+    assert isinstance(p, cliparse.BoostArgumentParser)
+
+
+def test_error_is_branded_and_exits_2(capsys):
+    p = cliparse.parser(prog="boost demo")
+    p.add_argument("name")
+    with pytest.raises(SystemExit) as exc:
+        p.parse_args([])            # missing required positional
+    assert exc.value.code == 2
+    err = capsys.readouterr().err
+    assert err.startswith("Error: ")           # branded, not bare "usage:"
+    assert "the following arguments are required" in err
+    assert "usage: boost demo" in err          # usage still shown, below
+
+
+def test_valid_args_do_not_error(capsys):
+    p = cliparse.parser(prog="boost demo")
+    p.add_argument("--flag", action="store_true")
+    ns = p.parse_args(["--flag"])
+    assert ns.flag is True
+    assert capsys.readouterr().err == ""
+
+
+def test_subparsers_inherit_branding():
+    p = cliparse.parser(prog="boost demo")
+    sub = p.add_subparsers()
+    child = sub.add_parser("go")
+    assert isinstance(child, cliparse.BoostArgumentParser)