Pārlūkot izejas kodu

Tokenize regex flags as 'keyword.other' like VS Code does

Sebastian Pahnke 6 gadi atpakaļ
vecāks
revīzija
ce3a0652b3

+ 34 - 1
src/javascript/javascript.test.ts

@@ -430,7 +430,7 @@ testTokenization('javascript', [
 		]
 	}],
 
-[{
+	[{
 		line: 'x = /foo/.test(\'\')',
 		tokens: [
 			{ startIndex: 0, type: 'identifier.js' },
@@ -446,6 +446,39 @@ testTokenization('javascript', [
 		]
 	}],
 
+	[{
+		line: '/foo/',
+		tokens: [
+			{ startIndex: 0, type: 'regexp.js' }
+		]
+	}],
+
+	[{
+		line: '/foo/g',
+		tokens: [
+			{ startIndex: 0, type: 'regexp.js' },
+			{ startIndex: 5, type: 'keyword.other.js' }
+		]
+	}],
+
+	[{
+		line: '/foo/gimsuy',
+		tokens: [
+			{ startIndex: 0, type: 'regexp.js' },
+			{ startIndex: 5, type: 'keyword.other.js' }
+		]
+	}],
+
+	[{
+		line: '/foo/q', // invalid flag
+		tokens: [
+			{ startIndex: 0, type: 'delimiter.js' },
+			{ startIndex: 1, type: 'identifier.js' },
+			{ startIndex: 4, type: 'delimiter.js' },
+			{ startIndex: 5, type: 'identifier.js' }
+		]
+	}],
+
 	[{
 		line: 'x = 1 + f(2 / 3, /foo/)',
 		tokens: [

+ 33 - 0
src/typescript/typescript.test.ts

@@ -446,6 +446,39 @@ testTokenization('typescript', [
 		]
 	}],
 
+	[{
+		line: '/foo/',
+		tokens: [
+			{ startIndex: 0, type: 'regexp.ts' }
+		]
+	}],
+
+	[{
+		line: '/foo/g',
+		tokens: [
+			{ startIndex: 0, type: 'regexp.ts' },
+			{ startIndex: 5, type: 'keyword.other.ts' }
+		]
+	}],
+
+	[{
+		line: '/foo/gimsuy',
+		tokens: [
+			{ startIndex: 0, type: 'regexp.ts' },
+			{ startIndex: 5, type: 'keyword.other.ts' }
+		]
+	}],
+
+	[{
+		line: '/foo/q', // invalid flag
+		tokens: [
+			{ startIndex: 0, type: 'delimiter.ts' },
+			{ startIndex: 1, type: 'identifier.ts' },
+			{ startIndex: 4, type: 'delimiter.ts' },
+			{ startIndex: 5, type: 'identifier.ts' }
+		]
+	}],
+
 	[{
 		line: 'x = 1 + f(2 / 3, /foo/)',
 		tokens: [

+ 1 - 1
src/typescript/typescript.ts

@@ -190,7 +190,7 @@ export const language = {
 			[/[^\\\/]/, 'regexp'],
 			[/@regexpesc/, 'regexp.escape'],
 			[/\\\./, 'regexp.invalid'],
-			['/', { token: 'regexp', bracket: '@close' }, '@pop'],
+			[/(\/)([gimsuy]*)/, [{ token: 'regexp', bracket: '@close', next: '@pop' }, 'keyword.other']],
 		],
 
 		regexrange: [