refactor: use SENTRY_CLI_NO_TELEMETRY env var instead of disableDbTracing

Replace the in-process tracingDisabled flag and disableDbTracing()
export with the existing SENTRY_CLI_NO_TELEMETRY=1 env var. The
__complete fast-path sets it before any imports, and db/index.ts
checks it when deciding whether to lazy-require the tracing wrapper.

Removes one export, one module-level variable, and one static import.

Author: BYK

src/bin.ts

@@ -6,8 +6,6 @@
  * the full CLI with telemetry, middleware, and error recovery.
  */
 
-import { disableDbTracing } from "./lib/db/index.js";
-
 // Exit cleanly when downstream pipe consumer closes (e.g., `sentry issue list | head`).
 // EPIPE (errno -32) is normal Unix behavior — not an error. Node.js/Bun ignore SIGPIPE
 // at the process level, so pipe write failures surface as async 'error' events on the
@@ -30,7 +28,8 @@ process.stderr.on("error", handleStreamError);
  * completion engine and SQLite cache modules.
  */
 async function runCompletion(completionArgs: string[]): Promise<void> {
-  disableDbTracing();
+  // Disable telemetry so db/index.ts skips the @sentry/bun lazy-require (~280ms)
+  process.env.SENTRY_CLI_NO_TELEMETRY = "1";
   const { handleComplete } = await import("./lib/complete.js");
   await handleComplete(completionArgs);
 }

src/lib/db/index.ts

@@ -21,23 +21,6 @@ const CACHE_TTL_MS = 7 * 24 * 60 * 60 * 1000;
 /** Probability of running cleanup on write operations */
 const CLEANUP_PROBABILITY = 0.1;
 
-/** When true, skip the Sentry tracing wrapper on the database connection. */
-let tracingDisabled = false;
-
-/**
- * Disable database query tracing for this process.
- *
- * Call before the first `getDatabase()` invocation to avoid loading
- * `@sentry/bun` (~280ms). Used by the `__complete` fast-path where
- * only cached reads are needed and telemetry adds unacceptable latency.
- *
- * Follows the same pattern as {@link disableResponseCache} and
- * {@link disableOrgCache}.
- */
-export function disableDbTracing(): void {
-  tracingDisabled = true;
-}
-
 /** Traced database wrapper (returned by getDatabase) */
 let db: Database | null = null;
 /** Raw database without tracing (used for repair operations) */
@@ -118,8 +101,8 @@ export function getDatabase(): Database {
 
     // Wrap with tracing proxy for automatic query instrumentation.
     // Lazy-require telemetry to avoid top-level import of @sentry/bun (~280ms).
-    // Shell completions disable tracing entirely via disableDbTracing().
-    if (tracingDisabled) {
+    // Shell completions set SENTRY_CLI_NO_TELEMETRY=1 to skip this entirely.
+    if (process.env.SENTRY_CLI_NO_TELEMETRY === "1") {
       db = rawDb;
     } else {
       const { createTracedDatabase } = require("../telemetry.js") as {