@inlang/sdk 0.7.0 → 0.9.0

package/dist/test-utilities/createMessage.test.js

@@ -10,7 +10,7 @@ test("should create a simple message", () => {
 		  "variants": [
 		    {
 		      "languageTag": "de",
-		      "match": {},
+		      "match": [],
 		      "pattern": [
 		        {
 		          "type": "Text",
@@ -36,7 +36,7 @@ test("should create a message with pattern", () => {
 			  "variants": [
 			    {
 			      "languageTag": "en",
-			      "match": {},
+			      "match": [],
 			      "pattern": [
 			        {
 			          "type": "Text",
@@ -67,7 +67,7 @@ test("should create a message with a pattern", () => {
 			  "variants": [
 			    {
 			      "languageTag": "en",
-			      "match": {},
+			      "match": [],
 			      "pattern": [
 			        {
 			          "type": "Text",
@@ -77,7 +77,7 @@ test("should create a message with a pattern", () => {
 			    },
 			    {
 			      "languageTag": "de",
-			      "match": {},
+			      "match": [],
 			      "pattern": [
 			        {
 			          "type": "Text",

package/package.json

@@ -1,7 +1,7 @@
 {
 	"name": "@inlang/sdk",
 	"type": "module",
-	"version": "0.7.0",
+	"version": "0.9.0",
 	"license": "Apache-2.0",
 	"publishConfig": {
 		"access": "public"

package/src/adapter/solidAdapter.test.ts

@@ -44,7 +44,7 @@ const exampleMessages: Message[] = [
 		variants: [
 			{
 				languageTag: "en",
-				match: {},
+				match: [],
 				pattern: [
 					{
 						type: "Text",
@@ -60,7 +60,7 @@ const exampleMessages: Message[] = [
 		variants: [
 			{
 				languageTag: "en",
-				match: {},
+				match: [],
 				pattern: [
 					{
 						type: "Text",
@@ -95,7 +95,7 @@ describe("config", () => {
 				nodeishFs: fs,
 				_import: $import,
 			}),
-			{ from },
+			{ from }
 		)
 
 		let counter = 0
@@ -125,7 +125,7 @@ describe("installed", () => {
 				nodeishFs: fs,
 				_import: $import,
 			}),
-			{ from },
+			{ from }
 		)
 		let counterPlugins = 0
 		let counterLint = 0
@@ -175,7 +175,7 @@ describe("messages", () => {
 			description: {
 				en: "wo",
 			},
-			loadMessages: ({ languageTags }) => (languageTags.length ? exampleMessages : []),
+			loadMessages: ({ settings }) => (settings.languageTags.length ? exampleMessages : []),
 			saveMessages: () => undefined,
 		}
 
@@ -188,7 +188,7 @@ describe("messages", () => {
 				nodeishFs: fs,
 				_import: mockImport,
 			}),
-			{ from },
+			{ from }
 		)
 
 		let counter = 0
@@ -217,7 +217,7 @@ describe("messages", () => {
 				nodeishFs: fs,
 				_import: $import,
 			}),
-			{ from },
+			{ from }
 		)
 
 		let counter = 0
@@ -232,7 +232,7 @@ describe("messages", () => {
 			(
 				Object.values(messagesBefore())[0]?.variants.find((variant) => variant.languageTag === "en")
 					?.pattern[0] as Text
-			).value,
+			).value
 		).toBe("test")
 
 		project.query.messages.update({
@@ -243,7 +243,7 @@ describe("messages", () => {
 				variants: [
 					{
 						languageTag: "en",
-						match: {},
+						match: [],
 						pattern: [
 							{
 								type: "Text",
@@ -262,7 +262,7 @@ describe("messages", () => {
 			(
 				Object.values(messagesAfter())[0]?.variants.find((variant) => variant.languageTag === "en")
 					?.pattern[0] as Text
-			).value,
+			).value
 		).toBe("test2")
 	})
 })
@@ -278,7 +278,7 @@ describe("lint", () => {
 					nodeishFs: fs,
 					_import: $import,
 				}),
-				{ from },
+				{ from }
 			)
 
 			let counter = 0
@@ -316,7 +316,7 @@ describe("lint", () => {
 					nodeishFs: fs,
 					_import: $import,
 				}),
-				{ from },
+				{ from }
 			)
 
 			let counter = 0
@@ -329,7 +329,7 @@ describe("lint", () => {
 				where: { id: "a" },
 				data: {
 					...exampleMessages[0],
-					variants: [{ languageTag: "en", match: {}, pattern: [{ type: "Text", value: "new" }] }],
+					variants: [{ languageTag: "en", match: [], pattern: [{ type: "Text", value: "new" }] }],
 				},
 			})
 
@@ -342,7 +342,7 @@ describe("lint", () => {
 				where: { id: "a" },
 				data: {
 					...exampleMessages[0],
-					variants: [{ languageTag: "en", match: {}, pattern: [{ type: "Text", value: "new" }] }],
+					variants: [{ languageTag: "en", match: [], pattern: [{ type: "Text", value: "new" }] }],
 				},
 			})
 

package/src/adapter/solidAdapter.ts

@@ -6,7 +6,7 @@ export const solidAdapter = (
 	project: InlangProject,
 	arg: {
 		from: typeof solidFrom
-	},
+	}
 ): InlangProjectWithSolidAdapter => {
 	const convert = <T>(signal: () => T): (() => T) => {
 		return arg.from(observable(signal)) as () => T

package/src/api.ts

@@ -64,7 +64,7 @@ export type MessageQueryApi = {
 	get: ((args: { where: { id: Message["id"] } }) => Readonly<Message>) & {
 		subscribe: (
 			args: { where: { id: Message["id"] } },
-			callback: (message: Message) => void,
+			callback: (message: Message) => void
 		) => void
 	}
 	includedMessageIds: Subscribable<Message["id"][]>
@@ -84,7 +84,7 @@ export type MessageLintReportsQueryApi = {
 	}) => Readonly<MessageLintReport[]>) & {
 		subscribe: (
 			args: { where: { messageId: MessageLintReport["messageId"] } },
-			callback: (MessageLintRules: Readonly<MessageLintReport[]>) => void,
+			callback: (MessageLintRules: Readonly<MessageLintReport[]>) => void
 		) => void
 	}
 }

package/src/createMessageLintReportsQuery.ts

@@ -14,7 +14,7 @@ export function createMessageLintReportsQuery(
 	messages: () => Array<Message> | undefined,
 	settings: () => ProjectSettings,
 	installedMessageLintRules: () => Array<InstalledMessageLintRule>,
-	resolvedModules: () => Awaited<ReturnType<typeof resolveModules>> | undefined,
+	resolvedModules: () => Awaited<ReturnType<typeof resolveModules>> | undefined
 ): InlangProject["query"]["messageLintReports"] {
 	// @ts-expect-error
 	const index = new ReactiveMap<MessageLintReport["messageId"], MessageLintReport[]>()
@@ -34,7 +34,7 @@ export function createMessageLintReportsQuery(
 					settings: {
 						..._settings,
 						messageLintRuleLevels: Object.fromEntries(
-							installedMessageLintRules().map((rule) => [rule.id, rule.level]),
+							installedMessageLintRules().map((rule) => [rule.id, rule.level])
 						),
 					},
 					messages: _messages,
@@ -58,13 +58,13 @@ export function createMessageLintReportsQuery(
 	return {
 		getAll: createSubscribable(() => {
 			return structuredClone(
-				[...index.values()].flat().length === 0 ? [] : [...index.values()].flat(),
+				[...index.values()].flat().length === 0 ? [] : [...index.values()].flat()
 			)
 		}),
 		get: Object.assign(get, {
 			subscribe: (
 				args: Parameters<MessageLintReportsQueryApi["get"]["subscribe"]>[0],
-				callback: Parameters<MessageLintReportsQueryApi["get"]["subscribe"]>[1],
+				callback: Parameters<MessageLintReportsQueryApi["get"]["subscribe"]>[1]
 			) => createSubscribable(() => get(args)).subscribe(callback),
 		}) as any,
 	}

package/src/createMessagesQuery.test.ts

@@ -54,10 +54,10 @@ describe("get", () => {
 		const message2 = query.get({ where: { id: "first-message" } })!
 
 		expect(
-			(message1.variants.find((v) => v.languageTag === "en")!.pattern![0]! as Text).value,
+			(message1.variants.find((v) => v.languageTag === "en")!.pattern![0]! as Text).value
 		).toBe("Hello World 2")
 		expect(
-			(message2.variants.find((v) => v.languageTag === "en")!.pattern![0]! as Text).value,
+			(message2.variants.find((v) => v.languageTag === "en")!.pattern![0]! as Text).value
 		).toBe("Hello World")
 	})
 })
@@ -94,7 +94,7 @@ describe("getAll", () => {
 			(
 				Object.values(query.getAll()!)[0]!.variants.find((v) => v.languageTag === "en")!
 					.pattern![0]! as Text
-			).value,
+			).value
 		).toBe("Hello World")
 	})
 })
@@ -194,13 +194,13 @@ describe("reactivity", () => {
 				expect(message).toBeDefined()
 				expect(
 					(message?.variants.find((variant) => variant.languageTag === "en")?.pattern[0] as Text)
-						.value,
+						.value
 				).toBe("before")
 
 				query.update({ where: { id: "1" }, data: createMessage("1", { en: "after" }) })
 				expect(
 					(message?.variants.find((variant) => variant.languageTag === "en")?.pattern[0] as Text)
-						.value,
+						.value
 				).toBe("after")
 			})
 		})
@@ -217,13 +217,13 @@ describe("reactivity", () => {
 				expect(message).toBeDefined()
 				expect(
 					(message?.variants.find((variant) => variant.languageTag === "en")?.pattern[0] as Text)
-						.value,
+						.value
 				).toBe("before")
 
 				query.upsert({ where: { id: "1" }, data: createMessage("1", { en: "after" }) })
 				expect(
 					(message?.variants.find((variant) => variant.languageTag === "en")?.pattern[0] as Text)
-						.value,
+						.value
 				).toBe("after")
 			})
 		})
@@ -254,14 +254,14 @@ describe("reactivity", () => {
 			expect(message).toBeDefined()
 			expect(
 				(message!.variants.find((variant) => variant.languageTag === "en")?.pattern[0] as Text)
-					.value,
+					.value
 			).toBe("before")
 
 			setMessages([createMessage("1", { en: "after" })])
 			expect(message).toBeDefined()
 			expect(
 				(message!.variants.find((variant) => variant.languageTag === "en")?.pattern[0] as Text)
-					.value,
+					.value
 			).toBe("after")
 		})
 	})
@@ -301,7 +301,7 @@ describe("reactivity", () => {
 					(
 						Object.values(messages!)![0]!.variants.find((variant) => variant.languageTag === "en")!
 							.pattern[0]! as Text
-					).value,
+					).value
 				).toBe("before")
 
 				query.update({ where: { id: "1" }, data: createMessage("1", { en: "after" }) })
@@ -310,7 +310,7 @@ describe("reactivity", () => {
 					(
 						Object.values(messages!)![0]!.variants.find((variant) => variant.languageTag === "en")!
 							.pattern[0]! as Text
-					).value,
+					).value
 				).toBe("after")
 			})
 		})
@@ -329,7 +329,7 @@ describe("reactivity", () => {
 					(
 						Object.values(messages!)![0]!.variants.find((variant) => variant.languageTag === "en")!
 							.pattern[0]! as Text
-					).value,
+					).value
 				).toBe("before")
 
 				query.upsert({ where: { id: "1" }, data: createMessage("1", { en: "after" }) })
@@ -337,7 +337,7 @@ describe("reactivity", () => {
 					(
 						Object.values(messages!)![0]!.variants.find((variant) => variant.languageTag === "en")!
 							.pattern[0]! as Text
-					).value,
+					).value
 				).toBe("after")
 			})
 		})
@@ -385,7 +385,7 @@ describe("reactivity", () => {
 				(
 					Object.values(messages!)![0]!.variants.find((variant) => variant.languageTag === "en")!
 						.pattern[0]! as Text
-				).value,
+				).value
 			).toBe("before")
 
 			setMessages([createMessage("1", { en: "after" })])
@@ -394,9 +394,22 @@ describe("reactivity", () => {
 				(
 					Object.values(messages!)![0]!.variants.find((variant) => variant.languageTag === "en")!
 						.pattern[0]! as Text
-				).value,
+				).value
 			).toBe("after")
 		})
+
+		it("should not mutate messages signal outside the query when using the query", async () => {
+			const [inputMessages] = createSignal<Message[]>([createMessage("1", { en: "before" })])
+			const query = createMessagesQuery(inputMessages)
+
+			let messages: Readonly<Message[]> | undefined = undefined
+			await createChangeListener(() => (messages = query.getAll()))
+			expect(Object.values(messages!)).toHaveLength(1)
+
+			query.create({ data: createMessage("2", { en: "" }) })
+
+			expect(inputMessages().length).toBe(1)
+		})
 	})
 })
 
@@ -421,11 +434,11 @@ it("instances should not share state", async () => {
 		query1.update({ where: { id: "1" }, data: createMessage("1", { en: "after" }) })
 		expect(
 			(message1!.variants.find((variant) => variant.languageTag === "en")!.pattern[0]! as Text)
-				.value,
+				.value
 		).toBe("after")
 		expect(
 			(message2!.variants.find((variant) => variant.languageTag === "en")!.pattern[0]! as Text)
-				.value,
+				.value
 		).toBe("before")
 
 		query1.delete({ where: { id: "1" } })

package/src/createMessagesQuery.ts

@@ -8,7 +8,7 @@ import type { InlangProject, MessageQueryApi } from "./api.js"
  * Creates a reactive query API for messages.
  */
 export function createMessagesQuery(
-	messages: () => Array<Message>,
+	messages: () => Array<Message>
 ): InlangProject["query"]["messages"] {
 	// @ts-expect-error
 	const index = new ReactiveMap<string, Message>()
@@ -31,7 +31,7 @@ export function createMessagesQuery(
 		get: Object.assign(get, {
 			subscribe: (
 				args: Parameters<MessageQueryApi["get"]["subscribe"]>[0],
-				callback: Parameters<MessageQueryApi["get"]["subscribe"]>[1],
+				callback: Parameters<MessageQueryApi["get"]["subscribe"]>[1]
 			) => createSubscribable(() => get(args)).subscribe(callback),
 		}) as any,
 		includedMessageIds: createSubscribable(() => {

package/src/lint/message/lintMessages.ts

@@ -13,7 +13,7 @@ export const lintMessages = async (args: {
 		lintSingleMessage({
 			...args,
 			message,
-		}),
+		})
 	)
 
 	const results = await Promise.all(promises)

package/src/lint/message/lintSingleMessage.test.ts

@@ -44,7 +44,7 @@ describe("lintSingleMessage", async () => {
 					messages,
 					message: message1,
 					rules: [lintRule1],
-				}),
+				})
 			)
 			expect(result.error).toBeDefined()
 			expect(result.data).toBeUndefined()

package/src/lint/message/lintSingleMessage.ts

@@ -42,8 +42,8 @@ export const lintSingleMessage = async (args: {
 			errors.push(
 				new MessagedLintRuleThrowedError(
 					`Lint rule '${rule.id}' throwed while linting message "${args.message.id}".`,
-					{ cause: error },
-				),
+					{ cause: error }
+				)
 			)
 		}
 	})

package/src/loadProject.test.ts

@@ -53,7 +53,7 @@ const exampleMessages: Message[] = [
 		variants: [
 			{
 				languageTag: "en",
-				match: {},
+				match: [],
 				pattern: [
 					{
 						type: "Text",
@@ -69,7 +69,7 @@ const exampleMessages: Message[] = [
 		variants: [
 			{
 				languageTag: "en",
-				match: {},
+				match: [],
 				pattern: [
 					{
 						type: "Text",
@@ -363,7 +363,7 @@ describe("functionality", () => {
 					sourceLanguageTag: "en",
 					languageTags: ["en"],
 					modules: ["plugin.js", "lintRule.js"],
-				} satisfies ProjectSettings),
+				} satisfies ProjectSettings)
 			)
 
 			const _import: ImportFunction = async (name) => {
@@ -411,7 +411,7 @@ describe("functionality", () => {
 					sourceLanguageTag: "en",
 					languageTags: ["en"],
 					modules: ["plugin.js", "lintRule.js"],
-				} satisfies ProjectSettings),
+				} satisfies ProjectSettings)
 			)
 			const _import: ImportFunction = async (name) => {
 				return {
@@ -428,7 +428,7 @@ describe("functionality", () => {
 			await new Promise((resolve) => setTimeout(resolve, 510))
 
 			expect(
-				project.query.messageLintReports.get({ where: { messageId: "some-message" } }),
+				project.query.messageLintReports.get({ where: { messageId: "some-message" } })
 			).toHaveLength(1)
 		})
 	})
@@ -482,17 +482,16 @@ describe("functionality", () => {
 		it("should call saveMessages() on updates", async () => {
 			const fs = createNodeishMemoryFs()
 
-			await fs.writeFile(
-				"./project.inlang.json",
-				JSON.stringify({
-					sourceLanguageTag: "en",
-					languageTags: ["en", "de"],
-					modules: ["plugin.js"],
-					"plugin.project.json": {
-						pathPattern: "./resources/{languageTag}.json",
-					},
-				}),
-			)
+			const settings: ProjectSettings = {
+				sourceLanguageTag: "en",
+				languageTags: ["en", "de"],
+				modules: ["plugin.js"],
+				"plugin.project.json": {
+					pathPattern: "./resources/{languageTag}.json",
+				},
+			}
+
+			await fs.writeFile("./project.inlang.json", JSON.stringify(settings))
 
 			await fs.mkdir("./resources")
 
@@ -526,7 +525,7 @@ describe("functionality", () => {
 					variants: [
 						{
 							languageTag: "en",
-							match: {},
+							match: [],
 							pattern: [
 								{
 									type: "Text",
@@ -536,7 +535,7 @@ describe("functionality", () => {
 						},
 						{
 							languageTag: "de",
-							match: {},
+							match: [],
 							pattern: [
 								{
 									type: "Text",
@@ -556,7 +555,7 @@ describe("functionality", () => {
 					variants: [
 						{
 							languageTag: "en",
-							match: {},
+							match: [],
 							pattern: [
 								{
 									type: "Text",
@@ -567,7 +566,7 @@ describe("functionality", () => {
 
 						{
 							languageTag: "de",
-							match: {},
+							match: [],
 							pattern: [
 								{
 									type: "Text",
@@ -583,9 +582,7 @@ describe("functionality", () => {
 
 			expect(mockSaveFn.mock.calls.length).toBe(1)
 
-			expect(mockSaveFn.mock.calls[0][0].settings).toStrictEqual({
-				pathPattern: "./resources/{languageTag}.json",
-			})
+			expect(mockSaveFn.mock.calls[0][0].settings).toStrictEqual(settings)
 
 			expect(Object.values(mockSaveFn.mock.calls[0][0].messages)).toStrictEqual([
 				{
@@ -594,7 +591,7 @@ describe("functionality", () => {
 					variants: [
 						{
 							languageTag: "en",
-							match: {},
+							match: [],
 							pattern: [
 								{
 									type: "Text",
@@ -604,7 +601,7 @@ describe("functionality", () => {
 						},
 						{
 							languageTag: "de",
-							match: {},
+							match: [],
 							pattern: [
 								{
 									type: "Text",
@@ -620,7 +617,7 @@ describe("functionality", () => {
 					variants: [
 						{
 							languageTag: "en",
-							match: {},
+							match: [],
 							pattern: [
 								{
 									type: "Text",
@@ -630,7 +627,7 @@ describe("functionality", () => {
 						},
 						{
 							languageTag: "de",
-							match: {},
+							match: [],
 							pattern: [
 								{
 									type: "Text",