Separate normal and wrapper hook implementations with distinct types

This change introduces type-safe separation between normal hook implementations and wrapper implementations by:

- Creating WrapperImpl subclass for wrapper/hookwrapper implementations
- Modifying _multicall to handle normal and wrapper implementations separately
- Updating hook callers to maintain separate lists and enforce type constraints
- Adding factory method in HookimplConfiguration for proper type instantiation
- Ensuring historic hooks reject wrapper implementations at registration time

The separation improves type safety, clarifies execution flow, and maintains backward compatibility while providing better error messages for misuse.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>

Author: RonnyPfannschmidt

src/pluggy/_callers.py

@@ -13,6 +13,7 @@
 
 from ._hook_callers import HookImpl
 from ._hook_callers import WrapperImpl
+from ._hook_callers import WrapperImpl
 from ._result import Result
 from ._warnings import PluggyTeardownRaisedWarning
 

src/pluggy/_hook_callers.py

@@ -60,7 +60,12 @@ def _insert_hookimpl_into_list(
         target_list.insert(i + 1, hookimpl)
 
 
-def _insert_hookimpl_into_list(hookimpl: HookImpl, target_list: list[HookImpl]) -> None:
+_T_HookImpl = TypeVar("_T_HookImpl", bound="HookImpl")
+
+
+def _insert_hookimpl_into_list(
+    hookimpl: _T_HookImpl, target_list: MutableSequence[_T_HookImpl]
+) -> None:
     """Insert a hookimpl into the target list maintaining proper ordering.
 
     The ordering is: [trylast, normal, tryfirst]
@@ -549,9 +554,11 @@ def __call__(self, **kwargs: object) -> Any:
         if self.spec:
             self.spec.verify_all_args_are_provided(kwargs)
         firstresult = self.spec.config.firstresult if self.spec else False
-        # Get the hookexec from the original
         hookexec = getattr(self._orig, "_hookexec")
-        return hookexec(self.name, self.get_hookimpls(), kwargs, firstresult)
+
+        normal_impls = self._get_filtered(self._orig._normal_hookimpls)
+        wrapper_impls = self._get_filtered(self._orig._wrapper_hookimpls)
+        return hookexec(self.name, normal_impls, wrapper_impls, kwargs, firstresult)
 
         normal_impls = self._get_filtered(self._orig._normal_hookimpls)
         wrapper_impls = self._get_filtered(self._orig._wrapper_hookimpls)
@@ -576,9 +583,7 @@ def call_historic(
         if self.spec:
             self.spec.verify_all_args_are_provided(kwargs)
 
-        # If the original is a HistoricHookCaller, add to its history
-        if hasattr(self._orig, "_call_history"):
-            self._orig._call_history.append((kwargs, result_callback))
+        self._orig._call_history.append((kwargs, result_callback))
 
         # Execute with filtered hookimpls (historic hooks don't support wrappers)
         hookexec = getattr(self._orig, "_hookexec")
@@ -631,7 +636,6 @@ def __repr__(self) -> str:
 _SubsetHookCaller = SubsetHookCaller
 
 
-@final
 class HookImpl:
     """Base class for hook implementations in a :class:`HookCaller`."""
 
@@ -648,6 +652,7 @@ class HookImpl:
         "trylast",
         "hookimpl_config",
     )
+
     function: Final[_HookImplFunction[object]]
     argnames: Final[tuple[str, ...]]
     kwargnames: Final[tuple[str, ...]]