hooks(refactor): remove redundant bulk operations methods

why: These methods are redundant with existing API
what:
- Remove get_hook_indices: use show_hook().keys() instead
- Remove get_hook_values: use show_hook() directly (returns SparseArray)
- Remove clear_hook: use unset_hook() without index (clears all)
- Remove append_hook: use set_hook(..., append=True) instead
- Update set_hooks to use unset_hook instead of clear_hook

Squash target: d4de173 (hooks feat: add bulk operations)

Author: tony

src/libtmux/hooks.py

@@ -403,117 +403,6 @@ def show_hook(
         hooks = Hooks.from_stdout(hooks_output)
         return getattr(hooks, hook.replace("-", "_"), None)
 
-    # --- Bulk operations API ---
-
-    def get_hook_indices(
-        self,
-        hook: str,
-        global_: bool = False,
-        scope: OptionScope | _DefaultOptionScope | None = DEFAULT_OPTION_SCOPE,
-        ignore_errors: bool | None = None,
-    ) -> list[int]:
-        """Get list of indices that exist for a hook.
-
-        Parameters
-        ----------
-        hook : str
-            Hook name, e.g. 'session-renamed'
-        global_ : bool
-            Use global hooks
-        scope : OptionScope | None
-            Scope for the hook
-        ignore_errors : bool | None
-            Suppress errors
-
-        Returns
-        -------
-        list[int]
-            Sorted list of indices that exist for this hook.
-
-        Examples
-        --------
-        >>> session.set_hook('session-renamed[0]', 'display-message "test"')
-        Session($...)
-
-        >>> session.set_hook('session-renamed[5]', 'display-message "test"')
-        Session($...)
-
-        >>> session.get_hook_indices('session-renamed')
-        [0, 5]
-
-        >>> session.clear_hook('session-renamed')
-        Session($...)
-        """
-        hooks_output = self._show_hook(
-            hook=hook,
-            global_=global_,
-            scope=scope,
-            ignore_errors=ignore_errors,
-        )
-        if hooks_output is None:
-            return []
-        hooks = Hooks.from_stdout(hooks_output)
-        hook_array = getattr(hooks, hook.replace("-", "_"), None)
-        if hook_array is None or not isinstance(hook_array, SparseArray):
-            return []
-        return sorted(hook_array.keys())
-
-    def get_hook_values(
-        self,
-        hook: str,
-        global_: bool = False,
-        scope: OptionScope | _DefaultOptionScope | None = DEFAULT_OPTION_SCOPE,
-        ignore_errors: bool | None = None,
-    ) -> SparseArray[str]:
-        """Get all indexed values for a hook as SparseArray.
-
-        Parameters
-        ----------
-        hook : str
-            Hook name, e.g. 'session-renamed'
-        global_ : bool
-            Use global hooks
-        scope : OptionScope | None
-            Scope for the hook
-        ignore_errors : bool | None
-            Suppress errors
-
-        Returns
-        -------
-        SparseArray[str]
-            SparseArray containing hook values at their original indices.
-
-        Examples
-        --------
-        >>> session.set_hook('session-renamed[0]', 'display-message "first"')
-        Session($...)
-
-        >>> values = session.get_hook_values('session-renamed')
-        >>> 0 in values
-        True
-
-        >>> 'display-message' in values[0]
-        True
-
-        >>> session.clear_hook('session-renamed')
-        Session($...)
-        """
-        hooks_output = self._show_hook(
-            hook=hook,
-            global_=global_,
-            scope=scope,
-            ignore_errors=ignore_errors,
-        )
-        if hooks_output is None:
-            return SparseArray()
-        hooks = Hooks.from_stdout(hooks_output)
-        hook_array = getattr(hooks, hook.replace("-", "_"), None)
-        if hook_array is None or not isinstance(hook_array, SparseArray):
-            return SparseArray()
-        # Type narrowing doesn't work with isinstance for generics, so cast
-        result: SparseArray[str] = hook_array
-        return result
-
     def set_hooks(
         self,
         hook: str,
@@ -589,7 +478,7 @@ def set_hooks(
         Session($...)
         """
         if clear_existing:
-            self.clear_hook(hook, global_=global_, scope=scope)
+            self.unset_hook(hook, global_=global_, scope=scope)
 
         # Convert list to dict with sequential indices
         if isinstance(values, list):
@@ -604,113 +493,3 @@ def set_hooks(
             )
 
         return self
-
-    def clear_hook(
-        self,
-        hook: str,
-        global_: bool | None = None,
-        scope: OptionScope | _DefaultOptionScope | None = DEFAULT_OPTION_SCOPE,
-        ignore_errors: bool | None = None,
-    ) -> Self:
-        """Unset all indexed values for a hook.
-
-        Parameters
-        ----------
-        hook : str
-            Hook name, e.g. 'session-renamed'
-        global_ : bool | None
-            Use global hooks
-        scope : OptionScope | None
-            Scope for the hook
-        ignore_errors : bool | None
-            Suppress errors when unsetting
-
-        Returns
-        -------
-        Self
-            Returns self for method chaining.
-
-        Examples
-        --------
-        >>> session.set_hook('session-renamed[0]', 'display-message "test"')
-        Session($...)
-
-        >>> session.set_hook('session-renamed[1]', 'display-message "test2"')
-        Session($...)
-
-        >>> session.get_hook_indices('session-renamed')
-        [0, 1]
-
-        >>> session.clear_hook('session-renamed')
-        Session($...)
-
-        >>> session.get_hook_indices('session-renamed')
-        []
-        """
-        indices = self.get_hook_indices(
-            hook,
-            global_=global_ if global_ is not None else False,
-            scope=scope,
-            ignore_errors=ignore_errors,
-        )
-        for index in indices:
-            self.unset_hook(
-                f"{hook}[{index}]",
-                global_=global_,
-                scope=scope,
-                ignore_errors=ignore_errors,
-            )
-        return self
-
-    def append_hook(
-        self,
-        hook: str,
-        value: str,
-        global_: bool | None = None,
-        scope: OptionScope | _DefaultOptionScope | None = DEFAULT_OPTION_SCOPE,
-    ) -> Self:
-        """Append a hook value at the next available index.
-
-        Parameters
-        ----------
-        hook : str
-            Hook name, e.g. 'session-renamed'
-        value : str
-            Hook command to append
-        global_ : bool | None
-            Use global hooks
-        scope : OptionScope | None
-            Scope for the hook
-
-        Returns
-        -------
-        Self
-            Returns self for method chaining.
-
-        Examples
-        --------
-        >>> session.set_hook('session-renamed[0]', 'display-message "first"')
-        Session($...)
-
-        >>> session.append_hook('session-renamed', 'display-message "appended"')
-        Session($...)
-
-        >>> session.get_hook_indices('session-renamed')
-        [0, 1]
-
-        >>> session.clear_hook('session-renamed')
-        Session($...)
-        """
-        indices = self.get_hook_indices(
-            hook,
-            global_=global_ if global_ is not None else False,
-            scope=scope,
-        )
-        next_index = max(indices) + 1 if indices else 0
-        self.set_hook(
-            f"{hook}[{next_index}]",
-            value,
-            global_=global_,
-            scope=scope,
-        )
-        return self