readline: fix to not access a property on an undefined value

PR-URL: nodejs/node#43543
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Tobias Nießen <[email protected]>
Reviewed-By: Yongsheng Zhang <[email protected]>
Reviewed-By: James M Snell <[email protected]>

Author: cola119

lib/internal/readline/utils.js

@@ -366,6 +366,9 @@ function* emitKeys(stream) {
 
 // This runs in O(n log n).
 function commonPrefix(strings) {
+  if (strings.length === 0) {
+    return '';
+  }
   if (strings.length === 1) {
     return strings[0];
   }

test/parallel/test-repl-tab-complete-on-editor-mode.js

@@ -0,0 +1,21 @@
+'use strict';
+
+require('../common');
+const ArrayStream = require('../common/arraystream');
+const repl = require('repl');
+
+const stream = new ArrayStream();
+const replServer = repl.start({
+  input: stream,
+  output: stream,
+  terminal: true,
+});
+
+// Editor mode
+replServer.write('.editor\n');
+
+// Regression test for https://github.com/nodejs/node/issues/43528
+replServer.write('a');
+replServer.write(null, { name: 'tab' }); // Should not throw
+
+replServer.close();