Change core client to be functional and expect fetch and localStorage to be available. Fix lots of types. Other simplifications.

Author: ejsmith

.vscode/settings.json

@@ -1,6 +1,7 @@
 // Place your settings in this file to overwrite default and user settings.
 {
   "editor.formatOnSave": true,
+  "editor.defaultFormatter": "dbaeumer.vscode-eslint",
   "files.insertFinalNewline": true,
   "files.exclude": {
     "**/tsconfig.tsbuildinfo": true,

.vscode/tasks.json

@@ -22,7 +22,10 @@
     {
       "type": "npm",
       "script": "test",
-      "group": "test",
+      "group": {
+        "kind": "test",
+        "isDefault": true
+      },
       "label": "npm: test"
     },
     {

package-lock.json

@@ -36,8 +36,6 @@
     "eslint-plugin-import": "2.23.2",
     "eslint-plugin-jest": "24.3.6",
     "eslint-plugin-jsdoc": "34.8.2",
-    "jest": "26.6.3",
-    "jest-ts-webcompat-resolver": "1.0.0",
     "lage": "^0.29.3",
     "rimraf": "3.0.2",
     "typescript": "4.2.4"

package.json

@@ -43,7 +43,6 @@
     "@types/angular": "1.8.1"
   },
   "dependencies": {
-    "@exceptionless/browser": "2.0.0-dev",
-    "angular": "1.8.2"
+    "@exceptionless/browser": "2.0.0-dev"
   }
 }

packages/angularjs/package.json

@@ -1,24 +1,20 @@
-import {
-  Configuration,
-  ExceptionlessClient
-} from "@exceptionless/core";
+import { Configuration, ExceptionlessClient } from "@exceptionless/core";
 
 import { BrowserGlobalHandlerPlugin } from "./plugins/BrowserGlobalHandlerPlugin.js";
 import { BrowserLifeCyclePlugin } from "./plugins/BrowserLifeCyclePlugin.js";
 import { BrowserModuleInfoPlugin } from "./plugins/BrowserModuleInfoPlugin.js";
 import { BrowserRequestInfoPlugin } from "./plugins/BrowserRequestInfoPlugin.js";
 import { BrowserWrapFunctions } from "./plugins/BrowserWrapFunctions.js";
 import { BrowserErrorParser } from "./services/BrowserErrorParser.js";
-import { BrowserLocalStorage } from "./storage/BrowserLocalStorage.js";
-import { BrowserFetchSubmissionClient } from "./submission/BrowserFetchSubmissionClient.js";
 
 export class BrowserExceptionlessClient extends ExceptionlessClient {
-  public async startup(configurationOrApiKey?: (config: Configuration) => void | string): Promise<void> {
+  public async startup(
+    configurationOrApiKey?: (config: Configuration) => void | string,
+  ): Promise<void> {
     const config = this.config;
     if (configurationOrApiKey) {
-      config.services.storage = new BrowserLocalStorage();
+      config.useLocalStorage();
       config.services.errorParser = new BrowserErrorParser();
-      config.services.submissionClient = new BrowserFetchSubmissionClient(config);
 
       config.addPlugin(new BrowserGlobalHandlerPlugin());
       config.addPlugin(new BrowserLifeCyclePlugin());

packages/browser/src/BrowserExceptionlessClient.ts

@@ -4,8 +4,6 @@ export { BrowserModuleInfoPlugin } from "./plugins/BrowserModuleInfoPlugin.js";
 export { BrowserRequestInfoPlugin } from "./plugins/BrowserRequestInfoPlugin.js";
 export { BrowserWrapFunctions } from "./plugins/BrowserWrapFunctions.js";
 export { BrowserErrorParser } from "./services/BrowserErrorParser.js";
-export { BrowserLocalStorage } from "./storage/BrowserLocalStorage.js";
-export { BrowserFetchSubmissionClient } from "./submission/BrowserFetchSubmissionClient.js";
 export { BrowserExceptionlessClient } from "./BrowserExceptionlessClient.js";
 
 import { BrowserExceptionlessClient } from "./BrowserExceptionlessClient.js";

packages/browser/src/index.ts

@@ -18,7 +18,6 @@ export class BrowserGlobalHandlerPlugin implements IEventPlugin {
     }
 
     this._client = context.client;
-    Error.stackTraceLimit = 50;
 
     // TODO: Discus if we want to unwire this handler in suspend?
     window.addEventListener("error", async event => {