From db6dd3615f40bb6ca0aefaa312be2af5e094a8fa Mon Sep 17 00:00:00 2001
From: Gilles Soulier <gil.soulier@gmail.com>
Date: Fri, 5 Jun 2026 04:27:22 +0200
Subject: [PATCH] fix: externaliser les deps npm dans le bundle tsup
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Le bundle ESM crashait au runtime (Dynamic require of 'events' via ws).
Externaliser les dépendances (skipNodeModulesBundle) — elles sont fournies
par pnpm install --prod dans l'image. Bug attrapé par la vérif end-to-end (Task 19).

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
---
 tsup.config.ts | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/tsup.config.ts b/tsup.config.ts
index 55989cb..f2d7ece 100644
--- a/tsup.config.ts
+++ b/tsup.config.ts
@@ -7,6 +7,9 @@ export default defineConfig({
   platform: "node",
   target: "node22",
   bundle: true,
-  noExternal: [/^(?!better-sqlite3|ssh2|cpu-features).*/],
-  external: ["better-sqlite3", "ssh2", "cpu-features"],
+  // Bundle uniquement le code first-party (server/, shared/). Les dépendances npm
+  // restent externes — elles sont installées dans l'image via `pnpm install --prod`.
+  // Bundler des paquets CommonJS (ws, ssh2, better-sqlite3…) en ESM casse au runtime
+  // ("Dynamic require ... is not supported"), d'où l'externalisation par défaut.
+  skipNodeModulesBundle: true,
 });