@inlang/sdk 0.36.1 → 0.36.2

package/dist/resolve-modules/message-lint-rules/resolveMessageLintRules.d.ts

@@ -3,7 +3,7 @@ import { MessageLintRuleIsInvalidError } from "./errors.js";
 export declare const resolveMessageLintRules: (args: {
     messageLintRules: Array<MessageLintRule>;
 }) => {
-    data: MessageLintRule[];
+    data: Array<MessageLintRule>;
     errors: MessageLintRuleIsInvalidError[];
 };
 //# sourceMappingURL=resolveMessageLintRules.d.ts.map

package/dist/resolve-modules/message-lint-rules/resolveMessageLintRules.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"resolveMessageLintRules.d.ts","sourceRoot":"","sources":["../../../src/resolve-modules/message-lint-rules/resolveMessageLintRules.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,eAAe,EAAE,MAAM,2BAA2B,CAAA;AAC3D,OAAO,EAAE,6BAA6B,EAAE,MAAM,aAAa,CAAA;AAE3D,eAAO,MAAM,uBAAuB,SAAU;IAAE,gBAAgB,EAAE,MAAM,eAAe,CAAC,CAAA;CAAE;;;CAqBzF,CAAA"}
+{"version":3,"file":"resolveMessageLintRules.d.ts","sourceRoot":"","sources":["../../../src/resolve-modules/message-lint-rules/resolveMessageLintRules.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,eAAe,EAAE,MAAM,2BAA2B,CAAA;AAC3D,OAAO,EAAE,6BAA6B,EAAE,MAAM,aAAa,CAAA;AAE3D,eAAO,MAAM,uBAAuB,SAAU;IAAE,gBAAgB,EAAE,KAAK,CAAC,eAAe,CAAC,CAAA;CAAE;UAE5E,KAAK,CAAC,eAAe,CAAC;YACpB,6BAA6B,EAAE;CAmB9C,CAAA"}

package/dist/resolve-modules/message-lint-rules/resolveMessageLintRules.js

@@ -10,6 +10,7 @@ export const resolveMessageLintRules = (args) => {
         if (Value.Check(MessageLintRule, rule) === false) {
             const errors = [...Value.Errors(MessageLintRule, rule)];
             result.errors.push(new MessageLintRuleIsInvalidError({
+                // @ts-ignore
                 id: rule.id,
                 errors,
             }));

package/dist/storage/helper.d.ts

@@ -6,24 +6,6 @@ export declare function getPathFromMessageId(id: string): string;
  * This produces a deterministic result when passed to stringify
  * independent of the initialization order.
  */
-export declare function normalizeMessage(message: Message): {
-    id: string;
-    alias: Record<string, string>;
-    selectors: {
-        type: "VariableReference";
-        name: string;
-    }[];
-    variants: {
-        languageTag: string;
-        match: string[];
-        pattern: ({
-            type: "Text";
-            value: string;
-        } | {
-            type: "VariableReference";
-            name: string;
-        })[];
-    }[];
-};
+export declare function normalizeMessage(message: Message): Message;
 export declare function stringifyMessage(message: Message): string;
 //# sourceMappingURL=helper.d.ts.map

package/dist/storage/helper.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"helper.d.ts","sourceRoot":"","sources":["../../src/storage/helper.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,OAAO,EAAW,MAAM,2BAA2B,CAAA;AAI5D,wBAAgB,oBAAoB,CAAC,IAAI,EAAE,MAAM,sBAahD;AAED,wBAAgB,oBAAoB,CAAC,EAAE,EAAE,MAAM,UAG9C;AAED;;;;GAIG;AACH,wBAAgB,gBAAgB,CAAC,OAAO,EAAE,OAAO;;;;;;;;;;;;;;;;;;EAwChD;AAED,wBAAgB,gBAAgB,CAAC,OAAO,EAAE,OAAO,UAEhD"}
+{"version":3,"file":"helper.d.ts","sourceRoot":"","sources":["../../src/storage/helper.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,OAAO,EAAW,MAAM,2BAA2B,CAAA;AAI5D,wBAAgB,oBAAoB,CAAC,IAAI,EAAE,MAAM,sBAahD;AAED,wBAAgB,oBAAoB,CAAC,EAAE,EAAE,MAAM,UAG9C;AAED;;;;GAIG;AACH,wBAAgB,gBAAgB,CAAC,OAAO,EAAE,OAAO,GAuChB,OAAO,CACvC;AAED,wBAAgB,gBAAgB,CAAC,OAAO,EAAE,OAAO,UAEhD"}

package/dist/telemetry/capture.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"capture.d.ts","sourceRoot":"","sources":["../../src/telemetry/capture.ts"],"names":[],"mappings":"AAEA;;;;;GAKG;AACH,QAAA,MAAM,MAAM,iCAAkC,CAAA;AAE9C;;;;GAIG;AACH,eAAO,MAAM,OAAO,UACZ,CAAC,aAAa,CAAC,CAAC,MAAM,CAAC,QACxB;IACL,SAAS,EAAE,MAAM,CAAA;IACjB;;OAEG;IACH,UAAU,EAAE,OAAO,MAAM,EAAE,GAAG,CAAC,CAAA;CAC/B,kBAyBD,CAAA"}
+{"version":3,"file":"capture.d.ts","sourceRoot":"","sources":["../../src/telemetry/capture.ts"],"names":[],"mappings":"AAEA;;;;;GAKG;AACH,QAAA,MAAM,MAAM,iCAAkC,CAAA;AAE9C;;;;GAIG;AACH,eAAO,MAAM,OAAO,UACZ,CAAC,OAAO,MAAM,EAAE,MAAM,CAAC,QACxB;IACL,SAAS,EAAE,MAAM,CAAA;IACjB;;OAEG;IACH,UAAU,EAAE,MAAM,CAAC,MAAM,EAAE,GAAG,CAAC,CAAA;CAC/B,kBAyBD,CAAA"}

package/dist/telemetry/groupIdentify.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"groupIdentify.d.ts","sourceRoot":"","sources":["../../src/telemetry/groupIdentify.ts"],"names":[],"mappings":"AAEA;;;;GAIG;AACH,eAAO,MAAM,eAAe,SAAgB;IAC3C,SAAS,EAAE,MAAM,CAAA;IACjB;;OAEG;IACH,UAAU,EAAE,OAAO,MAAM,EAAE,GAAG,CAAC,CAAA;CAC/B,kBA2BA,CAAA"}
+{"version":3,"file":"groupIdentify.d.ts","sourceRoot":"","sources":["../../src/telemetry/groupIdentify.ts"],"names":[],"mappings":"AAEA;;;;GAIG;AACH,eAAO,MAAM,eAAe,SAAgB;IAC3C,SAAS,EAAE,MAAM,CAAA;IACjB;;OAEG;IACH,UAAU,EAAE,MAAM,CAAC,MAAM,EAAE,GAAG,CAAC,CAAA;CAC/B,kBA2BA,CAAA"}

package/dist/test-utilities/createMessage.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"createMessage.d.ts","sourceRoot":"","sources":["../../src/test-utilities/createMessage.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAW,OAAO,EAAE,MAAM,iBAAiB,CAAA;AAEvD,eAAO,MAAM,aAAa,OAAQ,MAAM,YAAY,OAAO,MAAM,EAAE,OAAO,GAAG,MAAM,CAAC;;;;;;;;;;;;;;;CAkB/D,CAAA"}
+{"version":3,"file":"createMessage.d.ts","sourceRoot":"","sources":["../../src/test-utilities/createMessage.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAW,OAAO,EAAE,MAAM,iBAAiB,CAAA;AAEvD,eAAO,MAAM,aAAa,OAAQ,MAAM,YAAY,MAAM,CAAC,MAAM,EAAE,OAAO,GAAG,MAAM,CAAC;;;;;;;;;;;;;;;CAkB/D,CAAA"}

package/dist/v2/helper.d.ts

@@ -35,95 +35,7 @@ export declare function addSlots(messageBundle: MessageBundle, locales: string[]
 /**
  * remove empty message slots without first creating a structured clone
  */
-export declare function removeSlots(messageBundle: MessageBundleWithSlots): {
-    id: string;
-    alias: Record<string, string>;
-    messages: {
-        selectors: {
-            annotation?: {
-                type: "function";
-                name: string;
-                options: {
-                    name: string;
-                    value: {
-                        type: "literal";
-                        value: string;
-                    } | {
-                        type: "variable";
-                        name: string;
-                    };
-                }[];
-            } | undefined;
-            type: "expression";
-            arg: {
-                type: "literal";
-                value: string;
-            } | {
-                type: "variable";
-                name: string;
-            };
-        }[];
-        variants: {
-            match: string[];
-            pattern: ({
-                type: "text";
-                value: string;
-            } | {
-                annotation?: {
-                    type: "function";
-                    name: string;
-                    options: {
-                        name: string;
-                        value: {
-                            type: "literal";
-                            value: string;
-                        } | {
-                            type: "variable";
-                            name: string;
-                        };
-                    }[];
-                } | undefined;
-                type: "expression";
-                arg: {
-                    type: "literal";
-                    value: string;
-                } | {
-                    type: "variable";
-                    name: string;
-                };
-            })[];
-        }[];
-        locale: string;
-        declarations: {
-            type: "input";
-            name: string;
-            value: {
-                annotation?: {
-                    type: "function";
-                    name: string;
-                    options: {
-                        name: string;
-                        value: {
-                            type: "literal";
-                            value: string;
-                        } | {
-                            type: "variable";
-                            name: string;
-                        };
-                    }[];
-                } | undefined;
-                type: "expression";
-                arg: {
-                    type: "literal";
-                    value: string;
-                } | {
-                    type: "variable";
-                    name: string;
-                };
-            };
-        }[];
-    }[];
-};
+export declare function removeSlots(messageBundle: MessageBundleWithSlots): MessageBundle;
 /**
  * Add newlines between bundles and messages to avoid merge conflicts
  */

package/dist/v2/helper.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"helper.d.ts","sourceRoot":"","sources":["../../src/v2/helper.ts"],"names":[],"mappings":"AAAA,OAAO,EACN,WAAW,EACX,aAAa,EACb,sBAAsB,EACtB,OAAO,EACP,WAAW,EACX,IAAI,EACJ,MAAM,YAAY,CAAA;AAEnB;;;;;;;;;GASG;AACH,wBAAgB,mBAAmB,CAAC,IAAI,EAAE;IACzC,EAAE,EAAE,MAAM,CAAA;IACV,QAAQ,EAAE,OAAO,EAAE,CAAA;IACnB,KAAK,CAAC,EAAE,aAAa,CAAC,OAAO,CAAC,CAAA;CAC9B,GAAG,aAAa,CAMhB;AAED;;;GAGG;AACH,wBAAgB,aAAa,CAAC,IAAI,EAAE;IACnC,MAAM,EAAE,WAAW,CAAA;IACnB,IAAI,EAAE,MAAM,CAAA;IACZ,KAAK,CAAC,EAAE,KAAK,CAAC,MAAM,CAAC,CAAA;CACrB,GAAG,OAAO,CAOV;AAED,wBAAgB,aAAa,CAAC,IAAI,EAAE,MAAM,GAAG,IAAI,CAKhD;AAMD;;GAEG;AACH,wBAAgB,iBAAiB,CAAC,MAAM,EAAE,WAAW,GAAG,WAAW,CAKlE;AAED;;GAEG;AACH,wBAAgB,QAAQ,CAAC,aAAa,EAAE,aAAa,EAAE,OAAO,EAAE,MAAM,EAAE,GAAG,sBAAsB,CAMhG;AAED;;GAEG;AACH,wBAAgB,WAAW,CAAC,aAAa,EAAE,sBAAsB;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;EAGhE;AAED;;GAEG;AACH,wBAAgB,kBAAkB,CAAC,IAAI,EAAE,MAAM,GAAG,MAAM,CAMvD"}
+{"version":3,"file":"helper.d.ts","sourceRoot":"","sources":["../../src/v2/helper.ts"],"names":[],"mappings":"AAAA,OAAO,EACN,WAAW,EACX,aAAa,EACb,sBAAsB,EACtB,OAAO,EACP,WAAW,EACX,IAAI,EACJ,MAAM,YAAY,CAAA;AAEnB;;;;;;;;;GASG;AACH,wBAAgB,mBAAmB,CAAC,IAAI,EAAE;IACzC,EAAE,EAAE,MAAM,CAAA;IACV,QAAQ,EAAE,OAAO,EAAE,CAAA;IACnB,KAAK,CAAC,EAAE,aAAa,CAAC,OAAO,CAAC,CAAA;CAC9B,GAAG,aAAa,CAMhB;AAED;;;GAGG;AACH,wBAAgB,aAAa,CAAC,IAAI,EAAE;IACnC,MAAM,EAAE,WAAW,CAAA;IACnB,IAAI,EAAE,MAAM,CAAA;IACZ,KAAK,CAAC,EAAE,KAAK,CAAC,MAAM,CAAC,CAAA;CACrB,GAAG,OAAO,CAOV;AAED,wBAAgB,aAAa,CAAC,IAAI,EAAE,MAAM,GAAG,IAAI,CAKhD;AAMD;;GAEG;AACH,wBAAgB,iBAAiB,CAAC,MAAM,EAAE,WAAW,GAAG,WAAW,CAKlE;AAED;;GAEG;AACH,wBAAgB,QAAQ,CAAC,aAAa,EAAE,aAAa,EAAE,OAAO,EAAE,MAAM,EAAE,GAAG,sBAAsB,CAMhG;AAED;;GAEG;AACH,wBAAgB,WAAW,CAAC,aAAa,EAAE,sBAAsB,GAExC,aAAa,CACrC;AAED;;GAEG;AACH,wBAAgB,kBAAkB,CAAC,IAAI,EAAE,MAAM,GAAG,MAAM,CAMvD"}

package/dist/validateProjectPath.d.ts

@@ -1,23 +1,17 @@
-import type { NodeishFilesystem } from "@lix-js/fs";
 /**
  * validate that a project path is absolute and ends with {name}.inlang.
  *
  * @throws if the path is not valid.
  */
-export declare function assertValidProjectPath(projectPath: string): void;
+export declare function assertValidProjectPath(projectPath: string): asserts projectPath is `${string}.inlang`;
 /**
  * tests whether a path ends with {name}.inlang
  * (does not remove trailing slash)
  */
-export declare function isInlangProjectPath(path: string): boolean;
+export declare function isInlangProjectPath(path: string): path is `${string}.inlang`;
 /**
  * tests whether a path starts with a forward slash (/) or a windows-style
  * drive letter (C: or D:, etc.) followed by a slash
  */
 export declare function isAbsolutePath(path: string): boolean;
-/**
- * Returns true if the path exists (file or directory), false otherwise.
- *
- */
-export declare function pathExists(filePath: string, nodeishFs: NodeishFilesystem): Promise<boolean>;
 //# sourceMappingURL=validateProjectPath.d.ts.map

package/dist/validateProjectPath.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"validateProjectPath.d.ts","sourceRoot":"","sources":["../src/validateProjectPath.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,iBAAiB,EAAE,MAAM,YAAY,CAAA;AAEnD;;;;GAIG;AACH,wBAAgB,sBAAsB,CAAC,WAAW,EAAE,MAAM,QASzD;AAED;;;GAGG;AACH,wBAAgB,mBAAmB,CAAC,IAAI,EAAE,MAAM,WAE/C;AAED;;;GAGG;AACH,wBAAgB,cAAc,CAAC,IAAI,EAAE,MAAM,WAO1C;AAED;;;GAGG;AACH,wBAAsB,UAAU,CAAC,QAAQ,EAAE,MAAM,EAAE,SAAS,EAAE,iBAAiB,oBAc9E"}
+{"version":3,"file":"validateProjectPath.d.ts","sourceRoot":"","sources":["../src/validateProjectPath.ts"],"names":[],"mappings":"AAAA;;;;GAIG;AACH,wBAAgB,sBAAsB,CACrC,WAAW,EAAE,MAAM,GACjB,OAAO,CAAC,WAAW,IAAI,GAAG,MAAM,SAAS,CAS3C;AAED;;;GAGG;AACH,wBAAgB,mBAAmB,CAAC,IAAI,EAAE,MAAM,GAAG,IAAI,IAAI,GAAG,MAAM,SAAS,CAE5E;AAED;;;GAGG;AACH,wBAAgB,cAAc,CAAC,IAAI,EAAE,MAAM,WAO1C"}

package/dist/validateProjectPath.js

@@ -29,24 +29,3 @@ export function isAbsolutePath(path) {
     // 	/^(?:[A-Za-z]:\\(?:[^\\]+\\)*[^\\]+|[A-Za-z]:\/(?:[^/]+\/)*[^/]+|\/(?:[^/]+\/)*[^/]+)$/
     // return matchPosixAndWindowsAbsolutePaths.test(path)
 }
-/**
- * Returns true if the path exists (file or directory), false otherwise.
- *
- */
-export async function pathExists(filePath, nodeishFs) {
-    // from paraglide-js/src/services/file-handling/exists.ts
-    // TODO: add fs.exists to @lix-js/fs
-    try {
-        await nodeishFs.stat(filePath);
-        return true;
-    }
-    catch (error) {
-        //@ts-ignore
-        if (error.code === "ENOENT") {
-            return false;
-        }
-        else {
-            throw new Error(`Failed to check if path exists: ${error}`, { cause: error });
-        }
-    }
-}

package/dist/validateProjectPath.test.js

@@ -1,6 +1,5 @@
-import { assert, describe, expect, it } from "vitest";
-import { assertValidProjectPath, isAbsolutePath, isInlangProjectPath, pathExists, } from "./validateProjectPath.js";
-import { mockRepo } from "@lix-js/client";
+import { assert, describe, it } from "vitest";
+import { assertValidProjectPath, isAbsolutePath, isInlangProjectPath, } from "./validateProjectPath.js";
 describe("isAbsolutePath", () => {
     it("should correctly identify Unix absolute paths", () => {
         assert.isTrue(isAbsolutePath("/home/user/documents/file.txt"));
@@ -39,18 +38,3 @@ describe("assertValidProjectPath", () => {
         assert.throws(() => assertValidProjectPath("/path/to/green-elephant.inlang/settings.json"));
     });
 });
-// moar tests in paraglide-js/src/services/file-handling/exists.test.ts
-describe("pathExists", () => {
-    it("should work for files", async () => {
-        const repo = await mockRepo();
-        await repo.nodeishFs.writeFile("/test.txt", "hello");
-        expect(await pathExists("/test.txt", repo.nodeishFs)).toBe(true);
-        expect(await pathExists("/does-not-exist.txt", repo.nodeishFs)).toBe(false);
-    });
-    it("should work for directories", async () => {
-        const repo = await mockRepo();
-        await repo.nodeishFs.mkdir("/test/project.inlang", { recursive: true });
-        expect(await pathExists("/test/project.inlang", repo.nodeishFs)).toBe(true);
-        expect(await pathExists("/test/white-gorilla.inlang", repo.nodeishFs)).toBe(false);
-    });
-});

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@inlang/sdk",
   "type": "module",
-  "version": "0.36.1",
+  "version": "0.36.2",
   "license": "Apache-2.0",
   "publishConfig": {
     "access": "public"
@@ -37,14 +37,14 @@
     "@inlang/json-types": "1.1.0",
     "@inlang/language-tag": "1.5.1",
     "@inlang/message": "2.1.0",
-    "@inlang/module": "1.2.13",
     "@inlang/message-lint-rule": "1.4.7",
     "@inlang/plugin": "2.4.13",
+    "@inlang/module": "1.2.13",
     "@inlang/project-settings": "2.4.2",
-    "@lix-js/client": "2.2.0",
-    "@lix-js/fs": "2.1.0",
     "@inlang/result": "1.1.0",
-    "@inlang/translatable": "1.3.1"
+    "@inlang/translatable": "1.3.1",
+    "@lix-js/fs": "2.1.0",
+    "@lix-js/client": "2.2.0"
   },
   "devDependencies": {
     "@types/debug": "^4.1.12",
@@ -54,7 +54,7 @@
     "jsdom": "22.1.0",
     "patch-package": "6.5.1",
     "tsd": "^0.25.0",
-    "typescript": "5.2.2",
+    "typescript": "^5.5.2",
     "vitest": "^0.33.0"
   },
   "scripts": {

package/src/createMessagesQuery.ts

@@ -107,7 +107,7 @@ export function createMessagesQuery({
 			nodeishFs: nodeishFs,
 			// this message is called whenever a file changes that was read earlier by this filesystem
 			// - the plugin loads messages -> reads the file messages.json -> start watching on messages.json -> updateMessages
-			updateMessages: () => {
+			onChange: () => {
 				// reload
 				loadMessagesViaPlugin(
 					fsWithWatcher,

package/src/createNewProject.ts

@@ -1,6 +1,7 @@
 import type { Repository } from "@lix-js/client"
+import type { NodeishFilesystem } from "@lix-js/fs"
 import { ProjectSettings } from "@inlang/project-settings"
-import { assertValidProjectPath, pathExists } from "./validateProjectPath.js"
+import { assertValidProjectPath } from "./validateProjectPath.js"
 import { defaultProjectSettings } from "./defaultProjectSettings.js"
 
 /**
@@ -15,7 +16,7 @@ export async function createNewProject(args: {
 	assertValidProjectPath(args.projectPath)
 
 	const nodeishFs = args.repo.nodeishFs
-	if (await pathExists(args.projectPath, nodeishFs)) {
+	if (await directoryExists(args.projectPath, nodeishFs)) {
 		throw new Error(`projectPath already exists, received "${args.projectPath}"`)
 	}
 	const settingsText = JSON.stringify(args.projectSettings ?? defaultProjectSettings, undefined, 2)
@@ -27,3 +28,19 @@ export async function createNewProject(args: {
 		nodeishFs.mkdir(`${args.projectPath}/cache/modules`, { recursive: true }),
 	])
 }
+
+/**
+ * Returns true if the path exists (file or directory), false otherwise.
+ */
+async function directoryExists(filePath: string, nodeishFs: NodeishFilesystem) {
+	try {
+		const stat = await nodeishFs.stat(filePath)
+		return stat.isDirectory()
+	} catch (error: any) {
+		if (error && "code" in error && error.code === "ENOENT") {
+			return false
+		} else {
+			throw new Error(`Failed to check if path exists: ${error}`, { cause: error })
+		}
+	}
+}

package/src/createNodeishFsWithWatcher.test.ts

@@ -8,7 +8,7 @@ describe("watcher", () => {
 
 		const fs = createNodeishFsWithWatcher({
 			nodeishFs: createNodeishMemoryFs(),
-			updateMessages: () => {
+			onChange: () => {
 				counter++
 			},
 		})

package/src/createNodeishFsWithWatcher.ts

@@ -8,67 +8,78 @@ import type { NodeishFilesystem } from "@lix-js/fs"
  */
 export const createNodeishFsWithWatcher = (args: {
 	nodeishFs: NodeishFilesystem
-	updateMessages: () => void
+	onChange: () => void
 }): NodeishFilesystem & {
 	stopWatching: () => void
 } => {
-	const pathList: string[] = []
-	let abortControllers: AbortController[] = []
+	const pathList = new Set<string>()
+	const abortControllers = new Set<AbortController>()
 
 	const stopWatching = () => {
 		for (const ac of abortControllers) {
 			ac.abort()
+			abortControllers.delete(ac) // release reference
 		}
-		// release references
-		abortControllers = []
 	}
 
-	const makeWatcher = (path: string) => {
-		;(async () => {
-			try {
-				const ac = new AbortController()
-				abortControllers.push(ac)
-				const watcher = args.nodeishFs.watch(path, {
-					signal: ac.signal,
-					persistent: false,
-				})
-				if (watcher) {
-					//eslint-disable-next-line @typescript-eslint/no-unused-vars
-					for await (const event of watcher) {
-						args.updateMessages()
-					}
+	const makeWatcher = async (path: string) => {
+		try {
+			const ac = new AbortController()
+			abortControllers.add(ac)
+			const watcher = args.nodeishFs.watch(path, {
+				signal: ac.signal,
+				recursive: true,
+				persistent: false,
+			})
+			if (watcher) {
+				//eslint-disable-next-line @typescript-eslint/no-unused-vars
+				for await (const event of watcher) {
+					// whenever the watcher changes we need to update the messages
+					args.onChange()
 				}
-			} catch (err: any) {
-				if (err.name === "AbortError") return
-				// https://github.com/opral/monorepo/issues/1647
-				// the file does not exist (yet)
-				// this is not testable beacause the fs.watch api differs
-				// from node and lix. lenghty
-				else if (err.code === "ENOENT") return
-				throw err
 			}
-		})()
+		} catch (err: any) {
+			if (err.name === "AbortError") return
+			// https://github.com/opral/monorepo/issues/1647
+			// the file does not exist (yet)
+			// this is not testable beacause the fs.watch api differs
+			// from node and lix. lenghty
+			else if (err.code === "ENOENT") return
+			throw err
+		}
 	}
 
-	const readFileAndExtractPath = (path: string, options: { encoding: "utf-8" | "binary" }) => {
-		if (!pathList.includes(path)) {
-			makeWatcher(path)
-			pathList.push(path)
+	/**
+	 * Creates watchers on-the-fly for any file or directory that is not yet watched.
+	 *
+	 * We do this instead of recursively watching the entire project because fs.watch does not support
+	 * recursive watching on linux in node 18. Once node 18 support is dropped this can be drastically simplified.
+	 */
+	const watched = <T extends any[], R>(
+		fn: (path: string, ...rest: T) => R
+	): ((path: string, ...rest: T) => R) => {
+		return (path: string, ...rest: T): R => {
+			if (!pathList.has(path)) {
+				makeWatcher(path)
+				pathList.add(path)
+			}
+			return fn(path, ...rest)
 		}
-		return args.nodeishFs.readFile(path, options)
 	}
 
 	return {
+		...args.nodeishFs,
+		/**
+		 * Reads the file and automatically adds it to the list of watched files.
+		 * Any changes to the file will trigger a message update.
+		 */
 		// @ts-expect-error
-		readFile: (path: string, options: { encoding: "utf-8" | "binary" }) =>
-			readFileAndExtractPath(path, options),
-		rm: args.nodeishFs.rm,
-		readdir: args.nodeishFs.readdir,
-		mkdir: args.nodeishFs.mkdir,
-		rmdir: (args.nodeishFs as any).rmdir,
-		writeFile: args.nodeishFs.writeFile,
-		watch: args.nodeishFs.watch,
-		stat: args.nodeishFs.stat,
+		readFile: watched(args.nodeishFs.readFile),
+		/**
+		 * Reads the directory and automatically adds it to the list of watched files.
+		 * Any changes to the directory will trigger a message update.
+		 */
+		readdir: watched(args.nodeishFs.readdir),
 		stopWatching,
 	}
 }

package/src/loadProject.test.ts

@@ -278,26 +278,19 @@ describe("initialization", () => {
 		expect(result.data).toBeDefined()
 	})
 
-	// TODO: fix this test
-	// https://github.com/opral/inlang-message-sdk/issues/76
-	// it doesn't work because failure to open the settings file doesn't throw
-	// errors are returned in project.errors()
-	it("should resolve from a windows path", async () => {
+	it.skipIf(() => process.platform !== "win32")("should resolve from a windows path", async () => {
 		const repo = await mockRepo()
 		const fs = repo.nodeishFs
 		await fs.mkdir("C:\\Users\\user\\project.inlang", { recursive: true })
 		await fs.writeFile("C:\\Users\\user\\project.inlang\\settings.json", JSON.stringify(settings))
 
-		const result = await tryCatch(() =>
-			loadProject({
-				projectPath: "C:\\Users\\user\\project.inlang",
-				repo,
-				_import,
-			})
-		)
+		const project = await loadProject({
+			projectPath: "C:\\Users\\user\\project.inlang",
+			repo,
+			_import,
+		})
 
-		expect(result.error).toBeUndefined()
-		expect(result.data).toBeDefined()
+		expect(project.errors()).toEqual([])
 	})
 
 	describe("settings", () => {
@@ -362,9 +355,9 @@ describe("initialization", () => {
 		it("should not re-write the settings to disk when initializing", async () => {
 			const repo = await mockRepo()
 			const fs = repo.nodeishFs
-			const settingsWithDeifferentFormatting = JSON.stringify(settings, undefined, 4)
+			const settingsWithDifferentFormatting = JSON.stringify(settings, undefined, 4)
 			await fs.mkdir("/user/project.inlang", { recursive: true })
-			await fs.writeFile("/user/project.inlang/settings.json", settingsWithDeifferentFormatting)
+			await fs.writeFile("/user/project.inlang/settings.json", settingsWithDifferentFormatting)
 
 			const project = await loadProject({
 				projectPath: "/user/project.inlang",
@@ -375,7 +368,7 @@ describe("initialization", () => {
 			const settingsOnDisk = await fs.readFile("/user/project.inlang/settings.json", {
 				encoding: "utf-8",
 			})
-			expect(settingsOnDisk).toBe(settingsWithDeifferentFormatting)
+			expect(settingsOnDisk).toBe(settingsWithDifferentFormatting)
 
 			project.setSettings(project.settings())
 			// TODO: how can we await `setsettings` correctly
@@ -384,7 +377,7 @@ describe("initialization", () => {
 			const newsettingsOnDisk = await fs.readFile("/user/project.inlang/settings.json", {
 				encoding: "utf-8",
 			})
-			expect(newsettingsOnDisk).not.toBe(settingsWithDeifferentFormatting)
+			expect(newsettingsOnDisk).not.toBe(settingsWithDifferentFormatting)
 		})
 	})