@khanacademy/perseus 7.2.0 → 8.0.0

package/src/widgets/label-image/marker.tsx

@@ -1,3 +1,4 @@
+/* eslint-disable @khanacademy/ts-no-error-suppressions */
 /**
  * Controlled question image marker, rendered as a styled button element.
  *
@@ -132,9 +133,9 @@ export default class Marker extends React.Component<Props, State> {
                 tabIndex={isDisabled ? -1 : 0}
                 onFocus={() => this.handleFocus()}
                 onBlur={() => this.handleBlur()}
-                // @ts-expect-error [FEI-5003] - TS2345 - Argument of type 'MouseEvent<HTMLButtonElement, MouseEvent>' is not assignable to parameter of type 'MouseEvent'.
+                // @ts-expect-error - TS2345 - Argument of type 'MouseEvent<HTMLButtonElement, MouseEvent>' is not assignable to parameter of type 'MouseEvent'.
                 onClick={(e) => this.props.onClick(e)}
-                // @ts-expect-error [FEI-5003] - TS2345 - Argument of type 'KeyboardEvent<HTMLButtonElement>' is not assignable to parameter of type 'KeyboardEvent'.
+                // @ts-expect-error - TS2345 - Argument of type 'KeyboardEvent<HTMLButtonElement>' is not assignable to parameter of type 'KeyboardEvent'.
                 onKeyDown={(e) => this.props.onKeyDown(e)}
             >
                 {this.renderIcon()}

package/src/widgets/label-image.test.ts

@@ -335,7 +335,7 @@ describe("LabelImage", function () {
 
             expect(
                 markers[
-                    // @ts-expect-error [FEI-5003] - TS2345 - Argument of type '({ label: string; answers: readonly []; selected: readonly []; x: 0; y: 0; } | { label: string; y: number; showCorrectness: string; answers: readonly []; selected: readonly []; x: 0; } | { label: string; x: number; showCorrectness: string; answers: readonly []; selected: readonly []; y: 0; })[]' is not assignable to parameter of type 'readonly InteractiveMarkerType[]'.
+                    // @ts-expect-error - TS2345 - Argument of type '({ label: string; answers: readonly []; selected: readonly []; x: 0; y: 0; } | { label: string; y: number; showCorrectness: string; answers: readonly []; selected: readonly []; x: 0; } | { label: string; x: number; showCorrectness: string; answers: readonly []; selected: readonly []; y: 0; })[]' is not assignable to parameter of type 'readonly InteractiveMarkerType[]'.
                     LabelImage.navigateToMarkerIndex({x: 0, y: 0}, markers, 0)
                 ].label,
             ).toEqual("this");

package/src/widgets/label-image.tsx

@@ -99,7 +99,7 @@ class LabelImage extends React.Component<LabelImageProps, LabelImageState> {
         } as const;
 
         if (marker.selected && marker.selected.length > 0) {
-            // @ts-expect-error [FEI-5003] - TS2540 - Cannot assign to 'hasAnswers' because it is a read-only property.
+            // @ts-expect-error - TS2540 - Cannot assign to 'hasAnswers' because it is a read-only property.
             score.hasAnswers = true;
         }
 
@@ -109,14 +109,14 @@ class LabelImage extends React.Component<LabelImageProps, LabelImageState> {
                 marker.selected.length === marker.answers.length
             ) {
                 // All correct answers are selected by the user.
-                // @ts-expect-error [FEI-5003] - TS2540 - Cannot assign to 'isCorrect' because it is a read-only property.
+                // @ts-expect-error - TS2540 - Cannot assign to 'isCorrect' because it is a read-only property.
                 score.isCorrect = marker.selected.every((choice) =>
                     marker.answers.includes(choice),
                 );
             }
         } else if (!marker.selected || marker.selected.length === 0) {
             // Correct as no answers should be selected by the user.
-            // @ts-expect-error [FEI-5003] - TS2540 - Cannot assign to 'isCorrect' because it is a read-only property.
+            // @ts-expect-error - TS2540 - Cannot assign to 'isCorrect' because it is a read-only property.
             score.isCorrect = true;
         }
 
@@ -217,9 +217,9 @@ class LabelImage extends React.Component<LabelImageProps, LabelImageState> {
         for (const side of Object.keys(triangles)) {
             const corners = triangles[side];
 
-            // @ts-expect-error [FEI-5003] - TS2556 - A spread argument must either have a tuple type or be passed to a rest parameter.
+            // @ts-expect-error - TS2556 - A spread argument must either have a tuple type or be passed to a rest parameter.
             if (LabelImage.pointInTriangle(p, ...corners)) {
-                // @ts-expect-error [FEI-5003] - TS2322 - Type 'string' is not assignable to type '"left" | "top" | "center" | "right" | "bottom"'.
+                // @ts-expect-error - TS2322 - Type 'string' is not assignable to type '"left" | "top" | "center" | "right" | "bottom"'.
                 return side;
             }
         }
@@ -385,7 +385,7 @@ class LabelImage extends React.Component<LabelImageProps, LabelImageState> {
 
         // Update Perseus widget state with user selected answers without
         // triggering interaction events for listeners.
-        // @ts-expect-error [FEI-5003] - TS2345 - Argument of type 'null' is not assignable to parameter of type '(() => unknown) | undefined'.
+        // @ts-expect-error - TS2345 - Argument of type 'null' is not assignable to parameter of type '(() => unknown) | undefined'.
         onChange({markers: updatedMarkers}, null, true);
     }
 
@@ -526,7 +526,7 @@ class LabelImage extends React.Component<LabelImageProps, LabelImageState> {
         const marker =
             this._markers[
                 LabelImage.navigateToMarkerIndex(
-                    // @ts-expect-error [FEI-5003] - TS2345 - Argument of type '{ x: number; y: number; } | { x: number; y: number; } | { x: number; y: number; } | { x: number; y: number; }' is not assignable to parameter of type 'Direction'.
+                    // @ts-expect-error - TS2345 - Argument of type '{ x: number; y: number; } | { x: number; y: number; } | { x: number; y: number; } | { x: number; y: number; }' is not assignable to parameter of type 'Direction'.
                     navigateDirection,
                     markers,
                     index,

package/src/widgets/lights-puzzle.tsx

@@ -173,7 +173,7 @@ const flipTilesPattern = (oldCells: any, tileY: any, tileX, pattern: any) => {
 class LightsPuzzle extends React.Component<any> {
     _currPattern: any;
     _nextPattern: any;
-    // @ts-expect-error [FEI-5003] - TS2564 - Property '_patternIndex' has no initializer and is not definitely assigned in the constructor.
+    // @ts-expect-error - TS2564 - Property '_patternIndex' has no initializer and is not definitely assigned in the constructor.
     _patternIndex: number;
 
     static propTypes = {
@@ -237,7 +237,7 @@ class LightsPuzzle extends React.Component<any> {
     }
 
     change: (...args: ReadonlyArray<unknown>) => any = (...args) => {
-        // @ts-expect-error [FEI-5003] - TS2345 - Argument of type 'readonly unknown[]' is not assignable to parameter of type 'any[]'.
+        // @ts-expect-error - TS2345 - Argument of type 'readonly unknown[]' is not assignable to parameter of type 'any[]'.
         return Changeable.change.apply(this, args);
     };
 

package/src/widgets/matcher.tsx

@@ -1,3 +1,4 @@
+/* eslint-disable @khanacademy/ts-no-error-suppressions */
 /* eslint-disable react/forbid-prop-types, react/sort-comp */
 import {
     linterContextProps,
@@ -156,7 +157,7 @@ class Matcher extends React.Component<any, any> {
                         </td>
                         <td className={css(styles.column, styles.columnRight)}>
                             <Sortable
-                                // @ts-expect-error [FEI-5003] - TS2322 - Type 'readonly unknown[]' is not assignable to type 'readonly string[]'.
+                                // @ts-expect-error - TS2322 - Type 'readonly unknown[]' is not assignable to type 'readonly string[]'.
                                 options={right}
                                 layout={Layout.VERTICAL}
                                 padding={this.props.padding}
@@ -203,10 +204,10 @@ class Matcher extends React.Component<any, any> {
 
         return {
             // eslint-disable-next-line react/no-string-refs
-            // @ts-expect-error [FEI-5003] - TS2339 - Property 'getOptions' does not exist on type 'ReactInstance'.
+            // @ts-expect-error - TS2339 - Property 'getOptions' does not exist on type 'ReactInstance'.
             left: this.refs.left.getOptions(),
             // eslint-disable-next-line react/no-string-refs
-            // @ts-expect-error [FEI-5003] - TS2339 - Property 'getOptions' does not exist on type 'ReactInstance'.
+            // @ts-expect-error - TS2339 - Property 'getOptions' does not exist on type 'ReactInstance'.
             right: this.refs.right.getOptions(),
         };
     };
@@ -218,7 +219,7 @@ class Matcher extends React.Component<any, any> {
         index,
     ) => {
         // eslint-disable-next-line react/no-string-refs
-        // @ts-expect-error [FEI-5003] - TS2339 - Property 'moveOptionToIndex' does not exist on type 'ReactInstance'.
+        // @ts-expect-error - TS2339 - Property 'moveOptionToIndex' does not exist on type 'ReactInstance'.
         this.refs.left.moveOptionToIndex(option, index);
     };
 
@@ -229,12 +230,12 @@ class Matcher extends React.Component<any, any> {
         index,
     ) => {
         // eslint-disable-next-line react/no-string-refs
-        // @ts-expect-error [FEI-5003] - TS2339 - Property 'moveOptionToIndex' does not exist on type 'ReactInstance'.
+        // @ts-expect-error - TS2339 - Property 'moveOptionToIndex' does not exist on type 'ReactInstance'.
         this.refs.right.moveOptionToIndex(option, index);
     };
 
     simpleValidate: (arg1: any) => any = (rubric) => {
-        // @ts-expect-error [FEI-5003] - TS2339 - Property 'validate' does not exist on type 'typeof Matcher'.
+        // @ts-expect-error - TS2339 - Property 'validate' does not exist on type 'typeof Matcher'.
         return Matcher.validate(this.getUserInput(), rubric);
     };
 }

package/src/widgets/matrix.tsx

@@ -123,7 +123,7 @@ type State = {
     enterTheMatrix: number;
 };
 class Matrix extends React.Component<Props, State> {
-    // @ts-expect-error [FEI-5003] - TS2564 - Property 'cursorPosition' has no initializer and is not definitely assigned in the constructor.
+    // @ts-expect-error - TS2564 - Property 'cursorPosition' has no initializer and is not definitely assigned in the constructor.
     cursorPosition: [number, number];
 
     static defaultProps: DefaultProps = {
@@ -323,7 +323,7 @@ class Matrix extends React.Component<Props, State> {
                                         } as const;
 
                                         MatrixInput = (
-                                            // @ts-expect-error [FEI-5003] - TS2322 - Type '{ style: CSSProperties; className: "inside" | "outside"; ref: string; value: number | null; disabled: boolean; onFocus: () => void; onBlur: () => void; onKeyDown: (e: any) => void; onChange: (value: any, cb: any) => void; }' is not assignable to type 'Pick<Readonly<Props> & Readonly<{ children?: ReactNode; }>, "children" | "style" | "id" | "className" | "placeholder" | "onFocus" | "onBlur" | "onChange" | "onKeyDown" | "labelText">'.
+                                            // @ts-expect-error - TS2322 - Type '{ style: CSSProperties; className: "inside" | "outside"; ref: string; value: number | null; disabled: boolean; onFocus: () => void; onBlur: () => void; onKeyDown: (e: any) => void; onChange: (value: any, cb: any) => void; }' is not assignable to type 'Pick<Readonly<Props> & Readonly<{ children?: ReactNode; }>, "children" | "style" | "id" | "className" | "placeholder" | "onFocus" | "onBlur" | "onChange" | "onKeyDown" | "labelText">'.
                                             <TextInput {...updatedProps} />
                                         );
                                     }
@@ -387,7 +387,7 @@ class Matrix extends React.Component<Props, State> {
     focusInputPath: (arg1: any) => void = (path) => {
         const inputID = getRefForPath(path);
         // eslint-disable-next-line react/no-string-refs
-        // @ts-expect-error [FEI-5003] - TS2339 - Property 'focus' does not exist on type 'ReactInstance'.
+        // @ts-expect-error - TS2339 - Property 'focus' does not exist on type 'ReactInstance'.
         this.refs[inputID].focus();
     };
 
@@ -398,7 +398,7 @@ class Matrix extends React.Component<Props, State> {
 
         const inputID = getRefForPath(path);
         // eslint-disable-next-line react/no-string-refs
-        // @ts-expect-error [FEI-5003] - TS2339 - Property 'blur' does not exist on type 'ReactInstance'.
+        // @ts-expect-error - TS2339 - Property 'blur' does not exist on type 'ReactInstance'.
         this.refs[inputID].blur();
     };
 
@@ -431,37 +431,37 @@ class Matrix extends React.Component<Props, State> {
 
         // eslint-disable-next-line react/no-string-refs
         const curInput = this.refs[getRefForPath(getInputPath(row, col))];
-        // @ts-expect-error [FEI-5003] - TS2339 - Property 'getStringValue' does not exist on type 'ReactInstance'.
+        // @ts-expect-error - TS2339 - Property 'getStringValue' does not exist on type 'ReactInstance'.
         const curValueString = curInput.getStringValue();
-        // @ts-expect-error [FEI-5003] - TS2339 - Property 'getSelectionStart' does not exist on type 'ReactInstance'.
+        // @ts-expect-error - TS2339 - Property 'getSelectionStart' does not exist on type 'ReactInstance'.
         const cursorStartPosition = curInput.getSelectionStart();
-        // @ts-expect-error [FEI-5003] - TS2339 - Property 'getSelectionEnd' does not exist on type 'ReactInstance'.
+        // @ts-expect-error - TS2339 - Property 'getSelectionEnd' does not exist on type 'ReactInstance'.
         const cursorEndPosition = curInput.getSelectionEnd();
 
         let nextPath = null;
         if (e.key === "ArrowUp" && row > 0) {
-            // @ts-expect-error [FEI-5003] - TS2322 - Type 'readonly string[]' is not assignable to type 'null'.
+            // @ts-expect-error - TS2322 - Type 'readonly string[]' is not assignable to type 'null'.
             nextPath = getInputPath(row - 1, col);
         } else if (e.key === "ArrowDown" && row + 1 < maxRow) {
-            // @ts-expect-error [FEI-5003] - TS2322 - Type 'readonly string[]' is not assignable to type 'null'.
+            // @ts-expect-error - TS2322 - Type 'readonly string[]' is not assignable to type 'null'.
             nextPath = getInputPath(row + 1, col);
         } else if (e.key === "ArrowLeft" && col > 0) {
             if (cursorStartPosition === 0 && cursorEndPosition === 0) {
                 // Only go to next input if we're at the *start* of the content
-                // @ts-expect-error [FEI-5003] - TS2322 - Type 'readonly string[]' is not assignable to type 'null'.
+                // @ts-expect-error - TS2322 - Type 'readonly string[]' is not assignable to type 'null'.
                 nextPath = getInputPath(row, col - 1);
             }
         } else if (e.key === "ArrowRight" && col + 1 < maxCol) {
             if (cursorStartPosition === curValueString.length) {
                 // Only go to next input if we're at the *end* of the content
-                // @ts-expect-error [FEI-5003] - TS2322 - Type 'readonly string[]' is not assignable to type 'null'.
+                // @ts-expect-error - TS2322 - Type 'readonly string[]' is not assignable to type 'null'.
                 nextPath = getInputPath(row, col + 1);
             }
         } else if (e.key === "Enter") {
-            // @ts-expect-error [FEI-5003] - TS2322 - Type 'number' is not assignable to type 'null'.
+            // @ts-expect-error - TS2322 - Type 'number' is not assignable to type 'null'.
             enterTheMatrix = this.state.enterTheMatrix + 1;
         } else if (e.key === "Escape") {
-            // @ts-expect-error [FEI-5003] - TS2322 - Type '0' is not assignable to type 'null'.
+            // @ts-expect-error - TS2322 - Type '0' is not assignable to type 'null'.
             enterTheMatrix = 0;
         }
 
@@ -475,17 +475,17 @@ class Matrix extends React.Component<Props, State> {
 
             // Multiply by 2 to ensure the cursor always ends up at the end;
             // Opera sometimes sees a carriage return as 2 characters.
-            // @ts-expect-error [FEI-5003] - TS2339 - Property 'getStringValue' does not exist on type 'ReactInstance'.
+            // @ts-expect-error - TS2339 - Property 'getStringValue' does not exist on type 'ReactInstance'.
             const inputValString = input.getStringValue();
             const valueLength = inputValString.length * 2;
 
-            // @ts-expect-error [FEI-5003] - TS2339 - Property 'focus' does not exist on type 'ReactInstance'.
+            // @ts-expect-error - TS2339 - Property 'focus' does not exist on type 'ReactInstance'.
             input.focus();
             if (e.key === "ArrowRight") {
-                // @ts-expect-error [FEI-5003] - TS2339 - Property 'setSelectionRange' does not exist on type 'ReactInstance'.
+                // @ts-expect-error - TS2339 - Property 'setSelectionRange' does not exist on type 'ReactInstance'.
                 input.setSelectionRange(0, 0);
             } else {
-                // @ts-expect-error [FEI-5003] - TS2339 - Property 'setSelectionRange' does not exist on type 'ReactInstance'.
+                // @ts-expect-error - TS2339 - Property 'setSelectionRange' does not exist on type 'ReactInstance'.
                 input.setSelectionRange(valueLength, valueLength);
             }
         }
@@ -524,7 +524,7 @@ class Matrix extends React.Component<Props, State> {
     };
 
     simpleValidate: (arg1: any) => any = (rubric) => {
-        // @ts-expect-error [FEI-5003] - TS2339 - Property 'validate' does not exist on type 'typeof Matrix'.
+        // @ts-expect-error - TS2339 - Property 'validate' does not exist on type 'typeof Matrix'.
         return Matrix.validate(this.getUserInput(), rubric);
     };
 }
@@ -559,7 +559,7 @@ _.extend(Matrix, {
                     });
                     const result = validator(supplied[row][col]);
                     if (result.message) {
-                        // @ts-expect-error [FEI-5003] - TS2322 - Type 'string' is not assignable to type 'null'.
+                        // @ts-expect-error - TS2322 - Type 'string' is not assignable to type 'null'.
                         message = result.message;
                     }
                     if (!result.correct) {
@@ -616,7 +616,7 @@ const staticTransform: (arg1: any) => any = (editorProps) => {
     );
     // We convert matrix cells from numbers to string to match the expected
     // input into the rendered widget.
-    // @ts-expect-error [FEI-5003] - TS2339 - Property 'answers' does not exist on type 'Pick<any, "suffix" | "prefix" | "matrixBoardSize">'.
+    // @ts-expect-error - TS2339 - Property 'answers' does not exist on type 'Pick<any, "suffix" | "prefix" | "matrixBoardSize">'.
     widgetProps.answers = _.map(editorProps.answers, (row) => {
         // Replace null values with empty string
         return _.map(row, (cell) => (cell != null ? String(cell) : ""));

package/src/widgets/measurer.tsx

@@ -79,7 +79,7 @@ const Measurer: any = createReactClass({
                             left: image.left,
                         }}
                     >
-                        {/* @ts-expect-error [FEI-5003] - TS2741 - Property 'alt' is missing in type '{ src: any; }' but required in type 'Pick<Readonly<Props> & Readonly<{ children?: ReactNode; }>, "children" | "height" | "width" | "title" | "alt" | "trackInteraction" | "preloader" | "allowFullBleed" | "extraGraphie" | "overrideAriaHidden">'. */}
+                        {/* @ts-expect-error - TS2741 - Property 'alt' is missing in type '{ src: any; }' but required in type 'Pick<Readonly<Props> & Readonly<{ children?: ReactNode; }>, "children" | "height" | "width" | "title" | "alt" | "trackInteraction" | "preloader" | "allowFullBleed" | "extraGraphie" | "overrideAriaHidden">'. */}
                         <SvgImage src={image.url} />
                     </div>
                 )}
@@ -105,7 +105,7 @@ const Measurer: any = createReactClass({
                 "rulerLength",
             ],
             function (prop) {
-                // @ts-expect-error [FEI-5003] - TS2683 - 'this' implicitly has type 'any' because it does not have a type annotation.
+                // @ts-expect-error - TS2683 - 'this' implicitly has type 'any' because it does not have a type annotation.
                 return prevProps[prop] !== this.props[prop];
             },
             this,
@@ -119,7 +119,7 @@ const Measurer: any = createReactClass({
     setupGraphie: function () {
         // eslint-disable-next-line react/no-string-refs
         const graphieDiv = ReactDOM.findDOMNode(this.refs.graphieDiv);
-        // @ts-expect-error [FEI-5003] - TS2769 - No overload matches this call. | TS2339 - Property 'empty' does not exist on type 'JQueryStatic'.
+        // @ts-expect-error - TS2769 - No overload matches this call. | TS2339 - Property 'empty' does not exist on type 'JQueryStatic'.
         $(graphieDiv).empty();
         const graphie = (this.graphie = GraphUtils.createGraphie(graphieDiv));
 

package/src/widgets/molecule/__tests__/layout.test.ts

@@ -7,7 +7,7 @@ function assertApproximately(actual: any, expected: number, allowed: number) {
 
 describe("Molecule layout", () => {
     describe("Simple molecules", () => {
-        // @ts-expect-error [FEI-5003] - TS2554 - Expected 2 arguments, but got 1.
+        // @ts-expect-error - TS2554 - Expected 2 arguments, but got 1.
         const parsedAtoms = SmilesParser.parse("CC");
         const converted = Layout._convertTree({}, [], parsedAtoms);
         const atoms = converted[0];
@@ -72,7 +72,7 @@ describe("Molecule layout", () => {
     });
 
     describe("Layout at a three-way junction", () => {
-        // @ts-expect-error [FEI-5003] - TS2554 - Expected 2 arguments, but got 1.
+        // @ts-expect-error - TS2554 - Expected 2 arguments, but got 1.
         const layoutItems = Layout.layout(SmilesParser.parse("CC(C)C"), 0);
         it("should place the second atom in the center", () => {
             const centerAtom = layoutItems.find((item) => item.idx === "2,0");
@@ -112,7 +112,7 @@ describe("Molecule layout", () => {
     });
 
     describe("Layout at a four-way junction", () => {
-        // @ts-expect-error [FEI-5003] - TS2554 - Expected 2 arguments, but got 1.
+        // @ts-expect-error - TS2554 - Expected 2 arguments, but got 1.
         const layoutItems = Layout.layout(SmilesParser.parse("CC(C)(C)C"), 0);
         it("should place the second atom in the center", () => {
             const centerAtom = layoutItems.find((item) => item.idx === "2,0");
@@ -148,7 +148,7 @@ describe("Molecule layout", () => {
     });
 
     describe("Triple bond layout", () => {
-        // @ts-expect-error [FEI-5003] - TS2554 - Expected 2 arguments, but got 1.
+        // @ts-expect-error - TS2554 - Expected 2 arguments, but got 1.
         const layoutItems = Layout.layout(SmilesParser.parse("CC#CC"), 0);
         it("should place all the atoms colinearly", () => {
             const atoms = layoutItems.filter((item) => item.idx);

package/src/widgets/molecule/__tests__/parser.test.ts

@@ -5,7 +5,7 @@ const parser = SmilesParser.parse;
 describe("SMILES parser", () => {
     describe("single atom parsing", () => {
         it("should parse a single bare atom", () => {
-            // @ts-expect-error [FEI-5003] - TS2554 - Expected 2 arguments, but got 1.
+            // @ts-expect-error - TS2554 - Expected 2 arguments, but got 1.
             const parsed = parser("C");
             expect(parsed).toEqual({
                 type: "atom",
@@ -16,7 +16,7 @@ describe("SMILES parser", () => {
         });
 
         it("should parse a bracketed atom", () => {
-            // @ts-expect-error [FEI-5003] - TS2554 - Expected 2 arguments, but got 1.
+            // @ts-expect-error - TS2554 - Expected 2 arguments, but got 1.
             const parsed = parser("[At]");
             expect(parsed).toEqual({
                 type: "atom",
@@ -27,7 +27,7 @@ describe("SMILES parser", () => {
         });
 
         it("should parse certain common two-letter atoms", () => {
-            // @ts-expect-error [FEI-5003] - TS2554 - Expected 2 arguments, but got 1.
+            // @ts-expect-error - TS2554 - Expected 2 arguments, but got 1.
             const parsed = parser("Br");
             expect(parsed).toEqual({
                 type: "atom",
@@ -40,7 +40,7 @@ describe("SMILES parser", () => {
         it("should fail to parse an unusual atom when not bracketed", () => {
             // Arrange
             // Act
-            // @ts-expect-error [FEI-5003] - TS2554 - Expected 2 arguments, but got 1.
+            // @ts-expect-error - TS2554 - Expected 2 arguments, but got 1.
             const underTest = () => parser("At");
 
             // Assert
@@ -50,7 +50,7 @@ describe("SMILES parser", () => {
 
     describe("bond parsing", () => {
         it("should parse a single bond", () => {
-            // @ts-expect-error [FEI-5003] - TS2554 - Expected 2 arguments, but got 1.
+            // @ts-expect-error - TS2554 - Expected 2 arguments, but got 1.
             const parsed = parser("CC");
             const expectedBond = {
                 type: "bond",
@@ -71,7 +71,7 @@ describe("SMILES parser", () => {
         });
 
         it("should parse a double bond", () => {
-            // @ts-expect-error [FEI-5003] - TS2554 - Expected 2 arguments, but got 1.
+            // @ts-expect-error - TS2554 - Expected 2 arguments, but got 1.
             const parsed = parser("C=C");
             const expectedBond = {
                 type: "bond",
@@ -92,7 +92,7 @@ describe("SMILES parser", () => {
         });
 
         it("should parse a triple bond", () => {
-            // @ts-expect-error [FEI-5003] - TS2554 - Expected 2 arguments, but got 1.
+            // @ts-expect-error - TS2554 - Expected 2 arguments, but got 1.
             const parsed = parser("C#C");
             const expectedBond = {
                 type: "bond",
@@ -115,7 +115,7 @@ describe("SMILES parser", () => {
 
     describe("branch parsing", () => {
         it("should parse a branch", () => {
-            // @ts-expect-error [FEI-5003] - TS2554 - Expected 2 arguments, but got 1.
+            // @ts-expect-error - TS2554 - Expected 2 arguments, but got 1.
             const parsed = parser("C(C)C");
             expect(parsed.bonds).toHaveLength(2);
             expect(parsed.bonds[0].type).toBe("bond");
@@ -129,12 +129,12 @@ describe("SMILES parser", () => {
         });
 
         it("should apply bond modifiers only to one branch", () => {
-            // @ts-expect-error [FEI-5003] - TS2554 - Expected 2 arguments, but got 1.
+            // @ts-expect-error - TS2554 - Expected 2 arguments, but got 1.
             let parsed = parser("C(=O)C");
             expect(parsed.bonds).toHaveLength(2);
             expect(parsed.bonds[0].bondType).toBe("double");
             expect(parsed.bonds[1].bondType).toBe("single");
-            // @ts-expect-error [FEI-5003] - TS2554 - Expected 2 arguments, but got 1.
+            // @ts-expect-error - TS2554 - Expected 2 arguments, but got 1.
             parsed = parser("C(O)=C");
             expect(parsed.bonds).toHaveLength(2);
             expect(parsed.bonds[0].bondType).toBe("single");
@@ -142,13 +142,13 @@ describe("SMILES parser", () => {
         });
 
         it("should error on mismatched parentheses", () => {
-            // @ts-expect-error [FEI-5003] - TS2554 - Expected 2 arguments, but got 1.
+            // @ts-expect-error - TS2554 - Expected 2 arguments, but got 1.
             expect(() => parser("C)")).toThrow();
-            // @ts-expect-error [FEI-5003] - TS2554 - Expected 2 arguments, but got 1.
+            // @ts-expect-error - TS2554 - Expected 2 arguments, but got 1.
             expect(() => parser("C(")).toThrow();
-            // @ts-expect-error [FEI-5003] - TS2554 - Expected 2 arguments, but got 1.
+            // @ts-expect-error - TS2554 - Expected 2 arguments, but got 1.
             expect(() => parser("C(()")).toThrow();
-            // @ts-expect-error [FEI-5003] - TS2554 - Expected 2 arguments, but got 1.
+            // @ts-expect-error - TS2554 - Expected 2 arguments, but got 1.
             expect(() => parser("C())")).toThrow();
         });
     });

package/src/widgets/molecule/smiles-parser.ts

@@ -10,7 +10,7 @@ const smilesRe = new RegExp("^[A-Za-z\\[\\]()=#+-]*$");
 const atomRe = new RegExp("^(Cl|Br|[CONPSFBI]|\\[)");
 
 function ParseError(message: string) {
-    // @ts-expect-error [FEI-5003] - TS2683 - 'this' implicitly has type 'any' because it does not have a type annotation.
+    // @ts-expect-error - TS2683 - 'this' implicitly has type 'any' because it does not have a type annotation.
     this.message = message;
 }
 
@@ -180,9 +180,9 @@ function readAtomSymbol(smiles: any, _ctx) {
         rest = smiles.slice(closingIdx + 1);
     } else {
         const match = atomRe.exec(smiles);
-        // @ts-expect-error [FEI-5003] - TS2322 - Type 'string' is not assignable to type 'null'. | TS2531 - Object is possibly 'null'.
+        // @ts-expect-error - TS2322 - Type 'string' is not assignable to type 'null'. | TS2531 - Object is possibly 'null'.
         sym = match[1];
-        // @ts-expect-error [FEI-5003] - TS2531 - Object is possibly 'null'.
+        // @ts-expect-error - TS2531 - Object is possibly 'null'.
         rest = smiles.slice(sym.length);
     }
 

package/src/widgets/molecule.tsx

@@ -44,13 +44,13 @@ export class Molecule extends React.Component<any, any> {
     stateFromSmiles: (arg1: any) => void = (smiles) => {
         try {
             this.setState({
-                // @ts-expect-error [FEI-5003] - TS2554 - Expected 2 arguments, but got 1.
+                // @ts-expect-error - TS2554 - Expected 2 arguments, but got 1.
                 parsedSmiles: parse(smiles),
                 error: null,
             });
         } catch (e: any) {
             if (e instanceof ParseError) {
-                // @ts-expect-error [FEI-5003] - TS2339 - Property 'message' does not exist on type '{}'.
+                // @ts-expect-error - TS2339 - Property 'message' does not exist on type '{}'.
                 this.setState({error: e.message});
             } else {
                 throw e;
@@ -89,9 +89,9 @@ export class Molecule extends React.Component<any, any> {
         // eslint-disable-next-line react/no-string-refs
         const canvas = this.refs.canvas;
         const translation = this.setCanvasBounds(canvas, items);
-        // @ts-expect-error [FEI-5003] - TS2339 - Property 'getContext' does not exist on type 'ReactInstance'.
+        // @ts-expect-error - TS2339 - Property 'getContext' does not exist on type 'ReactInstance'.
         const ctx = canvas.getContext("2d");
-        // @ts-expect-error [FEI-5003] - TS2339 - Property 'width' does not exist on type 'ReactInstance'. | TS2339 - Property 'height' does not exist on type 'ReactInstance'.
+        // @ts-expect-error - TS2339 - Property 'width' does not exist on type 'ReactInstance'. | TS2339 - Property 'height' does not exist on type 'ReactInstance'.
         ctx.clearRect(0, 0, canvas.width, canvas.height);
         ctx.save();
         ctx.translate(translation[0], translation[1]);

package/src/widgets/number-line.tsx

@@ -16,9 +16,9 @@ import KhanMath from "../util/math";
 import type {ChangeableProps} from "../mixins/changeable";
 import type {APIOptions, WidgetExports} from "../types";
 
-// @ts-expect-error [FEI-5003] - TS2339 - Property 'MovablePoint' does not exist on type 'typeof Graphie'.
+// @ts-expect-error - TS2339 - Property 'MovablePoint' does not exist on type 'typeof Graphie'.
 const MovablePoint = Graphie.MovablePoint;
-// @ts-expect-error [FEI-5003] - TS2339 - Property 'Line' does not exist on type 'typeof Graphie'.
+// @ts-expect-error - TS2339 - Property 'Line' does not exist on type 'typeof Graphie'.
 const Line = Graphie.Line;
 const {assert} = InteractiveUtil;
 
@@ -111,7 +111,7 @@ const _label = (
     }
 };
 
-// @ts-expect-error [FEI-5003] - TS2339 - Property 'createSimpleClass' does not exist on type 'typeof Graphie'.
+// @ts-expect-error - TS2339 - Property 'createSimpleClass' does not exist on type 'typeof Graphie'.
 const TickMarks: any = Graphie.createSimpleClass((graphie, props) => {
     // Avoid infinite loop
     if (!_.isFinite(props.tickStep) || props.tickStep <= 0) {
@@ -252,7 +252,7 @@ class NumberLine extends React.Component<Props, State> {
     };
 
     change: (...args: ReadonlyArray<unknown>) => any = (...args) => {
-        // @ts-expect-error [FEI-5003] - TS2345 - Argument of type 'readonly unknown[]' is not assignable to parameter of type 'any[]'.
+        // @ts-expect-error - TS2345 - Argument of type 'readonly unknown[]' is not assignable to parameter of type 'any[]'.
         return Changeable.change.apply(this, args);
     };
 
@@ -333,7 +333,7 @@ class NumberLine extends React.Component<Props, State> {
     focus: () => boolean | null | undefined = () => {
         if (this.props.isTickCtrl) {
             // eslint-disable-next-line react/no-string-refs
-            // @ts-expect-error [FEI-5003] - TS2339 - Property 'focus' does not exist on type 'ReactInstance'.
+            // @ts-expect-error - TS2339 - Property 'focus' does not exist on type 'ReactInstance'.
             this.refs["tick-ctrl"].focus();
             return true;
         }
@@ -342,7 +342,7 @@ class NumberLine extends React.Component<Props, State> {
     focusInputPath: (arg1: any) => void = (path) => {
         if (path.length === 1) {
             // eslint-disable-next-line react/no-string-refs
-            // @ts-expect-error [FEI-5003] - TS2339 - Property 'focus' does not exist on type 'ReactInstance'.
+            // @ts-expect-error - TS2339 - Property 'focus' does not exist on type 'ReactInstance'.
             this.refs[path[0]].focus();
         }
     };
@@ -350,7 +350,7 @@ class NumberLine extends React.Component<Props, State> {
     blurInputPath: (arg1: any) => void = (path) => {
         if (path.length === 1) {
             // eslint-disable-next-line react/no-string-refs
-            // @ts-expect-error [FEI-5003] - TS2339 - Property 'blur' does not exist on type 'ReactInstance'.
+            // @ts-expect-error - TS2339 - Property 'blur' does not exist on type 'ReactInstance'.
             this.refs[path[0]].blur();
         }
     };
@@ -414,7 +414,7 @@ class NumberLine extends React.Component<Props, State> {
                 options={options}
                 onMouseDown={(coord) => {
                     // eslint-disable-next-line react/no-string-refs
-                    // @ts-expect-error [FEI-5003] - TS2339 - Property 'movables' does not exist on type 'ReactInstance'.
+                    // @ts-expect-error - TS2339 - Property 'movables' does not exist on type 'ReactInstance'.
                     this.refs.graphie.movables.numberLinePoint.grab(coord);
                 }}
                 setup={this._setupGraphie}
@@ -532,7 +532,7 @@ class NumberLine extends React.Component<Props, State> {
         this.props.onChange({rel: newRel});
     };
 
-    // @ts-expect-error [FEI-5003] - TS2322 - Type '(props: any) => any[]' is not assignable to type '(arg1: any) => [number, number]'.
+    // @ts-expect-error - TS2322 - Type '(props: any) => any[]' is not assignable to type '(arg1: any) => [number, number]'.
     _getInequalityEndpoint: (arg1: any) => [number, number] = (props) => {
         const isGreater = _(["ge", "gt"]).contains(props.rel);
         const widthInPixels = 400;
@@ -545,7 +545,7 @@ class NumberLine extends React.Component<Props, State> {
         return end;
     };
 
-    // @ts-expect-error [FEI-5003] - TS2322 - Type '(props: any) => Element | null' is not assignable to type '(arg1: any) => ReactElement<any, string | JSXElementConstructor<any>>'.
+    // @ts-expect-error - TS2322 - Type '(props: any) => Element | null' is not assignable to type '(arg1: any) => ReactElement<any, string | JSXElementConstructor<any>>'.
     _renderInequality: (arg1: any) => React.ReactElement = (props) => {
         if (props.isInequality) {
             const end = this._getInequalityEndpoint(props);
@@ -625,7 +625,7 @@ class NumberLine extends React.Component<Props, State> {
     };
 
     simpleValidate: (arg1: any) => any = (rubric) => {
-        // @ts-expect-error [FEI-5003] - TS2339 - Property 'validate' does not exist on type 'typeof NumberLine'.
+        // @ts-expect-error - TS2339 - Property 'validate' does not exist on type 'typeof NumberLine'.
         return NumberLine.validate(this.getUserInput(), rubric);
     };