boost-skill-cli 1.0.178__tar.gz → 1.0.179__tar.gz

{boost_skill_cli-1.0.178 → boost_skill_cli-1.0.179}/PKG-INFO

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

{boost_skill_cli-1.0.178 → boost_skill_cli-1.0.179}/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.178'
-__version_tuple__ = version_tuple = (1, 0, 178)
+__version__ = version = '1.0.179'
+__version_tuple__ = version_tuple = (1, 0, 179)
 
 __commit_id__ = commit_id = None

{boost_skill_cli-1.0.178 → boost_skill_cli-1.0.179}/boost_cli/core/output.py

@@ -419,6 +419,11 @@ def table(rows, headers=None) -> None:
     numeric columns are right-aligned, and when a row would overflow the
     terminal the widest text column is shrunk (its cells clipped with an
     ellipsis) so wide catalogs stay on one line instead of wrapping.
+
+    On a color terminal, columns are joined by a dim ``│`` separator — the
+    terminal cousin of the web stat blocks' hairline borders. Non-color output
+    (pipes, NO_COLOR, tests) keeps the plain two-space gutter byte-for-byte,
+    so scripts that parse table output never see the ornament.
     """
     rows = [[str(x) for x in r] for r in rows]
     all_rows = ([list(map(str, headers))] if headers else []) + rows
@@ -429,17 +434,23 @@ def table(rows, headers=None) -> None:
               for i in range(ncols)]
     numeric = [_numeric_col([r[i] for r in rows if i < len(r)])
                for i in range(ncols)]
-    widths = _fit_widths(widths, numeric, term_width())
+    if use_color():
+        sep, sep_w = " " + DIM + "│" + RESET + " ", 3
+    else:
+        sep, sep_w = "  ", 2
+    widths = _fit_widths(widths, numeric, term_width(), sep=sep_w)
 
     def fmt(cell: str, i: int) -> str:
         cell = _clip_visible(cell, widths[i])
         return _rpad(cell, widths[i]) if numeric[i] else _pad(cell, widths[i])
 
     if headers:
-        line = "  ".join(fmt(str(h), i) for i, h in enumerate(headers))
-        print(c(line.rstrip(), BOLD))
+        # Bold each header cell individually: a whole-line wrap would be
+        # cancelled at the first separator's RESET on color terminals.
+        cells = [c(fmt(str(h), i), BOLD) for i, h in enumerate(headers)]
+        print(sep.join(cells).rstrip())
     for r in rows:
-        print("  ".join(fmt(cell, i) for i, cell in enumerate(r)).rstrip())
+        print(sep.join(fmt(cell, i) for i, cell in enumerate(r)).rstrip())
 
 
 def confirm(prompt: str, default: bool = False) -> bool:

{boost_skill_cli-1.0.178 → boost_skill_cli-1.0.179}/boost_skill_cli.egg-info/PKG-INFO

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

{boost_skill_cli-1.0.178 → boost_skill_cli-1.0.179}/docs/design-roadmap.html

@@ -305,11 +305,11 @@
 </article>
 
 <article class="ritem" data-track="layout" data-impact="med">
-  <div class="ritem-top"><span class="rid">BOOST-D08</span><span class="rstatus progress">In progress</span></div>
+  <div class="ritem-top"><span class="rid">BOOST-D08</span><span class="rstatus done">Shipped</span></div>
   <h4>Width-aware, right-aligned tables</h4>
   <p><code>table()</code> is a naive <code>ljust</code> that ignores terminal width and left-aligns numeric columns. Make it width-aware (shrink the widest text column to fit), right-align counts with tabular figures, and add dim column separators — matching the web's <code>font-variant-numeric: tabular-nums</code> stat blocks.</p>
   <div class="rmeta"><span class="m-cat">layout</span><span>Complexity M</span><span class="m-imp med">Impact Med</span><span class="m-wow">★★★☆☆</span></div>
-  <div class="rref"><code>core/output.py:71–83</code></div>
+  <div class="rref"><code>core/output.py · table()</code></div>
 </article>
 
 <article class="ritem" data-track="layout" data-impact="med">

{boost_skill_cli-1.0.178 → boost_skill_cli-1.0.179}/docs/roadmap/items/BOOST-D08.md

@@ -2,15 +2,15 @@
 id: BOOST-D08
 board: design
 track: layout
-status: progress
+status: done
 impact: med
 complexity: M
 wow: 3
 category: layout
-ref: "core/output.py:71–83"
+ref: "core/output.py · table()"
 order: 4
-owner:
-pr:
+owner: loop/d08-tables
+pr: 202
 title: Width-aware, right-aligned tables
 ---
 <code>table()</code> is a naive <code>ljust</code> that ignores terminal width and left-aligns numeric columns. Make it width-aware (shrink the widest text column to fit), right-align counts with tabular figures, and add dim column separators — matching the web's <code>font-variant-numeric: tabular-nums</code> stat blocks.

{boost_skill_cli-1.0.178 → boost_skill_cli-1.0.179}/tests/unit/test_output.py

@@ -602,16 +602,55 @@ class TestTableColor:
         import re
         monkeypatch.setenv("CLICOLOR_FORCE", "1")
         # a colored cell must not push the next column out of alignment
+        # (color mode joins columns with the dim │ separator)
         output.table([[output.aurora("ab", "cyan"), "x"], ["cd", "y"]])
         vis = [re.sub(r"\x1b\[[0-9;]*m", "", ln)
                for ln in capsys.readouterr().out.splitlines()]
-        assert vis == ["ab  x", "cd  y"]
+        assert vis == ["ab │ x", "cd │ y"]
 
     def test_header_row_is_bold_when_forced(self, capsys, monkeypatch):
         monkeypatch.setenv("CLICOLOR_FORCE", "1")
         output.table([["x", "1"]], headers=["NAME", "N"])
         assert capsys.readouterr().out.startswith("\033[1mNAME")
 
+    def test_every_header_cell_is_bold(self, capsys, monkeypatch):
+        # A whole-line BOLD wrap would be cancelled by the separator's RESET;
+        # each header cell must carry its own bold instead.
+        monkeypatch.setenv("CLICOLOR_FORCE", "1")
+        output.table([["x", "y"]], headers=["AA", "BB"])
+        header = capsys.readouterr().out.splitlines()[0]
+        assert header.count(output.BOLD) == 2
+        assert "\033[1mAA" in header and "\033[1mBB" in header
+
+    def test_separator_is_dim_pipe_in_color_mode(self, capsys, monkeypatch):
+        monkeypatch.setenv("CLICOLOR_FORCE", "1")
+        output.table([("a", "b")])
+        out = capsys.readouterr().out
+        assert " " + output.DIM + "│" + output.RESET + " " in out
+
+    def test_no_separator_glyph_when_color_off(self, capsys, monkeypatch):
+        monkeypatch.setenv("NO_COLOR", "1")
+        monkeypatch.delenv("CLICOLOR_FORCE", raising=False)
+        output.table([("a", "b"), ("cc", "d")], headers=["X", "Y"])
+        out = capsys.readouterr().out
+        assert "│" not in out                       # plain two-space gutter,
+        assert out == "X   Y\na   b\ncc  d\n"       # byte-identical to before
+
+    def test_separator_width_counts_in_fit_budget(self, capsys, monkeypatch):
+        import os as _os
+        import re
+        # 3 columns of visible width 4 + two 3-wide separators = 18 > 17,
+        # so exactly one text column must shrink; with the old 2-wide gutter
+        # (total 16) nothing would shrink. Proves sep=3 reaches _fit_widths.
+        monkeypatch.setenv("CLICOLOR_FORCE", "1")
+        monkeypatch.setattr(output.shutil, "get_terminal_size",
+                            lambda fb: _os.terminal_size((17, 24)))
+        output.table([("aaaa", "bbbb", "cccc")])
+        vis = re.sub(r"\x1b\[[0-9;]*m", "",
+                     capsys.readouterr().out.splitlines()[0])
+        assert len(vis.rstrip()) <= 17
+        assert "…" in vis                            # a cell was clipped
+
 
 class TestTable:
     @pytest.fixture(autouse=True)