patchwork-conventions 0.1.2__tar.gz → 0.1.4__tar.gz

{patchwork_conventions-0.1.2 → patchwork_conventions-0.1.4}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: patchwork-conventions
-Version: 0.1.2
+Version: 0.1.4
 Summary: Mine your codebase. Generate CONVENTIONS.md. Stop AI agents from making up your style.
 Author: patchwork contributors
 License: MIT

{patchwork_conventions-0.1.2 → patchwork_conventions-0.1.4}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "patchwork-conventions"
-version = "0.1.2"
+version = "0.1.4"
 description = "Mine your codebase. Generate CONVENTIONS.md. Stop AI agents from making up your style."
 readme = "README.md"
 license = { text = "MIT" }

{patchwork_conventions-0.1.2 → patchwork_conventions-0.1.4}/src/patchwork/mcp/server.py

@@ -36,7 +36,7 @@ def _get_or_scan(root: Path) -> ConventionReport:
     cached = _CACHE.get(key)
     if cached is not None:
         return cached
-    opts = ScanOptions(root=root, max_files=300)
+    opts = ScanOptions(root=root, max_files=500)
     report = do_scan(opts)
     _CACHE[key] = report
     return report
@@ -330,6 +330,8 @@ async def run_server(root: Path, port: int = 3742, stdio: bool = True) -> None:
                     lines.append(f"  logging: {er.logging_framework}")
                 if er.propagation_style:
                     lines.append(f"  propagation: {er.propagation_style}")
+                if er.custom_exceptions:
+                    lines.append(f"  custom exceptions: {', '.join(er.custom_exceptions[:6])}")
                 for note in er.notes:
                     lines.append(f"  note: {note}")
             return [types.TextContent(type="text", text="\n".join(lines) or "No error patterns found.")]
@@ -385,8 +387,13 @@ async def run_server(root: Path, port: int = 3742, stdio: bool = True) -> None:
             return [types.TextContent(type="text", text="\n".join(lines))]
 
         elif name == "patchwork_check":
-            sym = arguments["name"]
-            kind = arguments["kind"]
+            sym = arguments.get("name")
+            kind = arguments.get("kind")
+            if not sym or not kind:
+                return [types.TextContent(
+                    type="text",
+                    text="Error: 'name' and 'kind' are required arguments.",
+                )]
             lang = arguments.get("language", "")
             return [types.TextContent(
                 type="text",

{patchwork_conventions-0.1.2 → patchwork_conventions-0.1.4}/src/patchwork/miners/api_patterns.py

@@ -34,10 +34,23 @@ _RESP_DATA_ERROR = re.compile(r'["\'](?:data|error)["\']', re.IGNORECASE)
 _RESP_SUCCESS_DATA = re.compile(r'["\']success["\'].*["\']data["\']', re.DOTALL | re.IGNORECASE)
 _RESP_RESULT = re.compile(r'["\']result["\']', re.IGNORECASE)
 
-# Route parameter styles
-_ROUTE_COLON = re.compile(r'(?:app|router)\.\w+\(["\'][^"\']*:\w+')   # Express :id
-_ROUTE_BRACE = re.compile(r'(?:path|url)\s*=\s*["\'][^"\']*\{[a-zA-Z_]+\}')  # FastAPI {id}
-_ROUTE_ANGLE = re.compile(r'(?:app|blueprint)\.\w+\(["\'][^"\']*<[a-zA-Z_:]+>')   # Flask <id>
+# Route parameter styles — match route strings passed to known route-registration calls.
+# We look for the route string ONLY when it appears as an argument to a route decorator
+# or router method, to avoid matching f-strings, format strings, and other /path usages.
+
+# FastAPI/Starlette: @app.get("/items/{item_id}") or @router.post("/x/{id:int}")
+_ROUTE_BRACE = re.compile(
+    r'(?:@?\w+\.(?:get|post|put|patch|delete|websocket|route|add_api_route)\s*\()'
+    r'\s*[f]?["\'][^"\']*\{[a-zA-Z_]\w*(?::[a-zA-Z]+)?\}'
+)
+# Flask: @app.route("/items/<int:item_id>")
+_ROUTE_ANGLE = re.compile(
+    r'(?:@?\w+\.route\s*\()\s*[f]?["\'][^"\']*<(?:[a-zA-Z_]+:)?[a-zA-Z_]\w*>'
+)
+# Express: router.get("/:id") or app.use("/items/:id")
+_ROUTE_COLON = re.compile(
+    r'(?:\w+\.(?:get|post|put|patch|delete|use|all)\s*\()\s*["\'][^"\']*(?<!\{):(?!\w*\})[a-zA-Z_]\w*'
+)
 
 # ORM signals
 _ORM_SIGNALS = {
@@ -57,7 +70,8 @@ _FRAMEWORK_SIGNALS = {
     "FastAPI": [r"from fastapi import", r"@app\.get\(", r"@router\."],
     "Flask": [r"from flask import", r"@app\.route\(", r"Blueprint\("],
     "Django": [r"from django", r"urlpatterns\s*=", r"HttpResponse"],
-    "Express": [r"require\(['\"]express['\"]", r"app\.use\(", r"router\.get\("],
+    # For Express: require both the require() AND usage — router.get alone is too generic
+    "Express": [r"require\(['\"]express['\"]\)", r"express\(\)"],
     "Fastify": [r"require\(['\"]fastify['\"]", r"fastify\.register"],
     "Hono": [r"from ['\"]hono['\"]", r"new Hono\("],
     "Gin": [r"\bgin\b.*\bDefault\(\)", r"r\.GET\(", r"c\.JSON\("],
@@ -105,9 +119,10 @@ def _detect_apis(paths: list[Path], lang: str) -> APIResult:
     data_error = 0
     success_data = 0
     result_shape = 0
-    colon_routes = 0
-    brace_routes = 0
-    angle_routes = 0
+    # Track route style per FILE (not per match) to avoid single-file anomalies
+    colon_route_files = 0
+    brace_route_files = 0
+    angle_route_files = 0
     # Track per-file hits (not per-match) to avoid false positives from
     # code that merely mentions framework names in strings, comments, or docs.
     orm_files: Counter[str] = Counter()
@@ -127,9 +142,9 @@ def _detect_apis(paths: list[Path], lang: str) -> APIResult:
         success_data += len(_RESP_SUCCESS_DATA.findall(text))
         result_shape += len(_RESP_RESULT.findall(text))
 
-        colon_routes += len(_ROUTE_COLON.findall(text))
-        brace_routes += len(_ROUTE_BRACE.findall(text))
-        angle_routes += len(_ROUTE_ANGLE.findall(text))
+        if _ROUTE_COLON.search(text): colon_route_files += 1
+        if _ROUTE_BRACE.search(text): brace_route_files += 1
+        if _ROUTE_ANGLE.search(text): angle_route_files += 1
 
         # Skip files that are themselves pattern-definition files (e.g. this miner)
         # to avoid self-matching on our own regex strings.
@@ -175,17 +190,17 @@ def _detect_apis(paths: list[Path], lang: str) -> APIResult:
     elif result_shape > 3:
         response_shape = "{result}"
 
-    # Route param style
-    route_total = colon_routes + brace_routes + angle_routes
+    # Route param style — pick the style seen in the most files.
+    # Require at least 2 files to avoid single-file anomalies.
     route_style = None
-    if route_total > 0:
-        best = max(colon_routes, brace_routes, angle_routes)
-        if colon_routes == best:
-            route_style = ":id (Express style)"
-        elif brace_routes == best:
-            route_style = "{id} (FastAPI style)"
-        else:
+    best = max(brace_route_files, angle_route_files, colon_route_files)
+    if best >= 2:
+        if brace_route_files == best:
+            route_style = "{id} (FastAPI/Starlette style)"
+        elif angle_route_files == best:
             route_style = "<id> (Flask style)"
+        else:
+            route_style = ":id (Express style)"
 
     return APIResult(
         response_shape=response_shape,

{patchwork_conventions-0.1.2 → patchwork_conventions-0.1.4}/src/patchwork_conventions.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: patchwork-conventions
-Version: 0.1.2
+Version: 0.1.4
 Summary: Mine your codebase. Generate CONVENTIONS.md. Stop AI agents from making up your style.
 Author: patchwork contributors
 License: MIT

patchwork_conventions-0.1.4/src/patchwork_conventions.egg-info/top_level.txt

@@ -0,0 +1 @@
+patchwork

patchwork_conventions-0.1.2/src/patchwork_conventions.egg-info/top_level.txt

@@ -1,5 +0,0 @@
-analyzers
-mcp
-miners
-output
-patchwork