modern-python-guidance 0.4.4__tar.gz → 0.4.5__tar.gz

{modern_python_guidance-0.4.4 → modern_python_guidance-0.4.5}/CHANGELOG.md

@@ -2,6 +2,17 @@
 
 All notable changes to this project will be documented in this file.
 
+## [0.4.5] — 2026-06-06
+
+### Fixed
+
+- MCP `retrieve_guides` maxItems hardcoded to 41: replaced with `_guide_limit()` that derives the limit from actual guide count at runtime. `_get_tools()` dynamically injects `maxItems` and description into the `tools/list` schema. Adding new guides no longer requires updating `mcp_server.py`. (closes #98)
+- `docs/design.md` out of sync with v0.4.4 implementation: consolidated overlapping non-goals, added `check`/`setup`/`uninstall` to CLI architecture diagram, added `check.py`, `setup_cmd.py`, `uninstall_cmd.py`, `mcp_server.py` to module responsibility table, fixed Layer 1 guide count from 16 to 18. (closes #99)
+
+### Added
+
+- 1 new test (1000 total).
+
 ## [0.4.4] — 2026-06-05
 
 ### Added

{modern_python_guidance-0.4.4 → modern_python_guidance-0.4.5}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: modern-python-guidance
-Version: 0.4.4
+Version: 0.4.5
 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

{modern_python_guidance-0.4.4 → modern_python_guidance-0.4.5}/docs/design.md

@@ -14,16 +14,16 @@ LLMs frequently generate outdated Python patterns: `typing.List` instead of `lis
 
 ## Non-goals
 
-- Automated code transformation (this is a reference tool, not a codemod)
+- Automated code transformation / codemod (the `check` command scans for outdated patterns via regex + tokenize, but does not rewrite code)
 - Language Server Protocol integration
-- Pattern detection in source code (no AST analysis)
 
 ## Architecture
 
 ```
 ┌─────────────────────────────────────────────────────────┐
 │ CLI (cli.py)                                            │
-│  search │ retrieve │ list │ detect-version              │
+│  search │ retrieve │ list │ detect-version │ check      │
+│  setup │ uninstall                                      │
 ├─────────┴──────────┴──────┴─────────────────────────────┤
 │                                                         │
 │  ┌──────────────┐  ┌──────────────┐  ┌───────────────┐  │
@@ -67,6 +67,10 @@ LLMs frequently generate outdated Python patterns: `typing.List` instead of `lis
 | `retrieve.py` | Renders guide content as JSON with version-match flag and token estimate |
 | `version_detect.py` | Detects target Python version from `--python-version` flag, `pyproject.toml` (`requires-python`), `.python-version` file, or default (3.11) |
 | `compat.py` | `version_compatible()` using `packaging.specifiers` and `token_estimate()` (chars / 4) |
+| `check.py` | Scan a Python file for outdated patterns against guide definitions (regex + tokenize, not AST) |
+| `setup_cmd.py` | Automate MCP server registration and Agent Skills symlink creation |
+| `uninstall_cmd.py` | Reverse `mpg setup`: deregister the MCP server and remove the Skills symlink |
+| `mcp_server.py` | MCP server — JSON-RPC 2.0 over stdio, zero external dependencies |
 
 ## Guide format
 
@@ -197,7 +201,7 @@ The CLI defaults to JSON when piped and human-readable when attached to a TTY. T
 
 | Layer | Scope | Categories | Count |
 |-------|-------|-----------|-------|
-| 1 — stdlib | Python standard library | typing, async, stdlib, data-structures | 16 |
+| 1 — stdlib | Python standard library | typing, async, stdlib, data-structures | 18 |
 | 2 — frameworks | Third-party frameworks | pydantic, fastapi, httpx, django, sqlalchemy, pytest | 18 |
 | 3 — toolchain | Development tools | toolchain | 5 |
 

{modern_python_guidance-0.4.4 → modern_python_guidance-0.4.5}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "modern-python-guidance"
-version = "0.4.4"
+version = "0.4.5"
 description = "Version-aware BAD/GOOD pattern guides that help AI coding agents generate modern Python"
 readme = "README.md"
 license = "Apache-2.0 OR MIT"

{modern_python_guidance-0.4.4 → modern_python_guidance-0.4.5}/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.4.4"
+__version__ = "0.4.5"

{modern_python_guidance-0.4.4 → modern_python_guidance-0.4.5}/src/modern_python_guidance/mcp_server.py

@@ -26,6 +26,30 @@ def _get_index() -> GuideIndex:
     return _index
 
 
+def _guide_limit() -> int:
+    return len(_get_index())
+
+
+def _get_tools() -> list[dict]:
+    n = _guide_limit()
+    tools = list(TOOLS)
+    tools[1] = {
+        **TOOLS[1],
+        "inputSchema": {
+            **TOOLS[1]["inputSchema"],
+            "properties": {
+                **TOOLS[1]["inputSchema"]["properties"],
+                "guide_ids": {
+                    **TOOLS[1]["inputSchema"]["properties"]["guide_ids"],
+                    "maxItems": n,
+                    "description": f"Guide IDs to retrieve (max {n})",
+                },
+            },
+        },
+    }
+    return tools
+
+
 # --- JSON-RPC framing (newline-delimited JSON) ---
 
 
@@ -122,8 +146,7 @@ TOOLS = [
                 "guide_ids": {
                     "type": "array",
                     "items": {"type": "string"},
-                    "description": "Guide IDs to retrieve (max 41)",
-                    "maxItems": 41,
+                    "description": "Guide IDs to retrieve",
                 },
                 "python_version": {
                     "type": "string",
@@ -270,8 +293,9 @@ def _tool_retrieve(arguments: dict) -> dict:
     guide_ids = arguments.get("guide_ids", [])
     if not guide_ids:
         return _tool_result("guide_ids is required and must not be empty", is_error=True)
-    if len(guide_ids) > 41:
-        return _tool_result("guide_ids exceeds maximum of 41", is_error=True)
+    limit = _guide_limit()
+    if len(guide_ids) > limit:
+        return _tool_result(f"guide_ids exceeds maximum of {limit}", is_error=True)
 
     pv = arguments.get("python_version")
     err = _validate_python_version(pv)
@@ -371,7 +395,7 @@ def _handle_request(msg: dict) -> dict | None:
         return None if is_notification else result
 
     if method == "tools/list":
-        result = _result_response(req_id, {"tools": TOOLS})
+        result = _result_response(req_id, {"tools": _get_tools()})
         return None if is_notification else result
 
     if method == "tools/call":

{modern_python_guidance-0.4.4 → modern_python_guidance-0.4.5}/tests/test_mcp_unit.py

@@ -217,16 +217,18 @@ class TestToolFunctions:
         assert data["results"][0]["id"] == "use-builtin-generics"
         assert data["not_found"][0]["id"] == "zzz-fake"
 
-    def test_retrieve_exactly_41_allowed(self):
-        ids = [f"fake-{i}" for i in range(41)]
+    def test_retrieve_at_limit_allowed(self):
+        limit = len(mcp._get_index())
+        ids = [f"fake-{i}" for i in range(limit)]
         r = mcp._tool_retrieve({"guide_ids": ids})
         assert r.get("isError") is not True
 
-    def test_retrieve_42_rejected(self):
-        ids = [f"fake-{i}" for i in range(42)]
+    def test_retrieve_over_limit_rejected(self):
+        limit = len(mcp._get_index())
+        ids = [f"fake-{i}" for i in range(limit + 1)]
         r = mcp._tool_retrieve({"guide_ids": ids})
         assert r["isError"] is True
-        assert "41" in r["content"][0]["text"]
+        assert str(limit) in r["content"][0]["text"]
 
     def test_retrieve_invalid_version(self):
         r = mcp._tool_retrieve({"guide_ids": ["use-builtin-generics"], "python_version": "x"})
@@ -318,6 +320,14 @@ class TestHandleRequest:
         assert "tools" in resp["result"]
         assert len(resp["result"]["tools"]) == 4
 
+    def test_tools_list_dynamic_max_items(self):
+        expected = len(mcp._get_index())
+        tools = mcp._get_tools()
+        retrieve = tools[1]
+        schema = retrieve["inputSchema"]["properties"]["guide_ids"]
+        assert schema["maxItems"] == expected
+        assert str(expected) in schema["description"]
+
     def test_tools_call(self):
         msg = {
             "jsonrpc": "2.0",