Create a rum-core package

packages/core/src/boot/init.ts

@@ -2,8 +2,8 @@ import { areCookiesAuthorized, CookieOptions } from '../browser/cookie'
 import { buildConfiguration, UserConfiguration } from '../domain/configuration'
 import { setDebugMode, startInternalMonitoring } from '../domain/internalMonitoring'
 
-export function makeGlobal<T>(stub: T): T & { onReady(callback: () => void): void } {
-  const global = {
+export function makePublicApi<T>(stub: T): T & { onReady(callback: () => void): void } {
+  const publicApi = {
     ...stub,
 
     // This API method is intentionally not monitored, since the only thing executed is the
@@ -16,14 +16,14 @@ export function makeGlobal<T>(stub: T): T & { onReady(callback: () => void): voi
 
   // Add an "hidden" property to set debug mode. We define it that way to hide it
   // as much as possible but of course it's not a real protection.
-  Object.defineProperty(global, '_setDebug', {
+  Object.defineProperty(publicApi, '_setDebug', {
     get() {
       return setDebugMode
     },
     enumerable: false,
   })
 
-  return global
+  return publicApi
 }
 
 export function defineGlobal<Global, Name extends keyof Global>(global: Global, name: Name, api: Global[Name]) {

packages/core/src/index.ts

@@ -6,7 +6,7 @@ export {
   BuildMode,
   Datacenter,
   defineGlobal,
-  makeGlobal,
+  makePublicApi,
   commonInit,
   checkCookiesAuthorized,
   checkIsNotLocalFile,

packages/logs/src/boot/logs.entry.spec.ts

@@ -1,7 +1,7 @@
 import { Context, monitor, ONE_SECOND } from '@datadog/browser-core'
 
 import { HandlerType, LogsMessage, StatusType } from '../domain/logger'
-import { LogsGlobal, makeLogsGlobal, StartLogs } from './logs.entry'
+import { LogsPublicApi, makeLogsPublicApi, StartLogs } from './logs.entry'
 
 const DEFAULT_INIT_CONFIGURATION = { clientToken: 'xxx' }
 
@@ -28,17 +28,17 @@ describe('logs entry', () => {
     startLogsGetGlobalContext = undefined
   })
 
-  it('should set global with init', () => {
-    const LOGS = makeLogsGlobal(startLogs)
+  it('should define the public API with init', () => {
+    const LOGS = makeLogsPublicApi(startLogs)
     expect(!!LOGS).toEqual(true)
     expect(!!LOGS.init).toEqual(true)
   })
 
   describe('configuration validation', () => {
-    let LOGS: LogsGlobal
+    let LOGS: LogsPublicApi
 
     beforeEach(() => {
-      LOGS = makeLogsGlobal(startLogs)
+      LOGS = makeLogsPublicApi(startLogs)
     })
 
     it('init should log an error with no public api key', () => {
@@ -123,10 +123,10 @@ describe('logs entry', () => {
   })
 
   describe('pre-init API usages', () => {
-    let LOGS: LogsGlobal
+    let LOGS: LogsPublicApi
 
     beforeEach(() => {
-      LOGS = makeLogsGlobal(startLogs)
+      LOGS = makeLogsPublicApi(startLogs)
       jasmine.clock().install()
       jasmine.clock().mockDate()
     })
@@ -196,10 +196,10 @@ describe('logs entry', () => {
   })
 
   describe('post-init API usages', () => {
-    let LOGS: LogsGlobal
+    let LOGS: LogsPublicApi
 
     beforeEach(() => {
-      LOGS = makeLogsGlobal(startLogs)
+      LOGS = makeLogsPublicApi(startLogs)
       LOGS.init(DEFAULT_INIT_CONFIGURATION)
     })
 

packages/logs/src/boot/logs.entry.ts

@@ -7,7 +7,7 @@ import {
   defineGlobal,
   getGlobalObject,
   isPercentage,
-  makeGlobal,
+  makePublicApi,
   monitor,
   UserConfiguration,
 } from '@datadog/browser-core'
@@ -26,18 +26,18 @@ export interface LoggerConfiguration {
   context?: object
 }
 
-export type LogsGlobal = ReturnType<typeof makeLogsGlobal>
+export type LogsPublicApi = ReturnType<typeof makeLogsPublicApi>
 
-export const datadogLogs = makeLogsGlobal(startLogs)
+export const datadogLogs = makeLogsPublicApi(startLogs)
 
 interface BrowserWindow extends Window {
-  DD_LOGS?: LogsGlobal
+  DD_LOGS?: LogsPublicApi
 }
 defineGlobal(getGlobalObject<BrowserWindow>(), 'DD_LOGS', datadogLogs)
 
 export type StartLogs = typeof startLogs
 
-export function makeLogsGlobal(startLogsImpl: StartLogs) {
+export function makeLogsPublicApi(startLogsImpl: StartLogs) {
   let isAlreadyInitialized = false
 
   const globalContextManager = createContextManager()
@@ -49,7 +49,7 @@ export function makeLogsGlobal(startLogsImpl: StartLogs) {
   }
   const logger = new Logger(sendLog)
 
-  return makeGlobal({
+  return makePublicApi({
     logger,
 
     init: monitor((userConfiguration: LogsUserConfiguration) => {

packages/logs/src/index.ts

@@ -1,3 +1,3 @@
 export { Logger, LogsMessage, StatusType, HandlerType } from './domain/logger'
-export { LogsUserConfiguration, LoggerConfiguration, LogsGlobal, datadogLogs } from './boot/logs.entry'
+export { LogsUserConfiguration, LoggerConfiguration, LogsPublicApi as LogsGlobal, datadogLogs } from './boot/logs.entry'
 export { LogsEvent } from './logsEvent.types'

packages/rum-core/.npmignore

@@ -0,0 +1,4 @@
+*
+!/cjs/**/*
+!/esm/**/*
+!/src/**/*

packages/rum-core/.yarnrc

@@ -0,0 +1,2 @@
+save-exact true
+

packages/rum-core/README.md

@@ -0,0 +1,3 @@
+# `rum-core`
+
+Datadog browser RUM core utilities.

packages/rum-core/package.json

@@ -0,0 +1,26 @@
+{
+  "name": "@datadog/browser-rum-core",
+  "version": "2.1.2",
+  "license": "Apache-2.0",
+  "main": "cjs/index.js",
+  "module": "esm/index.js",
+  "types": "cjs/index.d.ts",
+  "scripts": {
+    "build": "run-p build:cjs build:esm",
+    "build:cjs": "rm -rf cjs && tsc -p tsconfig.cjs.json && yarn replace-build-env cjs",
+    "build:esm": "rm -rf esm && tsc -p tsconfig.esm.json && yarn replace-build-env esm",
+    "replace-build-env": "node ../../scripts/replace-build-env.js"
+  },
+  "dependencies": {
+    "@datadog/browser-core": "2.1.2",
+    "tslib": "^1.10.0"
+  },
+  "devDependencies": {
+    "ajv": "6.12.6"
+  },
+  "repository": {
+    "type": "git",
+    "url": "https://github.com/DataDog/browser-sdk.git",
+    "directory": "packages/rum-core"
+  }
+}

packages/rum/src/boot/rum.ts → packages/rum-core/src/boot/rum.ts

@@ -16,7 +16,7 @@ import { CommonContext } from '../rawRumEvent.types'
 import { startRumBatch } from '../transport/batch'
 
 import { buildEnv } from './buildEnv'
-import { RumUserConfiguration } from './rum.entry'
+import { RumUserConfiguration } from './rumPublicApi'
 
 export function startRum(userConfiguration: RumUserConfiguration, getCommonContext: () => CommonContext) {
   const lifeCycle = new LifeCycle()