entrygraph 0.1.41__tar.gz → 0.1.42__tar.gz

{entrygraph-0.1.41 → entrygraph-0.1.42}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: entrygraph
-Version: 0.1.41
+Version: 0.1.42
 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.41 → entrygraph-0.1.42}/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.41'
-__version_tuple__ = version_tuple = (0, 1, 41)
+__version__ = version = '0.1.42'
+__version_tuple__ = version_tuple = (0, 1, 42)
 
 __commit_id__ = commit_id = None

{entrygraph-0.1.41 → entrygraph-0.1.42}/src/entrygraph/detect/entrypoints/javascript.py

@@ -239,6 +239,47 @@ def _next_handlers(x: FileExtraction) -> list[EntrypointHint]:
 
 
 _HTTP_METHODS_UPPER = frozenset(m.upper() for m in _HTTP_METHODS)
+_NEXT_PAGES_EXTS = (".ts", ".tsx", ".js", ".jsx", ".mjs", ".cjs")
+
+
+def _next_pages_handlers(x: FileExtraction) -> list[EntrypointHint]:
+    """Next.js Pages Router API routes: every file under `pages/api/**` is a route
+    (file-based routing), whatever the default-export shape — a named `handler`
+    function, a `createNextApiHandler(router)` wrapper, or a bare arrow. The IR has
+    no `export default` marker to key on, so we rely on the file convention itself.
+
+    The handler dispatches on `req.method` internally, so the route carries no
+    specific verb. Bind to a `handler`-named function when present (the Pages
+    Router convention) so taint reaches the real body; otherwise the module.
+    """
+    normalized = f"/{x.path}"  # leading slash so a top-level `pages/` dir also matches
+    if "/pages/api/" not in normalized or not x.path.endswith(_NEXT_PAGES_EXTS):
+        return []
+    if x.path.endswith(".d.ts") or ".test." in x.path or ".spec." in x.path:
+        return []  # type decls and colocated tests aren't routes (#33)
+    tail = normalized.split("/pages/api/", 1)[1].rsplit(".", 1)[0]  # after prefix, no ext
+    if tail == "index":
+        tail = ""
+    elif tail.endswith("/index"):
+        tail = tail[: -len("/index")]
+    route = "/api" + (f"/{tail}" if tail else "")
+    handler = next(
+        (
+            s.qualified_name
+            for s in x.symbols
+            if s.kind in (SymbolKind.FUNCTION, SymbolKind.METHOD) and s.name == "handler"
+        ),
+        None,
+    )
+    return [
+        EntrypointHint(
+            rule_id="javascript.next.pages-route",
+            kind=EntrypointKind.HTTP_ROUTE,
+            handler_qualified_name=handler,
+            route=route,
+            framework="next",
+        )
+    ]
 
 
 register(
@@ -323,3 +364,12 @@ register(
         "javascript.next.route", "javascript", "next", EntrypointKind.HTTP_ROUTE, _next_handlers
     )
 )
+register(
+    EntrypointRule(
+        "javascript.next.pages-route",
+        "javascript",
+        "next",
+        EntrypointKind.HTTP_ROUTE,
+        _next_pages_handlers,
+    )
+)

{entrygraph-0.1.41 → entrygraph-0.1.42}/tests/test_entrypoints_javascript.py

@@ -73,3 +73,55 @@ def test_next_ignores_non_method_symbols_and_non_route_files():
     assert _next_rule().match(_js_ext([_sym("helper", SymbolKind.VARIABLE)])) == []
     off_route = _js_ext([_sym("GET", SymbolKind.VARIABLE)], path="apps/web/app/api/me/page.ts")
     assert _next_rule().match(off_route) == []
+
+
+def _pages_rule():
+    return {r.id: r for r in rules_for("javascript", {"next"})}["javascript.next.pages-route"]
+
+
+def _handler_sym(qname="apps.web.pages.api.book.event.handler"):
+    return RawSymbol(
+        kind=SymbolKind.FUNCTION, name="handler", qualified_name=qname, span=Span(1, 0, 1, 40)
+    )
+
+
+def test_next_pages_route_binds_named_handler():
+    # Pages Router is file-based: pages/api/book/event.ts is the route /api/book/event,
+    # and the conventional `export default function handler` binds it (#37).
+    ext = _js_ext([_handler_sym()], path="apps/web/pages/api/book/event.ts")
+    hints = _pages_rule().match(ext)
+    assert len(hints) == 1
+    h = hints[0]
+    assert h.kind is EntrypointKind.HTTP_ROUTE
+    assert h.route == "/api/book/event"
+    assert h.http_methods == []  # method dispatched inside the handler
+    assert h.handler_qualified_name == "apps.web.pages.api.book.event.handler"
+
+
+def test_next_pages_route_without_handler_falls_back_to_module():
+    # `export default createNextApiHandler(router)` (tRPC) defines no handler symbol;
+    # the route is still detected, left unbound for the scanner to anchor on module.
+    ext = _js_ext(path="apps/web/pages/api/trpc/[trpc].ts")
+    hint = _pages_rule().match(ext)[0]
+    assert hint.route == "/api/trpc/[trpc]"
+    assert hint.handler_qualified_name is None
+
+
+def test_next_pages_index_files_map_to_parent():
+    foo = _pages_rule().match(_js_ext(path="apps/web/pages/api/foo/index.ts"))[0]
+    assert foo.route == "/api/foo"
+    assert _pages_rule().match(_js_ext(path="pages/api/index.ts"))[0].route == "/api"
+
+
+def test_next_pages_excludes_tests_and_type_decls():
+    for path in (
+        "apps/web/pages/api/book/recurring-event.test.ts",
+        "apps/web/pages/api/foo.spec.ts",
+        "apps/web/pages/api/types.d.ts",
+    ):
+        assert _pages_rule().match(_js_ext(path=path)) == []
+
+
+def test_next_pages_ignores_non_api_pages():
+    # A React page under pages/ (not pages/api/) is not an API route.
+    assert _pages_rule().match(_js_ext(path="apps/web/pages/about.tsx")) == []