From 65b9677a6b0016b161397ad130f5c3cb2b2c751d Mon Sep 17 00:00:00 2001 From: hatayama Date: Thu, 9 Jul 2026 08:26:05 +0900 Subject: [PATCH] Remove dead worker directory deleter seam Restore direct temporary-directory deletion after the only test that swapped the delegate was removed as part of the EditMode freeze prevention work. --- .../SharedRoslynCompilerWorkerHost.cs | 5 ----- .../SharedRoslynCompilerWorkerSession.cs | 12 +----------- 2 files changed, 1 insertion(+), 16 deletions(-) diff --git a/Packages/src/Editor/FirstPartyTools/ExecuteDynamicCode/DynamicCompilation/SharedRoslynCompilerWorkerHost.cs b/Packages/src/Editor/FirstPartyTools/ExecuteDynamicCode/DynamicCompilation/SharedRoslynCompilerWorkerHost.cs index be7d466233..586de3c946 100644 --- a/Packages/src/Editor/FirstPartyTools/ExecuteDynamicCode/DynamicCompilation/SharedRoslynCompilerWorkerHost.cs +++ b/Packages/src/Editor/FirstPartyTools/ExecuteDynamicCode/DynamicCompilation/SharedRoslynCompilerWorkerHost.cs @@ -467,11 +467,6 @@ internal static void ShutdownForServerReset() Shutdown(); } - internal static Action SwapWorkerDirectoryDeleterForTests(Action deleter) - { - return ServiceValue.SwapWorkerDirectoryDeleterForTests(deleter); - } - internal static Action SwapCompileRequestSenderForTests(Action sender) { return ServiceValue.SwapCompileRequestSenderForTests(sender); diff --git a/Packages/src/Editor/FirstPartyTools/ExecuteDynamicCode/DynamicCompilation/SharedRoslynCompilerWorkerSession.cs b/Packages/src/Editor/FirstPartyTools/ExecuteDynamicCode/DynamicCompilation/SharedRoslynCompilerWorkerSession.cs index da39886346..8b4b71a4f3 100644 --- a/Packages/src/Editor/FirstPartyTools/ExecuteDynamicCode/DynamicCompilation/SharedRoslynCompilerWorkerSession.cs +++ b/Packages/src/Editor/FirstPartyTools/ExecuteDynamicCode/DynamicCompilation/SharedRoslynCompilerWorkerSession.cs @@ -16,7 +16,6 @@ namespace io.github.hatayama.UnityCliLoop.FirstPartyTools internal sealed class SharedRoslynCompilerWorkerSession { private readonly object _syncRoot = new(); - private Action _deleteWorkerDirectory = path => Directory.Delete(path, true); private Func _startProcess = ProcessStartHelper.TryStart; private Action _sendCompileRequest = SendCompileRequestCore; private Func @@ -228,15 +227,6 @@ internal void Shutdown(string fallbackWorkerDirectoryPath) } } - internal Action SwapWorkerDirectoryDeleterForTests(Action deleter) - { - Debug.Assert(deleter != null, "deleter must not be null"); - - Action previous = _deleteWorkerDirectory; - _deleteWorkerDirectory = deleter; - return previous; - } - internal Func SwapProcessStarterForTests( Func starter) { @@ -291,7 +281,7 @@ private void TryDeleteWorkerDirectory(string workerDirectoryPath) { try { - _deleteWorkerDirectory(workerDirectoryPath); + Directory.Delete(workerDirectoryPath, true); } catch (IOException ex) {