Lint/Format check in CI

.github/workflows/code-style.yml

@@ -0,0 +1,19 @@
+name: Code Style Checks
+
+on:
+  push:
+    branches:
+      - main
+  pull_request:
+    branches:
+      - main
+
+jobs:
+  build:
+    runs-on: windows-latest
+    steps:
+    - uses: actions/checkout@v3
+
+    - name: Check Formatting
+      run: dotnet format ./src/ --verify-no-changes -v:diag
+

src/Cody.AgentTester/AssemblyLoader.cs

@@ -70,10 +70,10 @@ private static string SelectInstallPath()
         private static string finalPath = null;
         private static string SelectStreamJsonRpcPath()
         {
-            if(!string.IsNullOrEmpty(finalPath)) return finalPath;
+            if (!string.IsNullOrEmpty(finalPath)) return finalPath;
 
             var vsPath = SelectInstallPath();
-            foreach(var folder in folders)
+            foreach (var folder in folders)
             {
                 var path = Path.Combine(vsPath, folder);
                 var libPath = Path.Combine(path, "StreamJsonRpc.dll");

src/Cody.AgentTester/ConsoleLogger.cs

@@ -1,4 +1,4 @@
-using Cody.Core.Logging;
+using Cody.Core.Logging;
 using System;
 using System.Collections.Generic;
 using System.Linq;
@@ -43,7 +43,7 @@ public void Warn(string message, [CallerMemberName] string callerName = "")
         private void WriteToConsole(string prefix, string message, string callerName, Exception ex = null)
         {
             Console.WriteLine($"{prefix.ToUpper()} {callerName}: {message}");
-            if(ex != null) Console.WriteLine(ex.ToString());
+            if (ex != null) Console.WriteLine(ex.ToString());
         }
     }
 }

src/Cody.AgentTester/Properties/AssemblyInfo.cs

@@ -1,4 +1,4 @@
-using System.Reflection;
+using System.Reflection;
 using System.Runtime.CompilerServices;
 using System.Runtime.InteropServices;
 

src/Cody.Core/Agent/NotificationHandlers.cs

@@ -50,10 +50,10 @@ public async Task SendWebviewMessage(string handle, string message)
             bool handled = _messageFilter.HandleMessage(message);
             if (!handled)
                 await agentClient.ReceiveMessageStringEncoded(new ReceiveMessageStringEncodedParams
-            {
-                Id = handle,
-                MessageStringEncoded = message
-            });
+                {
+                    Id = handle,
+                    MessageStringEncoded = message
+                });
         }
 
         [AgentCallback("debug/message")]

src/Cody.Core/Agent/ProgressNotificationHandlers.cs

@@ -25,7 +25,7 @@ public ProgressNotificationHandlers(IProgressService progressService)
         public void Start(ProgressStartParams progressStart)
         {
             Action cancelAction = null;
-            if(progressStart.Options.Cancellable == true)
+            if (progressStart.Options.Cancellable == true)
             {
                 cancelAction = () => agentService.CancelProgress(progressStart.Id);
             };

src/Cody.Core/Agent/Protocol/AuthStatus.cs

@@ -1,4 +1,4 @@
-using System;
+using System;
 using System.Collections.Generic;
 using System.Linq;
 using System.Text;

src/Cody.Core/Agent/Protocol/ChatPanelInfo.cs

@@ -1,4 +1,4 @@
-namespace Cody.Core.Agent.Protocol
+namespace Cody.Core.Agent.Protocol
 {
     public class ChatPanelInfo
     {

src/Cody.Core/Agent/Protocol/CodyFilePath.cs

@@ -1,4 +1,4 @@
-namespace Cody.Core.Agent.Protocol
+namespace Cody.Core.Agent.Protocol
 {
     public class CodyFilePath
     {

src/Cody.Core/Agent/Protocol/ConfigOverwrites.cs

@@ -1,4 +1,4 @@
-using System;
+using System;
 using System.Collections.Generic;
 using System.Linq;
 using System.Text;

src/Cody.Core/Agent/Protocol/CurrentUserCodySubscription.cs

@@ -1,4 +1,4 @@
-using System;
+using System;
 using System.Collections.Generic;
 using System.Linq;
 using System.Text;

src/Cody.Core/Agent/Protocol/Position.cs

@@ -1,4 +1,4 @@
-using System;
+using System;
 using System.Collections.Generic;
 using System.Linq;
 using System.Text;

src/Cody.Core/Agent/Protocol/ProtocolTextDocument.cs

@@ -1,4 +1,4 @@
-using System;
+using System;
 using System.Collections.Generic;
 using System.Linq;
 using System.Text;

src/Cody.Core/Agent/Protocol/ProtocolTextDocumentContentChangeEvent.cs

@@ -1,4 +1,4 @@
-using System;
+using System;
 using System.Collections.Generic;
 using System.Linq;
 using System.Text;

src/Cody.Core/Agent/Protocol/Range.cs

@@ -1,4 +1,4 @@
-using System;
+using System;
 using System.Collections.Generic;
 using System.Linq;
 using System.Text;

src/Cody.Core/Agent/Protocol/ReceiveMessageStringEncodedParams.cs

@@ -1,4 +1,4 @@
-namespace Cody.Core.Agent.Protocol
+namespace Cody.Core.Agent.Protocol
 {
     public class ReceiveMessageStringEncodedParams
     {

src/Cody.Core/Agent/Protocol/ResolveWebviewViewParams.cs

@@ -1,4 +1,4 @@
-using System;
+using System;
 
 namespace Cody.Core.Agent.Protocol
 {

src/Cody.Core/Agent/Protocol/ServerInfo.cs

@@ -1,4 +1,4 @@
-using System;
+using System;
 using System.Collections.Generic;
 using System.Linq;
 using System.Text;

src/Cody.Core/Agent/Protocol/TextDocumentShowParams.cs

@@ -1,4 +1,4 @@
-namespace Cody.Core.Agent.Protocol
+namespace Cody.Core.Agent.Protocol
 {
     public class TextDocumentShowParams
     {

src/Cody.Core/Agent/Protocol/TextDocumentShowParamsOptions.cs

@@ -1,4 +1,4 @@
-using System;
+using System;
 using System.Collections.Generic;
 using System.Linq;
 using System.Text;

src/Cody.Core/Agent/SetHtmlEvent.cs

@@ -1,4 +1,4 @@
-using System;
+using System;
 
 namespace Cody.Core.Agent
 {
@@ -20,4 +20,4 @@ public class AgentResponseEvent : EventArgs
         public string Id { get; set; }
         public string StringEncodedMessage { get; set; }
     }
-}
+}

src/Cody.Core/DocumentSync/DocumentSyncCallback.cs

@@ -1,4 +1,4 @@
-using Cody.Core.Agent;
+using Cody.Core.Agent;
 using Cody.Core.Agent.Protocol;
 using Cody.Core.Logging;
 using System;

src/Cody.Core/DocumentSync/IDocumentSyncActions.cs

@@ -1,4 +1,4 @@
-using System;
+using System;
 using System.Collections.Generic;
 using System.Linq;
 using System.Text;

src/Cody.Core/Ide/IVsVersionService.cs

@@ -1,4 +1,4 @@
-using System;
+using System;
 
 namespace Cody.Core.Ide
 {

src/Cody.Core/Infrastructure/IStatusbarService.cs

@@ -1,4 +1,4 @@
-using System;
+using System;
 using System.Collections.Generic;
 using System.Linq;
 using System.Text;

src/Cody.Core/Infrastructure/IVersionService.cs

@@ -1,4 +1,4 @@
-using System;
+using System;
 
 namespace Cody.Core.Inf
 {
@@ -11,4 +11,4 @@ public interface IVersionService
         DateTime BuildDate { get; }
         void AddBuildMetadata(string build, bool isDebug);
     }
-}
+}

src/Cody.Core/Infrastructure/VersionService.cs

@@ -1,4 +1,4 @@
-using System;
+using System;
 using System.Reflection;
 
 namespace Cody.Core.Inf

src/Cody.Core/Infrastructure/WebViewsManager.cs

@@ -29,7 +29,7 @@ public WebViewEvent(WebViewsEventTypes type, string viewId = null)
             ViewId = viewId;
         }
 
-        public WebViewsEventTypes Type { get;}
+        public WebViewsEventTypes Type { get; }
         public string ViewId { get; }
     }
 
@@ -181,5 +181,5 @@ public void Dispose()
             _events?.Dispose();
         }
     }
-    
+
 }

src/Cody.Core/Logging/ILog.cs

@@ -1,4 +1,4 @@
-using System;
+using System;
 using System.Runtime.CompilerServices;
 
 namespace Cody.Core.Logging

src/Cody.Core/Logging/IOutputWindowPane.cs

@@ -1,4 +1,4 @@
-namespace Cody.Core.Logging
+namespace Cody.Core.Logging
 {
     public interface IOutputWindowPane
     {

src/Cody.Core/Properties/AssemblyInfo.cs

@@ -1,4 +1,4 @@
-using System.Reflection;
+using System.Reflection;
 using System.Runtime.CompilerServices;
 using System.Runtime.InteropServices;
 

src/Cody.Core/Settings/IUserSettingsProvider.cs

@@ -1,4 +1,4 @@
-using System;
+using System;
 using System.Collections.Generic;
 using System.Linq;
 using System.Text;

src/Cody.UI/Controls/WebView2Dev.xaml.cs

@@ -67,7 +67,7 @@ private async Task InitializeWebView()
 
                 Logger?.Error("Failed.", ex);
             }
-            
+
         }
 
         private async void WebView_NavigationCompleted(object sender, CoreWebView2NavigationCompletedEventArgs e)
@@ -118,7 +118,7 @@ public static async Task PostWebMessageAsJson(string message)
 
         public static readonly DependencyProperty HtmlProperty = DependencyProperty.Register(
             "Html", typeof(string), typeof(WebView2Dev),
-            new PropertyMetadata(null, async (d, e) =>
+            new PropertyMetadata(null, (d, e) =>
             {
                 _controller.SetHtml(e.NewValue as string);
             }));
@@ -159,7 +159,7 @@ public ICommand SendMessage
 
         public static readonly DependencyProperty LoggerProperty = DependencyProperty.Register(
             "Logger", typeof(ILog), typeof(WebView2Dev),
-            new PropertyMetadata(null, async (d, e) =>
+            new PropertyMetadata(null, (d, e) =>
             {
                 var logger = e.NewValue as ILog;
                 if (logger != null)

src/Cody.UI/MVVM/DelegateCommand.cs

@@ -1,4 +1,4 @@
-using System;
+using System;
 using System.ComponentModel;
 using System.Windows.Input;
 

src/Cody.UI/MVVM/NotifyPropertyChangedBase.cs

@@ -1,4 +1,4 @@
-using System;
+using System;
 using System.ComponentModel;
 using System.Linq.Expressions;
 using System.Runtime.CompilerServices;

src/Cody.UI/Properties/AssemblyInfo.cs

@@ -1,4 +1,4 @@
-using System.Reflection;
+using System.Reflection;
 using System.Runtime.CompilerServices;
 using System.Runtime.InteropServices;
 
@@ -29,4 +29,4 @@
 //
 // You can specify all the values or you can default the Build and Revision Numbers
 // by using the '*' as shown below:
-// [assembly: AssemblyVersion("1.0.*")]
+// [assembly: AssemblyVersion("1.0.*")]

src/Cody.UI/ViewModels/GeneralOptionsViewModel.cs

@@ -5,7 +5,7 @@
 
 namespace Cody.UI.ViewModels
 {
-    public class GeneralOptionsViewModel: NotifyPropertyChangedBase
+    public class GeneralOptionsViewModel : NotifyPropertyChangedBase
     {
         private readonly ILog _logger;
 
@@ -31,7 +31,7 @@ private void GetHelpCommandHandler()
                 uri = new Uri("https://community.sourcegraph.com/").AbsoluteUri;
                 Process.Start(new ProcessStartInfo(uri));
             }
-            catch(Exception ex)
+            catch (Exception ex)
             {
                 _logger.Error($"Opening '{uri}' failed.", ex);
             }

src/Cody.UI/ViewModels/MainViewModel.cs

@@ -36,7 +36,7 @@ private void OnPostMessageHandler(object sender, AgentResponseEvent e)
             };
         }
 
-        private async void OnWebviewRequestHandler(object sender, SetWebviewRequestEvent e)
+        private void OnWebviewRequestHandler(object sender, SetWebviewRequestEvent e)
         {
             NotificationHandlers.SendWebviewMessage(e.Handle, e.Messsage);
         }

src/Cody.UI/Views/MainView.xaml.cs

@@ -1,4 +1,4 @@
-using System;
+using System;
 using System.Collections.Generic;
 using System.Linq;
 using System.Text;

src/Cody.UITester/App.xaml.cs

@@ -1,4 +1,4 @@
-using System;
+using System;
 using System.Collections.Generic;
 using System.Configuration;
 using System.Data;

src/Cody.UITester/MainWindow.xaml.cs

@@ -1,4 +1,4 @@
-using System;
+using System;
 using System.Collections.Generic;
 using System.Linq;
 using System.Text;

src/Cody.UITester/Properties/AssemblyInfo.cs

@@ -1,4 +1,4 @@
-using System.Reflection;
+using System.Reflection;
 using System.Resources;
 using System.Runtime.CompilerServices;
 using System.Runtime.InteropServices;

src/Cody.VisualStudio.Tests/ChatLoggedBasicTests.cs

@@ -7,7 +7,7 @@
 
 namespace Cody.VisualStudio.Tests
 {
-    public class ChatLoggedBasicTests: PlaywrightInitializationTests
+    public class ChatLoggedBasicTests : PlaywrightInitializationTests
     {
         public ChatLoggedBasicTests(ITestOutputHelper output) : base(output)
         {

src/Cody.VisualStudio.Tests/PlaywrightTestsBase.cs

@@ -11,7 +11,7 @@
 
 namespace Cody.VisualStudio.Tests
 {
-    public abstract class PlaywrightTestsBase: TestsBase
+    public abstract class PlaywrightTestsBase : TestsBase
     {
         protected string CdpAddress = $"http://localhost:{9222}";
 
@@ -145,10 +145,10 @@ protected async Task<IReadOnlyCollection<ContextTag>> GetChatContextTags()
                 var content = await item.TextContentAsync();
                 var parts = content.Split(':');
                 tag.Name = parts.First();
-                if(parts.Length > 1)
+                if (parts.Length > 1)
                 {
                     var lines = parts[1].Split('-');
-                    if(lines.Length > 1)
+                    if (lines.Length > 1)
                     {
                         tag.StartLine = int.Parse(lines[0]);
                         tag.EndLine = int.Parse(lines[1]);