@khanacademy/perseus-linter 0.3.0 → 0.3.2

package/src/__tests__/rules.test.ts

@@ -47,7 +47,7 @@ describe("Individual lint rules tests", () => {
             if (TreeTransformer.isTextNode(node)) {
                 let next = state.nextSibling();
                 while (TreeTransformer.isTextNode(next)) {
-                    // @ts-expect-error [FEI-5003] - TS2339 - Property 'content' does not exist on type 'TreeNode'. | TS2533 - Object is possibly 'null' or 'undefined'. | TS2339 - Property 'content' does not exist on type 'TreeNode'.
+                    // @ts-expect-error - TS2339 - Property 'content' does not exist on type 'TreeNode'. | TS2533 - Object is possibly 'null' or 'undefined'. | TS2339 - Property 'content' does not exist on type 'TreeNode'.
                     node.content += next.content;
                     state.removeNextSibling();
                     next = state.nextSibling();
@@ -66,7 +66,7 @@ describe("Individual lint rules tests", () => {
         tt.traverse((node, state, content) => {
             const check = rule.check(node, state, content, context);
             if (check) {
-                // @ts-expect-error [FEI-5003] - TS2345 - Argument of type 'any' is not assignable to parameter of type 'never'.
+                // @ts-expect-error - TS2345 - Argument of type 'any' is not assignable to parameter of type 'never'.
                 warnings.push(check);
             }
         });
@@ -102,62 +102,62 @@ describe("Individual lint rules tests", () => {
     const sentence = new Array(25).fill("lorem ipsum").join(" ");
 
     // long-paragraph rule warns about paragraphs over 500 characters
-    // @ts-expect-error [FEI-5003] - TS2554 - Expected 3 arguments, but got 2.
+    // @ts-expect-error - TS2554 - Expected 3 arguments, but got 2.
     expectWarning(longParagraphRule, sentence + sentence);
-    // @ts-expect-error [FEI-5003] - TS2554 - Expected 3 arguments, but got 2.
+    // @ts-expect-error - TS2554 - Expected 3 arguments, but got 2.
     expectPass(longParagraphRule, [sentence, sentence + "\n\n" + sentence]);
 
-    // @ts-expect-error [FEI-5003] - TS2554 - Expected 3 arguments, but got 2.
+    // @ts-expect-error - TS2554 - Expected 3 arguments, but got 2.
     expectWarning(headingLevel1Rule, "# Level 1 heading");
-    // @ts-expect-error [FEI-5003] - TS2554 - Expected 3 arguments, but got 2.
+    // @ts-expect-error - TS2554 - Expected 3 arguments, but got 2.
     expectPass(headingLevel1Rule, "## Level 1 heading\n\n### Level 3 heading");
 
-    // @ts-expect-error [FEI-5003] - TS2554 - Expected 3 arguments, but got 2.
+    // @ts-expect-error - TS2554 - Expected 3 arguments, but got 2.
     expectWarning(headingLevelSkipRule, "## heading 1\n\n#### heading 2");
-    // @ts-expect-error [FEI-5003] - TS2554 - Expected 3 arguments, but got 2.
+    // @ts-expect-error - TS2554 - Expected 3 arguments, but got 2.
     expectPass(headingLevelSkipRule, [
         "## heading 1\n\n### heading 2\n\n#### heading 3\n\n### heading 4",
         "## heading 1\n\n##heading 2\n\n##heading3",
     ]);
 
-    // @ts-expect-error [FEI-5003] - TS2554 - Expected 3 arguments, but got 2.
+    // @ts-expect-error - TS2554 - Expected 3 arguments, but got 2.
     expectWarning(
         headingTitleCaseRule,
         "## This Heading is in Title Case and the but nor for Too",
     );
-    // @ts-expect-error [FEI-5003] - TS2554 - Expected 3 arguments, but got 2.
+    // @ts-expect-error - TS2554 - Expected 3 arguments, but got 2.
     expectPass(headingTitleCaseRule, [
         "## This heading is in sentence case",
         "## Acronyms: The CIA, NSA, DNI, and FBI",
         "## The Great War",
     ]);
 
-    // @ts-expect-error [FEI-5003] - TS2554 - Expected 3 arguments, but got 2.
+    // @ts-expect-error - TS2554 - Expected 3 arguments, but got 2.
     expectWarning(headingSentenceCaseRule, [
         "## this heading is uncapitalized",
         "## 'this' heading is uncapitalized",
         "##   this heading is uncapitalized",
     ]);
-    // @ts-expect-error [FEI-5003] - TS2554 - Expected 3 arguments, but got 2.
+    // @ts-expect-error - TS2554 - Expected 3 arguments, but got 2.
     expectPass(headingSentenceCaseRule, [
         "## This heading is in sentence case",
         "## 'This heading too'",
         "## 2 + 2 = 4",
     ]);
 
-    // @ts-expect-error [FEI-5003] - TS2554 - Expected 3 arguments, but got 2.
+    // @ts-expect-error - TS2554 - Expected 3 arguments, but got 2.
     expectWarning(nestedListsRule, [
         "1. outer\n  * nested\n  *nested",
         " + outer\n\n   1. nested",
     ]);
-    // @ts-expect-error [FEI-5003] - TS2554 - Expected 3 arguments, but got 2.
+    // @ts-expect-error - TS2554 - Expected 3 arguments, but got 2.
     expectPass(nestedListsRule, [
         "-one\n-two\n-three",
         "1. one\n 2. two\n3. three",
         " * one\n\n * two\n\n * three",
     ]);
 
-    // @ts-expect-error [FEI-5003] - TS2554 - Expected 3 arguments, but got 2.
+    // @ts-expect-error - TS2554 - Expected 3 arguments, but got 2.
     expectWarning(imageAltTextRule, [
         "![](http://google.com/)",
         '![](http://google.com/ "title")',
@@ -167,42 +167,42 @@ describe("Individual lint rules tests", () => {
         "![blah](http://google.com/)", // too short to be meaningful
     ]);
 
-    // @ts-expect-error [FEI-5003] - TS2554 - Expected 3 arguments, but got 2.
+    // @ts-expect-error - TS2554 - Expected 3 arguments, but got 2.
     expectPass(imageAltTextRule, [
         "![alt-text](http://google.com)",
         '![alternative text](http://google.com/ "title")',
         "![alt alt alt][url-ref]",
     ]);
 
-    // @ts-expect-error [FEI-5003] - TS2554 - Expected 3 arguments, but got 2.
+    // @ts-expect-error - TS2554 - Expected 3 arguments, but got 2.
     expectWarning(blockquotedMathRule, ["> $1$", "Quote:\n\n> $x$\n\n"]);
-    // @ts-expect-error [FEI-5003] - TS2554 - Expected 3 arguments, but got 2.
+    // @ts-expect-error - TS2554 - Expected 3 arguments, but got 2.
     expectPass(blockquotedMathRule, [
         "$x$",
         "\n$x$\n  $y$\n",
         "> bq #1\n\n$x+y=1$\n\n> bq #2",
     ]);
 
-    // @ts-expect-error [FEI-5003] - TS2554 - Expected 3 arguments, but got 2.
+    // @ts-expect-error - TS2554 - Expected 3 arguments, but got 2.
     expectWarning(blockquotedWidgetRule, ["> [[☃ passage 1]]"]);
-    // @ts-expect-error [FEI-5003] - TS2554 - Expected 3 arguments, but got 2.
+    // @ts-expect-error - TS2554 - Expected 3 arguments, but got 2.
     expectPass(blockquotedWidgetRule, [
         "[[☃ passage 1]]",
         "> bq #1\n\nTesting [[☃ passage 1]] testing\n\n> bq #2",
     ]);
 
-    // @ts-expect-error [FEI-5003] - TS2554 - Expected 3 arguments, but got 2.
+    // @ts-expect-error - TS2554 - Expected 3 arguments, but got 2.
     expectWarning(linkClickHereRule, [
         "[click here](http://google.com)",
         "[Click here, please](http://google.com)",
         "[For a good time, Click Here](http://google.com)",
     ]);
-    // @ts-expect-error [FEI-5003] - TS2554 - Expected 3 arguments, but got 2.
+    // @ts-expect-error - TS2554 - Expected 3 arguments, but got 2.
     expectPass(linkClickHereRule, [
         "[click to activate this link here](http://google.com)",
     ]);
 
-    // @ts-expect-error [FEI-5003] - TS2554 - Expected 3 arguments, but got 2.
+    // @ts-expect-error - TS2554 - Expected 3 arguments, but got 2.
     expectWarning(absoluteUrlRule, [
         // Warn about absolute khanacademy.org urls
         "[target](http://khanacademy.org/about)",
@@ -219,7 +219,7 @@ describe("Individual lint rules tests", () => {
         "![alt text](https://www.khanacademy.org/about)",
         "![alt text](https://es.khanacademy.org/about)",
     ]);
-    // @ts-expect-error [FEI-5003] - TS2554 - Expected 3 arguments, but got 2.
+    // @ts-expect-error - TS2554 - Expected 3 arguments, but got 2.
     expectPass(absoluteUrlRule, [
         "[target](/about)", // relative URLs okay
         "[target](https://kasandbox.org/path)",
@@ -234,25 +234,25 @@ describe("Individual lint rules tests", () => {
         "![alt text](https://ka-perseus-images.s3.amazonaws.com/path)",
     ]);
 
-    // @ts-expect-error [FEI-5003] - TS2554 - Expected 3 arguments, but got 2.
+    // @ts-expect-error - TS2554 - Expected 3 arguments, but got 2.
     expectWarning(imageInTableRule, [
         "|col1|col2|\n|----|----|\n|![alt-text](/link.gif)|cell2|",
     ]);
-    // @ts-expect-error [FEI-5003] - TS2554 - Expected 3 arguments, but got 2.
+    // @ts-expect-error - TS2554 - Expected 3 arguments, but got 2.
     expectPass(imageInTableRule, [
         "![alt-text](/link.gif)\n|col1|col2|\n|----|----|\n|cell1|cell2|",
     ]);
 
-    // @ts-expect-error [FEI-5003] - TS2554 - Expected 3 arguments, but got 2.
+    // @ts-expect-error - TS2554 - Expected 3 arguments, but got 2.
     expectWarning(widgetInTableRule, [
         "|col1|col2|\n|----|----|\n|[[☃ passage 1]]|cell2|",
     ]);
-    // @ts-expect-error [FEI-5003] - TS2554 - Expected 3 arguments, but got 2.
+    // @ts-expect-error - TS2554 - Expected 3 arguments, but got 2.
     expectPass(widgetInTableRule, [
         "[[☃ passage 1]]\n|col1|col2|\n|----|----|\n|cell1|cell2|",
     ]);
 
-    // @ts-expect-error [FEI-5003] - TS2554 - Expected 3 arguments, but got 2.
+    // @ts-expect-error - TS2554 - Expected 3 arguments, but got 2.
     expectWarning(tableMissingCellsRule, [
         "|col1|col2|col3|\n|----|----|----|\n|col1|col2|col3|\n|cell1|cell2|",
         "|col1|col2|col3|\n|----|----|----|\n|col1|col2|\n|cell1|cell2|",
@@ -260,19 +260,19 @@ describe("Individual lint rules tests", () => {
         "|col1|\n|----|----|\n|col1|\n|cell1|cell2|",
         "|col1|col2|\n|----|----|\n|col1|\n|cell1|cell2|",
     ]);
-    // @ts-expect-error [FEI-5003] - TS2554 - Expected 3 arguments, but got 2.
+    // @ts-expect-error - TS2554 - Expected 3 arguments, but got 2.
     expectPass(tableMissingCellsRule, [
         "|col1|col2|\n|----|----|\n|cell1|cell2|\n|cell1|cell2|",
         "|cell1|\n|----|\n|cell2|\n|cell3|",
     ]);
 
-    // @ts-expect-error [FEI-5003] - TS2554 - Expected 3 arguments, but got 2.
+    // @ts-expect-error - TS2554 - Expected 3 arguments, but got 2.
     expectWarning(unescapedDollarRule, ["It costs $10", "It costs $$10$"]);
 
-    // @ts-expect-error [FEI-5003] - TS2554 - Expected 3 arguments, but got 2.
+    // @ts-expect-error - TS2554 - Expected 3 arguments, but got 2.
     expectPass(unescapedDollarRule, ["It costs \\$10", "It costs $10x$"]);
 
-    // @ts-expect-error [FEI-5003] - TS2554 - Expected 3 arguments, but got 2.
+    // @ts-expect-error - TS2554 - Expected 3 arguments, but got 2.
     expectWarning(mathStartsWithSpaceRule, [
         "foo$~ x$bar",
         "$\\qquad x$",
@@ -285,7 +285,7 @@ describe("Individual lint rules tests", () => {
         "$\\enspace x$",
         "$\\phantom{xyz} x$",
     ]);
-    // @ts-expect-error [FEI-5003] - TS2554 - Expected 3 arguments, but got 2.
+    // @ts-expect-error - TS2554 - Expected 3 arguments, but got 2.
     expectPass(mathStartsWithSpaceRule, [
         "$a~ x$",
         "$a\\qquad x$",
@@ -299,51 +299,51 @@ describe("Individual lint rules tests", () => {
         "$a\\phantom{xyz} x$",
     ]);
 
-    // @ts-expect-error [FEI-5003] - TS2554 - Expected 3 arguments, but got 2.
+    // @ts-expect-error - TS2554 - Expected 3 arguments, but got 2.
     expectWarning(mathEmptyRule, [
         "foo $$ bar",
         "foo\n\n$$\n\nbar",
         "$$ | $$ | $$\n- | - | -\ndata 1 | data 2 | data 3",
     ]);
-    // @ts-expect-error [FEI-5003] - TS2554 - Expected 3 arguments, but got 2.
+    // @ts-expect-error - TS2554 - Expected 3 arguments, but got 2.
     expectPass(mathEmptyRule, [
         "foo $x$ bar",
         "foo\n\n$x$\n\nbar",
         "$x$ | $y$ | $z$\n- | - | -\ndata 1 | data 2 | data 3",
     ]);
 
-    // @ts-expect-error [FEI-5003] - TS2554 - Expected 3 arguments, but got 2.
+    // @ts-expect-error - TS2554 - Expected 3 arguments, but got 2.
     expectWarning(mathFracRule, ["$\\frac 12$", "$\\frac{1}{2}$"]);
-    // @ts-expect-error [FEI-5003] - TS2554 - Expected 3 arguments, but got 2.
+    // @ts-expect-error - TS2554 - Expected 3 arguments, but got 2.
     expectPass(mathFracRule, [
         "$\\dfrac 12$",
         "$\\dfrac{1}{2}$",
         "$\\fraction 12$",
     ]);
 
-    // @ts-expect-error [FEI-5003] - TS2554 - Expected 3 arguments, but got 2.
+    // @ts-expect-error - TS2554 - Expected 3 arguments, but got 2.
     expectWarning(mathTextEmptyRule, [
         "$x\\text{}y$",
         "$x\\text{ }y$",
         "$x\\text{\n}y$",
         "$x\\text{\t}y$",
     ]);
-    // @ts-expect-error [FEI-5003] - TS2554 - Expected 3 arguments, but got 2.
+    // @ts-expect-error - TS2554 - Expected 3 arguments, but got 2.
     expectPass(mathTextEmptyRule, ["$x\\text{z}y$"]);
 
-    // @ts-expect-error [FEI-5003] - TS2554 - Expected 3 arguments, but got 2.
+    // @ts-expect-error - TS2554 - Expected 3 arguments, but got 2.
     expectWarning(mathAdjacentRule, ["$x=b+c$\n\n$x-b=c$"]);
-    // @ts-expect-error [FEI-5003] - TS2554 - Expected 3 arguments, but got 2.
+    // @ts-expect-error - TS2554 - Expected 3 arguments, but got 2.
     expectPass(mathAdjacentRule, ["$x=b+c$\n\nnew paragraph\n\n$x-b=c$"]);
 
-    // @ts-expect-error [FEI-5003] - TS2554 - Expected 3 arguments, but got 2.
+    // @ts-expect-error - TS2554 - Expected 3 arguments, but got 2.
     expectWarning(mathAlignLinebreaksRule, [
         "$\\begin{align}x\\\\y\\end{align}$",
         "$\\begin{align} x \\\\ y \\end{align}$",
         "$\\begin{align}x\\\\\\\\\\\\y\\end{align}$",
         "$\\begin{align}\nx\\\\\n\\\\\\\\\ny\n\\end{align}$",
     ]);
-    // @ts-expect-error [FEI-5003] - TS2554 - Expected 3 arguments, but got 2.
+    // @ts-expect-error - TS2554 - Expected 3 arguments, but got 2.
     expectPass(mathAlignLinebreaksRule, [
         "$\\begin{align}x\\sqrty\\end{align}$",
         "$\\begin{align}x\\\\\\\\y\\end{align}$",
@@ -351,26 +351,26 @@ describe("Individual lint rules tests", () => {
         "$\\begin{align}x \\\\  \\\\ y\\end{align}$",
     ]);
 
-    // @ts-expect-error [FEI-5003] - TS2554 - Expected 3 arguments, but got 2.
+    // @ts-expect-error - TS2554 - Expected 3 arguments, but got 2.
     expectWarning(mathAlignExtraBreakRule, [
         "$\\begin{align}x \\\\\\\\ y \\\\ \\end{align}$",
         "$\\begin{align}x \\\\\\\\ y \\\\\\\\ \\end{align}$",
     ]);
-    // @ts-expect-error [FEI-5003] - TS2554 - Expected 3 arguments, but got 2.
+    // @ts-expect-error - TS2554 - Expected 3 arguments, but got 2.
     expectPass(mathAlignExtraBreakRule, [
         "$\\begin{align} x \\\\\\\\ y  \\end{align}$",
     ]);
 
-    // @ts-expect-error [FEI-5003] - TS2554 - Expected 3 arguments, but got 2.
+    // @ts-expect-error - TS2554 - Expected 3 arguments, but got 2.
     expectWarning(mathNestedRule, [
         "$\\text{4$x$}$",
         "inline $\\text{4$x$}$ math",
         "$\\text{$$}$",
     ]);
-    // @ts-expect-error [FEI-5003] - TS2554 - Expected 3 arguments, but got 2.
+    // @ts-expect-error - TS2554 - Expected 3 arguments, but got 2.
     expectPass(mathNestedRule, ["$\\text{4}x$", "inline $\\text{4}x$ math"]);
 
-    // @ts-expect-error [FEI-5003] - TS2554 - Expected 3 arguments, but got 2.
+    // @ts-expect-error - TS2554 - Expected 3 arguments, but got 2.
     expectWarning(mathFontSizeRule, [
         "$\\tiny{x}$",
         "inline $\\Tiny{x}$ math",
@@ -383,10 +383,10 @@ describe("Individual lint rules tests", () => {
         "$\\normalsize{x}$",
         "$\\scriptsize{x}$",
     ]);
-    // @ts-expect-error [FEI-5003] - TS2554 - Expected 3 arguments, but got 2.
+    // @ts-expect-error - TS2554 - Expected 3 arguments, but got 2.
     expectPass(mathFontSizeRule, ["$\\sqrt{x}$", "inline $\\sqrt{x}$ math"]);
 
-    // @ts-expect-error [FEI-5003] - TS2554 - Expected 3 arguments, but got 2.
+    // @ts-expect-error - TS2554 - Expected 3 arguments, but got 2.
     expectWarning(profanityRule, [
         "Shit",
         "taking a piss",
@@ -395,17 +395,17 @@ describe("Individual lint rules tests", () => {
         "cocksucker",
         "motherfucker",
     ]);
-    // @ts-expect-error [FEI-5003] - TS2554 - Expected 3 arguments, but got 2.
+    // @ts-expect-error - TS2554 - Expected 3 arguments, but got 2.
     expectPass(profanityRule, ["spit", "miss", "duck"]);
 
-    // @ts-expect-error [FEI-5003] - TS2554 - Expected 3 arguments, but got 2.
+    // @ts-expect-error - TS2554 - Expected 3 arguments, but got 2.
     expectWarning(mathWithoutDollarsRule, [
         "One half: \\frac{1}{2}!",
         "\\Large{BIG}!",
         "This looks like someone's ear: {",
         "Here's the other ear: }. Weird!",
     ]);
-    // @ts-expect-error [FEI-5003] - TS2554 - Expected 3 arguments, but got 2.
+    // @ts-expect-error - TS2554 - Expected 3 arguments, but got 2.
     expectPass(mathWithoutDollarsRule, [
         "One half: $\\frac{1}{2}$",
         "$\\Large{BIG}$!",
@@ -417,14 +417,14 @@ describe("Individual lint rules tests", () => {
         "\n    \\frac{1}{2}\n    {\n    }\n",
     ]);
 
-    // @ts-expect-error [FEI-5003] - TS2554 - Expected 3 arguments, but got 2.
+    // @ts-expect-error - TS2554 - Expected 3 arguments, but got 2.
     expectWarning(unbalancedCodeDelimitersRule, [
         "`code``",
         "``code```",
         "```code\n",
         "~~~\ncode\n~~",
     ]);
-    // @ts-expect-error [FEI-5003] - TS2554 - Expected 3 arguments, but got 2.
+    // @ts-expect-error - TS2554 - Expected 3 arguments, but got 2.
     expectPass(unbalancedCodeDelimitersRule, [
         "`code`",
         "``code``",
@@ -436,7 +436,7 @@ describe("Individual lint rules tests", () => {
         "$`~$",
     ]);
 
-    // @ts-expect-error [FEI-5003] - TS2554 - Expected 3 arguments, but got 2.
+    // @ts-expect-error - TS2554 - Expected 3 arguments, but got 2.
     expectWarning(imageSpacesAroundUrlsRule, [
         "![alternative]( http://example.com/image.jpg )",
         "![alternative]( http://example.com/image.jpg)",
@@ -446,7 +446,7 @@ describe("Individual lint rules tests", () => {
         "![alternative](\nhttp://example.com/image.jpg)",
         "![alternative](http://example.com/image.jpg\n)",
     ]);
-    // @ts-expect-error [FEI-5003] - TS2554 - Expected 3 arguments, but got 2.
+    // @ts-expect-error - TS2554 - Expected 3 arguments, but got 2.
     expectPass(imageSpacesAroundUrlsRule, [
         "![alternative](http://example.com/image.jpg)",
         "![alternative](image.jpg)",
@@ -520,26 +520,26 @@ describe("Individual lint rules tests", () => {
         },
     });
 
-    // @ts-expect-error [FEI-5003] - TS2554 - Expected 3 arguments, but got 2.
+    // @ts-expect-error - TS2554 - Expected 3 arguments, but got 2.
     expectWarning(doubleSpacingAfterTerminalRule, [
         "Good times.  Great oldies.",
         "End of the line!  ",
         "You?  Me!",
     ]);
-    // @ts-expect-error [FEI-5003] - TS2554 - Expected 3 arguments, but got 2.
+    // @ts-expect-error - TS2554 - Expected 3 arguments, but got 2.
     expectPass(doubleSpacingAfterTerminalRule, [
         "This is okay.",
         "This is definitely okay. Yeah.",
         "$a == 3.  125$",
     ]);
 
-    // @ts-expect-error [FEI-5003] - TS2554 - Expected 3 arguments, but got 2.
+    // @ts-expect-error - TS2554 - Expected 3 arguments, but got 2.
     expectWarning(extraContentSpacingRule, [
         "There's extra spaces here.     ",
         "There's extra spaces here    ",
         "  ",
     ]);
-    // @ts-expect-error [FEI-5003] - TS2554 - Expected 3 arguments, but got 2.
+    // @ts-expect-error - TS2554 - Expected 3 arguments, but got 2.
     expectPass(extraContentSpacingRule, [
         "This is okay.",
         "This is definitely okay. Yeah.",

package/src/__tests__/tree-transformer.test.ts

@@ -56,7 +56,7 @@ describe("PerseusLinter tree transformer", () => {
     function getTraversalOrder(tree: any) {
         const order: Array<any> = [];
         new TreeTransformer(tree).traverse((n, state) => {
-            // @ts-expect-error [FEI-5003] - TS2339 - Property 'id' does not exist on type 'TreeNode'.
+            // @ts-expect-error - TS2339 - Property 'id' does not exist on type 'TreeNode'.
             order.push(n.id);
         });
         return order;
@@ -250,7 +250,7 @@ describe("PerseusLinter tree transformer", () => {
                 new TreeTransformer(copy).traverse((n: any, state) => {
                     if (n.id === id) {
                         state.replace({
-                            // @ts-expect-error [FEI-5003] - TS2345 - Argument of type '{ id: number; type: string; }' is not assignable to parameter of type 'TreeNode'.
+                            // @ts-expect-error - TS2345 - Argument of type '{ id: number; type: string; }' is not assignable to parameter of type 'TreeNode'.
                             id: 99,
                             type: "replacement",
                         });
@@ -288,7 +288,7 @@ describe("PerseusLinter tree transformer", () => {
                         // Ensure that we don't traverse the node more than once
                         expect(++count).toEqual(1);
                         state.replace({
-                            // @ts-expect-error [FEI-5003] - TS2345 - Argument of type '{ id: number; type: string; content: any; }' is not assignable to parameter of type 'TreeNode'.
+                            // @ts-expect-error - TS2345 - Argument of type '{ id: number; type: string; content: any; }' is not assignable to parameter of type 'TreeNode'.
                             id: 99,
                             type: "reparent",
                             content: n,
@@ -323,7 +323,7 @@ describe("PerseusLinter tree transformer", () => {
                 // Replace the node with two new ones
                 new TreeTransformer(copy).traverse((n: any, state) => {
                     if (n.id === id) {
-                        // @ts-expect-error [FEI-5003] - TS2345 - Argument of type '({ id: number; type: string; } | { id: number; type: string; content: { id: number; type: string; }; })[]' is not assignable to parameter of type 'TreeNode'.
+                        // @ts-expect-error - TS2345 - Argument of type '({ id: number; type: string; } | { id: number; type: string; content: { id: number; type: string; }; })[]' is not assignable to parameter of type 'TreeNode'.
                         state.replace([
                             {
                                 id: 99,
@@ -372,7 +372,7 @@ describe("PerseusLinter tree transformer", () => {
                     if (n.id === id) {
                         state.replace(
                             {
-                                // @ts-expect-error [FEI-5003] - TS2345 - Argument of type '{ id: number; type: string; }' is not assignable to parameter of type 'TreeNode'.
+                                // @ts-expect-error - TS2345 - Argument of type '{ id: number; type: string; }' is not assignable to parameter of type 'TreeNode'.
                                 id: 99,
                                 type: "replacement",
                             },

package/src/index.ts

@@ -50,7 +50,7 @@ export function runLinter(
         if (TreeTransformer.isTextNode(node)) {
             let next = state.nextSibling();
             while (TreeTransformer.isTextNode(next)) {
-                // @ts-expect-error [FEI-5003] - TS2339 - Property 'content' does not exist on type 'TreeNode'. | TS2533 - Object is possibly 'null' or 'undefined'. | TS2339 - Property 'content' does not exist on type 'TreeNode'.
+                // @ts-expect-error - TS2339 - Property 'content' does not exist on type 'TreeNode'. | TS2533 - Object is possibly 'null' or 'undefined'. | TS2339 - Property 'content' does not exist on type 'TreeNode'.
                 node.content += next.content;
                 state.removeNextSibling();
                 next = state.nextSibling();
@@ -157,7 +157,7 @@ export function runLinter(
         // this node, then we need to save the warnings for display
         // on the table itself
         if (insideTable && nodeWarnings.length) {
-            // @ts-expect-error [FEI-5003] - TS2345 - Argument of type 'any' is not assignable to parameter of type 'never'.
+            // @ts-expect-error - TS2345 - Argument of type 'any' is not assignable to parameter of type 'never'.
             tableWarnings.push(...nodeWarnings);
         }
 
@@ -185,7 +185,7 @@ export function runLinter(
                 // node under a new lint node and put the warnings there.
                 state.replace({
                     type: "lint",
-                    // @ts-expect-error [FEI-5003] - TS2345 - Argument of type '{ type: string; content: TreeNode; message: string; ruleName: any; blockHighlight: any; insideTable: boolean; severity: any; }' is not assignable to parameter of type 'TreeNode'.
+                    // @ts-expect-error - TS2345 - Argument of type '{ type: string; content: TreeNode; message: string; ruleName: any; blockHighlight: any; insideTable: boolean; severity: any; }' is not assignable to parameter of type 'TreeNode'.
                     content: node,
                     message: nodeWarnings.map((w) => w.message).join("\n\n"),
                     ruleName: nodeWarnings[0].rule,
@@ -219,7 +219,7 @@ export function runLinter(
                 // single line, so keeping them combined in that case might
                 // be the best thing, anyway.
                 //
-                // @ts-expect-error [FEI-5003] - TS2339 - Property 'content' does not exist on type 'TreeNode'.
+                // @ts-expect-error - TS2339 - Property 'content' does not exist on type 'TreeNode'.
                 const content = node.content; // Text nodes have content
                 const warning = nodeWarnings[0]; // There is only one warning.
                 // These are the lint boundaries within the content

package/src/rule.ts

@@ -129,7 +129,7 @@ import type {TraversalState, TreeNode} from "./tree-transformer";
 
 // This represents the type returned by String.match(). It is an
 // array of strings, but also has index:number and input:string properties.
-// Flow doesn't handle it well, so we punt and just use any.
+// TypeScript doesn't handle it well, so we punt and just use any.
 export type PatternMatchType = any;
 
 // This is the return type of the check() method of a Rule object
@@ -382,7 +382,7 @@ ${e.stack}`,
             const lastSlash = pattern.lastIndexOf("/");
             const expression = pattern.substring(1, lastSlash);
             const flags = pattern.substring(lastSlash + 1);
-            // @ts-expect-error [FEI-5003] - TS2713 - Cannot access 'RegExp.flags' because 'RegExp' is a type, but not a namespace. Did you mean to retrieve the type of the property 'flags' in 'RegExp' with 'RegExp["flags"]'?
+            // @ts-expect-error - TS2713 - Cannot access 'RegExp.flags' because 'RegExp' is a type, but not a namespace. Did you mean to retrieve the type of the property 'flags' in 'RegExp' with 'RegExp["flags"]'?
             return new RegExp(expression, flags as RegExp.flags);
         }
         return new RegExp(pattern);

package/src/tree-transformer.ts

@@ -135,11 +135,9 @@ export default class TreeTransformer {
             // Record the node's text content if it has any.
             // Usually this is for nodes with a type property of "text",
             // but other nodes types like "math" may also have content.
-            // TODO(mdr): We found a new Flow error when upgrading:
-            //     "node.content (property `content` is missing in `TreeNode` [1].)"
-            // @ts-expect-error [FEI-5003] - TS2339 - Property 'content' does not exist on type 'TreeNode'.
+            // @ts-expect-error - TS2339 - Property 'content' does not exist on type 'TreeNode'.
             if (typeof node.content === "string") {
-                // @ts-expect-error [FEI-5003] - TS2339 - Property 'content' does not exist on type 'TreeNode'.
+                // @ts-expect-error - TS2339 - Property 'content' does not exist on type 'TreeNode'.
                 content = node.content;
             }
 
@@ -204,7 +202,7 @@ export default class TreeTransformer {
             while (index < nodes.length) {
                 state._indexes.push(index);
                 content += this._traverse(nodes[index], state, f);
-                // Casting to convince Flow that this is a number
+                // Casting to convince TypeScript that this is a number
                 index = (state._indexes.pop() as number) + 1;
             }
 
@@ -231,7 +229,7 @@ export default class TreeTransformer {
  * for that traversal, and the instance is passed to the traversal callback
  * function for each node that is traversed. This class is not intended to be
  * instantiated directly, but is exported so that its type can be used for
- * Flow annotaions.
+ * type annotaions.
  **/
 export class TraversalState {
     // The root node of the tree being traversed
@@ -241,8 +239,8 @@ export class TraversalState {
     // below instead of using these properties directly. Note that the
     // _containers and _indexes stacks can have two different types of
     // elements, depending on whether we just recursed on an array or on a
-    // node. This is hard for Flow to deal with, so you'll see a number of
-    // Flow casts through the any type when working with these two properties.
+    // node. This is hard for TypeScript to deal with, so you'll see a number of
+    // type casts through the any type when working with these two properties.
     _currentNode: TreeNode | null | undefined;
     _containers: Stack<TreeNode | Array<TreeNode>>;
     _indexes: Stack<string | number>;
@@ -383,7 +381,7 @@ export class TraversalState {
 
         // The top of the container stack is either an array or an object
         // and the top of the indexes stack is a corresponding array index
-        // or object property. This is hard for Flow, so we have to do some
+        // or object property. This is hard for TypeScript, so we have to do some
         // unsafe casting and be careful when we use which cast version
         if (Array.isArray(parent)) {
             const index = this._indexes.top() as number;
@@ -440,7 +438,7 @@ export class TraversalState {
         this._currentNode = this.previousSibling();
         // Since we know that we have a previous sibling, we know that
         // the value on top of the stack is a number, but we have to do
-        // this unsafe cast because Flow doesn't know that.
+        // this unsafe cast because TypeScript doesn't know that.
         const index = this._indexes.pop() as number;
         this._indexes.push(index - 1);
     }
@@ -479,8 +477,6 @@ export class TraversalState {
         //
         while (
             this._containers.size() &&
-            // This is safe, but easier to just disable flow than do casts
-            // $FlowFixMe[incompatible-use]
             this._containers.top()[this._indexes.top()] !== this._currentNode
         ) {
             this._containers.pop();
@@ -540,7 +536,7 @@ class Stack<T> {
 
     /** Pop a value off of the stack. */
     pop(): T {
-        // @ts-expect-error [FEI-5003] - TS2322 - Type 'T | undefined' is not assignable to type 'T'.
+        // @ts-expect-error - TS2322 - Type 'T | undefined' is not assignable to type 'T'.
         return this.stack.pop();
     }
 

package/tsconfig-build.json

@@ -0,0 +1,11 @@
+{
+    "extends": "../tsconfig-shared.json",
+    "compilerOptions": {
+        "outDir": "./dist",
+        "rootDir": "src",
+    },
+    "references": [
+        {"path": "../perseus-error/tsconfig-build.json"},
+        {"path": "../pure-markdown/tsconfig-build.json"},
+    ]
+}