modern-python-guidance 0.3.5__tar.gz → 0.3.6__tar.gz

{modern_python_guidance-0.3.5 → modern_python_guidance-0.3.6}/CHANGELOG.md

@@ -2,6 +2,22 @@
 
 All notable changes to this project will be documented in this file.
 
+## [0.3.6] — 2026-05-31
+
+### Added
+
+- Rule-based delivery via symlink: `mpg setup` creates `.claude/rules/modern-python.md` that auto-injects modern Python guidance whenever Python-related files are touched, replacing reliance on probabilistic skill matching (closes #79)
+- `setup_rules()` / `uninstall_rules()` mirroring skills symlink pattern
+- `source.is_symlink()` security check to refuse symlink-to-symlink chains
+- CI sync test enforcing SKILL.md body == rule body consistency
+- 21 new tests (V-037 to V-060) for setup, uninstall, CI sync, and security
+
+### Changed
+
+- `--skills-only` now includes Rules (both are project-local artifacts)
+- README updated to document 4 delivery methods (was 3)
+- `--project-dir` help text updated to mention Skills/Rules symlinks
+
 ## [0.3.5] — 2026-05-30
 
 ### Added

{modern_python_guidance-0.3.5 → modern_python_guidance-0.3.6}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: modern-python-guidance
-Version: 0.3.5
+Version: 0.3.6
 Summary: Version-aware BAD/GOOD pattern guides that help AI coding agents generate modern Python
 Project-URL: Homepage, https://github.com/yottayoshida/modern-python-guidance
 Project-URL: Repository, https://github.com/yottayoshida/modern-python-guidance
@@ -44,7 +44,7 @@ Stop your AI from writing `typing.List`, `@validator`, and `setup.py`. 41 versio
 - **Measurable impact**: AI writes modern Python 98% of the time with mpg, vs 79% without — even with vague prompts (Opus 4.8, [V5 benchmark details](docs/benchmark-v5.md))
 - **41 guides** across stdlib, Pydantic, FastAPI, Django, SQLAlchemy, pytest, and toolchain
 - **Version-aware**: auto-detects your project's Python version and filters guides accordingly
-- **3 delivery methods**: MCP server, CLI, Agent Skills plugin
+- **4 delivery methods**: MCP server, CLI, Agent Skills, and Rules (auto-injects on `.py` file touch)
 - **Not Ruff**: Ruff auto-fixes syntax (`List` → `list`). mpg guides design decisions that Ruff can't touch — `TaskGroup` over `gather`, Pydantic V2 migration, SQLAlchemy 2.0 style
 
 > **Note:** The tool itself requires Python 3.11+ to run. Guides cover patterns from Python 3.9 onward, and `--python-version` filters guides for your target environment.
@@ -58,7 +58,7 @@ pip install modern-python-guidance
 mpg setup
 ```
 
-This registers the MCP server and links Agent Skills in one command. Start a new Claude Code session afterwards — newly registered MCP servers and skills take effect on the next launch.
+This registers the MCP server, links Agent Skills, and creates a Rules file (`.claude/rules/modern-python.md`) in one command. The Rules file auto-injects modern Python guidance whenever you touch Python-related files. Start a new Claude Code session afterwards — newly registered MCP servers, skills, and rules take effect on the next launch.
 
 ### CLI
 
@@ -90,7 +90,7 @@ claude mcp add mpg -- mpg mcp
 }
 ```
 
-**Agent Skills symlink (Claude Code):**
+**Agent Skills + Rules only (Claude Code):**
 ```bash
 mpg setup --skills-only
 ```
@@ -99,25 +99,25 @@ mpg setup --skills-only
 | Flag | Purpose |
 |------|---------|
 | `--mcp-only` | MCP registration only |
-| `--skills-only` | Agent Skills symlink only |
+| `--skills-only` | Project-local artifacts only (Skills + Rules) |
 | `--scope {user,local}` | MCP scope (default: user) |
-| `--project-dir PATH` | Target project for Skills symlink |
+| `--project-dir PATH` | Target project for Skills/Rules symlinks |
 | `--dry-run` | Show what would be done |
 
-**Uninstall** — reverse `mpg setup` (deregister the MCP server and unlink Agent Skills):
+**Uninstall** — reverse `mpg setup` (deregister the MCP server and unlink Agent Skills + Rules):
 ```bash
-mpg uninstall            # remove both
+mpg uninstall            # remove all
 mpg uninstall --dry-run  # preview what would be removed
 ```
 
 | Flag | Purpose |
 |------|---------|
 | `--mcp-only` | MCP deregistration only |
-| `--skills-only` | Agent Skills unlink only |
-| `--project-dir PATH` | Target project for the Skills symlink |
+| `--skills-only` | Project-local artifacts only (Skills + Rules) |
+| `--project-dir PATH` | Target project for Skills/Rules symlinks |
 | `--dry-run` | Show what would be done |
 
-`mpg uninstall` clears the MCP registration from every scope `setup` can write to (user and local), removes only the symlink mpg created (never its target or other skills), and is idempotent — running it on an already-clean state is a harmless no-op.
+`mpg uninstall` clears the MCP registration from every scope `setup` can write to (user and local), removes only the symlinks mpg created (never their targets or other files), and is idempotent — running it on an already-clean state is a harmless no-op.
 
 </details>
 

{modern_python_guidance-0.3.5 → modern_python_guidance-0.3.6}/README.md

@@ -12,7 +12,7 @@ Stop your AI from writing `typing.List`, `@validator`, and `setup.py`. 41 versio
 - **Measurable impact**: AI writes modern Python 98% of the time with mpg, vs 79% without — even with vague prompts (Opus 4.8, [V5 benchmark details](docs/benchmark-v5.md))
 - **41 guides** across stdlib, Pydantic, FastAPI, Django, SQLAlchemy, pytest, and toolchain
 - **Version-aware**: auto-detects your project's Python version and filters guides accordingly
-- **3 delivery methods**: MCP server, CLI, Agent Skills plugin
+- **4 delivery methods**: MCP server, CLI, Agent Skills, and Rules (auto-injects on `.py` file touch)
 - **Not Ruff**: Ruff auto-fixes syntax (`List` → `list`). mpg guides design decisions that Ruff can't touch — `TaskGroup` over `gather`, Pydantic V2 migration, SQLAlchemy 2.0 style
 
 > **Note:** The tool itself requires Python 3.11+ to run. Guides cover patterns from Python 3.9 onward, and `--python-version` filters guides for your target environment.
@@ -26,7 +26,7 @@ pip install modern-python-guidance
 mpg setup
 ```
 
-This registers the MCP server and links Agent Skills in one command. Start a new Claude Code session afterwards — newly registered MCP servers and skills take effect on the next launch.
+This registers the MCP server, links Agent Skills, and creates a Rules file (`.claude/rules/modern-python.md`) in one command. The Rules file auto-injects modern Python guidance whenever you touch Python-related files. Start a new Claude Code session afterwards — newly registered MCP servers, skills, and rules take effect on the next launch.
 
 ### CLI
 
@@ -58,7 +58,7 @@ claude mcp add mpg -- mpg mcp
 }
 ```
 
-**Agent Skills symlink (Claude Code):**
+**Agent Skills + Rules only (Claude Code):**
 ```bash
 mpg setup --skills-only
 ```
@@ -67,25 +67,25 @@ mpg setup --skills-only
 | Flag | Purpose |
 |------|---------|
 | `--mcp-only` | MCP registration only |
-| `--skills-only` | Agent Skills symlink only |
+| `--skills-only` | Project-local artifacts only (Skills + Rules) |
 | `--scope {user,local}` | MCP scope (default: user) |
-| `--project-dir PATH` | Target project for Skills symlink |
+| `--project-dir PATH` | Target project for Skills/Rules symlinks |
 | `--dry-run` | Show what would be done |
 
-**Uninstall** — reverse `mpg setup` (deregister the MCP server and unlink Agent Skills):
+**Uninstall** — reverse `mpg setup` (deregister the MCP server and unlink Agent Skills + Rules):
 ```bash
-mpg uninstall            # remove both
+mpg uninstall            # remove all
 mpg uninstall --dry-run  # preview what would be removed
 ```
 
 | Flag | Purpose |
 |------|---------|
 | `--mcp-only` | MCP deregistration only |
-| `--skills-only` | Agent Skills unlink only |
-| `--project-dir PATH` | Target project for the Skills symlink |
+| `--skills-only` | Project-local artifacts only (Skills + Rules) |
+| `--project-dir PATH` | Target project for Skills/Rules symlinks |
 | `--dry-run` | Show what would be done |
 
-`mpg uninstall` clears the MCP registration from every scope `setup` can write to (user and local), removes only the symlink mpg created (never its target or other skills), and is idempotent — running it on an already-clean state is a harmless no-op.
+`mpg uninstall` clears the MCP registration from every scope `setup` can write to (user and local), removes only the symlinks mpg created (never their targets or other files), and is idempotent — running it on an already-clean state is a harmless no-op.
 
 </details>
 

{modern_python_guidance-0.3.5 → modern_python_guidance-0.3.6}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "modern-python-guidance"
-version = "0.3.5"
+version = "0.3.6"
 description = "Version-aware BAD/GOOD pattern guides that help AI coding agents generate modern Python"
 readme = "README.md"
 license = "Apache-2.0 OR MIT"
@@ -49,11 +49,12 @@ Issues = "https://github.com/yottayoshida/modern-python-guidance/issues"
 
 [tool.hatch.build.targets.wheel]
 packages = ["src/modern_python_guidance"]
-only-include = ["src/modern_python_guidance", "skills"]
+only-include = ["src/modern_python_guidance", "skills", "rules"]
 
 [tool.hatch.build.targets.wheel.sources]
 "src" = ""
 "skills" = "modern_python_guidance/skills"
+"rules" = "modern_python_guidance/rules"
 
 [tool.ruff]
 target-version = "py311"

modern_python_guidance-0.3.6/rules/modern-python.md

@@ -0,0 +1,70 @@
+---
+paths: ["**/*.py", "*.py", "**/pyproject.toml", "**/requirements*.txt", "**/setup.py", "**/setup.cfg", "**/.python-version", "**/Pipfile"]
+---
+
+# Modern Python Guidance
+
+Pre-generation design patterns that Ruff cannot auto-fix. Apply these transformations when writing or reviewing Python code.
+
+## When to use
+
+- Writing new Python code (use modern patterns from the start)
+- Reviewing Python code (flag outdated patterns)
+- Migrating from Pydantic V1 to V2
+- Upgrading Python version (check which new features are available)
+- Replacing legacy tooling (setup.py, flake8, pip)
+
+## Embedded patterns (high-frequency, Ruff-uncovered)
+
+### Pydantic V2 (>=3.9)
+
+- `@validator("f")` → `@field_validator("f")`
+- `@root_validator` → `@model_validator(mode="after")`
+- `class Config:` → `model_config = ConfigDict(...)`
+- `orm_mode` → `from_attributes`, `allow_population_by_field_name` → `populate_by_name`
+- `.parse_obj(d)` → `.model_validate(d)`, `.parse_raw(j)` → `.model_validate_json(j)`
+- `.dict()` → `.model_dump()`, `.json()` → `.model_dump_json()`
+- `.schema()` → `.model_json_schema()`, `.copy()` → `.model_copy()`
+
+### FastAPI (>=3.9)
+
+- `@app.on_event("startup")`/`"shutdown"` → `@asynccontextmanager` lifespan + `FastAPI(lifespan=lifespan)`; yield dict becomes `request.state`
+- `db: Session = Depends(get_db)` → `DbDep = Annotated[Session, Depends(get_db)]`; reusable type alias per PEP 593
+
+### httpx
+
+- Per-request `async with httpx.AsyncClient()` → shared `AsyncClient` with `base_url`
+  - Caveat: shared client must be closed via `async with` or lifespan management
+
+### asyncio (>=3.11)
+
+- `await asyncio.gather(a(), b())` → `async with asyncio.TaskGroup() as tg:` + `tg.create_task()`; access results via `task.result()`
+  - Caveat: 3.11+ only. `TaskGroup` cancels siblings on error and raises `ExceptionGroup`; `gather` preserves return order and supports `return_exceptions=True`
+
+### SQLAlchemy 2.0 (>=3.9)
+
+- `session.query(User).filter()` → `session.execute(select(User).where())`; use `session.scalars()` for ORM results
+- `Column(Integer)` → `Mapped[int] = mapped_column()`; type inferred from annotation, nullability from `Optional`/`| None`
+- Sync `Session` with `asyncio.to_thread` → `AsyncSession` + `create_async_engine` + `async_sessionmaker`
+
+### Toolchain
+
+- `setup.py` / `setup.cfg` → `pyproject.toml` with `[build-system]` + `[project]` (PEP 621)
+- `subprocess.run(f"cmd {arg}", shell=True)` → `subprocess.run(["cmd", arg], check=True)`
+  - Caveat: `shell=True` is valid when pipes/globs are needed; use `shlex.quote()` for user input
+
+## All 41 guides by category
+
+- **typing** (7): `use-builtin-generics`, `union-syntax`, `type-parameter-syntax`, `override-decorator`, `typeis-vs-typeguard`, `paramspec-decorators`, `deferred-annotations`
+- **async** (3): `taskgroup-over-gather`, `exception-groups`, `async-timeout-context`
+- **stdlib** (5): `datetime-utc`, `pathlib-over-os-path`, `tomllib-builtin`, `removeprefix-removesuffix`, `template-strings`
+- **data-structures** (3): `dict-merge-operator`, `match-case-patterns`, `dataclass-modern`
+- **pydantic** (4): `pydantic-v2-validators`, `pydantic-v2-config`, `pydantic-v2-model-api`, `pydantic-v2-serialization`
+- **fastapi** (3): `fastapi-lifespan`, `fastapi-annotated-depends`, `fastapi-typed-state`
+- **httpx** (2): `httpx-async-client-reuse`, `httpx-streaming`
+- **django** (3): `django-json-field`, `django-async-views`, `django-check-constraints`
+- **sqlalchemy** (3): `sqlalchemy-2-style`, `sqlalchemy-mapped-column`, `sqlalchemy-async-session`
+- **pytest** (3): `pytest-parametrize`, `pytest-tmp-path`, `pytest-raises-match`
+- **toolchain** (5): `pyproject-toml-over-setup`, `uv-over-pip`, `ruff-over-flake8`, `no-pickle`, `safe-subprocess`
+
+For full code examples, use `mpg retrieve <guide-id>` or MCP tool `retrieve_guides`.

{modern_python_guidance-0.3.5 → modern_python_guidance-0.3.6}/src/modern_python_guidance/__init__.py

@@ -1,3 +1,3 @@
 """Modern Python Guidance — version-aware BAD/GOOD pattern guides for AI coding agents."""
 
-__version__ = "0.3.5"
+__version__ = "0.3.6"

{modern_python_guidance-0.3.5 → modern_python_guidance-0.3.6}/src/modern_python_guidance/cli.py

@@ -74,10 +74,12 @@ def main(argv: list[str] | None = None) -> None:
     # setup
     p_setup = subparsers.add_parser(
         "setup",
-        help="Register MCP server and link Agent Skills",
+        help="Register MCP server and link Agent Skills + Rules",
     )
     p_setup.add_argument("--mcp-only", action="store_true", help="MCP registration only")
-    p_setup.add_argument("--skills-only", action="store_true", help="Skills symlink only")
+    p_setup.add_argument(
+        "--skills-only", action="store_true", help="Project-local artifacts only (Skills + Rules)"
+    )
     p_setup.add_argument(
         "--scope",
         choices=["user", "local"],
@@ -87,21 +89,25 @@ def main(argv: list[str] | None = None) -> None:
     p_setup.add_argument(
         "--project-dir",
         type=Path,
-        help="Project directory for Skills symlink",
+        help="Project directory for Skills/Rules symlinks",
     )
     p_setup.add_argument("--dry-run", action="store_true", help="Show what would be done")
 
     # uninstall
     p_uninstall = subparsers.add_parser(
         "uninstall",
-        help="Reverse 'setup': deregister MCP server and unlink Agent Skills",
+        help="Reverse 'setup': deregister MCP server and unlink Agent Skills + Rules",
     )
     p_uninstall.add_argument("--mcp-only", action="store_true", help="MCP deregistration only")
-    p_uninstall.add_argument("--skills-only", action="store_true", help="Skills unlink only")
+    p_uninstall.add_argument(
+        "--skills-only",
+        action="store_true",
+        help="Project-local artifacts only (Skills + Rules)",
+    )
     p_uninstall.add_argument(
         "--project-dir",
         type=Path,
-        help="Project directory for Skills symlink",
+        help="Project directory for Skills/Rules symlinks",
     )
     p_uninstall.add_argument("--dry-run", action="store_true", help="Show what would be done")
 

{modern_python_guidance-0.3.5 → modern_python_guidance-0.3.6}/src/modern_python_guidance/setup_cmd.py

@@ -11,8 +11,16 @@ import sys
 from pathlib import Path
 
 SKILLS_LINK_NAME = "modern-python-guidance"
+RULE_FILE_NAME = "modern-python.md"
 MCP_SERVER_NAME = "mpg"
 
+RULE_FRONTMATTER = (
+    "---\n"
+    'paths: ["**/*.py", "*.py", "**/pyproject.toml", "**/requirements*.txt",'
+    ' "**/setup.py", "**/setup.cfg", "**/.python-version", "**/Pipfile"]\n'
+    "---"
+)
+
 
 def _find_skills_dir() -> Path:
     """Resolve the bundled skills directory (package install or editable)."""
@@ -33,6 +41,25 @@ def _find_skills_dir() -> Path:
     raise FileNotFoundError(msg)
 
 
+def _find_rule_source() -> Path:
+    """Resolve the bundled rule file (package install or editable)."""
+    try:
+        pkg = importlib.resources.files("modern_python_guidance") / "rules"
+        rule_path = Path(str(pkg)) / RULE_FILE_NAME
+        if rule_path.is_file():
+            return rule_path
+    except (TypeError, FileNotFoundError):
+        pass
+
+    src_root = Path(__file__).resolve().parent.parent.parent
+    dev_path = src_root / "rules" / RULE_FILE_NAME
+    if dev_path.is_file():
+        return dev_path
+
+    msg = "Cannot locate bundled rule file"
+    raise FileNotFoundError(msg)
+
+
 def _find_project_root(start: Path | None = None) -> Path:
     """Walk upward from *start* to find the project root."""
     current = (start or Path.cwd()).resolve()
@@ -63,6 +90,26 @@ def _skills_link_path(project_dir: Path | None = None) -> Path:
     return root / ".claude" / "skills" / SKILLS_LINK_NAME
 
 
+def _rules_file_path(project_dir: Path | None = None) -> Path:
+    """Resolve the rule file symlink path: ``<root>/.claude/rules/modern-python.md``."""
+    root = project_dir or _find_project_root()
+    return root / ".claude" / "rules" / RULE_FILE_NAME
+
+
+def _build_rule_text() -> str:
+    """Generate rule file content from SKILL.md body with rule-specific frontmatter.
+
+    Used by CI sync tests to verify the bundled ``rules/modern-python.md`` matches
+    what would be generated from SKILL.md. Strips SKILL.md frontmatter and prepends
+    rule-only frontmatter (no name/description keys).
+    """
+    skills_dir = _find_skills_dir()
+    skill_md = (skills_dir / "SKILL.md").read_text(encoding="utf-8")
+    parts = skill_md.split("---", 2)
+    body = parts[2].lstrip("\n")
+    return RULE_FRONTMATTER + "\n\n" + body
+
+
 def setup_mcp(
     *,
     scope: str = "user",
@@ -80,7 +127,7 @@ def setup_mcp(
         print("Error: 'claude' command not found.", file=sys.stderr)
         print("Install Claude Code: https://claude.ai/download", file=sys.stderr)
         print(
-            "Run 'mpg setup --skills-only' to set up Agent Skills without MCP.",
+            "Run 'mpg setup --skills-only' to set up project-local artifacts without MCP.",
             file=sys.stderr,
         )
         return False
@@ -156,6 +203,58 @@ def setup_skills(
     return True
 
 
+def setup_rules(
+    *,
+    project_dir: Path | None = None,
+    dry_run: bool = False,
+) -> bool:
+    """Create a rule file symlink. Returns True on success."""
+    try:
+        source = _find_rule_source()
+    except FileNotFoundError as e:
+        print(f"Error: {e}", file=sys.stderr)
+        return False
+
+    if source.is_symlink():
+        print("Error: rule source is itself a symlink (unexpected).", file=sys.stderr)
+        return False
+
+    root = project_dir or _find_project_root()
+    link_path = _rules_file_path(project_dir)
+    rules_parent = link_path.parent
+
+    if dry_run:
+        print(f"Would link: {link_path} -> {source}")
+        return True
+
+    if link_path.is_symlink():
+        current_target = Path(os.readlink(link_path))
+        if current_target == source or link_path.resolve() == source.resolve():
+            print(f"Rule already linked at {link_path.relative_to(root)}")
+            return True
+        link_path.unlink()
+    elif link_path.exists():
+        print(
+            f"Error: {link_path.relative_to(root)} exists and is not a symlink.",
+            file=sys.stderr,
+        )
+        print(
+            f"Remove it manually: rm {shlex.quote(str(link_path))}",
+            file=sys.stderr,
+        )
+        return False
+
+    try:
+        rules_parent.mkdir(parents=True, exist_ok=True)
+        os.symlink(source, link_path)
+    except OSError as e:
+        print(f"Error creating symlink: {e}", file=sys.stderr)
+        return False
+
+    print(f"Rule linked to {link_path.relative_to(root)}")
+    return True
+
+
 def run_setup(
     *,
     scope: str = "user",
@@ -171,9 +270,11 @@ def run_setup(
 
     do_mcp = not skills_only
     do_skills = not mcp_only
+    do_rules = not mcp_only
 
     mcp_ok = True
     skills_ok = True
+    rules_ok = True
 
     if do_mcp:
         mcp_ok = setup_mcp(scope=scope, dry_run=dry_run)
@@ -181,7 +282,10 @@ def run_setup(
     if do_skills:
         skills_ok = setup_skills(project_dir=project_dir, dry_run=dry_run)
 
-    if mcp_ok and skills_ok:
+    if do_rules:
+        rules_ok = setup_rules(project_dir=project_dir, dry_run=dry_run)
+
+    if mcp_ok and skills_ok and rules_ok:
         if not dry_run and do_mcp and do_skills:
             print("Ready. Start Claude Code to use mpg guides.")
         return 0

{modern_python_guidance-0.3.5 → modern_python_guidance-0.3.6}/src/modern_python_guidance/uninstall_cmd.py

@@ -11,6 +11,7 @@ from pathlib import Path
 from modern_python_guidance.setup_cmd import (
     MCP_SERVER_NAME,
     _find_project_root,
+    _rules_file_path,
     _skills_link_path,
 )
 
@@ -48,7 +49,7 @@ def uninstall_mcp(*, dry_run: bool = False) -> bool:
         print("Error: 'claude' command not found.", file=sys.stderr)
         print("Install Claude Code: https://claude.ai/download", file=sys.stderr)
         print(
-            "Run 'mpg uninstall --skills-only' to remove Agent Skills without MCP.",
+            "Run 'mpg uninstall --skills-only' to remove project-local artifacts without MCP.",
             file=sys.stderr,
         )
         return False
@@ -139,6 +140,47 @@ def uninstall_skills(
     return True
 
 
+def uninstall_rules(
+    *,
+    project_dir: Path | None = None,
+    dry_run: bool = False,
+) -> bool:
+    """Remove the rule file symlink. Returns True on success.
+
+    Only a symlink is removed. A non-symlink entity at the path is refused.
+    Idempotent: if no symlink is present, this is a no-op success.
+    """
+    root = project_dir or _find_project_root()
+    link_path = _rules_file_path(project_dir)
+
+    if not link_path.is_symlink():
+        if link_path.exists():
+            print(
+                f"Error: {link_path.relative_to(root)} exists and is not a symlink.",
+                file=sys.stderr,
+            )
+            print(
+                f"Remove it manually: rm {shlex.quote(str(link_path))}",
+                file=sys.stderr,
+            )
+            return False
+        print(f"Rule not linked at {link_path.relative_to(root)} — nothing to remove.")
+        return True
+
+    if dry_run:
+        print(f"Would remove: {link_path}")
+        return True
+
+    try:
+        link_path.unlink()
+    except OSError as e:
+        print(f"Error removing symlink: {e}", file=sys.stderr)
+        return False
+
+    print(f"Rule unlinked from {link_path.relative_to(root)}")
+    return True
+
+
 def run_uninstall(
     *,
     mcp_only: bool = False,
@@ -153,9 +195,11 @@ def run_uninstall(
 
     do_mcp = not skills_only
     do_skills = not mcp_only
+    do_rules = not mcp_only
 
     mcp_ok = True
     skills_ok = True
+    rules_ok = True
 
     if do_mcp:
         mcp_ok = uninstall_mcp(dry_run=dry_run)
@@ -163,7 +207,10 @@ def run_uninstall(
     if do_skills:
         skills_ok = uninstall_skills(project_dir=project_dir, dry_run=dry_run)
 
-    if mcp_ok and skills_ok:
+    if do_rules:
+        rules_ok = uninstall_rules(project_dir=project_dir, dry_run=dry_run)
+
+    if mcp_ok and skills_ok and rules_ok:
         if not dry_run and do_mcp and do_skills:
             print("Done. mpg has been removed.")
         return 0

{modern_python_guidance-0.3.5 → modern_python_guidance-0.3.6}/tests/test_guide_structure.py

@@ -18,6 +18,7 @@ import pytest
 
 from modern_python_guidance.frontmatter import parse_frontmatter
 from modern_python_guidance.guide_index import _find_guides_dir
+from modern_python_guidance.setup_cmd import _build_rule_text
 
 GUIDES_DIR = _find_guides_dir()
 EXPECTED_GUIDE_COUNT = 41
@@ -112,6 +113,56 @@ class TestGuideStructure:
         assert body.startswith("# "), "body does not start with H1 heading"
 
 
+class TestRuleFileSync:
+    """CI sync tests: rules/modern-python.md body matches SKILL.md body."""
+
+    def _skill_body(self) -> str:
+        skill_md = (GUIDES_DIR.parent / "SKILL.md").read_text(encoding="utf-8")
+        parts = skill_md.split("---", 2)
+        return parts[2].lstrip("\n")
+
+    def _rule_path(self) -> Path:
+        return GUIDES_DIR.parent.parent.parent / "rules" / "modern-python.md"
+
+    def _rule_parts(self) -> tuple[str, str]:
+        text = self._rule_path().read_text(encoding="utf-8")
+        parts = text.split("---", 2)
+        return parts[1].strip(), parts[2].lstrip("\n")
+
+    def test_body_matches_skill(self):
+        """rules/modern-python.md body == SKILL.md body (content sync)."""
+        skill_body = self._skill_body()
+        _, rule_body = self._rule_parts()
+        assert rule_body == skill_body
+
+    def test_matches_build_rule_text(self):
+        """rules/modern-python.md == _build_rule_text() output (SoT enforcement)."""
+        actual = self._rule_path().read_text(encoding="utf-8")
+        expected = _build_rule_text()
+        assert actual == expected
+
+    def test_frontmatter_has_paths(self):
+        """rules/modern-python.md frontmatter contains expected paths patterns."""
+        fm, _ = self._rule_parts()
+        for pattern in [
+            "**/*.py",
+            "*.py",
+            "**/pyproject.toml",
+            "**/requirements*.txt",
+            "**/setup.py",
+            "**/setup.cfg",
+            "**/.python-version",
+            "**/Pipfile",
+        ]:
+            assert pattern in fm, f"missing path pattern: {pattern}"
+
+    def test_frontmatter_no_name_or_description(self):
+        """rules/modern-python.md frontmatter has NO name/description keys."""
+        fm, _ = self._rule_parts()
+        assert "name:" not in fm
+        assert "description:" not in fm
+
+
 class TestGuideInventory:
     def test_no_duplicate_ids(self):
         seen: dict[str, Path] = {}