Allow Chrome to exit after forced exporter termination

.env.sample

@@ -1,4 +1,5 @@
 # PUPPETEER CONFIG
+PUPPETEER_PIPE = false
 PUPPETEER_TEMP_DIR = ./tmp/
 PUPPETEER_LAUNCH_RETRY_WINDOW = 30000
 

README.md

@@ -161,6 +161,7 @@ The format, along with its default values, is as follows (using the recommended
 {
   "puppeteer": {
     "args": [],
+    "pipe": false,
     "launchRetryWindow": 30000,
     "tempDir": "./tmp/"
   },
@@ -358,6 +359,7 @@ These variables are set in your environment and take precedence over options fro
 
 ### Puppeteer Config
 
+- `PUPPETEER_PIPE`: Use a pipe instead of a WebSocket for the browser connection (defaults to `false`). This disables the browser's WebSocket debugging endpoint. When `DEBUG_ENABLE` is `true`, set `DEBUG_DEBUGGING_PORT` to `0` to avoid combining pipe transport with a nonzero debugging port.
 - `PUPPETEER_TEMP_DIR`: The directory for Puppeteer to store temporary files (defaults to `./tmp/`).
 - `PUPPETEER_LAUNCH_RETRY_WINDOW`: The duration, in milliseconds, to keep retrying a browser launch before reporting it as failed (defaults to `30000`).
 

lib/browser.js

@@ -291,6 +291,7 @@ async function launchBrowser(puppeteerArgs) {
 
   const launchOptions = {
     headless: other.browserShellMode ? 'shell' : true,
+    pipe: puppeteerOptions.pipe,
     userDataDir: puppeteerOptions.tempDir || './tmp/',
     args: puppeteerArgs,
     handleSIGINT: false,

lib/envs.js

@@ -119,6 +119,7 @@ const v = {
 
 export const Config = z.object({
   // puppeteer
+  PUPPETEER_PIPE: v.boolean(),
   PUPPETEER_TEMP_DIR: v.path(),
   PUPPETEER_LAUNCH_RETRY_WINDOW: v.nonNegativeNum(),
 

lib/schemas/config.js

@@ -148,6 +148,13 @@ export const defaultConfig = {
       type: 'string[]',
       description: 'Arguments array to send to Puppeteer.'
     },
+    pipe: {
+      value: false,
+      type: 'boolean',
+      envLink: 'PUPPETEER_PIPE',
+      description:
+        'Use a pipe instead of a WebSocket for the browser connection.'
+    },
     launchRetryWindow: {
       value: 30000,
       type: 'number',
@@ -772,6 +779,12 @@ export const promptsConfig = {
       message: 'Puppeteer arguments',
       initial: defaultConfig.puppeteer.args.value.join(','),
       separator: ','
+    },
+    {
+      type: 'toggle',
+      name: 'pipe',
+      message: 'Use a pipe for the browser connection',
+      initial: defaultConfig.puppeteer.pipe.value
     }
   ],
   highcharts: [

tests/unit/envs.test.js

@@ -1,6 +1,21 @@
 import { Config } from '../../lib/envs';
 
 describe('Environment variables should be correctly parsed', () => {
+  test('PUPPETEER_PIPE should be a boolean', () => {
+    expect(
+      Config.partial().parse({ PUPPETEER_PIPE: 'true' }).PUPPETEER_PIPE
+    ).toEqual(true);
+    expect(
+      Config.partial().parse({ PUPPETEER_PIPE: 'false' }).PUPPETEER_PIPE
+    ).toEqual(false);
+    expect(
+      Config.partial().parse({ PUPPETEER_PIPE: '' }).PUPPETEER_PIPE
+    ).toBeUndefined();
+    expect(() =>
+      Config.partial().parse({ PUPPETEER_PIPE: 'invalid' })
+    ).toThrow();
+  });
+
   test('PUPPETEER_TEMP_DIR should be a valid path', () => {
     const env = { PUPPETEER_TEMP_DIR: '/path/to/dir' };
     expect(Config.partial().parse(env).PUPPETEER_TEMP_DIR).toEqual(