@imenam/simple-scraper 1.0.9 → 1.0.12

package/dist/gui.js

@@ -20,6 +20,47 @@ if (!process.env.PROXY_URL) {
     process.stderr.write('[GUI] PROXY_URL non défini — arrêt.\n');
     process.exit(0);
 }
+const APP_PATH = process.env.PROXY_APP_PATH ?? '/simple-scraper-mcp';
+const APP_NAME = process.env.PROXY_APP_NAME ?? 'Simple Scraper MCP';
+// ─── Cycle de vie du process ─────────────────────────────────────────────────
+let proxyClient = null;
+/**
+ * Libère la route avant de mourir. Sans ça le proxy continue de router vers un
+ * port mort (502 opaques) jusqu'à ce que son health check finisse par nettoyer.
+ */
+async function cleanupProxy() {
+    if (proxyClient)
+        await proxyClient.unregister();
+    process.exit(0);
+}
+process.on('SIGTERM', () => void cleanupProxy());
+process.on('SIGINT', () => void cleanupProxy());
+// Le master peut disparaître sans nous tuer (crash, kill -9) : sans ce watchdog
+// la GUI reste orpheline et garde le port et la route du proxy.
+const parentPid = process.ppid;
+const parentCheckInterval = setInterval(() => {
+    try {
+        process.kill(parentPid, 0);
+    }
+    catch {
+        console.error(`[GUI] Master (PID ${parentPid}) disparu — arrêt.`);
+        clearInterval(parentCheckInterval);
+        void cleanupProxy();
+    }
+}, 2000);
+process.on('disconnect', () => {
+    console.error('[GUI] Canal IPC fermé par le master — arrêt.');
+    void cleanupProxy();
+});
+process.on('uncaughtException', (err) => {
+    console.error(`[GUI] Uncaught Exception: ${err.message}`);
+    console.error(err.stack ?? 'No stack trace');
+    process.exit(1);
+});
+process.on('unhandledRejection', (reason) => {
+    console.error(`[GUI] Unhandled Rejection: ${reason}`);
+    process.exit(1);
+});
 const getCookiesDir = () => process.env.COOKIES_DIR ?? null;
 const app = express();
 app.use(bodyParser.json());
@@ -130,20 +171,33 @@ app.get('/api/sessions/:id/screenshot', async (req, res) => {
     }
 });
 async function startServer() {
-    const proxyClient = new ProxyClient(process.env.PROXY_URL);
-    const result = await proxyClient.register({
-        path: process.env.PROXY_APP_PATH ?? '/simple-scraper-mcp',
-        name: process.env.PROXY_APP_NAME ?? 'Simple Scraper MCP',
-    });
+    const client = new ProxyClient(process.env.PROXY_URL);
+    const result = await client.register({ path: APP_PATH, name: APP_NAME });
+    // Une autre instance du serveur MCP sert déjà cette GUI. C'est le cas nominal
+    // quand le client redémarre le serveur : il lance le nouveau master avant de
+    // couper l'ancien, dont la GUI tient encore la route. Ce n'est pas un crash —
+    // on sort en 0 (sinon le launcher log "Giving up") sans toucher au proxy, dont
+    // la route appartient à l'autre instance.
+    if (result.error === 'HTTP 409') {
+        console.error('[GUI] Une autre instance sert déjà cette GUI — arrêt.');
+        ipc.send({
+            type: 'ALREADY_RUNNING',
+            data: { url: `${process.env.PROXY_URL}${APP_PATH}` },
+            timestamp: new Date().toISOString(),
+        });
+        process.exit(0);
+    }
     if (!result.success) {
-        process.stderr.write('[GUI] Enregistrement proxy échoué — arrêt.\n');
+        process.stderr.write(`[GUI] Enregistrement proxy échoué (${result.error}) — arrêt.\n`);
         process.exit(1);
     }
+    // À partir d'ici la route nous appartient : cleanupProxy peut la libérer.
+    proxyClient = client;
     const port = result.port;
     app.post('/api/stop', async (_req, res) => {
         console.error('POST /api/stop called');
         res.json({ success: true, message: 'Stopping...' });
-        await proxyClient.unregister();
+        await client.unregister();
         setTimeout(() => {
             if (process.send && process.connected) {
                 process.send({ type: 'STOP' });

package/dist/index.js

@@ -20,8 +20,12 @@ setupLogging('MASTER');
 const launcher = new GuiLauncher({
     guiPath: path.join(__dirname, 'gui.js'),
     onMessage: (msg) => {
-        if (msg.type === 'STOP')
+        if (msg.type === 'STOP') {
             process.exit(0);
+        }
+        else if (msg.type === 'ALREADY_RUNNING') {
+            console.error(`[MASTER] GUI déjà servie par une autre instance : ${msg.data?.url ?? 'URL inconnue'}`);
+        }
     },
 });
 const server = new McpServer({
@@ -609,17 +613,21 @@ server.tool('screenshot', 'Take a screenshot of a page. Provide either session_i
         }
     }
 });
-async function main() {
+function launchGUI() {
     if (!process.env.PROXY_URL) {
         console.error('[MASTER] PROXY_URL non défini — GUI désactivée.');
+        return;
     }
-    else {
-        launcher.start();
-        registerSessionRpc(launcher);
-    }
+    launcher.start();
+    registerSessionRpc(launcher);
+}
+async function main() {
     const transport = new StdioServerTransport();
     await server.connect(transport);
     console.error('[MCP] Simple Scraper MCP server started');
+    // Après le connect, comme mcp-documentor : au redémarrage du serveur par le
+    // client, le master sortant a alors eu le temps de libérer la route.
+    launchGUI();
 }
 main().catch(async (err) => {
     console.error('[MCP] Fatal error:', err);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@imenam/simple-scraper",
-  "version": "1.0.9",
+  "version": "1.0.12",
   "description": "MCP server for web scraping and JavaScript execution using Puppeteer",
   "type": "module",
   "main": "dist/index.js",