@imenam/simple-scraper 1.0.9 → 1.0.13

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,40 @@ 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;
+    // Signale au master que la GUI est enregistrée et joignable, pour que
+    // reconnect_gui se résolve immédiatement au lieu d'attendre le timeout.
+    ipc.send({
+        type: 'READY',
+        data: { url: result.url ?? `${process.env.PROXY_URL}${APP_PATH}` },
+        timestamp: new Date().toISOString(),
+    });
     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

@@ -9,7 +9,7 @@ import { z } from 'zod';
 import { setupLogging } from './logger.js';
 import { getBrowser, closeBrowser, getDefaultTimeout } from './browser.js';
 import { loadAllCookies } from './cookies.js';
-import { GuiLauncher } from '@imenam/mcp-gui-interface';
+import { GuiLauncher, reconnectGuiToolDefinition } from '@imenam/mcp-gui-interface';
 import { createSession, getSession, closeSession, closeAllSessions, listSessions, getSessionLogs, clearSessionLogs } from './sessions.js';
 import { registerSessionRpc } from './session-rpc.js';
 const __dirname = path.dirname(fileURLToPath(import.meta.url));
@@ -19,9 +19,14 @@ dotenv.config({ path: envPath, override: true });
 setupLogging('MASTER');
 const launcher = new GuiLauncher({
     guiPath: path.join(__dirname, 'gui.js'),
+    reloadEnv: () => dotenv.config({ path: envPath, override: true }),
     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({
@@ -193,6 +198,18 @@ server.tool('scrape_page', 'Navigate to a URL using a headless browser and retur
         await page.close();
     }
 });
+// ─── reconnect_gui ──────────────────────────────────────────────────────────────
+server.tool('reconnect_gui', reconnectGuiToolDefinition().description, {
+    force: z
+        .boolean()
+        .optional()
+        .describe('Restart the GUI worker even if it is already running and registered (default: false)'),
+}, async ({ force }) => {
+    const res = await launcher.handleReconnectTool({ force });
+    return res.isError
+        ? { content: res.content, isError: true }
+        : { content: res.content };
+});
 // ─── execute_js ───────────────────────────────────────────────────────────────
 server.tool('execute_js', 'Navigate to a URL and execute custom JavaScript in the page context. The script is executed as a JavaScript function body with new Function(script)(), so it must use an explicit return statement to send data back to the tool. A bare expression such as document.title returns undefined. Return serializable values only; objects and arrays are returned as formatted JSON, primitives as text. For async work, return a Promise, for example: return (async () => { /* ... */ return data; })();', {
     url: z.string().url().describe('URL of the page'),
@@ -609,17 +626,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.13",
   "description": "MCP server for web scraping and JavaScript execution using Puppeteer",
   "type": "module",
   "main": "dist/index.js",
@@ -35,7 +35,7 @@
     "node": ">=18.0.0"
   },
   "dependencies": {
-    "@imenam/mcp-gui-interface": "^1.0.5",
+    "@imenam/mcp-gui-interface": "^1.0.10",
     "@modelcontextprotocol/sdk": "^1.28.0",
     "body-parser": "^2.2.2",
     "dotenv": "^17.3.1",