Added readonly flag to non-reassigned attributes

src/core/brsTypes/components/RoAudioPlayerEvent.ts

@@ -8,9 +8,9 @@ import { MediaEvent } from "../../common";
 
 export class RoAudioPlayerEvent extends BrsComponent implements BrsValue {
     readonly kind = ValueKind.Object;
-    private flags: number;
-    private index: number;
-    private message: string;
+    private readonly flags: number;
+    private readonly index: number;
+    private readonly message: string;
 
     constructor(flags: number, index: number) {
         super("roAudioPlayerEvent");

src/core/brsTypes/components/RoAudioResource.ts

@@ -8,12 +8,12 @@ import { DataType } from "../../common";
 
 export class RoAudioResource extends BrsComponent implements BrsValue {
     readonly kind = ValueKind.Object;
-    private audioName: string;
-    private audioId?: number;
+    private readonly audioName: string;
+    private readonly audioId?: number;
+    private readonly maxStreams: number;
+    private readonly valid: boolean;
     private currentIndex: number;
     private playing: boolean;
-    private maxStreams: number;
-    private valid: boolean;
 
     constructor(interpreter: Interpreter, name: BrsString) {
         super("roAudioResource");

src/core/brsTypes/components/RoBitmap.ts

@@ -35,11 +35,11 @@ export class RoBitmap extends BrsComponent implements BrsValue {
     private readonly height: number;
     private readonly name: string;
     private readonly disposeCanvas: boolean;
+    private readonly valid: boolean;
     private alphaEnable: boolean;
     private rgbaCanvas?: BrsCanvas;
     private rgbaLast: number;
     private rgbaRedraw: boolean;
-    private valid: boolean;
 
     constructor(interpreter: Interpreter, param: BrsComponent) {
         super("roBitmap");

src/core/brsTypes/components/RoChannelStore.ts

@@ -8,11 +8,11 @@ import { RoChannelStoreEvent } from "./RoChannelStoreEvent";
 import { RoAssociativeArray, AAMember } from "./RoAssociativeArray";
 
 export class RoChannelStore extends BrsComponent implements BrsValue {
-    private id: number;
+    readonly kind = ValueKind.Object;
+    private readonly id: number;
     private order: BrsType[];
     private credData: string;
     private port?: RoMessagePort;
-    readonly kind = ValueKind.Object;
 
     constructor() {
         super("roChannelStore");

src/core/brsTypes/components/RoChannelStoreEvent.ts

@@ -7,8 +7,8 @@ import { Int32 } from "../Int32";
 
 export class RoChannelStoreEvent extends BrsComponent implements BrsValue {
     readonly kind = ValueKind.Object;
-    private id: number;
-    private response?: BrsType[];
+    private readonly id: number;
+    private readonly response?: BrsType[];
 
     constructor(id: number, response?: BrsType[]) {
         super("roChannelStoreEvent");

src/core/brsTypes/components/RoFont.ts

@@ -9,11 +9,11 @@ import { BrsCanvasContext2D, createNewCanvas, releaseCanvas } from "../draw2d";
 
 export class RoFont extends BrsComponent implements BrsValue {
     readonly kind = ValueKind.Object;
-    private family: string;
-    private size: number;
-    private bold: boolean;
-    private italic: boolean;
-    private metrics: FontMetrics;
+    private readonly family: string;
+    private readonly size: number;
+    private readonly bold: boolean;
+    private readonly italic: boolean;
+    private readonly metrics: FontMetrics;
 
     // Constructor can only be used by RoFontRegistry()
     constructor(

src/core/brsTypes/components/RoFontRegistry.ts

@@ -20,10 +20,10 @@ export interface FontMetrics {
 
 export class RoFontRegistry extends BrsComponent implements BrsValue {
     readonly kind = ValueKind.Object;
-    readonly defaultFontSize = 40;
-    readonly fallbackFontFamily = "Arial, Helvetica, sans-serif";
-    private defaultFontFamily: string;
-    private fontRegistry: Map<string, FontMetrics[]>;
+    private readonly defaultFontSize = 40;
+    private readonly fallbackFontFamily = "Arial, Helvetica, sans-serif";
+    private readonly defaultFontFamily: string;
+    private readonly fontRegistry: Map<string, FontMetrics[]>;
 
     constructor(interpreter: Interpreter) {
         super("roFontRegistry");

src/core/brsTypes/components/RoInputEvent.ts

@@ -6,8 +6,8 @@ import { Interpreter } from "../../interpreter";
 
 export class RoInputEvent extends BrsComponent implements BrsValue {
     readonly kind = ValueKind.Object;
-    readonly id: number;
-    readonly response?: BrsType;
+    private readonly id: number;
+    private readonly response?: BrsType;
 
     constructor(response?: BrsType) {
         super("roInputEvent");

src/core/brsTypes/components/RoInvalid.ts

@@ -7,7 +7,7 @@ import { Unboxable } from "../Boxing";
 
 export class RoInvalid extends BrsComponent implements BrsValue, Unboxable {
     readonly kind = ValueKind.Object;
-    private intrinsic: BrsInvalid;
+    private readonly intrinsic: BrsInvalid;
 
     public getValue(): BrsInvalid {
         return this.intrinsic;

src/core/brsTypes/components/RoRegion.ts

@@ -20,7 +20,7 @@ import UPNG from "upng-js";
 
 export class RoRegion extends BrsComponent implements BrsValue {
     readonly kind = ValueKind.Object;
-    private valid: boolean;
+    private readonly valid: boolean;
     private alphaEnable: boolean;
     private bitmap: RoBitmap | RoScreen;
     private x: number;

src/core/brsTypes/components/RoSprite.ts

@@ -11,9 +11,9 @@ import { BrsImageData } from "../draw2d";
 
 export class RoSprite extends BrsComponent implements BrsValue {
     readonly kind = ValueKind.Object;
+    private readonly id: number;
+    private readonly regions?: RoArray;
     private region: RoRegion;
-    private regions?: RoArray;
-    private id: number;
     private x: number;
     private y: number;
     private z: number;

src/core/brsTypes/components/RoURLEvent.ts

@@ -7,12 +7,12 @@ import { Int32 } from "../Int32";
 
 export class RoURLEvent extends BrsComponent implements BrsValue, Comparable {
     readonly kind = ValueKind.Object;
-    private id: number;
-    private responseCode: number;
-    private responseString: string;
-    private failureReason: string;
-    private headers: string;
-    private targetIp: string;
+    private readonly id: number;
+    private readonly responseCode: number;
+    private readonly responseString: string;
+    private readonly failureReason: string;
+    private readonly headers: string;
+    private readonly targetIp: string;
 
     constructor(id: number, response: string, status: number, statusText: string, headers: string) {
         super("roUrlEvent");

src/core/brsTypes/components/RoVideoPlayerEvent.ts

@@ -8,9 +8,9 @@ import { MediaEvent } from "../../common";
 
 export class RoVideoPlayerEvent extends BrsComponent implements BrsValue {
     readonly kind = ValueKind.Object;
-    readonly flags: number;
-    readonly index: number;
-    readonly message: string;
+    private readonly flags: number;
+    private readonly index: number;
+    private readonly message: string;
 
     constructor(flags: number, index: number) {
         super("roVideoPlayerEvent");

src/core/interpreter/FileSystem.ts

@@ -6,7 +6,7 @@ import * as nodeFS from "fs";
 /** Proxy Object to make File System volumes case insensitive, same as Roku devices */
 
 export class FileSystem {
-    private paths: Map<string, string>;
+    private readonly paths: Map<string, string>;
     readonly root?: string;
     readonly ext?: string;
     readonly pfs: typeof zenFS.fs | typeof nodeFS; // pkg:

src/core/preprocessor/index.ts

@@ -1,13 +1,11 @@
 import { EventEmitter } from "events";
-
 import { Token } from "../lexer";
-
 import { Parser } from "./Parser";
 import { Preprocessor as InternalPreprocessor, FilterResults } from "./Preprocessor";
 
 export class Preprocessor {
-    private parser = new Parser();
-    private _preprocessor = new InternalPreprocessor();
+    private readonly parser = new Parser();
+    private readonly _preprocessor = new InternalPreprocessor();
 
     readonly events = new EventEmitter();