Implement SkippDiff

Azure · Jul 19, 2024 · 2233383 · 2233383
1 parent 1661611
commit 2233383
Show file tree

Hide file tree

Showing 4 changed files with 15 additions and 4 deletions.
diff --git a/src/dotnet/APIView/APIViewWeb/Helpers/CodeFileHelpers.cs b/src/dotnet/APIView/APIViewWeb/Helpers/CodeFileHelpers.cs
@@ -394,6 +394,11 @@ private static void BuildTokensForDiffNodes(CodePanelData codePanelData, CodePan
                 {
                     var token = beforeTokens[beforeIndex++];
 
+                    if (codePanelRawData.ApplySkipDiff && token.TagsObj.Contains(StructuredToken.SKIPP_DIFF))
+                    {
+                        continue;
+                    }
+
                     if (token.Kind == StructuredTokenKind.LineBreak)
                     {
                         break;
@@ -429,6 +434,11 @@ private static void BuildTokensForDiffNodes(CodePanelData codePanelData, CodePan
                 {
                     var token = afterTokens[afterIndex++];
 
+                    if (codePanelRawData.ApplySkipDiff && token.TagsObj.Contains(StructuredToken.SKIPP_DIFF))
+                    {
+                        continue;
+                    }
+
                     if (token.Kind == StructuredTokenKind.LineBreak)
                     {
                         break;

diff --git a/src/dotnet/APIView/APIViewWeb/LeanModels/ReviewRevisionPageModels.cs b/src/dotnet/APIView/APIViewWeb/LeanModels/ReviewRevisionPageModels.cs
@@ -67,6 +67,7 @@ public class CodePanelRawData
         public List<APITreeNode> APIForest { get; set; } = new List<APITreeNode>();
         public CodeDiagnostic[] Diagnostics { get; set; } = new CodeDiagnostic[0];
         public string Language { get; set; }
+        public bool ApplySkipDiff { get; set; }
     }
 
     public class CodePanelRowData

diff --git a/src/dotnet/APIView/APIViewWeb/Managers/CodeFileManager.cs b/src/dotnet/APIView/APIViewWeb/Managers/CodeFileManager.cs
@@ -9,7 +9,6 @@
 using APIViewWeb.Managers.Interfaces;
 using APIViewWeb.Models;
 using APIViewWeb.Repositories;
-using Microsoft.CodeAnalysis.Host;
 
 namespace APIViewWeb.Managers
 {
@@ -208,11 +207,12 @@ public async Task<bool> AreAPICodeFilesTheSame(RenderedCodeFile codeFileA, Rende
 
             if (LanguageServiceHelpers.UseTreeStyleParser(codeFileA.CodeFile.Language))
             {
-                var diffTree =CodeFileHelpers.ComputeAPIForestDiff(codeFileA.CodeFile.APIForest, codeFileB.CodeFile.APIForest);
+                var diffTree = CodeFileHelpers.ComputeAPIForestDiff(codeFileA.CodeFile.APIForest, codeFileB.CodeFile.APIForest);
                 var codePanelRawData = new CodePanelRawData()
                 {
                     APIForest = diffTree,
-                    Language = codeFileA.CodeFile.Language
+                    Language = codeFileA.CodeFile.Language,
+                    ApplySkipDiff = true
                 };
                 var result = await CodeFileHelpers.GenerateCodePanelDataAsync(codePanelRawData);
                 return !result.HasDiff;

diff --git a/src/dotnet/APIView/apiview.yml b/src/dotnet/APIView/apiview.yml
@@ -4,7 +4,7 @@ parameters:
     default: 'https://pkgs.dev.azure.com/azure-sdk/public/_packaging/azure-sdk-for-net/nuget/v3/index.json'
   - name: CSharpAPIParserVersion
     type: string
-    default: '1.0.0-dev.20240716.16'
+    default: '1.0.0-dev.20240719.1'
 
 trigger:
   branches: