fork-version 5.0.0 → 5.0.1

package/CHANGELOG.md

@@ -1,5 +1,14 @@
 # Fork Version
 
+## [5.0.1](https://github.com/eglavin/fork-version/compare/v5.0.0...v5.0.1) (2026-04-10)
+
+
+### Refactor
+
+* remove logger from file manager classes ([7800d8b](https://github.com/eglavin/fork-version/commit/7800d8b29fd959f82216ee7061ce5fb44a883368))
+* show file name in the missing property exception log ([f7c80fc](https://github.com/eglavin/fork-version/commit/f7c80fcaeadaba897f637619931562f82cc9a707))
+
+
 ## [5.0.0](https://github.com/eglavin/fork-version/compare/v4.1.10...v5.0.0) (2026-04-09)
 
 

package/dist/files/arm-bicep.js

@@ -1,3 +1,4 @@
+import { MissingPropertyException } from "./file-manager.js";
 import { basename } from "node:path";
 import { readFileSync, writeFileSync } from "node:fs";
 //#region src/files/arm-bicep.ts
@@ -11,26 +12,19 @@ import { readFileSync, writeFileSync } from "node:fs";
 * ```
 */
 var ARMBicep = class {
-	#logger;
-	constructor(logger) {
-		this.#logger = logger;
-	}
 	/** https://regex101.com/r/Lriphb/2 */
 	#metadataRegex = /(metadata contentVersion *= *['"])(?<version>[^'"]+)(['"])/;
 	/** https://regex101.com/r/iKCTF9/1 */
 	#varRegex = /(var contentVersion(?: string)? *= *['"])(?<version>[^'"]+)(['"])/;
 	read(filePath) {
-		const fileName = basename(filePath);
 		const fileContents = readFileSync(filePath, "utf8");
 		const metadataMatch = this.#metadataRegex.exec(fileContents);
-		const varMatch = this.#varRegex.exec(fileContents);
-		if (metadataMatch?.groups?.version && varMatch?.groups?.version) return {
-			name: fileName,
+		if (metadataMatch?.groups?.version) return {
+			name: basename(filePath),
 			path: filePath,
 			version: metadataMatch.groups.version
 		};
-		if (!metadataMatch) this.#logger.warn(`[File Manager] Missing 'metadata contentVersion' in bicep file: ${fileName}`);
-		if (!varMatch) this.#logger.warn(`[File Manager] Missing 'var contentVersion' in bicep file: ${fileName}`);
+		throw new MissingPropertyException("ARM Bicep", "metadata contentVersion");
 	}
 	write(fileState, newVersion) {
 		const updatedContent = readFileSync(fileState.path, "utf8").replace(this.#metadataRegex, `$1${newVersion}$3`).replace(this.#varRegex, `$1${newVersion}$3`);

package/dist/files/file-manager.d.ts

@@ -2,6 +2,15 @@ import { ForkConfig } from "../config/types.js";
 import { Logger } from "../services/logger.js";
 
 //#region src/files/file-manager.d.ts
+/**
+ * Exception thrown if a file manager encounters a file missing a required property,
+ * such as the "version" property in a JSON package file.
+ */
+declare class MissingPropertyException extends Error {
+  fileType: string;
+  propertyName: string;
+  constructor(fileType: string, propertyName: string);
+}
 interface FileState {
   name: string;
   path: string;
@@ -44,4 +53,4 @@ declare class FileManager {
   write(fileState: FileState, newVersion: string): void;
 }
 //#endregion
-export { FileManager, FileState, IFileManager };
+export { FileManager, FileState, IFileManager, MissingPropertyException };

package/dist/files/file-manager.js

@@ -5,8 +5,22 @@ import { PlainText } from "./plain-text.js";
 import { MSBuildProject } from "./ms-build-project.js";
 import { ARMBicep } from "./arm-bicep.js";
 import { InstallShieldISM } from "./install-shield-ism.js";
-import { isAbsolute, resolve } from "node:path";
+import { basename, isAbsolute, resolve } from "node:path";
 //#region src/files/file-manager.ts
+/**
+* Exception thrown if a file manager encounters a file missing a required property,
+* such as the "version" property in a JSON package file.
+*/
+var MissingPropertyException = class extends Error {
+	fileType;
+	propertyName;
+	constructor(fileType, propertyName) {
+		super(`Missing '${propertyName}' property in ${fileType}`);
+		this.name = "MissingPropertyException";
+		this.fileType = fileType;
+		this.propertyName = propertyName;
+	}
+};
 var FileManager = class {
 	#config;
 	#logger;
@@ -15,12 +29,12 @@ var FileManager = class {
 		this.#config = config;
 		this.#logger = logger;
 		this.#fileManagers = [
-			new JSONPackage(logger),
-			new YAMLPackage(logger),
-			new PlainText(logger),
-			new MSBuildProject(logger),
-			new ARMBicep(logger),
-			new InstallShieldISM(logger)
+			new JSONPackage(),
+			new YAMLPackage(),
+			new PlainText(),
+			new MSBuildProject(),
+			new ARMBicep(),
+			new InstallShieldISM()
 		];
 	}
 	/**
@@ -40,7 +54,15 @@ var FileManager = class {
 		const _fileName = pathOrName.toLowerCase();
 		const filePath = isAbsolute(pathOrName) ? pathOrName : resolve(this.#config.path, pathOrName);
 		if (!fileExists(filePath)) return;
-		for (const fileManager of this.#fileManagers) if (fileManager.isSupportedFile(_fileName)) return fileManager.read(filePath);
+		for (const fileManager of this.#fileManagers) if (fileManager.isSupportedFile(_fileName)) {
+			try {
+				return fileManager.read(filePath);
+			} catch (error) {
+				if (error instanceof MissingPropertyException) this.#logger.warn(`[File Manager] Missing '${error.propertyName}' property in ${error.fileType} file: ${basename(_fileName)}`);
+				else throw new Error(`An unexpected error occurred while reading file: ${filePath}`, { cause: error });
+			}
+			return;
+		}
 		this.#logger.error(`[File Manager] Unsupported file: ${pathOrName}`);
 	}
 	/**
@@ -62,4 +84,4 @@ var FileManager = class {
 	}
 };
 //#endregion
-export { FileManager };
+export { FileManager, MissingPropertyException };

package/dist/files/install-shield-ism.js

@@ -1,3 +1,4 @@
+import { MissingPropertyException } from "./file-manager.js";
 import { basename } from "node:path";
 import { readFileSync, writeFileSync } from "node:fs";
 import * as cheerio from "cheerio/slim";
@@ -24,24 +25,19 @@ import * as cheerio from "cheerio/slim";
 * ```
 */
 var InstallShieldISM = class {
-	#logger;
-	constructor(logger) {
-		this.#logger = logger;
-	}
 	#cheerioOptions = {
 		xmlMode: true,
 		xml: { decodeEntities: false }
 	};
 	read(filePath) {
-		const fileName = basename(filePath);
 		const fileContents = readFileSync(filePath, "utf8");
 		const version = cheerio.load(fileContents, this.#cheerioOptions)("msi > table[name=\"Property\"] > row > td:contains(\"ProductVersion\")").next().text().trim();
 		if (version) return {
-			name: fileName,
+			name: basename(filePath),
 			path: filePath,
 			version
 		};
-		this.#logger.warn(`[File Manager] Unable to determine InstallShield ISM version: ${fileName}`);
+		throw new MissingPropertyException("InstallShield ISM", "ProductVersion");
 	}
 	write(fileState, newVersion) {
 		const fileContents = readFileSync(fileState.path, "utf8");

package/dist/files/json-package.js

@@ -1,3 +1,4 @@
+import { MissingPropertyException } from "./file-manager.js";
 import { basename } from "node:path";
 import { readFileSync, writeFileSync } from "node:fs";
 import { applyEdits, modify, parse as parse$1 } from "jsonc-parser";
@@ -16,10 +17,6 @@ import { applyEdits, modify, parse as parse$1 } from "jsonc-parser";
 * ```
 */
 var JSONPackage = class {
-	#logger;
-	constructor(logger) {
-		this.#logger = logger;
-	}
 	/** Options for parsing JSON and JSONC files. */
 	#jsoncOptions = {
 		allowEmptyContent: false,
@@ -37,17 +34,16 @@ var JSONPackage = class {
 		return applyEdits(jsonc, modify(jsonc, jsonPath, newString, {}));
 	}
 	read(filePath) {
-		const fileName = basename(filePath);
 		const fileContents = readFileSync(filePath, "utf8");
 		const parseErrors = [];
 		const parsedJson = parse$1(fileContents, parseErrors, this.#jsoncOptions);
 		if (parsedJson?.version && parseErrors.length === 0) return {
-			name: fileName,
+			name: basename(filePath),
 			path: filePath,
 			version: parsedJson.version,
 			isPrivate: typeof parsedJson?.private === "boolean" ? parsedJson.private : true
 		};
-		this.#logger.warn(`[File Manager] Unable to determine json version: ${fileName}`);
+		throw new MissingPropertyException("JSON", "version");
 	}
 	write(fileState, newVersion) {
 		let fileContents = readFileSync(fileState.path, "utf8");

package/dist/files/ms-build-project.js

@@ -1,3 +1,4 @@
+import { MissingPropertyException } from "./file-manager.js";
 import { basename } from "node:path";
 import { readFileSync, writeFileSync } from "node:fs";
 import * as cheerio from "cheerio/slim";
@@ -17,24 +18,19 @@ import * as cheerio from "cheerio/slim";
 * ```
 */
 var MSBuildProject = class {
-	#logger;
-	constructor(logger) {
-		this.#logger = logger;
-	}
 	#cheerioOptions = {
 		xmlMode: true,
 		xml: { decodeEntities: false }
 	};
 	read(filePath) {
-		const fileName = basename(filePath);
 		const fileContents = readFileSync(filePath, "utf8");
 		const version = cheerio.load(fileContents, this.#cheerioOptions)("Project > PropertyGroup > Version").text();
 		if (version) return {
-			name: fileName,
+			name: basename(filePath),
 			path: filePath,
 			version
 		};
-		this.#logger.warn(`[File Manager] Unable to determine ms-build version: ${fileName}`);
+		throw new MissingPropertyException("MSBuild", "Version");
 	}
 	write(fileState, newVersion) {
 		const fileContents = readFileSync(fileState.path, "utf8");

package/dist/files/plain-text.js

@@ -1,3 +1,4 @@
+import { MissingPropertyException } from "./file-manager.js";
 import { basename } from "node:path";
 import { readFileSync, writeFileSync } from "node:fs";
 //#region src/files/plain-text.ts
@@ -10,19 +11,14 @@ import { readFileSync, writeFileSync } from "node:fs";
 * ```
 */
 var PlainText = class {
-	#logger;
-	constructor(logger) {
-		this.#logger = logger;
-	}
 	read(filePath) {
-		const fileName = basename(filePath);
 		const fileContents = readFileSync(filePath, "utf8").trim();
 		if (fileContents) return {
-			name: fileName,
+			name: basename(filePath),
 			path: filePath,
 			version: fileContents
 		};
-		this.#logger.warn(`[File Manager] Unable to determine plain text version: ${fileName}`);
+		throw new MissingPropertyException("Plain Text", "version");
 	}
 	write(fileState, newVersion) {
 		writeFileSync(fileState.path, newVersion, "utf8");

package/dist/files/yaml-package.js

@@ -1,3 +1,4 @@
+import { MissingPropertyException } from "./file-manager.js";
 import { basename } from "node:path";
 import { readFileSync, writeFileSync } from "node:fs";
 import { parse as parse$1, parseDocument } from "yaml";
@@ -15,10 +16,6 @@ import { parse as parse$1, parseDocument } from "yaml";
 * ```
 */
 var YAMLPackage = class {
-	#logger;
-	constructor(logger) {
-		this.#logger = logger;
-	}
 	/**
 	* If the version is returned with a "+" symbol in the value then the version might be from a
 	* flutter `pubspec.yaml` file, if so we want to retain the input builderNumber by splitting it
@@ -33,18 +30,17 @@ var YAMLPackage = class {
 		return { version: fileVersion };
 	}
 	read(filePath) {
-		const fileName = basename(filePath);
 		const fileVersion = parse$1(readFileSync(filePath, "utf-8"))?.version;
 		if (fileVersion) {
 			const parsedVersion = this.#handleBuildNumber(fileVersion);
 			return {
-				name: fileName,
+				name: basename(filePath),
 				path: filePath,
 				version: parsedVersion.version || "",
 				builderNumber: parsedVersion.builderNumber ?? void 0
 			};
 		}
-		this.#logger.warn(`[File Manager] Unable to determine yaml version: ${fileName}`);
+		throw new MissingPropertyException("YAML", "version");
 	}
 	write(fileState, newVersion) {
 		const yamlDocument = parseDocument(readFileSync(fileState.path, "utf8"));

package/dist/index.d.ts

@@ -1,7 +1,7 @@
 import { ParserOptions, createParserOptions } from "./commit-parser/options.js";
 import { ChangelogPresetConfig, ChangelogPresetConfigType, Config, ForkConfig } from "./config/types.js";
 import { Logger } from "./services/logger.js";
-import { FileManager, FileState, IFileManager } from "./files/file-manager.js";
+import { FileManager, FileState, IFileManager, MissingPropertyException } from "./files/file-manager.js";
 import { Git } from "./services/git.js";
 import { inspect } from "./commands/inspect.js";
 import { main } from "./commands/main.js";
@@ -18,4 +18,4 @@ import { NextVersion, getNextVersion } from "./process/get-next-version.js";
 import { updateChangelog } from "./process/changelog.js";
 import { commitChanges } from "./process/commit.js";
 import { tagChanges } from "./process/tag.js";
-export { type ChangelogPresetConfig, type ChangelogPresetConfigType, type Commit, type CommitMerge, type CommitNote, CommitParser, type CommitReference, type CommitRevert, type CommitsSinceTag, type Config, type CurrentVersion, FileManager, type FileState, type ForkConfig, ForkConfigSchema, Git, type IFileManager, Logger, type NextVersion, type ParserOptions, commitChanges, createParserOptions, defineConfig, filterRevertedCommits, getCommitsSinceTag, getCurrentVersion, getNextVersion, getUserConfig, inspect, main, tagChanges, updateChangelog, validateConfig };
+export { type ChangelogPresetConfig, type ChangelogPresetConfigType, type Commit, type CommitMerge, type CommitNote, CommitParser, type CommitReference, type CommitRevert, type CommitsSinceTag, type Config, type CurrentVersion, FileManager, type FileState, type ForkConfig, ForkConfigSchema, Git, type IFileManager, Logger, MissingPropertyException, type NextVersion, type ParserOptions, commitChanges, createParserOptions, defineConfig, filterRevertedCommits, getCommitsSinceTag, getCurrentVersion, getNextVersion, getUserConfig, inspect, main, tagChanges, updateChangelog, validateConfig };

package/dist/index.js

@@ -14,6 +14,6 @@ import { ForkConfigSchema } from "./config/schema.js";
 import { defineConfig } from "./config/define-config.js";
 import { Git } from "./services/git.js";
 import { getUserConfig } from "./config/user-config.js";
-import { FileManager } from "./files/file-manager.js";
+import { FileManager, MissingPropertyException } from "./files/file-manager.js";
 import { Logger } from "./services/logger.js";
-export { CommitParser, FileManager, ForkConfigSchema, Git, Logger, commitChanges, createParserOptions, defineConfig, filterRevertedCommits, getCommitsSinceTag, getCurrentVersion, getNextVersion, getUserConfig, inspect, main, tagChanges, updateChangelog, validateConfig };
+export { CommitParser, FileManager, ForkConfigSchema, Git, Logger, MissingPropertyException, commitChanges, createParserOptions, defineConfig, filterRevertedCommits, getCommitsSinceTag, getCurrentVersion, getNextVersion, getUserConfig, inspect, main, tagChanges, updateChangelog, validateConfig };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "fork-version",
-  "version": "5.0.0",
+  "version": "5.0.1",
   "license": "MIT",
   "description": "Fork-Version automates version control tasks such as determining, updating, and committing versions, files, and changelogs, simplifying the process when adhering to the conventional commit standard.",
   "keywords": [