fix: address review feedback

- Fix BunClient → LightNodeClient type in telemetry-session test
- Remove dead empty alias object from esbuild config
- Override LightNodeClient runtime metadata to report bun when running
  as a compiled Bun binary (the client hardcodes 'node')

Author: BYK

script/bundle.ts

@@ -89,7 +89,6 @@ const result = await build({
   entryPoints: ["./src/bin.ts"],
   bundle: true,
   minify: true,
-  alias: {},
   banner: {
     // Check Node.js version (>= 22 required for node:sqlite) and suppress warnings
     js: `#!/usr/bin/env node

src/lib/telemetry.ts

@@ -338,6 +338,14 @@ export function initSentry(
     const isBun = typeof process.versions.bun !== "undefined";
     const runtime = isBun ? "bun" : "node";
 
+    // LightNodeClient hardcodes runtime to { name: 'node' }. Override it so
+    // events carry the correct runtime when running as a compiled Bun binary.
+    const opts = client.getOptions();
+    opts.runtime = {
+      name: runtime,
+      version: isBun ? process.versions.bun : process.version,
+    };
+
     // Tag whether running as bun binary or node (npm package).
     // Kept alongside the SDK's promoted 'runtime' tag for explicit signaling
     // and backward compatibility with existing dashboards/alerts.

test/lib/telemetry-session.test.ts

@@ -21,13 +21,13 @@ import {
 
 describe("createBeforeExitHandler", () => {
   /**
-   * Create a minimal mock BunClient for testing the beforeExit handler.
+   * Create a minimal mock LightNodeClient for testing the beforeExit handler.
    * Only needs flush() since that's all the handler uses from the client.
    */
-  function createMockClient(): Sentry.BunClient {
+  function createMockClient(): Sentry.LightNodeClient {
     return {
       flush: () => Promise.resolve(true),
-    } as unknown as Sentry.BunClient;
+    } as unknown as Sentry.LightNodeClient;
   }
 
   test("ends OK session on beforeExit", () => {