Remove Create factory method on AuAuditLog as now covered by the AiAuditLogFactory

Author: mattbrailsford

Umbraco.Ai/src/Umbraco.Ai.Core/AuditLog/AiAuditLog.cs

@@ -1,8 +1,4 @@
-using System.Text.RegularExpressions;
-using Microsoft.Extensions.AI;
-using Microsoft.Extensions.Logging;
 using Umbraco.Ai.Core.Models;
-using Umbraco.Cms.Core.Models.Membership;
 
 namespace Umbraco.Ai.Core.AuditLog;
 
@@ -144,119 +140,4 @@ public sealed class AiAuditLog
     /// Stored as a JSON dictionary in the database.
     /// </summary>
     public IReadOnlyDictionary<string, string>? Metadata { get; init; }
-
-    /// <summary>
-    /// Creates a new AiAuditLog instance from the given AiAuditContext.
-    /// </summary>
-    /// <param name="context">The AiAuditContext containing operation details.</param>
-    /// <param name="options">The AiAuditLogOptions for configuring audit-log behavior.</param>
-    /// <param name="metadata">Optional metadata to include in the audit-log.</param>
-    /// <param name="detailLevel">The detail level for this audit-log.</param>
-    /// <param name="user">The user who initiated the operation.</param>
-    /// <param name="id">Optional specific ID for the audit-log; a new GUID will be generated if not provided.</param>
-    /// <param name="parentId">Optional parent audit-log ID if this log is part of a nested operation.</param>
-    /// <param name="loggerFactory">Optional logger factory for logging during creation.</param>
-    /// <returns></returns>
-    public static AiAuditLog Create(
-        AiAuditContext context,
-        AiAuditLogOptions options,
-        IReadOnlyDictionary<string, string>? metadata = null,
-        AiAuditLogDetailLevel detailLevel = AiAuditLogDetailLevel.FailuresOnly,
-        IUser? user = null,
-        Guid? id = null,
-        Guid? parentId = null,
-        ILoggerFactory? loggerFactory = null)
-    {
-        if (!context.ProfileId.HasValue) throw new ArgumentException("ProfileId must be set in the AiAuditContext.", nameof(context));
-        if (string.IsNullOrWhiteSpace(context.ProfileAlias)) throw new ArgumentException("ProfileAlias must be set in the AiAuditContext.", nameof(context));
-        if (string.IsNullOrWhiteSpace(context.ProviderId)) throw new ArgumentException("ProviderId must be set in the AiAuditContext.", nameof(context));
-        if (string.IsNullOrWhiteSpace(context.ModelId)) throw new ArgumentException("ModelId must be set in the AiAuditContext.", nameof(context));
-        
-        var auditLog = new AiAuditLog
-        {
-            Id = id ?? Guid.NewGuid(),
-            StartTime = DateTime.UtcNow,
-            ParentAuditLogId = parentId,
-            UserId = user?.Id.ToString(),
-            UserName = user?.Name,
-            Capability = context.Capability,
-            ProfileId = context.ProfileId.Value,
-            ProfileAlias = context.ProfileAlias,
-            ProviderId = context.ProviderId,
-            ModelId = context.ModelId,
-            EntityId = context.EntityId,
-            EntityType = context.EntityType,
-            FeatureType = context.FeatureType,
-            FeatureId = context.FeatureId,
-            Metadata = metadata != null
-                ? new Dictionary<string, string>(metadata)
-                : null,
-            DetailLevel = detailLevel
-        };
-
-        // Set initial status to Running
-        auditLog.Status = AiAuditLogStatus.Running;
-        
-        var logger = loggerFactory?.CreateLogger<AiAuditLog>();
-        
-        // Capture response snapshot if configured
-        if (options.PersistPrompts && context.Prompt is not null)
-        {
-            var prompt = FormatPromptSnapshot(context.Prompt, context.Capability);
-            prompt = ApplyRedaction(options, prompt, logger);
-            auditLog.PromptSnapshot = prompt;
-            logger?.LogDebug("Captured prompt snapshot for audit-log {AuditLogId}: {Length} characters",
-                auditLog.Id, prompt?.Length ?? 0);
-        }
-
-        return auditLog;
-    }
-    
-    private static string? FormatPromptSnapshot(object? promptObj, AiCapability capability)
-    {
-        if (promptObj is null)
-        {
-            return null;
-        }
-
-        try
-        {
-            return capability switch
-            {
-                AiCapability.Chat when promptObj is IEnumerable<ChatMessage> messages =>
-                    string.Join("\n", messages.Select(m => $"[{m.Role}] {m.Text}")),
-
-                AiCapability.Embedding when promptObj is IEnumerable<string> values =>
-                    string.Join("\n", values.Select((v, i) => $"[{i}] {v}")),
-
-                _ => promptObj.ToString()
-            };
-        }
-        catch
-        {
-            // If formatting fails, return a fallback representation
-            return $"[Unable to format {capability} prompt]";
-        }
-    }
-    
-    private static string? ApplyRedaction(AiAuditLogOptions options, string? input,
-        ILogger<AiAuditLog>? logger = null)
-    {
-        if (string.IsNullOrEmpty(input) || options.RedactionPatterns.Count == 0)
-            return input;
-
-        var result = input;
-        foreach (var pattern in options.RedactionPatterns)
-        {
-            try
-            {
-                result = Regex.Replace(result, pattern, "[REDACTED]", RegexOptions.IgnoreCase);
-            }
-            catch (Exception ex)
-            {
-                logger?.LogWarning(ex, "Failed to apply redaction pattern: {Pattern}", pattern);
-            }
-        }
-        return result;
-    }
 }