Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

repl: add completion preview #30907

Closed
wants to merge 12 commits into from
Closed

repl: add completion preview #30907

Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
19dac98
repl: remove dead code
BridgeAR Dec 10, 2019
c65f3f1
repl: simplify repl autocompletion
BridgeAR Dec 10, 2019
ec47482
repl: simplify code
BridgeAR Dec 10, 2019
43ae689
readline: update ansi-regex
BridgeAR Dec 11, 2019
1426e6c
repl,readline: refactor common code
BridgeAR Dec 11, 2019
42d3590
repl,readline: refactor for simplicity
BridgeAR Dec 11, 2019
ca14259
repl: improve completion
BridgeAR Dec 11, 2019
1cbef42
repl: add completion preview
BridgeAR Dec 11, 2019
707cba4
repl: fix preview bug in case of long lines
BridgeAR Dec 11, 2019
ca4c894
test: add multiple repl preview tests
BridgeAR Dec 13, 2019
0cd4153
fixup: repl: add completion preview
BridgeAR Dec 14, 2019
2138fd7
fixup: test: add multiple repl preview tests
BridgeAR Dec 14, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
readline: update ansi-regex 
This updates the used regular expression to the latest version.
It includes a number of additional escape codes.
  • Loading branch information
@BridgeAR
BridgeAR committed Dec 14, 2019
commit 43ae68937b5f39f15b9b1bebda68c34e41a53ee0
11 changes: 6 additions & 5 deletions lib/internal/readline/utils.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,12 +7,13 @@ const {

// Regex used for ansi escape code splitting
// Adopted from https://github.com/chalk/ansi-regex/blob/master/index.js
// License: MIT, authors: @sindresorhus, Qix-, and arjunmehta
// License: MIT, authors: @sindresorhus, Qix-, arjunmehta and LitoMore
// Matches all ansi escape code sequences in a string
/* eslint-disable no-control-regex */
const ansi =
/[\u001b\u009b][[()#;?]*(?:[0-9]{1,4}(?:;[0-9]{0,4})*)?[0-9A-ORZcf-nqry=><]/g;
/* eslint-enable no-control-regex */
const ansiPattern = '[\\u001B\\u009B][[\\]()#;?]*' +
'(?:(?:(?:[a-zA-Z\\d]*(?:;[-a-zA-Z\\d\\/#&.:=?%@~_]*)*)?\\u0007)' +
'|(?:(?:\\d{1,4}(?:;\\d{0,4})*)?[\\dA-PR-TZcf-ntqry=><~]))';
const ansi = new RegExp(ansiPattern, 'g');

const kUTF16SurrogateThreshold = 0x10000; // 2 ** 16
const kEscape = '\x1b';

Expand Down