PYTHON: Fix documentation for copy command (#4859)

fix documentation for copy command

Signed-off-by: affonsov <[email protected]>

Author: affonsov

python/glide-async/python/glide/async_commands/cluster_commands.py

@@ -1062,6 +1062,16 @@ async def flushdb(
             await self._execute_command(RequestType.FlushDB, args, route),
         )
 
+    async def copy(
+        self,
+        source: TEncodable,
+        destination: TEncodable,
+        # TODO next major release the arguments replace and destinationDB must have their order
+        # swapped to align with the standalone order.
+        # At the moment of the patch release 2.1.1. we can't have a breaking change
+        replace: Optional[bool] = None,
+        destinationDB: Optional[int] = None,
+    ) -> bool:
         """
         Copies the value stored at the `source` to the `destination` key. If `destinationDB`
         is specified, the value will be copied to the database specified by `destinationDB`,
@@ -1070,57 +1080,31 @@ async def flushdb(
 
         See [valkey.io](https://valkey.io/commands/copy) for more details.
 
+        Note:
+            Both `source` and `destination` must map to the same hash slot.
+
         Args:
             source (TEncodable): The key to the source value.
             destination (TEncodable): The key where the value should be copied to.
-            destinationDB (Optional[Union[int, str]]): The alternative logical database index for the destination key.
             replace (Optional[bool]): If the destination key should be removed before copying the value to it.
+            destinationDB (Optional[int]): The alternative logical database index for the destination key.
 
         Returns:
-            bool: True if the source was copied. Otherwise, return False.
+            bool: True if the source was copied. Otherwise, returns False.
 
         Examples:
             >>> await client.set("source", "sheep")
+            >>> await client.copy(b"source", b"destination")
+                True # Source was copied
             >>> await client.copy(b"source", b"destination", destinationDB=1)
                 True # Source was copied to DB 1
             >>> await client.select(1)
             >>> await client.get("destination")
                 b"sheep"
 
-        Since: Valkey version 9.0.0.
-        """
-        ...
-
-    async def copy(
-        self,
-        source: TEncodable,
-        destination: TEncodable,
-        # TODO next major release the arguments replace and destinationDB must have their order
-        # swapped to align with the standalone order.
-        # At the moment of the patch release 2.1.1. we can't have a breaking change
-        replace: Optional[bool] = None,
-        destinationDB: Optional[int] = None,
-    ) -> bool:
-        """
-        Copies the value stored at the `source` to the `destination` key.
-
-        This method supports two signatures:
-        1. copy(source, destination, replace=None) - original API
-        2. copy(source, destination, replace=None, destinationDB=None) - new API with DB support
-
-        When `replace` is True, removes the `destination` key first if it already exists,
-        otherwise performs no action.
-
-        See [valkey.io](https://valkey.io/commands/copy) for more details.
-
-        Note:
-            When `destinationDB` is not provided, both `source` and `destination` must map to the same hash slot.
+        Since: Valkey version 6.2.0.
+               The destinationDB argument is available since Valkey 9.0.0
         """
-        # Handle backward compatibility for positional arguments
-        # If destinationDB is a boolean, it's likely the old API: copy(source, destination, replace)
-        if isinstance(destinationDB, bool):
-            replace = destinationDB
-            destinationDB = None
 
         # Build command arguments
         args: List[TEncodable] = [source, destination]

python/glide-sync/glide_sync/sync_commands/cluster_commands.py

@@ -952,6 +952,16 @@ def flushdb(
             self._execute_command(RequestType.FlushDB, args, route),
         )
 
+    def copy(
+        self,
+        source: TEncodable,
+        destination: TEncodable,
+        # TODO next major release the arguments replace and destinationDB must have their order
+        # swapped to align with the standalone order.
+        # At the moment of the patch release 2.1.1. we can't have a breaking change
+        replace: Optional[bool] = None,
+        destinationDB: Optional[int] = None,
+    ) -> bool:
         """
         Copies the value stored at the `source` to the `destination` key. If `destinationDB`
         is specified, the value will be copied to the database specified by `destinationDB`,
@@ -963,8 +973,8 @@ def flushdb(
         Args:
             source (TEncodable): The key to the source value.
             destination (TEncodable): The key where the value should be copied to.
-            destinationDB (Optional[Union[int, str]]): The alternative logical database index for the destination key.
             replace (Optional[bool]): If the destination key should be removed before copying the value to it.
+            destinationDB (Optional[int]): The alternative logical database index for the destination key.
 
         Returns:
             bool: True if the source was copied. Otherwise, return False.
@@ -977,40 +987,9 @@ def flushdb(
             >>> client.get("destination")
                 b"sheep"
 
-        Since: Valkey version 9.0.0.
-        """
-        ...
-
-    def copy(
-        self,
-        source: TEncodable,
-        destination: TEncodable,
-        # TODO next major release the arguments replace and destinationDB must have their order
-        # swapped to align with the standalone order.
-        # At the moment of the patch release 2.1.1. we can't have a breaking change
-        replace: Optional[bool] = None,
-        destinationDB: Optional[int] = None,
-    ) -> bool:
-        """
-        Copies the value stored at the `source` to the `destination` key.
-
-        This method supports two signatures:
-        1. copy(source, destination, replace=None) - original API
-        2. copy(source, destination, replace=None, destinationDB=None) - new API with DB support
-
-        When `replace` is True, removes the `destination` key first if it already exists,
-        otherwise performs no action.
-
-        See [valkey.io](https://valkey.io/commands/copy) for more details.
-
-        Note:
-            When `destinationDB` is not provided, both `source` and `destination` must map to the same hash slot.
+        Since: Valkey version 6.2.0.
+               The destinationDB argument is available since Valkey 9.0.0
         """
-        # Handle backward compatibility for positional arguments
-        # If destinationDB is a boolean, it's likely the old API: copy(source, destination, replace)
-        if isinstance(destinationDB, bool):
-            replace = destinationDB
-            destinationDB = None
 
         # Build command arguments
         args: List[TEncodable] = [source, destination]