@finos/legend-application 0.2.1 → 1.0.3

package/src/stores/ApplicationStore.ts

@@ -14,9 +14,13 @@
  * limitations under the License.
  */
 
-import type { Log, SuperGenericFunction } from '@finos/legend-shared';
+import type { SuperGenericFunction } from '@finos/legend-shared';
 import {
+  EventNotifierService,
+  TracerService,
+  TelemetryService,
   assertTrue,
+  Log,
   LogEvent,
   assertErrorThrown,
   isString,
@@ -26,6 +30,7 @@ import { makeAutoObservable, action } from 'mobx';
 import { APPLICATION_LOG_EVENT } from './ApplicationLogEvent';
 import type { LegendApplicationConfig } from './ApplicationConfig';
 import type { WebApplicationNavigator } from './WebApplicationNavigator';
+import type { LegendApplicationPluginManager } from '../application/LegendApplicationPluginManager';
 
 export enum ActionAlertType {
   STANDARD = 'STANDARD',
@@ -97,12 +102,20 @@ export class Notification {
 export class ApplicationStore<T extends LegendApplicationConfig> {
   navigator: WebApplicationNavigator;
   notification?: Notification | undefined;
-  log: Log;
   blockingAlertInfo?: BlockingAlertInfo | undefined;
   actionAlertInfo?: ActionAlertInfo | undefined;
   config: T;
 
-  constructor(config: T, navigator: WebApplicationNavigator, log: Log) {
+  log: Log = new Log();
+  telemetryService = new TelemetryService();
+  tracerService = new TracerService();
+  eventNotifierService = new EventNotifierService();
+
+  constructor(
+    config: T,
+    navigator: WebApplicationNavigator,
+    pluginManager: LegendApplicationPluginManager,
+  ) {
     makeAutoObservable(this, {
       navigator: false,
       setBlockingAlert: action,
@@ -117,7 +130,16 @@ export class ApplicationStore<T extends LegendApplicationConfig> {
 
     this.config = config;
     this.navigator = navigator;
-    this.log = log;
+
+    // Register plugins
+    this.log.registerPlugins(pluginManager.getLoggerPlugins());
+    this.telemetryService.registerPlugins(
+      pluginManager.getTelemetryServicePlugins(),
+    );
+    this.tracerService.registerPlugins(pluginManager.getTracerServicePlugins());
+    this.eventNotifierService.registerPlugins(
+      pluginManager.getEventNotifierPlugins(),
+    );
   }
 
   setBlockingAlert(alertInfo: BlockingAlertInfo | undefined): void {

package/src/stores/ApplicationStoreTestUtils.ts

@@ -17,8 +17,8 @@
 import { ApplicationStore } from './ApplicationStore';
 import { createBrowserHistory } from 'history';
 import { WebApplicationNavigator } from './WebApplicationNavigator';
-import { Log } from '@finos/legend-shared';
 import type { LegendApplicationConfig } from './ApplicationConfig';
+import type { LegendApplicationPluginManager } from '../application/LegendApplicationPluginManager';
 
 export const TEST_DATA__applicationVersion = {
   buildTime: '2001-01-01T00:00:00-0000',
@@ -30,9 +30,10 @@ export const TEST__getTestApplicationStore = <
   T extends LegendApplicationConfig,
 >(
   config: T,
+  pluginManager: LegendApplicationPluginManager,
 ): ApplicationStore<T> =>
   new ApplicationStore(
     config,
     new WebApplicationNavigator(createBrowserHistory()),
-    new Log(),
+    pluginManager,
   );

package/src/stores/WebApplicationNavigator.ts

@@ -34,6 +34,13 @@ import { guaranteeNonNullable } from '@finos/legend-shared';
  *
  * As such, instead, we should design a more generic concept `Location` to pass around.
  * We would need to flesh out the details, but this is the rough idea.
+ *
+ * Another thought is that we should also generalize Router so it handles more than just
+ * URLs. If we make `router` and `navigator` work together, we can potentially generalize
+ * application navigation
+ *
+ * However, this depends on how and when we move to another platform, like `electron` for example
+ * See https://github.com/finos/legend-studio/issues/718
  */
 interface ApplicationNavigator<T> {
   reload(): void;

package/tsconfig.json

@@ -34,7 +34,6 @@
     "./src/const.ts",
     "./src/index.ts",
     "./src/components/ApplicationTestID.ts",
-    "./src/network/TelemetryEvent.ts",
     "./src/stores/ApplicationConfig.ts",
     "./src/stores/ApplicationLogEvent.ts",
     "./src/stores/ApplicationStore.ts",
@@ -44,6 +43,7 @@
     "./src/stores/PureLanguageSupport.ts",
     "./src/stores/WebApplicationNavigator.ts",
     "./src/application/LegendApplication.tsx",
+    "./src/application/LegendApplicationPluginManager.tsx",
     "./src/components/ActionAlert.tsx",
     "./src/components/AppHeader.tsx",
     "./src/components/ApplicationBackdrop.tsx",

package/lib/network/TelemetryEvent.d.ts

@@ -1,19 +0,0 @@
-/**
- * Copyright (c) 2020-present, Goldman Sachs
- *
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- *     http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-export declare enum CORE_TELEMETRY_EVENT {
-    APPLICATION_LOADED = "Application loaded"
-}
-//# sourceMappingURL=TelemetryEvent.d.ts.map

package/lib/network/TelemetryEvent.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"TelemetryEvent.d.ts","sourceRoot":"","sources":["../../src/network/TelemetryEvent.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;GAcG;AAEH,oBAAY,oBAAoB;IAC9B,kBAAkB,uBAAuB;CAC1C"}

package/lib/network/TelemetryEvent.js

@@ -1,20 +0,0 @@
-/**
- * Copyright (c) 2020-present, Goldman Sachs
- *
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- *     http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-export var CORE_TELEMETRY_EVENT;
-(function (CORE_TELEMETRY_EVENT) {
-    CORE_TELEMETRY_EVENT["APPLICATION_LOADED"] = "Application loaded";
-})(CORE_TELEMETRY_EVENT || (CORE_TELEMETRY_EVENT = {}));
-//# sourceMappingURL=TelemetryEvent.js.map

package/lib/network/TelemetryEvent.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"TelemetryEvent.js","sourceRoot":"","sources":["../../src/network/TelemetryEvent.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;GAcG;AAEH,MAAM,CAAN,IAAY,oBAEX;AAFD,WAAY,oBAAoB;IAC9B,iEAAyC,CAAA;AAC3C,CAAC,EAFW,oBAAoB,KAApB,oBAAoB,QAE/B"}

package/src/network/TelemetryEvent.ts

@@ -1,19 +0,0 @@
-/**
- * Copyright (c) 2020-present, Goldman Sachs
- *
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- *     http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-
-export enum CORE_TELEMETRY_EVENT {
-  APPLICATION_LOADED = 'Application loaded',
-}