Merge pull request #480 from GeneralLibrary/fix/copy-unknown-files-atomic-replace

fix(DiffPipeline): CopyUnknownFiles 使用原子替换避免文件占用冲突

Author: JusterZhu

src/c#/GeneralUpdate.Core/Pipeline/DiffPipeline.cs

@@ -507,7 +507,22 @@ private static Task CopyUnknownFiles(string appPath, string patchPath)
                 if (parentFolder?.Exists == false)
                     parentFolder.Create();
 
-                File.Copy(file.FullName, targetPath, true);
+                // Atomic replace via temp file, same strategy as ApplyPatch.
+                // Avoids file-in-use errors when the process just exited.
+                var safeName = targetFileName.Replace(Path.DirectorySeparatorChar, '_');
+                var tempPath = Path.Combine(appPath, $"{Path.GetRandomFileName()}_{safeName}");
+                File.Copy(file.FullName, tempPath, true);
+
+                // Use File.Replace for atomic overwrite (handles locked targets gracefully)
+                if (File.Exists(targetPath))
+                {
+                    File.SetAttributes(targetPath, FileAttributes.Normal);
+                    File.Replace(tempPath, targetPath, null, true);
+                }
+                else
+                {
+                    File.Move(tempPath, targetPath);
+                }
             }
 
             if (Directory.Exists(patchPath))

tests/ClientTest/ClientTest.csproj

@@ -11,10 +11,4 @@
     <ProjectReference Include="..\..\src\c#\GeneralUpdate.Core\GeneralUpdate.Core.csproj" />
   </ItemGroup>
 
-  <ItemGroup>
-    <None Update="generalupdate.manifest.json">
-      <CopyToOutputDirectory>PreserveNewest</CopyToOutputDirectory>
-    </None>
-  </ItemGroup>
-
 </Project>