fix: remove syntax highlighting upon encountering invalid JS

Snehil-Shah · web-flow · commit e9c6d40b6d4d · 2024-08-08T00:23:21.000-07:00
Previously, if you entered `varlkdf`, the `var` would remain highlighted. This commit addresses this syntax highlighting bug. PR-URL: stdlib-js#2758 Reviewed-by: Athan Reines <kgryte@gmail.com> Signed-off-by: Snehil Shah <snehilshah.989@gmail.com>
diff --git a/lib/node_modules/@stdlib/repl/lib/syntax_highlighter.js b/lib/node_modules/@stdlib/repl/lib/syntax_highlighter.js
@@ -328,20 +328,20 @@ setNonEnumerableReadOnly( SyntaxHighlighter.prototype, 'onKeypress', function on
 	if ( this._line !== this._rli.line ) {
 		// Tokenize:
 		debug( 'Line change detected. Tokenizing line: %s', this._rli.line );
-		tokens = tokenizer( this._rli.line, this._repl._context );
+		this._line = this._rli.line; // update line buffer
+		tokens = tokenizer( this._line, this._repl._context );
 		if ( isEmptyArray( tokens ) ) {
 			debug( 'No tokens found. Skipping highlighting...' );
-			this._multilineHandler.updateLine( this._rli.line ); // save displayed line
-			return;
+			this._highlightedLine = this._line;
+		} else {
+			// Process tokens:
+			tokens.sort( tokenComparator );
+			tokens = removeDuplicateTokens( tokens );
+
+			// Highlight:
+			debug( '%d tokens found. Highlighting...', tokens.length );
+			this._highlightedLine = this._highlightLine( this._line, tokens );
 		}
-		// Process tokens:
-		tokens.sort( tokenComparator );
-		tokens = removeDuplicateTokens( tokens );
-
-		// Highlight:
-		debug( '%d tokens found. Highlighting...', tokens.length );
-		this._line = this._rli.line; // updated line buffer
-		this._highlightedLine = this._highlightLine( this._line, tokens );
 	}
 	// Replace:
 	debug( 'Replacing current line with the highlighted line...' );
