@finos/legend-application-studio 28.15.4 → 28.15.6

package/src/stores/project-reviewer/ProjectReviewerStore.ts

@@ -22,6 +22,7 @@ import {
   guaranteeNonNullable,
   ActionState,
   assertNonEmptyString,
+  isNonNullable,
 } from '@finos/legend-shared';
 import {
   makeObservable,
@@ -53,38 +54,57 @@ export class ProjectReviewReport {
   diffs: EntityDiff[];
   fromEntities: Entity[] = [];
   toEntities: Entity[] = [];
+  private fromConfig: PlainObject<ProjectConfiguration> | undefined;
+  private toConfig: PlainObject<ProjectConfiguration> | undefined;
   fromToProjectConfig:
     | [PlainObject<ProjectConfiguration>, PlainObject<ProjectConfiguration>]
     | undefined;
 
   constructor(diffs: EntityDiff[]) {
     this.diffs = diffs;
+
+    makeObservable(this, {
+      fromEntities: observable,
+      toEntities: observable,
+      fromToProjectConfig: observable,
+      addFromConfig: action,
+      addToConfig: action,
+    });
   }
 
-  findFromEntity(entityPath: string): Entity | undefined {
-    return this.fromEntities.find((e) => e.path === entityPath);
+  addFromConfig(config: PlainObject<ProjectConfiguration>): void {
+    this.fromConfig = config;
+    this.createFromToConfigIfPossible();
   }
 
-  findToEntity(entityPath: string): Entity | undefined {
-    return this.toEntities.find((e) => e.path === entityPath);
+  addToConfig(config: PlainObject<ProjectConfiguration>): void {
+    this.toConfig = config;
+    this.createFromToConfigIfPossible();
   }
 
-  withFromEntities(val: Entity[]): ProjectReviewReport {
-    this.fromEntities = val;
-    return this;
+  private createFromToConfigIfPossible(): void {
+    if (this.fromConfig && this.toConfig) {
+      this.fromToProjectConfig = [this.fromConfig, this.toConfig];
+    }
   }
 
-  withToEntities(val: Entity[]): ProjectReviewReport {
-    this.toEntities = val;
+  findFromEntity(entityPath: string): Entity | undefined {
+    return this.fromEntities.find((e) => e.path === entityPath);
+  }
+
+  addFromEntity(entity: Entity): ProjectReviewReport {
+    this.fromEntities.push(entity);
     return this;
   }
 
-  withProjectConfigChange(
-    val: [PlainObject<ProjectConfiguration>, PlainObject<ProjectConfiguration>],
-  ): ProjectReviewReport {
-    this.fromToProjectConfig = val;
+  addToEntity(entity: Entity): ProjectReviewReport {
+    this.toEntities.push(entity);
     return this;
   }
+
+  findToEntity(entityPath: string): Entity | undefined {
+    return this.toEntities.find((e) => e.path === entityPath);
+  }
 }
 
 export class ProjectReviewerStore {
@@ -96,8 +116,7 @@ export class ProjectReviewerStore {
   currentReviewId?: string | undefined;
   currentReview?: Review | undefined;
   // comparison
-  fetchComparisonState = ActionState.create();
-  reviewComparison?: Comparison | undefined;
+  buildReviewReportState = ActionState.create();
   reviewReport?: ProjectReviewReport | undefined;
 
   fetchCurrentReviewState = ActionState.create();
@@ -116,7 +135,7 @@ export class ProjectReviewerStore {
       currentReviewId: observable,
       currentReview: observable,
       fetchCurrentReviewState: observable,
-      fetchComparisonState: observable,
+      buildReviewReportState: observable,
       approveState: observable,
       closeState: observable,
       commitState: observable,
@@ -160,13 +179,6 @@ export class ProjectReviewerStore {
     return guaranteeNonNullable(this.currentReview, 'Review must exist');
   }
 
-  get comparison(): Comparison {
-    return guaranteeNonNullable(
-      this.reviewComparison,
-      'review Comparison must exist',
-    );
-  }
-
   get approvalString(): string | undefined {
     const approvals = this.reviewApproval?.approvedBy;
     if (approvals?.length) {
@@ -267,7 +279,6 @@ export class ProjectReviewerStore {
 
   refresh(): void {
     this.editorStore.tabManagerState.closeAllTabs();
-    this.reviewComparison = undefined;
     this.reviewReport = undefined;
     flowResult(this.fetchReviewComparison()).catch(
       this.editorStore.applicationStore.alertUnhandledError,
@@ -279,52 +290,66 @@ export class ProjectReviewerStore {
    * assume the review has completed been fetched
    */
   *fetchReviewComparison(): GeneratorFn<void> {
-    this.fetchComparisonState.inProgress();
+    this.buildReviewReportState.inProgress();
     try {
-      const [comparison, fromEntities, toEntities] = (yield Promise.all([
-        this.editorStore.sdlcServerClient.getReviewComparision(
-          this.projectId,
-          this.patchReleaseVersionId,
-          this.reviewId,
-        ),
-        this.editorStore.sdlcServerClient.getReviewFromEntities(
-          this.projectId,
-          this.patchReleaseVersionId,
-          this.reviewId,
-        ),
-        this.editorStore.sdlcServerClient.getReviewToEntities(
+      const comparison = Comparison.serialization.fromJson(
+        (yield this.editorStore.sdlcServerClient.getReviewComparision(
           this.projectId,
           this.patchReleaseVersionId,
           this.reviewId,
-        ),
-      ])) as [PlainObject<Comparison>, Entity[], Entity[]];
-      const resolvedComparison = Comparison.serialization.fromJson(comparison);
-      this.reviewComparison = resolvedComparison;
+        )) as PlainObject<Comparison>,
+      );
       const report = new ProjectReviewReport(
-        reprocessEntityDiffs(resolvedComparison.entityDiffs),
+        reprocessEntityDiffs(comparison.entityDiffs),
       );
-      report.withFromEntities(fromEntities).withToEntities(toEntities);
       this.reviewReport = report;
-      if (comparison.projectConfigurationUpdated) {
-        const [fromConfig, toConfig] = (yield Promise.all([
-          this.editorStore.sdlcServerClient.getReviewFromConfiguration(
-            this.projectId,
-            this.patchReleaseVersionId,
-            this.reviewId,
+      const fromToRequests = Promise.all([
+        ...report.diffs
+          .map((diff) => diff.oldPath)
+          .filter(isNonNullable)
+          .map((fromDiff) =>
+            this.editorStore.sdlcServerClient
+              .getReviewFromEntity(
+                this.projectId,
+                this.patchReleaseVersionId,
+                this.reviewId,
+                fromDiff,
+              )
+              .then((fromEntity: Entity) => report.addFromEntity(fromEntity)),
           ),
-          this.editorStore.sdlcServerClient.getReviewToConfiguration(
-            this.projectId,
-            this.patchReleaseVersionId,
-            this.reviewId,
+        ...report.diffs
+          .map((diff) => diff.newPath)
+          .filter(isNonNullable)
+          .map((toDiff) =>
+            this.editorStore.sdlcServerClient
+              .getReviewToEntity(
+                this.projectId,
+                this.patchReleaseVersionId,
+                this.reviewId,
+                toDiff,
+              )
+              .then((toEntity: Entity) => report.addToEntity(toEntity)),
           ),
-        ])) as [
-          PlainObject<ProjectConfiguration> | undefined,
-          PlainObject<ProjectConfiguration> | undefined,
+      ]);
+      if (comparison.projectConfigurationUpdated) {
+        [
+          this.editorStore.sdlcServerClient
+            .getReviewFromConfiguration(
+              this.projectId,
+              this.patchReleaseVersionId,
+              this.reviewId,
+            )
+            .then((config) => report.addFromConfig(config)),
+          this.editorStore.sdlcServerClient
+            .getReviewToConfiguration(
+              this.projectId,
+              this.patchReleaseVersionId,
+              this.reviewId,
+            )
+            .then((config) => report.addToConfig(config)),
         ];
-        if (fromConfig && toConfig) {
-          report.withProjectConfigChange([fromConfig, toConfig]);
-        }
       }
+      yield fromToRequests;
     } catch (error) {
       assertErrorThrown(error);
       this.editorStore.applicationStore.logService.error(
@@ -333,7 +358,7 @@ export class ProjectReviewerStore {
       );
       this.editorStore.applicationStore.notificationService.notifyError(error);
     } finally {
-      this.fetchComparisonState.complete();
+      this.buildReviewReportState.complete();
     }
   }
 

package/tsconfig.json

@@ -107,6 +107,7 @@
     "./src/stores/editor/editor-state/element-editor-state/database/QueryDatabaseState.ts",
     "./src/stores/editor/editor-state/element-editor-state/external-format/DSL_ExternalFormat_BindingEditorState.ts",
     "./src/stores/editor/editor-state/element-editor-state/external-format/DSL_ExternalFormat_SchemaSetEditorState.ts",
+    "./src/stores/editor/editor-state/element-editor-state/function-activator/HostedServiceFunctionActivatorEditorState.ts",
     "./src/stores/editor/editor-state/element-editor-state/function-activator/INTERNAL__UnknownFunctionActivatorEditorState.ts",
     "./src/stores/editor/editor-state/element-editor-state/function-activator/SnowflakeAppFunctionActivatorEditorState.ts",
     "./src/stores/editor/editor-state/element-editor-state/function-activator/testable/FunctionTestableState.ts",
@@ -167,6 +168,7 @@
     "./src/stores/extensions/STO_Relational_LegendStudioApplicationPlugin_Extension.ts",
     "./src/stores/graph-modifier/DSL_Data_GraphModifierHelper.ts",
     "./src/stores/graph-modifier/DSL_ExternalFormat_GraphModifierHelper.ts",
+    "./src/stores/graph-modifier/DSL_FunctionActivator_GraphModifierHelper.ts",
     "./src/stores/graph-modifier/DSL_Generation_GraphModifierHelper.ts",
     "./src/stores/graph-modifier/DSL_Mapping_GraphModifierHelper.ts",
     "./src/stores/graph-modifier/DSL_Service_GraphModifierHelper.ts",
@@ -234,6 +236,7 @@
     "./src/components/editor/editor-group/external-format-editor/DSL_ExternalFormat_SchemaSetElementEditor.tsx",
     "./src/components/editor/editor-group/external-format-editor/DSL_ExternalFormat_SchemaSetModelGenerationEditor.tsx",
     "./src/components/editor/editor-group/function-activator/FunctionEditor.tsx",
+    "./src/components/editor/editor-group/function-activator/HostedServiceFunctionActivatorEditor.tsx",
     "./src/components/editor/editor-group/function-activator/INTERNAL__UnknownFunctionActivatorEdtior.tsx",
     "./src/components/editor/editor-group/function-activator/SnowflakeAppFunctionActivatorEditor.tsx",
     "./src/components/editor/editor-group/function-activator/testable/FunctionTestableEditor.tsx",