@inlang/sdk 0.6.0 → 0.8.0

package/src/resolve-modules/import.test.ts

@@ -12,7 +12,7 @@ describe("$import", async () => {
 		export function hello() {
 			return "hello";
 		}
-		`,
+		`
 	)
 
 	// mock module in a directory
@@ -23,7 +23,7 @@ describe("$import", async () => {
 		export function hello() {
 			return "world";
 		}
-		`,
+		`
 	)
 
 	const _import = createImport({

package/src/resolve-modules/import.ts

@@ -35,7 +35,7 @@ async function $import(
 		 * Required to import from a local path.
 		 */
 		readFile: NodeishFilesystemSubset["readFile"]
-	},
+	}
 ): Promise<any> {
 	let moduleAsText: string
 

package/src/resolve-modules/message-lint-rules/resolveMessageLintRules.ts

@@ -12,7 +12,7 @@ export const resolveMessageLintRules = (args: { messageLintRules: Array<MessageL
 			result.errors.push(
 				new MessageLintRuleIsInvalidError(`Couldn't parse lint rule "${rule.id}"`, {
 					module: "not implemented",
-				}),
+				})
 			)
 			continue
 		} else {

package/src/resolve-modules/plugins/errors.ts

@@ -1,7 +1,7 @@
 import type { Plugin } from "@inlang/plugin"
 
 type PluginErrorOptions = {
-	plugin: Plugin["id"]
+	plugin: Plugin["id"] | undefined
 } & Partial<Error>
 
 class PluginError extends Error {
@@ -10,7 +10,7 @@ class PluginError extends Error {
 	constructor(message: string, options: PluginErrorOptions) {
 		super(message)
 		this.name = "PluginError"
-		this.plugin = options.plugin
+		this.plugin = options.plugin ?? "unknown"
 	}
 }
 

package/src/resolve-modules/plugins/resolvePlugins.test.ts

@@ -91,7 +91,7 @@ describe("loadMessages", () => {
 			await resolved.data.loadMessages!({
 				languageTags: ["en"],
 				sourceLanguageTag: "en",
-			}),
+			})
 		).toEqual([{ id: "test", expressions: [], selectors: [], variants: [] }])
 	})
 
@@ -198,37 +198,6 @@ describe("saveMessages", () => {
 	})
 })
 
-describe("detectedLanguageTags", () => {
-	it("should merge language tags from plugins", async () => {
-		const mockPlugin: Plugin = {
-			id: "plugin.namepsace.detectedLanguageTags",
-			description: { en: "My plugin description" },
-			displayName: { en: "My plugin" },
-			detectedLanguageTags: async () => ["de", "en"],
-			addCustomApi: () => {
-				return {}
-			},
-		}
-		const mockPlugin2: Plugin = {
-			id: "plugin.namepsace.detectedLanguageTags2",
-			description: { en: "My plugin description" },
-			displayName: { en: "My plugin" },
-			addCustomApi: () => {
-				return {}
-			},
-			detectedLanguageTags: async () => ["de", "fr"],
-		}
-
-		const resolved = await resolvePlugins({
-			plugins: [mockPlugin, mockPlugin2],
-			settings: {},
-			nodeishFs: {} as any,
-		})
-
-		expect(resolved.data.detectedLanguageTags).toEqual(["de", "en", "fr"])
-	})
-})
-
 describe("addCustomApi", () => {
 	it("it should resolve app specific api", async () => {
 		const mockPlugin: Plugin = {
@@ -329,7 +298,7 @@ describe("addCustomApi", () => {
 
 		expect(resolved.data.customApi).toHaveProperty("app.inlang.placeholder")
 		expect(
-			(resolved.data.customApi?.["app.inlang.placeholder"] as any).messageReferenceMatcher(),
+			(resolved.data.customApi?.["app.inlang.placeholder"] as any).messageReferenceMatcher()
 		).toEqual({
 			hello: "world",
 		})

package/src/resolve-modules/plugins/resolvePlugins.ts

@@ -27,7 +27,6 @@ export const resolvePlugins: ResolvePluginsFunction = async (args) => {
 		data: {
 			loadMessages: undefined as any,
 			saveMessages: undefined as any,
-			detectedLanguageTags: [],
 			customApi: {},
 		},
 		errors: [],
@@ -46,8 +45,8 @@ export const resolvePlugins: ResolvePluginsFunction = async (args) => {
 			result.errors.push(
 				new PluginHasInvalidIdError(
 					`Plugin ${plugin.id} has an invalid id "${plugin.id}". It must be kebap-case and contain a namespace like project.my-plugin.`,
-					{ plugin: plugin.id },
-				),
+					{ plugin: plugin.id }
+				)
 			)
 		}
 
@@ -58,8 +57,8 @@ export const resolvePlugins: ResolvePluginsFunction = async (args) => {
 					`Plugin ${plugin.id} uses reserved namespace 'inlang'.`,
 					{
 						plugin: plugin.id,
-					},
-				),
+					}
+				)
 			)
 		}
 
@@ -71,8 +70,8 @@ export const resolvePlugins: ResolvePluginsFunction = async (args) => {
 					{
 						plugin: plugin.id,
 						cause: errors,
-					},
-				),
+					}
+				)
 			)
 		}
 
@@ -81,8 +80,8 @@ export const resolvePlugins: ResolvePluginsFunction = async (args) => {
 			result.errors.push(
 				new PluginLoadMessagesFunctionAlreadyDefinedError(
 					`Plugin ${plugin.id} defines the loadMessages function, but it was already defined by another plugin.`,
-					{ plugin: plugin.id },
-				),
+					{ plugin: plugin.id }
+				)
 			)
 		}
 
@@ -90,8 +89,8 @@ export const resolvePlugins: ResolvePluginsFunction = async (args) => {
 			result.errors.push(
 				new PluginSaveMessagesFunctionAlreadyDefinedError(
 					`Plugin ${plugin.id} defines the saveMessages function, but it was already defined by another plugin.`,
-					{ plugin: plugin.id },
-				),
+					{ plugin: plugin.id }
+				)
 			)
 		}
 
@@ -101,7 +100,7 @@ export const resolvePlugins: ResolvePluginsFunction = async (args) => {
 			const { data: customApi, error } = tryCatch(() =>
 				plugin.addCustomApi!({
 					settings: args.settings?.[plugin.id] ?? {},
-				}),
+				})
 			)
 			if (error) {
 				// @ts-ignore
@@ -112,8 +111,8 @@ export const resolvePlugins: ResolvePluginsFunction = async (args) => {
 				result.errors.push(
 					new PluginReturnedInvalidCustomApiError(
 						`Plugin ${plugin.id} defines the addCustomApi function, but it does not return an object.`,
-						{ plugin: plugin.id, cause: error },
-					),
+						{ plugin: plugin.id, cause: error }
+					)
 				)
 			}
 		}
@@ -145,21 +144,11 @@ export const resolvePlugins: ResolvePluginsFunction = async (args) => {
 				})
 		}
 
-		if (typeof plugin.detectedLanguageTags === "function") {
-			const detectedLangugeTags = await plugin.detectedLanguageTags!({
-				settings: args.settings?.[plugin.id] ?? {},
-				nodeishFs: args.nodeishFs,
-			})
-			result.data.detectedLanguageTags = [
-				...new Set([...result.data.detectedLanguageTags, ...detectedLangugeTags]),
-			]
-		}
-
 		if (typeof plugin.addCustomApi === "function") {
 			const { data: customApi } = tryCatch(() =>
 				plugin.addCustomApi!({
 					settings: args.settings?.[plugin.id] ?? {},
-				}),
+				})
 			)
 			if (customApi) {
 				result.data.customApi = deepmerge(result.data.customApi, customApi)
@@ -175,8 +164,8 @@ export const resolvePlugins: ResolvePluginsFunction = async (args) => {
 		result.errors.push(
 			new PluginsDoNotProvideLoadOrSaveMessagesError(
 				"It seems you did not install any plugin that handles messages. Please add one to make inlang work. See https://inlang.com/documentation/plugins/registry.",
-				{ plugin: "plugin.inlang.missing" },
-			),
+				{ plugin: undefined }
+			)
 		)
 	}
 

package/src/resolve-modules/plugins/types.ts

@@ -52,10 +52,6 @@ export type ResolvedPluginApi = {
 		sourceLanguageTag: LanguageTag
 	}) => Promise<Message[]> | Message[]
 	saveMessages: (args: { messages: Message[] }) => Promise<void> | void
-	/**
-	 * Detect language tags in the project provided plugins.
-	 */
-	detectedLanguageTags: LanguageTag[]
 	/**
 	 * App specific APIs.
 	 *

package/src/resolve-modules/resolveModules.ts

@@ -38,7 +38,7 @@ export const resolveModules: ResolveModuleFunction = async (args) => {
 				new ModuleImportError(`Couldn't import the plugin "${module}"`, {
 					module: module,
 					cause: importedModule.error as Error,
-				}),
+				})
 			)
 			continue
 		}
@@ -48,7 +48,7 @@ export const resolveModules: ResolveModuleFunction = async (args) => {
 			moduleErrors.push(
 				new ModuleHasNoExportsError(`Module "${module}" has no exports.`, {
 					module: module,
-				}),
+				})
 			)
 			continue
 		}
@@ -57,12 +57,12 @@ export const resolveModules: ResolveModuleFunction = async (args) => {
 
 		if (isValidModule === false) {
 			const errors = [...ModuleCompiler.Errors(importedModule.data)].map(
-				(e) => `${e.path} ${e.message}`,
+				(e) => `${e.path} ${e.message}`
 			)
 			moduleErrors.push(
 				new ModuleExportIsInvalidError(`Module "${module}" is invalid: ` + errors.join("\n"), {
 					module: module,
-				}),
+				})
 			)
 			continue
 		}

package/src/test-utilities/createMessage.test.ts

@@ -5,7 +5,7 @@ test("should create a simple message", () => {
 	expect(
 		createMessage("welcome", {
 			de: "Hallo inlang",
-		}),
+		})
 	).toMatchInlineSnapshot(`
 		{
 		  "id": "welcome",
@@ -13,7 +13,7 @@ test("should create a simple message", () => {
 		  "variants": [
 		    {
 		      "languageTag": "de",
-		      "match": {},
+		      "match": [],
 		      "pattern": [
 		        {
 		          "type": "Text",
@@ -34,7 +34,7 @@ test("should create a message with pattern", () => {
 				{ type: "VariableReference", name: "name" },
 				{ type: "Text", value: '"' },
 			],
-		}),
+		})
 	).toMatchInlineSnapshot(`
 			{
 			  "id": "greeting",
@@ -42,7 +42,7 @@ test("should create a message with pattern", () => {
 			  "variants": [
 			    {
 			      "languageTag": "en",
-			      "match": {},
+			      "match": [],
 			      "pattern": [
 			        {
 			          "type": "Text",
@@ -68,7 +68,7 @@ test("should create a message with a pattern", () => {
 		createMessage("welcome", {
 			en: "hello inlang",
 			de: [{ type: "Text", value: "Hallo inlang" }],
-		}),
+		})
 	).toMatchInlineSnapshot(`
 			{
 			  "id": "welcome",
@@ -76,7 +76,7 @@ test("should create a message with a pattern", () => {
 			  "variants": [
 			    {
 			      "languageTag": "en",
-			      "match": {},
+			      "match": [],
 			      "pattern": [
 			        {
 			          "type": "Text",
@@ -86,7 +86,7 @@ test("should create a message with a pattern", () => {
 			    },
 			    {
 			      "languageTag": "de",
-			      "match": {},
+			      "match": [],
 			      "pattern": [
 			        {
 			          "type": "Text",

package/src/test-utilities/createMessage.ts

@@ -6,7 +6,7 @@ export const createMessage = (id: string, patterns: Record<string, Pattern | str
 		selectors: [],
 		variants: Object.entries(patterns).map(([languageTag, patterns]) => ({
 			languageTag,
-			match: {},
+			match: [],
 			pattern:
 				typeof patterns === "string"
 					? [