feat: add python-architecture-review skill with Cosmic Python guards

.github/workflows/ci.yml

@@ -73,6 +73,7 @@ jobs:
           uvx --python 3.11 --from 'skills-ref==0.1.1' agentskills validate src/python-property-based-testing
           uvx --python 3.11 --from 'skills-ref==0.1.1' agentskills validate src/python-test-suite-audit
           uvx --python 3.11 --from 'skills-ref==0.1.1' agentskills validate src/python-type-safety
+          uvx --python 3.11 --from 'skills-ref==0.1.1' agentskills validate src/python-architecture-review
 
       - name: skills.sh.json syntax
         run: uv run --locked --group dev python -m json.tool skills.sh.json >/dev/null
@@ -137,7 +138,7 @@ jobs:
           # line after the CLI's `│    ` box-drawing indentation. A bare substring
           # match would false-pass on a renamed skill (`-v2` suffix) or on another
           # skill's description that merely mentions this name.
-          for skill in python-blackbox-testing python-parameterized-testing python-property-based-testing python-test-suite-audit python-type-safety; do
+          for skill in python-blackbox-testing python-parameterized-testing python-property-based-testing python-test-suite-audit python-type-safety python-architecture-review; do
             if grep -Eq "^[^[:alnum:]]*${skill}[^[:alnum:]]*$" "$plain"; then
               echo "OK: $skill is listed by skills.sh discovery"
             else

CHANGELOG.md

@@ -34,6 +34,12 @@ All notable changes to this project are documented in this file. The format foll
   three references, eval fixtures with a depth-split near-miss against parameterized tables, and a
   standard-library-only `plan_property_matrix.py` planner. Stateful RuleBasedStateMachine work is
   explicitly deferred.
+- `python-architecture-review`, an independently installable read-only skill for Cosmic Python
+  architecture reviews (seams, segmentation, inheritance, boundaries, scalability,
+  maintainability) with complexity-gated proposals, forward-looking triggers, folder hierarchy,
+  import-linter and scanner guards, and test-pyramid placement. It ships three references, eval
+  fixtures with CRUD carve-out and forward-looking cases, and a standard-library-only
+  `scan_architecture.py` scanner. Refactoring stays out of scope.
 
 ## [0.1.0] - 2026-09-24
 

CONTRIBUTING.md

@@ -171,6 +171,7 @@ uvx --python 3.11 --from 'skills-ref==0.1.1' agentskills validate src/python-par
 uvx --python 3.11 --from 'skills-ref==0.1.1' agentskills validate src/python-property-based-testing
 uvx --python 3.11 --from 'skills-ref==0.1.1' agentskills validate src/python-test-suite-audit
 uvx --python 3.11 --from 'skills-ref==0.1.1' agentskills validate src/python-type-safety
+uvx --python 3.11 --from 'skills-ref==0.1.1' agentskills validate src/python-architecture-review
 ```
 
 Run the validator for each changed skill rather than assuming all skills were checked. Do not claim a

README.md

@@ -23,6 +23,7 @@ to inspect, not proof that a program is correct.
 | `python-property-based-testing` | Proving a quantified invariant with Hypothesis strategies and shrinking; designing oracles, bounding assume/filtering, replaying a shrunken counterexample with seed and settings. |
 | `python-test-suite-audit` | Auditing an existing suite for tautological tests, weak assertions, over-mocking, private coupling, missing error paths, or order-dependent structure; producing a severity scorecard without changing code. |
 | `python-type-safety` | Adding missing annotations boundary-first; gating with mypy or pyright to zero errors; triaging error codes; recording checker divergences without chasing them. |
+| `python-architecture-review` | Reviewing seams, segmentation, inheritance, boundaries, scalability, and maintainability through Cosmic Python; proposing a gated target architecture with guards, folder hierarchy, and tests that make violation hard. |
 
 Each skill is independently installable. None requires another, and all are designed to work
 with the conventions already present in the target repository.
@@ -44,6 +45,7 @@ with the conventions already present in the target repository.
 - `python-type-safety` for annotation health: boundary-first annotation coverage, strict
   checker gates with mypy or pyright, error-code triage, and report-only cross-checks.
   Type first, then use black-box or parameterized skills to prove runtime behavior.
+- `python-architecture-review` for architecture proposals: seam and dependency mapping, complexity-gated Cosmic Python targets, forward-looking scalability, and enforceable guards. Read-only by default with chat findings and an optional proposal file; it never refactors code.
 - Combine them on libraries with a CLI or a broad public surface: black-box for the acceptance
   matrix, parameterized for the input-domain depth, audit for suite health. Keep the skills
   independent; each owns its workflow and its own evidence report.
@@ -89,6 +91,12 @@ Install the type safety skill:
 npx skills add nexusnv/python-agentic-skills --skill python-type-safety
 ```
 
+Install the architecture review skill:
+
+```bash
+npx skills add nexusnv/python-agentic-skills --skill python-architecture-review
+```
+
 The install commands above are **project-local by default** because they omit `--global`; keep the
 canonical skill sources in `src/`. For a personal installation shared across projects, add
 `--global`: