refactor(core): Extract validation utilities

- validation.ts moved to @hawk.so/core

Author: Reversean

packages/core/src/index.ts

@@ -3,3 +3,4 @@ export type { RandomGenerator } from './utils/random';
 export { HawkUserManager } from './users/hawk-user-manager';
 export type { Logger, LogType } from './logger/logger';
 export { isLoggerSet, setLogger, resetLogger, log } from './logger/logger';
+export { isPlainObject, validateUser, validateContext, isValidEventPayload, isValidBreadcrumb } from './utils/validation';

…kages/javascript/src/utils/validation.ts packages/core/src/utils/validation.tspackages/javascript/src/utils/validation.ts renamed to packages/core/src/utils/validation.ts packages/javascript/src/utils/validation.ts renamed to packages/core/src/utils/validation.ts

@@ -1,14 +1,13 @@
-import { log } from '@hawk.so/core';
-import type { AffectedUser, Breadcrumb, EventContext, EventData, JavaScriptAddons } from '@hawk.so/types';
-import Sanitizer from '../modules/sanitizer';
+import { log } from '../logger/logger';
+import type { AffectedUser, Breadcrumb, EventAddons, EventContext, EventData } from '@hawk.so/types';
 
 /**
  * Validates user data - basic security checks
  *
  * @param user - user data to validate
  */
 export function validateUser(user: AffectedUser): boolean {
-  if (!user || !Sanitizer.isObject(user)) {
+  if (!user || !isPlainObject(user)) {
     log('validateUser: User must be an object', 'warn');
 
     return false;
@@ -30,7 +29,7 @@ export function validateUser(user: AffectedUser): boolean {
  * @param context - context data to validate
  */
 export function validateContext(context: EventContext | undefined): boolean {
-  if (context && !Sanitizer.isObject(context)) {
+  if (context && !isPlainObject(context)) {
     log('validateContext: Context must be an object', 'warn');
 
     return false;
@@ -40,22 +39,23 @@ export function validateContext(context: EventContext | undefined): boolean {
 }
 
 /**
- * Checks if value is a plain object (not array, Date, etc.)
+ * Checks if value is a plain object (not null, array, Date, Map, etc.)
  *
  * @param value - value to check
+ * @returns `true` if value is a plain object, otherwise `false`
  */
-function isPlainObject(value: unknown): value is Record<string, unknown> {
+export function isPlainObject(value: unknown): value is Record<string, unknown> {
   return Object.prototype.toString.call(value) === '[object Object]';
 }
 
 /**
  * Runtime check for required EventData fields.
  * Per @hawk.so/types EventData, `title` is the only non-optional field.
- * Additionally validates `backtrace` shape if present (must be an array).
+ * Additionally, validates `backtrace` shape if present (must be an array).
  *
  * @param payload - value to validate
  */
-export function isValidEventPayload(payload: unknown): payload is EventData<JavaScriptAddons> {
+export function isValidEventPayload(payload: unknown): payload is EventData<EventAddons> {
   if (!isPlainObject(payload)) {
     return false;
   }
@@ -64,11 +64,7 @@ export function isValidEventPayload(payload: unknown): payload is EventData<Java
     return false;
   }
 
-  if (payload.backtrace !== undefined && !Array.isArray(payload.backtrace)) {
-    return false;
-  }
-
-  return true;
+  return !(payload.backtrace !== undefined && !Array.isArray(payload.backtrace));
 }
 
 /**
@@ -86,9 +82,5 @@ export function isValidBreadcrumb(breadcrumb: unknown): breadcrumb is Breadcrumb
     return false;
   }
 
-  if (breadcrumb.timestamp !== undefined && typeof breadcrumb.timestamp !== 'number') {
-    return false;
-  }
-
-  return true;
+  return !(breadcrumb.timestamp !== undefined && typeof breadcrumb.timestamp !== 'number');
 }

…avascript/tests/utils/validation.test.ts …ages/core/tests/utils/validation.test.tspackages/javascript/tests/utils/validation.test.ts renamed to packages/core/tests/utils/validation.test.ts packages/javascript/tests/utils/validation.test.ts renamed to packages/core/tests/utils/validation.test.ts

@@ -1,8 +1,8 @@
-import { describe, it, expect, vi } from 'vitest';
-import { validateUser, validateContext, isValidEventPayload, isValidBreadcrumb } from '../../src/utils/validation';
+import { describe, expect, it, vi } from 'vitest';
+import { isValidBreadcrumb, isValidEventPayload, validateContext, validateUser } from '../../src';
 
 // Suppress log output produced by log() calls inside validation failures.
-vi.mock('@hawk.so/core', () => ({ log: vi.fn(), isLoggerSet: vi.fn(() => true), setLogger: vi.fn() }));
+vi.mock('../../src/logger/logger', () => ({ log: vi.fn(), isLoggerSet: vi.fn(() => true), setLogger: vi.fn() }));
 
 describe('validateUser', () => {
   it('should return false when user is null', () => {

packages/javascript/src/addons/breadcrumbs.ts

@@ -4,8 +4,7 @@
 import type { Breadcrumb, BreadcrumbLevel, BreadcrumbType, Json, JsonNode } from '@hawk.so/types';
 import Sanitizer from '../modules/sanitizer';
 import { buildElementSelector } from '../utils/selector';
-import { log } from '@hawk.so/core';
-import { isValidBreadcrumb } from '../utils/validation';
+import { isValidBreadcrumb, log } from '@hawk.so/core';
 
 /**
  * Default maximum number of breadcrumbs to store

packages/javascript/src/catcher.ts

@@ -18,7 +18,7 @@ import { isErrorProcessed, markErrorAsProcessed } from './utils/event';
 import { BrowserRandomGenerator } from './utils/random';
 import { ConsoleCatcher } from './addons/consoleCatcher';
 import { BreadcrumbManager } from './addons/breadcrumbs';
-import { isValidEventPayload, validateContext, validateUser } from './utils/validation';
+import { isValidEventPayload, validateContext, validateUser } from '@hawk.so/core';
 import { HawkUserManager, isLoggerSet, log, setLogger } from '@hawk.so/core';
 import { HawkLocalStorage } from './storages/hawk-local-storage';
 import { createBrowserLogger } from './logger/logger';