feat(core): Deprecate spanStatusfromHttpCode in favour of getSpanStatusFromHttpCode (#10361)

The old export contained a typo/camel-case inconsistency which I fixed
in the new export and I opted to prepend the `get` verb.

Author: Lms24

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

@@ -71,7 +71,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

@@ -49,7 +49,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,