fix: Update regex for removing ANSI colors output (#271)

* fix: Disable color output

* fix: Update regex for removing ANSI colors output

* revert FORCE_COLOR update

* strip stderr as well

* also strip process.stdout

* fix typo

* test output

* test output file

* what if I disable stdout/stderr?

* test different log file name

* put console log under assets folder

* remove debug code

* fix filepath

* let spawn error

* revert debug

* bump package

* remove console.log directly

Author: tianfeng92

package-lock.json

@@ -32,7 +32,7 @@
   "homepage": "https://github.com/saucelabs/sauce-playwright-runner",
   "dependencies": {
     "@playwright/test": "1.45.2",
-    "@saucelabs/playwright-reporter": "1.2.1",
+    "@saucelabs/playwright-reporter": "1.2.2",
     "@saucelabs/testcomposer": "3.0.0",
     "dotenv": "16.4.5",
     "lodash": "4.17.21",

package.json

@@ -1,17 +1,14 @@
-import * as fs from 'node:fs';
 import * as path from 'node:path';
 import { Transform } from 'node:stream';
 import * as childProcess from 'node:child_process';
 import * as process from 'node:process';
 
-const escapeSequenceRegex = new RegExp('[\\u001b]\\[2K|[\\u001b]\\[0G', 'g');
+const escapeSequenceRegex = new RegExp(
+  '[\\u001B\\u009B][[\\]()#;?]*(?:(?:(?:[a-zA-Z\\d]*(?:;[-a-zA-Z\\d\\/#&.:=?%@~_]*)*)?\\u0007)|(?:(?:\\d{1,4}(?:;\\d{0,4})*)?[\\dA-PR-TZcf-ntqry=><~]))',
+  'g',
+);
 
 export function playwrightRecorder() {
-  // console.log is saved out of reportsDir since it is cleared on startup.
-  const ws = fs.createWriteStream(path.join(process.cwd(), 'console.log'), {
-    flags: 'w+',
-    mode: 0o644,
-  });
   const stripAsciiTransform = new Transform({
     transform(chunk, _, callback) {
       // list reporter uses escape codes to rewrite lines, strip them to make console output more readable
@@ -25,13 +22,10 @@ export function playwrightRecorder() {
     ...process.argv.slice(2),
   ]);
 
-  child.stdout.pipe(process.stdout);
+  child.stdout.pipe(stripAsciiTransform).pipe(process.stdout);
   child.stderr.pipe(process.stderr);
-  child.stdout.pipe(stripAsciiTransform).pipe(ws);
-  child.stderr.pipe(ws);
 
   child.on('exit', (exitCode) => {
-    ws.end();
     process.exit(exitCode ?? 1);
   });
 }