gh-143052: add exist_ok in copy_info func, allow merge same dir

Author: fatelei

Doc/library/pathlib.rst

@@ -1590,12 +1590,16 @@ Copying, moving and deleting
    .. versionadded:: 3.14
 
 
-.. method:: Path.copy_into(target_dir, *, follow_symlinks=True, \
+.. method:: Path.copy_into(target_dir, *, exist_ok=True, follow_symlinks=True, \
                            preserve_metadata=False)
 
    Copy this file or directory tree into the given *target_dir*, which should
-   be an existing directory. Other arguments are handled identically to
-   :meth:`Path.copy`. Returns a new :class:`!Path` instance pointing to the
+   be an existing directory. If *exist_ok* is true (the default), copying a file
+   overwrites an existing file with the same name and copying a directory merges
+   its contents into an existing directory with the same name. If *exist_ok* is
+   false, :exc:`FileExistsError` is raised if *target_dir* already contains an
+   entry with the same name as the source. Other arguments are handled identically
+   to :meth:`Path.copy`. Returns a new :class:`!Path` instance pointing to the
    copy.
 
    .. versionadded:: 3.14

Lib/pathlib/__init__.py

@@ -1301,17 +1301,33 @@ def copy(self, target, **kwargs):
         target._copy_from(self, **kwargs)
         return target.joinpath()  # Empty join to ensure fresh metadata.
 
-    def copy_into(self, target_dir, **kwargs):
+    def copy_into(self, target_dir, exist_ok=True, **kwargs):
         """
         Copy this file or directory tree into the given existing directory.
         """
         name = self.name
         if not name:
             raise ValueError(f"{self!r} has an empty name")
-        elif hasattr(target_dir, 'with_segments'):
-            target = target_dir / name
-        else:
-            target = self.with_segments(target_dir, name)
+
+        target = (target_dir / name) if hasattr(target_dir, "with_segments") \
+                else self.with_segments(target_dir, name)
+
+        if self.info.is_file():
+            if not exist_ok and target.info.is_file():
+                raise FileExistsError(EEXIST, "File exists", str(target))
+            return self.copy(target, **kwargs)
+
+        if self.info.is_dir():
+            if target.info.exists() and target.info.is_dir():
+                if not exist_ok:
+                    raise FileExistsError(EEXIST, "File exists", str(target))
+
+                for child in self.iterdir():
+                    child.copy_into(target, exist_ok=exist_ok, **kwargs)
+                return target.joinpath()
+
+            return self.copy(target, **kwargs)
+
         return self.copy(target, **kwargs)
 
     def _copy_from(self, source, follow_symlinks=True, preserve_metadata=False):

Lib/pathlib/types.py

@@ -386,13 +386,22 @@ def copy(self, target, **kwargs):
         target._copy_from(self, **kwargs)
         return target.joinpath()  # Empty join to ensure fresh metadata.
 
-    def copy_into(self, target_dir, **kwargs):
+    def copy_into(self, target_dir, exist_ok=True, **kwargs):
         """
         Copy this file or directory tree into the given existing directory.
         """
         name = self.name
         if not name:
             raise ValueError(f"{self!r} has an empty name")
+        
+        target = target_dir / name
+        if hasattr(target, 'info') and self.info.is_dir() and target.info.exists() and target.info.is_dir():
+            if not exist_ok:
+                raise FileExistsError("File exists", str(target))
+            for child in self.iterdir():
+                child.copy_into(target, exist_ok=exist_ok, **kwargs)
+            return target.joinpath()
+
         return self.copy(target_dir / name, **kwargs)
 
 

Lib/test/test_pathlib/test_copy.py

@@ -146,6 +146,33 @@ def test_copy_into_empty_name(self):
         self.target_ground.create_dir(target_dir)
         self.assertRaises(ValueError, source.copy_into, target_dir)
 
+    def test_copy_dir_into_existing_dir_merges_when_exist_ok_true(self):
+        if isinstance(self.target_root, WritableZipPath):
+            self.skipTest('needs local target')
+        source = self.source_root / 'dirC'
+        target_dir = self.target_root
+        preexisting = target_dir / 'dirC'
+        self.target_ground.create_dir(preexisting)
+        self.target_ground.create_file(preexisting / 'pre.txt', b'pre\n')
+
+        result = source.copy_into(target_dir)
+        self.assertEqual(result, preexisting)
+
+        self.assertTrue(self.target_ground.isfile(preexisting / 'pre.txt'))
+
+        self.assertTrue(self.target_ground.isfile(preexisting / 'fileC'))
+        self.assertTrue(self.target_ground.isdir(preexisting / 'dirD'))
+        self.assertTrue(self.target_ground.isfile(preexisting / 'dirD' / 'fileD'))
+
+    def test_copy_dir_into_existing_dir_exist_ok_false(self):
+        if isinstance(self.target_root, WritableZipPath):
+            self.skipTest('needs local target')
+        source = self.source_root / 'dirC'
+        target_dir = self.target_root
+        self.target_ground.create_dir(target_dir / 'dirC')
+        with self.assertRaises(FileExistsError):
+            source.copy_into(target_dir, exist_ok=False)
+
 
 class ZipToZipPathCopyTest(CopyTestBase, unittest.TestCase):
     source_ground = ZipPathGround(ReadableZipPath)

Misc/NEWS.d/next/Library/2025-12-22-11-21-41.gh-issue-143052.3BhjY1.rst

@@ -0,0 +1 @@
+copy_into add exist_ok allow overwrite same directory.