@khanacademy/perseus-linter 0.2.4 → 0.3.0

package/src/__tests__/{rules_test.js → rules.test.ts}

@@ -1,41 +1,42 @@
-// @flow
 import * as PureMarkdown from "@khanacademy/pure-markdown";
 
-import absoluteUrlRule from "../rules/absolute-url.js";
-import blockquotedMathRule from "../rules/blockquoted-math.js";
-import blockquotedWidgetRule from "../rules/blockquoted-widget.js";
-import doubleSpacingAfterTerminalRule from "../rules/double-spacing-after-terminal.js";
-import extraContentSpacingRule from "../rules/extra-content-spacing.js";
-import headingLevel1Rule from "../rules/heading-level-1.js";
-import headingLevelSkipRule from "../rules/heading-level-skip.js";
-import headingSentenceCaseRule from "../rules/heading-sentence-case.js";
-import headingTitleCaseRule from "../rules/heading-title-case.js";
-import imageAltTextRule from "../rules/image-alt-text.js";
-import imageInTableRule from "../rules/image-in-table.js";
-import imageSpacesAroundUrlsRule from "../rules/image-spaces-around-urls.js";
-import imageWidgetRule from "../rules/image-widget.js";
-import linkClickHereRule from "../rules/link-click-here.js";
-import longParagraphRule from "../rules/long-paragraph.js";
-import mathAdjacentRule from "../rules/math-adjacent.js";
-import mathAlignExtraBreakRule from "../rules/math-align-extra-break.js";
-import mathAlignLinebreaksRule from "../rules/math-align-linebreaks.js";
-import mathEmptyRule from "../rules/math-empty.js";
-import mathFontSizeRule from "../rules/math-font-size.js";
-import mathFracRule from "../rules/math-frac.js";
-import mathNestedRule from "../rules/math-nested.js";
-import mathStartsWithSpaceRule from "../rules/math-starts-with-space.js";
-import mathTextEmptyRule from "../rules/math-text-empty.js";
-import mathWithoutDollarsRule from "../rules/math-without-dollars.js";
-import nestedListsRule from "../rules/nested-lists.js";
-import profanityRule from "../rules/profanity.js";
-import tableMissingCellsRule from "../rules/table-missing-cells.js";
-import unbalancedCodeDelimitersRule from "../rules/unbalanced-code-delimiters.js";
-import unescapedDollarRule from "../rules/unescaped-dollar.js";
-import widgetInTableRule from "../rules/widget-in-table.js";
-import TreeTransformer from "../tree-transformer.js";
+import absoluteUrlRule from "../rules/absolute-url";
+import blockquotedMathRule from "../rules/blockquoted-math";
+import blockquotedWidgetRule from "../rules/blockquoted-widget";
+import doubleSpacingAfterTerminalRule from "../rules/double-spacing-after-terminal";
+import extraContentSpacingRule from "../rules/extra-content-spacing";
+import headingLevel1Rule from "../rules/heading-level-1";
+import headingLevelSkipRule from "../rules/heading-level-skip";
+import headingSentenceCaseRule from "../rules/heading-sentence-case";
+import headingTitleCaseRule from "../rules/heading-title-case";
+import imageAltTextRule from "../rules/image-alt-text";
+import imageInTableRule from "../rules/image-in-table";
+import imageSpacesAroundUrlsRule from "../rules/image-spaces-around-urls";
+import imageWidgetRule from "../rules/image-widget";
+import linkClickHereRule from "../rules/link-click-here";
+import longParagraphRule from "../rules/long-paragraph";
+import mathAdjacentRule from "../rules/math-adjacent";
+import mathAlignExtraBreakRule from "../rules/math-align-extra-break";
+import mathAlignLinebreaksRule from "../rules/math-align-linebreaks";
+import mathEmptyRule from "../rules/math-empty";
+import mathFontSizeRule from "../rules/math-font-size";
+import mathFracRule from "../rules/math-frac";
+import mathNestedRule from "../rules/math-nested";
+import mathStartsWithSpaceRule from "../rules/math-starts-with-space";
+import mathTextEmptyRule from "../rules/math-text-empty";
+import mathWithoutDollarsRule from "../rules/math-without-dollars";
+import nestedListsRule from "../rules/nested-lists";
+import profanityRule from "../rules/profanity";
+import tableMissingCellsRule from "../rules/table-missing-cells";
+import unbalancedCodeDelimitersRule from "../rules/unbalanced-code-delimiters";
+import unescapedDollarRule from "../rules/unescaped-dollar";
+import widgetInTableRule from "../rules/widget-in-table";
+import TreeTransformer from "../tree-transformer";
+
+type Rule = any;
 
 describe("Individual lint rules tests", () => {
-    function testRule(rule, markdown, context) {
+    function testRule(rule: Rule, markdown: string, context) {
         const tree = PureMarkdown.parse(markdown);
         const tt = new TreeTransformer(tree);
         const warnings = [];
@@ -46,8 +47,7 @@ describe("Individual lint rules tests", () => {
             if (TreeTransformer.isTextNode(node)) {
                 let next = state.nextSibling();
                 while (TreeTransformer.isTextNode(next)) {
-                    // $FlowFixMe[prop-missing]
-                    // $FlowFixMe[incompatible-use]
+                    // @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'.
                     node.content += next.content;
                     state.removeNextSibling();
                     next = state.nextSibling();
@@ -56,7 +56,6 @@ describe("Individual lint rules tests", () => {
         });
 
         if (context) {
-            // $FlowFixMe[prop-missing]
             context.content = markdown;
         } else {
             context = {
@@ -67,6 +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'.
                 warnings.push(check);
             }
         });
@@ -74,7 +74,7 @@ describe("Individual lint rules tests", () => {
         return warnings.length === 0 ? null : warnings;
     }
 
-    function expectWarning(rule, strings, context) {
+    function expectWarning(rule, strings: string | Array<string>, context) {
         if (typeof strings === "string") {
             strings = [strings];
         }
@@ -86,7 +86,7 @@ describe("Individual lint rules tests", () => {
         });
     }
 
-    function expectPass(rule, strings, context) {
+    function expectPass(rule, strings: string | Array<string>, context) {
         if (typeof strings === "string") {
             strings = [strings];
         }
@@ -102,49 +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.
     expectWarning(longParagraphRule, sentence + sentence);
+    // @ts-expect-error [FEI-5003] - 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.
     expectWarning(headingLevel1Rule, "# Level 1 heading");
+    // @ts-expect-error [FEI-5003] - 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.
     expectWarning(headingLevelSkipRule, "## heading 1\n\n#### heading 2");
+    // @ts-expect-error [FEI-5003] - 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.
     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.
     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.
     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.
     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.
     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.
     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.
     expectWarning(imageAltTextRule, [
         "![](http://google.com/)",
         '![](http://google.com/ "title")',
@@ -154,34 +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.
     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.
     expectWarning(blockquotedMathRule, ["> $1$", "Quote:\n\n> $x$\n\n"]);
+    // @ts-expect-error [FEI-5003] - 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.
     expectWarning(blockquotedWidgetRule, ["> [[☃ passage 1]]"]);
+    // @ts-expect-error [FEI-5003] - 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.
     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.
     expectPass(linkClickHereRule, [
         "[click to activate this link here](http://google.com)",
     ]);
 
+    // @ts-expect-error [FEI-5003] - TS2554 - Expected 3 arguments, but got 2.
     expectWarning(absoluteUrlRule, [
         // Warn about absolute khanacademy.org urls
         "[target](http://khanacademy.org/about)",
@@ -198,6 +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.
     expectPass(absoluteUrlRule, [
         "[target](/about)", // relative URLs okay
         "[target](https://kasandbox.org/path)",
@@ -212,20 +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.
     expectWarning(imageInTableRule, [
         "|col1|col2|\n|----|----|\n|![alt-text](/link.gif)|cell2|",
     ]);
+    // @ts-expect-error [FEI-5003] - 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.
     expectWarning(widgetInTableRule, [
         "|col1|col2|\n|----|----|\n|[[☃ passage 1]]|cell2|",
     ]);
+    // @ts-expect-error [FEI-5003] - 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.
     expectWarning(tableMissingCellsRule, [
         "|col1|col2|col3|\n|----|----|----|\n|col1|col2|col3|\n|cell1|cell2|",
         "|col1|col2|col3|\n|----|----|----|\n|col1|col2|\n|cell1|cell2|",
@@ -233,15 +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.
     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.
     expectWarning(unescapedDollarRule, ["It costs $10", "It costs $$10$"]);
 
+    // @ts-expect-error [FEI-5003] - 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.
     expectWarning(mathStartsWithSpaceRule, [
         "foo$~ x$bar",
         "$\\qquad x$",
@@ -254,6 +285,7 @@ describe("Individual lint rules tests", () => {
         "$\\enspace x$",
         "$\\phantom{xyz} x$",
     ]);
+    // @ts-expect-error [FEI-5003] - TS2554 - Expected 3 arguments, but got 2.
     expectPass(mathStartsWithSpaceRule, [
         "$a~ x$",
         "$a\\qquad x$",
@@ -267,41 +299,51 @@ describe("Individual lint rules tests", () => {
         "$a\\phantom{xyz} x$",
     ]);
 
+    // @ts-expect-error [FEI-5003] - 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.
     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.
     expectWarning(mathFracRule, ["$\\frac 12$", "$\\frac{1}{2}$"]);
+    // @ts-expect-error [FEI-5003] - 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.
     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.
     expectPass(mathTextEmptyRule, ["$x\\text{z}y$"]);
 
+    // @ts-expect-error [FEI-5003] - 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.
     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.
     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.
     expectPass(mathAlignLinebreaksRule, [
         "$\\begin{align}x\\sqrty\\end{align}$",
         "$\\begin{align}x\\\\\\\\y\\end{align}$",
@@ -309,21 +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.
     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.
     expectPass(mathAlignExtraBreakRule, [
         "$\\begin{align} x \\\\\\\\ y  \\end{align}$",
     ]);
 
+    // @ts-expect-error [FEI-5003] - 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.
     expectPass(mathNestedRule, ["$\\text{4}x$", "inline $\\text{4}x$ math"]);
 
+    // @ts-expect-error [FEI-5003] - TS2554 - Expected 3 arguments, but got 2.
     expectWarning(mathFontSizeRule, [
         "$\\tiny{x}$",
         "inline $\\Tiny{x}$ math",
@@ -336,8 +383,10 @@ describe("Individual lint rules tests", () => {
         "$\\normalsize{x}$",
         "$\\scriptsize{x}$",
     ]);
+    // @ts-expect-error [FEI-5003] - 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.
     expectWarning(profanityRule, [
         "Shit",
         "taking a piss",
@@ -346,14 +395,17 @@ describe("Individual lint rules tests", () => {
         "cocksucker",
         "motherfucker",
     ]);
+    // @ts-expect-error [FEI-5003] - TS2554 - Expected 3 arguments, but got 2.
     expectPass(profanityRule, ["spit", "miss", "duck"]);
 
+    // @ts-expect-error [FEI-5003] - 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.
     expectPass(mathWithoutDollarsRule, [
         "One half: $\\frac{1}{2}$",
         "$\\Large{BIG}$!",
@@ -365,12 +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.
     expectWarning(unbalancedCodeDelimitersRule, [
         "`code``",
         "``code```",
         "```code\n",
         "~~~\ncode\n~~",
     ]);
+    // @ts-expect-error [FEI-5003] - TS2554 - Expected 3 arguments, but got 2.
     expectPass(unbalancedCodeDelimitersRule, [
         "`code`",
         "``code``",
@@ -382,6 +436,7 @@ describe("Individual lint rules tests", () => {
         "$`~$",
     ]);
 
+    // @ts-expect-error [FEI-5003] - TS2554 - Expected 3 arguments, but got 2.
     expectWarning(imageSpacesAroundUrlsRule, [
         "![alternative]( http://example.com/image.jpg )",
         "![alternative]( http://example.com/image.jpg)",
@@ -391,6 +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.
     expectPass(imageSpacesAroundUrlsRule, [
         "![alternative](http://example.com/image.jpg)",
         "![alternative](image.jpg)",
@@ -464,22 +520,26 @@ describe("Individual lint rules tests", () => {
         },
     });
 
+    // @ts-expect-error [FEI-5003] - 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.
     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.
     expectWarning(extraContentSpacingRule, [
         "There's extra spaces here.     ",
         "There's extra spaces here    ",
         "  ",
     ]);
+    // @ts-expect-error [FEI-5003] - TS2554 - Expected 3 arguments, but got 2.
     expectPass(extraContentSpacingRule, [
         "This is okay.",
         "This is definitely okay. Yeah.",

package/src/__tests__/{selector-parser_test.js → selector-parser.test.ts}

@@ -1,5 +1,4 @@
-// @flow
-import Selector from "../selector.js";
+import Selector from "../selector";
 
 describe("PerseusLinter selector parser", () => {
     const validExpressions = [

package/src/__tests__/{tree-transformer_test.js → tree-transformer.test.ts}

@@ -1,5 +1,6 @@
-// @flow
-import TreeTransformer from "../tree-transformer.js";
+import TreeTransformer from "../tree-transformer";
+
+import type {TraversalState} from "../tree-transformer";
 
 describe("PerseusLinter tree transformer", () => {
     function clone(o) {
@@ -30,7 +31,7 @@ describe("PerseusLinter tree transformer", () => {
                 ],
             },
         ],
-    };
+    } as const;
 
     // These are three variants on the same tree where we use arrays
     // instead of single nodes. The tests below will be run over all
@@ -50,25 +51,25 @@ describe("PerseusLinter tree transformer", () => {
 
     // The first test will fill in this array mapping numbers to nodes
     // Then subsequent tests can use it
-    const nodes = [];
+    const nodes: Array<any> = [];
 
-    function getTraversalOrder(tree) {
-        const order = [];
+    function getTraversalOrder(tree: any) {
+        const order: Array<any> = [];
         new TreeTransformer(tree).traverse((n, state) => {
-            // $FlowFixMe[prop-missing]
+            // @ts-expect-error [FEI-5003] - TS2339 - Property 'id' does not exist on type 'TreeNode'.
             order.push(n.id);
         });
         return order;
     }
 
-    trees.forEach((tree: $FlowFixMe, treenum: number) => {
+    trees.forEach((tree: any, treenum: number) => {
         it(
             "does post-order traversal of each node in the tree " + treenum,
             () => {
                 const tt = new TreeTransformer(tree);
-                const ids = [];
+                const ids: Array<any> = [];
 
-                tt.traverse((n: $FlowFixMe) => {
+                tt.traverse((n: any) => {
                     nodes[n.id] = n; // Remember the nodes by id for later tests
                     ids.push(n.id);
                 });
@@ -86,7 +87,7 @@ describe("PerseusLinter tree transformer", () => {
         });
 
         it("correctly gets the siblings for each node " + treenum, () => {
-            new TreeTransformer(tree).traverse((n: $FlowFixMe, state) => {
+            new TreeTransformer(tree).traverse((n: any, state) => {
                 const previd = previousNodeIds[n.id];
                 expect(state.hasPreviousSibling()).toEqual(previd >= 0);
                 expect(state.previousSibling()).toEqual(
@@ -122,7 +123,7 @@ describe("PerseusLinter tree transformer", () => {
                 ["root", "list"],
             ];
 
-            new TreeTransformer(tree).traverse((n: $FlowFixMe, state) => {
+            new TreeTransformer(tree).traverse((n: any, state) => {
                 expect(state.hasParent()).toEqual(
                     ancestorsById[n.id].length > 0,
                 );
@@ -151,11 +152,9 @@ describe("PerseusLinter tree transformer", () => {
                 "C",
             ];
 
-            new TreeTransformer(tree).traverse(
-                (n: $FlowFixMe, state, content) => {
-                    expect(content).toEqual(textContentForNode[n.id]);
-                },
-            );
+            new TreeTransformer(tree).traverse((n: any, state, content) => {
+                expect(content).toEqual(textContentForNode[n.id]);
+            });
         });
 
         it("can remove the next sibling " + treenum, () => {
@@ -177,16 +176,14 @@ describe("PerseusLinter tree transformer", () => {
                 const copy = clone(tree);
 
                 // Remove the next sibling of the node with this id
-                new TreeTransformer(copy).traverse(
-                    (n: $FlowFixMe, state: $FlowFixMe) => {
-                        if (n.id === id) {
-                            state.removeNextSibling();
-                        }
-
-                        // Ensure that we don't iterate the removed sibling
-                        expect(n.id).not.toEqual(nextNodeIds[id]);
-                    },
-                );
+                new TreeTransformer(copy).traverse((n: any, state: any) => {
+                    if (n.id === id) {
+                        state.removeNextSibling();
+                    }
+
+                    // Ensure that we don't iterate the removed sibling
+                    expect(n.id).not.toEqual(nextNodeIds[id]);
+                });
 
                 // And then get the traversal order of the resulting tree
                 const traversal = getTraversalOrder(copy);
@@ -222,13 +219,11 @@ describe("PerseusLinter tree transformer", () => {
                 // Make a copy of the tree
                 const copy = clone(tree);
                 // Remove this node from it
-                new TreeTransformer(copy).traverse(
-                    (n: $FlowFixMe, state: $FlowFixMe) => {
-                        if (n.id === id) {
-                            state.replace();
-                        }
-                    },
-                );
+                new TreeTransformer(copy).traverse((n: any, state: any) => {
+                    if (n.id === id) {
+                        state.replace();
+                    }
+                });
 
                 // Traverse what remains and see if we get what is expected
                 expect(getTraversalOrder(copy)).toEqual(expectedTraversals[id]);
@@ -252,9 +247,10 @@ describe("PerseusLinter tree transformer", () => {
                 // Make a copy of the tree
                 const copy = clone(tree);
                 // Replace the node with a different one
-                new TreeTransformer(copy).traverse((n: $FlowFixMe, state) => {
+                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'.
                             id: 99,
                             type: "replacement",
                         });
@@ -287,11 +283,12 @@ describe("PerseusLinter tree transformer", () => {
                 const copy = clone(tree);
                 let count = 0;
                 // Replace the node with a different one
-                new TreeTransformer(copy).traverse((n: $FlowFixMe, state) => {
+                new TreeTransformer(copy).traverse((n: any, state) => {
                     if (n.id === id) {
                         // 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'.
                             id: 99,
                             type: "reparent",
                             content: n,
@@ -324,9 +321,9 @@ describe("PerseusLinter tree transformer", () => {
                 // Make a copy of the tree
                 const copy = clone(tree);
                 // Replace the node with two new ones
-                new TreeTransformer(copy).traverse((n: $FlowFixMe, state) => {
+                new TreeTransformer(copy).traverse((n: any, state) => {
                     if (n.id === id) {
-                        // $FlowFixMe[incompatible-call]
+                        // @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'.
                         state.replace([
                             {
                                 id: 99,
@@ -371,10 +368,11 @@ describe("PerseusLinter tree transformer", () => {
                 // Make a copy of the tree
                 const copy = clone(tree);
                 // Replace the node with two new ones
-                new TreeTransformer(copy).traverse((n: $FlowFixMe, state) => {
+                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'.
                                 id: 99,
                                 type: "replacement",
                             },
@@ -405,10 +403,10 @@ describe("PerseusLinter tree transformer", () => {
             // for each node. Then check that goToParent() and
             // goToPreviousSibling() on each saved state object
             // modify the states in apprpriate ways.
-            const states = [];
+            const states: Array<TraversalState> = [];
 
             const tt = new TreeTransformer(tree);
-            tt.traverse((n: $FlowFixMe, state, content) => {
+            tt.traverse((n: any, state, content) => {
                 states[n.id] = state.clone();
                 // Verify that a clone() and equal() work as expected
                 expect(state.equals(states[n.id])).toBeTruthy();