vite-plugin-fvtt 0.2.2 → 0.2.3

package/CHANGELOG.md

@@ -1,8 +1,21 @@
 # Changelog
 
-## [0.2.2]
+## [0.2.3] - 2025-09-20
 
-## Fixed
+### Fixed
+
+- Return absolute paths from globbing.
+- Refactor path-utils import and usage across codebase, for better Windows support.
+- Refactor logger to use static Logger class methods.
+- Bump dependencies.
+
+### Added
+
+- Add Vitest and fixture-based build test.
+
+## [0.2.2] - 2025-09-10
+
+### Fixed
 
 - Improve windows path resolution.
 
@@ -75,7 +88,8 @@
 
 - Initial Release
 
-[unreleased]: https://github.com/MatyeusM/vite-plugin-fvtt/compare/v0.2.2...HEAD
+[unreleased]: https://github.com/MatyeusM/vite-plugin-fvtt/compare/v0.2.3...HEAD
+[0.2.3]: https://github.com/MatyeusM/vite-plugin-fvtt/compare/v0.2.2...v0.2.3
 [0.2.2]: https://github.com/MatyeusM/vite-plugin-fvtt/compare/v0.2.1...v0.2.2
 [0.2.1]: https://github.com/MatyeusM/vite-plugin-fvtt/compare/v0.2.0...v0.2.1
 [0.2.0]: https://github.com/MatyeusM/vite-plugin-fvtt/compare/v0.1.4...v0.2.0

package/README.md

@@ -104,7 +104,8 @@ Merging follows your manifest’s declared language paths, searching in root or
 
 ### **6. Packs**
 
-Packs are tried to be auto-discovered in the source directory. If the paths match, they are automatically compiled.
+Packs are tried to be auto-discovered in the source directory. If the paths match, they are
+automatically compiled.
 
 **Note:** Packs are currently not watched for changes.
 

package/dist/index.js

@@ -1,7 +1,6 @@
 import fs from "fs-extra";
-import posix from "path/posix";
-import dotenv from "dotenv";
 import path from "path";
+import dotenv from "dotenv";
 import { globSync } from "tinyglobby";
 import { compilePack } from "@foundryvtt/foundryvtt-cli";
 import { Server } from "socket.io";
@@ -26,49 +25,52 @@ function loadEnv() {
 
 //#endregion
 //#region src/utils/logger.ts
-var Logger = class {
-	namespace;
-	colors = {
-		info: "\x1B[32m",
+var Logger = class Logger {
+	static namespace = "vite-plugin-fvtt";
+	static colors = {
+		info: "\x1B[36m",
 		warn: "\x1B[33m",
 		error: "\x1B[31m"
 	};
-	constructor({ namespace = "vite-plugin-fvtt" } = {}) {
-		this.namespace = namespace;
+	static reset = "\x1B[0m";
+	initialize(namespace = "vite-plugin-fvtt") {
+		Logger.namespace = namespace;
+	}
+	static format(level, message) {
+		return `${Logger.colors[level] ?? ""}[${Logger.namespace}] [${level.toUpperCase()}]${Logger.reset} ${message}`;
 	}
-	format(level, message) {
-		const color = this.colors[level] ?? "";
-		const reset = "\x1B[0m";
-		return `${color}[${this.namespace}] [${level.toUpperCase()}]${reset} ${message}`;
+	static info(message) {
+		console.log(Logger.format("info", message));
 	}
-	info(message) {
-		console.log(this.format("info", message));
+	static warn(message) {
+		console.warn(Logger.format("warn", message));
 	}
-	warn(message) {
-		console.warn(this.format("warn", message));
+	static error(message) {
+		console.error(Logger.format("error", message));
 	}
-	error(message) {
-		console.error(this.format("error", message));
+	static fail(message) {
+		const formatted = Logger.format("error", Logger.stringify(message));
+		console.error(formatted);
+		throw new Error(formatted);
 	}
-	fail(message) {
-		this.error(message);
-		throw new Error(typeof message === "string" ? message : JSON.stringify(message, null, 2));
+	static stringify(message) {
+		if (message instanceof Error) return message.stack ?? message.message;
+		return typeof message === "string" ? message : JSON.stringify(message, null, 2);
 	}
 };
-var logger_default = new Logger();
+var logger_default = Logger;
 
 //#endregion
 //#region src/config/foundryvtt-manifest.ts
 function loadManifest(config) {
 	if (context?.manifest) return context.manifest;
 	const publicDir = config.publicDir || "public";
-	const MANIFEST_LOCATIONS = [
+	const foundPath = [
 		"system.json",
 		"module.json",
 		`${publicDir}/system.json`,
 		`${publicDir}/module.json`
-	];
-	const foundPath = MANIFEST_LOCATIONS.map((relPath) => path.resolve(process.cwd(), relPath)).find((absPath) => fs.pathExistsSync(absPath));
+	].map((relPath) => path.resolve(process.cwd(), relPath)).find((absPath) => fs.pathExistsSync(absPath));
 	if (!foundPath) logger_default.fail(`Could not find a manifest file (system.json or module.json) in project root or ${publicDir}/.`);
 	try {
 		const data = fs.readJsonSync(foundPath);
@@ -76,7 +78,7 @@ function loadManifest(config) {
 		const hasEsmodules = Array.isArray(data.esmodules) && data.esmodules.length > 0;
 		const hasScripts = Array.isArray(data.scripts) && data.scripts.length > 0;
 		if (hasEsmodules === hasScripts) logger_default.fail(`Manifest at ${foundPath} must define exactly one of "esmodules" or "scripts".`);
-		const result = {
+		return {
 			manifestType: foundPath.includes("module.json") ? "module" : "system",
 			id: data.id,
 			esmodules: Array.isArray(data.esmodules) ? data.esmodules : [],
@@ -85,7 +87,6 @@ function loadManifest(config) {
 			languages: Array.isArray(data.languages) ? data.languages : [],
 			packs: Array.isArray(data.packs) ? data.packs : []
 		};
-		return result;
 	} catch (err) {
 		logger_default.fail(`Failed to read manifest at ${foundPath}: ${err?.message || err}`);
 	}
@@ -122,8 +123,7 @@ function createPartialViteConfig(config) {
 			rollupOptions: { output: {
 				entryFileNames: fileName,
 				assetFileNames: (assetInfo) => {
-					const names = assetInfo.names ?? [];
-					if (names.some((n) => n.endsWith(".css"))) return cssFileName;
+					if ((assetInfo.names ?? []).some((n) => n.endsWith(".css"))) return cssFileName;
 					return "[name][extname]";
 				}
 			} }
@@ -220,8 +220,7 @@ var PathUtils = class PathUtils {
 	static getSourceDirectory() {
 		if (!PathUtils._sourceDirectory) {
 			const config = PathUtils.getConfig();
-			const normalizedEntry = path.normalize(config.build.lib.entry.toString());
-			const segments = normalizedEntry.split(path.sep).filter(Boolean).filter((s) => s !== ".");
+			const segments = path.normalize(config.build.lib.entry.toString()).split(path.sep).filter(Boolean).filter((s) => s !== ".");
 			const firstFolder = segments.length > 0 ? segments[0] : ".";
 			PathUtils._sourceDirectory = path.join(config.root, firstFolder);
 		}
@@ -257,17 +256,15 @@ var PathUtils = class PathUtils {
 		return fs.existsSync(file) ? file : "";
 	}
 	static findLocalFilePath(p) {
-		const fileCandidates = [
+		return [
 			PathUtils.getPublicDir(),
 			PathUtils.getSourceDirectory(),
 			PathUtils.getRoot()
-		].map((pth) => path.join(pth, p));
-		return fileCandidates.find((pth) => fs.existsSync(pth)) ?? null;
+		].map((pth) => path.join(pth, p)).find((pth) => fs.existsSync(pth)) ?? null;
 	}
 	static isFoundryVTTUrl(p) {
 		const decodedBase = PathUtils.getDecodedBase();
-		const pathToCheck = path.posix.normalize(p);
-		return pathToCheck.startsWith(decodedBase);
+		return path.posix.normalize(p).startsWith(decodedBase);
 	}
 	static foundryVTTUrlToLocal(p) {
 		const decodedBase = PathUtils.getDecodedBase();
@@ -290,8 +287,7 @@ var PathUtils = class PathUtils {
 	}
 	static getLanguageSourcePath(p, lang) {
 		const dir = path.parse(p).dir;
-		const lastDirName = path.basename(dir);
-		const finalSegments = lastDirName === lang ? dir : path.join(dir, lang);
+		const finalSegments = path.basename(dir) === lang ? dir : path.join(dir, lang);
 		return path.join(PathUtils.getSourceDirectory(), finalSegments);
 	}
 };
@@ -300,14 +296,10 @@ var path_utils_default = PathUtils;
 //#endregion
 //#region src/language/loader.ts
 function getLocalLanguageFiles(lang, outDir = false) {
-	const manifest = context.manifest;
-	const language = manifest.languages.find((l) => l.lang === lang);
+	const language = context.manifest.languages.find((l) => l.lang === lang);
 	if (!language) logger_default.fail(`Cannot find language "${lang}"`);
 	const langPath = language?.path ?? "";
-	if (outDir) {
-		const languageFile = path_utils_default.getOutDirFile(langPath);
-		return [languageFile];
-	}
+	if (outDir) return [path_utils_default.getOutDirFile(langPath)];
 	const publicDirFile = path_utils_default.getPublicDirFile(langPath);
 	if (publicDirFile !== "") return [publicDirFile];
 	const sourcePath = path_utils_default.getLanguageSourcePath(langPath, lang);
@@ -315,7 +307,7 @@ function getLocalLanguageFiles(lang, outDir = false) {
 		logger_default.warn(`No language folder found at: ${sourcePath}`);
 		return [];
 	}
-	return globSync(posix.join(sourcePath, "**/*.json"));
+	return globSync(path.join(sourcePath, "**/*.json"), { absolute: true });
 }
 function loadLanguage(lang, outDir = false) {
 	const files = getLocalLanguageFiles(lang, outDir);
@@ -376,15 +368,15 @@ function validator() {
 		if (lang.lang === "en") continue;
 		const currentLanguageData = loadLanguage(lang.lang, true);
 		if (currentLanguageData.size === 0) {
-			console.warn(`Summary for language [${lang.lang}]: Could not be loaded.`);
+			logger_default.warn(`Summary for language [${lang.lang}]: Could not be loaded.`);
 			continue;
 		}
 		const current = flattenKeys(currentLanguageData.values().next().value);
 		const missing = Object.keys(base).filter((key) => !(key in current));
 		const extra = Object.keys(current).filter((key) => !(key in base));
-		console.log(`Summary for language [${lang.lang}]:`);
-		if (missing.length) console.warn(`\tMissing keys: ${missing.length}`, missing.slice(0, 5));
-		if (extra.length) console.warn(`\tExtra keys: ${extra.length}`, extra.slice(0, 5));
+		logger_default.info(`Summary for language [${lang.lang}]:`);
+		if (missing.length) console.warn(`Missing keys: ${missing.length}`, missing.slice(0, 5));
+		if (extra.length) console.warn(`Extra keys: ${extra.length}`, extra.slice(0, 5));
 		if (!missing.length && !extra.length) console.log("	✅ All keys match.");
 	}
 }
@@ -401,11 +393,10 @@ async function compileManifestPacks() {
 			logger_default.warn(`Pack path not found for ${pack.path}, skipped.`);
 			continue;
 		}
-		const entries = fs.readdirSync(chosenSrc, {
+		const hasYaml = fs.readdirSync(chosenSrc, {
 			recursive: true,
 			encoding: "utf8"
-		});
-		const hasYaml = entries.some((entry) => entry.endsWith(".yaml") || entry.endsWith(".yml"));
+		}).some((entry) => entry.endsWith(".yaml") || entry.endsWith(".yml"));
 		await compilePack(chosenSrc, dest, {
 			yaml: hasYaml,
 			recursive: true
@@ -461,8 +452,7 @@ function httpMiddlewareHook(server) {
 //#region src/server/socket-proxy.ts
 function socketProxy(server) {
 	const env = context.env;
-	const ioProxy = new Server(server.httpServer, { path: "/socket.io" });
-	ioProxy.on("connection", (socket) => {
+	new Server(server.httpServer, { path: "/socket.io" }).on("connection", (socket) => {
 		const upstream = io(`http://${env.foundryUrl}:${env.foundryPort}`, {
 			transports: ["websocket"],
 			upgrade: false,
@@ -486,8 +476,7 @@ function socketProxy(server) {
 			});
 		});
 		upstream.onAny((event, ...args) => {
-			const lastArg = args[args.length - 1];
-			const maybeAck = typeof lastArg === "function" ? args.pop() : null;
+			const maybeAck = typeof args[args.length - 1] === "function" ? args.pop() : null;
 			socket.emit(event, ...args, (response) => {
 				if (maybeAck) maybeAck(response);
 			});
@@ -599,7 +588,7 @@ function foundryVTTPlugin(options = { buildPacks: true }) {
 		async generateBundle() {
 			const manifestCandidates = ["system.json", "module.json"];
 			for (const file of manifestCandidates) {
-				const src = posix.resolve(file);
+				const src = path.resolve(file);
 				if (!path_utils_default.getPublicDirFile(file) && fs.existsSync(src)) {
 					this.addWatchFile(src);
 					const manifest = fs.readJsonSync(src);
@@ -618,7 +607,7 @@ function foundryVTTPlugin(options = { buildPacks: true }) {
 				const languageData = transform(languageDataRaw);
 				this.emitFile({
 					type: "asset",
-					fileName: posix.join(language.path),
+					fileName: path.join(language.path),
 					source: JSON.stringify(languageData, null, 2)
 				});
 			}
@@ -627,17 +616,12 @@ function foundryVTTPlugin(options = { buildPacks: true }) {
 			if (options.buildPacks) await compileManifestPacks();
 		},
 		closeBundle() {
-			const languages = context.manifest?.languages ?? [];
-			if (languages.length > 0) validator();
+			if ((context.manifest?.languages ?? []).length > 0) validator();
 		},
 		load(id) {
 			const config = context.config;
 			const jsFileName = (config.build.rollupOptions?.output).entryFileNames;
-			if (id === jsFileName || id === `/${jsFileName}`) {
-				const entryPath = posix.resolve(config.build.lib.entry);
-				const viteId = `/@fs/${entryPath}`;
-				return `import '${viteId}';\n${hmr_client_default}`;
-			}
+			if (id === jsFileName || id === `/${jsFileName}`) return `import '${`/@fs/${path.resolve(config.build.lib.entry)}`}';\n${hmr_client_default}`;
 		},
 		configureServer: setupDevServer
 	};

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "vite-plugin-fvtt",
-  "version": "0.2.2",
+  "version": "0.2.3",
   "description": "A Vite plugin for module and system development for Foundry VTT",
   "keywords": [
     "vite",
@@ -28,7 +28,8 @@
   ],
   "scripts": {
     "dev": "tsdown --watch",
-    "build": "tsdown"
+    "build": "tsdown",
+    "test": "vitest"
   },
   "devDependencies": {
     "@eslint/js": "^9.34.0",
@@ -36,10 +37,11 @@
     "@types/node": "^24.3.0",
     "eslint": "^9.34.0",
     "prettier": "^3.6.2",
-    "tsdown": "^0.14.2",
+    "tsdown": "^0.15.1",
     "typescript": "^5.9.2",
     "typescript-eslint": "^8.41.0",
-    "vite": "*"
+    "vite": "*",
+    "vitest": "^3.2.4"
   },
   "dependencies": {
     "@foundryvtt/foundryvtt-cli": "^3.0.0",