Updated dependency versions.

Added safety dupe checks in app start & load functions.

Author: mbrich

COPYRIGHT

@@ -1 +1 @@
-Copyright 2024 Toreda, Inc. All Rights Reserved.
+Copyright 2025 Toreda, Inc. All Rights Reserved.

README.md

@@ -97,7 +97,7 @@ We recommend `yarn` to manage project packages. Although you can use `npm instal
 
 ### *What are resolutions?*
 
-Yarn `resolutions` support selective package version overrides used by the project & dependencies.
+Yarn `resolutions` allow per-package version overrides. All dependencies use the specified package version.
 
 ### *Why are there packages in the `resolutions` field?*
 

package.json

@@ -36,8 +36,8 @@
 	},
 	"homepage": "https://github.com/toreda/typescript-cli-starter#readme",
 	"devDependencies": {
-		"@swc/core": "^1.11.13",
-		"@swc/jest": "^0.2.37",
+		"@swc/core": "^1.11.24",
+		"@swc/jest": "^0.2.38",
 		"@toreda/build-tools": "^0.8.0",
 		"@toreda/eslint-config": "^2.2.0",
 		"@toreda/lifecycle": "^2.2.1",
@@ -52,13 +52,13 @@
 		"@types/yargs": "^17.0.32",
 		"@typescript-eslint/eslint-plugin": "^7.13.0",
 		"@typescript-eslint/parser": "^7.13.0",
-		"esbuild": "^0.25.1",
+		"esbuild": "0.25.1",
 		"esbuild-config": "^1.0.1",
 		"esbuild-loader": "^4.3.0",
 		"esbuild-node-externals": "^1.18.0",
-		"eslint": "^9.23.0",
-		"eslint-config-prettier": "^10.1.1",
-		"eslint-plugin-prettier": "^5.2.5",
+		"eslint": "^9.27.0",
+		"eslint-config-prettier": "^10.1.5",
+		"eslint-plugin-prettier": "^5.4.0",
 		"fork-ts-checker-webpack-plugin": "^9.0.2",
 		"gulp": "^5.0.0",
 		"gulp-typescript": "^6.0.0-alpha.1",

src/app.ts

@@ -2,6 +2,8 @@ import {AppConfig} from './app/config';
 import type {CliArgs} from './cli/args';
 import {Log} from '@toreda/log';
 import {type ClientDelegate, ClientLifecycle, clientPhase} from '@toreda/lifecycle';
+import {Bool, boolMake} from '@toreda/strong-types';
+import {Defaults} from './defaults';
 
 /**
  * Example app class called by the CLI.
@@ -10,21 +12,62 @@ export class App implements ClientDelegate {
 	public readonly lifecycle: ClientLifecycle;
 	public readonly log: Log;
 	public readonly cfg: AppConfig;
+	private readonly loaded: Bool;
+	private readonly running: Bool;
 
 	constructor(args: CliArgs, log: Log) {
 		this.lifecycle = new ClientLifecycle();
 		this.log = log;
 
+		this.loaded = boolMake(Defaults.App.Loaded);
+		this.running = boolMake(Defaults.App.Running);
 		this.cfg = new AppConfig(args, log);
+
+		this.bindListeners();
 	}
 
-	public async start(): Promise<App> {
-		this.log.info(`CLI App starting..`);
+	private bindListeners(): void {
+		this.load = this.load.bind(this);
+		this.start = this.start.bind(this);
+	}
 
+	public async load(): Promise<App> {
+		// Ignore duplicate load calls after app finishes loading. Not strictly necessary
+		// when `load` contains only Lifecycle phase calls as lifecycle methods set their own
+		// flags to prevent dupe calls. Prevents problematic behavior if unprotected calls
+		// are added in the future.
+		if (this.loaded()) {
+			return this;
+		}
+		await clientPhase('clientWillInit', this);
 		await clientPhase('clientOnInit', this);
+		await clientPhase('clientDidInit', this);
+
+		await clientPhase('clientWillLoad', this);
 		await clientPhase('clientOnLoad', this);
+		await clientPhase('clientDidLoad', this);
+
+		this.loaded(true);
+		return this;
+
+	}
+
+	public async start(): Promise<App> {
+		// Ignore dupe calls after app start. Lifecycle phases maintain their own flags
+		// to prevent dupe calls, but other logic here isn't protected without the check.
+		if (this.running()) {
+			return this;
+		}
+
+		this.log.info(`CLI App starting..`);
+		await this.load();
+		await clientPhase('clientWillStart', this);
 		await clientPhase('clientOnStart', this);
+		await clientPhase('clientDidStart', this);
+
+		await clientPhase('clientWillBecomeReady', this);
 		await clientPhase('clientOnReady', this);
+		await clientPhase('clientDidBecomeReady', this);
 
 		return this;
 	}
@@ -42,7 +85,7 @@ export class App implements ClientDelegate {
 
 		this.log.debug(`CLI app init complete.`);
 
-		return this.lifecycle.phase('clientOnInit');
+		return this.lifecycle.endPhase('clientOnInit');
 	}
 
 	public async clientOnLoad(): Promise<boolean> {
@@ -53,7 +96,7 @@ export class App implements ClientDelegate {
 			return false;
 		}
 
-		return this.lifecycle.phase('clientOnLoad');
+		return this.lifecycle.endPhase('clientOnLoad');
 	}
 
 	public async clientOnStart(): Promise<boolean> {
@@ -64,7 +107,7 @@ export class App implements ClientDelegate {
 			return false;
 		}
 
-		return this.lifecycle.phase('clientOnStart');
+		return this.lifecycle.endPhase('clientOnStart');
 	}
 
 	public async clientOnReady(): Promise<boolean> {
@@ -75,7 +118,7 @@ export class App implements ClientDelegate {
 			return false;
 		}
 
-		return this.lifecycle.phase('clientOnReady');
+		return this.lifecycle.endPhase('clientOnReady');
 	}
 
 	public reset(): void {

src/defaults.ts

@@ -5,5 +5,9 @@ import type {Env} from './env';
  *
  */
 export class Defaults {
+	public static App = {
+		Loaded: false as const,
+		Running: false as const
+	} as const;
 	public static Env: Env = 'prod' as const;
 }