refactor(core): Move hasProp

Signed-off-by: Kaung Zin Hein <[email protected]>

Author: Zen-cronic

packages/browser/src/integrations/graphqlClient.ts

@@ -5,6 +5,7 @@ import {
   SEMANTIC_ATTRIBUTE_SENTRY_OP,
   SEMANTIC_ATTRIBUTE_URL_FULL,
   defineIntegration,
+  hasProp,
   spanToJSON,
 } from '@sentry/core';
 import type { Client, IntegrationFn } from '@sentry/core';
@@ -136,11 +137,6 @@ export function getRequestPayloadXhrOrFetch(hint: XhrHint | FetchHint): string |
   return body;
 }
 
-// Duplicate from deprecated @sentry-utils/src/instrument/fetch.ts
-function hasProp<T extends string>(obj: unknown, prop: T): obj is Record<string, string> {
-  return !!obj && typeof obj === 'object' && !!(obj as Record<string, string>)[prop];
-}
-
 /**
  * Parses the fetch arguments to extract the request payload.
  * Exported for tests only.

packages/core/src/index.ts

@@ -96,6 +96,7 @@ export {
   extractQueryParamsFromUrl,
   headersToDict,
 } from './utils/request';
+export { hasProp } from './utils/hasProp';
 export { DEFAULT_ENVIRONMENT } from './constants';
 export { addBreadcrumb } from './breadcrumbs';
 export { functionToStringIntegration } from './integrations/functiontostring';

packages/core/src/utils/hasProp.ts

@@ -0,0 +1,6 @@
+/**
+ * A more comprehensive key property check.
+ */
+export function hasProp<T extends string>(obj: unknown, prop: T): obj is Record<string, unknown> {
+  return !!obj && typeof obj === 'object' && !!(obj as Record<string, unknown>)[prop];
+}

packages/core/test/lib/utils/hasProp.test.ts

@@ -0,0 +1,27 @@
+import { hasProp } from '../../../src/utils/hasProp';
+
+describe('hasProp', () => {
+  it('should return true if the object has the provided property', () => {
+    const obj = { a: 1 };
+    const result = hasProp(obj, 'a');
+    expect(result).toBe(true);
+  });
+
+  it('should return false if the object does not have the provided property', () => {
+    const obj = { a: 1 };
+    const result = hasProp(obj, 'b');
+    expect(result).toBe(false);
+  });
+
+  it('should return false if the object is null', () => {
+    const obj = null;
+    const result = hasProp(obj, 'a');
+    expect(result).toBe(false);
+  });
+
+  it('should return false if the object is undefined', () => {
+    const obj = undefined;
+    const result = hasProp(obj, 'a');
+    expect(result).toBe(false);
+  });
+});