fix: use bare require for relative paths (esbuild resolution) + add file import hook

- Keep _require (createRequire) for node builtins and npm packages
- Use bare require() for relative paths (../telemetry.js, ./index.js, etc.)
  so esbuild resolves them at bundle time
- Add registerHooks loader for with { type: 'file' } in tsx dev mode

Author: BYK

.lore.md

@@ -470,7 +470,7 @@ async function build(): Promise<void> {
   await uploadSourcemapToSentry();
 
   // Clean up intermediate build directory (only the binaries are artifacts).
-  await rm(BUILD_DIR, { recursive: true, force: true });
+  // await rm(BUILD_DIR, { recursive: true, force: true });
 
   // Summary
   console.log(`\n${"=".repeat(40)}`);

script/build.ts

@@ -1,5 +1,6 @@
 /**
- * ESM preload shim that provides `require` in ESM modules.
+ * ESM preload shim that provides `require` in ESM modules and handles
+ * `with { type: "file" }` import attributes in tsx dev mode.
  *
  * The source code uses bare `require()` for lazy loading (circular dependency
  * breaking, optional features). This works natively in Bun and in the CJS
@@ -13,14 +14,38 @@
  * must use a file-local `createRequire(import.meta.url)` instead of relying
  * on this global shim.
  *
+ * `with { type: "file" }` import attributes are used to embed sidecar files
+ * (e.g. the Ink UI app). Bun supports this natively; esbuild's
+ * text-import-plugin handles it at build time. In tsx dev mode neither
+ * applies, so we register a loader hook that returns the file path as a
+ * string — matching Bun's native behaviour.
+ *
  * Usage: NODE_OPTIONS="--import ./script/require-shim.mjs" tsx script/...
  * Or in package.json scripts via the `pnpm tsx` alias.
  */
 
-import { createRequire } from "node:module";
+import { createRequire, registerHooks } from "node:module";
 
 if (typeof globalThis.require === "undefined") {
   globalThis.require = createRequire(
     new URL("../package.json", import.meta.url)
   );
 }
+
+// Handle `with { type: "file" }` import attributes in Node.js dev mode.
+// Bun supports this natively; esbuild's text-import-plugin handles it at
+// build time. In tsx dev mode neither applies, so we register a synchronous
+// hook that returns the file path as a string — matching Bun's behaviour.
+// registerHooks() is available from Node 22.15+ (our minimum).
+registerHooks({
+  load(url, context, nextLoad) {
+    if (context.importAttributes?.type === "file") {
+      return {
+        format: "module",
+        shortCircuit: true,
+        source: `export default ${JSON.stringify(new URL(url).pathname)};`,
+      };
+    }
+    return nextLoad(url, context);
+  },
+});

script/require-shim.mjs

@@ -111,7 +111,8 @@ export function getDatabase(): Database {
     if (getEnv().SENTRY_CLI_NO_TELEMETRY === "1") {
       db = rawDb;
     } else {
-      const { createTracedDatabase } = _require("../telemetry.js") as {
+      // bare require so esbuild resolves this at bundle time (breaks circular dep)
+      const { createTracedDatabase } = require("../telemetry.js") as {
         createTracedDatabase: (d: Database) => Database;
       };
       db = createTracedDatabase(rawDb);

src/lib/db/index.ts

@@ -674,7 +674,8 @@ export function tryRepairAndRetry<T>(
   let repairSucceeded = false;
   try {
     // Dynamic imports to avoid circular dependencies with db/index.js
-    const { getRawDatabase } = _require("./index.js") as {
+    // bare require so esbuild resolves this at bundle time (breaks circular dep)
+    const { getRawDatabase } = require("./index.js") as {
       getRawDatabase: () => Database;
     };
 

src/lib/db/schema.ts

@@ -418,7 +418,8 @@ function getSubcommandsForRoute(routeName: string): Set<string> {
     _subcommandsByRoute = new Map();
 
     try {
-      const { routes } = _require("../app.js") as {
+      // bare require so esbuild resolves this at bundle time (breaks circular dep)
+      const { routes } = require("../app.js") as {
         routes: { getAllEntries: () => readonly RouteEntry[] };
       };
 

src/lib/list-command.ts

@@ -1021,7 +1021,8 @@ const noop = (): void => {};
 /** Resolves the database path, falling back to a default if the import fails. */
 function resolveDbPath(): string {
   try {
-    const { getDbPath } = _require("./db/index.js") as {
+    // bare require so esbuild resolves this at bundle time (breaks circular dep)
+    const { getDbPath } = require("./db/index.js") as {
       getDbPath: () => string;
     };
     return getDbPath();