feat: run cs-ide as a persistent JSON-RPC server

.github/coverage-mine.ps1

@@ -46,9 +46,8 @@ $totalLines = 0
 $coveredLines = 0
 
 foreach ($package in $coverage.coverage.packages.package) {
-    $packageName = $package.name
     foreach ($class in $package.classes.class) {
-        $filename = Join-Path $packageName $class.filename
+        $filename = ($class.filename -replace '/', '\')
 
         if ($changedFilesSet.ContainsKey($filename)) {
             $classLines = 0

.github/cwf.ps1

@@ -1,3 +1,43 @@
+param(
+    [switch]$Local
+)
+
+$localCwf = Join-Path (Get-Location) "..\cs-webview"
+$localPackage = Join-Path $localCwf "package.json"
+
+if ($Local) {
+    if (-not (Test-Path $localPackage)) {
+        Write-Error "Local CWF checkout not found at $localCwf"
+        exit 1
+    }
+
+    Write-Host "Building CWF locally from $localCwf..."
+    Push-Location $localCwf
+    try {
+        & npm run build
+        if ($LASTEXITCODE -ne 0) { exit $LASTEXITCODE }
+    }
+    finally {
+        Pop-Location
+    }
+
+    $assetsPath = Join-Path $localCwf "build\assets"
+    if (-not (Test-Path $assetsPath)) {
+        Write-Error "Local CWF build did not produce build\assets."
+        exit 1
+    }
+
+    if (Test-Path cs-cwf) { Remove-Item cs-cwf -Recurse -Force }
+    New-Item -ItemType Directory -Path cs-cwf | Out-Null
+    Copy-Item (Join-Path $assetsPath "*") cs-cwf
+    $webDir = "Codescene.VSExtension.VS2022/Codescene.VSExtension.VS2022/ToolWindows/WebComponent"
+    Copy-Item cs-cwf/index.css $webDir -Force
+    Copy-Item cs-cwf/index.js $webDir -Force
+    if (Test-Path cs-cwf.zip) { Remove-Item cs-cwf.zip -Force }
+    Compress-Archive -Path "cs-cwf\*" -DestinationPath cs-cwf.zip
+    exit 0
+}
+
 $Token = $env:CODESCENE_IDE_DOCS_AND_WEBVIEW_TOKEN
 
 $headers = @{

.github/workflows/benchmarks.yml

@@ -107,20 +107,3 @@ jobs:
           comment-on-alert: true
           fail-on-alert: true
           alert-comment-cc-users: '@martinsafsten-codescene'
-
-      - name: Run CachingCodeReviewer benchmarks
-        shell: pwsh
-        working-directory: Codescene.VSExtension.VS2022
-        run: dotnet run --project Codescene.VSExtension.Core.Benchmarks/Codescene.VSExtension.Core.Benchmarks.csproj -c Release --no-build -- --filter "*CachingCodeReviewerBenchmarks*" --exporters json
-
-      - name: Store CachingCodeReviewer benchmark result
-        uses: benchmark-action/github-action-benchmark@v1
-        with:
-          name: Code Review - CachingCodeReviewer
-          tool: 'benchmarkdotnet'
-          output-file-path: Codescene.VSExtension.VS2022/BenchmarkDotNet.Artifacts/results/Codescene.VSExtension.Core.Benchmarks.CachingCodeReviewerBenchmarks-report-full-compressed.json
-          github-token: ${{ secrets.GITHUB_TOKEN }}
-          auto-push: true
-          comment-on-alert: true
-          fail-on-alert: true
-          alert-comment-cc-users: '@martinsafsten-codescene'

.github/workflows/github-release.yml

@@ -46,7 +46,7 @@ jobs:
     needs: fetch-docs
     # env:
       # Feature flags here. Example:
-      # FEATURE_INITIAL_GIT_OBSERVER: true
+      # FEATURE_ACE: true
     steps:
       - name: Checkout code
         uses: actions/checkout@v4

Codescene.VSExtension.VS2022/Codescene.VSExtension.Core.Benchmarks/BenchmarkSupport.cs

@@ -1,16 +1,10 @@
 // Copyright (c) CodeScene. All rights reserved.
 
-using System.Collections.Concurrent;
-using System.Reflection;
-using Codescene.VSExtension.Core.Application.Cache.Review;
 using Codescene.VSExtension.Core.Application.Cli;
 using Codescene.VSExtension.Core.Interfaces;
 using Codescene.VSExtension.Core.Interfaces.Ace;
 using Codescene.VSExtension.Core.Interfaces.Cli;
 using Codescene.VSExtension.Core.Interfaces.Extension;
-using Codescene.VSExtension.Core.Interfaces.Git;
-using Codescene.VSExtension.Core.Models.Cache.Delta;
-using Codescene.VSExtension.Core.Models.Cache.Review;
 using Codescene.VSExtension.Core.Models.Cli.Refactor;
 using AutoRefactorConfig = Codescene.VSExtension.Core.Models.WebComponent.Data.AutoRefactorConfig;
 
@@ -22,8 +16,7 @@ internal sealed class BenchmarkEnvironment : IDisposable
     private readonly ICliSettingsProvider _cliSettingsProvider = new CliSettingsProvider();
     private readonly ModelMapper _modelMapper = new ModelMapper();
     private readonly FixedSettingsProvider _settingsProvider = new FixedSettingsProvider();
-    private readonly IProcessExecutor _processExecutor;
-    private readonly ICliCommandProvider _commandProvider;
+    private readonly IIdeServerClient _ideServerClient;
     private readonly BenchmarkCacheStorageService _cacheStorageService;
     private bool _disposed;
 
@@ -38,8 +31,7 @@ public BenchmarkEnvironment()
         System.IO.File.WriteAllText(ExistingFilePath, BenchmarkInputs.CurrentCode);
 
         _cacheStorageService = new BenchmarkCacheStorageService(CacheDirectory, RootDirectory);
-        _commandProvider = new CliCommandProvider(new CliObjectScoreCreator(_logger));
-        _processExecutor = CreateProcessExecutor(_cliSettingsProvider, _logger);
+        _ideServerClient = new IdeServerClient(_cliSettingsProvider, _logger, _settingsProvider);
     }
 
     public string RootDirectory { get; }
@@ -52,36 +44,12 @@ public CliExecutor CreateCliExecutor()
     {
         return new CliExecutor(
             _logger,
-            new BenchmarkCliServices(_commandProvider, _processExecutor, _cacheStorageService),
+            _ideServerClient,
+            _cacheStorageService,
             _settingsProvider,
             null);
     }
 
-    public CodeReviewer CreateCodeReviewer(IGitService gitService, IPreflightManager? preflightManager = null)
-    {
-        return new CodeReviewer(
-            _logger,
-            _modelMapper,
-            CreateCliExecutor(),
-            null,
-            gitService,
-            null,
-            preflightManager);
-    }
-
-    public CachingCodeReviewer CreateCachingCodeReviewer(IGitService gitService, IPreflightManager? preflightManager = null)
-    {
-        return new CachingCodeReviewer(
-            CreateCodeReviewer(gitService, preflightManager),
-            new ReviewCacheService(new ConcurrentDictionary<string, ReviewCacheItem>()),
-            new BaselineReviewCacheService(new ConcurrentDictionary<string, string>()),
-            new DeltaCacheService(new ConcurrentDictionary<string, DeltaCacheItem>()),
-            _logger,
-            gitService,
-            null,
-            null);
-    }
-
     public void Dispose()
     {
         if (_disposed)
@@ -90,6 +58,7 @@ public void Dispose()
         }
 
         _disposed = true;
+        _ideServerClient.Dispose();
         try
         {
             Directory.Delete(RootDirectory, true);
@@ -98,20 +67,6 @@ public void Dispose()
         {
         }
     }
-
-    private static IProcessExecutor CreateProcessExecutor(ICliSettingsProvider cliSettingsProvider, ILogger logger)
-    {
-        var processExecutorType = typeof(CliSettingsProvider).Assembly.GetType(
-            "Codescene.VSExtension.Core.Application.Cli.ProcessExecutor",
-            throwOnError: true);
-        var processExecutor = (IProcessExecutor?)Activator.CreateInstance(
-            processExecutorType,
-            BindingFlags.Instance | BindingFlags.Public | BindingFlags.NonPublic,
-            null,
-            new object[] { cliSettingsProvider, logger },
-            null);
-        return processExecutor!;
-    }
 }
 
 internal static class BenchmarkInputs
@@ -153,22 +108,6 @@ public int Calculate(int a, int b, int c, int d, int e, int f, int g, int h)
 }";
 }
 
-internal sealed class BenchmarkCliServices : ICliServices
-{
-    public BenchmarkCliServices(ICliCommandProvider commandProvider, IProcessExecutor processExecutor, ICacheStorageService cacheStorage)
-    {
-        CommandProvider = commandProvider;
-        ProcessExecutor = processExecutor;
-        CacheStorage = cacheStorage;
-    }
-
-    public ICliCommandProvider CommandProvider { get; }
-
-    public IProcessExecutor ProcessExecutor { get; }
-
-    public ICacheStorageService CacheStorage { get; }
-}
-
 internal sealed class BenchmarkCacheStorageService : ICacheStorageService
 {
     public BenchmarkCacheStorageService(string cacheDirectory, string workspaceDirectory)
@@ -220,53 +159,12 @@ internal sealed class FixedSettingsProvider : ISettingsProvider
     public bool ShowDebugLogs => false;
 
     public string AuthToken => string.Empty;
-}
 
-internal sealed class FixedGitService : IGitService
-{
-    private readonly string _defaultBaselineContent;
-    private readonly Dictionary<string, string> _pathContents = new Dictionary<string, string>(StringComparer.OrdinalIgnoreCase);
+    public int ServerWorkerThreads => 0;
 
-    public FixedGitService(string defaultBaselineContent)
-    {
-        _defaultBaselineContent = defaultBaselineContent;
-    }
-
-    public void SetContent(string path, string content)
-    {
-        _pathContents[path] = content;
-    }
-
-    public string GetBaselineCommit(string repoRootPath)
-    {
-        return string.Empty;
-    }
-
-    public string GetFileContentForCommit(string path)
-    {
-        return GetFileContentForCommit(path, string.Empty);
-    }
+    public bool EnableAutomaticAnalysis => true;
 
-    public string GetFileContentForCommit(string path, string baselineCommit)
-    {
-        if (path != null && _pathContents.TryGetValue(path, out var content))
-        {
-            return content;
-        }
-
-        return _defaultBaselineContent;
-    }
-
-    public bool IsFileIgnored(string filePath) => false;
-
-    public HashSet<string> FilterIgnoredFiles(IEnumerable<string> absolutePaths)
-    {
-        return new HashSet<string>(absolutePaths, StringComparer.OrdinalIgnoreCase);
-    }
-
-    public void Dispose()
-    {
-    }
+    public bool EnableTelemetry => true;
 }
 
 internal sealed class FixedPreflightManager : IPreflightManager