SF-3609 Use CanDenormalizeQuotes flag from Serval

Author: Nateowami

src/SIL.XForge.Scripture/ClientApp/src/app/machine-api/build-dto.ts

@@ -1,6 +1,5 @@
 import { ProjectScriptureRange } from 'realtime-server/lib/esm/scriptureforge/models/translate-config';
 import { BuildStates } from './build-states';
-import { QuotationAnalysis } from './quotation-denormalization';
 import { ResourceDto } from './resource-dto';
 
 export interface BuildDto extends ResourceDto {
@@ -26,5 +25,5 @@ export interface ServalBuildAdditionalInfo {
   translationScriptureRanges: ProjectScriptureRange[];
   trainingDataFileIds: string[];
   requestedByUserId?: string;
-  quotationDenormalization: QuotationAnalysis;
+  canDenormalizeQuotes: boolean;
 }

src/SIL.XForge.Scripture/ClientApp/src/app/machine-api/quotation-denormalization.ts

@@ -27,7 +27,6 @@ import { OnlineStatusService } from 'xforge-common/online-status.service';
 import { filterNullish, quietTakeUntilDestroyed } from 'xforge-common/util/rxjs-util';
 import { TextDocId } from '../../../core/models/text-doc';
 import { SFProjectService } from '../../../core/sf-project.service';
-import { QuotationAnalysis } from '../../../machine-api/quotation-denormalization';
 import { ServalAdministrationService } from '../../../serval-administration/serval-administration.service';
 import { BookChapterChooserComponent } from '../../../shared/book-chapter-chooser/book-chapter-chooser.component';
 import { NoticeComponent } from '../../../shared/notice/notice.component';
@@ -79,7 +78,7 @@ export class DraftUsfmFormatComponent extends DataLoadingComponent implements Af
 
   private updateDraftConfig$: Subject<DraftUsfmConfig | undefined> = new Subject<DraftUsfmConfig | undefined>();
   private lastSavedState?: DraftUsfmConfig;
-  private quotationDenormalization: QuotationAnalysis = QuotationAnalysis.Successful;
+  private canDenormalizeQuotes?: boolean = undefined;
 
   constructor(
     private readonly activatedRoute: ActivatedRoute,
@@ -100,10 +99,7 @@ export class DraftUsfmFormatComponent extends DataLoadingComponent implements Af
       .pipe(filterNullish(), first(), quietTakeUntilDestroyed(this.destroyRef))
       .subscribe(async projectId => {
         const currentBuild = await firstValueFrom(this.draftGenerationService.getLastCompletedBuild(projectId));
-        this.quotationDenormalization =
-          currentBuild?.additionalInfo?.quotationDenormalization === QuotationAnalysis.Successful
-            ? QuotationAnalysis.Successful
-            : QuotationAnalysis.Unsuccessful;
+        this.canDenormalizeQuotes = currentBuild?.additionalInfo?.canDenormalizeQuotes;
       });
   }
 
@@ -125,7 +121,7 @@ export class DraftUsfmFormatComponent extends DataLoadingComponent implements Af
   }
 
   get showQuoteFormatWarning(): boolean {
-    return this.quotationDenormalization !== QuotationAnalysis.Successful;
+    return this.canDenormalizeQuotes === false;
   }
 
   private get currentFormat(): DraftUsfmConfig | undefined {

src/SIL.XForge.Scripture/ClientApp/src/app/translate/draft-generation/draft-usfm-format/draft-usfm-format.component.ts

@@ -1,8 +1,5 @@
 using System;
 using System.Collections.Generic;
-using Newtonsoft.Json;
-using Newtonsoft.Json.Converters;
-using SIL.Converters.Usj;
 
 namespace SIL.XForge.Scripture.Models;
 
@@ -20,13 +17,5 @@ public class ServalBuildAdditionalInfo
     public HashSet<string> TrainingDataFileIds { get; init; } = [];
     public string TranslationEngineId { get; init; } = string.Empty;
     public HashSet<ProjectScriptureRange> TranslationScriptureRanges { get; init; } = [];
-
-    [JsonConverter(typeof(StringEnumConverter), typeof(LowerCaseNamingStrategy))]
-    public QuotationAnalysis QuotationDenormalization { get; set; } = QuotationAnalysis.Unsuccessful;
-}
-
-public enum QuotationAnalysis
-{
-    Successful,
-    Unsuccessful,
+    public bool CanDenormalizeQuotes { get; set; } = false;
 }

src/SIL.XForge.Scripture/Models/ServalBuildAdditionalInfo.cs

@@ -2251,6 +2251,8 @@ string bookId in ScriptureRangeParser
     private static ServalBuildDto CreateDto(TranslationBuild translationBuild)
     {
         string? parallelCorpusId = translationBuild.Pretranslate?.FirstOrDefault()?.ParallelCorpus?.Id;
+        var matchingCorpus = translationBuild.Analysis?.FirstOrDefault(a => a.ParallelCorpusRef == parallelCorpusId);
+
         var buildDto = new ServalBuildDto
         {
             Id = translationBuild.Id,
@@ -2285,16 +2287,7 @@ .. translationBuild
                             .Where(id => !string.IsNullOrEmpty(id)) ?? [],
                     ]
                 ),
-                QuotationDenormalization =
-                    parallelCorpusId is not null
-                    && translationBuild.Analysis?.FirstOrDefault(a =>
-                        a.ParallelCorpusRef == parallelCorpusId
-                        && !string.IsNullOrEmpty(a.SourceQuoteConvention)
-                        && !string.IsNullOrEmpty(a.TargetQuoteConvention)
-                    )
-                        is not null
-                        ? QuotationAnalysis.Successful
-                        : QuotationAnalysis.Unsuccessful,
+                CanDenormalizeQuotes = matchingCorpus?.CanDenormalizeQuotes == true,
                 DateFinished = translationBuild.DateFinished,
                 Step = translationBuild.Step,
                 TranslationEngineId = translationBuild.Engine.Id,

src/SIL.XForge.Scripture/Services/MachineApiService.cs

@@ -989,6 +989,7 @@ public async Task GetBuildAsync_IncludesAdditionalInfo()
                     ParallelCorpusRef = parallelCorpusId1,
                     SourceQuoteConvention = "standard_english",
                     TargetQuoteConvention = "standard_english",
+                    CanDenormalizeQuotes = true,
                 },
             ],
         };
@@ -1027,7 +1028,7 @@ public async Task GetBuildAsync_IncludesAdditionalInfo()
         Assert.AreEqual(parallelCorpusId1, actual.AdditionalInfo.ParallelCorporaIds!.ElementAt(0));
         Assert.AreEqual(parallelCorpusId2, actual.AdditionalInfo.ParallelCorporaIds.ElementAt(1));
         Assert.AreEqual(TrainingDataId01, actual.AdditionalInfo.TrainingDataFileIds.Single());
-        Assert.AreEqual(actual.AdditionalInfo.QuotationDenormalization, QuotationAnalysis.Successful);
+        Assert.That(actual.AdditionalInfo.CanDenormalizeQuotes, Is.True);
     }
 
     [Test]
@@ -1274,6 +1275,7 @@ public async Task GetBuildsAsync_SuccessWithEventMetrics()
                 ParallelCorpusRef = ParallelCorpusId01,
                 SourceQuoteConvention = "standard_english",
                 TargetQuoteConvention = "standard_english",
+                CanDenormalizeQuotes = true,
             },
         ];
         translationBuild.Pretranslate =
@@ -1361,7 +1363,7 @@ public async Task GetBuildsAsync_SuccessWithEventMetrics()
             builds[0].AdditionalInfo.TrainingScriptureRanges.Single().ScriptureRange
         );
         Assert.AreEqual(TrainingDataId01, builds[0].AdditionalInfo.TrainingDataFileIds.Single());
-        Assert.AreEqual(builds[0].AdditionalInfo!.QuotationDenormalization, QuotationAnalysis.Successful);
+        Assert.That(builds[0].AdditionalInfo.CanDenormalizeQuotes, Is.True);
     }
 
     [Test]
@@ -1403,7 +1405,7 @@ public async Task GetBuildsAsync_SuccessWithFallbackToLegacyBuild()
             trainingScriptureRange,
             builds[0].AdditionalInfo?.TrainingScriptureRanges.Single().ScriptureRange
         );
-        Assert.AreEqual(builds[0].AdditionalInfo!.QuotationDenormalization, QuotationAnalysis.Unsuccessful);
+        Assert.That(builds[0].AdditionalInfo!.CanDenormalizeQuotes, Is.False);
     }
 
     [Test]
@@ -1455,7 +1457,7 @@ public async Task GetBuildsAsync_SuccessWithFallbackToPreTranslateBuild()
             trainingScriptureRange,
             builds[0].AdditionalInfo?.TrainingScriptureRanges.Single().ScriptureRange
         );
-        Assert.AreEqual(builds[0].AdditionalInfo!.QuotationDenormalization, QuotationAnalysis.Unsuccessful);
+        Assert.That(builds[0].AdditionalInfo!.CanDenormalizeQuotes, Is.False);
     }
 
     [Test]