fixup!

Author: avivkeller

lib/internal/repl/highlight.js

@@ -56,7 +56,7 @@ function highlight(code) {
   let result = '';
   let offset = 0;
   function write(start, end, style) {
-    result += 
+    result +=
       StringPrototypeSlice(code, offset, start) +
       stylizeWithColor(StringPrototypeSlice(code, start, end), style);
     offset = end;

test/parallel/test-repl-preview-newlines.mjs

@@ -4,7 +4,14 @@ import { startNewREPLServer } from '../common/repl.js';
 
 common.skipIfInspectorDisabled();
 
-const { output, run, replServer } = startNewREPLServer({ useColors: true });
+// Ignore terminal settings so the preview remains active under TERM=dumb.
+process.env.TERM = '';
+
+// Keep syntax highlighting out of this test so it only covers preview layout.
+// Preview and result colors are enabled after readline is initialized.
+const { output, run, replServer } = startNewREPLServer({ useColors: false });
+replServer.useColors = true;
+replServer.writer.options.colors = true;
 
 for (const char of ['\\n', '\\v', '\\r']) {
   output.accumulator = '';

test/parallel/test-repl-preview.mjs

@@ -83,9 +83,13 @@ async function tests(options) {
     prompt: PROMPT,
     stream: new REPLStream(),
     ignoreUndefined: true,
-    useColors: true,
+    // Keep syntax highlighting out of these preview transcript assertions.
+    // Preview and result colors are enabled after readline is initialized.
+    useColors: false,
     ...options
   });
+  repl.useColors = true;
+  repl.writer.options.colors = true;
 
   await runAndWait([
     'function foo(x) { return x; } ' +

test/parallel/test-repl-reverse-search.js

@@ -324,7 +324,9 @@ function runTest() {
     }),
     completer: opts.completer,
     prompt,
-    useColors: opts.useColors || false,
+    // Keep syntax highlighting out of the reverse-search transcript. Result
+    // colors are enabled below after readline has been initialized.
+    useColors: false,
     terminal: true
   }, common.mustCall((err, repl) => {
     if (err) {
@@ -346,6 +348,11 @@ function runTest() {
       setImmediate(runTestWrap, true);
     }));
 
+    if (opts.useColors) {
+      repl.useColors = true;
+      repl.writer.options.colors = true;
+    }
+
     if (opts.columns) {
       Object.defineProperty(repl, 'columns', {
         value: opts.columns,