Add readonly to a few props and types

src/cli/index.ts

@@ -80,8 +80,8 @@ export async function selectTasks(
     d: Pick<D, "simplifyPath">,
     herebyfile: Herebyfile,
     herebyfilePath: string,
-    taskNames: string[],
-): Promise<Task[]> {
+    taskNames: readonly string[],
+): Promise<readonly Task[]> {
     if (taskNames.length === 0) {
         if (herebyfile.defaultTask) return [herebyfile.defaultTask];
         throw new UserError(

src/cli/parseArgs.ts

@@ -4,11 +4,11 @@ import minimist from "minimist";
 import type { TaskFormat } from "./formatTasks.js";
 
 interface CLIOptions {
-    help: boolean;
-    run: string[];
-    herebyfile: string | undefined;
-    printTasks: TaskFormat | undefined;
-    version: boolean;
+    readonly help: boolean;
+    readonly run: readonly string[];
+    readonly herebyfile: string | undefined;
+    readonly printTasks: TaskFormat | undefined;
+    readonly version: boolean;
 }
 
 export function parseArgs(argv: string[]): CLIOptions {

src/cli/runner.ts

@@ -8,12 +8,12 @@ import type { D } from "./utils.js";
 export type RunnerD = Pick<D, "log" | "error" | "prettyMilliseconds">;
 
 export class Runner {
-    private _addedTasks = new Map<Task, Promise<void>>();
+    private readonly _addedTasks = new Map<Task, Promise<void>>();
 
     private _errored = false;
-    private _startTimes = new Map<Task, number>();
+    private readonly _startTimes = new Map<Task, number>();
 
-    constructor(private _d: RunnerD) {}
+    constructor(private readonly _d: RunnerD) {}
 
     async runTasks(...tasks: Task[]): Promise<void> {
         // Using allSettled here so that we don't immediately exit; it could be