Merge pull request #153 from codex-team/refactor/javascript-package-decomposition

Javascript package decomposition

Author: Reversean

packages/javascript/src/errors.ts packages/core/src/errors.tspackages/javascript/src/errors.ts renamed to packages/core/src/errors.ts

@@ -7,3 +7,8 @@ export { validateUser, validateContext, isValidEventPayload, isValidBreadcrumb }
 export { isPlainObject, isArray, isClassPrototype, isClassInstance, isString } from './utils/type-guards';
 export { Sanitizer } from './modules/sanitizer';
 export type { Transport } from './transports/transport';
+export type { SanitizerTypeHandler } from './modules/sanitizer';
+export { StackParser } from './modules/stack-parser';
+export { buildElementSelector } from './utils/selector';
+export { EventRejectedError } from './errors';
+export { isErrorProcessed, markErrorAsProcessed } from './utils/event';

packages/core/src/index.ts

@@ -1,4 +1,4 @@
-import { log } from '@hawk.so/core';
+import { log } from '../logger/logger';
 
 /**
  * Sends AJAX request and wait for some time.

…ges/javascript/src/modules/fetchTimer.ts packages/core/src/modules/fetch-timer.tspackages/javascript/src/modules/fetchTimer.ts renamed to packages/core/src/modules/fetch-timer.ts packages/javascript/src/modules/fetchTimer.ts renamed to packages/core/src/modules/fetch-timer.ts

@@ -1,12 +1,12 @@
 import type { StackFrame } from 'error-stack-parser';
 import ErrorStackParser from 'error-stack-parser';
 import type { BacktraceFrame, SourceCodeLine } from '@hawk.so/types';
-import fetchTimer from './fetchTimer';
+import fetchTimer from './fetch-timer';
 
 /**
  * This module prepares parsed backtrace
  */
-export default class StackParser {
+export class StackParser {
   /**
    * Prevents loading one file several times
    * name -> content
@@ -48,7 +48,7 @@ export default class StackParser {
     try {
       if (!frame.fileName) {
         return null;
-      };
+      }
 
       if (!this.isValidUrl(frame.fileName)) {
         return null;
@@ -118,9 +118,9 @@ export default class StackParser {
   /**
    * Downloads source file
    *
-   * @param {string} fileName - name of file to download
+   * @param fileName - name of file to download
    */
-  private async loadSourceFile(fileName): Promise<string | null> {
+  private async loadSourceFile(fileName: string): Promise<string | null> {
     if (this.sourceFilesCache[fileName] !== undefined) {
       return this.sourceFilesCache[fileName];
     }

…es/javascript/src/modules/stackParser.ts packages/core/src/modules/stack-parser.tspackages/javascript/src/modules/stackParser.ts renamed to packages/core/src/modules/stack-parser.ts packages/javascript/src/modules/stackParser.ts renamed to packages/core/src/modules/stack-parser.ts

@@ -1,12 +1,12 @@
-import { log } from '@hawk.so/core';
+import { log } from '../logger/logger';
 
 /**
  * Symbol to mark error as processed by Hawk
  */
 const errorSentShadowProperty = Symbol('__hawk_processed__');
 
 /**
- * Check if the error has alrady been sent to Hawk.
+ * Check if the error has already been sent to Hawk.
  *
  * Motivation:
  * Some integrations may catch errors on their own side and then normally re-throw them down.
@@ -20,7 +20,7 @@ export function isErrorProcessed(error: unknown): boolean {
     return false;
   }
 
-  return error[errorSentShadowProperty] === true;
+  return (error as Record<symbol, unknown>)[errorSentShadowProperty] === true;
 }
 
 /**
@@ -35,7 +35,7 @@ export function markErrorAsProcessed(error: unknown): void {
     }
 
     Object.defineProperty(error, errorSentShadowProperty, {
-      enumerable: false, // Prevent from beight collected by Hawk
+      enumerable: false,  // Prevent from being collected by Hawk
       value: true,
       writable: true,
       configurable: true,

packages/javascript/src/utils/event.ts packages/core/src/utils/event.tspackages/javascript/src/utils/event.ts renamed to packages/core/src/utils/event.ts

@@ -2,8 +2,7 @@
  * @file Breadcrumbs module - captures chronological trail of events before an error
  */
 import type { Breadcrumb, BreadcrumbLevel, BreadcrumbType, Json, JsonNode } from '@hawk.so/types';
-import { isValidBreadcrumb, log, Sanitizer } from '@hawk.so/core';
-import { buildElementSelector } from '../utils/selector';
+import { buildElementSelector, isValidBreadcrumb, log, Sanitizer } from '@hawk.so/core';
 
 /**
  * Default maximum number of breadcrumbs to store

packages/javascript/src/utils/selector.ts packages/core/src/utils/selector.tspackages/javascript/src/utils/selector.ts renamed to packages/core/src/utils/selector.ts

@@ -1,6 +1,5 @@
 import './modules/element-sanitizer';
 import Socket from './modules/socket';
-import StackParser from './modules/stackParser';
 import type { BreadcrumbsAPI, CatcherMessage, HawkInitialSettings, HawkJavaScriptEvent, Transport } from './types';
 import { VueIntegration } from './integrations/vue';
 import type {
@@ -12,24 +11,26 @@ import type {
   Json,
   VueIntegrationAddons
 } from '@hawk.so/types';
-import type { JavaScriptCatcherIntegrations } from './types/integrations';
-import { EventRejectedError } from './errors';
-import { isErrorProcessed, markErrorAsProcessed } from './utils/event';
-import { BrowserRandomGenerator } from './utils/random';
+import type { JavaScriptCatcherIntegrations } from '@/types';
 import { ConsoleCatcher } from './addons/consoleCatcher';
 import { BreadcrumbManager } from './addons/breadcrumbs';
 import {
+  EventRejectedError,
   HawkUserManager,
+  isErrorProcessed,
   isLoggerSet,
   isValidEventPayload,
   log,
+  markErrorAsProcessed,
   Sanitizer,
   setLogger,
+  StackParser,
   validateContext,
   validateUser
 } from '@hawk.so/core';
 import { HawkLocalStorage } from './storages/hawk-local-storage';
 import { createBrowserLogger } from './logger/logger';
+import { BrowserRandomGenerator } from './utils/random';
 
 /**
  * Allow to use global VERSION, that will be overwritten by Webpack
@@ -694,6 +695,6 @@ export default class Catcher {
    * @param integrationAddons - extra addons
    */
   private appendIntegrationAddons(errorFormatted: CatcherMessage<typeof Catcher.type>, integrationAddons: JavaScriptCatcherIntegrations): void {
-    Object.assign(errorFormatted.payload.addons!, integrationAddons);
+    Object.assign(errorFormatted.payload.addons, integrationAddons);
   }
 }

packages/javascript/src/addons/breadcrumbs.ts

@@ -1,6 +1,6 @@
-import type { EventContext, AffectedUser } from '@hawk.so/types';
+import type { AffectedUser, EventContext } from '@hawk.so/types';
 import type { HawkJavaScriptEvent } from './event';
-import type { Transport } from './transport';
+import type { Transport } from '@hawk.so/core';
 import type { BreadcrumbsOptions } from '../addons/breadcrumbs';
 
 /**