fix: let phase decorator reprs handle callables without __qualname__

repr() of a middleware built by before_request, after_response, on_error
or their async siblings no longer raises AttributeError for a
functools.partial or a callable instance; it falls back to repr(f).

Author: lesnik512

src/httpware/middleware/__init__.py

@@ -44,7 +44,7 @@ async def __call__(self, request: httpx2.Request, next: AsyncNext) -> httpx2.Res
             return await next(await f(request))
 
         def __repr__(self) -> str:
-            return f"<async_before_request({f.__qualname__})>"  # ty: ignore[unresolved-attribute]
+            return f"<async_before_request({getattr(f, '__qualname__', repr(f))})>"
 
     return _BeforeRequestMiddleware()
 
@@ -60,7 +60,7 @@ async def __call__(self, request: httpx2.Request, next: AsyncNext) -> httpx2.Res
             return await f(request, response)
 
         def __repr__(self) -> str:
-            return f"<async_after_response({f.__qualname__})>"  # ty: ignore[unresolved-attribute]
+            return f"<async_after_response({getattr(f, '__qualname__', repr(f))})>"
 
     return _AfterResponseMiddleware()
 
@@ -85,7 +85,7 @@ async def __call__(self, request: httpx2.Request, next: AsyncNext) -> httpx2.Res
                 return result
 
         def __repr__(self) -> str:
-            return f"<async_on_error({f.__qualname__})>"  # ty: ignore[unresolved-attribute]
+            return f"<async_on_error({getattr(f, '__qualname__', repr(f))})>"
 
     return _OnErrorMiddleware()
 
@@ -110,7 +110,7 @@ def __call__(self, request: httpx2.Request, next: Next) -> httpx2.Response:  # n
             return next(f(request))
 
         def __repr__(self) -> str:
-            return f"<before_request({f.__qualname__})>"  # ty: ignore[unresolved-attribute]
+            return f"<before_request({getattr(f, '__qualname__', repr(f))})>"
 
     return _BeforeRequestMiddleware()
 
@@ -126,7 +126,7 @@ def __call__(self, request: httpx2.Request, next: Next) -> httpx2.Response:  # n
             return f(request, response)
 
         def __repr__(self) -> str:
-            return f"<after_response({f.__qualname__})>"  # ty: ignore[unresolved-attribute]
+            return f"<after_response({getattr(f, '__qualname__', repr(f))})>"
 
     return _AfterResponseMiddleware()
 
@@ -151,6 +151,6 @@ def __call__(self, request: httpx2.Request, next: Next) -> httpx2.Response:  # n
                 return result
 
         def __repr__(self) -> str:
-            return f"<on_error({f.__qualname__})>"  # ty: ignore[unresolved-attribute]
+            return f"<on_error({getattr(f, '__qualname__', repr(f))})>"
 
     return _OnErrorMiddleware()

tests/test_middleware.py

@@ -1,7 +1,9 @@
 """Tests for the AsyncMiddleware protocol, AsyncNext type, chain composition, and decorators."""
 
 import asyncio
+import functools
 import typing
+from collections.abc import Callable
 from http import HTTPStatus
 
 import httpx2
@@ -189,3 +191,34 @@ def test_compose_get_type_hints_resolves_without_nameerror() -> None:
     """Sync mirror — sync `compose` get_type_hints must also resolve."""
     hints = typing.get_type_hints(compose)
     assert "middleware" in hints
+
+
+async def _passthrough(*args: typing.Any) -> typing.Any:  # noqa: ANN401
+    return args[-1]  # pragma: no cover — never invoked; the test only checks repr()
+
+
+class _CallableTransform:
+    async def __call__(self, *args: typing.Any) -> typing.Any:  # noqa: ANN401
+        return args[-1]  # pragma: no cover — never invoked; the test only checks repr()
+
+
+@pytest.mark.parametrize(
+    ("decorator", "name"),
+    [
+        (async_before_request, "async_before_request"),
+        (async_after_response, "async_after_response"),
+        (async_on_error, "async_on_error"),
+    ],
+)
+@pytest.mark.parametrize(
+    "factory",
+    [lambda: functools.partial(_passthrough), _CallableTransform],
+    ids=["partial", "callable_instance"],
+)
+def test_decorator_repr_falls_back_to_repr_without_qualname(
+    decorator: Callable[[typing.Any], AsyncMiddleware],
+    name: str,
+    factory: Callable[[], typing.Any],
+) -> None:
+    f = factory()
+    assert repr(decorator(f)) == f"<{name}({f!r})>"

tests/test_middleware_sync.py

@@ -1,5 +1,8 @@
 """Tests for the sync Middleware protocol, Next type, chain composition, and decorators."""
 
+import functools
+import typing
+from collections.abc import Callable
 from http import HTTPStatus
 
 import httpx2
@@ -197,3 +200,34 @@ def terminal_se(request: httpx2.Request) -> httpx2.Response:  # noqa: ARG001
     dispatch = compose((swallow_all,), terminal_se)
     with pytest.raises(SystemExit):
         dispatch(_make_request())
+
+
+def _passthrough(*args: typing.Any) -> typing.Any:  # noqa: ANN401
+    return args[-1]  # pragma: no cover — never invoked; the test only checks repr()
+
+
+class _CallableTransform:
+    def __call__(self, *args: typing.Any) -> typing.Any:  # noqa: ANN401
+        return args[-1]  # pragma: no cover — never invoked; the test only checks repr()
+
+
+@pytest.mark.parametrize(
+    ("decorator", "name"),
+    [
+        (before_request, "before_request"),
+        (after_response, "after_response"),
+        (on_error, "on_error"),
+    ],
+)
+@pytest.mark.parametrize(
+    "factory",
+    [lambda: functools.partial(_passthrough), _CallableTransform],
+    ids=["partial", "callable_instance"],
+)
+def test_decorator_repr_falls_back_to_repr_without_qualname(
+    decorator: Callable[[typing.Any], Middleware],
+    name: str,
+    factory: Callable[[], typing.Any],
+) -> None:
+    f = factory()
+    assert repr(decorator(f)) == f"<{name}({f!r})>"