django-orm-lens 1.2.0__tar.gz → 1.2.1__tar.gz

{django_orm_lens-1.2.0 → django_orm_lens-1.2.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: django-orm-lens
-Version: 1.2.0
+Version: 1.2.1
 Summary: Static analysis + MCP server for Django models. Sidebar tree, ER diagrams, and JSON output for terminals and AI coding agents.
 Author: frowningdev
 License: MIT

{django_orm_lens-1.2.0 → django_orm_lens-1.2.1}/django_orm_lens/cli.py

@@ -23,6 +23,8 @@ from typing import List, Optional, Sequence
 from . import __version__
 from .diff import diff_schemas, format_diff
 from .formatters import format_hover, format_index, format_model
+from .migrations_parser import analyze_migration_risks
+from .query_analyzer import scan_for_nplusone
 from .models import (
     ParsedModel,
     WorkspaceIndex,
@@ -205,8 +207,10 @@ def _cmd_diff(args: argparse.Namespace) -> int:
 def _cmd_nplusone(args: argparse.Namespace) -> int:
     """Static N+1 detector — flag FK/M2M access inside loops without select/prefetch."""
     idx = _load_index(args)
-    schema = _build_schema_from_index(idx)
-    findings = scan_for_nplusone(args.path, schema)
+    # scan_for_nplusone accepts a WorkspaceIndex directly (see
+    # query_analyzer._normalise_schema) — no need to pre-flatten. Passing idx
+    # avoids importing the private _build_schema_from_index helper here.
+    findings = scan_for_nplusone(args.path, idx)
     if args.confidence == "high":
         findings = [f for f in findings if f.confidence == "high"]
     elif args.confidence == "medium":
@@ -248,7 +252,7 @@ def _cmd_migration_risk(args: argparse.Namespace) -> int:
             print("No risky migration operations found.")
         for f in findings:
             print(
-                f"{f.filePath}:{f.lineNumber}  [{f.severity}/{f.confidence}]  "
+                f"{f.file_path}:{f.line_number}  [{f.severity}/{f.confidence}]  "
                 f"{f.operation} {f.app}.{f.model}"
                 + (f".{f.field}" if f.field else "")
             )

{django_orm_lens-1.2.0 → django_orm_lens-1.2.1}/django_orm_lens.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: django-orm-lens
-Version: 1.2.0
+Version: 1.2.1
 Summary: Static analysis + MCP server for Django models. Sidebar tree, ER diagrams, and JSON output for terminals and AI coding agents.
 Author: frowningdev
 License: MIT

{django_orm_lens-1.2.0 → django_orm_lens-1.2.1}/django_orm_lens.egg-info/SOURCES.txt

@@ -19,6 +19,7 @@ django_orm_lens.egg-info/requires.txt
 django_orm_lens.egg-info/top_level.txt
 tests/test_abstract_and_auth_user.py
 tests/test_cli_hint.py
+tests/test_cli_subcommands_smoke.py
 tests/test_cli_verbose.py
 tests/test_cross_module_bases.py
 tests/test_diff_command.py

{django_orm_lens-1.2.0 → django_orm_lens-1.2.1}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "django-orm-lens"
-version = "1.2.0"
+version = "1.2.1"
 description = "Static analysis + MCP server for Django models. Sidebar tree, ER diagrams, and JSON output for terminals and AI coding agents."
 readme = "README.md"
 requires-python = ">=3.9"

django_orm_lens-1.2.1/tests/test_cli_subcommands_smoke.py

@@ -0,0 +1,118 @@
+"""Regression: every CLI subcommand must import and execute cleanly.
+
+The v0.6.0 → v0.7.0 audit surfaced two bugs where a subcommand's body
+referenced module-level names that were never imported (or that used the
+wrong attribute case) — the unit tests still passed because they exercised
+the helper functions directly, never the argparse dispatch path.
+
+This suite runs each subcommand end-to-end via ``main([...])`` against the
+miniapp fixture and the migration_risks fixture. Any `NameError`,
+`ImportError`, or `AttributeError` in a subcommand body surfaces as a
+test failure instead of a crash the next user sees.
+"""
+
+from __future__ import annotations
+
+import contextlib
+import io
+import json as _json
+import unittest
+from pathlib import Path
+
+from django_orm_lens.cli import main
+
+FIXTURE_ROOT = Path(__file__).parent / "fixtures"
+MINIAPP = FIXTURE_ROOT / "miniapp"
+MIGRATION_RISKS = FIXTURE_ROOT / "migration_risks"
+
+
+def _run(argv):
+    out, err = io.StringIO(), io.StringIO()
+    with contextlib.redirect_stdout(out), contextlib.redirect_stderr(err):
+        exit_code = main(argv)
+    return exit_code, out.getvalue(), err.getvalue()
+
+
+class SubcommandsSmokeTest(unittest.TestCase):
+    """Every subcommand runs without a NameError / ImportError / AttributeError."""
+
+    def test_scan_runs(self) -> None:
+        code, out, _err = _run(
+            ["scan", "--path", str(MINIAPP), "--format", "table"]
+        )
+        self.assertEqual(code, 0)
+        self.assertIn("Order", out)
+
+    def test_list_runs(self) -> None:
+        code, out, _err = _run(["list", "--path", str(MINIAPP)])
+        self.assertEqual(code, 0)
+        self.assertIn("orders.Order", out)
+
+    def test_er_runs(self) -> None:
+        code, out, _err = _run(["er", "--path", str(MINIAPP)])
+        self.assertEqual(code, 0)
+        self.assertIn("erDiagram", out)
+
+    def test_describe_runs(self) -> None:
+        code, out, _err = _run(
+            ["describe", "orders.Order", "--path", str(MINIAPP)]
+        )
+        self.assertEqual(code, 0)
+        self.assertIn("Order", out)
+
+    def test_hover_runs(self) -> None:
+        code, out, _err = _run(
+            ["hover", "orders.Order", "--path", str(MINIAPP)]
+        )
+        self.assertEqual(code, 0)
+        self.assertIn("Order", out)
+
+    def test_nplusone_runs(self) -> None:
+        # Miniapp has no n+1 patterns — but the subcommand must dispatch
+        # without a NameError (the pre-v0.7 bug was scan_for_nplusone +
+        # _build_schema_from_index not being imported at module level).
+        code, out, _err = _run(
+            ["nplusone", "--path", str(MINIAPP), "--exit-zero"]
+        )
+        self.assertEqual(code, 0)
+        self.assertIn("N+1", out)
+
+    def test_migration_risk_text_format(self) -> None:
+        # Text format previously crashed with AttributeError because the
+        # print used camelCase attrs (filePath / lineNumber) instead of the
+        # snake_case dataclass fields (file_path / line_number).
+        code, out, _err = _run(
+            [
+                "migration-risk",
+                "--path",
+                str(MIGRATION_RISKS / "add_field_notnull"),
+                "--exit-zero",
+            ]
+        )
+        self.assertEqual(code, 0)
+        # A finding should be printed with "file:line [severity/confidence]".
+        self.assertTrue(
+            any(":" in line and "[" in line for line in out.splitlines()),
+            f"expected 'file:line [severity/confidence]' line in output, got: {out!r}",
+        )
+
+    def test_migration_risk_json_format(self) -> None:
+        # JSON path uses to_dict() — separate from the text path bug,
+        # but worth pinning too.
+        code, out, _err = _run(
+            [
+                "migration-risk",
+                "--path",
+                str(MIGRATION_RISKS / "add_field_notnull"),
+                "--format",
+                "json",
+                "--exit-zero",
+            ]
+        )
+        self.assertEqual(code, 0)
+        payload = _json.loads(out)
+        self.assertIn("findings", payload)
+
+
+if __name__ == "__main__":
+    unittest.main()