boost-skill-cli 1.0.156__tar.gz → 1.0.158__tar.gz

{boost_skill_cli-1.0.156 → boost_skill_cli-1.0.158}/.gitignore

@@ -195,9 +195,9 @@ cython_debug/
 .abstra/
 
 # Visual Studio Code
-#   Visual Studio Code specific template is maintained in a separate VisualStudioCode.gitignore 
+#   Visual Studio Code specific template is maintained in a separate VisualStudioCode.gitignore
 #   that can be found at https://github.com/github/gitignore/blob/main/Global/VisualStudioCode.gitignore
-#   and can be added to the global gitignore or merged into this file. However, if you prefer, 
+#   and can be added to the global gitignore or merged into this file. However, if you prefer,
 #   you could uncomment the following to ignore the entire vscode folder
 # .vscode/
 # Temporary file for partial code execution

boost_skill_cli-1.0.158/.pre-commit-config.yaml

@@ -0,0 +1,67 @@
+# Shift-left the `make lint` gate to *before* the commit, so ruff / mypy /
+# codespell / whitespace fixes land locally instead of failing in CI.
+#
+#   pip install pre-commit && pre-commit install   # once — hooks then run on `git commit`
+#   pre-commit run --all-files                      # run everything right now
+#
+# pre-commit.ci (free for open source) runs these on every PR and pushes the
+# auto-fixes back onto the branch — see the `ci:` block. Versions below are
+# pinned to match the tools `make lint` runs; bump them together.
+ci:
+  autofix_prs: true
+  autoupdate_schedule: monthly
+  # mypy is a `repo: local` hook that needs the project's venv — pre-commit.ci
+  # builds isolated hook envs and can't provide it, so `make lint` / CI own it.
+  skip: [mypy]
+
+minimum_pre_commit_version: "3.5.0"
+
+repos:
+  - repo: https://github.com/pre-commit/pre-commit-hooks
+    rev: v6.0.0
+    hooks:
+      # Whitespace fixers skip the generated boards (build_roadmap.py owns their
+      # bytes — a drift check would fight this hook), the loop-edited roadmap
+      # item data (many parallel branches touch these; a whitespace churn there
+      # means needless merge conflicts) and IDE files under .idea/.
+      - id: trailing-whitespace
+        exclude: &data_and_generated '^(docs/(roadmap|design-roadmap)\.html$|docs/roadmap/items/|\.idea/)'
+      - id: end-of-file-fixer
+        exclude: *data_and_generated
+      - id: mixed-line-ending
+        args: [--fix=lf]
+        exclude: *data_and_generated
+      - id: check-merge-conflict
+      - id: check-yaml
+      - id: check-toml
+      - id: check-added-large-files
+
+  # Ruff — pinned to the version in `make lint`; reads the same [tool.ruff]
+  # config from pyproject.toml and is scoped to the gate's paths.
+  - repo: https://github.com/astral-sh/ruff-pre-commit
+    rev: v0.15.22
+    hooks:
+      - id: ruff
+        args: [--fix]
+        files: ^(boost_cli|tests)/.*\.py$
+
+  # codespell — same version and [tool.codespell] config (skip / ignore lists)
+  # as `make lint`, scoped to the same targets: boost_cli, docs, README.
+  - repo: https://github.com/codespell-project/codespell
+    rev: v2.4.3
+    hooks:
+      - id: codespell
+        additional_dependencies: [tomli]
+        files: ^(boost_cli/|docs/|README\.md)
+
+  # mypy — run from the project venv so it matches `make lint` exactly
+  # (config drives it: files = ["boost_cli"]). Local so stdlib/editable
+  # resolution is identical; skipped on pre-commit.ci (see `ci.skip`).
+  - repo: local
+    hooks:
+      - id: mypy
+        name: mypy (boost_cli)
+        entry: mypy
+        language: system
+        pass_filenames: false
+        types: [python]

{boost_skill_cli-1.0.156 → boost_skill_cli-1.0.158}/CONTRIBUTING.md

@@ -13,6 +13,21 @@ make venv          # pytest, coverage, mutmut, ruff, mypy — runtime stays stdl
 Every test runs against a throwaway `$HOME`, so nothing touches your real
 agent configs.
 
+### Pre-commit hooks (recommended)
+
+Catch the `make lint` findings *before* they reach CI:
+
+```bash
+pip install pre-commit && pre-commit install   # once
+pre-commit run --all-files                      # or run everything now
+```
+
+`git commit` then runs ruff, mypy, codespell and the whitespace fixers on your
+staged files. The hooks in [`.pre-commit-config.yaml`](.pre-commit-config.yaml)
+are pinned to the same tool versions the lint gate uses, and
+[pre-commit.ci](https://pre-commit.ci) runs them on every PR and pushes any
+auto-fixes straight back onto the branch.
+
 ## The gates (CI runs all of these)
 
 | Command | Gate |
@@ -22,12 +37,18 @@ agent configs.
 | `make mutation` | mutmut over `boost_cli/core`, **≥80% mutants killed** |
 | `ruff check boost_cli tests` | lint, zero findings |
 | `mypy` | type check, zero errors |
+| `lint-imports` | layering: `core/` imports no `commands/`/`cli`; CLI depends inward |
 
 ## Ground rules
 
 - **Stdlib only** at runtime. No third-party imports in `boost_cli/`.
 - New commands live in `boost_cli/commands/<group>.py` as
   `def cmd_<name>(argv) -> int` and are dispatched lazily from `cli.py`.
+- **Respect the layers** (`cli → commands → core`): `core/` is the engine and
+  must import neither `commands/` nor `cli`; `commands/` must not reach up into
+  `cli`. `import-linter` enforces this — see `[tool.importlinter]` in
+  `pyproject.toml` (the one allowlisted edge is `completions` reading the
+  `COMMANDS` registry).
 - Behavior changes need tests — functional tests drive the CLI in-process via
   `boost_cli.cli.main` (see `tests/conftest.py` for the fixtures).
 - Anything under `boost_cli/core/` is mutation-tested; expect to add unit

{boost_skill_cli-1.0.156 → boost_skill_cli-1.0.158}/Makefile

@@ -14,7 +14,7 @@ EVAL_HOME := $(CURDIR)/.eval-home
 
 venv:
 	python3 -m venv $(VENV)
-	$(VENV)/bin/pip -q install pytest pytest-cov coverage mutmut ruff mypy codespell hypothesis
+	$(VENV)/bin/pip -q install pytest pytest-cov coverage mutmut ruff mypy codespell hypothesis import-linter
 
 unit:
 	$(PYTEST) tests/unit -q
@@ -39,6 +39,7 @@ mutation:
 lint:
 	$(VENV)/bin/ruff check boost_cli tests
 	$(VENV)/bin/mypy
+	$(VENV)/bin/lint-imports
 	$(VENV)/bin/codespell boost_cli docs README.md
 	@command -v actionlint >/dev/null 2>&1 && actionlint || echo "actionlint not on PATH — skipping (CI enforces it)"
 	$(PY) scripts/build_registries.py --check

{boost_skill_cli-1.0.156 → boost_skill_cli-1.0.158}/PKG-INFO

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

{boost_skill_cli-1.0.156 → boost_skill_cli-1.0.158}/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.156'
-__version_tuple__ = version_tuple = (1, 0, 156)
+__version__ = version = '1.0.158'
+__version_tuple__ = version_tuple = (1, 0, 158)
 
 __commit_id__ = commit_id = None

{boost_skill_cli-1.0.156 → boost_skill_cli-1.0.158}/boost_skill_cli.egg-info/PKG-INFO

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

{boost_skill_cli-1.0.156 → boost_skill_cli-1.0.158}/boost_skill_cli.egg-info/SOURCES.txt

@@ -1,6 +1,7 @@
 .gitattributes
 .gitignore
 .gitleaks.toml
+.pre-commit-config.yaml
 CLAUDE.md
 CONTRIBUTING.md
 LICENSE

{boost_skill_cli-1.0.156 → boost_skill_cli-1.0.158}/docs/roadmap/items/layering-guard-import-linter.md

@@ -2,15 +2,15 @@
 id: layering-guard-import-linter
 board: code
 section: dx
-status: planned
+status: shipped
 category: Quality · Architecture
 complexity: M
 impact: Med
 wow: 4
 note: core/ ↛ commands/
 order: 3
-owner:
-pr:
+owner: loop/layering
+pr: 182
 title: Layering guard — <code>import-linter</code>
 ---
 Enforce the architecture boost already assumes: <code>core/</code> must

{boost_skill_cli-1.0.156 → boost_skill_cli-1.0.158}/docs/roadmap/items/shift-left-gate-pre-commit-pre-commit-ci.md

@@ -2,15 +2,15 @@
 id: shift-left-gate-pre-commit-pre-commit-ci
 board: code
 section: dx
-status: planned
+status: shipped
 category: DX · Infra
 complexity: S
 impact: High
 wow: 4
 note: auto-fix PRs
 order: 2
-owner:
-pr:
+owner: loop/pre-commit
+pr: 181
 title: Shift-left gate — <code>pre-commit</code> + pre-commit.ci
 ---
 A <code>.pre-commit-config.yaml</code> runs ruff, mypy, codespell and the

{boost_skill_cli-1.0.156 → boost_skill_cli-1.0.158}/docs/roadmap.html

@@ -1325,7 +1325,7 @@ improvement is real (or isn't)."</p>
       </article>
 
       <article class="cap rcard">
-        <div class="head"><span class="pill planned">Planned</span><span class="cat">DX · Infra</span></div>
+        <div class="head"><span class="pill shipped">Shipped</span><span class="cat">DX · Infra</span></div>
         <h3>Shift-left gate — <code>pre-commit</code> + pre-commit.ci</h3>
         <p>A <code>.pre-commit-config.yaml</code> runs ruff, mypy, codespell and the
            whitespace fixers <em>before</em> code ever reaches CI, and the free-for-OSS
@@ -1341,7 +1341,7 @@ improvement is real (or isn't)."</p>
       </article>
 
       <article class="cap rcard">
-        <div class="head"><span class="pill planned">Planned</span><span class="cat">Quality · Architecture</span></div>
+        <div class="head"><span class="pill shipped">Shipped</span><span class="cat">Quality · Architecture</span></div>
         <h3>Layering guard — <code>import-linter</code></h3>
         <p>Enforce the architecture boost already assumes: <code>core/</code> must
            never import <code>commands/</code>, and the CLI depends inward only.

{boost_skill_cli-1.0.156 → boost_skill_cli-1.0.158}/pyproject.toml

@@ -155,3 +155,24 @@ exclude_lines = [
     "if TYPE_CHECKING:",
     "raise NotImplementedError",
 ]
+
+# import-linter: turn boost's assumed layering into an enforced contract, so a
+# cross-layer import can't silently erode the engine. Run by `make lint`
+# (lint-imports) and the CI lint job. Dev/CI-only — never imported at runtime.
+[tool.importlinter]
+root_package = "boost_cli"
+
+[[tool.importlinter.contracts]]
+name = "boost_cli is layered: cli -> commands -> core"
+type = "layers"
+# Higher layers may import lower ones, never the reverse: core/ (the engine)
+# must import neither commands/ nor the CLI, and commands/ must not reach up
+# into cli. This is the structural rule CLAUDE.md documents — core is the
+# mutation-tested engine, commands/ is thin glue on top of it.
+layers = ["cli", "commands", "core"]
+containers = ["boost_cli"]
+# The one intentional upward edge: `boost completions` reads COMMANDS — the
+# command registry, which lives in cli.py by design (see CLAUDE.md) — to list
+# every command. That single metadata lookup is allowlisted; any other
+# commands -> cli import is still a violation.
+ignore_imports = ["boost_cli.commands.configuration -> boost_cli.cli"]