EditorApplication.update will not work so revert

Author: ekcoh

Packages/com.unity.inputsystem/InputSystem/Editor/AssetImporter/InputActionImporter.cs

@@ -256,28 +256,14 @@ private static void GenerateWrapperCode(AssetImportContext ctx, InputActionAsset
 
             if (InputActionCodeGenerator.GenerateWrapperCode(wrapperFilePath, asset, options))
             {
-                // When we generate the wrapper code cs file during asset import, we cannot call ImportAsset on that directly because
-                // script assets have to be imported before all other assets, and are not allowed to be added to the import queue during
-                // asset import. So instead we register a callback to trigger a delayed asset refresh which should then pick up the
-                // changed/added script, and trigger a new import.
-                //AssetDatabase.ImportAsset(wrapperFilePath);
-                // TODO Temporary disabled, reenable: EditorApplication.delayCall += AssetDatabase.Refresh;
-                EditorApplication.delayCall += AssetDatabase.Refresh;
-                if (++_importedAssetsSinceLastRefresh == 1)
-                    EditorApplication.update += AssetDatabase.Refresh;
+                // This isn't ideal and may have side effects, but we cannot avoid compiling again.  
+                // Previously we attempted to run a EditorApplication.delayCall += AssetDatabase.Refresh 
+                // but this would lead to "error: Error building Player because scripts are compiling" in CI.
+                // Previous comment here warned against not being able to reimport here directly, but it seems it's ok.
+                AssetDatabase.ImportAsset(wrapperFilePath);
             }
         }
 
-        private static int _importedAssetsSinceLastRefresh = 0;
-
-        private void RefreshGeneratedAssets()
-        {
-            EditorApplication.update -= RefreshGeneratedAssets;
-            _importedAssetsSinceLastRefresh = 0;
-            AssetDatabase.Refresh();
-        }
-
-
 #if UNITY_INPUT_SYSTEM_PROJECT_WIDE_ACTIONS
         internal static IEnumerable<InputActionReference> LoadInputActionReferencesFromAsset(string assetPath)
         {