use new Primitive type rather than listing primitives

Author: lobsterkatie

MIGRATION.md

@@ -88,11 +88,11 @@ like this:
 
 ## New Scope functions
 
-We realized how annoying it is to set a whole object using `setExtra`, that's why there are now a few new methods on the
+We realized how annoying it is to set a whole object using `setExtra`, so there are now a few new methods on the
 `Scope`.
 
 ```typescript
-setTags(tags: { [key: string]: string | number | boolean | undefined }): this;
+setTags(tags: { [key: string]: string | number | boolean | bigint| symbol| null | undefined }): this;
 setExtras(extras: { [key: string]: any }): this;
 clearBreadcrumbs(): this;
 ```

packages/hub/src/hub.ts

@@ -11,6 +11,7 @@ import {
   Hub as HubInterface,
   Integration,
   IntegrationClass,
+  Primitive,
   SessionContext,
   Severity,
   Span,
@@ -261,7 +262,7 @@ export class Hub implements HubInterface {
   /**
    * @inheritDoc
    */
-  public setTags(tags: { [key: string]: string | number | boolean | undefined }): void {
+  public setTags(tags: { [key: string]: Primitive }): void {
     const scope = this.getScope();
     if (scope) scope.setTags(tags);
   }
@@ -277,7 +278,7 @@ export class Hub implements HubInterface {
   /**
    * @inheritDoc
    */
-  public setTag(key: string, value: string | number | boolean | undefined): void {
+  public setTag(key: string, value: Primitive): void {
     const scope = this.getScope();
     if (scope) scope.setTag(key, value);
   }

packages/hub/src/scope.ts

@@ -9,6 +9,7 @@ import {
   EventProcessor,
   Extra,
   Extras,
+  Primitive,
   Scope as ScopeInterface,
   ScopeContext,
   Severity,
@@ -41,7 +42,7 @@ export class Scope implements ScopeInterface {
   protected _user: User = {};
 
   /** Tags */
-  protected _tags: { [key: string]: string | number | boolean | undefined } = {};
+  protected _tags: { [key: string]: Primitive } = {};
 
   /** Extra */
   protected _extra: Extras = {};
@@ -124,7 +125,7 @@ export class Scope implements ScopeInterface {
   /**
    * @inheritDoc
    */
-  public setTags(tags: { [key: string]: string | number | boolean | undefined }): this {
+  public setTags(tags: { [key: string]: Primitive }): this {
     this._tags = {
       ...this._tags,
       ...tags,
@@ -136,7 +137,7 @@ export class Scope implements ScopeInterface {
   /**
    * @inheritDoc
    */
-  public setTag(key: string, value: string | number | boolean | undefined): this {
+  public setTag(key: string, value: Primitive): this {
     this._tags = { ...this._tags, [key]: value };
     this._notifyScopeListeners();
     return this;

packages/minimal/src/index.ts

@@ -6,6 +6,7 @@ import {
   Event,
   Extra,
   Extras,
+  Primitive,
   Severity,
   Transaction,
   TransactionContext,
@@ -127,7 +128,7 @@ export function setExtras(extras: Extras): void {
  * Set an object that will be merged sent as tags data with the event.
  * @param tags Tags context object to merge into current context.
  */
-export function setTags(tags: { [key: string]: string | number | boolean | undefined }): void {
+export function setTags(tags: { [key: string]: Primitive }): void {
   callOnHub<void>('setTags', tags);
 }
 
@@ -148,7 +149,7 @@ export function setExtra(key: string, extra: Extra): void {
  * @param key String key of tag
  * @param value Value of tag
  */
-export function setTag(key: string, value: string | number | boolean | undefined): void {
+export function setTag(key: string, value: Primitive): void {
   callOnHub<void>('setTag', key, value);
 }
 

packages/react/src/reactrouterv3.ts

@@ -1,4 +1,4 @@
-import { Transaction, TransactionContext } from '@sentry/types';
+import { Primitive, Transaction, TransactionContext } from '@sentry/types';
 import { getGlobalObject } from '@sentry/utils';
 
 import { Location, ReactRouterInstrumentation } from './types';
@@ -62,7 +62,7 @@ export function reactRouterV3Instrumentation(
           if (activeTransaction) {
             activeTransaction.finish();
           }
-          const tags: Record<string, string | number | boolean | undefined> = {
+          const tags: Record<string, Primitive> = {
             'routing.instrumentation': 'react-router-v3',
           };
           if (prevName) {

packages/tracing/src/span.ts

@@ -1,5 +1,5 @@
 /* eslint-disable max-lines */
-import { Span as SpanInterface, SpanContext, Transaction } from '@sentry/types';
+import { Primitive, Span as SpanInterface, SpanContext, Transaction } from '@sentry/types';
 import { dropUndefinedKeys, timestampWithMs, uuid4 } from '@sentry/utils';
 
 import { SpanStatus } from './spanstatus';
@@ -86,7 +86,7 @@ export class Span implements SpanInterface {
   /**
    * @inheritDoc
    */
-  public tags: { [key: string]: string | number | boolean | undefined } = {};
+  public tags: { [key: string]: Primitive } = {};
 
   /**
    * @inheritDoc
@@ -187,7 +187,7 @@ export class Span implements SpanInterface {
   /**
    * @inheritDoc
    */
-  public setTag(key: string, value: string | number | boolean | undefined): this {
+  public setTag(key: string, value: Primitive): this {
     this.tags = { ...this.tags, [key]: value };
     return this;
   }
@@ -257,7 +257,7 @@ export class Span implements SpanInterface {
     parent_span_id?: string;
     span_id: string;
     status?: string;
-    tags?: { [key: string]: string | number | boolean | undefined };
+    tags?: { [key: string]: Primitive };
     trace_id: string;
   } {
     return dropUndefinedKeys({
@@ -284,7 +284,7 @@ export class Span implements SpanInterface {
     span_id: string;
     start_timestamp: number;
     status?: string;
-    tags?: { [key: string]: string | number | boolean | undefined };
+    tags?: { [key: string]: Primitive };
     timestamp?: number;
     trace_id: string;
   } {

packages/types/src/event.ts

@@ -2,6 +2,7 @@ import { Breadcrumb } from './breadcrumb';
 import { Contexts } from './context';
 import { Exception } from './exception';
 import { Extras } from './extra';
+import { Primitive } from './misc';
 import { Request } from './request';
 import { CaptureContext } from './scope';
 import { SdkInfo } from './sdkinfo';
@@ -35,7 +36,7 @@ export interface Event {
   stacktrace?: Stacktrace;
   breadcrumbs?: Breadcrumb[];
   contexts?: Contexts;
-  tags?: { [key: string]: string | number | boolean | undefined };
+  tags?: { [key: string]: Primitive };
   extra?: Extras;
   user?: User;
   type?: EventType;

packages/types/src/hub.ts

@@ -3,6 +3,7 @@ import { Client } from './client';
 import { Event, EventHint } from './event';
 import { Extra, Extras } from './extra';
 import { Integration, IntegrationClass } from './integration';
+import { Primitive } from './misc';
 import { Scope } from './scope';
 import { Session, SessionContext } from './session';
 import { Severity } from './severity';
@@ -127,7 +128,7 @@ export interface Hub {
    *
    * @param tags Tags context object to merge into current context.
    */
-  setTags(tags: { [key: string]: string | number | boolean | undefined }): void;
+  setTags(tags: { [key: string]: Primitive }): void;
 
   /**
    * Set key:value that will be sent as tags data with the event.
@@ -137,7 +138,7 @@ export interface Hub {
    * @param key String key of tag
    * @param value Value of tag
    */
-  setTag(key: string, value: string | number | boolean | undefined): void;
+  setTag(key: string, value: Primitive): void;
 
   /**
    * Set key:value that will be sent as extra data with the event.

packages/types/src/index.ts

@@ -11,7 +11,7 @@ export { Hub } from './hub';
 export { Integration, IntegrationClass } from './integration';
 export { LogLevel } from './loglevel';
 export { Mechanism } from './mechanism';
-export { ExtractedNodeRequestData, WorkerLocation } from './misc';
+export { ExtractedNodeRequestData, NonPrimitive, Primitive, WorkerLocation } from './misc';
 export { Options } from './options';
 export { Package } from './package';
 export { Request, SentryRequest, SentryRequestType } from './request';

packages/types/src/misc.ts

@@ -59,3 +59,8 @@ export interface WorkerLocation {
   /** Synonym for `href` attribute */
   toString(): string;
 }
+
+export type Primitive = number | string | boolean | bigint | symbol | null | undefined;
+
+// at eslint's suggestion, not using 'object' because https://github.com/microsoft/TypeScript/issues/21732 is still open
+export type NonPrimitive = Record<string, unknown>;