@spotlightjs/spotlight 4.4.0 → 4.5.0

package/dist/sidecar.cjs

@@ -1 +1,7 @@
-"use strict";Object.defineProperty(exports,Symbol.toStringTag,{value:"Module"});const e=require("@spotlightjs/sidecar");Object.defineProperty(exports,"setupSidecar",{enumerable:!0,get:()=>e.setupSidecar});
+"use strict";
+Object.defineProperty(exports, Symbol.toStringTag, { value: "Module" });
+const sidecar = require("@spotlightjs/sidecar");
+Object.defineProperty(exports, "setupSidecar", {
+  enumerable: true,
+  get: () => sidecar.setupSidecar
+});

package/dist/sidecar.js

@@ -1,4 +1,4 @@
-import { setupSidecar as o } from "@spotlightjs/sidecar";
+import { setupSidecar } from "@spotlightjs/sidecar";
 export {
-  o as setupSidecar
+  setupSidecar
 };

package/package.json

@@ -1,24 +1,23 @@
 {
   "name": "@spotlightjs/spotlight",
   "description": "Spotlight - Sentry for development. Containing the overlay and the sidecar.",
-  "version": "4.4.0",
+  "version": "4.5.0",
   "license": "Apache-2.0",
   "type": "module",
   "files": [
-    "dist",
-    "bin"
+    "dist"
   ],
   "bin": {
-    "spotlight": "./bin/run.js"
+    "spotlight": "./dist/run.js"
   },
-  "main": "./dist/overlay.cjs",
-  "module": "./dist/overlay.js",
-  "types": "./dist/overlay.d.ts",
+  "main": "./dist/sidecar.cjs",
+  "module": "./dist/sidecar.js",
+  "types": "./dist/sidecar.d.ts",
   "exports": {
     ".": {
-      "types": "./dist/overlay.d.ts",
-      "import": "./dist/overlay.js",
-      "require": "./dist/overlay.cjs"
+      "types": "./dist/sidecar.d.ts",
+      "import": "./dist/sidecar.js",
+      "require": "./dist/sidecar.cjs"
     },
     "./sidecar": {
       "types": "./dist/sidecar.d.ts",
@@ -29,8 +28,8 @@
   "dependencies": {
     "@sentry/node": "^10.22.0",
     "import-meta-resolve": "^4.1.0",
-    "@spotlightjs/sidecar": "2.4.0",
-    "@spotlightjs/overlay": "4.4.0"
+    "@spotlightjs/overlay": "4.5.0",
+    "@spotlightjs/sidecar": "2.4.1"
   },
   "devDependencies": {
     "@types/node": "^22.15.21",
@@ -46,12 +45,9 @@
     "node": ">=20"
   },
   "scripts": {
-    "start": "./bin/run.js",
-    "dev": "vite build --watch",
+    "start": "node --disable-warning=ExperimentalWarning --experimental-transform-types --watch ./src/run.ts",
+    "dev": "node --disable-warning=ExperimentalWarning --experimental-transform-types --watch ./src/run.ts",
     "build:sea": "fossilize -m dist/overlay/manifest.json",
-    "build": "vite build && vite build --config vite.overlay.config.ts && npm run build:sea && tsc",
-    "build:watch": "vite build --watch",
-    "yalc:publish": "yalc publish --push --sig --private",
-    "clean": "rimraf dist"
+    "build": "vite build && vite build --config vite.overlay.config.ts && pnpm build:sea && tsc"
   }
 }

package/bin/instrument.js

@@ -1,107 +0,0 @@
-import { consoleLoggingIntegration, getClient, init } from "@sentry/node";
-import { parseCLIArgs } from "@spotlightjs/sidecar/cli";
-import { DEFAULT_PORT } from "@spotlightjs/sidecar/constants";
-
-// Parse CLI arguments to determine the port this Spotlight instance will use
-const { port: instancePort } = parseCLIArgs();
-
-// Check if SENTRY_SPOTLIGHT points to this instance
-const spotlightEnv = process.env.SENTRY_SPOTLIGHT;
-let disableSpotlight = false;
-
-// Note: If port is set to 0 (dynamic port assignment), we cannot detect the actual port
-// before starting the server, so we cannot prevent the feedback loop in that case.
-// The server will be assigned a port by the OS after it starts, and if SENTRY_SPOTLIGHT
-// happens to point to that same port, a feedback loop may occur.
-if (spotlightEnv && instancePort !== 0) {
-  let targetPort;
-  let targetHost;
-
-  // SENTRY_SPOTLIGHT can be:
-  // 1. A full URL like "http://localhost:8969"
-  // 2. A truthy value (true, t, y, yes, on, 1) which means use the default URL
-  const TRUTHY_ENV_VALUES = new Set(["true", "t", "y", "yes", "on", "1"]);
-  const isTruthy = TRUTHY_ENV_VALUES.has(spotlightEnv.toLowerCase());
-
-  if (isTruthy) {
-    // Use default Spotlight URL
-    targetHost = "localhost";
-    targetPort = DEFAULT_PORT;
-  } else {
-    // Try to parse as URL
-    try {
-      const spotlightUrl = new URL(spotlightEnv);
-      targetHost = spotlightUrl.hostname;
-      targetPort = spotlightUrl.port ? Number(spotlightUrl.port) : spotlightUrl.protocol === "https:" ? 443 : 80;
-    } catch (_err) {
-      // If we can't parse it, we can't determine if it's a feedback loop, so do nothing
-      targetPort = null;
-    }
-  }
-
-  // Only disable if we successfully determined the target and it matches our instance
-  if (targetPort !== null && targetPort !== undefined) {
-    const isLocalhost = targetHost === "localhost" || targetHost === "127.0.0.1" || targetHost === "::1";
-
-    if (isLocalhost && targetPort === instancePort) {
-      disableSpotlight = true;
-      console.warn(
-        `⚠️  [Spotlight] SENTRY_SPOTLIGHT is set to ${spotlightEnv} which points to this Spotlight instance (port ${instancePort}). Disabling Spotlight integration to prevent feedback loop.`,
-      );
-    }
-  }
-}
-
-const sentry = init({
-  dsn: "https://51bcd92dba1128934afd1c5726c84442@o1.ingest.us.sentry.io/4508404727283713",
-  environment: process.env.NODE_ENV || "development",
-  release: `spotlight@${process.env.npm_package_version}`,
-  debug: Boolean(process.env.SENTRY_DEBUG),
-
-  tracesSampleRate: 1,
-  enableLogs: true,
-  ...(disableSpotlight && { spotlight: false }),
-
-  integrations: [
-    consoleLoggingIntegration({
-      levels: ["log", "info", "warn", "error", "debug"],
-    }),
-  ],
-
-  beforeSendTransaction: event => {
-    event.server_name = undefined; // Server name might contain PII
-    return event;
-  },
-
-  beforeSend: event => {
-    const exceptions = event.exception?.values;
-    if (!exceptions) {
-      return event;
-    }
-    for (const exception of exceptions) {
-      if (!exception.stacktrace) {
-        continue;
-      }
-
-      for (const frame of exception.stacktrace.frames) {
-        if (!frame.filename) {
-          continue;
-        }
-
-        const homeDir = process.env.HOME || process.env.USERPROFILE;
-        frame.filename = frame.filename?.replace(homeDir, "~");
-      }
-    }
-
-    event.server_name = undefined; // Server name might contain PII
-    return event;
-  },
-});
-
-function shutdown() {
-  sentry.close();
-}
-
-process.on("SIGINT", shutdown);
-process.on("SIGTERM", shutdown);
-// process.on("beforeExit", shutdown);

package/bin/run.js

@@ -1,63 +0,0 @@
-#!/usr/bin/env node
-import { readFileSync } from "node:fs";
-import Module from "node:module";
-import { join } from "node:path";
-import { fileURLToPath } from "node:url";
-import { setContext, startSpan } from "@sentry/node";
-import { main } from "@spotlightjs/sidecar/cli";
-import "./instrument.js";
-const require = Module.createRequire(import.meta.url);
-let sea = null;
-try {
-  // This is to maintain compatibility with Node 20.11- as
-  // the `node:sea` module is added in Node 20.12+
-  sea = require("node:sea");
-} catch {
-  sea = { isSea: () => false };
-}
-
-const homeDir = process.env.HOME || process.env.USERPROFILE;
-setContext("CLI", {
-  sea: sea.isSea(),
-  // TODO: Be less naive with path obscuring
-  argv: process.argv.map(arg => arg.replace(homeDir, "~")),
-});
-
-const withTracing =
-  (fn, spanArgs = {}) =>
-  (...args) =>
-    startSpan({ name: fn.name, attributes: { args }, ...spanArgs }, () => fn(...args));
-
-const readAsset = withTracing(
-  sea.isSea()
-    ? name => Buffer.from(sea.getRawAsset(name))
-    : (() => {
-        const ASSET_DIR = join(fileURLToPath(import.meta.url), "../../dist/overlay/");
-
-        return name => readFileSync(join(ASSET_DIR, name));
-      })(),
-  { name: "readAsset", op: "cli.asset.read" },
-);
-
-startSpan({ name: "Spotlight CLI", op: "cli" }, async () => {
-  await startSpan({ name: "Setup Sidecar", op: "cli.setup.sidecar" }, async () => {
-    const MANIFEST_NAME = "manifest.json";
-    const ENTRY_POINT_NAME = "src/index.html";
-    const filesToServe = Object.create(null);
-
-    startSpan({ name: "Setup Server Assets", op: "cli.setup.sidecar.assets" }, () => {
-      // Following the guide here: https://vite.dev/guide/backend-integration.html
-      const manifest = JSON.parse(readAsset(MANIFEST_NAME));
-      filesToServe[ENTRY_POINT_NAME] = readAsset(ENTRY_POINT_NAME);
-      const entries = Object.values(manifest);
-      for (const entry of entries) {
-        filesToServe[entry.file] = readAsset(entry.file);
-      }
-    });
-
-    await main({
-      basePath: process.cwd(),
-      filesToServe,
-    });
-  });
-});

package/dist/overlay.cjs

@@ -1 +0,0 @@
-"use strict";Object.defineProperty(exports,Symbol.toStringTag,{value:"Module"});const t=require("@spotlightjs/overlay");Object.keys(t).forEach(e=>{e!=="default"&&!Object.prototype.hasOwnProperty.call(exports,e)&&Object.defineProperty(exports,e,{enumerable:!0,get:()=>t[e]})});

package/dist/overlay.d.ts

@@ -1 +0,0 @@
-export * from "@spotlightjs/overlay";

package/dist/overlay.js

@@ -1 +0,0 @@
-export * from "@spotlightjs/overlay";