entrygraph 0.1.34__tar.gz → 0.1.35__tar.gz

{entrygraph-0.1.34 → entrygraph-0.1.35}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: entrygraph
-Version: 0.1.34
+Version: 0.1.35
 Summary: Language-agnostic code graph: query symbols, entrypoints, and source-to-sink call paths from a SQLite index
 Project-URL: Repository, https://github.com/brettbergin/entrygraph
 Author-email: Brett Bergin <brettberginbc@yahoo.com>

{entrygraph-0.1.34 → entrygraph-0.1.35}/src/entrygraph/_version.py

@@ -18,7 +18,7 @@ version_tuple: tuple[int | str, ...]
 commit_id: str | None
 __commit_id__: str | None
 
-__version__ = version = '0.1.34'
-__version_tuple__ = version_tuple = (0, 1, 34)
+__version__ = version = '0.1.35'
+__version_tuple__ = version_tuple = (0, 1, 35)
 
 __commit_id__ = commit_id = None

{entrygraph-0.1.34 → entrygraph-0.1.35}/src/entrygraph/detect/entrypoints/ruby.py

@@ -21,6 +21,13 @@ _SINATRA_VERBS = frozenset({"get", "post", "put", "delete", "patch"})
 _RAILS_VERBS = frozenset({"get", "post", "put", "patch", "delete", "resources", "resource", "root"})
 
 
+def _is_rails_routes_file(path: str) -> bool:
+    """The main `config/routes.rb` plus split route files loaded via `draw(:x)`
+    (`config/routes/api.rb`, `config/routes/admin.rb`, ...). Missing the split
+    files hid most of the route surface on large Rails apps (mastodon/forem)."""
+    return path.endswith("config/routes.rb") or "/config/routes/" in f"/{path}"
+
+
 def _sinatra_routes(x: FileExtraction) -> list[EntrypointHint]:
     hints = []
     for ref in x.references:
@@ -46,7 +53,7 @@ def _sinatra_routes(x: FileExtraction) -> list[EntrypointHint]:
 
 
 def _rails_routes(x: FileExtraction) -> list[EntrypointHint]:
-    if not x.path.endswith("config/routes.rb"):
+    if not _is_rails_routes_file(x.path):
         return []
     hints = []
     for ref in x.references:

{entrygraph-0.1.34 → entrygraph-0.1.35}/tests/test_entrypoint_expansion.py

@@ -402,3 +402,41 @@ def test_nestjs_decorator_routes_with_controller_prefix():
     hints = {h.http_methods[0]: h.route for h in rules["javascript.nestjs.route"].match(x)}
     assert hints["GET"] == "/users/:id"  # controller prefix + method path
     assert hints["POST"] == "/users"
+
+
+def _ruby_ext(references, path):
+    return FileExtraction(
+        path=path,
+        language="ruby",
+        module_path=path.replace("/", ".").removesuffix(".rb"),
+        parse_ok=True,
+        error_count=0,
+        symbols=[],
+        references=list(references),
+    )
+
+
+def _ruby_route_ref(verb, route):
+    return RawReference(
+        kind="call",
+        callee_text=verb,
+        callee_name=verb,
+        receiver_text=None,
+        span=SPAN,
+        caller_qualified_name=None,
+        arg_preview=f"('{route}', to: 'x#y')",
+    )
+
+
+def test_rails_routes_read_from_split_route_files():
+    # Rails apps split routes into config/routes/*.rb loaded via `draw(:api)`; the
+    # rule must read those, not only the top-level config/routes.rb (#37 / F-H22).
+    from entrygraph.detect.entrypoints import rules_for
+
+    rule = {r.id: r for r in rules_for("ruby", {"rails"})}["ruby.rails.routes"]
+    main = rule.match(_ruby_ext([_ruby_route_ref("get", "/health")], "config/routes.rb"))
+    split = rule.match(_ruby_ext([_ruby_route_ref("post", "/users")], "config/routes/api.rb"))
+    unrelated = rule.match(_ruby_ext([_ruby_route_ref("get", "/x")], "app/models/user.rb"))
+    assert [h.route for h in main] == ["/health"]
+    assert [h.route for h in split] == ["/users"]  # the split file is now read
+    assert unrelated == []  # a non-routes file is still ignored