add readonly keyword where applicable - AI

src/ViewModel/Chat/chatListViewModel.ts

@@ -14,7 +14,7 @@ export default class ChatListViewModel {
     notificationViewModel: NotificationViewModel;
 
     // data
-    chatIndexManager = new IndexManager<ChatViewModel>(
+    readonly chatIndexManager = new IndexManager<ChatViewModel>(
         (chatViewModel: ChatViewModel) =>
             chatViewModel.settingsPageViewModel.primaryChannel.value,
     );
@@ -96,9 +96,9 @@ export default class ChatListViewModel {
 
     // init
     constructor(
-        public coreViewModel: CoreViewModel,
-        public settingsViewModel: SettingsViewModel,
-        public connectionViewModel: ConnectionViewModel,
+        public readonly coreViewModel: CoreViewModel,
+        public readonly settingsViewModel: SettingsViewModel,
+        public readonly connectionViewModel: ConnectionViewModel,
     ) {
         this.notificationViewModel = new NotificationViewModel(this);
 

src/ViewModel/Chat/chatMessageViewModel.ts

@@ -123,8 +123,8 @@ export default class ChatMessageViewModel {
 
     // init
     constructor(
-        public coreViewModel: CoreViewModel,
-        public messagePageViewModel: MessagePageViewModel,
+        public readonly coreViewModel: CoreViewModel,
+        public readonly messagePageViewModel: MessagePageViewModel,
         chatMessage: ChatMessage,
         sentByUser: boolean,
     ) {

src/ViewModel/Chat/chatViewModel.ts

@@ -145,12 +145,12 @@ export default class ChatViewModel extends ContextHost<ChatPageTypes> {
 
     // init
     constructor(
-        public coreViewModel: CoreViewModel,
-        public chatModel: ChatModel,
-        public settingsViewModel: SettingsViewModel,
-        public notificationViewModel: NotificationViewModel,
-        public connectionViewModel: ConnectionViewModel,
-        public chatListViewModel: ChatListViewModel,
+        public readonly coreViewModel: CoreViewModel,
+        public readonly chatModel: ChatModel,
+        public readonly settingsViewModel: SettingsViewModel,
+        public readonly notificationViewModel: NotificationViewModel,
+        public readonly connectionViewModel: ConnectionViewModel,
+        public readonly chatListViewModel: ChatListViewModel,
     ) {
         super();
 

src/ViewModel/Global/connectionViewModel.ts

@@ -87,7 +87,7 @@ export default class ConnectionViewModel {
     };
 
     // init
-    constructor(public coreViewModel: CoreViewModel) {
+    constructor(public readonly coreViewModel: CoreViewModel) {
         this.updatePreviousAddresses();
 
         coreViewModel.connectionModel.connectionChangeHandlerManager.addHandler(

src/ViewModel/Global/coreViewModel.ts

@@ -66,11 +66,11 @@ export default class CoreViewModel {
 
     // init
     constructor(
-        public storageModel: StorageModel,
-        public settingsModel: SettingsModel,
-        public connectionModel: ConnectionModel,
-        public chatListModel: ChatListModel,
-        public fileTransferModel: FileTransferModel,
+        public readonly storageModel: StorageModel,
+        public readonly settingsModel: SettingsModel,
+        public readonly connectionModel: ConnectionModel,
+        public readonly chatListModel: ChatListModel,
+        public readonly fileTransferModel: FileTransferModel,
     ) {
         this.translations =
             allTranslations[settingsModel.language] || allTranslations.en;

src/ViewModel/Global/fileTransferViewModel.ts

@@ -13,7 +13,7 @@ import CoreViewModel, { Context } from "./coreViewModel";
 import { CommonKeys } from "../../View/keystrokes";
 
 export default class FileTransferViewModel extends Context {
-    contextDebugDescription: string = "transfer";
+    contextDebugDescription = "transfer";
 
     // state
     presentedModal: React.State<FileTransferModals | undefined> =
@@ -172,7 +172,7 @@ export default class FileTransferViewModel extends Context {
     };
 
     // init
-    constructor(public coreViewModel: CoreViewModel) {
+    constructor(public readonly coreViewModel: CoreViewModel) {
         super();
 
         this.coreViewModel.fileTransferModel.fileHandlerManager.addHandler(

src/ViewModel/Global/notificationViewModel.ts

@@ -76,7 +76,7 @@ export default class NotificationViewModel {
     };
 
     // init
-    constructor(public chatListViewModel: ChatListViewModel) {}
+    constructor(public readonly chatListViewModel: ChatListViewModel) {}
 
     // util
     static createNotification(message: ChatMessage): Notification {

src/ViewModel/Global/settingsViewModel.ts

@@ -5,7 +5,7 @@ import { Languages, ThemeSettings } from "../../Model/Global/settingsModel";
 import { CommonKeys } from "../../View/keystrokes";
 
 export default class SettingsViewModel extends Context {
-    contextDebugDescription: string = "settings";
+    contextDebugDescription = "settings";
 
     // state
     username: React.State<string> = new React.State("");
@@ -72,7 +72,7 @@ export default class SettingsViewModel extends Context {
     };
 
     // init
-    constructor(public coreViewModel: CoreViewModel) {
+    constructor(public readonly coreViewModel: CoreViewModel) {
         super();
 
         this.username.value = coreViewModel.settingsModel.username;

src/ViewModel/Global/storageViewModel.ts

@@ -65,7 +65,7 @@ export default class StorageViewModel extends Context {
     };
 
     // init
-    constructor(public coreViewModel: CoreViewModel) {
+    constructor(public readonly coreViewModel: CoreViewModel) {
         super();
 
         this.selectedFileName = React.createProxyState(

src/ViewModel/Pages/boardViewModel.ts

@@ -16,7 +16,7 @@ import TaskViewModel from "./taskViewModel";
 import { CommonKeys } from "../../View/keystrokes";
 
 export default class BoardViewModel extends TaskContainingPageViewModel {
-    contextDebugDescription: string = "board";
+    contextDebugDescription = "board";
 
     // state
     name: React.State<string> = new React.State("");
@@ -247,11 +247,11 @@ export default class BoardViewModel extends TaskContainingPageViewModel {
 
     // init
     constructor(
-        public coreViewModel: CoreViewModel,
-        public chatViewModel: ChatViewModel,
-        public boardsAndTasksModel: BoardsAndTasksModel,
-        public taskPageViewModel: TaskPageViewModel,
-        public boardInfo: BoardInfoFileContent,
+        public readonly coreViewModel: CoreViewModel,
+        public readonly chatViewModel: ChatViewModel,
+        public readonly boardsAndTasksModel: BoardsAndTasksModel,
+        public readonly taskPageViewModel: TaskPageViewModel,
+        public readonly boardInfo: BoardInfoFileContent,
     ) {
         super(coreViewModel, chatViewModel, boardsAndTasksModel);
 

src/ViewModel/Pages/calendarPageViewModel.ts

@@ -185,10 +185,10 @@ export default class CalendarPageViewModel extends TaskContainingPageViewModel {
 
     // init
     constructor(
-        public coreViewModel: CoreViewModel,
-        public chatViewModel: ChatViewModel,
-        public calendarModel: CalendarModel,
-        public boardsAndTasksModel: BoardsAndTasksModel,
+        public readonly coreViewModel: CoreViewModel,
+        public readonly chatViewModel: ChatViewModel,
+        public readonly calendarModel: CalendarModel,
+        public readonly boardsAndTasksModel: BoardsAndTasksModel,
     ) {
         super(coreViewModel, chatViewModel, boardsAndTasksModel);
 

src/ViewModel/Pages/homeViewModel.ts

@@ -9,11 +9,11 @@ export default class HomeViewModel extends Context {
     contextDebugDescription = "home";
 
     constructor(
-        public coreViewModel: CoreViewModel,
-        public settingsViewModel: SettingsViewModel,
-        public fileTransferViewModel: FileTransferViewModel,
-        public storageViewModel: StorageViewModel,
-        public connectionViewModel: ConnectionViewModel,
+        public readonly coreViewModel: CoreViewModel,
+        public readonly settingsViewModel: SettingsViewModel,
+        public readonly fileTransferViewModel: FileTransferViewModel,
+        public readonly storageViewModel: StorageViewModel,
+        public readonly connectionViewModel: ConnectionViewModel,
     ) {
         super();
 

src/ViewModel/Pages/messagePageViewModel.ts

@@ -127,8 +127,8 @@ export default class MessagePageViewModel extends Context {
 
     // init
     constructor(
-        public coreViewModel: CoreViewModel,
-        public chatViewModel: ChatViewModel,
+        public readonly coreViewModel: CoreViewModel,
+        public readonly chatViewModel: ChatViewModel,
     ) {
         super();
 

src/ViewModel/Pages/settingsPageViewModel.ts

@@ -112,8 +112,8 @@ export default class SettingsPageViewModel extends Context {
 
     // init
     constructor(
-        public coreViewModel: CoreViewModel,
-        public chatViewModel: ChatViewModel,
+        public readonly coreViewModel: CoreViewModel,
+        public readonly chatViewModel: ChatViewModel,
     ) {
         super();
 

src/ViewModel/Pages/taskContainingPageViewModel.ts

@@ -11,7 +11,7 @@ import TaskViewModel from "./taskViewModel";
 
 export default class TaskContainingPageViewModel extends Context {
     // state
-    taskIndexManager = new IndexManager<TaskViewModel>(
+    readonly taskIndexManager = new IndexManager<TaskViewModel>(
         (taskViewModel: TaskViewModel) => taskViewModel.sortingString,
     );
 
@@ -56,9 +56,9 @@ export default class TaskContainingPageViewModel extends Context {
 
     // init
     constructor(
-        public coreViewModel: CoreViewModel,
-        public chatViewModel: ChatViewModel,
-        public boardsAndTasksModel: BoardsAndTasksModel,
+        public readonly coreViewModel: CoreViewModel,
+        public readonly chatViewModel: ChatViewModel,
+        public readonly boardsAndTasksModel: BoardsAndTasksModel,
     ) {
         super();
     }

src/ViewModel/Pages/taskPageViewModel.ts

@@ -11,10 +11,10 @@ import { IndexManager } from "../../Model/Utility/utility";
 import { CommonKeys } from "../../View/keystrokes";
 
 export default class TaskPageViewModel extends ContextHost<string> {
-    contextDebugDescription: string = "task-page";
+    contextDebugDescription = "task-page";
 
     // data
-    boardIndexManager = new IndexManager<BoardViewModel>(
+    readonly boardIndexManager = new IndexManager<BoardViewModel>(
         (boardViewModel: BoardViewModel) => boardViewModel.name.value,
     );
 
@@ -162,9 +162,9 @@ export default class TaskPageViewModel extends ContextHost<string> {
 
     // init
     constructor(
-        public coreViewModel: CoreViewModel,
-        public chatViewModel: ChatViewModel,
-        public boardsAndTasksModel: BoardsAndTasksModel,
+        public readonly coreViewModel: CoreViewModel,
+        public readonly chatViewModel: ChatViewModel,
+        public readonly boardsAndTasksModel: BoardsAndTasksModel,
     ) {
         super();
 

src/ViewModel/Pages/taskViewModel.ts

@@ -210,10 +210,10 @@ export default class TaskViewModel extends Context {
 
     // init
     constructor(
-        public coreViewModel: CoreViewModel,
-        public chatViewModel: ChatViewModel,
-        public boardsAndTasksModel: BoardsAndTasksModel,
-        public containingModel: TaskContainingPageViewModel,
+        public readonly coreViewModel: CoreViewModel,
+        public readonly chatViewModel: ChatViewModel,
+        public readonly boardsAndTasksModel: BoardsAndTasksModel,
+        public readonly containingModel: TaskContainingPageViewModel,
         public task: TaskFileContent,
     ) {
         super();

src/ViewModel/Utility/taskPropertyBulkChangeViewModel.ts

@@ -19,7 +19,7 @@ export default class TaskPropertyBulkChangeViewModel {
         });
     };
 
-    setValue: (newValue: string, taskViewModel: TaskViewModel) => void = () => {};
+    setValue: (newValue: string, taskViewModel: TaskViewModel) => void;
 
     // init
     constructor(