fix: add missing context information for Plugin logging

Author: pandinocoder

Framework/Intersect.Framework.Core/Plugins/Interfaces/CreateLoggerOptions.cs

@@ -24,7 +24,12 @@ public record struct CreateLoggerOptions
     public string Name { get; init; }
 
     /// <summary>
-    /// The context type of the logger.
+    /// The context name of the logger. This takes priority over <see cref="ContextType"/>.
     /// </summary>
-    public Type ContextType { get; init; }
+    public string? ContextName { get; init; }
+
+    /// <summary>
+    /// The context type of the logger. This will be ignored if <see cref="ContextName"/> is set.
+    /// </summary>
+    public Type? ContextType { get; init; }
 }

Intersect (Core)/Plugins/Helpers/LoggingHelper.cs

@@ -51,7 +51,7 @@ private static Logger CreateLogger(IManifestHelper manifest, CreateLoggerOptions
 
     public ILogger Plugin { get; }
 
-    internal LoggingHelper(ILogger applicationLogger, IManifestHelper manifest)
+    internal LoggingHelper(ILogger applicationLogger, IManifestHelper manifest, PluginReference pluginReference)
     {
         mManifest = manifest;
 
@@ -60,15 +60,28 @@ internal LoggingHelper(ILogger applicationLogger, IManifestHelper manifest)
             new CreateLoggerOptions
             {
                 Console = Debugger.IsAttached ? LogLevel.Debug : LogLevel.None, File = LogLevel.Information,
+                ContextName = manifest.Name,
+                ContextType = pluginReference.EntryType,
             }
         );
     }
 
     /// <inheritdoc />
-    public ILogger CreateLogger(CreateLoggerOptions createLoggerOptions) =>
-        new SerilogLoggerFactory(CreateLogger(mManifest, createLoggerOptions)).CreateLogger(
-            createLoggerOptions.ContextType
-        );
+    public ILogger CreateLogger(CreateLoggerOptions createLoggerOptions)
+    {
+        var factory = new SerilogLoggerFactory(CreateLogger(mManifest, createLoggerOptions));
+        if (createLoggerOptions.ContextName is { } contextName && !string.IsNullOrWhiteSpace(contextName))
+        {
+            return factory.CreateLogger(contextName);
+        }
+
+        if (createLoggerOptions.ContextType is { } contextType)
+        {
+            return factory.CreateLogger(contextType);
+        }
+
+        return factory.CreateLogger("Plugin");
+    }
 
     /// <inheritdoc />
     public ILogger<TContext> CreateLogger<TContext>(CreateLoggerOptions createLoggerOptions) =>

Intersect (Core)/Plugins/Plugin.cs

@@ -20,7 +20,7 @@ internal static Plugin Create(
         IApplicationContext applicationContext,
         IManifestHelper manifest,
         PluginReference reference
-    ) => new Plugin(manifest, new LoggingHelper(applicationContext.Logger, manifest), reference);
+    ) => new Plugin(manifest, new LoggingHelper(applicationContext.Logger, manifest, reference), reference);
 
     // ReSharper disable once NotNullMemberIsNotInitialized
     // Plugin instance is created at the Discovery phase and Configuration is loaded afterwards