libtmux(fix[logging]): guard None/int values in structured extra keys

why: Extra keys with None or int values violate the AGENTS.md schema
requiring str-typed scalar values. session_name, session_id, pane_id
are all str | None per neo.py dataclass definitions.
what:
- session.py new_window: use conditional extra pattern for nullable fields
- session.py kill_window: fix unbound target, use session_name not window_name,
  accept str | int | None type for target_window
- session.py kill: wrap session_name and session_id in None guards
- pane.py split: use conditional extra pattern for nullable fields
- pane.py kill: wrap pane_id in None guard

Author: tony

src/libtmux/pane.py

@@ -567,14 +567,13 @@ def kill(
         if proc.stderr:
             raise exc.LibTmuxException(proc.stderr)
 
-        logger.info(
-            "pane killed",
-            extra={
-                "tmux_subcommand": "kill-pane",
-                "tmux_pane": self.pane_id,
-                "tmux_target": self.pane_id,
-            },
-        )
+        extra: dict[str, str] = {
+            "tmux_subcommand": "kill-pane",
+        }
+        if self.pane_id is not None:
+            extra["tmux_pane"] = str(self.pane_id)
+            extra["tmux_target"] = str(self.pane_id)
+        logger.info("pane killed", extra=extra)
 
     """
     Commands ("climber"-helpers)
@@ -780,16 +779,18 @@ def split(
 
         pane = self.from_pane_id(server=self.server, pane_id=pane_formatters["pane_id"])
 
-        logger.info(
-            "pane created",
-            extra={
-                "tmux_subcommand": "split-window",
-                "tmux_session": self.session.session_name,
-                "tmux_window": self.window.window_name,
-                "tmux_pane": pane.pane_id,
-                "tmux_target": target,
-            },
-        )
+        extra: dict[str, str] = {
+            "tmux_subcommand": "split-window",
+            "tmux_pane": str(pane.pane_id),
+        }
+        if self.session.session_name is not None:
+            extra["tmux_session"] = str(self.session.session_name)
+        if self.window.window_name is not None:
+            extra["tmux_window"] = str(self.window.window_name)
+        if target is not None:
+            extra["tmux_target"] = str(target)
+
+        logger.info("pane created", extra=extra)
 
         return pane
 

src/libtmux/session.py

@@ -395,14 +395,14 @@ def kill(
         if proc.stderr:
             raise exc.LibTmuxException(proc.stderr)
 
-        logger.info(
-            "session killed",
-            extra={
-                "tmux_subcommand": "kill-session",
-                "tmux_session": self.session_name,
-                "tmux_target": self.session_id,
-            },
-        )
+        extra: dict[str, str] = {
+            "tmux_subcommand": "kill-session",
+        }
+        if self.session_name is not None:
+            extra["tmux_session"] = str(self.session_name)
+        if self.session_id is not None:
+            extra["tmux_target"] = str(self.session_id)
+        logger.info("session killed", extra=extra)
 
     def switch_client(self) -> Session:
         """Switch client to session.
@@ -584,19 +584,21 @@ def new_window(
             window_id=window_formatters["window_id"],
         )
 
-        logger.info(
-            "window created",
-            extra={
-                "tmux_subcommand": "new-window",
-                "tmux_session": self.session_name,
-                "tmux_window": window_name,
-                "tmux_target": target,
-            },
-        )
+        extra: dict[str, str] = {
+            "tmux_subcommand": "new-window",
+        }
+        if self.session_name is not None:
+            extra["tmux_session"] = str(self.session_name)
+        if window.window_name is not None:
+            extra["tmux_window"] = str(window.window_name)
+        if target is not None:
+            extra["tmux_target"] = str(target)
+
+        logger.info("window created", extra=extra)
 
         return window
 
-    def kill_window(self, target_window: str | None = None) -> None:
+    def kill_window(self, target_window: str | int | None = None) -> None:
         """Close a tmux window, and all panes inside it, ``$ tmux kill-window``.
 
         Kill the current window or the window at ``target-window``. removing it
@@ -612,9 +614,10 @@ def kill_window(self, target_window: str | None = None) -> None:
         :exc:`libtmux.exc.LibTmuxException`
             If tmux returns an error.
         """
-        if target_window:
+        target: str | int | None = target_window
+        if target_window is not None:
             if isinstance(target_window, int):
-                target = f"{self.window_name}:{target_window}"
+                target = f"{self.session_name}:{target_window}"
             else:
                 target = f"{target_window}"
 
@@ -623,6 +626,15 @@ def kill_window(self, target_window: str | None = None) -> None:
         if proc.stderr:
             raise exc.LibTmuxException(proc.stderr)
 
+        extra: dict[str, str] = {
+            "tmux_subcommand": "kill-window",
+        }
+        if self.session_name is not None:
+            extra["tmux_session"] = str(self.session_name)
+        if target is not None:
+            extra["tmux_target"] = str(target)
+        logger.info("window killed", extra=extra)
+
     #
     # Dunder
     #