fix: Address PR review feedback for backward compatibility (#1693)

- Change Clean() default to removeReadOnlyAttribute: false for backward compat
- Move CopyTo log to start of method for consistency with other operations
- Add XML remarks documenting the backward-compatible behavior

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>

Author: thomhurst

src/ModularPipelines/FileSystem/Folder.cs

@@ -95,9 +95,13 @@ public void Delete()
     /// <summary>
     /// Removes all files and subdirectories within the folder.
     /// </summary>
+    /// <remarks>
+    /// This method preserves backward compatibility by not removing read-only attributes.
+    /// Use <see cref="Clean(bool)"/> with <c>removeReadOnlyAttribute: true</c> to handle read-only files.
+    /// </remarks>
     public void Clean()
     {
-        Clean(removeReadOnlyAttribute: true);
+        Clean(removeReadOnlyAttribute: false);
     }
 
     /// <summary>
@@ -153,6 +157,8 @@ public Folder CopyTo(string targetPath)
     /// <returns>A new <see cref="Folder"/> instance representing the copied folder.</returns>
     public Folder CopyTo(string targetPath, bool preserveTimestamps)
     {
+        LogFolderOperationWithDestination("Copying Folder: {Source} > {Destination} [Module: {ModuleName}, Activity: {ActivityId}]", this, targetPath);
+
         Directory.CreateDirectory(targetPath);
 
         // Copy all subdirectories first
@@ -206,8 +212,6 @@ public Folder CopyTo(string targetPath, bool preserveTimestamps)
             targetRootDir.LastAccessTimeUtc = DirectoryInfo.LastAccessTimeUtc;
         }
 
-        LogFolderOperationWithDestination("Copied Folder: {Source} > {Destination} [Module: {ModuleName}, Activity: {ActivityId}]", this, targetPath);
-
         return new Folder(targetPath);
     }