Session(refactor): Replace deprecation warnings with DeprecatedError

why: Enforce immediate migration from deprecated APIs.

what:
- attached_window raises DeprecatedError, use active_window instead
- attached_pane raises DeprecatedError, use active_pane instead
- attach_session() raises DeprecatedError, use attach() instead
- kill_session() raises DeprecatedError, use kill() instead
- get(), __getitem__() raise DeprecatedError, use attribute access
- get_by_id(), where(), find_where() raise DeprecatedError
- list_windows(), _windows, _list_windows(), children raise DeprecatedError

Author: tony

src/libtmux/session.py

@@ -11,7 +11,6 @@
 import logging
 import pathlib
 import typing as t
-import warnings
 
 from libtmux._internal.query_list import QueryList
 from libtmux.common import tmux_cmd
@@ -649,12 +648,11 @@ def attached_pane(self) -> Pane | None:
 
            Deprecated in favor of :meth:`.active_pane`.
         """
-        warnings.warn(
-            "Session.attached_pane() is deprecated in favor of Session.active_pane()",
-            category=DeprecationWarning,
-            stacklevel=2,
+        raise exc.DeprecatedError(
+            deprecated="Session.attached_pane",
+            replacement="Session.active_pane",
+            version="0.31.0",
         )
-        return self.active_window.active_pane
 
     @property
     def attached_window(self) -> Window:
@@ -666,13 +664,11 @@ def attached_window(self) -> Window:
 
            Deprecated in favor of :meth:`.active_window`.
         """
-        warnings.warn(
-            "Session.attached_window() is deprecated in favor of "
-            "Session.active_window()",
-            category=DeprecationWarning,
-            stacklevel=2,
+        raise exc.DeprecatedError(
+            deprecated="Session.attached_window",
+            replacement="Session.active_window",
+            version="0.31.0",
         )
-        return self.active_window
 
     def attach_session(self) -> Session:
         """Return ``$ tmux attach-session`` aka alias: ``$ tmux attach``.
@@ -683,17 +679,11 @@ def attach_session(self) -> Session:
 
            Deprecated in favor of :meth:`.attach()`.
         """
-        warnings.warn(
-            "Session.attach_session() is deprecated in favor of Session.attach()",
-            category=DeprecationWarning,
-            stacklevel=2,
+        raise exc.DeprecatedError(
+            deprecated="Session.attach_session()",
+            replacement="Session.attach()",
+            version="0.30.0",
         )
-        proc = self.cmd("attach-session")
-
-        if proc.stderr:
-            raise exc.LibTmuxException(proc.stderr)
-
-        return self
 
     def kill_session(self) -> None:
         """Destroy session.
@@ -704,15 +694,11 @@ def kill_session(self) -> None:
 
            Deprecated in favor of :meth:`.kill()`.
         """
-        warnings.warn(
-            "Session.kill_session() is deprecated in favor of Session.kill()",
-            category=DeprecationWarning,
-            stacklevel=2,
+        raise exc.DeprecatedError(
+            deprecated="Session.kill_session()",
+            replacement="Session.kill()",
+            version="0.30.0",
         )
-        proc = self.cmd("kill-session")
-
-        if proc.stderr:
-            raise exc.LibTmuxException(proc.stderr)
 
     def get(self, key: str, default: t.Any | None = None) -> t.Any:
         """Return key-based lookup. Deprecated by attributes.
@@ -723,12 +709,11 @@ def get(self, key: str, default: t.Any | None = None) -> t.Any:
            accessed via ``session.session_name``.
 
         """
-        warnings.warn(
-            "Session.get() is deprecated",
-            category=DeprecationWarning,
-            stacklevel=2,
+        raise exc.DeprecatedError(
+            deprecated="Session.get()",
+            replacement="direct attribute access (e.g., session.session_name)",
+            version="0.16.0",
         )
-        return getattr(self, key, default)
 
     def __getitem__(self, key: str) -> t.Any:
         """Return item lookup by key. Deprecated in favor of attributes.
@@ -739,12 +724,11 @@ def __getitem__(self, key: str) -> t.Any:
            accessed via ``session.session_name``.
 
         """
-        warnings.warn(
-            f"Item lookups, e.g. session['{key}'] is deprecated",
-            category=DeprecationWarning,
-            stacklevel=2,
+        raise exc.DeprecatedError(
+            deprecated="Session[key] lookup",
+            replacement="direct attribute access (e.g., session.session_name)",
+            version="0.16.0",
         )
-        return getattr(self, key)
 
     def get_by_id(self, session_id: str) -> Window | None:
         """Return window by id. Deprecated in favor of :meth:`.windows.get()`.
@@ -754,12 +738,11 @@ def get_by_id(self, session_id: str) -> Window | None:
            Deprecated by :meth:`.windows.get()`.
 
         """
-        warnings.warn(
-            "Session.get_by_id() is deprecated",
-            category=DeprecationWarning,
-            stacklevel=2,
+        raise exc.DeprecatedError(
+            deprecated="Session.get_by_id()",
+            replacement="Session.windows.get(window_id=..., default=None)",
+            version="0.16.0",
         )
-        return self.windows.get(window_id=session_id, default=None)
 
     def where(self, kwargs: dict[str, t.Any]) -> list[Window]:
         """Filter through windows, return list of :class:`Window`.
@@ -769,15 +752,11 @@ def where(self, kwargs: dict[str, t.Any]) -> list[Window]:
            Deprecated by :meth:`.windows.filter()`.
 
         """
-        warnings.warn(
-            "Session.where() is deprecated",
-            category=DeprecationWarning,
-            stacklevel=2,
+        raise exc.DeprecatedError(
+            deprecated="Session.where()",
+            replacement="Session.windows.filter()",
+            version="0.16.0",
         )
-        try:
-            return self.windows.filter(**kwargs)
-        except IndexError:
-            return []
 
     def find_where(self, kwargs: dict[str, t.Any]) -> Window | None:
         """Filter through windows, return first :class:`Window`.
@@ -787,12 +766,11 @@ def find_where(self, kwargs: dict[str, t.Any]) -> Window | None:
            Slated to be removed in favor of :meth:`.windows.get()`.
 
         """
-        warnings.warn(
-            "Session.find_where() is deprecated",
-            category=DeprecationWarning,
-            stacklevel=2,
+        raise exc.DeprecatedError(
+            deprecated="Session.find_where()",
+            replacement="Session.windows.get(default=None, **kwargs)",
+            version="0.16.0",
         )
-        return self.windows.get(default=None, **kwargs)
 
     def _list_windows(self) -> list[WindowDict]:
         """Return list of windows (deprecated in favor of :attr:`.windows`).
@@ -802,12 +780,11 @@ def _list_windows(self) -> list[WindowDict]:
            Slated to be removed in favor of :attr:`.windows`.
 
         """
-        warnings.warn(
-            "Session._list_windows() is deprecated",
-            category=DeprecationWarning,
-            stacklevel=2,
+        raise exc.DeprecatedError(
+            deprecated="Session._list_windows()",
+            replacement="Session.windows property",
+            version="0.16.0",
         )
-        return [w.__dict__ for w in self.windows]
 
     @property
     def _windows(self) -> list[WindowDict]:
@@ -818,12 +795,11 @@ def _windows(self) -> list[WindowDict]:
            Slated to be removed in favor of :attr:`.windows`.
 
         """
-        warnings.warn(
-            "Session._windows is deprecated",
-            category=DeprecationWarning,
-            stacklevel=2,
+        raise exc.DeprecatedError(
+            deprecated="Session._windows",
+            replacement="Session.windows property",
+            version="0.16.0",
         )
-        return self._list_windows()
 
     def list_windows(self) -> list[Window]:
         """Return a list of :class:`Window` from the ``tmux(1)`` session.
@@ -833,12 +809,11 @@ def list_windows(self) -> list[Window]:
            Slated to be removed in favor of :attr:`.windows`.
 
         """
-        warnings.warn(
-            "Session.list_windows() is deprecated",
-            category=DeprecationWarning,
-            stacklevel=2,
+        raise exc.DeprecatedError(
+            deprecated="Session.list_windows()",
+            replacement="Session.windows property",
+            version="0.16.0",
         )
-        return self.windows
 
     @property
     def children(self) -> QueryList[Window]:
@@ -849,9 +824,8 @@ def children(self) -> QueryList[Window]:
            Slated to be removed in favor of :attr:`.windows`.
 
         """
-        warnings.warn(
-            "Session.children is deprecated",
-            category=DeprecationWarning,
-            stacklevel=2,
+        raise exc.DeprecatedError(
+            deprecated="Session.children",
+            replacement="Session.windows property",
+            version="0.16.0",
         )
-        return self.windows