Ops(fix): Correct move-window -k and paste-buffer -r

why: An adversarial review of the new ops against tmux's command
grammar found two defects: move-window could not request its
kill-on-collision behavior, and paste-buffer's -r flag was
documented as a space replacement it never performs.

what:
- MoveWindow: add kill (-k) field; tmux move-window's option string
  is "abdkrs:t:" and -k replaces any window already at the
  destination index
- PasteBuffer: rename no_format to no_replace and fix the docstring;
  -r keeps linefeeds instead of converting them to the default
  carriage-return separator (it has nothing to do with spaces)
- Add render cases for move-window -k/-r and paste-buffer -r

Author: tony

src/libtmux/experimental/ops/_ops/move_window.py

@@ -24,6 +24,8 @@ class MoveWindow(Operation[AckResult]):
         Do not change the active window (``-d``).
     before, after : bool
         Insert before (``-b``) or after (``-a``) the destination index.
+    kill : bool
+        Replace (kill) any window already at the destination (``-k``).
     renumber : bool
         Renumber windows to close gaps (``-r``).
 
@@ -44,6 +46,7 @@ class MoveWindow(Operation[AckResult]):
     detach: bool = False
     before: bool = False
     after: bool = False
+    kill: bool = False
     renumber: bool = False
 
     def args(self, *, version: str | None = None) -> tuple[str, ...]:
@@ -55,6 +58,8 @@ def args(self, *, version: str | None = None) -> tuple[str, ...]:
             out.append("-b")
         if self.after:
             out.append("-a")
+        if self.kill:
+            out.append("-k")
         if self.renumber:
             out.append("-r")
         out.extend(self.src_args())

src/libtmux/experimental/ops/_ops/paste_buffer.py

@@ -25,8 +25,9 @@ class PasteBuffer(Operation[AckResult]):
         Delete the buffer after pasting (``-d``).
     bracket : bool
         Use bracketed paste mode (``-p``).
-    no_format : bool
-        Do not replace separators with spaces (``-r``).
+    no_replace : bool
+        Do no separator replacement: keep linefeeds (LF) instead of
+        converting them to the default carriage-return separator (``-r``).
     separator : str or None
         Separator inserted between lines (``-s``).
 
@@ -49,7 +50,7 @@ class PasteBuffer(Operation[AckResult]):
     buffer_name: str | None = None
     delete: bool = False
     bracket: bool = False
-    no_format: bool = False
+    no_replace: bool = False
     separator: str | None = None
 
     def args(self, *, version: str | None = None) -> tuple[str, ...]:
@@ -59,7 +60,7 @@ def args(self, *, version: str | None = None) -> tuple[str, ...]:
             out.append("-d")
         if self.bracket:
             out.append("-p")
-        if self.no_format:
+        if self.no_replace:
             out.append("-r")
         if self.buffer_name is not None:
             out.extend(("-b", self.buffer_name))

tests/experimental/ops/test_buffer_ops.py

@@ -82,6 +82,11 @@ class RenderCase(t.NamedTuple):
         op=PasteBuffer(target=PaneId("%1"), delete=True),
         expected=("paste-buffer", "-t", "%1", "-d"),
     ),
+    RenderCase(
+        test_id="paste_buffer_no_replace",
+        op=PasteBuffer(target=PaneId("%1"), no_replace=True),
+        expected=("paste-buffer", "-t", "%1", "-r"),
+    ),
     RenderCase(
         test_id="show_buffer",
         op=ShowBuffer(buffer_name="b0"),

tests/experimental/ops/test_window_ops.py

@@ -97,6 +97,16 @@ class RenderCase(t.NamedTuple):
         op=MoveWindow(target=SessionId("$0"), src_target=WindowId("@2")),
         expected=("move-window", "-t", "$0", "-s", "@2"),
     ),
+    RenderCase(
+        test_id="move_window_kill_renumber",
+        op=MoveWindow(
+            target=SessionId("$0"),
+            src_target=WindowId("@2"),
+            kill=True,
+            renumber=True,
+        ),
+        expected=("move-window", "-t", "$0", "-k", "-r", "-s", "@2"),
+    ),
     RenderCase(
         test_id="link_window",
         op=LinkWindow(target=SessionId("$0"), src_target=WindowId("@2")),