feat(core): Deprecate spanStatusfromHttpCode in favour of getSpanStatusFromHttpCode

Author: Lms24

.vscode/settings.json

@@ -38,5 +38,6 @@
   "editor.codeActionsOnSave": {
     "source.organizeImports.biome": "explicit",
   },
-  "editor.defaultFormatter": "biomejs.biome"
+  "editor.defaultFormatter": "biomejs.biome",
+  "cSpell.words": ["Statusfrom"]
 }

packages/astro/src/index.server.ts

@@ -46,7 +46,9 @@ export {
   setTag,
   setTags,
   setUser,
+  // eslint-disable-next-line deprecation/deprecation
   spanStatusfromHttpCode,
+  getSpanStatusFromHttpCode,
   // eslint-disable-next-line deprecation/deprecation
   trace,
   withScope,

packages/browser/src/index.ts

@@ -66,7 +66,9 @@ export {
   extractTraceparentData,
   // eslint-disable-next-line deprecation/deprecation
   getActiveTransaction,
+  // eslint-disable-next-line deprecation/deprecation
   spanStatusfromHttpCode,
+  getSpanStatusFromHttpCode,
   // eslint-disable-next-line deprecation/deprecation
   trace,
   makeMultiplexedTransport,

packages/bun/src/index.ts

@@ -66,7 +66,9 @@ export {
   setTag,
   setTags,
   setUser,
+  // eslint-disable-next-line deprecation/deprecation
   spanStatusfromHttpCode,
+  getSpanStatusFromHttpCode,
   // eslint-disable-next-line deprecation/deprecation
   trace,
   withScope,

packages/core/src/tracing/index.ts

@@ -7,7 +7,12 @@ export { Transaction } from './transaction';
 export { extractTraceparentData, getActiveTransaction } from './utils';
 // eslint-disable-next-line deprecation/deprecation
 export { SpanStatus } from './spanstatus';
-export { setHttpStatus, spanStatusfromHttpCode } from './spanstatus';
+export {
+  setHttpStatus,
+  // eslint-disable-next-line deprecation/deprecation
+  spanStatusfromHttpCode,
+  getSpanStatusFromHttpCode,
+} from './spanstatus';
 export type { SpanStatusType } from './spanstatus';
 export {
   // eslint-disable-next-line deprecation/deprecation

packages/core/src/tracing/spanstatus.ts

@@ -83,7 +83,7 @@ export type SpanStatusType =
  * @param httpStatus The HTTP response status code.
  * @returns The span status or unknown_error.
  */
-export function spanStatusfromHttpCode(httpStatus: number): SpanStatusType {
+export function getSpanStatusFromHttpCode(httpStatus: number): SpanStatusType {
   if (httpStatus < 400 && httpStatus >= 100) {
     return 'ok';
   }
@@ -123,6 +123,17 @@ export function spanStatusfromHttpCode(httpStatus: number): SpanStatusType {
   return 'unknown_error';
 }
 
+/**
+ * Converts a HTTP status code into a {@link SpanStatusType}.
+ *
+ * @deprecated Use {@link spanStatusFromHttpCode} instead.
+ * This export will be removed in v8 as the signature contains a typo.
+ *
+ * @param httpStatus The HTTP response status code.
+ * @returns The span status or unknown_error.
+ */
+export const spanStatusfromHttpCode = getSpanStatusFromHttpCode;
+
 /**
  * Sets the Http status attributes on the current span based on the http code.
  * Additionally, the span's status is updated, depending on the http code.
@@ -140,7 +151,7 @@ export function setHttpStatus(span: Span, httpStatus: number): void {
   // eslint-disable-next-line deprecation/deprecation
   span.setData('http.response.status_code', httpStatus);
 
-  const spanStatus = spanStatusfromHttpCode(httpStatus);
+  const spanStatus = getSpanStatusFromHttpCode(httpStatus);
   if (spanStatus !== 'unknown_error') {
     span.setStatus(spanStatus);
   }

packages/deno/src/index.ts

@@ -65,7 +65,9 @@ export {
   setTag,
   setTags,
   setUser,
+  // eslint-disable-next-line deprecation/deprecation
   spanStatusfromHttpCode,
+  getSpanStatusFromHttpCode,
   // eslint-disable-next-line deprecation/deprecation
   trace,
   withScope,

packages/node-experimental/src/index.ts

@@ -66,7 +66,9 @@ export {
   Hub,
   runWithAsyncContext,
   SDK_VERSION,
+  // eslint-disable-next-line deprecation/deprecation
   spanStatusfromHttpCode,
+  getSpanStatusFromHttpCode,
   // eslint-disable-next-line deprecation/deprecation
   trace,
   captureCheckIn,

packages/node/src/index.ts

@@ -65,7 +65,9 @@ export {
   setTag,
   setTags,
   setUser,
+  // eslint-disable-next-line deprecation/deprecation
   spanStatusfromHttpCode,
+  getSpanStatusFromHttpCode,
   // eslint-disable-next-line deprecation/deprecation
   trace,
   withScope,

packages/remix/src/index.server.ts

@@ -44,7 +44,9 @@ export {
   setTag,
   setTags,
   setUser,
+  // eslint-disable-next-line deprecation/deprecation
   spanStatusfromHttpCode,
+  getSpanStatusFromHttpCode,
   // eslint-disable-next-line deprecation/deprecation
   trace,
   withScope,

packages/sveltekit/src/server/index.ts

@@ -43,7 +43,9 @@ export {
   setTag,
   setTags,
   setUser,
+  // eslint-disable-next-line deprecation/deprecation
   spanStatusfromHttpCode,
+  getSpanStatusFromHttpCode,
   // eslint-disable-next-line deprecation/deprecation
   trace,
   withScope,

packages/tracing-internal/src/exports/index.ts

@@ -8,6 +8,7 @@ export {
   Span,
   // eslint-disable-next-line deprecation/deprecation
   SpanStatus,
+  // eslint-disable-next-line deprecation/deprecation
   spanStatusfromHttpCode,
   startIdleTransaction,
   Transaction,

packages/tracing/src/index.ts

@@ -78,6 +78,7 @@ export const extractTraceparentData = extractTraceparentDataT;
  *
  * `spanStatusfromHttpCode` can be imported from `@sentry/node`, `@sentry/browser`, or your framework SDK
  */
+// eslint-disable-next-line deprecation/deprecation
 export const spanStatusfromHttpCode = spanStatusfromHttpCodeT;
 
 /**

packages/vercel-edge/src/index.ts

@@ -65,7 +65,9 @@ export {
   setTag,
   setTags,
   setUser,
+  // eslint-disable-next-line deprecation/deprecation
   spanStatusfromHttpCode,
+  getSpanStatusFromHttpCode,
   // eslint-disable-next-line deprecation/deprecation
   trace,
   withScope,