Walker now only returns path that need changes to be applied.

src/ansible_creator/utils.py

@@ -240,17 +240,22 @@ def each_obj(
         if obj.is_file():
             dest_path.set_content(template_data, self.templar)
 
-        conflict_msg = dest_path.conflict
-        if dest_path.needs_write and conflict_msg:
-            self.output.warning(conflict_msg)
-        if obj.is_dir() and obj.name not in SKIP_DIRS:
-            return [
-                dest_path,
-                *self._recursive_walk(root=obj, resource=resource, template_data=template_data),
-            ]
+        if dest_path.needs_write:
+            # Warn on conflict
+            conflict_msg = dest_path.conflict
+            if conflict_msg:
+                self.output.warning(conflict_msg)
+
+            if obj.is_dir() and obj.name not in SKIP_DIRS:
+                return [
+                    dest_path,
+                    *self._recursive_walk(root=obj, resource=resource, template_data=template_data),
+                ]
+            if obj.is_file():
+                return [dest_path]
 
-        if obj.is_file():
-            return [dest_path]
+        if obj.is_dir() and obj.name not in SKIP_DIRS:
+            return self._recursive_walk(root=obj, resource=resource, template_data=template_data)
 
         return []
 
@@ -345,8 +350,7 @@ def copy_containers(self: Copier, paths: list[DestinationFile]) -> None:
             paths: A list of paths to create in the destination.
         """
         for path in paths:
-            if path.conflict:
-                path.remove_existing()
+            path.remove_existing()
 
             if path.source.is_dir():
                 path.dest.mkdir(parents=True, exist_ok=True)

tests/units/test_utils.py

@@ -104,3 +104,30 @@ def test_overwrite(tmp_path: Path, output: Output) -> None:
     assert base_file.read_text() == base_contents
     assert podman_dir.is_dir()
     assert docker_file.is_file()
+
+
+def test_skip_repeats(tmp_path: Path, output: Output) -> None:
+    """Test Copier skipping existing files.
+
+    Args:
+        tmp_path: Temporary directory path.
+        output: Output class object.
+    """
+    walker = Walker(
+        resources=("common.devcontainer",),
+        resource_id="common.devcontainer",
+        dest=tmp_path,
+        output=output,
+        template_data=TemplateData(),
+    )
+    paths = walker.collect_paths()
+    assert paths
+
+    copier = Copier(
+        output=output,
+    )
+    copier.copy_containers(paths)
+
+    # Re-walk directory to generate new path list
+    paths = walker.collect_paths()
+    assert not paths