Don't use regex

Author: jakebailey

src/compiler/scanner.ts

@@ -327,11 +327,6 @@ const commentDirectiveRegExSingleLine = /^\/\/\/?\s*@(ts-expect-error|ts-ignore)
  */
 const commentDirectiveRegExMultiLine = /^(?:\/|\*)*\s*@(ts-expect-error|ts-ignore)/;
 
-/**
- * Test for whether a comment contains a JSDoc tag needed by the checker when run in tsc.
- */
-const semanticJSDocTagRegEx = /@(?:see|link)/i;
-
 function lookupInUnicodeMap(code: number, map: readonly number[]): boolean {
     // Bail out quickly if it couldn't possibly be in the map.
     if (code < map[0]) {
@@ -1834,6 +1829,7 @@ export function createScanner(languageVersion: ScriptTarget,
                     if (text.charCodeAt(pos + 1) === CharacterCodes.asterisk) {
                         pos += 2;
                         const isJSDoc = text.charCodeAt(pos) === CharacterCodes.asterisk && text.charCodeAt(pos + 1) !== CharacterCodes.slash;
+                        let containsSeeOrLink = false;
 
                         let commentClosed = false;
                         let lastLineStart = tokenPos;
@@ -1846,6 +1842,36 @@ export function createScanner(languageVersion: ScriptTarget,
                                 break;
                             }
 
+                            if (skipJSDoc && isJSDoc && !containsSeeOrLink) {
+                                if (ch === CharacterCodes.at) {
+                                    const ch1 = text.charCodeAt(pos + 1);
+                                    const ch2 = text.charCodeAt(pos + 2);
+                                    const ch3 = text.charCodeAt(pos + 3);
+                                    if (ch1 === CharacterCodes.s || ch1 === CharacterCodes.S) {
+                                        if (
+                                            (ch2 === CharacterCodes.e || ch2 === CharacterCodes.E)
+                                            && (ch3 === CharacterCodes.e || ch3 === CharacterCodes.E)
+                                        ) {
+                                            containsSeeOrLink = true;
+                                            pos += 3;
+                                            continue;
+                                        }
+                                    }
+                                    else if (ch1 === CharacterCodes.l || ch1 === CharacterCodes.L) {
+                                        const ch4 = text.charCodeAt(pos + 4);
+                                        if (
+                                            (ch2 === CharacterCodes.i || ch2 === CharacterCodes.I)
+                                            && (ch3 === CharacterCodes.n || ch3 === CharacterCodes.N)
+                                            && (ch4 === CharacterCodes.k || ch4 === CharacterCodes.K)
+                                        ) {
+                                            containsSeeOrLink = true;
+                                            pos += 4;
+                                            continue;
+                                        }
+                                    }
+                                }
+                            }
+
                             pos++;
 
                             if (isLineBreak(ch)) {
@@ -1854,7 +1880,7 @@ export function createScanner(languageVersion: ScriptTarget,
                             }
                         }
 
-                        if (isJSDoc && (!skipJSDoc || semanticJSDocTagRegEx.test(text.slice(tokenPos, pos)))) {
+                        if (isJSDoc && (!skipJSDoc || containsSeeOrLink)) {
                             tokenFlags |= TokenFlags.PrecedingJSDocComment;
                         }
 

tests/cases/fourslash/jsdocDeprecated_suggestion_skipJSDoc.ts

@@ -10,5 +10,19 @@
 ////function imDeprecated() {}
 ////[|imDeprecated|]()
 
-const [range] = test.ranges();
-verify.getSuggestionDiagnostics([]);
+/////**
+//// * {@see imDeprecated}
+//// * @deprecated
+//// */
+////function imDeprecated2() {}
+////[|imDeprecated2|]()
+
+const [, range] = test.ranges();
+verify.getSuggestionDiagnostics([
+    {
+        "code": 6387,
+        "message": "The signature '(): void' of 'imDeprecated2' is deprecated.",
+        "reportsDeprecated": true,
+        "range": range
+    },
+]);