scar-cli 0.8.0__tar.gz → 0.9.0__tar.gz

scar_cli-0.9.0/.release-please-manifest.json

@@ -0,0 +1,3 @@
+{
+  ".": "0.9.0"
+}

scar_cli-0.8.0/.scars/candidates/rich-output-nontty-must-bypass-rich.md → scar_cli-0.9.0/.scars/0008-rich-output-nontty-must-bypass-rich.landmine.md

@@ -1,25 +1,22 @@
 ---
-# COPY THIS FILE — do not edit the template itself.
-# New scars: write to .scars/candidates/<slug>.md with status: candidate.
-# A human reviewer promotes to .scars/NNNN-<slug>.<type>.md with status: active.
-id: 0                      # assigned at promotion (next free NNNN)
-type: landmine             # deadend = tried+failed | fence = looks wrong, intentional | landmine = touching A breaks B
+id: 8
+type: landmine
 title: Read commands' non-tty branch must bypass Rich — Rich wraps to 80 cols and breaks path-substring tests
 severity: high
 confidence: 0.9
 created: 2026-06-30
-authors: ["claude-code"]
+authors: ["claude-code", "kibukx"]
 anchors:
   - path: src/scar/cli.py
   - path: src/scar/output.py
   - pattern: "output\.render\("
 evidence:
   - issue: 78
-  - note: "187-test suite asserts plain substrings like '0001-bad.deadend.md' and long anchor paths on main([...]) under capsys"
+  - note: 187-test suite asserts plain substrings like '0001-bad.deadend.md' and long anchor paths on main([...]) under capsys
 expires:
   condition: "the test suite stops asserting on plain stdout substrings (e.g. moves to asserting structured --json only)"
   review_after: 2027-06-30
-status: candidate
+status: active
 ---
 
 The five read commands (status, lint, check, why, orphan) route output 3 ways:

{scar_cli-0.8.0 → scar_cli-0.9.0}/CHANGELOG.md

@@ -1,5 +1,12 @@
 # Changelog
 
+## [0.9.0](https://github.com/Daily-Nerd/Scar/compare/v0.8.0...v0.9.0) (2026-07-01)
+
+
+### Features
+
+* **cli:** Rich-colored --help via rich-argparse ([#81](https://github.com/Daily-Nerd/Scar/issues/81)) ([7983a85](https://github.com/Daily-Nerd/Scar/commit/7983a85a2069a548ffa676b8c07f5fe928f8971c)), closes [#80](https://github.com/Daily-Nerd/Scar/issues/80)
+
 ## [0.8.0](https://github.com/Daily-Nerd/Scar/compare/v0.7.0...v0.8.0) (2026-06-30)
 
 

{scar_cli-0.8.0 → scar_cli-0.9.0}/PKG-INFO

@@ -1,10 +1,11 @@
 Metadata-Version: 2.4
 Name: scar-cli
-Version: 0.8.0
+Version: 0.9.0
 Summary: SCAR — version control for negative knowledge (deadends, fences, landmines)
 License: MIT
 License-File: LICENSE
 Requires-Python: >=3.10
+Requires-Dist: rich-argparse>=1.8.0
 Requires-Dist: rich>=15.0.0
 Description-Content-Type: text/markdown
 

{scar_cli-0.8.0 → scar_cli-0.9.0}/plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "scar",
-  "version": "0.8.0",
+  "version": "0.9.0",
   "description": "SCAR — version control for negative knowledge (deadends, fences, landmines)",
   "hooks": {
     "PreToolUse": [

{scar_cli-0.8.0 → scar_cli-0.9.0}/pyproject.toml

@@ -1,12 +1,13 @@
 [project]
 name = "scar-cli"
-version = "0.8.0"
+version = "0.9.0"
 description = "SCAR — version control for negative knowledge (deadends, fences, landmines)"
 readme = "README.md"
 requires-python = ">=3.10"
 license = { text = "MIT" }
 dependencies = [
     "rich>=15.0.0",
+    "rich-argparse>=1.8.0",
 ]
 
 [project.scripts]

{scar_cli-0.8.0 → scar_cli-0.9.0}/src/scar/cli.py

@@ -14,6 +14,8 @@ import time
 from importlib.metadata import PackageNotFoundError, version as _dist_version
 from pathlib import Path
 
+from rich_argparse import RichHelpFormatter
+
 from .lint import lint_text
 from .match import rank_for_edit, rank_matches_for_diff, rank_matches_for_edit
 from .model import parse_scar_text
@@ -729,45 +731,56 @@ def _scar_version() -> str:
         return "unknown"
 
 
-def main(argv: list[str] | None = None) -> int:
+def build_parser() -> argparse.ArgumentParser:
+    """Construct the fully-wired argparse parser.
+
+    Subparsers do NOT inherit ``formatter_class`` from their parent, so every
+    ``add_parser`` call is routed through ``_add`` to wire RichHelpFormatter in
+    one place instead of repeating it per command.
+    """
+
+    def _add(subparsers, name, **kw):
+        return subparsers.add_parser(name, formatter_class=RichHelpFormatter, **kw)
+
     parser = argparse.ArgumentParser(prog="scar",
-                                     description="version control for negative knowledge")
+                                     description="version control for negative knowledge",
+                                     formatter_class=RichHelpFormatter)
     # argparse's version action prints and exits during optional parsing, before
     # the required-subcommand check below — so `scar --version` needs no command.
     parser.add_argument("--version", action="version", version=f"scar {_scar_version()}")
     sub = parser.add_subparsers(dest="command", required=True)
 
-    sub.add_parser("init", help="create .scars/ layout in the current repo")
-    p = sub.add_parser("lint", help="validate every scar and candidate")
+    _add(sub, "init", help="create .scars/ layout in the current repo")
+    p = _add(sub, "lint", help="validate every scar and candidate")
     p.add_argument("--fail-orphans", action="store_true",
                    help="exit non-zero when any scar is orphan-detected")
     p.add_argument("--json", action="store_true", help="emit machine-readable JSON")
-    p = sub.add_parser("status", help="counts, titles, broken-file warnings")
+    p = _add(sub, "status", help="counts, titles, broken-file warnings")
     p.add_argument("--json", action="store_true", help="emit machine-readable JSON")
 
-    p = sub.add_parser("promote", help="review a candidate into an active scar")
+    p = _add(sub, "promote", help="review a candidate into an active scar")
     p.add_argument("candidate", help="candidate filename (or unique substring)")
     p.add_argument("--reviewer", default="", help="human reviewer to add to authors")
 
-    p = sub.add_parser("check", help="scars anchored to a path")
+    p = _add(sub, "check", help="scars anchored to a path")
     p.add_argument("path")
     p.add_argument("--content", default="", help="new code to test pattern anchors against")
     p.add_argument("--top-k", type=int, default=10)
     p.add_argument("--json", action="store_true", help="emit machine-readable JSON")
 
-    p = sub.add_parser("why", help="history of pain for a path (any status)")
+    p = _add(sub, "why", help="history of pain for a path (any status)")
     p.add_argument("path")
     p.add_argument("--json", action="store_true", help="emit machine-readable JSON")
 
-    p = sub.add_parser("challenge", help="dispute a scar (still fires, marked challenged)")
+    p = _add(sub, "challenge", help="dispute a scar (still fires, marked challenged)")
     p.add_argument("id", type=int)
     p.add_argument("--reason", required=True, help="why the scar may no longer hold")
 
-    p = sub.add_parser("archive", help="retire a scar (never fires; history kept)")
+    p = _add(sub, "archive", help="retire a scar (never fires; history kept)")
     p.add_argument("id", type=int)
     p.add_argument("--reason", required=True, help="why it is retired (e.g. expiry condition met)")
 
-    p = sub.add_parser("orphan", help="list scars whose anchors all went dead")
+    p = _add(sub, "orphan", help="list scars whose anchors all went dead")
     p.add_argument("--apply", action="store_true",
                    help="persist status: orphaned (human review only — never CI)")
     p.add_argument("--id", type=int, default=None,
@@ -777,7 +790,7 @@ def main(argv: list[str] | None = None) -> int:
     p.add_argument("--json", action="store_true",
                    help="emit machine-readable JSON (read mode only)")
 
-    p = sub.add_parser("harvest", help="mine git history for candidate scars")
+    p = _add(sub, "harvest", help="mine git history for candidate scars")
     p.add_argument("repo", nargs="?", default=".")
     p.add_argument("--top-k", type=int, default=None,
                    help="show the N highest-scoring candidates across all sections "
@@ -792,33 +805,38 @@ def main(argv: list[str] | None = None) -> int:
     p.add_argument("--at", default=None,
                    help="with --precision: comma-separated N values (default 5,10,20)")
 
-    p = sub.add_parser("hook", help="install, remove, inspect, or run Claude Code hooks")
+    p = _add(sub, "hook", help="install, remove, inspect, or run Claude Code hooks")
     p.add_argument("kind", choices=["install", "uninstall", "status",
                                     "precheck", "session-notice", "stop-drafter"])
     p.add_argument("--dry-run", action="store_true",
                    help="show lifecycle changes without writing settings")
 
-    p = sub.add_parser("skill", help="install, remove, or inspect the scar-authoring skill")
+    p = _add(sub, "skill", help="install, remove, or inspect the scar-authoring skill")
     p.add_argument("kind", choices=["install", "uninstall", "status"])
     p.add_argument("--dry-run", action="store_true",
                    help="show changes without writing to ~/.claude/skills")
 
-    sub.add_parser("mcp", help="run the SCAR MCP stdio server")
+    _add(sub, "mcp", help="run the SCAR MCP stdio server")
 
-    p = sub.add_parser("agent", help="agent integration helpers")
+    p = _add(sub, "agent", help="agent integration helpers")
     agent_sub = p.add_subparsers(dest="agent_command", required=True)
-    agent_sub.add_parser("doctor", help="show local agent integration readiness")
-    cfg = agent_sub.add_parser("config", help="print config for an agent runtime")
+    _add(agent_sub, "doctor", help="show local agent integration readiness")
+    cfg = _add(agent_sub, "config", help="print config for an agent runtime")
     cfg.add_argument("target", choices=["codex", "cursor", "opencode", "windsurf"])
-    agent_sub.add_parser("skill", help="print the scar-authoring skill body")
+    _add(agent_sub, "skill", help="print the scar-authoring skill body")
 
-    p = sub.add_parser("inject", help="machine mode for hooks: JSON or silence")
+    p = _add(sub, "inject", help="machine mode for hooks: JSON or silence")
     p.add_argument("--path")
     p.add_argument("--content", default="")
     p.add_argument("--diff", help="unified diff text, or path to a diff file")
     p.add_argument("--top-k", type=int, default=3)
     p.add_argument("--hook-event", default="PreToolUse")
 
+    return parser
+
+
+def main(argv: list[str] | None = None) -> int:
+    parser = build_parser()
     args = parser.parse_args(argv)
     if args.command == "mcp":
         from .mcp import serve

{scar_cli-0.8.0 → scar_cli-0.9.0}/tests/test_cli.py

@@ -63,6 +63,49 @@ def test_version_flag_prints_version_and_exits_zero(capsys):
     assert re.search(r"\d+\.\d+", out)
 
 
+def test_root_parser_uses_rich_help_formatter():
+    from rich_argparse import RichHelpFormatter
+
+    from scar.cli import build_parser
+
+    parser = build_parser()
+    assert parser.formatter_class is RichHelpFormatter
+
+
+def _subparsers_choices(parser):
+    import argparse
+
+    for action in parser._actions:
+        if isinstance(action, argparse._SubParsersAction):
+            return action.choices
+    raise AssertionError("no subparsers action found")
+
+
+def test_subparser_uses_rich_help_formatter():
+    from rich_argparse import RichHelpFormatter
+
+    from scar.cli import build_parser
+
+    lint = _subparsers_choices(build_parser())["lint"]
+    assert lint.formatter_class is RichHelpFormatter
+
+
+def test_nested_agent_subparser_uses_rich_help_formatter():
+    from rich_argparse import RichHelpFormatter
+
+    from scar.cli import build_parser
+
+    agent = _subparsers_choices(build_parser())["agent"]
+    config = _subparsers_choices(agent)["config"]
+    assert config.formatter_class is RichHelpFormatter
+
+
+def test_help_still_exits_zero(capsys):
+    with pytest.raises(SystemExit) as exc:
+        main(["--help"])
+    assert exc.value.code == 0
+
+
 def test_status_counts(repo, capsys):
     init_scars(repo)
     (repo / ".scars" / "candidates" / "x.md").write_text(CANDIDATE)

{scar_cli-0.8.0 → scar_cli-0.9.0}/uv.lock

@@ -111,12 +111,25 @@ wheels = [
     { url = "https://files.pythonhosted.org/packages/82/3b/64d4899d73f91ba49a8c18a8ff3f0ea8f1c1d75481760df8c68ef5235bf5/rich-15.0.0-py3-none-any.whl", hash = "sha256:33bd4ef74232fb73fe9279a257718407f169c09b78a87ad3d296f548e27de0bb", size = 310654, upload-time = "2026-04-12T08:24:02.83Z" },
 ]
 
+[[package]]
+name = "rich-argparse"
+version = "1.8.0"
+source = { registry = "https://pypi.org/simple" }
+dependencies = [
+    { name = "rich" },
+]
+sdist = { url = "https://files.pythonhosted.org/packages/6a/e5/1064c43203a357d668cd42435f7a15fe6af51512d85b2104fecb937aa861/rich_argparse-1.8.0.tar.gz", hash = "sha256:679df3d832fa94ad6e4bdb07ded088cd7ea2dddc58ae9b2b46346a40b06cbc0c", size = 38940, upload-time = "2026-05-01T15:18:43.604Z" }
+wheels = [
+    { url = "https://files.pythonhosted.org/packages/0b/35/1cceccc5fcb50fa2ed53e2aa278cd032f3902682a73e763fb1ac3be8e6fa/rich_argparse-1.8.0-py3-none-any.whl", hash = "sha256:d2a3ce7854654e2253c578763ab0a32f05016f23a55fadba7b9a91b6c0e92142", size = 25616, upload-time = "2026-05-01T15:18:42.395Z" },
+]
+
 [[package]]
 name = "scar-cli"
-version = "0.7.0"
+version = "0.8.0"
 source = { editable = "." }
 dependencies = [
     { name = "rich" },
+    { name = "rich-argparse" },
 ]
 
 [package.dev-dependencies]
@@ -125,7 +138,10 @@ dev = [
 ]
 
 [package.metadata]
-requires-dist = [{ name = "rich", specifier = ">=15.0.0" }]
+requires-dist = [
+    { name = "rich", specifier = ">=15.0.0" },
+    { name = "rich-argparse", specifier = ">=1.8.0" },
+]
 
 [package.metadata.requires-dev]
 dev = [{ name = "pytest", specifier = ">=8.0" }]

scar_cli-0.8.0/.release-please-manifest.json

@@ -1,3 +0,0 @@
-{
-  ".": "0.8.0"
-}