From e79d3bc3b7070404e090036d1bd4e28d4bed54ca Mon Sep 17 00:00:00 2001
From: Alexander Skvortsov <sasha.skvortsov109@gmail.com>
Date: Sun, 12 Dec 2021 15:41:59 -0500
Subject: [PATCH 01/13] Don't assume `app.session.user` is present

---
 framework/core/js/src/forum/states/NotificationListState.ts | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/framework/core/js/src/forum/states/NotificationListState.ts b/framework/core/js/src/forum/states/NotificationListState.ts
index cfe214c02..75bb07158 100644
--- a/framework/core/js/src/forum/states/NotificationListState.ts
+++ b/framework/core/js/src/forum/states/NotificationListState.ts
@@ -15,7 +15,7 @@ export default class NotificationListState extends PaginatedListState<Notificati
    * Load the next page of notification results.
    */
   load(): Promise<void> {
-    if (app.session.user.newNotificationCount()) {
+    if (app.session.user?.newNotificationCount()) {
       this.pages = [];
       this.location = { page: 1 };
     }
@@ -24,7 +24,7 @@ export default class NotificationListState extends PaginatedListState<Notificati
       return Promise.resolve();
     }
 
-    app.session.user.pushAttributes({ newNotificationCount: 0 });
+    app.session.user?.pushAttributes({ newNotificationCount: 0 });
 
     return super.loadNext();
   }
@@ -35,7 +35,7 @@ export default class NotificationListState extends PaginatedListState<Notificati
   markAllAsRead() {
     if (this.pages.length === 0) return;
 
-    app.session.user.pushAttributes({ unreadNotificationCount: 0 });
+    app.session.user?.pushAttributes({ unreadNotificationCount: 0 });
 
     this.pages.forEach((page) => {
       page.items.forEach((notification) => notification.pushAttributes({ isRead: true }));

From a8db0b7bb24ad70d46ded1581c8187884fee23ef Mon Sep 17 00:00:00 2001
From: Alexander Skvortsov <sasha.skvortsov109@gmail.com>
Date: Sun, 12 Dec 2021 15:42:49 -0500
Subject: [PATCH 02/13] Fix import of `mithril` for `VnodeElementTag` usage

---
 framework/core/js/src/@types/global.d.ts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/framework/core/js/src/@types/global.d.ts b/framework/core/js/src/@types/global.d.ts
index e3ff2fe27..37d6b81e7 100644
--- a/framework/core/js/src/@types/global.d.ts
+++ b/framework/core/js/src/@types/global.d.ts
@@ -21,7 +21,7 @@ declare type KeysOfType<Type extends object, Match> = {
  */
 declare type KeyOfType<Type extends object, Match> = KeysOfType<Type, Match>[keyof Type];
 
-declare type VnodeElementTag<Attrs = Record<string, unknown>, State = Record<string, unknown>> = string | ComponentTypes<Attrs, State>;
+declare type VnodeElementTag<Attrs = Record<string, unknown>, State = Record<string, unknown>> = string | import('mithril').ComponentTypes<Attrs, State>;
 
 /**
  * @deprecated Please import `app` from a namespace instead of using it as a global variable.

From d307331603ae27bc5f205795d613cc50662c1058 Mon Sep 17 00:00:00 2001
From: Alexander Skvortsov <sasha.skvortsov109@gmail.com>
Date: Sun, 12 Dec 2021 15:43:19 -0500
Subject: [PATCH 03/13] extractText from translations where strings expected

---
 framework/core/js/src/admin/components/ExtensionPage.tsx | 3 ++-
 framework/core/js/src/common/utils/abbreviateNumber.ts   | 5 +++--
 framework/core/js/src/forum/ForumApplication.ts          | 8 +++++---
 3 files changed, 10 insertions(+), 6 deletions(-)

diff --git a/framework/core/js/src/admin/components/ExtensionPage.tsx b/framework/core/js/src/admin/components/ExtensionPage.tsx
index fa7f70751..fa6acccaa 100644
--- a/framework/core/js/src/admin/components/ExtensionPage.tsx
+++ b/framework/core/js/src/admin/components/ExtensionPage.tsx
@@ -16,6 +16,7 @@ import RequestError from '../../common/utils/RequestError';
 import { Extension } from '../AdminApplication';
 import { IPageAttrs } from '../../common/components/Page';
 import type Mithril from 'mithril';
+import extractText from '../../common/utils/extractText';
 
 export interface ExtensionPageAttrs extends IPageAttrs {
   id: string;
@@ -156,7 +157,7 @@ export default class ExtensionPage<Attrs extends ExtensionPageAttrs = ExtensionP
 
     if (!this.isEnabled()) {
       const purge = () => {
-        if (confirm(app.translator.trans('core.admin.extension.confirm_purge'))) {
+        if (confirm(extractText(app.translator.trans('core.admin.extension.confirm_purge')))) {
           app
             .request({
               url: app.forum.attribute('apiUrl') + '/extensions/' + this.extension.id,
diff --git a/framework/core/js/src/common/utils/abbreviateNumber.ts b/framework/core/js/src/common/utils/abbreviateNumber.ts
index 9f3c5366e..43586876a 100644
--- a/framework/core/js/src/common/utils/abbreviateNumber.ts
+++ b/framework/core/js/src/common/utils/abbreviateNumber.ts
@@ -1,4 +1,5 @@
 import app from '../../common/app';
+import extractText from './extractText';
 
 /**
  * The `abbreviateNumber` utility converts a number to a shorter localized form.
@@ -10,9 +11,9 @@ import app from '../../common/app';
 export default function abbreviateNumber(number: number): string {
   // TODO: translation
   if (number >= 1000000) {
-    return Math.floor(number / 1000000) + app.translator.trans('core.lib.number_suffix.mega_text');
+    return Math.floor(number / 1000000) + extractText(app.translator.trans('core.lib.number_suffix.mega_text'));
   } else if (number >= 1000) {
-    return (number / 1000).toFixed(1) + app.translator.trans('core.lib.number_suffix.kilo_text');
+    return (number / 1000).toFixed(1) + extractText(app.translator.trans('core.lib.number_suffix.kilo_text'));
   } else {
     return number.toString();
   }
diff --git a/framework/core/js/src/forum/ForumApplication.ts b/framework/core/js/src/forum/ForumApplication.ts
index 169e28135..3c88f4e1e 100644
--- a/framework/core/js/src/forum/ForumApplication.ts
+++ b/framework/core/js/src/forum/ForumApplication.ts
@@ -23,6 +23,7 @@ import isSafariMobile from './utils/isSafariMobile';
 import type Notification from './components/Notification';
 import type Post from './components/Post';
 import Discussion from '../common/models/Discussion';
+import extractText from '../common/utils/extractText';
 
 export default class ForumApplication extends Application {
   /**
@@ -99,7 +100,7 @@ export default class ForumApplication extends Application {
     }
 
     this.routes[defaultAction].path = '/';
-    this.history.push(defaultAction, this.translator.trans('core.forum.header.back_to_index_tooltip'), '/');
+    this.history.push(defaultAction, extractText(this.translator.trans('core.forum.header.back_to_index_tooltip')), '/');
 
     this.pane = new Pane(document.getElementById('app'));
 
@@ -124,8 +125,9 @@ export default class ForumApplication extends Application {
       app.history.home();
 
       // Reload the current user so that their unread notification count is refreshed.
-      if (app.session.user) {
-        app.store.find('users', app.session.user.id());
+      const userId = app.session.user?.id()
+      if (userId) {
+        app.store.find('users', userId);
         m.redraw();
       }
     });

From 0e2053da1f7e1b9e1eb63221a6a186bb6fee355e Mon Sep 17 00:00:00 2001
From: Alexander Skvortsov <sasha.skvortsov109@gmail.com>
Date: Sun, 12 Dec 2021 15:43:37 -0500
Subject: [PATCH 04/13] Allow any Mithril Children where appropriate

---
 framework/core/js/src/admin/components/AdminPage.tsx    | 4 ++--
 framework/core/js/src/admin/components/UserListPage.tsx | 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/framework/core/js/src/admin/components/AdminPage.tsx b/framework/core/js/src/admin/components/AdminPage.tsx
index 981a246cc..8a3b4bc1f 100644
--- a/framework/core/js/src/admin/components/AdminPage.tsx
+++ b/framework/core/js/src/admin/components/AdminPage.tsx
@@ -13,8 +13,8 @@ import generateElementId from '../utils/generateElementId';
 import ColorPreviewInput from '../../common/components/ColorPreviewInput';
 
 export interface AdminHeaderOptions {
-  title: string;
-  description: string;
+  title: Mithril.Children;
+  description: Mithril.Children;
   icon: string;
   /**
    * Will be used as the class for the AdminPage.
diff --git a/framework/core/js/src/admin/components/UserListPage.tsx b/framework/core/js/src/admin/components/UserListPage.tsx
index 675104aea..3502a24f9 100644
--- a/framework/core/js/src/admin/components/UserListPage.tsx
+++ b/framework/core/js/src/admin/components/UserListPage.tsx
@@ -21,7 +21,7 @@ type ColumnData = {
   /**
    * Column title
    */
-  name: String;
+  name: Mithril.Children;
   /**
    * Component(s) to show for this column.
    */

From 74f223cccc7c3e689e8497bb9ce71044652a2279 Mon Sep 17 00:00:00 2001
From: Alexander Skvortsov <sasha.skvortsov109@gmail.com>
Date: Sun, 12 Dec 2021 15:43:50 -0500
Subject: [PATCH 05/13] Fix `LoadingModal` attr typings

---
 framework/core/js/src/admin/components/LoadingModal.tsx | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/framework/core/js/src/admin/components/LoadingModal.tsx b/framework/core/js/src/admin/components/LoadingModal.tsx
index 2a9555d88..5594cb278 100644
--- a/framework/core/js/src/admin/components/LoadingModal.tsx
+++ b/framework/core/js/src/admin/components/LoadingModal.tsx
@@ -1,7 +1,7 @@
 import app from '../../admin/app';
-import Modal from '../../common/components/Modal';
+import Modal, { IInternalModalAttrs } from '../../common/components/Modal';
 
-export default class LoadingModal<ModalAttrs = {}> extends Modal<ModalAttrs> {
+export default class LoadingModal<ModalAttrs extends IInternalModalAttrs = IInternalModalAttrs> extends Modal<ModalAttrs> {
   /**
    * @inheritdoc
    */

From da6ae898b2c2b264baab720f9bc3817a53a1289d Mon Sep 17 00:00:00 2001
From: Alexander Skvortsov <sasha.skvortsov109@gmail.com>
Date: Sun, 12 Dec 2021 15:44:09 -0500
Subject: [PATCH 06/13] `alertAttrs` can be null

---
 framework/core/js/src/common/components/Modal.tsx | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/framework/core/js/src/common/components/Modal.tsx b/framework/core/js/src/common/components/Modal.tsx
index 24594f5a7..a6e17f4d5 100644
--- a/framework/core/js/src/common/components/Modal.tsx
+++ b/framework/core/js/src/common/components/Modal.tsx
@@ -30,7 +30,7 @@ export default abstract class Modal<ModalAttrs extends IInternalModalAttrs = IIn
   /**
    * Attributes for an alert component to show below the header.
    */
-  alertAttrs!: AlertAttrs;
+  alertAttrs: AlertAttrs | null = null;
 
   oninit(vnode: Mithril.VnodeDOM<ModalAttrs, this>) {
     super.oninit(vnode);
@@ -122,7 +122,7 @@ export default abstract class Modal<ModalAttrs extends IInternalModalAttrs = IIn
   /**
    * Get the title of the modal dialog.
    */
-  abstract title(): string;
+  abstract title(): Mithril.Children;
 
   /**
    * Get the content of the modal.

From e1c6028dce9aa96bac8fc5682c719667dea8d433 Mon Sep 17 00:00:00 2001
From: Alexander Skvortsov <sasha.skvortsov109@gmail.com>
Date: Sun, 12 Dec 2021 18:36:51 -0500
Subject: [PATCH 07/13] Modal typescript cleanup and conversions

---
 .../js/src/admin/components/LoadingModal.tsx  |  4 +-
 .../{ReadmeModal.js => ReadmeModal.tsx}       | 16 +++-
 framework/core/js/src/common/Model.ts         |  4 +-
 framework/core/js/src/common/Session.ts       |  5 +-
 .../{EditUserModal.js => EditUserModal.tsx}   | 93 ++++++++++++-------
 .../core/js/src/common/components/Modal.tsx   |  2 +-
 ...estErrorModal.js => RequestErrorModal.tsx} | 20 ++--
 .../core/js/src/forum/ForumApplication.ts     |  2 +-
 ...sswordModal.js => ForgotPasswordModal.tsx} | 43 ++++-----
 .../{LogInModal.js => LogInModal.tsx}         | 71 +++++++-------
 .../{SignUpModal.js => SignUpModal.tsx}       | 84 ++++++++---------
 11 files changed, 182 insertions(+), 162 deletions(-)
 rename framework/core/js/src/admin/components/{ReadmeModal.js => ReadmeModal.tsx} (70%)
 rename framework/core/js/src/common/components/{EditUserModal.js => EditUserModal.tsx} (67%)
 rename framework/core/js/src/common/components/{RequestErrorModal.js => RequestErrorModal.tsx} (59%)
 rename framework/core/js/src/forum/components/{ForgotPasswordModal.js => ForgotPasswordModal.tsx} (78%)
 rename framework/core/js/src/forum/components/{LogInModal.js => LogInModal.tsx} (79%)
 rename framework/core/js/src/forum/components/{SignUpModal.js => SignUpModal.tsx} (75%)

diff --git a/framework/core/js/src/admin/components/LoadingModal.tsx b/framework/core/js/src/admin/components/LoadingModal.tsx
index 5594cb278..082efd002 100644
--- a/framework/core/js/src/admin/components/LoadingModal.tsx
+++ b/framework/core/js/src/admin/components/LoadingModal.tsx
@@ -1,7 +1,9 @@
 import app from '../../admin/app';
 import Modal, { IInternalModalAttrs } from '../../common/components/Modal';
 
-export default class LoadingModal<ModalAttrs extends IInternalModalAttrs = IInternalModalAttrs> extends Modal<ModalAttrs> {
+export interface ILoadingModalAttrs extends IInternalModalAttrs {}
+
+export default class LoadingModal<ModalAttrs extends ILoadingModalAttrs = ILoadingModalAttrs> extends Modal<ModalAttrs> {
   /**
    * @inheritdoc
    */
diff --git a/framework/core/js/src/admin/components/ReadmeModal.js b/framework/core/js/src/admin/components/ReadmeModal.tsx
similarity index 70%
rename from framework/core/js/src/admin/components/ReadmeModal.js
rename to framework/core/js/src/admin/components/ReadmeModal.tsx
index 425e5aac2..f932de229 100644
--- a/framework/core/js/src/admin/components/ReadmeModal.js
+++ b/framework/core/js/src/admin/components/ReadmeModal.tsx
@@ -1,11 +1,21 @@
 import app from '../../admin/app';
-import Modal from '../../common/components/Modal';
+import Modal, { IInternalModalAttrs } from '../../common/components/Modal';
 import LoadingIndicator from '../../common/components/LoadingIndicator';
 import Placeholder from '../../common/components/Placeholder';
 import ExtensionReadme from '../models/ExtensionReadme';
+import type Mithril from 'mithril';
+import { Extension } from '../AdminApplication';
 
-export default class ReadmeModal extends Modal {
-  oninit(vnode) {
+export interface IReadmeModalAttrs extends IInternalModalAttrs {
+  extension: Extension;
+}
+
+export default class ReadmeModal<CustomAttrs extends IReadmeModalAttrs = IReadmeModalAttrs> extends Modal<CustomAttrs> {
+  protected name!: string;
+  protected extName!: string;
+  protected readme!: ExtensionReadme;
+
+  oninit(vnode: Mithril.Vnode<CustomAttrs, this>) {
     super.oninit(vnode);
 
     app.store.models['extension-readmes'] = ExtensionReadme;
diff --git a/framework/core/js/src/common/Model.ts b/framework/core/js/src/common/Model.ts
index 1bc29431e..9fce10624 100644
--- a/framework/core/js/src/common/Model.ts
+++ b/framework/core/js/src/common/Model.ts
@@ -32,11 +32,11 @@ export interface SavedModelData {
 
 export type ModelData = UnsavedModelData | SavedModelData;
 
-interface SaveRelationships {
+export interface SaveRelationships {
   [relationship: string]: Model | Model[];
 }
 
-interface SaveAttributes {
+export interface SaveAttributes {
   [key: string]: unknown;
   relationships?: SaveRelationships;
 }
diff --git a/framework/core/js/src/common/Session.ts b/framework/core/js/src/common/Session.ts
index 0997bf99f..e4e15a6dc 100644
--- a/framework/core/js/src/common/Session.ts
+++ b/framework/core/js/src/common/Session.ts
@@ -7,11 +7,8 @@ export type LoginParams = {
    * The username/email
    */
   identification: string;
-
-  /**
-   * Password
-   */
   password: string;
+  remember: boolean;
 };
 
 /**
diff --git a/framework/core/js/src/common/components/EditUserModal.js b/framework/core/js/src/common/components/EditUserModal.tsx
similarity index 67%
rename from framework/core/js/src/common/components/EditUserModal.js
rename to framework/core/js/src/common/components/EditUserModal.tsx
index 70b3ca9c5..01083ec76 100644
--- a/framework/core/js/src/common/components/EditUserModal.js
+++ b/framework/core/js/src/common/components/EditUserModal.tsx
@@ -1,17 +1,28 @@
 import app from '../../common/app';
-import Modal from './Modal';
+import Modal, { IInternalModalAttrs } from './Modal';
 import Button from './Button';
 import GroupBadge from './GroupBadge';
 import Group from '../models/Group';
 import extractText from '../utils/extractText';
 import ItemList from '../utils/ItemList';
 import Stream from '../utils/Stream';
+import Mithril from 'mithril';
+import User from '../models/User';
+import { SaveAttributes, SaveRelationships } from '../Model';
 
-/**
- * The `EditUserModal` component displays a modal dialog with a login form.
- */
-export default class EditUserModal extends Modal {
-  oninit(vnode) {
+export interface IEditUserModalAttrs extends IInternalModalAttrs {
+  user: User;
+}
+
+export default class EditUserModal<CustomAttrs extends IEditUserModalAttrs = IEditUserModalAttrs> extends Modal<CustomAttrs> {
+  protected username!: Stream<string>;
+  protected email!: Stream<string>;
+  protected isEmailConfirmed!: Stream<boolean>;
+  protected setPassword!: Stream<boolean>;
+  protected password!: Stream<string>;
+  protected groups: Record<string, Stream<boolean>> = {};
+
+  oninit(vnode: Mithril.Vnode<CustomAttrs, this>) {
     super.oninit(vnode);
 
     const user = this.attrs.user;
@@ -19,14 +30,15 @@ export default class EditUserModal extends Modal {
     this.username = Stream(user.username() || '');
     this.email = Stream(user.email() || '');
     this.isEmailConfirmed = Stream(user.isEmailConfirmed() || false);
-    this.setPassword = Stream(false);
+    this.setPassword = Stream(false as boolean);
     this.password = Stream(user.password() || '');
-    this.groups = {};
+
+    const userGroups = user.groups() || [];
 
     app.store
-      .all('groups')
-      .filter((group) => [Group.GUEST_ID, Group.MEMBER_ID].indexOf(group.id()) === -1)
-      .forEach((group) => (this.groups[group.id()] = Stream(user.groups().indexOf(group) !== -1)));
+      .all<Group>('groups')
+      .filter((group) => ![Group.GUEST_ID, Group.MEMBER_ID].includes(group.id()!))
+      .forEach((group) => (this.groups[group.id()!] = Stream(userGroups.includes(group))));
   }
 
   className() {
@@ -49,7 +61,7 @@ export default class EditUserModal extends Modal {
   fields() {
     const items = new ItemList();
 
-    if (app.session.user.canEditCredentials()) {
+    if (app.session.user?.canEditCredentials()) {
       items.add(
         'username',
         <div className="Form-group">
@@ -103,10 +115,11 @@ export default class EditUserModal extends Modal {
               <label className="checkbox">
                 <input
                   type="checkbox"
-                  onchange={(e) => {
-                    this.setPassword(e.target.checked);
+                  onchange={(e: KeyboardEvent) => {
+                    const target = e.target as HTMLInputElement;
+                    this.setPassword(target.checked);
                     m.redraw.sync();
-                    if (e.target.checked) this.$('[name=password]').select();
+                    if (target.checked) this.$('[name=password]').select();
                     e.redraw = false;
                   }}
                   disabled={this.nonAdminEditingAdmin()}
@@ -132,24 +145,31 @@ export default class EditUserModal extends Modal {
       }
     }
 
-    if (app.session.user.canEditGroups()) {
+    if (app.session.user?.canEditGroups()) {
       items.add(
         'groups',
         <div className="Form-group EditUserModal-groups">
           <label>{app.translator.trans('core.lib.edit_user.groups_heading')}</label>
           <div>
             {Object.keys(this.groups)
-              .map((id) => app.store.getById('groups', id))
-              .map((group) => (
-                <label className="checkbox">
-                  <input
-                    type="checkbox"
-                    bidi={this.groups[group.id()]}
-                    disabled={group.id() === Group.ADMINISTRATOR_ID && (this.attrs.user === app.session.user || !this.userIsAdmin(app.session.user))}
-                  />
-                  {GroupBadge.component({ group, label: '' })} {group.nameSingular()}
-                </label>
-              ))}
+              .map((id) => app.store.getById<Group>('groups', id))
+              .filter(Boolean)
+              .map(
+                (group) =>
+                  // Necessary because filter(Boolean) doesn't narrow out falsy values.
+                  group && (
+                    <label className="checkbox">
+                      <input
+                        type="checkbox"
+                        bidi={this.groups[group.id()!]}
+                        disabled={
+                          group.id() === Group.ADMINISTRATOR_ID && (this.attrs.user === app.session.user || !this.userIsAdmin(app.session.user))
+                        }
+                      />
+                      {GroupBadge.component({ group, label: '' })} {group.nameSingular()}
+                    </label>
+                  )
+              )}
           </div>
         </div>,
         10
@@ -194,9 +214,8 @@ export default class EditUserModal extends Modal {
   }
 
   data() {
-    const data = {
-      relationships: {},
-    };
+    const data: SaveAttributes = {};
+    const relationships: SaveRelationships = {};
 
     if (this.attrs.user.canEditCredentials() && !this.nonAdminEditingAdmin()) {
       data.username = this.username();
@@ -211,15 +230,18 @@ export default class EditUserModal extends Modal {
     }
 
     if (this.attrs.user.canEditGroups()) {
-      data.relationships.groups = Object.keys(this.groups)
+      relationships.groups = Object.keys(this.groups)
         .filter((id) => this.groups[id]())
-        .map((id) => app.store.getById('groups', id));
+        .map((id) => app.store.getById<Group>('groups', id))
+        .filter((g): g is Group => g instanceof Group);
     }
 
+    data.relationships = relationships;
+
     return data;
   }
 
-  onsubmit(e) {
+  onsubmit(e: SubmitEvent) {
     e.preventDefault();
 
     this.loading = true;
@@ -239,9 +261,8 @@ export default class EditUserModal extends Modal {
 
   /**
    * @internal
-   * @protected
    */
-  userIsAdmin(user) {
-    return user.groups().some((g) => g.id() === Group.ADMINISTRATOR_ID);
+  protected userIsAdmin(user: User | null) {
+    return user && (user.groups() || []).some((g) => g?.id() === Group.ADMINISTRATOR_ID);
   }
 }
diff --git a/framework/core/js/src/common/components/Modal.tsx b/framework/core/js/src/common/components/Modal.tsx
index a6e17f4d5..b8656860d 100644
--- a/framework/core/js/src/common/components/Modal.tsx
+++ b/framework/core/js/src/common/components/Modal.tsx
@@ -32,7 +32,7 @@ export default abstract class Modal<ModalAttrs extends IInternalModalAttrs = IIn
    */
   alertAttrs: AlertAttrs | null = null;
 
-  oninit(vnode: Mithril.VnodeDOM<ModalAttrs, this>) {
+  oninit(vnode: Mithril.Vnode<ModalAttrs, this>) {
     super.oninit(vnode);
 
     // TODO: [Flarum 2.0] Remove the code below.
diff --git a/framework/core/js/src/common/components/RequestErrorModal.js b/framework/core/js/src/common/components/RequestErrorModal.tsx
similarity index 59%
rename from framework/core/js/src/common/components/RequestErrorModal.js
rename to framework/core/js/src/common/components/RequestErrorModal.tsx
index 297f2a462..f624e1c17 100644
--- a/framework/core/js/src/common/components/RequestErrorModal.js
+++ b/framework/core/js/src/common/components/RequestErrorModal.tsx
@@ -1,6 +1,12 @@
-import Modal from './Modal';
+import RequestError from '../utils/RequestError';
+import Modal, { IInternalModalAttrs } from './Modal';
 
-export default class RequestErrorModal extends Modal {
+export interface IRequestErrorModalAttrs extends IInternalModalAttrs {
+  error: RequestError;
+  formattedError: string[];
+};
+
+export default class RequestErrorModal<CustomAttrs extends IRequestErrorModalAttrs = IRequestErrorModalAttrs> extends Modal<CustomAttrs> {
   className() {
     return 'RequestErrorModal Modal--large';
   }
@@ -18,14 +24,10 @@ export default class RequestErrorModal extends Modal {
     // else try to parse it as JSON and stringify it with indentation
     if (formattedError) {
       responseText = formattedError.join('\n\n');
+    } else if (error.response) {
+      responseText = JSON.stringify(error.response, null, 2);
     } else {
-      try {
-        const json = error.response || JSON.parse(error.responseText);
-
-        responseText = JSON.stringify(json, null, 2);
-      } catch (e) {
-        responseText = error.responseText;
-      }
+      responseText = error.responseText;
     }
 
     return (
diff --git a/framework/core/js/src/forum/ForumApplication.ts b/framework/core/js/src/forum/ForumApplication.ts
index 3c88f4e1e..2566d2766 100644
--- a/framework/core/js/src/forum/ForumApplication.ts
+++ b/framework/core/js/src/forum/ForumApplication.ts
@@ -125,7 +125,7 @@ export default class ForumApplication extends Application {
       app.history.home();
 
       // Reload the current user so that their unread notification count is refreshed.
-      const userId = app.session.user?.id()
+      const userId = app.session.user?.id();
       if (userId) {
         app.store.find('users', userId);
         m.redraw();
diff --git a/framework/core/js/src/forum/components/ForgotPasswordModal.js b/framework/core/js/src/forum/components/ForgotPasswordModal.tsx
similarity index 78%
rename from framework/core/js/src/forum/components/ForgotPasswordModal.js
rename to framework/core/js/src/forum/components/ForgotPasswordModal.tsx
index 64dc67c2f..4c11d478e 100644
--- a/framework/core/js/src/forum/components/ForgotPasswordModal.js
+++ b/framework/core/js/src/forum/components/ForgotPasswordModal.tsx
@@ -1,34 +1,31 @@
 import app from '../../forum/app';
-import Modal from '../../common/components/Modal';
+import Modal, { IInternalModalAttrs } from '../../common/components/Modal';
 import Button from '../../common/components/Button';
 import extractText from '../../common/utils/extractText';
 import Stream from '../../common/utils/Stream';
+import Mithril from 'mithril';
+import RequestError from '../../common/utils/RequestError';
+
+export interface IForgotPasswordModalAttrs extends IInternalModalAttrs {
+  email?: string;
+}
 
 /**
  * The `ForgotPasswordModal` component displays a modal which allows the user to
  * enter their email address and request a link to reset their password.
- *
- * ### Attrs
- *
- * - `email`
  */
-export default class ForgotPasswordModal extends Modal {
-  oninit(vnode) {
+export default class ForgotPasswordModal<CustomAttrs extends IForgotPasswordModalAttrs = IForgotPasswordModalAttrs> extends Modal<CustomAttrs> {
+  /**
+   * The value of the email input.
+   */
+  email!: Stream<string>;
+
+  success: boolean = false;
+
+  oninit(vnode: Mithril.Vnode<CustomAttrs, this>) {
     super.oninit(vnode);
 
-    /**
-     * The value of the email input.
-     *
-     * @type {Function}
-     */
     this.email = Stream(this.attrs.email || '');
-
-    /**
-     * Whether or not the password reset email was sent successfully.
-     *
-     * @type {Boolean}
-     */
-    this.success = false;
   }
 
   className() {
@@ -84,7 +81,7 @@ export default class ForgotPasswordModal extends Modal {
     );
   }
 
-  onsubmit(e) {
+  onsubmit(e: SubmitEvent) {
     e.preventDefault();
 
     this.loading = true;
@@ -98,14 +95,14 @@ export default class ForgotPasswordModal extends Modal {
       })
       .then(() => {
         this.success = true;
-        this.alert = null;
+        this.alertAttrs = null;
       })
       .catch(() => {})
       .then(this.loaded.bind(this));
   }
 
-  onerror(error) {
-    if (error.status === 404) {
+  onerror(error: RequestError) {
+    if (error.status === 404 && error.alert) {
       error.alert.content = app.translator.trans('core.forum.forgot_password.not_found_message');
     }
 
diff --git a/framework/core/js/src/forum/components/LogInModal.js b/framework/core/js/src/forum/components/LogInModal.tsx
similarity index 79%
rename from framework/core/js/src/forum/components/LogInModal.js
rename to framework/core/js/src/forum/components/LogInModal.tsx
index 6e2097d34..cbbe3c723 100644
--- a/framework/core/js/src/forum/components/LogInModal.js
+++ b/framework/core/js/src/forum/components/LogInModal.tsx
@@ -1,5 +1,5 @@
 import app from '../../forum/app';
-import Modal from '../../common/components/Modal';
+import Modal, { IInternalModalAttrs } from '../../common/components/Modal';
 import ForgotPasswordModal from './ForgotPasswordModal';
 import SignUpModal from './SignUpModal';
 import Button from '../../common/components/Button';
@@ -7,38 +7,34 @@ import LogInButtons from './LogInButtons';
 import extractText from '../../common/utils/extractText';
 import ItemList from '../../common/utils/ItemList';
 import Stream from '../../common/utils/Stream';
+import type Mithril from 'mithril';
+import RequestError from '../../common/utils/RequestError';
 
-/**
- * The `LogInModal` component displays a modal dialog with a login form.
- *
- * ### Attrs
- *
- * - `identification`
- * - `password`
- */
-export default class LogInModal extends Modal {
-  oninit(vnode) {
+export interface ILoginModalAttrs extends IInternalModalAttrs {
+  identification?: string;
+  password?: string;
+  remember?: boolean;
+}
+
+export default class LogInModal<CustomAttrs extends ILoginModalAttrs = ILoginModalAttrs> extends Modal<CustomAttrs> {
+  /**
+   * The value of the identification input.
+   */
+  identification!: Stream<string>;
+  /**
+   * The value of the password input.
+   */
+  password!: Stream<string>;
+  /**
+   * The value of the remember me input.
+   */
+  remember!: Stream<boolean>;
+
+  oninit(vnode: Mithril.Vnode<CustomAttrs, this>) {
     super.oninit(vnode);
 
-    /**
-     * The value of the identification input.
-     *
-     * @type {Function}
-     */
     this.identification = Stream(this.attrs.identification || '');
-
-    /**
-     * The value of the password input.
-     *
-     * @type {Function}
-     */
     this.password = Stream(this.attrs.password || '');
-
-    /**
-     * The value of the remember me input.
-     *
-     * @type {Function}
-     */
     this.remember = Stream(!!this.attrs.remember);
   }
 
@@ -140,12 +136,10 @@ export default class LogInModal extends Modal {
   /**
    * Open the forgot password modal, prefilling it with an email if the user has
    * entered one.
-   *
-   * @public
    */
   forgotPassword() {
     const email = this.identification();
-    const attrs = email.indexOf('@') !== -1 ? { email } : undefined;
+    const attrs = email.includes('@') ? { email } : undefined;
 
     app.modal.show(ForgotPasswordModal, attrs);
   }
@@ -153,13 +147,14 @@ export default class LogInModal extends Modal {
   /**
    * Open the sign up modal, prefilling it with an email/username/password if
    * the user has entered one.
-   *
-   * @public
    */
   signUp() {
-    const attrs = { password: this.password() };
     const identification = this.identification();
-    attrs[identification.indexOf('@') !== -1 ? 'email' : 'username'] = identification;
+
+    const attrs = {
+      [identification.includes('@') ? 'email' : 'username']: identification,
+      password: this.password(),
+    };
 
     app.modal.show(SignUpModal, attrs);
   }
@@ -168,7 +163,7 @@ export default class LogInModal extends Modal {
     this.$('[name=' + (this.identification() ? 'password' : 'identification') + ']').select();
   }
 
-  onsubmit(e) {
+  onsubmit(e: SubmitEvent) {
     e.preventDefault();
 
     this.loading = true;
@@ -182,8 +177,8 @@ export default class LogInModal extends Modal {
       .then(() => window.location.reload(), this.loaded.bind(this));
   }
 
-  onerror(error) {
-    if (error.status === 401) {
+  onerror(error: RequestError) {
+    if (error.status === 401 && error.alert) {
       error.alert.content = app.translator.trans('core.forum.log_in.invalid_login_message');
     }
 
diff --git a/framework/core/js/src/forum/components/SignUpModal.js b/framework/core/js/src/forum/components/SignUpModal.tsx
similarity index 75%
rename from framework/core/js/src/forum/components/SignUpModal.js
rename to framework/core/js/src/forum/components/SignUpModal.tsx
index 53593134f..95fdd9b64 100644
--- a/framework/core/js/src/forum/components/SignUpModal.js
+++ b/framework/core/js/src/forum/components/SignUpModal.tsx
@@ -1,45 +1,47 @@
 import app from '../../forum/app';
-import Modal from '../../common/components/Modal';
+import Modal, { IInternalModalAttrs } from '../../common/components/Modal';
 import LogInModal from './LogInModal';
 import Button from '../../common/components/Button';
 import LogInButtons from './LogInButtons';
 import extractText from '../../common/utils/extractText';
 import ItemList from '../../common/utils/ItemList';
 import Stream from '../../common/utils/Stream';
+import type Mithril from 'mithril';
 
-/**
- * The `SignUpModal` component displays a modal dialog with a singup form.
- *
- * ### Attrs
- *
- * - `username`
- * - `email`
- * - `password`
- * - `token` An email token to sign up with.
- */
-export default class SignUpModal extends Modal {
-  oninit(vnode) {
+export interface ISignupModalAttrs extends IInternalModalAttrs {
+  username?: string;
+  email?: string;
+  password?: string;
+  token?: string;
+  provided?: string[];
+}
+
+export type SignupBody = {
+  username: string;
+  email: string;
+} & ({ token: string } | { password: string });
+
+export default class SignUpModal<CustomAttrs extends ISignupModalAttrs = ISignupModalAttrs> extends Modal<CustomAttrs> {
+  /**
+   * The value of the username input.
+   */
+  username!: Stream<string>;
+
+  /**
+   * The value of the email input.
+   */
+  email!: Stream<string>;
+
+  /**
+   * The value of the password input.
+   */
+  password!: Stream<string>;
+
+  oninit(vnode: Mithril.Vnode<CustomAttrs, this>) {
     super.oninit(vnode);
 
-    /**
-     * The value of the username input.
-     *
-     * @type {Function}
-     */
     this.username = Stream(this.attrs.username || '');
-
-    /**
-     * The value of the email input.
-     *
-     * @type {Function}
-     */
     this.email = Stream(this.attrs.email || '');
-
-    /**
-     * The value of the password input.
-     *
-     * @type {Function}
-     */
     this.password = Stream(this.attrs.password || '');
   }
 
@@ -55,12 +57,12 @@ export default class SignUpModal extends Modal {
     return [<div className="Modal-body">{this.body()}</div>, <div className="Modal-footer">{this.footer()}</div>];
   }
 
-  isProvided(field) {
-    return this.attrs.provided && this.attrs.provided.indexOf(field) !== -1;
+  isProvided(field: string): boolean {
+    return this.attrs.provided?.includes(field) ?? false;
   }
 
   body() {
-    return [this.attrs.token ? '' : <LogInButtons />, <div className="Form Form--centered">{this.fields().toArray()}</div>];
+    return [!this.attrs.token && <LogInButtons />, <div className="Form Form--centered">{this.fields().toArray()}</div>];
   }
 
   fields() {
@@ -156,7 +158,7 @@ export default class SignUpModal extends Modal {
     }
   }
 
-  onsubmit(e) {
+  onsubmit(e: SubmitEvent) {
     e.preventDefault();
 
     this.loading = true;
@@ -175,22 +177,16 @@ export default class SignUpModal extends Modal {
 
   /**
    * Get the data that should be submitted in the sign-up request.
-   *
-   * @return {Object}
-   * @protected
    */
-  submitData() {
+  submitData(): SignupBody {
+    const authData = this.attrs.token ? { token: this.attrs.token } : { password: this.password() };
+
     const data = {
       username: this.username(),
       email: this.email(),
+      ...authData,
     };
 
-    if (this.attrs.token) {
-      data.token = this.attrs.token;
-    } else {
-      data.password = this.password();
-    }
-
     return data;
   }
 }

From 959662f2e2e2428a00dc6c83b23e3d82708a595b Mon Sep 17 00:00:00 2001
From: Alexander Skvortsov <sasha.skvortsov109@gmail.com>
Date: Sun, 12 Dec 2021 18:38:19 -0500
Subject: [PATCH 08/13] Format VnodeElementTag fix

---
 framework/core/js/src/@types/global.d.ts | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/framework/core/js/src/@types/global.d.ts b/framework/core/js/src/@types/global.d.ts
index 37d6b81e7..7e537b1ec 100644
--- a/framework/core/js/src/@types/global.d.ts
+++ b/framework/core/js/src/@types/global.d.ts
@@ -21,7 +21,9 @@ declare type KeysOfType<Type extends object, Match> = {
  */
 declare type KeyOfType<Type extends object, Match> = KeysOfType<Type, Match>[keyof Type];
 
-declare type VnodeElementTag<Attrs = Record<string, unknown>, State = Record<string, unknown>> = string | import('mithril').ComponentTypes<Attrs, State>;
+declare type VnodeElementTag<Attrs = Record<string, unknown>, State = Record<string, unknown>> =
+  | string
+  | import('mithril').ComponentTypes<Attrs, State>;
 
 /**
  * @deprecated Please import `app` from a namespace instead of using it as a global variable.

From 7b8d507139e444f8986c3f9bb776cf10b1c7fb4c Mon Sep 17 00:00:00 2001
From: Alexander Skvortsov <sasha.skvortsov109@gmail.com>
Date: Sun, 12 Dec 2021 22:31:03 -0500
Subject: [PATCH 09/13] `VnodeElementTag` must be either a string or a class.

Unfortunately, TypeScript only supports strings and classes for JSX tags.
Therefore, our type definition should only allow for those two types.
see https://github.com/microsoft/TypeScript/issues/14789#issuecomment-412247771
---
 framework/core/js/src/@types/global.d.ts | 18 +++++++++++++++---
 1 file changed, 15 insertions(+), 3 deletions(-)

diff --git a/framework/core/js/src/@types/global.d.ts b/framework/core/js/src/@types/global.d.ts
index 7e537b1ec..a0d84324f 100644
--- a/framework/core/js/src/@types/global.d.ts
+++ b/framework/core/js/src/@types/global.d.ts
@@ -21,9 +21,21 @@ declare type KeysOfType<Type extends object, Match> = {
  */
 declare type KeyOfType<Type extends object, Match> = KeysOfType<Type, Match>[keyof Type];
 
-declare type VnodeElementTag<Attrs = Record<string, unknown>, State = Record<string, unknown>> =
-  | string
-  | import('mithril').ComponentTypes<Attrs, State>;
+type Component<A> = import('mithril').Component<A>;
+
+declare type ComponentClass< 
+Attrs = Record<string, unknown>, C extends Component<Attrs> = Component<Attrs>> = { new (...args: any[]): Component<Attrs>, prototype: C };
+
+/**
+ * Unfortunately, TypeScript only supports strings and classes for JSX tags.
+ * Therefore, our type definition should only allow for those two types.
+ * 
+ * @see https://github.com/microsoft/TypeScript/issues/14789#issuecomment-412247771
+ */
+declare type VnodeElementTag<
+  Attrs = Record<string, unknown>,
+  C extends Component<Attrs> = Component<Attrs>
+> = string | ComponentClass<Attrs, C>;
 
 /**
  * @deprecated Please import `app` from a namespace instead of using it as a global variable.

From 850d5558661b7fa8fc945e5ad280a7cf787b1c0a Mon Sep 17 00:00:00 2001
From: Alexander Skvortsov <sasha.skvortsov109@gmail.com>
Date: Sun, 12 Dec 2021 22:34:47 -0500
Subject: [PATCH 10/13] Slightly improve AlertManagerState `show` typing

`typeof Alert` is more correct than `Alert`, since we're accepting classes not instances.
---
 framework/core/js/src/common/states/AlertManagerState.ts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/framework/core/js/src/common/states/AlertManagerState.ts b/framework/core/js/src/common/states/AlertManagerState.ts
index 076322ac1..a5e479546 100644
--- a/framework/core/js/src/common/states/AlertManagerState.ts
+++ b/framework/core/js/src/common/states/AlertManagerState.ts
@@ -27,7 +27,7 @@ export default class AlertManagerState {
    */
   show(children: Mithril.Children): AlertIdentifier;
   show(attrs: AlertAttrs, children: Mithril.Children): AlertIdentifier;
-  show(componentClass: Alert, attrs: AlertAttrs, children: Mithril.Children): AlertIdentifier;
+  show(componentClass: typeof Alert, attrs: AlertAttrs, children: Mithril.Children): AlertIdentifier;
 
   show(arg1: any, arg2?: any, arg3?: any) {
     // Assigns variables as per the above signatures

From 7b8eaf9045dcae097efe3eb30e6358e2c31f78b7 Mon Sep 17 00:00:00 2001
From: Alexander Skvortsov <sasha.skvortsov109@gmail.com>
Date: Sun, 12 Dec 2021 22:38:55 -0500
Subject: [PATCH 11/13] Fix typing errors with `app.modal.show`

Unfortunately TypeScript doesn't support higher-kinded types, so we can't write this in a type-safe way.
---
 .../core/js/src/common/states/ModalManagerState.ts | 14 ++++++++++++--
 1 file changed, 12 insertions(+), 2 deletions(-)

diff --git a/framework/core/js/src/common/states/ModalManagerState.ts b/framework/core/js/src/common/states/ModalManagerState.ts
index 401d81b21..44c2be05d 100644
--- a/framework/core/js/src/common/states/ModalManagerState.ts
+++ b/framework/core/js/src/common/states/ModalManagerState.ts
@@ -1,5 +1,15 @@
+import Component from '../Component';
 import Modal from '../components/Modal';
 
+/**
+ * Ideally, `show` would take a higher-kinded generic, ala:
+ *  `show<Attrs, C>(componentClass: C<Attrs>, attrs: Attrs): void`
+ * Unfortunately, TypeScript does not support this:
+ * https://github.com/Microsoft/TypeScript/issues/1213
+ * Therefore, we have to use this ugly, messy workaround.
+ */
+type UnsafeModalClass = ComponentClass<any, Modal> & {isDismissible: boolean, component: typeof Component.component};
+
 /**
  * Class used to manage modal state.
  *
@@ -10,7 +20,7 @@ export default class ModalManagerState {
    * @internal
    */
   modal: null | {
-    componentClass: typeof Modal;
+    componentClass: UnsafeModalClass;
     attrs?: Record<string, unknown>;
   } = null;
 
@@ -28,7 +38,7 @@ export default class ModalManagerState {
    * // This "hack" is needed due to quirks with nested redraws in Mithril.
    * setTimeout(() => app.modal.show(MyCoolModal, { attr: 'value' }), 0);
    */
-  show(componentClass: typeof Modal, attrs: Record<string, unknown> = {}): void {
+  show(componentClass: UnsafeModalClass, attrs: Record<string, unknown> = {}): void {
     if (!(componentClass.prototype instanceof Modal)) {
       // This is duplicated so that if the error is caught, an error message still shows up in the debug console.
       const invalidModalWarning = 'The ModalManager can only show Modals.';

From 2dea2f09a43ebabd2c3d35a149fab7b1dd0df862 Mon Sep 17 00:00:00 2001
From: Alexander Skvortsov <sasha.skvortsov109@gmail.com>
Date: Sun, 12 Dec 2021 22:51:05 -0500
Subject: [PATCH 12/13] Error in GH actions on type failures

---
 framework/core/.github/workflows/js.yml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/framework/core/.github/workflows/js.yml b/framework/core/.github/workflows/js.yml
index 3b44e6d7d..cd1e65982 100644
--- a/framework/core/.github/workflows/js.yml
+++ b/framework/core/.github/workflows/js.yml
@@ -49,7 +49,7 @@ jobs:
         working-directory: ./js
 
       - name: Typecheck
-        run: yarn run check-typings || true  # REMOVE THIS ONCE TYPE SAFETY REACHED
+        run: yarn run check-typings
         working-directory: ./js
 
   type-coverage:

From a82a160ecbbd0d8515506a7bf45741681ffb2916 Mon Sep 17 00:00:00 2001
From: Alexander Skvortsov <sasha.skvortsov109@gmail.com>
Date: Sun, 12 Dec 2021 22:56:22 -0500
Subject: [PATCH 13/13] Format and Type Import Cleanup

Co-authored-by: David Wheatley <hi@davwheat.dev>
---
 framework/core/js/src/@types/global.d.ts            | 13 ++++++-------
 .../core/js/src/admin/components/ReadmeModal.tsx    |  2 +-
 .../core/js/src/common/components/EditUserModal.tsx |  6 +++---
 .../js/src/common/components/RequestErrorModal.tsx  |  4 ++--
 .../core/js/src/common/states/ModalManagerState.ts  |  4 ++--
 5 files changed, 14 insertions(+), 15 deletions(-)

diff --git a/framework/core/js/src/@types/global.d.ts b/framework/core/js/src/@types/global.d.ts
index a0d84324f..26bc910f3 100644
--- a/framework/core/js/src/@types/global.d.ts
+++ b/framework/core/js/src/@types/global.d.ts
@@ -23,19 +23,18 @@ declare type KeyOfType<Type extends object, Match> = KeysOfType<Type, Match>[key
 
 type Component<A> = import('mithril').Component<A>;
 
-declare type ComponentClass< 
-Attrs = Record<string, unknown>, C extends Component<Attrs> = Component<Attrs>> = { new (...args: any[]): Component<Attrs>, prototype: C };
+declare type ComponentClass<Attrs = Record<string, unknown>, C extends Component<Attrs> = Component<Attrs>> = {
+  new (...args: any[]): Component<Attrs>;
+  prototype: C;
+};
 
 /**
  * Unfortunately, TypeScript only supports strings and classes for JSX tags.
  * Therefore, our type definition should only allow for those two types.
- * 
+ *
  * @see https://github.com/microsoft/TypeScript/issues/14789#issuecomment-412247771
  */
-declare type VnodeElementTag<
-  Attrs = Record<string, unknown>,
-  C extends Component<Attrs> = Component<Attrs>
-> = string | ComponentClass<Attrs, C>;
+declare type VnodeElementTag<Attrs = Record<string, unknown>, C extends Component<Attrs> = Component<Attrs>> = string | ComponentClass<Attrs, C>;
 
 /**
  * @deprecated Please import `app` from a namespace instead of using it as a global variable.
diff --git a/framework/core/js/src/admin/components/ReadmeModal.tsx b/framework/core/js/src/admin/components/ReadmeModal.tsx
index f932de229..e2cc9f0bc 100644
--- a/framework/core/js/src/admin/components/ReadmeModal.tsx
+++ b/framework/core/js/src/admin/components/ReadmeModal.tsx
@@ -4,7 +4,7 @@ import LoadingIndicator from '../../common/components/LoadingIndicator';
 import Placeholder from '../../common/components/Placeholder';
 import ExtensionReadme from '../models/ExtensionReadme';
 import type Mithril from 'mithril';
-import { Extension } from '../AdminApplication';
+import type { Extension } from '../AdminApplication';
 
 export interface IReadmeModalAttrs extends IInternalModalAttrs {
   extension: Extension;
diff --git a/framework/core/js/src/common/components/EditUserModal.tsx b/framework/core/js/src/common/components/EditUserModal.tsx
index 01083ec76..994d6b8b3 100644
--- a/framework/core/js/src/common/components/EditUserModal.tsx
+++ b/framework/core/js/src/common/components/EditUserModal.tsx
@@ -6,9 +6,9 @@ import Group from '../models/Group';
 import extractText from '../utils/extractText';
 import ItemList from '../utils/ItemList';
 import Stream from '../utils/Stream';
-import Mithril from 'mithril';
-import User from '../models/User';
-import { SaveAttributes, SaveRelationships } from '../Model';
+import type Mithril from 'mithril';
+import type User from '../models/User';
+import type { SaveAttributes, SaveRelationships } from '../Model';
 
 export interface IEditUserModalAttrs extends IInternalModalAttrs {
   user: User;
diff --git a/framework/core/js/src/common/components/RequestErrorModal.tsx b/framework/core/js/src/common/components/RequestErrorModal.tsx
index f624e1c17..895ef5acc 100644
--- a/framework/core/js/src/common/components/RequestErrorModal.tsx
+++ b/framework/core/js/src/common/components/RequestErrorModal.tsx
@@ -1,10 +1,10 @@
-import RequestError from '../utils/RequestError';
+import type RequestError from '../utils/RequestError';
 import Modal, { IInternalModalAttrs } from './Modal';
 
 export interface IRequestErrorModalAttrs extends IInternalModalAttrs {
   error: RequestError;
   formattedError: string[];
-};
+}
 
 export default class RequestErrorModal<CustomAttrs extends IRequestErrorModalAttrs = IRequestErrorModalAttrs> extends Modal<CustomAttrs> {
   className() {
diff --git a/framework/core/js/src/common/states/ModalManagerState.ts b/framework/core/js/src/common/states/ModalManagerState.ts
index 44c2be05d..425de5150 100644
--- a/framework/core/js/src/common/states/ModalManagerState.ts
+++ b/framework/core/js/src/common/states/ModalManagerState.ts
@@ -1,4 +1,4 @@
-import Component from '../Component';
+import type Component from '../Component';
 import Modal from '../components/Modal';
 
 /**
@@ -8,7 +8,7 @@ import Modal from '../components/Modal';
  * https://github.com/Microsoft/TypeScript/issues/1213
  * Therefore, we have to use this ugly, messy workaround.
  */
-type UnsafeModalClass = ComponentClass<any, Modal> & {isDismissible: boolean, component: typeof Component.component};
+type UnsafeModalClass = ComponentClass<any, Modal> & { isDismissible: boolean; component: typeof Component.component };
 
 /**
  * Class used to manage modal state.