refactor(live_component): keep _AutoRefresh.__name__ honest via __coco_subpath_name__

The previous implementation overrode `_AutoRefresh.__name__` to the wrapped
fn's name so `coco.mount(auto_refresh(fn, ...))` would derive the same
subpath as `coco.mount(fn)`. That conflated subpath identity with class
identity — tracebacks and `processor_name` in the exception handler
reported the wrapper as if it were the wrapped fn, hiding the live-loop
layer when something went wrong inside the scaffold.

Introduce an opt-in `__coco_subpath_name__` attribute that
`mount`/`use_mount`/`mount_each` consult before falling back to
`__name__`. `auto_refresh` now keeps `_AutoRefresh.__name__` accurate
and only overrides the subpath via the new attribute. Same user-visible
subpath; honest class identity for debugging and for any future wrapper
(or user-defined LiveComponent) that wants the same behavior.

Author: georgeh0

python/cocoindex/_internal/api.py

@@ -112,6 +112,23 @@
 _ChildHandlerT = TypeVar("_ChildHandlerT", bound="TargetHandler[Any, Any, Any] | None")
 
 
+def _default_subpath_name(processor_fn: Any) -> str | None:
+    """Resolve the default subpath name for a mount target.
+
+    Honors an explicit ``__coco_subpath_name__`` attribute (set by wrappers
+    like ``coco.auto_refresh`` so the wrapper class can keep an honest
+    ``__name__`` while still mounting under the wrapped function's name),
+    falling back to ``__name__``.
+    """
+    name = getattr(processor_fn, "__coco_subpath_name__", None)
+    if isinstance(name, str):
+        return name
+    name = getattr(processor_fn, "__name__", None)
+    if isinstance(name, str):
+        return name
+    return None
+
+
 class ComponentMountHandle:
     """Handle for processing unit(s) started with `mount()` or `mount_each()`. Allows waiting until ready."""
 
@@ -223,7 +240,7 @@ async def use_mount(*pos_args: Any, **kwargs: Any) -> Any:
     else:
         processor_fn = pos_args[0]
         args = pos_args[1:]
-        name = getattr(processor_fn, "__name__", None)
+        name = _default_subpath_name(processor_fn)
         if name is None:
             raise TypeError(
                 "use_mount() requires a ComponentSubpath when the function has no "
@@ -333,7 +350,7 @@ async def mount(*pos_args: Any, **kwargs: Any) -> ComponentMountHandle:
     else:
         processor_fn = pos_args[0]
         args = pos_args[1:]
-        name = getattr(processor_fn, "__name__", None)
+        name = _default_subpath_name(processor_fn)
         if name is None:
             raise TypeError(
                 "mount() requires a ComponentSubpath when the function has no "
@@ -423,7 +440,7 @@ async def mount_each(*pos_args: Any, **kwargs: Any) -> ComponentMountHandle:
         fn = pos_args[0]
         items = pos_args[1]
         extra_args = pos_args[2:]
-        name = getattr(fn, "__name__", None)
+        name = _default_subpath_name(fn)
         if name is None:
             raise TypeError(
                 "mount_each() requires a ComponentSubpath when the function has no "

python/cocoindex/_internal/live_component.py

@@ -586,6 +586,5 @@ async def process_live(self, operator: LiveComponentOperator) -> None:
                 await asyncio.sleep(sleep_seconds)
                 await operator.update_full()
 
-    _AutoRefresh.__name__ = f"AutoRefresh[{fn_name}]"
-    _AutoRefresh.__qualname__ = _AutoRefresh.__name__
+    _AutoRefresh.__coco_subpath_name__ = fn_name  # type: ignore[attr-defined]
     return _AutoRefresh