django-orm-lens 1.2.5__tar.gz → 1.2.7__tar.gz

{django_orm_lens-1.2.5 → django_orm_lens-1.2.7}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: django-orm-lens
-Version: 1.2.5
+Version: 1.2.7
 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.5 → django_orm_lens-1.2.7}/django_orm_lens/parser.py

@@ -398,7 +398,17 @@ def _collect_defs(file_path: str, content: str) -> List[ParsedModel]:
     result with definitions from other files before calling
     ``_resolve_and_filter``.
     """
-    lines = content.splitlines()
+    # Strip leading BOM (U+FEFF) — Windows editors (Notepad, VS Code with
+    # certain settings) save UTF-8 files with a byte-order mark. Without this
+    # the first line becomes "class Foo..." and CLASS_RE fails to match.
+    if content.startswith(""):
+        content = content[1:]
+    # Expand tab characters to 4 spaces so the ``\s{indent}`` prefix used in
+    # FIELD_RE / BARE_FIELD_RE / META_ITEM_RE matches tab-indented code too.
+    # A single ``\t`` character otherwise contributes 1 to ``\s`` even though
+    # it's visually 4 columns — tab-indented models used to have zero fields
+    # detected. Line numbers are preserved (per-line expansion only).
+    lines = [line.expandtabs(4) for line in content.splitlines()]
     all_defs: List[ParsedModel] = []
     parent = os.path.basename(os.path.dirname(file_path))
     if parent == "models":

{django_orm_lens-1.2.5 → django_orm_lens-1.2.7}/django_orm_lens.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: django-orm-lens
-Version: 1.2.5
+Version: 1.2.7
 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.5 → django_orm_lens-1.2.7}/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_aliased_module_and_third_party_fields.py
+tests/test_bom_prefix.py
 tests/test_cli_hint.py
 tests/test_cli_subcommands_smoke.py
 tests/test_cli_verbose.py
@@ -35,4 +36,5 @@ tests/test_pep526_type_hints.py
 tests/test_pep695_generic_classes.py
 tests/test_query_analyzer.py
 tests/test_scan_root.py
-tests/test_signals_parser.py
+tests/test_signals_parser.py
+tests/test_tab_indented_models.py

{django_orm_lens-1.2.5 → django_orm_lens-1.2.7}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "django-orm-lens"
-version = "1.2.5"
+version = "1.2.7"
 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.7/tests/test_bom_prefix.py

@@ -0,0 +1,40 @@
+"""Regression: leading BOM (U+FEFF) must be stripped before parsing.
+
+Windows editors - Notepad, older Sublime, VS Code with certain UTF-8
+encoding settings - save files with a byte-order mark. Without this
+fix the first line becomes "class Foo..." and CLASS_RE fails
+to match. Result: zero models parsed for the whole file, silent.
+"""
+
+from __future__ import annotations
+
+import unittest
+
+from django_orm_lens.parser import parse_models_file
+
+BOM = ""
+
+
+class BomPrefixTest(unittest.TestCase):
+    def test_bom_prefixed_file_parses(self) -> None:
+        src = (
+            BOM
+            + "class Article(models.Model):\n"
+            "    title = models.CharField(max_length=100)\n"
+            "    body = models.TextField()\n"
+        )
+        models = parse_models_file("blog/models.py", src)
+        self.assertEqual(len(models), 1)
+        self.assertEqual(models[0].name, "Article")
+        self.assertEqual(
+            [f.name for f in models[0].fields], ["title", "body"]
+        )
+
+    def test_no_bom_still_works(self) -> None:
+        src = "class A(models.Model):\n    x = models.CharField(max_length=5)\n"
+        models = parse_models_file("t/models.py", src)
+        self.assertEqual([f.name for f in models[0].fields], ["x"])
+
+
+if __name__ == "__main__":
+    unittest.main()

django_orm_lens-1.2.7/tests/test_tab_indented_models.py

@@ -0,0 +1,54 @@
+"""Regression: tab-indented model bodies must parse.
+
+Editors that default to tabs (or projects that use PEP-8 exceptions with
+tabs) produced empty models - every field silently dropped. Root cause:
+``FIELD_RE`` uses ``\\s{indent}`` as its column prefix and
+``_detect_class_indent`` expands tabs to width 4, but a single ``\\t``
+character is only one match for ``\\s``, not four. Fixed by pre-expanding
+tabs to spaces in the line buffer before regex matching.
+"""
+
+from __future__ import annotations
+
+import unittest
+
+from django_orm_lens.parser import parse_models_file
+
+
+class TabIndentationTest(unittest.TestCase):
+    def test_pure_tab_indented_body(self) -> None:
+        src = (
+            "class Post(models.Model):\n"
+            "\ttitle = models.CharField(max_length=100)\n"
+            "\tauthor = models.CharField(max_length=100)\n"
+            "\tbody = models.TextField()\n"
+        )
+        models = parse_models_file("blog/models.py", src)
+        self.assertEqual(len(models), 1)
+        self.assertEqual(
+            [f.name for f in models[0].fields],
+            ["title", "author", "body"],
+        )
+
+    def test_mixed_tab_then_space(self) -> None:
+        # Editors that mid-migrate to tabs sometimes end up with a tab
+        # followed by a space; that used to fail equally silently.
+        src = (
+            "class M(models.Model):\n"
+            "\t x = models.CharField(max_length=5)\n"
+        )
+        models = parse_models_file("t/models.py", src)
+        self.assertEqual([f.name for f in models[0].fields], ["x"])
+
+    def test_space_indented_still_works(self) -> None:
+        # Backward-compat guard.
+        src = (
+            "class M(models.Model):\n"
+            "    x = models.CharField(max_length=5)\n"
+        )
+        models = parse_models_file("t/models.py", src)
+        self.assertEqual([f.name for f in models[0].fields], ["x"])
+
+
+if __name__ == "__main__":
+    unittest.main()