repl: copying tabs shouldn't trigger completion

PR-URL: #5958Fixes: #5954 Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Jeremiah Senkpiel <[email protected]>

Author: ghaiklor

doc/api/readline.md

@@ -354,6 +354,13 @@ a `'resize'` event on the `output` if/when the columns ever change
 
 Move cursor to the specified position in a given TTY stream.
 
+## readline.emitKeypressEvents(stream[, interface])
+
+Causes `stream` to begin emitting `'keypress'` events corresponding to its
+input.
+Optionally, `interface` specifies a `readline.Interface` instance for which
+autocompletion is disabled when copy-pasted input is detected.
+
 ## readline.moveCursor(stream, dx, dy)
 
 Move cursor relative to it's current position in a given TTY stream.

lib/readline.js

@@ -36,6 +36,7 @@ function Interface(input, output, completer, terminal){
 }
 
 this._sawReturn=false;
+this.isCompletionEnabled=true;
 
 EventEmitter.call(this);
 varhistorySize;
@@ -122,7 +123,7 @@ function Interface(input, output, completer, terminal){
 
 }else{
 
-exports.emitKeypressEvents(input);
+exports.emitKeypressEvents(input,this);
 
 // input usually refers to stdin
 input.on('keypress',onkeypress);
@@ -868,7 +869,7 @@ Interface.prototype._ttyWrite = function(s, key){
 
 case'tab':
 // If tab completion enabled, do that...
-if(typeofthis.completer==='function'){
+if(typeofthis.completer==='function'&&this.isCompletionEnabled){
 this._tabComplete();
 break;
 }
@@ -902,7 +903,7 @@ exports.Interface = Interface;
 constKEYPRESS_DECODER=Symbol('keypress-decoder');
 constESCAPE_DECODER=Symbol('escape-decoder');
 
-functionemitKeypressEvents(stream){
+functionemitKeypressEvents(stream,iface){
 if(stream[KEYPRESS_DECODER])return;
 varStringDecoder=require('string_decoder').StringDecoder;// lazy load
 stream[KEYPRESS_DECODER]=newStringDecoder('utf8');
@@ -915,6 +916,10 @@ function emitKeypressEvents(stream){
 varr=stream[KEYPRESS_DECODER].write(b);
 if(r){
 for(vari=0;i<r.length;i++){
+if(r[i]==='\t'&&typeofr[i+1]==='string'&&iface){
+iface.isCompletionEnabled=false;
+}
+
 try{
 stream[ESCAPE_DECODER].next(r[i]);
 }catch(err){
@@ -923,6 +928,10 @@ function emitKeypressEvents(stream){
 stream[ESCAPE_DECODER]=emitKeys(stream);
 stream[ESCAPE_DECODER].next();
 throwerr;
+}finally{
+if(iface){
+iface.isCompletionEnabled=true;
+}
 }
 }
 }

test/parallel/test-readline-interface.js

@@ -208,7 +208,9 @@ function isWarned(emitter){
 assert.strictEqual(called,false);
 called=true;
 });
-fi.emit('data','\tfo\to\t');
+for(varcharacterof'\tfo\to\t'){
+fi.emit('data',character);
+}
 fi.emit('data','\n');
 assert.ok(called);
 rli.close();