@finos/legend-application-studio 19.0.0 → 20.0.0

package/src/stores/sidebar-state/LocalChangesState.ts

@@ -14,7 +14,14 @@
  * limitations under the License.
  */
 
-import { action, makeAutoObservable, flowResult, flow } from 'mobx';
+import {
+  action,
+  makeObservable,
+  flowResult,
+  flow,
+  observable,
+  computed,
+} from 'mobx';
 import type { EditorStore } from '../EditorStore.js';
 import type { EditorSDLCState } from '../EditorSDLCState.js';
 import { CHANGE_DETECTION_EVENT } from '../ChangeDetectionEvent.js';
@@ -55,8 +62,8 @@ import { EntityChangeConflictEditorState } from '../editor-state/entity-diff-edi
 import { DATE_TIME_FORMAT } from '@finos/legend-graph';
 
 class PatchLoaderState {
-  editorStore: EditorStore;
-  sdlcState: EditorSDLCState;
+  readonly editorStore: EditorStore;
+  readonly sdlcState: EditorSDLCState;
 
   changes: EntityChange[] | undefined;
   currentChanges: EntityChange[] = [];
@@ -65,19 +72,37 @@ class PatchLoaderState {
   isValidPatch = false;
 
   constructor(editorStore: EditorStore, sdlcState: EditorSDLCState) {
-    makeAutoObservable(this, {
-      editorStore: false,
-      sdlcState: false,
+    makeObservable(this, {
+      changes: observable,
+      currentChanges: observable,
+      isLoadingChanges: observable,
+      showModal: observable,
+      isValidPatch: observable,
+      overiddingChanges: computed,
       openModal: action,
       closeModal: action,
+      setIsValidPatch: action,
+      setPatchChanges: action,
       deleteChange: action,
       loadPatchFile: flow,
+      applyChanges: flow,
     });
 
     this.editorStore = editorStore;
     this.sdlcState = sdlcState;
   }
 
+  get overiddingChanges(): EntityChange[] {
+    if (this.changes?.length) {
+      return this.changes.filter((change) =>
+        this.currentChanges.find(
+          (local) => local.entityPath === change.entityPath,
+        ),
+      );
+    }
+    return [];
+  }
+
   openModal(localChanges: EntityChange[]): void {
     this.currentChanges = localChanges;
     this.showModal = true;
@@ -103,17 +128,6 @@ class PatchLoaderState {
     }
   }
 
-  get overiddingChanges(): EntityChange[] {
-    if (this.changes?.length) {
-      return this.changes.filter((change) =>
-        this.currentChanges.find(
-          (local) => local.entityPath === change.entityPath,
-        ),
-      );
-    }
-    return [];
-  }
-
   *loadPatchFile(file: File): GeneratorFn<void> {
     try {
       this.setPatchChanges(undefined);
@@ -161,20 +175,21 @@ class PatchLoaderState {
 }
 
 export class LocalChangesState {
-  editorStore: EditorStore;
-  sdlcState: EditorSDLCState;
-  workspaceSyncState: WorkspaceSyncState;
-  pushChangesState = ActionState.create();
-  refreshLocalChangesDetectorState = ActionState.create();
-  patchLoaderState: PatchLoaderState;
-  refreshWorkspaceSyncStatusState = ActionState.create();
+  readonly editorStore: EditorStore;
+  readonly sdlcState: EditorSDLCState;
+  readonly workspaceSyncState: WorkspaceSyncState;
+  readonly pushChangesState = ActionState.create();
+  readonly refreshLocalChangesDetectorState = ActionState.create();
+  readonly refreshWorkspaceSyncStatusState = ActionState.create();
+  readonly patchLoaderState: PatchLoaderState;
 
   constructor(editorStore: EditorStore, sdlcState: EditorSDLCState) {
-    makeAutoObservable(this, {
-      editorStore: false,
-      sdlcState: false,
-      openLocalChange: action,
+    makeObservable(this, {
+      hasUnpushedChanges: computed,
+      openPotentialWorkspacePullConflict: action,
       refreshWorkspaceSyncStatus: flow,
+      refreshLocalChanges: flow,
+      pushLocalChanges: flow,
     });
 
     this.editorStore = editorStore;
@@ -190,15 +205,57 @@ export class LocalChangesState {
     );
   }
 
+  downloadLocalChanges(): void {
+    const fileName = `entityChanges_(${this.sdlcState.currentProject?.name}_${
+      this.sdlcState.activeWorkspace.workspaceId
+    })_${formatDate(new Date(Date.now()), DATE_TIME_FORMAT)}.json`;
+    const content = JSON.stringify(
+      {
+        message: '', // TODO?
+        entityChanges: this.editorStore.graphState.computeLocalEntityChanges(),
+        revisionId: this.sdlcState.activeRevision.id,
+      },
+      undefined,
+      TAB_SIZE,
+    );
+    downloadFileUsingDataURI(fileName, content, ContentType.APPLICATION_JSON);
+  }
+
+  alertUnsavedChanges(onProceed: () => void): void {
+    if (this.hasUnpushedChanges) {
+      this.editorStore.applicationStore.setActionAlertInfo({
+        message:
+          'Unsaved changes to your query will be lost if you continue. Do you still want to proceed?',
+        type: ActionAlertType.CAUTION,
+        actions: [
+          {
+            label: 'Proceed',
+            type: ActionAlertActionType.PROCEED_WITH_CAUTION,
+            handler: (): void => onProceed(),
+          },
+          {
+            label: 'Abort',
+            type: ActionAlertActionType.PROCEED,
+            default: true,
+          },
+        ],
+      });
+    } else {
+      onProceed();
+    }
+  }
+
   openLocalChange(diff: EntityDiff): void {
     const fromEntityGetter = (
       entityPath: string | undefined,
-    ): Entity | undefined =>
-      entityPath
-        ? this.editorStore.changeDetectionState.workspaceLocalLatestRevisionState.entities.find(
-            (e) => e.path === entityPath,
-          )
-        : undefined;
+    ): Entity | undefined => {
+      if (entityPath) {
+        return this.editorStore.changeDetectionState.workspaceLocalLatestRevisionState.entities.find(
+          (e) => e.path === entityPath,
+        );
+      }
+      return undefined;
+    };
     const toEntityGetter = (
       entityPath: string | undefined,
     ): Entity | undefined => {
@@ -250,31 +307,23 @@ export class LocalChangesState {
     const fromEntityGetter = (
       entityPath: string | undefined,
     ): Entity | undefined => {
-      if (!entityPath) {
-        return undefined;
-      }
-      const element =
-        this.editorStore.graphManagerState.graph.getNullableElement(entityPath);
-      if (!element) {
-        return undefined;
-      }
-      const entity =
-        this.editorStore.graphManagerState.graphManager.elementToEntity(
-          element,
-          {
-            pruneSourceInformation: true,
-          },
+      if (entityPath) {
+        return this.editorStore.changeDetectionState.workspaceLocalLatestRevisionState.entities.find(
+          (e) => e.path === entityPath,
         );
-      return entity;
+      }
+      return undefined;
     };
     const toEntityGetter = (
       entityPath: string | undefined,
-    ): Entity | undefined =>
-      entityPath
-        ? this.editorStore.changeDetectionState.workspaceRemoteLatestRevisionState.entities.find(
-            (e) => e.path === entityPath,
-          )
-        : undefined;
+    ): Entity | undefined => {
+      if (entityPath) {
+        return this.editorStore.changeDetectionState.workspaceRemoteLatestRevisionState.entities.find(
+          (e) => e.path === entityPath,
+        );
+      }
+      return undefined;
+    };
     const fromEntity = EntityDiff.shouldOldEntityExist(diff)
       ? guaranteeNonNullable(
           fromEntityGetter(diff.getValidatedOldPath()),
@@ -302,6 +351,53 @@ export class LocalChangesState {
     );
   }
 
+  openPotentialWorkspacePullConflict(conflict: EntityChangeConflict): void {
+    const baseEntityGetter = (
+      entityPath: string | undefined,
+    ): Entity | undefined =>
+      entityPath
+        ? this.editorStore.changeDetectionState.workspaceLocalLatestRevisionState.entities.find(
+            (e) => e.path === entityPath,
+          )
+        : undefined;
+    const currentChangeEntityGetter = (
+      entityPath: string | undefined,
+    ): Entity | undefined =>
+      entityPath
+        ? this.editorStore.graphManagerState.graph.allOwnElements
+            .map((element) =>
+              this.editorStore.graphManagerState.graphManager.elementToEntity(
+                element,
+              ),
+            )
+            .find((e) => e.path === entityPath)
+        : undefined;
+    const incomingChangeEntityGetter = (
+      entityPath: string | undefined,
+    ): Entity | undefined =>
+      entityPath
+        ? this.editorStore.changeDetectionState.workspaceRemoteLatestRevisionState.entities.find(
+            (e) => e.path === entityPath,
+          )
+        : undefined;
+    const conflictEditorState = new EntityChangeConflictEditorState(
+      this.editorStore,
+      this.editorStore.conflictResolutionState,
+      conflict.entityPath,
+      SPECIAL_REVISION_ALIAS.WORKSPACE_BASE,
+      SPECIAL_REVISION_ALIAS.LOCAL,
+      SPECIAL_REVISION_ALIAS.WORKSPACE_HEAD,
+      baseEntityGetter(conflict.entityPath),
+      currentChangeEntityGetter(conflict.entityPath),
+      incomingChangeEntityGetter(conflict.entityPath),
+      baseEntityGetter,
+      currentChangeEntityGetter,
+      incomingChangeEntityGetter,
+    );
+    conflictEditorState.setReadOnly(true);
+    this.editorStore.openEntityChangeConflict(conflictEditorState);
+  }
+
   *refreshLocalChanges(): GeneratorFn<void> {
     const startTime = Date.now();
     this.refreshLocalChangesDetectorState.inProgress();
@@ -392,69 +488,6 @@ export class LocalChangesState {
     }
   }
 
-  openPotentialWorkspacePullConflict(conflict: EntityChangeConflict): void {
-    const baseEntityGetter = (
-      entityPath: string | undefined,
-    ): Entity | undefined =>
-      entityPath
-        ? this.editorStore.changeDetectionState.workspaceLocalLatestRevisionState.entities.find(
-            (e) => e.path === entityPath,
-          )
-        : undefined;
-    const currentChangeEntityGetter = (
-      entityPath: string | undefined,
-    ): Entity | undefined =>
-      entityPath
-        ? this.editorStore.graphManagerState.graph.allOwnElements
-            .map((element) =>
-              this.editorStore.graphManagerState.graphManager.elementToEntity(
-                element,
-              ),
-            )
-            .find((e) => e.path === entityPath)
-        : undefined;
-    const incomingChangeEntityGetter = (
-      entityPath: string | undefined,
-    ): Entity | undefined =>
-      entityPath
-        ? this.editorStore.changeDetectionState.workspaceRemoteLatestRevisionState.entities.find(
-            (e) => e.path === entityPath,
-          )
-        : undefined;
-    const conflictEditorState = new EntityChangeConflictEditorState(
-      this.editorStore,
-      this.editorStore.conflictResolutionState,
-      conflict.entityPath,
-      SPECIAL_REVISION_ALIAS.WORKSPACE_BASE,
-      SPECIAL_REVISION_ALIAS.LOCAL,
-      SPECIAL_REVISION_ALIAS.WORKSPACE_HEAD,
-      baseEntityGetter(conflict.entityPath),
-      currentChangeEntityGetter(conflict.entityPath),
-      incomingChangeEntityGetter(conflict.entityPath),
-      baseEntityGetter,
-      currentChangeEntityGetter,
-      incomingChangeEntityGetter,
-    );
-    conflictEditorState.setReadOnly(true);
-    this.editorStore.openEntityChangeConflict(conflictEditorState);
-  }
-
-  downloadLocalChanges = (): void => {
-    const fileName = `entityChanges_(${this.sdlcState.currentProject?.name}_${
-      this.sdlcState.activeWorkspace.workspaceId
-    })_${formatDate(new Date(Date.now()), DATE_TIME_FORMAT)}.json`;
-    const content = JSON.stringify(
-      {
-        message: '', // TODO?
-        entityChanges: this.editorStore.graphState.computeLocalEntityChanges(),
-        revisionId: this.sdlcState.activeRevision.id,
-      },
-      undefined,
-      TAB_SIZE,
-    );
-    downloadFileUsingDataURI(fileName, content, ContentType.APPLICATION_JSON);
-  };
-
   *pushLocalChanges(pushMessage?: string): GeneratorFn<void> {
     if (
       this.pushChangesState.isInProgress ||
@@ -689,28 +722,4 @@ export class LocalChangesState {
       this.pushChangesState.complete();
     }
   }
-
-  alertUnsavedChanges(onProceed: () => void): void {
-    if (this.hasUnpushedChanges) {
-      this.editorStore.applicationStore.setActionAlertInfo({
-        message:
-          'Unsaved changes to your query will be lost if you continue. Do you still want to proceed?',
-        type: ActionAlertType.CAUTION,
-        actions: [
-          {
-            label: 'Proceed',
-            type: ActionAlertActionType.PROCEED_WITH_CAUTION,
-            handler: (): void => onProceed(),
-          },
-          {
-            label: 'Abort',
-            type: ActionAlertActionType.PROCEED,
-            default: true,
-          },
-        ],
-      });
-    } else {
-      onProceed();
-    }
-  }
 }

package/src/stores/sidebar-state/ProjectOverviewState.ts

@@ -16,7 +16,7 @@
 
 import type { EditorStore } from '../EditorStore.js';
 import type { EditorSDLCState } from '../EditorSDLCState.js';
-import { action, flowResult, makeAutoObservable } from 'mobx';
+import { action, flow, flowResult, makeObservable, observable } from 'mobx';
 import {
   type GeneratorFn,
   type PlainObject,
@@ -63,10 +63,25 @@ export class ProjectOverviewState {
   isFetchingCurrentProjectRevision = false;
 
   constructor(editorStore: EditorStore, sdlcState: EditorSDLCState) {
-    makeAutoObservable(this, {
-      editorStore: false,
-      sdlcState: false,
+    makeObservable(this, {
+      activityMode: observable,
+      releaseVersion: observable,
+      committedReviewsBetweenMostRecentVersionAndProjectLatest: observable,
+      latestProjectVersion: observable,
+      currentProjectRevision: observable,
+      projectWorkspaces: observable,
+      isCreatingVersion: observable,
+      isFetchingProjectWorkspaces: observable,
+      isDeletingWorkspace: observable,
+      isUpdatingProject: observable,
+      isFetchingLatestVersion: observable,
+      isFetchingCurrentProjectRevision: observable,
       setActivityMode: action,
+      fetchProjectWorkspaces: flow,
+      deleteWorkspace: flow,
+      updateProject: flow,
+      fetchLatestProjectVersion: flow,
+      createVersion: flow,
     });
 
     this.editorStore = editorStore;

package/src/stores/sidebar-state/WorkflowManagerState.ts

@@ -15,14 +15,7 @@
  */
 
 import type { TreeNodeData, TreeData } from '@finos/legend-art';
-import {
-  makeAutoObservable,
-  observable,
-  action,
-  flowResult,
-  makeObservable,
-  flow,
-} from 'mobx';
+import { observable, action, flowResult, makeObservable, flow } from 'mobx';
 import { LEGEND_STUDIO_APP_EVENT } from '../LegendStudioAppEvent.js';
 import type { EditorStore } from '../EditorStore.js';
 import type { EditorSDLCState } from '../EditorSDLCState.js';
@@ -102,11 +95,12 @@ const updateWorkflowJobData = (
 };
 
 export class WorkflowLogState {
-  editorStore: EditorStore;
-  workflowManagerState: WorkflowManagerState;
+  readonly editorStore: EditorStore;
+  readonly workflowManagerState: WorkflowManagerState;
+  readonly fetchJobLogState = ActionState.create();
+
   job: WorkflowJob | undefined;
   logs: string;
-  fetchJobLogState = ActionState.create();
 
   constructor(
     editorStore: EditorStore,
@@ -114,12 +108,14 @@ export class WorkflowLogState {
     job: WorkflowJob | undefined,
     logs: string | undefined,
   ) {
-    makeAutoObservable(this, {
-      editorStore: false,
+    makeObservable(this, {
       job: observable,
       logs: observable,
       setLogs: action,
       setJob: action,
+      closeModal: action,
+      refreshJobLogs: flow,
+      viewJobLogs: flow,
     });
 
     this.editorStore = editorStore;
@@ -187,8 +183,9 @@ export class WorkflowLogState {
 
 export class WorkflowState {
   readonly uuid = uuid();
-  editorStore: EditorStore;
-  workflowManagerState: WorkflowManagerState;
+  readonly editorStore: EditorStore;
+  readonly workflowManagerState: WorkflowManagerState;
+
   treeData: TreeData<WorkflowExplorerTreeNodeData>;
   isExecutingWorkflowRequest = false;
 
@@ -198,10 +195,15 @@ export class WorkflowState {
     workflow: Workflow,
     jobs: WorkflowJob[] | undefined,
   ) {
-    makeAutoObservable(this, {
-      editorStore: false,
+    makeObservable(this, {
       treeData: observable.ref,
+      isExecutingWorkflowRequest: observable,
       setWorkflowTreeData: action,
+      fetchAllWorkspaceWorkJobs: flow,
+      onTreeNodeSelect: flow,
+      cancelJob: flow,
+      refreshWorkflow: flow,
+      retryJob: flow,
     });
 
     this.editorStore = editorStore;
@@ -326,19 +328,15 @@ export class WorkflowState {
 }
 
 export abstract class WorkflowManagerState {
-  editorStore: EditorStore;
-  sdlcState: EditorSDLCState;
-  fetchWorkflowsState = ActionState.create();
+  readonly editorStore: EditorStore;
+  readonly sdlcState: EditorSDLCState;
+  readonly fetchWorkflowsState = ActionState.create();
+
   logState: WorkflowLogState;
   workflowStates: WorkflowState[] = [];
 
   constructor(editorStore: EditorStore, sdlcState: EditorSDLCState) {
-    this.editorStore = editorStore;
-    this.sdlcState = sdlcState;
     makeObservable(this, {
-      editorStore: false,
-      sdlcState: false,
-      fetchWorkflowsState: observable,
       logState: observable,
       workflowStates: observable,
       fetchAllWorkflows: flow,
@@ -350,6 +348,10 @@ export abstract class WorkflowManagerState {
       cancelJob: flow,
       getJobLogs: flow,
     });
+
+    this.editorStore = editorStore;
+    this.sdlcState = sdlcState;
+
     this.logState = new WorkflowLogState(
       this.editorStore,
       this,
@@ -357,18 +359,13 @@ export abstract class WorkflowManagerState {
       undefined,
     );
   }
-  abstract getWorkflows(): GeneratorFn<Workflow[]>;
 
+  abstract getWorkflows(): GeneratorFn<Workflow[]>;
   abstract getWorkflow(workflowId: string): GeneratorFn<Workflow>;
-
   abstract getJobs(workflowId: string): GeneratorFn<WorkflowJob[]>;
-
   abstract getJob(job: WorkflowJob): GeneratorFn<WorkflowJob[]>;
-
   abstract retryJob(workflowJob: WorkflowJob): GeneratorFn<void>;
-
   abstract cancelJob(workflowJob: WorkflowJob): GeneratorFn<void>;
-
   abstract getJobLogs(workflowJob: WorkflowJob): GeneratorFn<string>;
 
   *fetchAllWorkflows(): GeneratorFn<void> {

package/src/stores/sidebar-state/WorkspaceReviewState.ts

@@ -14,7 +14,7 @@
  * limitations under the License.
  */
 
-import { action, makeAutoObservable, flowResult } from 'mobx';
+import { action, makeObservable, flowResult, observable, flow } from 'mobx';
 import type { EditorStore } from '../EditorStore.js';
 import type { EditorSDLCState } from '../EditorSDLCState.js';
 import { CHANGE_DETECTION_EVENT } from '../ChangeDetectionEvent.js';
@@ -41,8 +41,9 @@ import {
 import { ActionAlertActionType } from '@finos/legend-application';
 
 export class WorkspaceReviewState {
-  editorStore: EditorStore;
-  sdlcState: EditorSDLCState;
+  readonly editorStore: EditorStore;
+  readonly sdlcState: EditorSDLCState;
+
   reviewTitle = '';
   committedReviewsBetweenWorkspaceBaseAndProjectLatest: Review[] = [];
   workspaceReview?: Review | undefined;
@@ -57,20 +58,35 @@ export class WorkspaceReviewState {
   isRecreatingWorkspaceAfterCommittingReview = false;
 
   constructor(editorStore: EditorStore, sdlcState: EditorSDLCState) {
-    makeAutoObservable(this, {
-      editorStore: false,
-      sdlcState: false,
+    makeObservable(this, {
+      reviewTitle: observable,
+      committedReviewsBetweenWorkspaceBaseAndProjectLatest: observable,
+      workspaceReview: observable,
+      isUpdatingWorkspace: observable,
+      isRefreshingWorkspaceUpdater: observable,
+      isFetchingCurrentWorkspaceReview: observable,
+      isRefreshingWorkspaceChangesDetector: observable,
+      isClosingWorkspaceReview: observable,
+      isCreatingWorkspaceReview: observable,
+      isCommittingWorkspaceReview: observable,
+      isRecreatingWorkspaceAfterCommittingReview: observable,
       setReviewTitle: action,
       openReviewChange: action,
+      refreshWorkspaceChanges: flow,
+      fetchCurrentWorkspaceReview: flow,
+      recreateWorkspaceAfterCommittingReview: flow,
+      closeWorkspaceReview: flow,
+      createWorkspaceReview: flow,
+      commitWorkspaceReview: flow,
     });
 
     this.editorStore = editorStore;
     this.sdlcState = sdlcState;
   }
 
-  setReviewTitle = (val: string): void => {
+  setReviewTitle(val: string): void {
     this.reviewTitle = val;
-  };
+  }
 
   openReviewChange(diff: EntityDiff): void {
     const fromEntityGetter = (

package/src/stores/sidebar-state/WorkspaceSyncState.ts

@@ -14,14 +14,7 @@
  * limitations under the License.
  */
 
-import {
-  makeAutoObservable,
-  flowResult,
-  action,
-  flow,
-  makeObservable,
-  observable,
-} from 'mobx';
+import { flowResult, action, flow, makeObservable, observable } from 'mobx';
 import type { EditorStore } from '../EditorStore.js';
 import type { EditorSDLCState } from '../EditorSDLCState.js';
 import type { Entity } from '@finos/legend-storage';
@@ -373,23 +366,25 @@ class WorkspaceSyncConflictResolutionState extends AbstractConflictResolutionSta
 }
 
 export class WorkspaceSyncState {
-  editorStore: EditorStore;
-  sdlcState: EditorSDLCState;
+  readonly editorStore: EditorStore;
+  readonly sdlcState: EditorSDLCState;
 
   pullChangesState = ActionState.create();
   incomingRevisions: Revision[] = [];
   workspaceSyncConflictResolutionState: WorkspaceSyncConflictResolutionState;
 
   constructor(editorStore: EditorStore, sdlcState: EditorSDLCState) {
-    makeAutoObservable(this, {
-      editorStore: false,
-      sdlcState: false,
-      fetchIncomingRevisions: flow,
+    makeObservable(this, {
+      pullChangesState: observable,
+      incomingRevisions: observable,
+      workspaceSyncConflictResolutionState: observable,
+      resetConflictState: action,
       setIncomingRevisions: action,
+      fetchIncomingRevisions: flow,
       pullChanges: flow,
-      resetConflictState: action,
-      forcePull: flow,
       loadChanges: flow,
+      forcePull: flow,
+      applyResolutionChanges: flow,
     });
 
     this.editorStore = editorStore;

package/src/stores/sidebar-state/WorkspaceUpdaterState.ts

@@ -16,7 +16,7 @@
 
 import type { EditorStore } from '../EditorStore.js';
 import type { EditorSDLCState } from '../EditorSDLCState.js';
-import { action, makeAutoObservable, flowResult } from 'mobx';
+import { action, makeObservable, flowResult, observable, flow } from 'mobx';
 import { CHANGE_DETECTION_EVENT } from '../ChangeDetectionEvent.js';
 import {
   type GeneratorFn,
@@ -45,19 +45,23 @@ import {
 import { LEGEND_STUDIO_APP_EVENT } from '../LegendStudioAppEvent.js';
 
 export class WorkspaceUpdaterState {
-  editorStore: EditorStore;
-  sdlcState: EditorSDLCState;
-  committedReviewsBetweenWorkspaceBaseAndProjectLatest: Review[] = [];
+  readonly editorStore: EditorStore;
+  readonly sdlcState: EditorSDLCState;
 
+  committedReviewsBetweenWorkspaceBaseAndProjectLatest: Review[] = [];
   isUpdatingWorkspace = false;
   isRefreshingWorkspaceUpdater = false;
 
   constructor(editorStore: EditorStore, sdlcState: EditorSDLCState) {
-    makeAutoObservable(this, {
-      editorStore: false,
-      sdlcState: false,
+    makeObservable(this, {
+      committedReviewsBetweenWorkspaceBaseAndProjectLatest: observable,
+      isUpdatingWorkspace: observable,
+      isRefreshingWorkspaceUpdater: observable,
       openProjectLatestChange: action,
       openPotentialWorkspaceUpdateConflict: action,
+      refreshWorkspaceUpdater: flow,
+      updateWorkspace: flow,
+      fetchLatestCommittedReviews: flow,
     });
 
     this.editorStore = editorStore;