@inlang/sdk 0.18.0 → 0.19.0

package/package.json

@@ -1,57 +1,57 @@
 {
-	"name": "@inlang/sdk",
-	"type": "module",
-	"version": "0.18.0",
-	"license": "Apache-2.0",
-	"publishConfig": {
-		"access": "public"
-	},
-	"exports": {
-		".": "./dist/index.js",
-		"./test-utilities": "./dist/test-utilities/index.js",
-		"./lint": "./dist/lint/index.js",
-		"./messages": "./dist/messages/index.js"
-	},
-	"files": [
-		"./dist",
-		"./src"
-	],
-	"scripts": {
-		"build": "tsc --build",
-		"dev": "tsc --watch",
-		"test": "tsc --noEmit && vitest run --passWithNoTests --coverage",
-		"lint": "eslint ./src --fix",
-		"format": "prettier ./src --write",
-		"clean": "rm -rf ./dist ./node_modules"
-	},
-	"engines": {
-		"node": ">=18.0.0"
-	},
-	"dependencies": {
-		"@inlang/json-types": "*",
-		"@inlang/translatable": "*",
-		"@inlang/message-lint-rule": "*",
-		"@inlang/module": "*",
-		"@inlang/language-tag": "*",
-		"@inlang/message": "*",
-		"@inlang/plugin": "*",
-		"@inlang/project-settings": "*",
-		"@inlang/result": "*",
-		"@lix-js/fs": "*",
-		"@sinclair/typebox": "^0.31.17",
-		"deepmerge-ts": "^5.1.0",
-		"solid-js": "1.6.12",
-		"throttle-debounce": "5.0.0",
-		"dedent": "1.5.1"
-	},
-	"devDependencies": {
-		"@lix-js/fs": "*",
-		"@types/throttle-debounce": "5.0.0",
-		"@vitest/coverage-v8": "^0.33.0",
-		"jsdom": "22.1.0",
-		"patch-package": "6.5.1",
-		"tsd": "^0.25.0",
-		"typescript": "5.2.2",
-		"vitest": "^0.33.0"
-	}
-}
+  "name": "@inlang/sdk",
+  "type": "module",
+  "version": "0.19.0",
+  "license": "Apache-2.0",
+  "publishConfig": {
+    "access": "public"
+  },
+  "exports": {
+    ".": "./dist/index.js",
+    "./test-utilities": "./dist/test-utilities/index.js",
+    "./lint": "./dist/lint/index.js",
+    "./messages": "./dist/messages/index.js"
+  },
+  "files": [
+    "./dist",
+    "./src"
+  ],
+  "engines": {
+    "node": ">=18.0.0"
+  },
+  "dependencies": {
+    "@sinclair/typebox": "^0.31.17",
+    "deepmerge-ts": "^5.1.0",
+    "solid-js": "1.6.12",
+    "throttle-debounce": "5.0.0",
+    "dedent": "1.5.1",
+    "@inlang/json-types": "1.1.0",
+    "@inlang/message-lint-rule": "1.4.0",
+    "@inlang/translatable": "1.2.0",
+    "@inlang/module": "1.2.0",
+    "@inlang/language-tag": "1.2.0",
+    "@inlang/message": "2.0.0",
+    "@inlang/plugin": "2.4.0",
+    "@inlang/result": "1.1.0",
+    "@inlang/project-settings": "2.2.0",
+    "@lix-js/fs": "0.4.0"
+  },
+  "devDependencies": {
+    "@types/throttle-debounce": "5.0.0",
+    "@vitest/coverage-v8": "^0.33.0",
+    "jsdom": "22.1.0",
+    "patch-package": "6.5.1",
+    "tsd": "^0.25.0",
+    "typescript": "5.2.2",
+    "vitest": "^0.33.0",
+    "@lix-js/fs": "0.4.0"
+  },
+  "scripts": {
+    "build": "tsc --build",
+    "dev": "tsc --watch",
+    "test": "tsc --noEmit && vitest run --passWithNoTests --coverage",
+    "lint": "eslint ./src --fix",
+    "format": "prettier ./src --write",
+    "clean": "rm -rf ./dist ./node_modules"
+  }
+}

package/src/adapter/solidAdapter.test.ts

@@ -90,11 +90,11 @@ const $import: ImportFunction = async (name) => ({
 describe("config", () => {
 	it("should react to changes in config", async () => {
 		const fs = createNodeishMemoryFs()
-		await fs.mkdir("/user/project", { recursive: true })
-		await fs.writeFile("/user/project/project.inlang.json", JSON.stringify(config))
+		await fs.mkdir("/user/project.inlang", { recursive: true })
+		await fs.writeFile("/user/project.inlang/settings.json", JSON.stringify(config))
 		const project = solidAdapter(
 			await loadProject({
-				settingsFilePath: "/user/project/project.inlang.json",
+				projectPath: "/user/project.inlang",
 				nodeishFs: fs,
 				_import: $import,
 			}),
@@ -123,11 +123,11 @@ describe("config", () => {
 describe("installed", () => {
 	it("react to changes that are unrelated to installed items", async () => {
 		const fs = createNodeishMemoryFs()
-		await fs.mkdir("/user/project", { recursive: true })
-		await fs.writeFile("/user/project/project.inlang.json", JSON.stringify(config))
+		await fs.mkdir("/user/project.inlang", { recursive: true })
+		await fs.writeFile("/user/project.inlang/settings.json", JSON.stringify(config))
 		const project = solidAdapter(
 			await loadProject({
-				settingsFilePath: "/user/project/project.inlang.json",
+				projectPath: "/user/project.inlang",
 				nodeishFs: fs,
 				_import: $import,
 			}),
@@ -188,11 +188,11 @@ describe("messages", () => {
 
 		const mockImport: ImportFunction = async () => ({ default: mockPlugin })
 
-		await fs.mkdir("/user/project", { recursive: true })
-		await fs.writeFile("/user/project/project.inlang.json", JSON.stringify(mockConfig))
+		await fs.mkdir("/user/project.inlang.inlang", { recursive: true })
+		await fs.writeFile("/user/project.inlang.inlang/settings.json", JSON.stringify(mockConfig))
 		const project = solidAdapter(
 			await loadProject({
-				settingsFilePath: "/user/project/project.inlang.json",
+				projectPath: "/user/project.inlang.inlang",
 				nodeishFs: fs,
 				_import: mockImport,
 			}),
@@ -218,11 +218,11 @@ describe("messages", () => {
 
 	it("should react to changes in messages", async () => {
 		const fs = createNodeishMemoryFs()
-		await fs.mkdir("/user/project", { recursive: true })
-		await fs.writeFile("/user/project/project.inlang.json", JSON.stringify(config))
+		await fs.mkdir("/user/project.inlang.inlang", { recursive: true })
+		await fs.writeFile("/user/project.inlang.inlang/settings.json", JSON.stringify(config))
 		const project = solidAdapter(
 			await loadProject({
-				settingsFilePath: "/user/project/project.inlang.json",
+				projectPath: "/user/project.inlang.inlang",
 				nodeishFs: fs,
 				_import: $import,
 			}),
@@ -280,10 +280,11 @@ describe("lint", () => {
 	it.todo("should react to changes in config", async () => {
 		await createRoot(async () => {
 			const fs = createNodeishMemoryFs()
-			await fs.writeFile("./project.config.json", JSON.stringify(config))
+			await fs.mkdir("./project.inlang", { recursive: true })
+			await fs.writeFile("./project.inlang/settings.json", JSON.stringify(config))
 			const project = solidAdapter(
 				await loadProject({
-					settingsFilePath: "./project.config.json",
+					projectPath: "./project.inlang",
 					nodeishFs: fs,
 					_import: $import,
 				}),
@@ -321,7 +322,7 @@ describe("lint", () => {
 			await fs.writeFile("./project.config.json", JSON.stringify(config))
 			const project = solidAdapter(
 				await loadProject({
-					settingsFilePath: "./project.config.json",
+					projectPath: "./project.config.json",
 					nodeishFs: fs,
 					_import: $import,
 				}),

package/src/createNodeishFsWithAbsolutePaths.test.ts

@@ -2,16 +2,16 @@ import { it, expect, vi } from "vitest"
 import { createNodeishFsWithAbsolutePaths } from "./createNodeishFsWithAbsolutePaths.js"
 import type { NodeishFilesystemSubset } from "./versionedInterfaces.js"
 
-it("throws an error if settingsFilePath is not an absolute path", () => {
+it("throws an error if projectPath is not an absolute path", () => {
 	const relativePath = "relative/path"
 
 	expect(() =>
-		createNodeishFsWithAbsolutePaths({ settingsFilePath: relativePath, nodeishFs: {} as any })
+		createNodeishFsWithAbsolutePaths({ projectPath: relativePath, nodeishFs: {} as any })
 	).toThrow()
 })
 
 it("intercepts paths correctly for readFile", async () => {
-	const settingsFilePath = `/Users/samuel/Documents/paraglide/example/project.inlang.json`
+	const projectPath = `/Users/samuel/Documents/paraglide/example/project.inlang`
 
 	const filePaths = [
 		["file.txt", `/Users/samuel/Documents/paraglide/example/file.txt`],
@@ -32,7 +32,7 @@ it("intercepts paths correctly for readFile", async () => {
 	} satisfies Record<keyof NodeishFilesystemSubset, any>
 
 	const interceptedFs = createNodeishFsWithAbsolutePaths({
-		settingsFilePath,
+		projectPath,
 		nodeishFs: mockNodeishFs,
 	})
 

package/src/createNodeishFsWithAbsolutePaths.ts

@@ -6,19 +6,19 @@ import { isAbsolutePath } from "./isAbsolutePath.js"
  * Wraps the nodeish filesystem subset with a function that intercepts paths
  * and prepends the base path.
  *
- * The paths are resolved from the `settingsFilePath` argument.
+ * The paths are resolved from the `projectPath` argument.
  */
 export const createNodeishFsWithAbsolutePaths = (args: {
-	settingsFilePath: string
+	projectPath: string
 	nodeishFs: NodeishFilesystemSubset
 }): NodeishFilesystemSubset => {
-	if (!isAbsolutePath(args.settingsFilePath)) {
-		throw new Error(`Expected an absolute path but received "${args.settingsFilePath}".`)
+	if (!isAbsolutePath(args.projectPath)) {
+		throw new Error(`Expected an absolute path but received "${args.projectPath}".`)
 	}
 
 	// get the base path of the settings file by
 	// removing the file name from the path
-	const basePath = normalizePath(args.settingsFilePath).split("/").slice(0, -1).join("/")
+	const basePath = normalizePath(args.projectPath).split("/").slice(0, -1).join("/")
 
 	const makeAbsolute = (path: string) => {
 		if (isAbsolutePath(path)) {

package/src/createNodeishFsWithWatcher.ts

@@ -4,7 +4,7 @@ import type { NodeishFilesystemSubset } from "@inlang/plugin"
  * Wraps the nodeish filesystem subset with a function that intercepts paths
  * and prepends the base path.
  *
- * The paths are resolved from the `settingsFilePath` argument.
+ * The paths are resolved from the `projectPath` argument.
  */
 export const createNodeishFsWithWatcher = (args: {
 	nodeishFs: NodeishFilesystemSubset
@@ -20,9 +20,11 @@ export const createNodeishFsWithWatcher = (args: {
 					signal: abortController.signal,
 					persistent: false,
 				})
-				//eslint-disable-next-line @typescript-eslint/no-unused-vars
-				for await (const event of watcher) {
-					args.updateMessages()
+				if (watcher) {
+					//eslint-disable-next-line @typescript-eslint/no-unused-vars
+					for await (const event of watcher) {
+						args.updateMessages()
+					}
 				}
 			} catch (err: any) {
 				if (err.name === "AbortError") return

package/src/isAbsolutePath.test.ts

@@ -10,7 +10,7 @@ describe("isAbsolutePath", () => {
 
 	it("should correctly identify Windows absolute paths", () => {
 		assert.isTrue(isAbsolutePath("C:\\Users\\User\\Documents\\File.txt"))
-		assert.isTrue(isAbsolutePath("C:/Users/user/project/project.inlang.json"))
+		assert.isTrue(isAbsolutePath("C:/Users/user/project.inlang/settings.json"))
 		assert.isFalse(isAbsolutePath("Projects\\Project1\\source\\file.txt"))
 	})