log exception of running an external application

src/Mscc.GenerativeAI/BaseModel.cs

@@ -338,7 +338,7 @@ private string RunExternalExe(string filename, string arguments)
             }
             catch (Exception e)
             {
-                // throw new Exception("OS error while executing " + Format(filename, arguments)+ ": " + e.Message, e);
+                Logger.LogRunExternalExe("OS error while executing " + Format(filename, arguments)+ ": " + e.Message);
                 return string.Empty;
             }
 

src/Mscc.GenerativeAI/CHANGELOG.md

@@ -10,14 +10,17 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 ### Added
 
 - Feature suggestion: Retry mechanism ([#2](https://github.com/mscraftsman/generative-ai/issues/2))
-- Feature suggestion: Add logs with LogLevel using the Standard logging in .NET ([#6](https://github.com/mscraftsman/generative-ai/issues/6))
 - implement Automatic Function Call (AFC)
 
 ### Changed
 ### Fixed
 
 ## 1.8.1
 
+### Added
+
+- add logs with LogLevel using the Standard logging in .NET ([#6](https://github.com/mscraftsman/generative-ai/issues/6)) - thanks @doggy8088 
+
 ### Changed
 
 - improve regarding XMLdoc, typos, and non-nullable properties

src/Mscc.GenerativeAI/Logging/GenerativeModelLogMessages.cs

@@ -44,5 +44,15 @@ public static partial void LogGenerativeModelInvokingRequest(
             string methodName,
             string url,
             string payload);
+
+        /// <summary>
+        /// Logs <see cref="BaseModel"/> when exception thrown to run an external application.
+        /// </summary>
+        /// <param name="logger">Optional. Logger instance used for logging</param>
+        /// <param name="message">Message of <see cref="System.Exception"/> to log.</param>
+        [LoggerMessage(EventId = 0, Level = LogLevel.Warning, Message = "{Message}")]
+        public static partial void LogRunExternalExe(
+            this ILogger logger,
+            string message);
     }
 }