Added logging when solution (workspace) is changed. (#116)

Improved logging for SolutionService.
sourcegraph · Oct 16, 2024 · e742b5e · e742b5e
1 parent d22b8b7
commit e742b5e
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 8 deletions.
diff --git a/src/Cody.VisualStudio/CodyPackage.cs b/src/Cody.VisualStudio/CodyPackage.cs
@@ -127,7 +127,7 @@ private void InitializeServices()
             Logger = loggerFactory.Create();
 
             var vsSolution = this.GetService<SVsSolution, IVsSolution>();
-            SolutionService = new SolutionService(vsSolution);
+            SolutionService = new SolutionService(vsSolution, Logger);
             VersionService = loggerFactory.GetVersionService();
             VsVersionService = new VsVersionService(Logger);
 
@@ -420,6 +420,7 @@ private void UpdateCurrentWorkspaceFolder()
                     Uris = new List<string> { solutionUri }
                 };
                 AgentService.WorkspaceFolderDidChange(workspaceFolderEvent);
+                Logger.Debug($"Workspace updated:{solutionUri}");
 
                 if (DocumentsSyncService == null)
                 {

diff --git a/src/Cody.VisualStudio/Services/SolutionService.cs b/src/Cody.VisualStudio/Services/SolutionService.cs
@@ -3,33 +3,40 @@
 using System;
 using System.Diagnostics;
 using System.IO;
+using Cody.Core.Logging;
 
 namespace Cody.VisualStudio.Services
 {
     public class SolutionService : ISolutionService
     {
-        private IVsSolution solutionService;
+        private readonly IVsSolution _vsSolution;
+        private readonly ILog _logger;
 
-        public SolutionService(IVsSolution solutionService)
+        public SolutionService(IVsSolution solutionService, ILog logger)
         {
-            this.solutionService = solutionService;
+            _vsSolution = solutionService;
+            _logger = logger;
         }
 
         public bool IsSolutionOpen()
         {
-            solutionService.GetProperty((int)__VSPROPID.VSPROPID_IsSolutionOpen, out object value);
-            return value is bool isSolutionOpen && isSolutionOpen;
+            _vsSolution.GetProperty((int)__VSPROPID.VSPROPID_IsSolutionOpen, out object value);
+            var isOpened = value is bool isSolutionOpen && isSolutionOpen;
+
+            _logger.Debug($"Is solution opened:{isOpened}");
+
+            return isOpened;
         }
 
         public string GetSolutionDirectory()
         {
-            solutionService.GetProperty((int)__VSPROPID.VSPROPID_SolutionFileName, out object value);
+            _vsSolution.GetProperty((int)__VSPROPID.VSPROPID_SolutionFileName, out object value);
             var solutionDir = Path.GetDirectoryName(value as string);
 
             // Use user directory if solution directory is null.
             var userDir = Environment.GetFolderPath(Environment.SpecialFolder.UserProfile);
 
-            Debug.WriteLine(solutionDir, "GetSolutionDirectory");
+            _logger.Debug($"'{solutionDir}'");
 
             return new Uri(solutionDir ?? userDir).ToString();
         }