⚡ Bolt: Optimize GetIdentifier string concatenation

.jules/bolt.md

@@ -0,0 +1,3 @@
+## 2024-05-22 - [Optimizing String Manipulations in Code Generation]
+**Learning:** The codebase relies heavily on string manipulations for code generation (e.g., `SafeIdentifier`). I observed a pattern of chained `string.Replace()` calls which creates unnecessary intermediate allocations.
+**Action:** Future optimizations should focus on replacing chained `Replace` calls with single-pass `StringBuilder` filtering or `regex` replacements where appropriate, similar to the `GetIdentifier` optimization.

v4/DynamicsCrm.DevKit.Shared/Helper.cs

@@ -104,13 +104,15 @@ public static string GetSchemaNameFromFile(string file, string endsWith)
 
         private static string GetIdentifier(string name)
         {
-            var value = string.Empty;
+            if (string.IsNullOrEmpty(name)) return string.Empty;
+            // Optimization: Use StringBuilder to avoid intermediate string allocations in loop
+            var sb = new StringBuilder(name.Length);
             for (int i = 0; i < name.Length; ++i)
             {
                 if (char.IsLetterOrDigit(name[i]) || name[i] == ' ' || name[i] == '-' || name[i] == '_')
-                    value += name[i];
+                    sb.Append(name[i]);
             }
-            return value;
+            return sb.ToString();
         }
 
         public static string SafeIdentifier(string name)