@khanacademy/perseus-editor 2.6.0 → 2.6.2

package/src/editor-page.tsx

@@ -69,7 +69,7 @@ type State = {
 
 class EditorPage extends React.Component<Props, State> {
     _isMounted: boolean;
-    // @ts-expect-error [FEI-5003] - TS2564 - Property 'rendererMountNode' has no initializer and is not definitely assigned in the constructor.
+    // @ts-expect-error - TS2564 - Property 'rendererMountNode' has no initializer and is not definitely assigned in the constructor.
     rendererMountNode: HTMLDivElement;
     renderer: RendererInterface | null | undefined;
 
@@ -87,7 +87,7 @@ class EditorPage extends React.Component<Props, State> {
         super(props);
 
         this.state = {
-            // @ts-expect-error [FEI-5003] - TS2322 - Type 'Pick<Readonly<Props> & Readonly<{ children?: ReactNode; }>, "hints" | "question" | "answerArea" | "itemDataVersion">' is not assignable to type 'PerseusJson'.
+            // @ts-expect-error - TS2322 - Type 'Pick<Readonly<Props> & Readonly<{ children?: ReactNode; }>, "hints" | "question" | "answerArea" | "itemDataVersion">' is not assignable to type 'PerseusJson'.
             json: _.pick(
                 this.props,
                 "question",
@@ -161,7 +161,7 @@ class EditorPage extends React.Component<Props, State> {
         };
 
         // eslint-disable-next-line react/no-string-refs
-        // @ts-expect-error [FEI-5003] - TS2339 - Property 'triggerPreviewUpdate' does not exist on type 'ReactInstance'.
+        // @ts-expect-error - TS2339 - Property 'triggerPreviewUpdate' does not exist on type 'ReactInstance'.
         this.refs.itemEditor.triggerPreviewUpdate({
             type: "question",
             data: _({
@@ -177,7 +177,7 @@ class EditorPage extends React.Component<Props, State> {
                 },
                 reviewMode: true,
                 // eslint-disable-next-line react/no-string-refs
-                // @ts-expect-error [FEI-5003] - TS2339 - Property 'getSaveWarnings' does not exist on type 'ReactInstance'.
+                // @ts-expect-error - TS2339 - Property 'getSaveWarnings' does not exist on type 'ReactInstance'.
                 legacyPerseusLint: this.refs.itemEditor.getSaveWarnings(),
             }).extend(
                 _(this.props).pick(
@@ -199,10 +199,10 @@ class EditorPage extends React.Component<Props, State> {
 
     getSaveWarnings(): any {
         // eslint-disable-next-line react/no-string-refs
-        // @ts-expect-error [FEI-5003] - TS2339 - Property 'getSaveWarnings' does not exist on type 'ReactInstance'.
+        // @ts-expect-error - TS2339 - Property 'getSaveWarnings' does not exist on type 'ReactInstance'.
         const issues1 = this.refs.itemEditor.getSaveWarnings();
         // eslint-disable-next-line react/no-string-refs
-        // @ts-expect-error [FEI-5003] - TS2339 - Property 'getSaveWarnings' does not exist on type 'ReactInstance'.
+        // @ts-expect-error - TS2339 - Property 'getSaveWarnings' does not exist on type 'ReactInstance'.
         const issues2 = this.refs.hintsEditor.getSaveWarnings();
         return issues1.concat(issues2);
     }
@@ -212,10 +212,10 @@ class EditorPage extends React.Component<Props, State> {
             return this.state.json;
         }
         // eslint-disable-next-line react/no-string-refs
-        // @ts-expect-error [FEI-5003] - TS2339 - Property 'serialize' does not exist on type 'ReactInstance'.
+        // @ts-expect-error - TS2339 - Property 'serialize' does not exist on type 'ReactInstance'.
         return _.extend(this.refs.itemEditor.serialize(options), {
             // eslint-disable-next-line react/no-string-refs
-            // @ts-expect-error [FEI-5003] - TS2339 - Property 'serialize' does not exist on type 'ReactInstance'.
+            // @ts-expect-error - TS2339 - Property 'serialize' does not exist on type 'ReactInstance'.
             hints: this.refs.hintsEditor.serialize(options),
         });
     }

package/src/editor.tsx

@@ -1,3 +1,4 @@
+/* eslint-disable @khanacademy/ts-no-error-suppressions */
 import {
     preprocessTex,
     Errors,
@@ -87,7 +88,7 @@ const IMAGE_REGEX = /!\[[^\]]*\]\(([^\s)]+)[^)]*\)/g;
  * ignores captures. If you don't need captures, use String::match
  */
 const allMatches = function (regex: RegExp, str: string) {
-    // @ts-expect-error [FEI-5003] - TS2702 - 'RegExp' only refers to a type, but is being used as a namespace here.
+    // @ts-expect-error - TS2702 - 'RegExp' only refers to a type, but is being used as a namespace here.
     const result: Array<any | RegExp.matchResult> = [];
     // eslint-disable-next-line no-constant-condition
     while (true) {
@@ -183,9 +184,9 @@ class Editor extends React.Component<Props, State> {
         // setState
         this._sizeImages(this.props);
         // eslint-disable-next-line react/no-string-refs
-        // @ts-expect-error [FEI-5003] - TS2769 - No overload matches this call.
+        // @ts-expect-error - TS2769 - No overload matches this call.
         $(ReactDOM.findDOMNode(this.refs.textarea))
-            // @ts-expect-error [FEI-5003] - TS2339 - Property 'on' does not exist on type 'JQueryStatic'.
+            // @ts-expect-error - TS2339 - Property 'on' does not exist on type 'JQueryStatic'.
             .on("copy cut", this._maybeCopyWidgets)
             .on("paste", this._maybePasteWidgets);
     }
@@ -215,13 +216,13 @@ class Editor extends React.Component<Props, State> {
              * textarea should be refined with an instanceof check to
              * HTMLTextAreaElement so that these props are available.
              */
-            // @ts-expect-error [FEI-5003] - TS2339 - Property 'focus' does not exist on type 'Element | Text'.
+            // @ts-expect-error - TS2339 - Property 'focus' does not exist on type 'Element | Text'.
             textarea.focus();
-            // @ts-expect-error [FEI-5003] - TS2339 - Property 'value' does not exist on type 'Element | Text'.
+            // @ts-expect-error - TS2339 - Property 'value' does not exist on type 'Element | Text'.
             textarea.value = this.lastUserValue;
-            // @ts-expect-error [FEI-5003] - TS2339 - Property 'selectionStart' does not exist on type 'Element | Text'.
+            // @ts-expect-error - TS2339 - Property 'selectionStart' does not exist on type 'Element | Text'.
             textarea.selectionStart = 0;
-            // @ts-expect-error [FEI-5003] - TS2339 - Property 'select' does not exist on type 'Element | Text'.
+            // @ts-expect-error - TS2339 - Property 'select' does not exist on type 'Element | Text'.
             textarea.select(0, prevProps.content.length);
             if (
                 document.execCommand(
@@ -232,7 +233,7 @@ class Editor extends React.Component<Props, State> {
             ) {
                 // This command is not implemented. Fall back to setting `value`
                 // directly.
-                // @ts-expect-error [FEI-5003] - TS2339 - Property 'value' does not exist on type 'Element | Text'.
+                // @ts-expect-error - TS2339 - Property 'value' does not exist on type 'Element | Text'.
                 textarea.value = this.props.content;
             }
             this.lastUserValue = null;
@@ -258,11 +259,11 @@ class Editor extends React.Component<Props, State> {
             }
             return (
                 <WidgetEditor
-                    // @ts-expect-error [FEI-5003] - TS2322 - Type 'string' is not assignable to type 'number | undefined'.
+                    // @ts-expect-error - TS2322 - Type 'string' is not assignable to type 'number | undefined'.
                     key={id}
                     ref={id}
                     id={id}
-                    // @ts-expect-error [FEI-5003] - TS2783 - 'type' is specified more than once, so this usage will be overwritten.
+                    // @ts-expect-error - TS2783 - 'type' is specified more than once, so this usage will be overwritten.
                     type={type}
                     // eslint-disable-next-line react/jsx-no-bind
                     onChange={this._handleWidgetEditorChange.bind(this, id)}
@@ -294,7 +295,7 @@ class Editor extends React.Component<Props, State> {
         // eslint-disable-next-line react/no-string-refs
         const textarea = this.refs.textarea;
         const re = new RegExp(widgetRegExp.replace("{id}", id), "gm");
-        // @ts-expect-error [FEI-5003] - TS2339 - Property 'value' does not exist on type 'ReactInstance'.
+        // @ts-expect-error - TS2339 - Property 'value' does not exist on type 'ReactInstance'.
         this.props.onChange({content: textarea.value.replace(re, "")});
     };
 
@@ -334,7 +335,7 @@ class Editor extends React.Component<Props, State> {
                     {
                         images: _.clone(images),
                     },
-                    // @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'.
                     null, // callback
                     true, // silent
                 );
@@ -410,13 +411,13 @@ class Editor extends React.Component<Props, State> {
                 this.props.onChange({content: content});
             })
             .each((fileAndSentinel) => {
-                // @ts-expect-error [FEI-5003] - TS2531 - Object is possibly 'null'. | TS2345 - Argument of type 'File' is not assignable to parameter of type 'string'.
+                // @ts-expect-error - TS2531 - Object is possibly 'null'. | TS2345 - Argument of type 'File' is not assignable to parameter of type 'string'.
                 imageUploader(fileAndSentinel.file, (url) => {
                     // See componentDidUpdate() for how this flag is used
                     this.lastUserValue = origContent;
                     this.props.onChange({
                         content: this.state.textAreaValue.replace(
-                            // @ts-expect-error [FEI-5003] - TS2531 - Object is possibly 'null'.
+                            // @ts-expect-error - TS2531 - Object is possibly 'null'.
                             fileAndSentinel.sentinel,
                             url,
                         ),
@@ -452,7 +453,7 @@ class Editor extends React.Component<Props, State> {
             const textarea = ReactDOM.findDOMNode(this.refs.textarea);
 
             // findDOMNode can also return Text, but we know it's an element.
-            // @ts-expect-error [FEI-5003] - TS2345 - Argument of type 'Element | Text | null' is not assignable to parameter of type 'HTMLTextAreaElement'.
+            // @ts-expect-error - TS2345 - Argument of type 'Element | Text | null' is not assignable to parameter of type 'HTMLTextAreaElement'.
             const word = Util.textarea.getWordBeforeCursor(textarea);
             const matches = word.string.toLowerCase().match(shortcutRegexp);
 
@@ -492,10 +493,10 @@ class Editor extends React.Component<Props, State> {
             );
 
             const widgetNames = _.map(
-                // @ts-expect-error [FEI-5003] - TS2345 - Argument of type 'RegExpMatchArray | null' is not assignable to parameter of type 'Collection<any>'.
+                // @ts-expect-error - TS2345 - Argument of type 'RegExpMatchArray | null' is not assignable to parameter of type 'Collection<any>'.
                 selectedText.match(rWidgetSplit),
                 (syntax) => {
-                    // @ts-expect-error [FEI-5003] - TS2531 - Object is possibly 'null'.
+                    // @ts-expect-error - TS2531 - Object is possibly 'null'.
                     return Util.rWidgetParts.exec(syntax)[1];
                 },
             );
@@ -519,7 +520,7 @@ class Editor extends React.Component<Props, State> {
             const widgetJSON = localStorage.perseusLastCopiedWidgets;
             const lastCopiedText = localStorage.perseusLastCopiedText;
             const textToBePasted =
-                // @ts-expect-error [FEI-5003] - TS2339 - Property 'originalEvent' does not exist on type 'SyntheticEvent<HTMLTextAreaElement, Event>'.
+                // @ts-expect-error - TS2339 - Property 'originalEvent' does not exist on type 'SyntheticEvent<HTMLTextAreaElement, Event>'.
                 e.originalEvent.clipboardData.getData("text");
 
             // Only intercept if we have widget data to paste and the user is
@@ -550,9 +551,9 @@ class Editor extends React.Component<Props, State> {
                     rWidgetSplit,
                     (syntax) => {
                         const match = Util.rWidgetParts.exec(syntax);
-                        // @ts-expect-error [FEI-5003] - TS2531 - Object is possibly 'null'.
+                        // @ts-expect-error - TS2531 - Object is possibly 'null'.
                         const completeWidget = match[0];
-                        // @ts-expect-error [FEI-5003] - TS2531 - Object is possibly 'null'.
+                        // @ts-expect-error - TS2531 - Object is possibly 'null'.
                         const widget = match[1];
                         return completeWidget.replace(
                             widget,
@@ -649,12 +650,12 @@ class Editor extends React.Component<Props, State> {
         // Note: we have to use _.map here instead of Array::map
         // because the results of a .match might be null if no
         // widgets were found.
-        // @ts-expect-error [FEI-5003] - TS2345 - Argument of type 'RegExpMatchArray | null' is not assignable to parameter of type 'Collection<any>'.
+        // @ts-expect-error - TS2345 - Argument of type 'RegExpMatchArray | null' is not assignable to parameter of type 'Collection<any>'.
         const allWidgetIds = _.map(oldContent.match(rWidgetSplit), (syntax) => {
             const match = Util.rWidgetParts.exec(syntax);
-            // @ts-expect-error [FEI-5003] - TS2531 - Object is possibly 'null'.
+            // @ts-expect-error - TS2531 - Object is possibly 'null'.
             const type = match[2];
-            // @ts-expect-error [FEI-5003] - TS2531 - Object is possibly 'null'.
+            // @ts-expect-error - TS2531 - Object is possibly 'null'.
             const num = +match[3];
             return [type, num];
         });
@@ -692,12 +693,12 @@ class Editor extends React.Component<Props, State> {
         const newWidgets = _.clone(this.props.widgets);
         newWidgets[id] = {
             options: Widgets.getEditor(widgetType)?.defaultProps,
-            // @ts-expect-error [FEI-5003] - TS2322 - Type 'string' is not assignable to type '"video" | "image" | "iframe" | "table" | "radio" | "definition" | "group" | "matrix" | "categorizer" | "cs-program" | "dropdown" | "example-graphie-widget" | "example-widget" | ... 26 more ... | "unit-input"'.
+            // @ts-expect-error - TS2322 - Type 'string' is not assignable to type '"video" | "image" | "iframe" | "table" | "radio" | "definition" | "group" | "matrix" | "categorizer" | "cs-program" | "dropdown" | "example-graphie-widget" | "example-widget" | ... 26 more ... | "unit-input"'.
             type: widgetType,
             // Track widget version on creation, so that a widget editor
             // without a valid version prop can only possibly refer to a
             // pre-versioning creation time.
-            // @ts-expect-error [FEI-5003] - TS2322 - Type 'Version | null | undefined' is not assignable to type 'Version | undefined'.
+            // @ts-expect-error - TS2322 - Type 'Version | null | undefined' is not assignable to type 'Version | undefined'.
             version: Widgets.getVersion(widgetType),
         };
 
@@ -711,7 +712,7 @@ class Editor extends React.Component<Props, State> {
             function () {
                 Util.textarea.moveCursor(
                     // findDOMNode can return Text but we know this is Element
-                    // @ts-expect-error [FEI-5003] - TS2345 - Argument of type 'Element | Text | null' is not assignable to parameter of type 'HTMLTextAreaElement'.
+                    // @ts-expect-error - TS2345 - Argument of type 'Element | Text | null' is not assignable to parameter of type 'HTMLTextAreaElement'.
                     textarea,
                     // We want to put the cursor after the widget
                     // and after any added newlines
@@ -727,11 +728,11 @@ class Editor extends React.Component<Props, State> {
 
         this._addWidgetToContent(
             this.props.content,
-            // @ts-expect-error [FEI-5003] - TS2339 - Property 'selectionStart' does not exist on type 'ReactInstance'. | TS2339 - Property 'selectionEnd' does not exist on type 'ReactInstance'.
+            // @ts-expect-error - TS2339 - Property 'selectionStart' does not exist on type 'ReactInstance'. | TS2339 - Property 'selectionEnd' does not exist on type 'ReactInstance'.
             [textarea.selectionStart, textarea.selectionEnd],
             widgetType,
         );
-        // @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'.
         textarea.focus();
     };
 
@@ -806,7 +807,7 @@ class Editor extends React.Component<Props, State> {
             .chain()
             .map((id) => {
                 // eslint-disable-next-line react/no-string-refs
-                // @ts-expect-error [FEI-5003] - TS2339 - Property 'getSaveWarnings' does not exist on type 'ReactInstance'.
+                // @ts-expect-error - TS2339 - Property 'getSaveWarnings' does not exist on type 'ReactInstance'.
                 const issuesFunc = this.refs[id].getSaveWarnings;
                 const issues = issuesFunc ? issuesFunc() : [];
                 return _.map(issues, (issue) => id + ": " + issue);
@@ -826,7 +827,7 @@ class Editor extends React.Component<Props, State> {
              * textarea should be refined with an instanceof check to
              * HTMLTextAreaElement so that these props are available.
              */
-            // @ts-expect-error [FEI-5003] - TS2339 - Property 'focus' does not exist on type 'Element | Text'.
+            // @ts-expect-error - TS2339 - Property 'focus' does not exist on type 'Element | Text'.
             textarea.focus();
         }
     };
@@ -841,9 +842,9 @@ class Editor extends React.Component<Props, State> {
              * textarea should be refined with an instanceof check to
              * HTMLTextAreaElement so that these props are available.
              */
-            // @ts-expect-error [FEI-5003] - TS2339 - Property 'selectionStart' does not exist on type 'Element | Text'. | TS2339 - Property 'value' does not exist on type 'Element | Text'.
+            // @ts-expect-error - TS2339 - Property 'selectionStart' does not exist on type 'Element | Text'. | TS2339 - Property 'value' does not exist on type 'Element | Text'.
             textarea.selectionStart = textarea.value.length;
-            // @ts-expect-error [FEI-5003] - TS2339 - Property 'selectionEnd' does not exist on type 'Element | Text'. | TS2339 - Property 'value' does not exist on type 'Element | Text'.
+            // @ts-expect-error - TS2339 - Property 'selectionEnd' does not exist on type 'Element | Text'. | TS2339 - Property 'value' does not exist on type 'Element | Text'.
             textarea.selectionEnd = textarea.value.length;
         }
     };
@@ -862,7 +863,7 @@ class Editor extends React.Component<Props, State> {
         const widgetIds = _.intersection(this.widgetIds, _.keys(this.refs));
         _.each(widgetIds, (id) => {
             // eslint-disable-next-line react/no-string-refs
-            // @ts-expect-error [FEI-5003] - TS2339 - Property 'serialize' does not exist on type 'ReactInstance'.
+            // @ts-expect-error - TS2339 - Property 'serialize' does not exist on type 'ReactInstance'.
             widgets[id] = this.refs[id].serialize();
         });
 
@@ -932,7 +933,7 @@ class Editor extends React.Component<Props, State> {
                     // Normal text
                     underlayPieces.push(pieces[i]);
 
-                    // @ts-expect-error [FEI-5003] - TS2554 - Expected 2 arguments, but got 1.
+                    // @ts-expect-error - TS2554 - Expected 2 arguments, but got 1.
                     const ast = PerseusMarkdown.parse(pieces[i]);
 
                     PerseusMarkdown.traverseContent(ast, (node) => {
@@ -945,7 +946,7 @@ class Editor extends React.Component<Props, State> {
                             } catch (e: any) {
                                 katexErrorList.push({
                                     math: content,
-                                    // @ts-expect-error [FEI-5003] - TS2322 - Type 'any' is not assignable to type 'never'.
+                                    // @ts-expect-error - TS2322 - Type 'any' is not assignable to type 'never'.
                                     message: e.message,
                                 });
                             }
@@ -1001,7 +1002,7 @@ class Editor extends React.Component<Props, State> {
 
             const insertTemplateString = "Insert template\u2026";
             templatesDropDown = (
-                // @ts-expect-error [FEI-5003] - TS2322 - Type '(e: SyntheticEvent<HTMLTextAreaElement, Event>) => void' is not assignable to type 'ChangeEventHandler<HTMLSelectElement>'.
+                // @ts-expect-error - TS2322 - Type '(e: SyntheticEvent<HTMLTextAreaElement, Event>) => void' is not assignable to type 'ChangeEventHandler<HTMLSelectElement>'.
                 <select onChange={this.addTemplate}>
                     <option value="">{insertTemplateString}</option>
                     <option disabled>--</option>

package/src/hint-editor.tsx

@@ -1,3 +1,4 @@
+/* eslint-disable @khanacademy/ts-no-error-suppressions */
 /* eslint-disable @babel/no-invalid-this */
 /**
  * Collection of classes for rendering the hint editor area,
@@ -81,19 +82,19 @@ export class HintEditor extends React.Component<HintEditorProps> {
 
     focus: () => void = () => {
         // 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.editor.focus();
     };
 
     getSaveWarnings: () => any = () => {
         // eslint-disable-next-line react/no-string-refs
-        // @ts-expect-error [FEI-5003] - TS2339 - Property 'getSaveWarnings' does not exist on type 'ReactInstance'.
+        // @ts-expect-error - TS2339 - Property 'getSaveWarnings' does not exist on type 'ReactInstance'.
         return this.refs.editor.getSaveWarnings();
     };
 
     serialize: (options?: any) => any = (options: any) => {
         // eslint-disable-next-line react/no-string-refs
-        // @ts-expect-error [FEI-5003] - TS2339 - Property 'serialize' does not exist on type 'ReactInstance'.
+        // @ts-expect-error - TS2339 - Property 'serialize' does not exist on type 'ReactInstance'.
         return this.refs.editor.serialize(options);
     };
 
@@ -146,7 +147,7 @@ export class HintEditor extends React.Component<HintEditorProps> {
                     )}
                     <input
                         type="checkbox"
-                        // @ts-expect-error [FEI-5003] - TS2322 - Type 'boolean | null | undefined' is not assignable to type 'boolean | undefined'.
+                        // @ts-expect-error - TS2322 - Type 'boolean | null | undefined' is not assignable to type 'boolean | undefined'.
                         checked={this.props.replace}
                         onChange={this.handleChange}
                     />
@@ -204,7 +205,7 @@ class CombinedHintEditor extends React.Component<CombinedHintEditorProps> {
             this.props.isLast && !/\*\*/.test(this.props.hint.content);
 
         // eslint-disable-next-line react/no-string-refs
-        // @ts-expect-error [FEI-5003] - TS2339 - Property 'sendNewData' does not exist on type 'ReactInstance'.
+        // @ts-expect-error - TS2339 - Property 'sendNewData' does not exist on type 'ReactInstance'.
         this.refs.frame.sendNewData({
             type: "hint",
             data: {
@@ -223,19 +224,19 @@ class CombinedHintEditor extends React.Component<CombinedHintEditorProps> {
 
     getSaveWarnings = () => {
         // eslint-disable-next-line react/no-string-refs
-        // @ts-expect-error [FEI-5003] - TS2339 - Property 'getSaveWarnings' does not exist on type 'ReactInstance'.
+        // @ts-expect-error - TS2339 - Property 'getSaveWarnings' does not exist on type 'ReactInstance'.
         return this.refs.editor.getSaveWarnings();
     };
 
     serialize = (options: any) => {
         // eslint-disable-next-line react/no-string-refs
-        // @ts-expect-error [FEI-5003] - TS2339 - Property 'serialize' does not exist on type 'ReactInstance'.
+        // @ts-expect-error - TS2339 - Property 'serialize' does not exist on type 'ReactInstance'.
         return this.refs.editor.serialize(options);
     };
 
     focus = () => {
         // 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.editor.focus();
     };
 
@@ -340,15 +341,15 @@ class CombinedHintsEditor extends React.Component<CombinedHintsEditorProps> {
             newProps,
         );
 
-        // @ts-expect-error [FEI-5003] - TS2740 - Type 'Hint' is missing the following properties from type 'readonly Hint[]': length, concat, join, slice, and 18 more.
+        // @ts-expect-error - TS2740 - Type 'Hint' is missing the following properties from type 'readonly Hint[]': length, concat, join, slice, and 18 more.
         this.props.onChange({hints: hints}, cb, silent);
     };
 
     handleHintRemove: (i: number) => void = (i: number) => {
         const hints = _(this.props.hints).clone();
-        // @ts-expect-error [FEI-5003] - TS2339 - Property 'splice' does not exist on type 'Hint'.
+        // @ts-expect-error - TS2339 - Property 'splice' does not exist on type 'Hint'.
         hints.splice(i, 1);
-        // @ts-expect-error [FEI-5003] - TS2322 - Type 'Hint' is not assignable to type 'readonly Hint[]'.
+        // @ts-expect-error - TS2322 - Type 'Hint' is not assignable to type 'readonly Hint[]'.
         this.props.onChange({hints: hints});
     };
 
@@ -357,14 +358,14 @@ class CombinedHintsEditor extends React.Component<CombinedHintsEditorProps> {
         dir: number,
     ) => {
         const hints = _(this.props.hints).clone();
-        // @ts-expect-error [FEI-5003] - TS2339 - Property 'splice' does not exist on type 'Hint'.
+        // @ts-expect-error - TS2339 - Property 'splice' does not exist on type 'Hint'.
         const hint = hints.splice(i, 1)[0];
-        // @ts-expect-error [FEI-5003] - TS2339 - Property 'splice' does not exist on type 'Hint'.
+        // @ts-expect-error - TS2339 - Property 'splice' does not exist on type 'Hint'.
         hints.splice(i + dir, 0, hint);
-        // @ts-expect-error [FEI-5003] - TS2322 - Type 'Hint' is not assignable to type 'readonly Hint[]'.
+        // @ts-expect-error - TS2322 - Type 'Hint' is not assignable to type 'readonly Hint[]'.
         this.props.onChange({hints: hints}, () => {
             // 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["hintEditor" + (i + dir)].focus();
         });
     };
@@ -372,12 +373,12 @@ class CombinedHintsEditor extends React.Component<CombinedHintsEditorProps> {
     addHint: () => void = () => {
         const hints = _(this.props.hints)
             .clone()
-            // @ts-expect-error [FEI-5003] - TS2339 - Property 'concat' does not exist on type 'Hint'.
+            // @ts-expect-error - TS2339 - Property 'concat' does not exist on type 'Hint'.
             .concat([{content: ""}]);
         this.props.onChange({hints: hints}, () => {
             const i = hints.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["hintEditor" + i].focus();
         });
     };
@@ -387,7 +388,7 @@ class CombinedHintsEditor extends React.Component<CombinedHintsEditorProps> {
             .map((hint, i) => {
                 return _.map(
                     // eslint-disable-next-line react/no-string-refs
-                    // @ts-expect-error [FEI-5003] - TS2339 - Property 'getSaveWarnings' does not exist on type 'ReactInstance'.
+                    // @ts-expect-error - TS2339 - Property 'getSaveWarnings' does not exist on type 'ReactInstance'.
                     this.refs["hintEditor" + i].getSaveWarnings(),
                     (issue) => "Hint " + (i + 1) + ": " + issue,
                 );
@@ -407,7 +408,7 @@ class CombinedHintsEditor extends React.Component<CombinedHintsEditorProps> {
         options: any,
     ): string => {
         // eslint-disable-next-line react/no-string-refs
-        // @ts-expect-error [FEI-5003] - TS2339 - Property 'serialize' does not exist on type 'ReactInstance'.
+        // @ts-expect-error - TS2339 - Property 'serialize' does not exist on type 'ReactInstance'.
         return this.refs["hintEditor" + index].serialize(options);
     };
 
@@ -425,24 +426,24 @@ class CombinedHintsEditor extends React.Component<CombinedHintsEditorProps> {
                         itemId={itemId}
                         hint={hint}
                         pos={i}
-                        // @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.
                         imageUploader={this.props.imageUploader}
                         // eslint-disable-next-line react/jsx-no-bind
-                        // @ts-expect-error [FEI-5003] - TS2683 - 'this' implicitly has type 'any' because it does not have a type annotation. | 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. | TS2683 - 'this' implicitly has type 'any' because it does not have a type annotation.
                         onChange={this.handleHintChange.bind(this, i)}
                         // eslint-disable-next-line react/jsx-no-bind
-                        // @ts-expect-error [FEI-5003] - TS2683 - 'this' implicitly has type 'any' because it does not have a type annotation. | 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. | TS2683 - 'this' implicitly has type 'any' because it does not have a type annotation.
                         onRemove={this.handleHintRemove.bind(this, i)}
                         // eslint-disable-next-line react/jsx-no-bind
-                        // @ts-expect-error [FEI-5003] - TS2683 - 'this' implicitly has type 'any' because it does not have a type annotation. | 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. | TS2683 - 'this' implicitly has type 'any' because it does not have a type annotation.
                         onMove={this.handleHintMove.bind(this, i)}
-                        // @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.
                         deviceType={this.props.deviceType}
-                        // @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.
                         apiOptions={this.props.apiOptions}
-                        // @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.
                         highlightLint={this.props.highlightLint}
-                        // @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.
                         previewURL={this.props.previewURL}
                         // TODO(CP-4838): what should be passed here?
                         contentPaths={[]}

package/src/i18n.ts

@@ -9,7 +9,7 @@ const {findContentNodesInItem, findHintNodesInItem, inferItemShape} =
 
 // Takes a renderer content and parses the markdown for images
 function findImagesInContent(content: any, images: Array<any>) {
-    // @ts-expect-error [FEI-5003] - TS2554 - Expected 2 arguments, but got 1.
+    // @ts-expect-error - TS2554 - Expected 2 arguments, but got 1.
     const parsed = PerseusMarkdown.parse(content);
 
     PerseusMarkdown.traverseContent(parsed, function (node) {
@@ -119,12 +119,12 @@ function findImagesInItemData(itemData: any): any {
     let renderers = [];
     if (itemData._multi) {
         const shape = inferItemShape(itemData);
-        // @ts-expect-error [FEI-5003] - TS2345 - Argument of type 'ContentNode' is not assignable to parameter of type 'never'.
+        // @ts-expect-error - TS2345 - Argument of type 'ContentNode' is not assignable to parameter of type 'never'.
         findContentNodesInItem(itemData, shape, (node) => renderers.push(node));
-        // @ts-expect-error [FEI-5003] - TS2345 - Argument of type 'HintNode' is not assignable to parameter of type 'never'.
+        // @ts-expect-error - TS2345 - Argument of type 'HintNode' is not assignable to parameter of type 'never'.
         findHintNodesInItem(itemData, shape, (node) => renderers.push(node));
     } else {
-        // @ts-expect-error [FEI-5003] - TS2322 - Type 'any' is not assignable to type 'never'. | TS2322 - Type 'any' is not assignable to type 'never'.
+        // @ts-expect-error - TS2322 - Type 'any' is not assignable to type 'never'. | TS2322 - Type 'any' is not assignable to type 'never'.
         renderers = [itemData.question, ...itemData.hints];
     }
     return findImagesInRenderers(renderers);

package/src/iframe-content-renderer.tsx

@@ -17,7 +17,7 @@ import * as React from "react";
 let nextIframeID = 0;
 const requestIframeData: Record<string, any> = {};
 const updateIframeHeight: Record<string, any> = {};
-// @ts-expect-error [FEI-5003] - TS2339 - Property 'iframeDataStore' does not exist on type 'Window & typeof globalThis'.
+// @ts-expect-error - TS2339 - Property 'iframeDataStore' does not exist on type 'Window & typeof globalThis'.
 window.iframeDataStore = {};
 
 // This is called once after Perseus is loaded and the iframe
@@ -66,12 +66,12 @@ type Props = {
 
 class IframeContentRenderer extends React.Component<Props> {
     _frame: HTMLIFrameElement | null | undefined;
-    // @ts-expect-error [FEI-5003] - TS2564 - Property '_isMounted' has no initializer and is not definitely assigned in the constructor.
+    // @ts-expect-error - TS2564 - Property '_isMounted' has no initializer and is not definitely assigned in the constructor.
     _isMounted: boolean;
     _lastData: any;
-    // @ts-expect-error [FEI-5003] - TS2564 - Property '_lastHeight' has no initializer and is not definitely assigned in the constructor.
+    // @ts-expect-error - TS2564 - Property '_lastHeight' has no initializer and is not definitely assigned in the constructor.
     _lastHeight: number;
-    // @ts-expect-error [FEI-5003] - TS2564 - Property 'iframeID' has no initializer and is not definitely assigned in the constructor.
+    // @ts-expect-error - TS2564 - Property 'iframeID' has no initializer and is not definitely assigned in the constructor.
     iframeID: number;
 
     componentDidMount() {
@@ -91,7 +91,7 @@ class IframeContentRenderer extends React.Component<Props> {
             this._lastHeight = height;
             if (this._isMounted && this.props.seamless) {
                 // eslint-disable-next-line react/no-string-refs
-                // @ts-expect-error [FEI-5003] - TS2339 - Property 'style' does not exist on type 'ReactInstance'.
+                // @ts-expect-error - TS2339 - Property 'style' does not exist on type 'ReactInstance'.
                 this.refs.container.style.height = height + "px";
             }
         };
@@ -107,11 +107,11 @@ class IframeContentRenderer extends React.Component<Props> {
     componentDidUpdate(prevProps: Props) {
         if (!this.props.seamless) {
             // eslint-disable-next-line react/no-string-refs
-            // @ts-expect-error [FEI-5003] - TS2339 - Property 'style' does not exist on type 'ReactInstance'.
+            // @ts-expect-error - TS2339 - Property 'style' does not exist on type 'ReactInstance'.
             this.refs.container.style.height = "100%";
         } else {
             // eslint-disable-next-line react/no-string-refs
-            // @ts-expect-error [FEI-5003] - TS2339 - Property 'style' does not exist on type 'ReactInstance'.
+            // @ts-expect-error - TS2339 - Property 'style' does not exist on type 'ReactInstance'.
             this.refs.container.style.height = this._lastHeight + "px";
         }
 
@@ -132,7 +132,7 @@ class IframeContentRenderer extends React.Component<Props> {
         // Don't initialize the iframe until the page has loaded
         if (this._frame) {
             // eslint-disable-next-line react/no-string-refs
-            // @ts-expect-error [FEI-5003] - TS2339 - Property 'removeChild' does not exist on type 'ReactInstance'.
+            // @ts-expect-error - TS2339 - Property 'removeChild' does not exist on type 'ReactInstance'.
             this.refs.container.removeChild(this._frame);
         }
 
@@ -160,7 +160,7 @@ class IframeContentRenderer extends React.Component<Props> {
         }
 
         // eslint-disable-next-line react/no-string-refs
-        // @ts-expect-error [FEI-5003] - TS2339 - Property 'appendChild' does not exist on type 'ReactInstance'.
+        // @ts-expect-error - TS2339 - Property 'appendChild' does not exist on type 'ReactInstance'.
         this.refs.container.appendChild(frame);
 
         this._frame = frame;
@@ -174,7 +174,7 @@ class IframeContentRenderer extends React.Component<Props> {
             // We can't use JSON.stringify/parse for this because the apiOptions
             // includes the functions GroupMetadataEditor, groupAnnotator,
             // onFocusChange, and onInputError.
-            // @ts-expect-error [FEI-5003] - TS2339 - Property 'iframeDataStore' does not exist on type 'Window & typeof globalThis'.
+            // @ts-expect-error - TS2339 - Property 'iframeDataStore' does not exist on type 'Window & typeof globalThis'.
             window.iframeDataStore[this.iframeID] = data;
             frame.contentWindow.postMessage(this.iframeID, "*");
         }

package/src/item-editor.tsx

@@ -51,7 +51,7 @@ class ItemEditor extends React.Component<Props> {
 
     triggerPreviewUpdate: (newData?: any) => void = (newData: any) => {
         // eslint-disable-next-line react/no-string-refs
-        // @ts-expect-error [FEI-5003] - TS2339 - Property 'sendNewData' does not exist on type 'ReactInstance'.
+        // @ts-expect-error - TS2339 - Property 'sendNewData' does not exist on type 'ReactInstance'.
         this.refs.frame.sendNewData(newData);
     };
 
@@ -67,7 +67,7 @@ class ItemEditor extends React.Component<Props> {
 
     getSaveWarnings: () => any = () => {
         // eslint-disable-next-line react/no-string-refs
-        // @ts-expect-error [FEI-5003] - TS2339 - Property 'getSaveWarnings' does not exist on type 'ReactInstance'.
+        // @ts-expect-error - TS2339 - Property 'getSaveWarnings' does not exist on type 'ReactInstance'.
         return this.refs.questionEditor.getSaveWarnings();
     };
 
@@ -81,10 +81,10 @@ class ItemEditor extends React.Component<Props> {
     } = (options: any) => {
         return {
             // eslint-disable-next-line react/no-string-refs
-            // @ts-expect-error [FEI-5003] - TS2339 - Property 'serialize' does not exist on type 'ReactInstance'.
+            // @ts-expect-error - TS2339 - Property 'serialize' does not exist on type 'ReactInstance'.
             question: this.refs.questionEditor.serialize(options),
             // eslint-disable-next-line react/no-string-refs
-            // @ts-expect-error [FEI-5003] - TS2339 - Property 'serialize' does not exist on type 'ReactInstance'.
+            // @ts-expect-error - TS2339 - Property 'serialize' does not exist on type 'ReactInstance'.
             answerArea: this.refs.itemExtrasEditor.serialize(options),
             itemDataVersion: ITEM_DATA_VERSION,
         };