readline: fix history manager property propagation

Fix historySize and history properties not being passed to setupHistoryManager when using options object pattern. Remove unused import and trailing whitespace.

Fixes: #61526

Author: Zohaibarif69

lib/internal/readline/interface.js

@@ -235,7 +235,24 @@ function InterfaceConstructor(input, output, completer, terminal) {
   this[kSubstringSearch] = null;
   this.output = output;
   this.input = input;
-  this.setupHistoryManager(inputOptions || input);
+
+  // If inputOptions was provided (constructor with options object),
+  // copy the stream's size/history properties back to it for setupHistoryManager
+  let historyOptions = inputOptions || input;
+  if (inputOptions !== undefined && input !== undefined &&
+      inputOptions !== input) {
+    if (input.size !== undefined && historyOptions.size === undefined) {
+      historyOptions.size = input.size;
+    }
+    if (input.history !== undefined && historyOptions.history === undefined) {
+      historyOptions.history = input.history;
+    }
+    if (input.removeHistoryDuplicates !== undefined &&
+        historyOptions.removeHistoryDuplicates === undefined) {
+      historyOptions.removeHistoryDuplicates = input.removeHistoryDuplicates;
+    }
+  }
+  this.setupHistoryManager(historyOptions);
   this[kUndoStack] = [];
   this[kRedoStack] = [];
   this[kPreviousCursorCols] = -1;
@@ -387,15 +404,15 @@ class Interface extends InterfaceConstructor {
     this.historyManager = new ReplHistory(this, options);
 
     // Only initialize REPL history when called from REPL.setupHistory(),
-    // not from the readline constructor. 
+    // not from the readline constructor.
     // Constructor passes: stream OR { input: stream, output: stream, ... }
     // setupHistory passes: { filePath: ..., size: ..., onHistoryFileLoaded: ... }
     // Detect constructor calls by checking for stream.on() or options.input
     if (options && typeof options === 'object') {
       const isStream = typeof options.on === 'function';
       const hasInputProperty = options.input !== undefined;
       const isFromConstructor = isStream || hasInputProperty;
-      
+
       if (!isFromConstructor && typeof options.onHistoryFileLoaded === 'function') {
         this.historyManager.initialize(options.onHistoryFileLoaded);
       }

test/parallel/test-readline-history-init-order.js

@@ -1,6 +1,5 @@
 'use strict';
 
-const common = require('../common');
 const readline = require('readline');
 const { PassThrough } = require('stream');